Re: [PATCH v2] arm, am335x: add support for Moxa UC-8100-ME-T open platform

[Tony Lindgren]

* Rob Herring  [170810 08:53]:
> On Wed, Aug 02, 2017 at 02:57:57PM +0800, SZ Lin wrote:
> > Add support for Moxa UC-8100-ME-T open platform
...

> > +   vbat: fixedregulator@0 {
> 
> Unit address without reg property is not valid (build with W=2). Perhaps 
> vbat-regulator instead.
...

> > +   /* Power supply provides a fixed 3.3V @3A */
> > +   vmmcsd_fixed: fixedregulator@1 {
> 
> Ditto
> 
> With those fixed,
> 
> Acked-by: Rob Herring 

I've fixed it up here and will apply into omap-for-v4.14/dt
thanks.

Tony

Re: [PATCH v2] arm, am335x: add support for Moxa UC-8100-ME-T open platform

[Tony Lindgren]

* Rob Herring  [170810 08:53]:
> On Wed, Aug 02, 2017 at 02:57:57PM +0800, SZ Lin wrote:
> > Add support for Moxa UC-8100-ME-T open platform
...

> > +   vbat: fixedregulator@0 {
> 
> Unit address without reg property is not valid (build with W=2). Perhaps 
> vbat-regulator instead.
...

> > +   /* Power supply provides a fixed 3.3V @3A */
> > +   vmmcsd_fixed: fixedregulator@1 {
> 
> Ditto
> 
> With those fixed,
> 
> Acked-by: Rob Herring 

I've fixed it up here and will apply into omap-for-v4.14/dt
thanks.

Tony

Re: [PATCH v2] arm, am335x: add support for Moxa UC-8100-ME-T open platform

[Rob Herring]

On Wed, Aug 02, 2017 at 02:57:57PM +0800, SZ Lin wrote:
> Add support for Moxa UC-8100-ME-T open platform
> 
> The UC-8100-ME-T computing platform is designed
> for embedded data acquisition industrial applications
> 
> The features of UC-8100-ME-T series are:
> * eMMC
> * SPI flash
> * SD slot
> * 2x LAN
> * 2 RS-232/422/485 ports, software-selectable
> * Mini PCIe form factor with USB signal
> * USB host
> * EEPROM
> * TPM
> * Watchdog
> * RTC
> * User gpio-keys
> * User LEDs
> * User button
> 
> Signed-off-by: SZ Lin 
> ---
> Changes from v1:
> - Remove the un-needed @ in led@x's
> - Use AM33XX_IOPAD pinmux macro
> - Properly alphabetize Makefile
> 
>  .../devicetree/bindings/arm/omap/omap.txt  |   3 +
>  arch/arm/boot/dts/Makefile |   1 +
>  arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts | 525 
> +
>  3 files changed, 529 insertions(+)
>  create mode 100644 arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts
> 
> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt 
> b/Documentation/devicetree/bindings/arm/omap/omap.txt
> index 8219b2c6bb29..72ad8700156a 100644
> --- a/Documentation/devicetree/bindings/arm/omap/omap.txt
> +++ b/Documentation/devicetree/bindings/arm/omap/omap.txt
> @@ -154,6 +154,9 @@ Boards:
>  - AM335X phyCORE-AM335x: Development kit
>compatible = "phytec,am335x-pcm-953", "phytec,am335x-phycore-som", 
> "ti,am33xx"
>  
> +- AM335X UC-8100-ME-T: Communication-centric industrial computing platform
> +  compatible = "moxa,uc-8100-me-t", "ti,am33xx";
> +
>  - OMAP5 EVM : Evaluation Module
>compatible = "ti,omap5-evm", "ti,omap5"
>  
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 4b17f35dc9a7..c293e841eea3 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -617,6 +617,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \
>   am335x-evmsk.dtb \
>   am335x-icev2.dtb \
>   am335x-lxm.dtb \
> + am335x-moxa-uc-8100-me-t.dtb \
>   am335x-nano.dtb \
>   am335x-pepper.dtb \
>   am335x-phycore-rdk.dtb \
> diff --git a/arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts 
> b/arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts
> new file mode 100644
> index ..0aa1d10a0bed
> --- /dev/null
> +++ b/arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts
> @@ -0,0 +1,525 @@
> +/*
> + * Copyright (C) 2017 MOXA Inc. - https://www.moxa.com/
> + *
> + * Author: SZ Lin (林上智) 
> + *
> + * 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.
> + */
> +
> +/dts-v1/;
> +
> +#include "am33xx.dtsi"
> +
> +/ {
> + model = "Moxa UC-8100-ME-T";
> + compatible = "moxa,uc-8100-me-t", "ti,am33xx";
> +
> + cpus {
> + cpu@0 {
> + cpu0-supply = <_reg>;
> + };
> + };
> +
> + memory {
> + device_type = "memory";
> + reg = <0x8000 0x2000>; /* 512 MB */
> + };
> +
> + vbat: fixedregulator@0 {

Unit address without reg property is not valid (build with W=2). Perhaps 
vbat-regulator instead.

> + compatible = "regulator-fixed";
> + };
> +
> + /* Power supply provides a fixed 3.3V @3A */
> + vmmcsd_fixed: fixedregulator@1 {

Ditto

With those fixed,

Acked-by: Rob Herring 

Re: [PATCH v2] arm, am335x: add support for Moxa UC-8100-ME-T open platform

[Rob Herring]

On Wed, Aug 02, 2017 at 02:57:57PM +0800, SZ Lin wrote:
> Add support for Moxa UC-8100-ME-T open platform
> 
> The UC-8100-ME-T computing platform is designed
> for embedded data acquisition industrial applications
> 
> The features of UC-8100-ME-T series are:
> * eMMC
> * SPI flash
> * SD slot
> * 2x LAN
> * 2 RS-232/422/485 ports, software-selectable
> * Mini PCIe form factor with USB signal
> * USB host
> * EEPROM
> * TPM
> * Watchdog
> * RTC
> * User gpio-keys
> * User LEDs
> * User button
> 
> Signed-off-by: SZ Lin 
> ---
> Changes from v1:
> - Remove the un-needed @ in led@x's
> - Use AM33XX_IOPAD pinmux macro
> - Properly alphabetize Makefile
> 
>  .../devicetree/bindings/arm/omap/omap.txt  |   3 +
>  arch/arm/boot/dts/Makefile |   1 +
>  arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts | 525 
> +
>  3 files changed, 529 insertions(+)
>  create mode 100644 arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts
> 
> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt 
> b/Documentation/devicetree/bindings/arm/omap/omap.txt
> index 8219b2c6bb29..72ad8700156a 100644
> --- a/Documentation/devicetree/bindings/arm/omap/omap.txt
> +++ b/Documentation/devicetree/bindings/arm/omap/omap.txt
> @@ -154,6 +154,9 @@ Boards:
>  - AM335X phyCORE-AM335x: Development kit
>compatible = "phytec,am335x-pcm-953", "phytec,am335x-phycore-som", 
> "ti,am33xx"
>  
> +- AM335X UC-8100-ME-T: Communication-centric industrial computing platform
> +  compatible = "moxa,uc-8100-me-t", "ti,am33xx";
> +
>  - OMAP5 EVM : Evaluation Module
>compatible = "ti,omap5-evm", "ti,omap5"
>  
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 4b17f35dc9a7..c293e841eea3 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -617,6 +617,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \
>   am335x-evmsk.dtb \
>   am335x-icev2.dtb \
>   am335x-lxm.dtb \
> + am335x-moxa-uc-8100-me-t.dtb \
>   am335x-nano.dtb \
>   am335x-pepper.dtb \
>   am335x-phycore-rdk.dtb \
> diff --git a/arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts 
> b/arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts
> new file mode 100644
> index ..0aa1d10a0bed
> --- /dev/null
> +++ b/arch/arm/boot/dts/am335x-moxa-uc-8100-me-t.dts
> @@ -0,0 +1,525 @@
> +/*
> + * Copyright (C) 2017 MOXA Inc. - https://www.moxa.com/
> + *
> + * Author: SZ Lin (林上智) 
> + *
> + * 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.
> + */
> +
> +/dts-v1/;
> +
> +#include "am33xx.dtsi"
> +
> +/ {
> + model = "Moxa UC-8100-ME-T";
> + compatible = "moxa,uc-8100-me-t", "ti,am33xx";
> +
> + cpus {
> + cpu@0 {
> + cpu0-supply = <_reg>;
> + };
> + };
> +
> + memory {
> + device_type = "memory";
> + reg = <0x8000 0x2000>; /* 512 MB */
> + };
> +
> + vbat: fixedregulator@0 {

Unit address without reg property is not valid (build with W=2). Perhaps 
vbat-regulator instead.

> + compatible = "regulator-fixed";
> + };
> +
> + /* Power supply provides a fixed 3.3V @3A */
> + vmmcsd_fixed: fixedregulator@1 {

Ditto

With those fixed,

Acked-by: Rob Herring