| Message ID | 20210617093330.9179-4-paul.barker@sancloud.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add support for the SanCloud BBE Lite | expand |
Hi Paul, On Thu, Jun 17, 2021 at 4:35 AM Paul Barker <paul.barker@sancloud.com> wrote: > > This adds support for the Sancloud BBE Lite which shares a common > hardware base with the non-Lite version of the BBE. > > Signed-off-by: Paul Barker <paul.barker@sancloud.com> > --- > arch/arm/boot/dts/Makefile | 1 + > .../arm/boot/dts/am335x-sancloud-bbe-lite.dts | 51 +++++++++++++++++++ > 2 files changed, 52 insertions(+) > create mode 100644 arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index f8f09c5066e7..8629c941f573 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -834,6 +834,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \ > am335x-pocketbeagle.dtb \ > am335x-regor-rdk.dtb \ > am335x-sancloud-bbe.dtb \ > + am335x-sancloud-bbe-lite.dtb \ > am335x-shc.dtb \ > am335x-sbc-t335.dtb \ > am335x-sl50.dtb \ > diff --git a/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > new file mode 100644 > index 000000000000..9c311bd106f6 > --- /dev/null > +++ b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > @@ -0,0 +1,51 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/ > + * Copyright (C) 2021 SanCloud Ltd > + */ > +/dts-v1/; > + > +#include "am33xx.dtsi" > +#include "am335x-bone-common.dtsi" > +#include "am335x-boneblack-common.dtsi" > +#include "am335x-sancloud-bbe-common.dtsi" > + > +/ { > + model = "SanCloud BeagleBone Enhanced Lite"; > + compatible = "sancloud,am335x-boneenhanced", > + "ti,am335x-bone-black", > + "ti,am335x-bone", > + "ti,am33xx"; > +}; > + > +&am33xx_pinmux { > + bb_spi0_pins: pinmux_bb_spi0_pins { > + pinctrl-single,pins = < > + AM33XX_PADCONF(AM335X_PIN_SPI0_SCLK, PIN_INPUT, MUX_MODE0) > + AM33XX_PADCONF(AM335X_PIN_SPI0_D0, PIN_INPUT, MUX_MODE0) > + AM33XX_PADCONF(AM335X_PIN_SPI0_D1, PIN_INPUT, MUX_MODE0) > + AM33XX_PADCONF(AM335X_PIN_SPI0_CS0, PIN_INPUT, MUX_MODE0) > + >; > + }; > +}; > + > +&spi0 { > + #address-cells = <1>; > + #size-cells = <0>; > + > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&bb_spi0_pins>; > + > + channel@0 { > + #address-cells = <1>; > + #size-cells = <0>; > + > + compatible = "micron,spi-authenta"; > + symlink = "spi/0.0"; Sorry, this "symlink" is an undocumented "feature" of the BeagleBoard.org tree.. We use it to help "identity" spi/usart/i2c nodes names when they dynamically change node numbers based on kernel versions or even 'drivers'.... https://github.com/beagleboard/customizations/blob/master/etc/udev/rules.d/10-of-symlink.rules For 'mainline' we need to remove it.. Regards,
On Fri, 18 Jun 2021 11:02:12 -0500 Robert Nelson <robertcnelson@gmail.com> wrote: > Hi Paul, > > On Thu, Jun 17, 2021 at 4:35 AM Paul Barker > <paul.barker@sancloud.com> wrote: > > > > This adds support for the Sancloud BBE Lite which shares a common > > hardware base with the non-Lite version of the BBE. > > > > Signed-off-by: Paul Barker <paul.barker@sancloud.com> > > --- > > arch/arm/boot/dts/Makefile | 1 + > > .../arm/boot/dts/am335x-sancloud-bbe-lite.dts | 51 > > +++++++++++++++++++ 2 files changed, 52 insertions(+) > > create mode 100644 arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > index f8f09c5066e7..8629c941f573 100644 > > --- a/arch/arm/boot/dts/Makefile > > +++ b/arch/arm/boot/dts/Makefile > > @@ -834,6 +834,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \ > > am335x-pocketbeagle.dtb \ > > am335x-regor-rdk.dtb \ > > am335x-sancloud-bbe.dtb \ > > + am335x-sancloud-bbe-lite.dtb \ > > am335x-shc.dtb \ > > am335x-sbc-t335.dtb \ > > am335x-sl50.dtb \ > > diff --git a/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts new file mode > > 100644 index 000000000000..9c311bd106f6 > > --- /dev/null > > +++ b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > @@ -0,0 +1,51 @@ > > +// SPDX-License-Identifier: GPL-2.0-only > > +/* > > + * Copyright (C) 2012 Texas Instruments Incorporated - > > http://www.ti.com/ > > + * Copyright (C) 2021 SanCloud Ltd > > + */ > > +/dts-v1/; > > + > > +#include "am33xx.dtsi" > > +#include "am335x-bone-common.dtsi" > > +#include "am335x-boneblack-common.dtsi" > > +#include "am335x-sancloud-bbe-common.dtsi" > > + > > +/ { > > + model = "SanCloud BeagleBone Enhanced Lite"; > > + compatible = "sancloud,am335x-boneenhanced", > > + "ti,am335x-bone-black", > > + "ti,am335x-bone", > > + "ti,am33xx"; > > +}; > > + > > +&am33xx_pinmux { > > + bb_spi0_pins: pinmux_bb_spi0_pins { > > + pinctrl-single,pins = < > > + AM33XX_PADCONF(AM335X_PIN_SPI0_SCLK, > > PIN_INPUT, MUX_MODE0) > > + AM33XX_PADCONF(AM335X_PIN_SPI0_D0, > > PIN_INPUT, MUX_MODE0) > > + AM33XX_PADCONF(AM335X_PIN_SPI0_D1, > > PIN_INPUT, MUX_MODE0) > > + AM33XX_PADCONF(AM335X_PIN_SPI0_CS0, > > PIN_INPUT, MUX_MODE0) > > + >; > > + }; > > +}; > > + > > +&spi0 { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + status = "okay"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&bb_spi0_pins>; > > + > > + channel@0 { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + compatible = "micron,spi-authenta"; > > + symlink = "spi/0.0"; > > Sorry, this "symlink" is an undocumented "feature" of the > BeagleBoard.org tree.. We use it to help "identity" spi/usart/i2c > nodes names when they dynamically change node numbers based on kernel > versions or even 'drivers'.... > > https://github.com/beagleboard/customizations/blob/master/etc/udev/rules.d/10-of-symlink.rules > > For 'mainline' we need to remove it.. Ok, that's a straightforward change. I'll send an updated patch series later in the week. Thanks,
On Tue, 29 Jun 2021 14:10:33 +0100 Paul Barker <paul.barker@sancloud.com> wrote: > On Fri, 18 Jun 2021 11:02:12 -0500 > Robert Nelson <robertcnelson@gmail.com> wrote: > > > Hi Paul, > > > > On Thu, Jun 17, 2021 at 4:35 AM Paul Barker > > <paul.barker@sancloud.com> wrote: > > > > > > This adds support for the Sancloud BBE Lite which shares a common > > > hardware base with the non-Lite version of the BBE. > > > > > > Signed-off-by: Paul Barker <paul.barker@sancloud.com> > > > --- > > > arch/arm/boot/dts/Makefile | 1 + > > > .../arm/boot/dts/am335x-sancloud-bbe-lite.dts | 51 > > > +++++++++++++++++++ 2 files changed, 52 insertions(+) > > > create mode 100644 arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > > > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > > index f8f09c5066e7..8629c941f573 100644 > > > --- a/arch/arm/boot/dts/Makefile > > > +++ b/arch/arm/boot/dts/Makefile > > > @@ -834,6 +834,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \ > > > am335x-pocketbeagle.dtb \ > > > am335x-regor-rdk.dtb \ > > > am335x-sancloud-bbe.dtb \ > > > + am335x-sancloud-bbe-lite.dtb \ > > > am335x-shc.dtb \ > > > am335x-sbc-t335.dtb \ > > > am335x-sl50.dtb \ > > > diff --git a/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > > b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts new file mode > > > 100644 index 000000000000..9c311bd106f6 > > > --- /dev/null > > > +++ b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts > > > @@ -0,0 +1,51 @@ > > > +// SPDX-License-Identifier: GPL-2.0-only > > > +/* > > > + * Copyright (C) 2012 Texas Instruments Incorporated - > > > http://www.ti.com/ > > > + * Copyright (C) 2021 SanCloud Ltd > > > + */ > > > +/dts-v1/; > > > + > > > +#include "am33xx.dtsi" > > > +#include "am335x-bone-common.dtsi" > > > +#include "am335x-boneblack-common.dtsi" > > > +#include "am335x-sancloud-bbe-common.dtsi" > > > + > > > +/ { > > > + model = "SanCloud BeagleBone Enhanced Lite"; > > > + compatible = "sancloud,am335x-boneenhanced", > > > + "ti,am335x-bone-black", > > > + "ti,am335x-bone", > > > + "ti,am33xx"; > > > +}; > > > + > > > +&am33xx_pinmux { > > > + bb_spi0_pins: pinmux_bb_spi0_pins { > > > + pinctrl-single,pins = < > > > + AM33XX_PADCONF(AM335X_PIN_SPI0_SCLK, > > > PIN_INPUT, MUX_MODE0) > > > + AM33XX_PADCONF(AM335X_PIN_SPI0_D0, > > > PIN_INPUT, MUX_MODE0) > > > + AM33XX_PADCONF(AM335X_PIN_SPI0_D1, > > > PIN_INPUT, MUX_MODE0) > > > + AM33XX_PADCONF(AM335X_PIN_SPI0_CS0, > > > PIN_INPUT, MUX_MODE0) > > > + >; > > > + }; > > > +}; > > > + > > > +&spi0 { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + > > > + status = "okay"; > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&bb_spi0_pins>; > > > + > > > + channel@0 { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + > > > + compatible = "micron,spi-authenta"; > > > + symlink = "spi/0.0"; > > > > Sorry, this "symlink" is an undocumented "feature" of the > > BeagleBoard.org tree.. We use it to help "identity" spi/usart/i2c > > nodes names when they dynamically change node numbers based on kernel > > versions or even 'drivers'.... > > > > https://github.com/beagleboard/customizations/blob/master/etc/udev/rules.d/10-of-symlink.rules > > > > For 'mainline' we need to remove it.. > > Ok, that's a straightforward change. I'll send an updated patch series > later in the week. > > Thanks, > Sorry folks, v2 got delayed due to illness. I've sent it today. Thanks,
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index f8f09c5066e7..8629c941f573 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -834,6 +834,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \ am335x-pocketbeagle.dtb \ am335x-regor-rdk.dtb \ am335x-sancloud-bbe.dtb \ + am335x-sancloud-bbe-lite.dtb \ am335x-shc.dtb \ am335x-sbc-t335.dtb \ am335x-sl50.dtb \ diff --git a/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts new file mode 100644 index 000000000000..9c311bd106f6 --- /dev/null +++ b/arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts @@ -0,0 +1,51 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/ + * Copyright (C) 2021 SanCloud Ltd + */ +/dts-v1/; + +#include "am33xx.dtsi" +#include "am335x-bone-common.dtsi" +#include "am335x-boneblack-common.dtsi" +#include "am335x-sancloud-bbe-common.dtsi" + +/ { + model = "SanCloud BeagleBone Enhanced Lite"; + compatible = "sancloud,am335x-boneenhanced", + "ti,am335x-bone-black", + "ti,am335x-bone", + "ti,am33xx"; +}; + +&am33xx_pinmux { + bb_spi0_pins: pinmux_bb_spi0_pins { + pinctrl-single,pins = < + AM33XX_PADCONF(AM335X_PIN_SPI0_SCLK, PIN_INPUT, MUX_MODE0) + AM33XX_PADCONF(AM335X_PIN_SPI0_D0, PIN_INPUT, MUX_MODE0) + AM33XX_PADCONF(AM335X_PIN_SPI0_D1, PIN_INPUT, MUX_MODE0) + AM33XX_PADCONF(AM335X_PIN_SPI0_CS0, PIN_INPUT, MUX_MODE0) + >; + }; +}; + +&spi0 { + #address-cells = <1>; + #size-cells = <0>; + + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&bb_spi0_pins>; + + channel@0 { + #address-cells = <1>; + #size-cells = <0>; + + compatible = "micron,spi-authenta"; + symlink = "spi/0.0"; + + reg = <0>; + spi-max-frequency = <16000000>; + spi-cpha; + }; +};
This adds support for the Sancloud BBE Lite which shares a common hardware base with the non-Lite version of the BBE. Signed-off-by: Paul Barker <paul.barker@sancloud.com> --- arch/arm/boot/dts/Makefile | 1 + .../arm/boot/dts/am335x-sancloud-bbe-lite.dts | 51 +++++++++++++++++++ 2 files changed, 52 insertions(+) create mode 100644 arch/arm/boot/dts/am335x-sancloud-bbe-lite.dts