arch/arm64/boot/dts/qcom/milos.dtsi | 54 ++++++++++++++++++------------------- 1 file changed, 27 insertions(+), 27 deletions(-)
As documented in the "Devicetree Sources (DTS) Coding Style" document,
pinctrl subnodes should be sorted by the pins property. Do this once for
milos.dtsi so that future additions can be added at the right places.
Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
arch/arm64/boot/dts/qcom/milos.dtsi | 54 ++++++++++++++++++-------------------
1 file changed, 27 insertions(+), 27 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/milos.dtsi b/arch/arm64/boot/dts/qcom/milos.dtsi
index e1a51d43943f..bd9a11ea703d 100644
--- a/arch/arm64/boot/dts/qcom/milos.dtsi
+++ b/arch/arm64/boot/dts/qcom/milos.dtsi
@@ -1667,6 +1667,21 @@ tlmm: pinctrl@f100000 {
wakeup-parent = <&pdc>;
+ qup_spi0_data_clk: qup-spi0-data-clk-state {
+ /* MISO, MOSI, CLK */
+ pins = "gpio0", "gpio1", "gpio2";
+ function = "qup0_se0";
+ drive-strength = <6>;
+ bias-disable;
+ };
+
+ qup_spi0_cs: qup-spi0-cs-state {
+ pins = "gpio3";
+ function = "qup0_se0";
+ drive-strength = <6>;
+ bias-disable;
+ };
+
qup_i2c1_data_clk: qup-i2c1-data-clk-state {
/* SDA, SCL */
pins = "gpio4", "gpio5";
@@ -1683,29 +1698,6 @@ qup_i2c3_data_clk: qup-i2c3-data-clk-state {
bias-pull-up = <2200>;
};
- qup_i2c7_data_clk: qup-i2c7-data-clk-state {
- /* SDA, SCL */
- pins = "gpio32", "gpio33";
- function = "qup1_se0";
- drive-strength = <2>;
- bias-pull-up;
- };
-
- qup_spi0_cs: qup-spi0-cs-state {
- pins = "gpio3";
- function = "qup0_se0";
- drive-strength = <6>;
- bias-disable;
- };
-
- qup_spi0_data_clk: qup-spi0-data-clk-state {
- /* MISO, MOSI, CLK */
- pins = "gpio0", "gpio1", "gpio2";
- function = "qup0_se0";
- drive-strength = <6>;
- bias-disable;
- };
-
qup_uart5_default: qup-uart5-default-state {
/* TX, RX */
pins = "gpio25", "gpio26";
@@ -1714,10 +1706,10 @@ qup_uart5_default: qup-uart5-default-state {
bias-disable;
};
- qup_uart11_default: qup-uart11-default-state {
- /* TX, RX */
- pins = "gpio50", "gpio51";
- function = "qup1_se4";
+ qup_i2c7_data_clk: qup-i2c7-data-clk-state {
+ /* SDA, SCL */
+ pins = "gpio32", "gpio33";
+ function = "qup1_se0";
drive-strength = <2>;
bias-pull-up;
};
@@ -1730,6 +1722,14 @@ qup_uart11_cts_rts: qup-uart11-cts-rts-state {
bias-pull-down;
};
+ qup_uart11_default: qup-uart11-default-state {
+ /* TX, RX */
+ pins = "gpio50", "gpio51";
+ function = "qup1_se4";
+ drive-strength = <2>;
+ bias-pull-up;
+ };
+
sdc2_default: sdc2-default-state {
clk-pins {
pins = "gpio62";
---
base-commit: 3daf23347bb5f4a375d0101ed29c97ce1a99721b
change-id: 20260213-milos-pinctrl-sort-16ed0fdb5aed
Best regards,
--
Luca Weiss <luca.weiss@fairphone.com>
On Fri, 13 Feb 2026 15:03:19 +0100, Luca Weiss wrote:
> As documented in the "Devicetree Sources (DTS) Coding Style" document,
> pinctrl subnodes should be sorted by the pins property. Do this once for
> milos.dtsi so that future additions can be added at the right places.
>
>
Applied, thanks!
[1/1] arm64: dts: qcom: milos: Sort pinctrl subnodes by pins
commit: 41626e8fef2e58bdfee4e3cb5b13c27f5b2b3abf
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
On 2/13/26 3:03 PM, Luca Weiss wrote: > As documented in the "Devicetree Sources (DTS) Coding Style" document, > pinctrl subnodes should be sorted by the pins property. Do this once for > milos.dtsi so that future additions can be added at the right places. > > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Konrad
© 2016 - 2026 Red Hat, Inc.