Re: [PATCH 2/3] media: microchip: Make sure clk_init_data is fully initialized

[Eugen Hristev <ehristev@kernel.org>]

On 8/21/26 13:04, Geert Uytterhoeven wrote:
> The clk_init_data structure contains several mutually-exclusive members
> for different methods to specify the possible parents of a clock,
> prompting drivers to initialize only the members they need.  However,
> not initializing all members may cause subtle issues, which are only
> exposed when CONFIG_INIT_STACK_ALL_PATTERN or CONFIG_INIT_STACK_NONE is
> enabled.
> 
> Make sure all members are fully initialized, to avoid such bugs, and to
> prevent future breakage when converting drivers to a different method
> for specifying the parents.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Compile-tested only.
> ---
>  drivers/media/platform/microchip/microchip-isc-clk.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

The only nitpick I have is that "media: microchip" holds multiple
drivers, while you are fixing this for the microchip ISC.

Regardless,
Reviewed-by: Eugen Hristev <ehristev@kernel.org>
> diff --git a/drivers/media/platform/microchip/microchip-isc-clk.c b/drivers/media/platform/microchip/microchip-isc-clk.c
> index 24358d804e75d75c..65d5fb8019b341b8 100644
> --- a/drivers/media/platform/microchip/microchip-isc-clk.c
> +++ b/drivers/media/platform/microchip/microchip-isc-clk.c
> @@ -231,7 +231,7 @@ static int isc_clk_register(struct isc_device *isc, unsigned int id)
>  	struct regmap *regmap = isc->regmap;
>  	struct device_node *np = isc->dev->of_node;
>  	struct isc_clk *isc_clk;
> -	struct clk_init_data init;
> +	struct clk_init_data init = {};
>  	const char *clk_name = np->name;
>  	const char *parent_names[3];
>  	int num_parents;


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic