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>,
"'Artur Świgoń'" <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>
Subject: RE: [PATCH v2] interconnect: constify of_phandle_args in xlate
Date: Wed, 21 Feb 2024 15:58:38 +0530 [thread overview]
Message-ID: <1f7401da64b0$bdba8430$392f8c90$@samsung.com> (raw)
In-Reply-To: <6c4bdd9f-6b1f-43a0-9baa-7c1eedd46f76@linaro.org>
Hi Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Sent: Wednesday, February 21, 2024 12:44 PM
> To: Alim Akhtar <alim.akhtar@samsung.com>; 'Georgi Djakov'
> <djakov@kernel.org>; 'Bjorn Andersson' <andersson@kernel.org>; 'Konrad
> Dybcio' <konrad.dybcio@linaro.org>; 'Sylwester Nawrocki'
> <s.nawrocki@samsung.com>; 'Artur Świgoń' <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>
> Subject: Re: [PATCH v2] interconnect: constify of_phandle_args in xlate
>
> On 20/02/2024 13:18, Alim Akhtar wrote:
> >
> >
> >> -----Original Message-----
> >> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >> Sent: Tuesday, February 20, 2024 12:52 PM
> >> To: Georgi Djakov <djakov@kernel.org>; Bjorn Andersson
> >> <andersson@kernel.org>; Konrad Dybcio <konrad.dybcio@linaro.org>;
> >> Sylwester Nawrocki <s.nawrocki@samsung.com>; Artur Świgoń
> >> <a.swigon@samsung.com>; Krzysztof Kozlowski
> >> <krzysztof.kozlowski@linaro.org>; Alim Akhtar
> >> <alim.akhtar@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>
> >> Subject: [PATCH v2] interconnect: constify of_phandle_args in xlate
> >>
> >> The xlate callbacks are supposed to translate of_phandle_args to
> >> proper provider without modifying the of_phandle_args. Make the
> >> argument pointer to const for code safety and readability.
> >>
> >> Acked-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> >> Acked-by: Thierry Reding <treding@nvidia.com> # Tegra
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>
> >> ---
> >>
> >> Changes in v2:
> >> 1. Drop unrelated gpiolib changes (Konrad, Thierry).
> >> 2. Add tags.
> >> ---
> >> drivers/interconnect/core.c | 4 ++--
> >> drivers/interconnect/qcom/icc-common.c | 3 ++-
> >> drivers/interconnect/qcom/icc-common.h | 3 ++-
> >> drivers/interconnect/samsung/exynos.c | 2 +-
> >> drivers/memory/tegra/mc.c | 2 +-
> >> drivers/memory/tegra/tegra124-emc.c | 2 +-
> >> drivers/memory/tegra/tegra124.c | 2 +-
> >> drivers/memory/tegra/tegra186-emc.c | 2 +-
> >> drivers/memory/tegra/tegra20-emc.c | 2 +-
> >> drivers/memory/tegra/tegra20.c | 2 +-
> >> drivers/memory/tegra/tegra30-emc.c | 2 +-
> >> drivers/memory/tegra/tegra30.c | 2 +-
> >> include/linux/interconnect-provider.h | 11 ++++++-----
> >> include/soc/tegra/mc.h | 7 ++++---
> >> 14 files changed, 25 insertions(+), 21 deletions(-)
> >>
> >
> > For Samsung/Exynos
> > Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
>
> Review tags are per patch. You probably meant Acked-by.
if that is more appropriate in this case, feel free to add
Acked-by: Alim Akhtar <alim.akhtar@samsung.com>
> https://protect2.fireeye.com/v1/url?k=bc64ffdb-ddefeaed-bc657494-
> 74fe485cbff1-62590d464cd6269e&q=1&e=ec8047f9-eeac-446c-8ba5-
> 476f581d9cfa&u=https%3A%2F%2Felixir.bootlin.com%2Flinux%2Fv6.8-
> rc5%2Fsource%2FDocumentation%2Fprocess%2Fsubmitting-
> patches.rst%23L544
>
> What is there anyway to review in Samsung? Adding one const in one driver?
>
I my opinion, any changes coming in, has to be reviewed to see if there is/are any side effect because of the change,
even though the changes looks trivial.
>
> Best regards,
> Krzysztof
prev parent reply other threads:[~2024-02-21 10:28 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20240220072224epcas5p4ff60556b0b5632abfc44d2b8ad932b8f@epcas5p4.samsung.com>
2024-02-20 7:22 ` [PATCH v2] interconnect: constify of_phandle_args in xlate Krzysztof Kozlowski
2024-02-20 12:18 ` Alim Akhtar
2024-02-21 7:13 ` Krzysztof Kozlowski
2024-02-21 10:28 ` Alim Akhtar [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='1f7401da64b0$bdba8430$392f8c90$@samsung.com' \
--to=alim.akhtar@samsung.com \
--cc=a.swigon@samsung.com \
--cc=andersson@kernel.org \
--cc=djakov@kernel.org \
--cc=jonathanh@nvidia.com \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=s.nawrocki@samsung.com \
--cc=thierry.reding@gmail.com \
--cc=treding@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox