Re: [PATCH] clk/axs10x: Clear init field in driver probe

From: Michael Turquette
Date: Mon Dec 12 2016 - 14:03:42 EST


Quoting Jose Abreu (2016-12-12 03:08:55)
> Init field must be cleared in driver probe as this structure is not dinamically allocated. If not, wrong flags can be passed to core.
>
> Signed-off-by: Jose Abreu <joabreu@xxxxxxxxxxxx>

Applied to clk-next. You can find this patch merged after v4.10-rc1 is
released in a couple of weeks.

Regards,
Mike

> Cc: Carlos Palminha <palminha@xxxxxxxxxxxx>
> Cc: Stephen Boyd <sboyd@xxxxxxxxxxxxxx>
> Cc: Michael Turquette <mturquette@xxxxxxxxxxxx>
> Cc: linux-clk@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> ---
> drivers/clk/axs10x/i2s_pll_clock.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/axs10x/i2s_pll_clock.c b/drivers/clk/axs10x/i2s_pll_clock.c
> index 411310d..02d3bcd 100644
> --- a/drivers/clk/axs10x/i2s_pll_clock.c
> +++ b/drivers/clk/axs10x/i2s_pll_clock.c
> @@ -182,6 +182,7 @@ static int i2s_pll_clk_probe(struct platform_device *pdev)
> if (IS_ERR(pll_clk->base))
> return PTR_ERR(pll_clk->base);
>
> + memset(&init, 0, sizeof(init));
> clk_name = node->name;
> init.name = clk_name;
> init.ops = &i2s_pll_ops;
> --
> 1.9.1
>
>