Re: [PATCH] ARM: dts: Add am335x-pocketbeagle

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Rob,

All fixed, posting v2 shortly..

On Mon, Apr 16, 2018 at 3:21 PM, Rob Herring <robh@xxxxxxxxxx> wrote:
> On Mon, Apr 16, 2018 at 2:29 PM, Robert Nelson <robertcnelson@xxxxxxxxx> wrote:
>> PocketBeagle Blue is an ultra-tiny-yet-complete open-source USB-key-fob computer.
>>
>> This board can be indentified by the BPxx value after A335BNLT (BBB)
>> in the at24 eeprom:
>> BPxx [aa 55 33 ee 41 33 33 35  42 4e 4c 54 42 50 30 30 |.U3.A335BNLTBP00|]
>>
>> http://beagleboard.org/pocket
>> https://github.com/beagleboard/pocketbeagle
>>
>> Signed-off-by: Robert Nelson <robertcnelson@xxxxxxxxx>
>> CC: Tony Lindgren <tony@xxxxxxxxxxx>
>> CC: Jason Kridner <jkridner@xxxxxxxxxxxxxxx>
>> CC: Drew Fustini <drew@xxxxxxxxxxxxxxx>
>> CC: Peter Robinson <pbrobinson@xxxxxxxxxx>
>> ---
>>  arch/arm/boot/dts/Makefile                   |   1 +
>>  arch/arm/boot/dts/am335x-osd335x-common.dtsi | 136 +++++++++++
>>  arch/arm/boot/dts/am335x-pocketbeagle.dts    | 244 +++++++++++++++++++
>>  3 files changed, 381 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/am335x-osd335x-common.dtsi
>>  create mode 100644 arch/arm/boot/dts/am335x-pocketbeagle.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 7e2424957809..5a09ff15743b 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -688,6 +688,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \
>>         am335x-pdu001.dtb \
>>         am335x-pepper.dtb \
>>         am335x-phycore-rdk.dtb \
>> +       am335x-pocketbeagle.dtb \
>>         am335x-shc.dtb \
>>         am335x-sbc-t335.dtb \
>>         am335x-sl50.dtb \
>> diff --git a/arch/arm/boot/dts/am335x-osd335x-common.dtsi b/arch/arm/boot/dts/am335x-osd335x-common.dtsi
>> new file mode 100644
>> index 000000000000..184b3a9f1538
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/am335x-osd335x-common.dtsi
>> @@ -0,0 +1,136 @@
>> +/*
>> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License version 2 as
>> + * published by the Free Software Foundation.
>
> Use SPDX tag.

Fixed

>> + */
>> +
>> +/ {
>> +       cpus {
>> +               cpu@0 {
>> +                       cpu0-supply = <&dcdc2_reg>;
>> +               };
>> +       };
>> +
>> +       memory@80000000 {
>> +               device_type = "memory";
>> +               reg = <0x80000000 0x20000000>; /* 512 MB */
>> +       };
>> +};
>> +
>> +&cpu0_opp_table {
>> +       /*
>> +       * Octavo Systems:
>> +       * The EFUSE_SMA register is not programmed for any of the AM335x wafers
>> +       * we get and we are not programming them during our production test.
>> +       * Therefore, from a DEVICE_ID revision point of view, the silicon looks
>> +       * like it is Revision 2.1.  However, from an EFUSE_SMA point of view for
>> +       * the HW OPP table, the silicon looks like it is Revision 1.0 (ie the
>> +       * EFUSE_SMA register reads as all zeros).
>> +       */
>> +       oppnitro-1000000000 {
>> +               opp-supported-hw = <0x06 0x0100>;
>> +       };
>> +};
>> +
>> +&am33xx_pinmux {
>> +       i2c0_pins: pinmux_i2c0_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x988, PIN_INPUT_PULLUP | MUX_MODE0)       /* (C17) I2C0_SDA.I2C0_SDA */
>> +                       AM33XX_IOPAD(0x98c, PIN_INPUT_PULLUP | MUX_MODE0)       /* (C16) I2C0_SCL.I2C0_SCL */
>> +               >;
>> +       };
>> +};
>> +
>> +&i2c0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&i2c0_pins>;
>> +
>> +       status = "okay";
>> +       clock-frequency = <400000>;
>> +
>> +       tps: tps@24 {
>> +               reg = <0x24>;
>> +       };
>> +
>> +       baseboard_eeprom: baseboard_eeprom@50 {
>
> eeprom@50

fixed..

>
> Build the dtb with "W=12" and fix all the warnings (unless you've
> inherited them from the includes).

It looks like i need to clean a few more am335x-* targets, as i used
pinmux_X_pins quite a bit...

>
>> +               compatible = "atmel,24c256";
>> +               reg = <0x50>;
>> +
>> +               #address-cells = <1>;
>> +               #size-cells = <1>;
>> +               baseboard_data: baseboard_data@0 {
>> +                       reg = <0 0x100>;
>> +               };
>> +       };
>> +};
>> +
>> +/include/ "tps65217.dtsi"
>> +
>> +&tps {
>> +       interrupts = <7>; /* NMI */
>> +       interrupt-parent = <&intc>;
>> +
>> +       ti,pmic-shutdown-controller;
>> +
>> +       pwrbutton {
>> +               interrupts = <2>;
>> +               status = "okay";
>> +       };
>> +
>> +       regulators {
>> +               dcdc1_reg: regulator@0 {
>> +                       regulator-name = "vdds_dpr";
>> +                       regulator-always-on;
>> +               };
>> +
>> +               dcdc2_reg: regulator@1 {
>> +                       /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
>> +                       regulator-name = "vdd_mpu";
>> +                       regulator-min-microvolt = <925000>;
>> +                       regulator-max-microvolt = <1351500>;
>> +                       regulator-boot-on;
>> +                       regulator-always-on;
>> +               };
>> +
>> +               dcdc3_reg: regulator@2 {
>> +                       /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
>> +                       regulator-name = "vdd_core";
>> +                       regulator-min-microvolt = <925000>;
>> +                       regulator-max-microvolt = <1150000>;
>> +                       regulator-boot-on;
>> +                       regulator-always-on;
>> +               };
>> +
>> +               ldo1_reg: regulator@3 {
>> +                       regulator-name = "vio,vrtc,vdds";
>> +                       regulator-always-on;
>> +               };
>> +
>> +               ldo2_reg: regulator@4 {
>> +                       regulator-name = "vdd_3v3aux";
>> +                       regulator-always-on;
>> +               };
>> +
>> +               ldo3_reg: regulator@5 {
>> +                       regulator-name = "vdd_1v8";
>> +                       regulator-min-microvolt = <1800000>;
>> +                       regulator-max-microvolt = <1800000>;
>> +                       regulator-always-on;
>> +               };
>> +
>> +               ldo4_reg: regulator@6 {
>> +                       regulator-name = "vdd_3v3a";
>> +                       regulator-always-on;
>> +               };
>> +       };
>> +};
>> +
>> +&aes {
>> +       status = "okay";
>> +};
>> +
>> +&sham {
>> +       status = "okay";
>> +};
>> diff --git a/arch/arm/boot/dts/am335x-pocketbeagle.dts b/arch/arm/boot/dts/am335x-pocketbeagle.dts
>> new file mode 100644
>> index 000000000000..c224cde88920
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/am335x-pocketbeagle.dts
>> @@ -0,0 +1,244 @@
>> +/*
>> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License version 2 as
>> + * published by the Free Software Foundation.
>
> Use SPDX tag.

fixed

>
>> + */
>> +/dts-v1/;
>> +
>> +#include "am33xx.dtsi"
>> +#include "am335x-osd335x-common.dtsi"
>> +
>> +/ {
>> +       model = "TI AM335x PocketBeagle";
>> +       compatible = "ti,am335x-pocketbeagle", "ti,am335x-bone", "ti,am33xx";
>> +
>> +       chosen {
>> +               stdout-path = &uart0;
>> +       };
>> +
>> +       leds {
>> +               pinctrl-names = "default";
>> +               pinctrl-0 = <&user_leds_s0>;
>> +
>> +               compatible = "gpio-leds";
>> +
>> +               usr_0_led {
>> +                       label = "beaglebone:green:usr0";
>> +                       gpios = <&gpio1 21 GPIO_ACTIVE_HIGH>;
>> +                       linux,default-trigger = "heartbeat";
>> +                       default-state = "off";
>> +               };
>> +
>> +               usr_1_led {
>> +                       label = "beaglebone:green:usr1";
>> +                       gpios = <&gpio1 22 GPIO_ACTIVE_HIGH>;
>> +                       linux,default-trigger = "mmc0";
>> +                       default-state = "off";
>> +               };
>> +
>> +               usr_2_led {
>> +                       label = "beaglebone:green:usr2";
>> +                       gpios = <&gpio1 23 GPIO_ACTIVE_HIGH>;
>> +                       linux,default-trigger = "cpu0";
>> +                       default-state = "off";
>> +               };
>> +
>> +               usr_3_led {
>> +                       label = "beaglebone:green:usr3";
>> +                       gpios = <&gpio1 24 GPIO_ACTIVE_HIGH>;
>> +                       default-state = "off";
>> +               };
>> +       };
>> +
>> +       vmmcsd_fixed: fixedregulator0 {
>> +               compatible = "regulator-fixed";
>> +               regulator-name = "vmmcsd_fixed";
>> +               regulator-min-microvolt = <3300000>;
>> +               regulator-max-microvolt = <3300000>;
>> +       };
>> +};
>> +
>> +&am33xx_pinmux {
>> +       i2c2_pins: pinmux_i2c2_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x97c, PIN_INPUT_PULLUP | MUX_MODE3)       /* (D17) uart1_rtsn.I2C2_SCL */
>> +                       AM33XX_IOPAD(0x978, PIN_INPUT_PULLUP | MUX_MODE3)       /* (D18) uart1_ctsn.I2C2_SDA */
>> +               >;
>> +       };
>> +
>> +       ehrpwm0_pins: pinmux_ehrpwm0_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x990, PIN_OUTPUT_PULLDOWN | MUX_MODE1)    /* (A13) mcasp0_aclkx.ehrpwm0A */
>> +               >;
>> +       };
>> +
>> +       ehrpwm1_pins: pinmux_ehrpwm1_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x848, PIN_OUTPUT_PULLDOWN | MUX_MODE6)    /* (U14) gpmc_a2.ehrpwm1A */
>> +               >;
>> +       };
>> +
>> +       mmc0_pins: pinmux_mmc0_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x960, PIN_INPUT | MUX_MODE7)              /* (C15) spi0_cs1.gpio0[6] */
>> +                       AM33XX_IOPAD(0x8fc, PIN_INPUT_PULLUP | MUX_MODE0)       /* (G16) mmc0_dat0.mmc0_dat0 */
>> +                       AM33XX_IOPAD(0x8f8, PIN_INPUT_PULLUP | MUX_MODE0)       /* (G15) mmc0_dat1.mmc0_dat1 */
>> +                       AM33XX_IOPAD(0x8f4, PIN_INPUT_PULLUP | MUX_MODE0)       /* (F18) mmc0_dat2.mmc0_dat2 */
>> +                       AM33XX_IOPAD(0x8f0, PIN_INPUT_PULLUP | MUX_MODE0)       /* (F17) mmc0_dat3.mmc0_dat3 */
>> +                       AM33XX_IOPAD(0x904, PIN_INPUT_PULLUP | MUX_MODE0)       /* (G18) mmc0_cmd.mmc0_cmd */
>> +                       AM33XX_IOPAD(0x900, PIN_INPUT_PULLUP | MUX_MODE0)       /* (G17) mmc0_clk.mmc0_clk */
>> +                       AM33XX_IOPAD(0x9a0, PIN_INPUT | MUX_MODE4)              /* (B12) mcasp0_aclkr.mmc0_sdwp */
>> +               >;
>> +       };
>> +
>> +       spi0_pins: pinmux_spi0_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x950, PIN_INPUT_PULLUP | MUX_MODE0)       /* (A17) spi0_sclk.spi0_sclk */
>> +                       AM33XX_IOPAD(0x954, PIN_INPUT_PULLUP | MUX_MODE0)       /* (B17) spi0_d0.spi0_d0 */
>> +                       AM33XX_IOPAD(0x958, PIN_INPUT_PULLUP | MUX_MODE0)       /* (B16) spi0_d1.spi0_d1 */
>> +                       AM33XX_IOPAD(0x95c, PIN_INPUT_PULLUP | MUX_MODE0)       /* (A16) spi0_cs0.spi0_cs0 */
>> +               >;
>> +       };
>> +
>> +       spi1_pins: pinmux_spi1_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x964, PIN_INPUT_PULLUP | MUX_MODE4)       /* (C18) eCAP0_in_PWM0_out.spi1_sclk */
>> +                       AM33XX_IOPAD(0x968, PIN_INPUT_PULLUP | MUX_MODE4)       /* (E18) uart0_ctsn.spi1_d0 */
>> +                       AM33XX_IOPAD(0x96c, PIN_INPUT_PULLUP | MUX_MODE4)       /* (E17) uart0_rtsn.spi1_d1 */
>> +                       AM33XX_IOPAD(0x9b0, PIN_INPUT_PULLUP | MUX_MODE4)       /* (A15) xdma_event_intr0.spi1_cs1 */
>> +               >;
>> +       };
>> +
>> +       user_leds_s0: user_leds_s0 {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x854, PIN_OUTPUT | MUX_MODE7)             /* (V15) gpmc_a5.gpio1[21] - USR_LED_0 */
>> +                       AM33XX_IOPAD(0x858, PIN_OUTPUT | MUX_MODE7)             /* (U15) gpmc_a6.gpio1[22] - USR_LED_1 */
>> +                       AM33XX_IOPAD(0x85c, PIN_OUTPUT | MUX_MODE7)             /* (T15) gpmc_a7.gpio1[23] - USR_LED_2 */
>> +                       AM33XX_IOPAD(0x860, PIN_OUTPUT | MUX_MODE7)             /* (V16) gpmc_a8.gpio1[24] - USR_LED_3 */
>> +               >;
>> +       };
>> +
>> +       uart0_pins: pinmux_uart0_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x970, PIN_INPUT_PULLUP | MUX_MODE0)       /* (E15) uart0_rxd.uart0_rxd */
>> +                       AM33XX_IOPAD(0x974, PIN_OUTPUT_PULLDOWN | MUX_MODE0)    /* (E16) uart0_txd.uart0_txd */
>> +               >;
>> +       };
>> +
>> +       uart4_pins: pinmux_uart4_pins {
>> +               pinctrl-single,pins = <
>> +                       AM33XX_IOPAD(0x870, PIN_INPUT_PULLUP | MUX_MODE6)       /* (T17) gpmc_wait0.uart4_rxd */
>> +                       AM33XX_IOPAD(0x874, PIN_OUTPUT_PULLDOWN | MUX_MODE6)    /* (U17) gpmc_wpn.uart4_txd */
>> +               >;
>> +       };
>> +};
>> +
>> +&epwmss0 {
>> +       status = "okay";
>> +};
>> +
>> +&ehrpwm0 {
>> +       status = "okay";
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&ehrpwm0_pins>;
>> +};
>> +
>> +&epwmss1 {
>> +       status = "okay";
>> +};
>> +
>> +&ehrpwm1 {
>> +       status = "okay";
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&ehrpwm1_pins>;
>> +};
>> +
>> +&i2c0 {
>> +       baseboard_eeprom: baseboard_eeprom@50 {
>
> This appears twice?

Correct, now removed and cleaned up...

>
>> +               compatible = "atmel,24c256";
>> +               reg = <0x50>;
>> +
>> +               #address-cells = <1>;
>> +               #size-cells = <1>;
>> +               baseboard_data: baseboard_data@0 {
>> +                       reg = <0 0x100>;
>> +               };
>> +       };
>> +};
>> +
>> +&i2c2 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&i2c2_pins>;
>> +
>> +       status = "okay";
>> +       clock-frequency = <400000>;
>> +};
>> +
>> +&mmc1 {
>> +       status = "okay";
>> +       vmmc-supply = <&vmmcsd_fixed>;
>> +       bus-width = <4>;
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc0_pins>;
>> +       cd-gpios = <&gpio0 6 GPIO_ACTIVE_LOW>;
>> +};
>> +
>> +&rtc {
>> +       system-power-controller;
>> +};
>> +
>> +&tscadc {
>> +       status = "okay";
>> +       adc {
>> +               ti,adc-channels = <0 1 2 3 4 5 6 7>;
>> +               ti,chan-step-avg = <16 16 16 16 16 16 16 16>;
>> +               ti,chan-step-opendelay = <0x98 0x98 0x98 0x98 0x98 0x98 0x98 0x98>;
>> +               ti,chan-step-sampledelay = <0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0>;
>> +       };
>> +};
>> +
>> +&uart0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart0_pins>;
>> +
>> +       status = "okay";
>> +};
>> +
>> +&uart4 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart4_pins>;
>> +
>> +       status = "okay";
>> +};
>> +
>> +&usb {
>> +       status = "okay";
>> +};
>> +
>> +&usb_ctrl_mod {
>> +       status = "okay";
>> +};
>> +
>> +&usb0_phy {
>> +       status = "okay";
>> +};
>> +
>> +&usb0 {
>> +       status = "okay";
>> +       dr_mode = "otg";
>> +};
>> +
>> +&usb1_phy {
>> +       status = "okay";
>> +};
>> +
>> +&usb1 {
>> +       status = "okay";
>> +       dr_mode = "host";
>> +};
>> +
>> +&cppi41dma  {
>> +       status = "okay";
>> +};
>> --
>> 2.17.0
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe devicetree" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Robert Nelson
https://rcn-ee.com/
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux