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: <019501dad2af$9e03a640$da0af2c0$@samsung.com>
Date: Wed, 10 Jul 2024 15:27:44 +0530
From: "Alim Akhtar" <alim.akhtar@...sung.com>
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



> -----Original Message-----
> From: sunyeal.hong <sunyeal.hong@...sung.com>
> Sent: Wednesday, July 10, 2024 12:29 PM
> 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.
Ok, you keep size as 0x8000

> > [snip]
> > > > > --
> > > > > 2.45.2
> > > >
> > > >
> > >
> > > Please review my answer again.
> > >
> > > Thanks,
> > > Sunyeal Hong
> > >
> >
> >
> 
> Thanks,
> Sunyeal Hong
>