Re: [U-Boot] [PATCH v2 3/3] am335x, guardian: Add support for the bosch guardian board

2019-02-25 Thread LTEC AG
On 22.02.19 18:52, Martyn Welch wrote:

> From: Sjoerd Simons 
> 
> Add support for the Bosch Guardian board.
> 
> CPU  : AM335X-GP rev 2.1
> Model: Bosch AM335x Guardian
> I2C:   ready
> DRAM:  256 MiB
> NAND:  512 MiB
> MMC:   OMAP SD/MMC: 0
> 
> Signed-off-by: Sjoerd Simons 
> Signed-off-by: Martyn Welch 
> 
> ---
> 
> Changes in v2:
> - Correct guard #ifdef naming in board config header
> - Remove missed cruft from board files
> 
>  arch/arm/Kconfig |   1 +
>  arch/arm/dts/am335x-guardian-u-boot.dtsi |  66 +++
>  arch/arm/dts/am335x-guardian.dts | 511 +++
>  arch/arm/mach-omap2/am33xx/Kconfig   |   7 +
>  board/bosch/guardian/Kconfig |  15 +
>  board/bosch/guardian/MAINTAINERS |   6 +
>  board/bosch/guardian/Makefile|  12 +
>  board/bosch/guardian/board.c | 186 +
>  board/bosch/guardian/board.h |  17 +
>  board/bosch/guardian/mux.c   |  99 +
>  configs/am335x_guardian_defconfig|  90 
>  include/configs/am335x_guardian.h| 111 +
>  12 files changed, 1121 insertions(+)
>  create mode 100644 arch/arm/dts/am335x-guardian-u-boot.dtsi
>  create mode 100644 arch/arm/dts/am335x-guardian.dts
>  create mode 100644 board/bosch/guardian/Kconfig
>  create mode 100644 board/bosch/guardian/MAINTAINERS
>  create mode 100644 board/bosch/guardian/Makefile
>  create mode 100644 board/bosch/guardian/board.c
>  create mode 100644 board/bosch/guardian/board.h
>  create mode 100644 board/bosch/guardian/mux.c
>  create mode 100644 configs/am335x_guardian_defconfig
>  create mode 100644 include/configs/am335x_guardian.h
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 455f06cfee..d2b3dce2da 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1524,6 +1524,7 @@ source "arch/arm/cpu/armv8/Kconfig"
>  source "arch/arm/mach-imx/Kconfig"
>  
>  source "board/bosch/shc/Kconfig"
> +source "board/bosch/guardian/Kconfig"
>  source "board/CarMediaLab/flea3/Kconfig"
>  source "board/Marvell/aspenite/Kconfig"
>  source "board/Marvell/gplugd/Kconfig"

[...]

> diff --git a/arch/arm/dts/am335x-guardian.dts 
> b/arch/arm/dts/am335x-guardian.dts
> new file mode 100644
> index 00..d8652ab014
> --- /dev/null
> +++ b/arch/arm/dts/am335x-guardian.dts
> @@ -0,0 +1,511 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
> + * Copyright (C) 2018 Robert Bosch Power Tools GmbH
> + */
> +/dts-v1/;
> +
> +#include "am33xx.dtsi"
> +#include 
> +#include 
> +
> +/ {
> + model = "Bosch AM335x Guardian";
> + compatible = "bosch,am335x-guardian", "ti,am33xx";
> +
> + chosen {
> + stdout-path = &uart0;
> + tick-timer = &timer2;
> + };
> +
> + cpus {
> + cpu@0 {
> + cpu0-supply = <&dcdc2_reg>;
> + };
> + };
> +
> + memory@8000 {
> + device_type = "memory";
> + reg = <0x8000 0x1000>; /* 256 MB */
> + };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> + #address-cells = <1>;
> + #size-cells = <0>;
The properties #address-cells and #size-cells can be removed as the only
child node does not have a reg property.

> + pinctrl-names = "default";
> + pinctrl-0 = <&gpio_keys_pins>;
> +
> + button21 {
> + label = "guardian-power-button";
> + linux,code = ;
> + gpios = <&gpio2 21 0>;
> + wakeup-source;
> + };
> + };
> +
> + leds {
> + compatible = "gpio-leds";
> + pinctrl-names = "default";
> + pinctrl-0 = <&leds_pins>;
> +
> + led1 {
> + label = "green:heartbeat";
> + gpios = <&gpio1 27 GPIO_ACTIVE_HIGH>;
> + linux,default-trigger = "heartbeat";
> + default-state = "off";
> + };
> +
> + led2 {
> + label = "green:mmc0";
> + gpios = <&gpio1 26 GPIO_ACTIVE_HIGH>;
> + linux,default-trigger = "mmc0";
> + default-state = "off";
> + };
> + };
> +

[...]

> diff --git a/board/bosch/guardian/board.c b/board/bosch/guardian/board.c
> new file mode 100644
> index 00..7f6f500535
> --- /dev/null
> +++ b/board/bosch/guardian/board.c
> @@ -0,0 +1,186 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * board.c
> + *
> + * Board functions for Bosch Guardian
> + *
> + * Copyright (C) 2011, Texas Instruments, Incorporated - http://www.ti.com/
> + * Copyright (C) 2018 Robert Bosch Power Tools GmbH
> + */
> +
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#inc

Re: [U-Boot] [PATCH v2 3/3] am335x, guardian: Add support for the bosch guardian board

2019-02-25 Thread Felix Brack
On 22.02.19 18:52, Martyn Welch wrote:

Same post as some minutes ago but with correct sender, sorry.

