[v1] arm64: dts: qcom: arduino-imola: fix faulty spidev node

[PATCH] arm64: dts: qcom: arduino-imola: fix faulty spidev node

Posted by Riccardo Mereu 1 month, 3 weeks ago

From: Riccardo Mereu <r.mereu@arduino.cc>

CS pin added on pinctrl0 property is causing spidev to return -ENODEV
since that GPIO is already part of spi5 pinmuxing.

Fixes: 3f745bc0f11f ("arm64: dts: qcom: qrb2210: add dts for Arduino unoq")
Signed-off-by: Riccardo Mereu <r.mereu@arduino.cc>
---
 arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts b/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
index 197ab6eb1666..d2c3359dd814 100644
--- a/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
+++ b/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
@@ -328,18 +328,11 @@ &spi5 {
 	spidev@0 {
 		reg = <0>;
 		compatible = "arduino,unoq-mcu";
-		pinctrl-0 = <&spidev_cs>;
 		pinctrl-names = "default";
 	};
 };
 
 &tlmm {
-	spidev_cs: spidev-cs-state {
-		pins = "gpio17";
-		function = "gpio";
-		drive-strength = <16>;
-	};
-
 	jmisc_gpio18: jmisc-gpio18-state {
 		pins = "gpio18";
 		function = "gpio";
-- 
2.53.0

Re: [PATCH] arm64: dts: qcom: arduino-imola: fix faulty spidev node

Posted by Konrad Dybcio 1 month, 2 weeks ago

On 2/10/26 11:39 AM, Riccardo Mereu wrote:
> From: Riccardo Mereu <r.mereu@arduino.cc>
> 
> CS pin added on pinctrl0 property is causing spidev to return -ENODEV
> since that GPIO is already part of spi5 pinmuxing.
> 
> Fixes: 3f745bc0f11f ("arm64: dts: qcom: qrb2210: add dts for Arduino unoq")
> Signed-off-by: Riccardo Mereu <r.mereu@arduino.cc>
> ---
>  arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts b/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
> index 197ab6eb1666..d2c3359dd814 100644
> --- a/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
> +++ b/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
> @@ -328,18 +328,11 @@ &spi5 {
>  	spidev@0 {
>  		reg = <0>;
>  		compatible = "arduino,unoq-mcu";
> -		pinctrl-0 = <&spidev_cs>;
>  		pinctrl-names = "default";

You should delete this line too

Konrad

Re: [PATCH] arm64: dts: qcom: arduino-imola: fix faulty spidev node

Posted by Riccardo Mereu Linux Kernel 1 month, 2 weeks ago

Sorry forgot about it.
Thanks

Best wishes.
Riccardo

On Thu, Feb 12, 2026 at 1:00 PM Konrad Dybcio
<konrad.dybcio@oss.qualcomm.com> wrote:
>
> On 2/10/26 11:39 AM, Riccardo Mereu wrote:
> > From: Riccardo Mereu <r.mereu@arduino.cc>
> >
> > CS pin added on pinctrl0 property is causing spidev to return -ENODEV
> > since that GPIO is already part of spi5 pinmuxing.
> >
> > Fixes: 3f745bc0f11f ("arm64: dts: qcom: qrb2210: add dts for Arduino unoq")
> > Signed-off-by: Riccardo Mereu <r.mereu@arduino.cc>
> > ---
> >  arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts | 7 -------
> >  1 file changed, 7 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts b/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
> > index 197ab6eb1666..d2c3359dd814 100644
> > --- a/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
> > +++ b/arch/arm64/boot/dts/qcom/qrb2210-arduino-imola.dts
> > @@ -328,18 +328,11 @@ &spi5 {
> >       spidev@0 {
> >               reg = <0>;
> >               compatible = "arduino,unoq-mcu";
> > -             pinctrl-0 = <&spidev_cs>;
> >               pinctrl-names = "default";
>
> You should delete this line too
>
> Konrad