clk: microchip: mpfs-ccc: Use devm_kasprintf() for allocating formatted strings
authorGeert Uytterhoeven <geert+renesas@glider.be>
Tue, 3 Jan 2023 16:45:30 +0000 (17:45 +0100)
committerClaudiu Beznea <claudiu.beznea@microchip.com>
Thu, 19 Jan 2023 09:45:35 +0000 (11:45 +0200)
commit86d884f5287f4369c198811aaa4931a3a11f36d2
tree9a6cc1aa3a5e4169061727a37039417c0186e623
parent1b929c02afd37871d5afb9d498426f83432e71c2
clk: microchip: mpfs-ccc: Use devm_kasprintf() for allocating formatted strings

In various places, string buffers of a fixed size are allocated, and
filled using snprintf() with the same fixed size, which is error-prone.

Replace this by calling devm_kasprintf() instead, which always uses the
appropriate size.

While at it, remove an unneeded intermediate variable, which allows us
to drop a cast as a bonus.

With the initial behavior it would have been possible to have a device tree
with a node address that would make "ccc<node_address>_pll<N>" exceed
18 characters. If that happened, the <N> would be cut off & both
pll 0 & 1 would be named identically. If that happens, pll1 would fail
to register. Thus, the fixes tag has been added to this commit.

Fixes: d39fb172760e ("clk: microchip: add PolarFire SoC fabric clock support")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Tested-by: Conor Dooley <conor.dooley@microchip.com>
[claudiu.beznea: added the rationale behind fixes tag]
Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Link: https://lore.kernel.org/r/f904fd28b2087d1463ea65f059924e3b1acc193c.1672764239.git.geert+renesas@glider.be
drivers/clk/microchip/clk-mpfs-ccc.c