[PATCH v3] clk: ti: Notify AVS driver upon setting clock rate

Kumar, Udit u-kumar1 at ti.com
Thu Sep 21 07:14:15 CEST 2023


Thanks Nishanth

On 9/20/2023 5:50 PM, Nishanth Menon wrote:
> On 17:32-20230920, Udit Kumar wrote:
>> AVS is enabled at R5 SPL stage, on few platforms like J721E
>> and J7200 clk-k3 is used instead if clk-sci driver.
>> [...]
>> I think it is better to put IS_ENABLED() as the first check.
>> See other usage:
>> $ git grep IS_ENABLED . |grep if|grep '&&'

  will address in next version

>>
>> [..]
> First look - it looks fine, but note: these are two different patches.
> the clk-sci.c is a fix for an existing implementation and clk-k3.c is a
> new feature addition. please don't mix the two.

Sure, Let me send two patches then,

One for new implementation clk-k3.c version v4.

and another for clk-sci.c with Fixes tag.



More information about the U-Boot mailing list