> -----Original Message-----
> From: sunyeal.hong <sunyeal.hong@xxxxxxxxxxx>
> Sent: Wednesday, July 10, 2024 12:29 PM
> To: 'Alim Akhtar' <alim.akhtar@xxxxxxxxxxx>; 'Krzysztof Kozlowski'
> <krzk@xxxxxxxxxx>; 'Sylwester Nawrocki' <s.nawrocki@xxxxxxxxxxx>;
> 'Chanwoo Choi' <cw00.choi@xxxxxxxxxxx>; 'Michael Turquette'
> <mturquette@xxxxxxxxxxxx>; 'Stephen Boyd' <sboyd@xxxxxxxxxx>; 'Rob
> Herring' <robh@xxxxxxxxxx>; 'Conor Dooley' <conor+dt@xxxxxxxxxx>
> Cc: linux-samsung-soc@xxxxxxxxxxxxxxx; linux-clk@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx
> Subject: RE: [PATCH v2 2/4] arm64: dts: exynos: add initial CMU clock nodes
> in Exynos Auto v920
>
> Hello Alim,
>
> > -----Original Message-----
> > From: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>
> > Sent: Wednesday, July 10, 2024 11:31 AM
> > To: 'sunyeal.hong' <sunyeal.hong@xxxxxxxxxxx>; 'Krzysztof Kozlowski'
> > <krzk@xxxxxxxxxx>; 'Sylwester Nawrocki' <s.nawrocki@xxxxxxxxxxx>;
> > 'Chanwoo Choi' <cw00.choi@xxxxxxxxxxx>; 'Michael Turquette'
> > <mturquette@xxxxxxxxxxxx>; 'Stephen Boyd' <sboyd@xxxxxxxxxx>; 'Rob
> > Herring' <robh@xxxxxxxxxx>; 'Conor Dooley' <conor+dt@xxxxxxxxxx>
> > Cc: linux-samsung-soc@xxxxxxxxxxxxxxx; linux-clk@xxxxxxxxxxxxxxx;
> > devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> > linux- kernel@xxxxxxxxxxxxxxx
> > Subject: RE: [PATCH v2 2/4] arm64: dts: exynos: add initial CMU clock
> > nodes in Exynos Auto v920
> >
> > Hello Sunyeal,
> >
> > > -----Original Message-----
> > > From: sunyeal.hong <sunyeal.hong@xxxxxxxxxxx>
> > >
> > [snip]
> > > > Subject: RE: [PATCH v2 2/4] arm64: dts: exynos: add initial CMU
> > > > clock nodes in Exynos Auto v920
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Sunyeal Hong <sunyeal.hong@xxxxxxxxxxx>
> > > > > Sent: Monday, July 8, 2024 4:43 AM
> > > > > To: Krzysztof Kozlowski <krzk@xxxxxxxxxx>; Sylwester Nawrocki
> > > > > <s.nawrocki@xxxxxxxxxxx>; Chanwoo Choi
> > > <cw00.choi@xxxxxxxxxxx>; Alim
> > > > > Akhtar <alim.akhtar@xxxxxxxxxxx>; Michael Turquette
> > [Snip]
> > > > > + cmu_peric0: clock-controller@10800000 {
> > > > > + compatible = "samsung,exynosautov920-
> cmu-
> > > > > peric0";
> > > > > + reg = <0x10800000 0x8000>;
> > > > Please cross check the size of the register range, this looks to
> > > > be more then what is needed.
> > > >
> > > In the case of preic0, the size is up to 0x7088. The CMU block SFR
> > > area of ExynosAuto v920 is generally specified up to 0x8000. There
> > > are differences for each block, but the settings are the same.
> > > Do you think it is necessary to change the actual size of each block?
> > To avoid any overlap between difference SFR region, better to mention
> > the exact size of the SFR region.
> >
> In ExynosAuto, the SFR of each block has an offset of 0x10000. So I don't
> think SFR will be overlapped.
> In the case of CMU SFR, there is no size over 0x8000, so I think it would be
> better to maintain that value.
> Please give me your opinion again.
Ok, you keep size as 0x8000
> > [snip]
> > > > > --
> > > > > 2.45.2
> > > >
> > > >
> > >
> > > Please review my answer again.
> > >
> > > Thanks,
> > > Sunyeal Hong
> > >
> >
> >
>
> Thanks,
> Sunyeal Hong
>
