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: <d4752e50-a1b6-2464-8bf9-6820e7f2d995@quicinc.com>
Date:   Mon, 4 Dec 2023 17:45:40 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.3.2
Subject: Re: [PATCH v5 0/3] Add UFS host controller and Phy nodes for sc7280
To:     Luca Weiss <luca.weiss@fairphone.com>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Manivannan Sadhasivam <mani@kernel.org>,
        "Alim Akhtar" <alim.akhtar@samsung.com>,
        Avri Altman <avri.altman@wdc.com>,
        "Bart Van Assche" <bvanassche@acm.org>,
        Rob Herring <robh+dt@kernel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        <cros-qcom-dts-watchers@chromium.org>
CC:     <~postmarketos/upstreaming@lists.sr.ht>,
        <phone-devel@vger.kernel.org>, <linux-arm-msm@vger.kernel.org>,
        <linux-scsi@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>,
        "Bao D. Nguyen" <quic_nguyenb@quicinc.com>
References: <20231204-sc7280-ufs-v5-0-926ceed550da@fairphone.com>
Content-Language: en-US
From:   Nitin Rawat <quic_nitirawa@quicinc.com>
In-Reply-To: <20231204-sc7280-ufs-v5-0-926ceed550da@fairphone.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 12/4/2023 3:54 PM, Luca Weiss wrote:
> This patch adds UFS host controller and Phy nodes for Qualcomm sc7280
> SoC and enable it on some sc7280-based boards.
> 
> Pick up the patchset from Nitin since the last revision (v4) has been
> sent end of September and is blocking qcm6490-fairphone-fp5 UFS.
> 
> ---
> Changes in v5:
> - Try to get patch tags in order
> - Drop patch reordering clocks/clock-names in dt-bindings example (Rob)
> - Use QCOM_ICC_TAG_ALWAYS for interconnect (Konrad)
> - Add missing interconnect-names (Luca)
> - Fix sorting of ufs nodes, place at correct location (Luca)
> - Provide ufs_mem_phy clock to gcc node (Luca)
> - Add missing power-domain to ufs_mem_phy (Luca)
> - Link to v4: https://lore.kernel.org/linux-arm-msm/20230929131936.29421-1-quic_nitirawa@quicinc.com/
> 
> ---
> Nitin Rawat (3):
>        scsi: ufs: qcom: dt-bindings: Add SC7280 compatible string
>        arm64: dts: qcom: sc7280: Add UFS nodes for sc7280 soc
>        arm64: dts: qcom: sc7280: Add UFS nodes for sc7280 IDP board
> 
>   .../devicetree/bindings/ufs/qcom,ufs.yaml          |  2 +
>   arch/arm64/boot/dts/qcom/sc7280-idp.dtsi           | 19 ++++++
>   arch/arm64/boot/dts/qcom/sc7280.dtsi               | 74 +++++++++++++++++++++-
>   3 files changed, 94 insertions(+), 1 deletion(-)
> ---
> base-commit: ce733604ab13d907655fd76ef5be55d16bbd0f8c
> change-id: 20231204-sc7280-ufs-b1e746ea60ed
> 
> Best regards,

Hi Luca,
Thanks for updating the patch. Posted a comment on 1 patch.

Regards,
Nitin