Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Message-ID: <1356b1fc-fcdb-42af-a8df-0f7c2e2be9f3@kernel.org>
Date:   Mon, 27 Nov 2023 16:16:22 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH V3 0/2] interconnect: qcom: Introduce interconnect drivers
 for X1E80100
Content-Language: en-US
To:     Sibi Sankar <quic_sibis@quicinc.com>, andersson@kernel.org
Cc:     agross@kernel.org, conor+dt@kernel.org, quic_rjendra@quicinc.com,
        abel.vesa@linaro.org, linux-arm-msm@vger.kernel.org,
        linux-pm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, quic_tsoni@quicinc.com,
        neil.armstrong@linaro.org, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org
References: <20231123135028.29433-1-quic_sibis@quicinc.com>
From:   Georgi Djakov <djakov@kernel.org>
In-Reply-To: <20231123135028.29433-1-quic_sibis@quicinc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 23.11.23 15:50, Sibi Sankar wrote:
> This series adds interconnect support for the Qualcomm X1E80100 platform,
> aka Snapdragon X Elite.
> 
> Our v1 post of the patchsets adding support for Snapdragon X Elite SoC had
> the part number sc8380xp which is now updated to the new part number x1e80100
> based on the new branding scheme and refers to the exact same SoC.
> 
> V3:
> * Fix the index numbers of pcie_center_anoc nodes. [Georgi]

Thanks for updating the patches, Sibi! Now LGTM.

Hi Bjorn,

Here is a stable branch with the DT header in case you need it:
https://git.kernel.org/pub/scm/linux/kernel/git/djakov/icc.git/log/?h=icc-x1e80100

Thanks,
Georgi

> 
> v2:
> * Update the part number from sc8380xp to x1e80100.
> * Fixup required property ordering [Krzysztof]
> * Pickup Rbs.
> 
> Dependencies: None
> Release Link: https://www.qualcomm.com/news/releases/2023/10/qualcomm-unleashes-snapdragon-x-elite--the-ai-super-charged-plat
> 
> 
> Rajendra Nayak (2):
>    dt-bindings: interconnect: Add Qualcomm X1E80100 SoC
>    interconnect: qcom: Add X1E80100 interconnect provider driver
> 
>   .../interconnect/qcom,x1e80100-rpmh.yaml      |   83 +
>   drivers/interconnect/qcom/Kconfig             |    9 +
>   drivers/interconnect/qcom/Makefile            |    2 +
>   drivers/interconnect/qcom/x1e80100.c          | 2328 +++++++++++++++++
>   drivers/interconnect/qcom/x1e80100.h          |  192 ++
>   .../interconnect/qcom,x1e80100-rpmh.h         |  207 ++
>   6 files changed, 2821 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/interconnect/qcom,x1e80100-rpmh.yaml
>   create mode 100644 drivers/interconnect/qcom/x1e80100.c
>   create mode 100644 drivers/interconnect/qcom/x1e80100.h
>   create mode 100644 include/dt-bindings/interconnect/qcom,x1e80100-rpmh.h
>