clk: move from strlcpy with unused retval to strscpy
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Thu, 18 Aug 2022 21:00:00 +0000 (23:00 +0200)
committerStephen Boyd <sboyd@kernel.org>
Mon, 22 Aug 2022 23:22:53 +0000 (16:22 -0700)
Follow the advice of the below link and prefer 'strscpy' in this
subsystem. Conversion is 1:1 because the return value is not used.
Generated by a coccinelle script.

Link: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@mail.gmail.com/
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Link: https://lore.kernel.org/r/20220818210000.6600-1-wsa+renesas@sang-engineering.com
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clkdev.c
drivers/clk/mvebu/dove-divider.c
drivers/clk/tegra/clk-bpmp.c

index 67f601a4102395e67ba4ef710379a5fb3518cc1b..a4d4bd3f5be555e95df8c70bcf7decb4d9bbfca4 100644 (file)
@@ -165,7 +165,7 @@ vclkdev_alloc(struct clk_hw *hw, const char *con_id, const char *dev_fmt,
 
        cla->cl.clk_hw = hw;
        if (con_id) {
-               strlcpy(cla->con_id, con_id, sizeof(cla->con_id));
+               strscpy(cla->con_id, con_id, sizeof(cla->con_id));
                cla->cl.con_id = cla->con_id;
        }
 
index 7e35c891e1684338460d0cba223ce656c4d52c25..0a90452ee808b6bf2f1f9f815624b99720c3a81f 100644 (file)
@@ -170,7 +170,7 @@ static struct clk *clk_register_dove_divider(struct device *dev,
                .num_parents = num_parents,
        };
 
-       strlcpy(name, dc->name, sizeof(name));
+       strscpy(name, dc->name, sizeof(name));
 
        dc->hw.init = &init;
        dc->base = base;
index 3748a39dae7cb442c5a29527f317a7605f83bdea..d82a71f10c2c182309653e4c1ea7230e79396af8 100644 (file)
@@ -349,7 +349,7 @@ static int tegra_bpmp_clk_get_info(struct tegra_bpmp *bpmp, unsigned int id,
        if (err < 0)
                return err;
 
-       strlcpy(info->name, response.name, MRQ_CLK_NAME_MAXLEN);
+       strscpy(info->name, response.name, MRQ_CLK_NAME_MAXLEN);
        info->num_parents = response.num_parents;
 
        for (i = 0; i < info->num_parents; i++)