> From: Sjoerd Simons 
> 
> Add support for the Bosch Guardian board.
> 
> CPU  : AM335X-GP rev 2.1
> Model: Bosch AM335x Guardian
> I2C:   ready
> DRAM:  256 MiB
> NAND:  512 MiB
> MMC:   OMAP SD/MMC: 0
> 
> Signed-off-by: Sjoerd Simons 
> Signed-off-by: Martyn Welch 
> 
> ---
> 
> Changes in v2:
> - Correct guard #ifdef naming in board config header
> - Remove missed cruft from board files
> 
>  arch/arm/Kconfig |   1 +
>  arch/arm/dts/am335x-guardian-u-boot.dtsi |  66 +++
>  arch/arm/dts/am335x-guardian.dts | 511 +++
>  arch/arm/mach-omap2/am33xx/Kconfig   |   7 +
>  board/bosch/guardian/Kconfig |  15 +
>  board/bosch/guardian/MAINTAINERS |   6 +
>  board/bosch/guardian/Makefile|  12 +
>  board/bosch/guardian/board.c | 186 +
>  board/bosch/guardian/board.h |  17 +
>  board/bosch/guardian/mux.c   |  99 +
>  configs/am335x_guardian_defconfig|  90 
>  include/configs/am335x_guardian.h| 111 +
>  12 files changed, 1121 insertions(+)
>  create mode 100644 arch/arm/dts/am335x-guardian-u-boot.dtsi
>  create mode 100644 arch/arm/dts/am335x-guardian.dts
>  create mode 100644 board/bosch/guardian/Kconfig
>  create mode 100644 board/bosch/guardian/MAINTAINERS
>  create mode 100644 board/bosch/guardian/Makefile
>  create mode 100644 board/bosch/guardian/board.c
>  create mode 100644 board/bosch/guardian/board.h
>  create mode 100644 board/bosch/guardian/mux.c
>  create mode 100644 configs/am335x_guardian_defconfig
>  create mode 100644 include/configs/am335x_guardian.h
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 455f06cfee..d2b3dce2da 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1524,6 +1524,7 @@ source "arch/arm/cpu/armv8/Kconfig"
>  source "arch/arm/mach-imx/Kconfig"
>  
>  source "board/bosch/shc/Kconfig"
> +source "board/bosch/guardian/Kconfig"
>  source "board/CarMediaLab/flea3/Kconfig"
>  source "board/Marvell/aspenite/Kconfig"
>  source "board/Marvell/gplugd/Kconfig"

[...]

> diff --git a/arch/arm/dts/am335x-guardian.dts 
> b/arch/arm/dts/am335x-guardian.dts
> new file mode 100644
> index 00..d8652ab014
> --- /dev/null
> +++ b/arch/arm/dts/am335x-guardian.dts
> @@ -0,0 +1,511 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
> + * Copyright (C) 2018 Robert Bosch Power Tools GmbH
> + */
> +/dts-v1/;
> +
> +#include "am33xx.dtsi"
> +#include 
> +#include 
> +
> +/ {
> + model = "Bosch AM335x Guardian";
> + compatible = "bosch,am335x-guardian", "ti,am33xx";
> +
> + chosen {
> + stdout-path = &uart0;
> + tick-timer = &timer2;
> + };
> +
> + cpus {
> + cpu@0 {
> + cpu0-supply = <&dcdc2_reg>;
> + };
> + };
> +
> + memory@8000 {
> + device_type = "memory";
> + reg = <0x8000 0x1000>; /* 256 MB */
> + };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> + #address-cells = <1>;
> + #size-cells = <0>;
The properties #address-cells and #size-cells can be removed as the only
child node does not have a reg property.

> + pinctrl-names = "default";
> + pinctrl-0 = <&gpio_keys_pins>;
> +
> + button21 {
> + label = "guardian-power-button";
> + linux,code = ;
> + gpios = <&gpio2 21 0>;
> + wakeup-source;
> + };
> + };
> +
> + leds {
> + compatible = "gpio-leds";
> + pinctrl-names = "default";
> + pinctrl-0 = <&leds_pins>;
> +
> + led1 {
> + label = "green:heartbeat";
> + gpios = <&gpio1 27 GPIO_ACTIVE_HIGH>;
> + linux,default-trigger = "heartbeat";
> + default-state = "off";
> + };
> +
> + led2 {
> + label = "green:mmc0";
> + gpios = <&gpio1 26 GPIO_ACTIVE_HIGH>;
> + linux,default-trigger = "mmc0";
> + default-state = "off";
> + };
> + };
> +

[...]

> diff --git a/board/bosch/guardian/board.c b/board/bosch/guardian/board.c
> new file mode 100644
> index 00..7f6f500535
> --- /dev/null
> +++ b/board/bosch/guardian/board.c
> @@ -0,0 +1,186 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * board.c
> + *
> + * Board functions for Bosch Guardian
> + *
> + * Copyright (C) 2011, Texas Instruments, Incorporated - http://www.ti.com/
> + * Copyright (C) 2018 Robert Bosch Power Tools GmbH
> + */
> +
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#inclu

Re: [U-Boot] [PATCH v2 3/3] am335x, guardian: Add support for the bosch guardian board

2019-02-22 Thread Tom Rini
On Fri, Feb 22, 2019 at 05:52:54PM +, Martyn Welch wrote:

> From: Sjoerd Simons 
> 
> Add support for the Bosch Guardian board.
> 
> CPU  : AM335X-GP rev 2.1
> Model: Bosch AM335x Guardian
> I2C:   ready
> DRAM:  256 MiB
> NAND:  512 MiB
> MMC:   OMAP SD/MMC: 0
> 
> Signed-off-by: Sjoerd Simons 
> Signed-off-by: Martyn Welch 
> 

Reviewed-by: Tom Rini 

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot