On 09/10/2022 13:46, Marijn Suijten wrote: >> diff --git a/arch/arm64/boot/dts/qcom/sm6125.dtsi b/arch/arm64/boot/dts/qcom/sm6125.dtsi >> index 8c582a9e4ada..1fe3fa3ad877 100644 >> --- a/arch/arm64/boot/dts/qcom/sm6125.dtsi >> +++ b/arch/arm64/boot/dts/qcom/sm6125.dtsi >> @@ -387,19 +387,19 @@ tlmm: pinctrl@500000 { >> #interrupt-cells = <2>; >> >> sdc2_off_state: sdc2-off-state { >> - clk { >> + clk-pins { >> pins = "sdc2_clk"; >> drive-strength = <2>; >> bias-disable; >> }; >> >> - cmd { >> + cmd-pins { >> pins = "sdc2_cmd"; >> drive-strength = <2>; >> bias-pull-up; >> }; >> >> - data { >> + data-pins { >> pins = "sdc2_data"; >> drive-strength = <2>; >> bias-pull-up; >> @@ -413,13 +413,13 @@ clk { >> bias-disable; >> }; >> >> - cmd { >> + cmd-pins-pins { > > Is this double -pins-pins suffix intended? > No, thanks for noticing it. Best regards, Krzysztof