linux-kernel - RE: [PATCH v2 2/4] arm64: dts: exynos: add initial CMU clock nodes in Exynos Auto v920

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <008201dad296$a904ea90$fb0ebfb0$@samsung.com>
Date: Wed, 10 Jul 2024 15:59:08 +0900
From: "sunyeal.hong" <sunyeal.hong@...sung.com>
To: "'Alim Akhtar'" <alim.akhtar@...sung.com>, "'Krzysztof Kozlowski'"
	<krzk@...nel.org>, "'Sylwester Nawrocki'" <s.nawrocki@...sung.com>,
	"'Chanwoo Choi'" <cw00.choi@...sung.com>, "'Michael Turquette'"
	<mturquette@...libre.com>, "'Stephen Boyd'" <sboyd@...nel.org>, "'Rob
 Herring'" <robh@...nel.org>, "'Conor Dooley'" <conor+dt@...nel.org>
Cc: <linux-samsung-soc@...r.kernel.org>, <linux-clk@...r.kernel.org>,
	<devicetree@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>
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@...sung.com>
> Sent: Wednesday, July 10, 2024 11:31 AM
> To: 'sunyeal.hong' <sunyeal.hong@...sung.com>; 'Krzysztof Kozlowski'
> <krzk@...nel.org>; 'Sylwester Nawrocki' <s.nawrocki@...sung.com>; 'Chanwoo
> Choi' <cw00.choi@...sung.com>; 'Michael Turquette'
> <mturquette@...libre.com>; 'Stephen Boyd' <sboyd@...nel.org>; 'Rob
> Herring' <robh@...nel.org>; 'Conor Dooley' <conor+dt@...nel.org>
> Cc: linux-samsung-soc@...r.kernel.org; linux-clk@...r.kernel.org;
> devicetree@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; linux-
> kernel@...r.kernel.org
> 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@...sung.com>
> >
> [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@...sung.com>
> > > > Sent: Monday, July 8, 2024 4:43 AM
> > > > To: Krzysztof Kozlowski <krzk@...nel.org>; Sylwester Nawrocki
> > > > <s.nawrocki@...sung.com>; Chanwoo Choi
> > <cw00.choi@...sung.com>; Alim
> > > > Akhtar <alim.akhtar@...sung.com>; Michael Turquette
> [Snip]
> > > > +		cmu_peric0: clock-controller@...00000 {
> > > > +			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.
> [snip]
> > > > --
> > > > 2.45.2
> > >
> > >
> >
> > Please review my answer again.
> >
> > Thanks,
> > Sunyeal Hong
> >
> 
> 

Thanks,
Sunyeal Hong