On 9/27/2023 2:55 PM, Konrad Dybcio wrote:
On 27.09.2023 10:18, Nitin Rawat wrote:Add UFS host controller and PHY nodes for sc7280 IDP board. Signed-off-by: Nitin Rawat <quic_nitirawa@xxxxxxxxxxx> --- arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi index 2ff549f4dc7a..a0059527d9e4 100644 --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi @@ -499,6 +499,25 @@ status = "okay"; }; +&ufs_mem_hc { + reset-gpios = <&tlmm 175 GPIO_ACTIVE_LOW>; + vcc-supply = <&vreg_l7b_2p9>; + vcc-max-microamp = <800000>; + vccq-supply = <&vreg_l9b_1p2>; + vccq-max-microamp = <900000>; + vccq2-supply = <&vreg_l9b_1p2>; + vccq2-max-microamp = <900000>;Were you able to confirm it's correct (see the q in [1])
Sorry I missed to reply to your query in my last reply . Yes Kodiak support both UFS2.x and UFS3.x and UFS2.x needs vccq2 . Hence we need that node.
Konrad [1] https://lore.kernel.org/linux-arm-msm/20230926162042.14180-1-quic_nitirawa@xxxxxxxxxxx/T/#m72ca82a9145af380ffd37415455d6ef3d4195795
