Received-SPF: pass (google.com: domain of linux-kernel+bounces-209593-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Tue, 11 Jun 2024 15:16:33 +0530
From: Varadarajan Narayanan <quic_varada@quicinc.com>
To: Konrad Dybcio <konrad.dybcio@linaro.org>
CC: <andersson@kernel.org>, <mturquette@baylibre.com>, <sboyd@kernel.org>,
        <robh@kernel.org>, <krzk+dt@kernel.org>, <conor+dt@kernel.org>,
        <djakov@kernel.org>, <dmitry.baryshkov@linaro.org>,
        <quic_anusha@quicinc.com>, <linux-arm-msm@vger.kernel.org>,
        <linux-clk@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <linux-pm@vger.kernel.org>,
        <bryan.odonoghue@linaro.org>
Subject: Re: [PATCH v11 0/6] Add interconnect driver for IPQ9574 SoC
Message-ID: <Zmgc+Qzwt6Zbg/w+@hu-varada-blr.qualcomm.com>
References: <20240430064214.2030013-1-quic_varada@quicinc.com>
 <ZjXrTywO6+iRaEYk@hu-varada-blr.qualcomm.com>
 <90bb9256-d54d-4e01-aa06-4184e2b95d48@linaro.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <90bb9256-d54d-4e01-aa06-4184e2b95d48@linaro.org>

On Thu, Jun 06, 2024 at 04:07:23PM +0200, Konrad Dybcio wrote:
> On 4.05.2024 10:01 AM, Varadarajan Narayanan wrote:
> > Bjorn,
> >
> >> On Tue, Apr 30, 2024 at 12:12:08PM +0530, Varadarajan Narayanan wrote:
> >> MSM platforms manage NoC related clocks and scaling from RPM.
> >> However, in IPQ SoCs, RPM is not involved in managing NoC
> >> related clocks and there is no NoC scaling.
> >>
> >> However, there is a requirement to enable some NoC interface
> >> clocks for the accessing the peripherals present in the
> >> system. Hence add a minimalistic interconnect driver that
> >> establishes a path from the processor/memory to those peripherals
> >> and vice versa.
> >>
> >> Change icc-clk driver to take master and slave ids instead
> >> of auto generating.
> >>
> >> Currently, drivers/clk/qcom/clk-cbf-8996.c is the only user of
> >> icc-clk. And, it had exactly one master and one slave node.
> >> For this the auto generated master (= 1) and slave (= 0) was
> >> enough.
> >>
> >> However, when drivers/clk/qcom/gcc-ipq9574.c wanted to make use
> >> of the icc-clk framework, it had more number of master and slave
> >> nodes and the auto generated ids did not suit the usage.
> >>
> >> ---
> >> v11:	No code changes
> >> 	Commit log changed for the first patch
> >> 	Added Acked-By: to 3 patches
> >
> > Can this be included in your driver changes for 6.10?
>
Konrad,

> FWIW there is still an open discussion at v9
> <CAA8EJpqENsojPQmCbma_nQLEZq8nK1fz1K0JdtvLd=kPrH_DBw@mail.gmail.com>

Thanks for reminding. Have responded to it.
https://lore.kernel.org/linux-arm-msm/Zmgb+OjdBNw71sC1@hu-varada-blr.qualcomm.com/

-Varada