[PATCH 2/3] clk: xilinx: vcu: don't set pll_ref as parent of VCU(enc/dec) clocks

[Visavalia, Rohit]

Hi Stephen,

Thanks for the review.

>-----Original Message-----
>From: Stephen Boyd <sboyd at kernel.org>
>Sent: Tuesday, December 31, 2024 6:06 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 2/3] clk: xilinx: vcu: don't set pll_ref as parent of
>VCU(enc/dec) clocks
>
>Quoting Rohit Visavalia (2024-12-26 04:20:22)
>> CCF will try to adjust parent clock to set desire clock frequency of
>> child clock. So if pll_ref is not a fixed-clock then while setting
>> rate of enc/dec clocks pll_ref may get change, which may make VCU
>malfunction.
>>
>> Signed-off-by: Rohit Visavalia <rohit.visavalia at amd.com>
>> ---
>>  drivers/clk/xilinx/xlnx_vcu.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/clk/xilinx/xlnx_vcu.c
>> b/drivers/clk/xilinx/xlnx_vcu.c index f294a2398cb4..c3a4df7e325a
>> 100644
>> --- a/drivers/clk/xilinx/xlnx_vcu.c
>> +++ b/drivers/clk/xilinx/xlnx_vcu.c
>> @@ -550,7 +550,7 @@ static int xvcu_register_clock_provider(struct
>xvcu_device *xvcu)
>>                 return PTR_ERR(hw);
>>         xvcu->pll_post = hw;
>>
>> -       parent_data[0].fw_name = "pll_ref";
>> +       parent_data[0].fw_name = "dummy_name";
>
>"dummy_name" isn't part of the binding. It sounds like you want to not set
>CLK_SET_RATE_PARENT flag sometimes?
Yes, if parent rate(pll_ref) has been changed then VCU is not working correctly. So, making name as dummy.

>
>>         parent_data[1].hw = xvcu->pll_post;
>>
>>         hws[CLK_XVCU_ENC_CORE] =

Thanks
Rohit