[PATCH 3/3] clk: xilinx: vcu: unregister pll_post only if registered correctly

[Visavalia, Rohit]

Hi Stephen,

Thanks for the review.

>-----Original Message-----
>From: Stephen Boyd <sboyd at kernel.org>
>Sent: Tuesday, December 31, 2024 6:07 AM
>To: Visavalia, Rohit <rohit.visavalia at amd.com>; Simek, Michal
><michal.simek at amd.com>; mturquette at baylibre.com; Sagar, Vishal
><vishal.sagar at amd.com>
>Cc: javier.carrasco.cruz at gmail.com; geert+renesas at glider.be; u.kleine-
>koenig at baylibre.com; linux-clk at vger.kernel.org; linux-arm-
>kernel at lists.infradead.org; linux-kernel at vger.kernel.org; Visavalia, Rohit
><rohit.visavalia at amd.com>
>Subject: Re: [PATCH 3/3] clk: xilinx: vcu: unregister pll_post only if registered
>correctly
>
>Quoting Rohit Visavalia (2024-12-26 04:20:23)
>> If registration of pll_post is failed, it will be set to NULL or ERR,
>> unregistering same will fail with following call trace:
>>
>> Unable to handle kernel NULL pointer dereference at virtual address
>> 008 pc : clk_hw_unregister+0xc/0x20 lr :
>> clk_hw_unregister_fixed_factor+0x18/0x30
>> sp : ffff800011923850
>> ...
>> Call trace:
>>  clk_hw_unregister+0xc/0x20
>>  clk_hw_unregister_fixed_factor+0x18/0x30
>>  xvcu_unregister_clock_provider+0xcc/0xf4 [xlnx_vcu]
>> xvcu_probe+0x2bc/0x53c [xlnx_vcu]
>>
>> Signed-off-by: Rohit Visavalia <rohit.visavalia at amd.com>
>
>Please add a Fixes tag, and put fixes earlier in the series so they can be applied
>while other patches go through review.
I will take care in next patch series.

Thanks
Rohit