[PATCH 1/2] ARM64: dts: marvell: cn9310: Use appropriate label for spi1 pins

Stefan Roese sr at denx.de
Thu Aug 24 13:29:16 CEST 2023


On 8/21/23 01:31, Chris Packham wrote:
> The CN9130-DB uses the SPI1 interface but had the pinctrl node labelled
> as "cp0_spi0_pins". Use the label "cp0_spi1_pins" and update the node
> name to "cp0-spi-pins-1" to avoid confusion with the pinctrl options for
> SPI0.
> 
> Signed-off-by: Chris Packham <judge.packham at gmail.com>

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan

> ---
> 
>   arch/arm/dts/cn9130-db.dtsi | 2 +-
>   arch/arm/dts/cn9130.dtsi    | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/dts/cn9130-db.dtsi b/arch/arm/dts/cn9130-db.dtsi
> index 1b28732ee752..4b21ff46d507 100644
> --- a/arch/arm/dts/cn9130-db.dtsi
> +++ b/arch/arm/dts/cn9130-db.dtsi
> @@ -183,7 +183,7 @@
>   /* U55 */
>   &cp0_spi1 {
>   	pinctrl-names = "default";
> -	pinctrl-0 = <&cp0_spi0_pins>;
> +	pinctrl-0 = <&cp0_spi1_pins>;
>   	reg = <0x700680 0x50>,		/* control */
>   	      <0x2000000 0x1000000>,	/* CS0 */
>   	      <0 0xffffffff>,		/* CS1 */
> diff --git a/arch/arm/dts/cn9130.dtsi b/arch/arm/dts/cn9130.dtsi
> index 68b767a70639..efcb2e906b91 100644
> --- a/arch/arm/dts/cn9130.dtsi
> +++ b/arch/arm/dts/cn9130.dtsi
> @@ -66,7 +66,7 @@
>   		marvell,pins = < 56 57 58 59 60 61 >;
>   		marvell,function = <14>;
>   	};
> -	cp0_spi0_pins: cp0-spi-pins-0 {
> +	cp0_spi1_pins: cp0-spi-pins-1 {
>   		marvell,pins = < 13 14 15 16 >;
>   		marvell,function = <3>;
>   	};

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de


More information about the U-Boot mailing list