Received-SPF: pass (google.com: domain of linux-kernel+bounces-74526-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: "Alim Akhtar" <alim.akhtar@samsung.com>
To: "'Krzysztof Kozlowski'" <krzysztof.kozlowski@linaro.org>, "'Georgi
 Djakov'" <djakov@kernel.org>, "'Bjorn Andersson'" <andersson@kernel.org>,
	"'Konrad	Dybcio'" <konrad.dybcio@linaro.org>, "'Sylwester Nawrocki'"
	<s.nawrocki@samsung.com>, =?utf-8?Q?'Artur_=C5=9Awigo=C5=84'?=
	<a.swigon@samsung.com>, "'Thierry Reding'" <thierry.reding@gmail.com>,
	"'Jonathan Hunter'" <jonathanh@nvidia.com>, <linux-pm@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <linux-arm-msm@vger.kernel.org>,
	<linux-samsung-soc@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>,
	<linux-tegra@vger.kernel.org>
Cc: "'Thierry Reding'" <treding@nvidia.com>
In-Reply-To: <6c4bdd9f-6b1f-43a0-9baa-7c1eedd46f76@linaro.org>
Subject: RE: [PATCH v2] interconnect: constify of_phandle_args in xlate
Date: Wed, 21 Feb 2024 15:58:38 +0530
Message-ID: <1f7401da64b0$bdba8430$392f8c90$@samsung.com>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQLQP0hpybpPMp0Wp7r3CvZ4KlHU5QIjg6hJAdtsHmwBHGOYhq7/5urQ
Content-Language: en-us
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 105P
DLP-Filter: Pass
References: <CGME20240220072224epcas5p4ff60556b0b5632abfc44d2b8ad932b8f@epcas5p4.samsung.com>
	<20240220072213.35779-1-krzysztof.kozlowski@linaro.org>
	<1d9b01da63f6$daeb0170$90c10450$@samsung.com>
	<6c4bdd9f-6b1f-43a0-9baa-7c1eedd46f76@linaro.org>

Hi Krzysztof,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski=40linaro.org>
> Sent: Wednesday, February 21, 2024 12:44 PM
> To: Alim Akhtar <alim.akhtar=40samsung.com>; 'Georgi Djakov'
> <djakov=40kernel.org>; 'Bjorn Andersson' <andersson=40kernel.org>; 'Konra=
d
> Dybcio' <konrad.dybcio=40linaro.org>; 'Sylwester Nawrocki'
> <s.nawrocki=40samsung.com>; 'Artur =C5=9Awigo=C5=84'=20<a.swigon=40samsun=
g.com>;=0D=0A>=20'Thierry=20Reding'=20<thierry.reding=40gmail.com>;=20'Jona=
than=20Hunter'=0D=0A>=20<jonathanh=40nvidia.com>;=20linux-pm=40vger.kernel.=
org;=20linux-=0D=0A>=20kernel=40vger.kernel.org;=20linux-arm-msm=40vger.ker=
nel.org;=20linux-samsung-=0D=0A>=20soc=40vger.kernel.org;=20linux-arm-kerne=
l=40lists.infradead.org;=20linux-=0D=0A>=20tegra=40vger.kernel.org=0D=0A>=
=20Cc:=20'Thierry=20Reding'=20<treding=40nvidia.com>=0D=0A>=20Subject:=20Re=
:=20=5BPATCH=20v2=5D=20interconnect:=20constify=20of_phandle_args=20in=20xl=
ate=0D=0A>=20=0D=0A>=20On=2020/02/2024=2013:18,=20Alim=20Akhtar=20wrote:=0D=
=0A>=20>=0D=0A>=20>=0D=0A>=20>>=20-----Original=20Message-----=0D=0A>=20>>=
=20From:=20Krzysztof=20Kozlowski=20<krzysztof.kozlowski=40linaro.org>=0D=0A=
>=20>>=20Sent:=20Tuesday,=20February=2020,=202024=2012:52=20PM=0D=0A>=20>>=
=20To:=20Georgi=20Djakov=20<djakov=40kernel.org>;=20Bjorn=20Andersson=0D=0A=
>=20>>=20<andersson=40kernel.org>;=20Konrad=20Dybcio=20<konrad.dybcio=40lin=
aro.org>;=0D=0A>=20>>=20Sylwester=20Nawrocki=20<s.nawrocki=40samsung.com>;=
=20Artur=20=C5=9Awigo=C5=84=0D=0A>=20>>=20<a.swigon=40samsung.com>;=20Krzys=
ztof=20Kozlowski=0D=0A>=20>>=20<krzysztof.kozlowski=40linaro.org>;=20Alim=
=20Akhtar=0D=0A>=20>>=20<alim.akhtar=40samsung.com>;=20Thierry=20Reding=0D=
=0A>=20<thierry.reding=40gmail.com>;=0D=0A>=20>>=20Jonathan=20Hunter=20<jon=
athanh=40nvidia.com>;=20linux-pm=40vger.kernel.org;=0D=0A>=20>>=20linux-ker=
nel=40vger.kernel.org;=20linux-arm-msm=40vger.kernel.org;=20linux-=0D=0A>=
=20>>=20samsung-soc=40vger.kernel.org;=20linux-arm-kernel=40lists.infradead=
org;=0D=0A>=20>>=20linux-=20tegra=40vger.kernel.org=0D=0A>=20>>=20Cc:=20Th=
ierry=20Reding=20<treding=40nvidia.com>=0D=0A>=20>>=20Subject:=20=5BPATCH=
=20v2=5D=20interconnect:=20constify=20of_phandle_args=20in=20xlate=0D=0A>=
=20>>=0D=0A>=20>>=20The=20xlate=20callbacks=20are=20supposed=20to=20transla=
te=20of_phandle_args=20to=0D=0A>=20>>=20proper=20provider=20without=20modif=
ying=20the=20of_phandle_args.=20=20Make=20the=0D=0A>=20>>=20argument=20poin=
ter=20to=20const=20for=20code=20safety=20and=20readability.=0D=0A>=20>>=0D=
=0A>=20>>=20Acked-by:=20Konrad=20Dybcio=20<konrad.dybcio=40linaro.org>=0D=
=0A>=20>>=20Acked-by:=20Thierry=20Reding=20<treding=40nvidia.com>=20=23=20T=
egra=0D=0A>=20>>=20Signed-off-by:=20Krzysztof=20Kozlowski=20<krzysztof.kozl=
owski=40linaro.org>=0D=0A>=20>>=0D=0A>=20>>=20---=0D=0A>=20>>=0D=0A>=20>>=
=20Changes=20in=20v2:=0D=0A>=20>>=201.=20Drop=20unrelated=20gpiolib=20chang=
es=20(Konrad,=20Thierry).=0D=0A>=20>>=202.=20Add=20tags.=0D=0A>=20>>=20---=
=0D=0A>=20>>=20=20drivers/interconnect/core.c=20=20=20=20=20=20=20=20=20=20=
=20=20=7C=20=204=20++--=0D=0A>=20>>=20=20drivers/interconnect/qcom/icc-comm=
on.c=20=7C=20=203=20++-=0D=0A>=20>>=20drivers/interconnect/qcom/icc-common.=
h=20=7C=20=203=20++-=0D=0A>=20>>=20drivers/interconnect/samsung/exynos.c=20=
=20=7C=20=202=20+-=0D=0A>=20>>=20=20drivers/memory/tegra/mc.c=20=20=20=20=
=20=20=20=20=20=20=20=20=20=20=7C=20=202=20+-=0D=0A>=20>>=20=20drivers/memo=
ry/tegra/tegra124-emc.c=20=20=20=20=7C=20=202=20+-=0D=0A>=20>>=20=20drivers=
/memory/tegra/tegra124.c=20=20=20=20=20=20=20=20=7C=20=202=20+-=0D=0A>=20>>=
=20=20drivers/memory/tegra/tegra186-emc.c=20=20=20=20=7C=20=202=20+-=0D=0A>=
=20>>=20=20drivers/memory/tegra/tegra20-emc.c=20=20=20=20=20=7C=20=202=20+-=
=0D=0A>=20>>=20=20drivers/memory/tegra/tegra20.c=20=20=20=20=20=20=20=20=20=
=7C=20=202=20+-=0D=0A>=20>>=20=20drivers/memory/tegra/tegra30-emc.c=20=20=
=20=20=20=7C=20=202=20+-=0D=0A>=20>>=20=20drivers/memory/tegra/tegra30.c=20=
=20=20=20=20=20=20=20=20=7C=20=202=20+-=0D=0A>=20>>=20=20include/linux/inte=
rconnect-provider.h=20=20=7C=2011=20++++++-----=0D=0A>=20>>=20=20include/so=
c/tegra/mc.h=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=7C=20=207=
=20++++---=0D=0A>=20>>=20=2014=20files=20changed,=2025=20insertions(+),=202=
1=20deletions(-)=0D=0A>=20>>=0D=0A>=20>=0D=0A>=20>=20For=20Samsung/Exynos=
=0D=0A>=20>=20Reviewed-by:=20Alim=20Akhtar=20<alim.akhtar=40samsung.com>=0D=
=0A>=20=0D=0A>=20Review=20tags=20are=20per=20patch.=20You=20probably=20mean=
t=20Acked-by.=0D=0Aif=20that=20is=20more=20appropriate=20in=20this=20case,=
=20feel=20free=20to=20add=0D=0AAcked-by:=20Alim=20Akhtar=20<alim.akhtar=40s=
amsung.com>=0D=0A=0D=0A>=20https://protect2.fireeye.com/v1/url?k=3Dbc64ffdb=
-ddefeaed-bc657494-=0D=0A>=2074fe485cbff1-62590d464cd6269e&q=3D1&e=3Dec8047=
f9-eeac-446c-8ba5-=0D=0A>=20476f581d9cfa&u=3Dhttps%3A%2F%2Felixir.bootlin.c=
om%2Flinux%2Fv6.8-=0D=0A>=20rc5%2Fsource%2FDocumentation%2Fprocess%2Fsubmit=
ting-=0D=0A>=20patches.rst%23L544=0D=0A>=20=0D=0A>=20What=20is=20there=20an=
yway=20to=20review=20in=20Samsung?=20Adding=20one=20const=20in=20one=20driv=
er?=0D=0A>=20=0D=0AI=20my=20opinion,=20any=20changes=20coming=20in,=20has=
=20to=20be=20reviewed=20to=20see=20if=20there=20is/are=20any=20side=20effec=
t=20because=20of=20the=20change,=0D=0Aeven=20though=20the=20changes=20looks=
=20trivial.=0D=0A=0D=0A>=20=0D=0A>=20Best=20regards,=0D=0A>=20Krzysztof=0D=
=0A=0D=0A=0D=0A