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: <015501dad271$30f54a10$92dfde30$@samsung.com>
Date: Wed, 10 Jul 2024 08:00:52 +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

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.

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