Hi Neha, On 19:03-20230926, Neha Malcom Francis wrote: > Hi Nishanth > [..] > > > diff --git a/arch/arm/dts/k3-j721e-r5-sk.dts > > > b/arch/arm/dts/k3-j721e-r5-sk.dts > > > index 1cc64d07f7..0274465fa4 100644 > > > --- a/arch/arm/dts/k3-j721e-r5-sk.dts > > > +++ b/arch/arm/dts/k3-j721e-r5-sk.dts > > > > Unrelated to this patch: I noticed also that the DDR configuration for > > J721e-sk is stale (0.6 rev of DDR tool Vs 0.10 for EVM) > > > > > > For SK's R5 dts and u-boot.dtsi: > > The above pattern of comments from EVM repeat on SK as well, I will skip > > repeating the comments. > > > > [...] > > > > Thanks for reviewing, all are valid points. Regarding why clock-frequency is > mentioned for a bunch of peripherals, I will be looking into it and > accordingly posting v4.
Not sure but could be that the clk-data.c is not populated with those specified clocks with frequencies due to which we have to provide them here. I could see that autogen does allow us to set some fixed clock frequency so maybe that might be it. Regards, Manorit > > Thanking You > Neha Malcom Francis