mbox series

[v2,0/5] ARM: dts: tq imx7 evaluation board

Message ID 20190403064427.30041-1-bruno.thomsen@gmail.com
Headers show
Series ARM: dts: tq imx7 evaluation board | expand

Message

Bruno Thomsen April 3, 2019, 6:44 a.m. UTC 
Thanks for all the review comments Shawn Guo, Rob Herring and
Clemens Koller. All issues that have been pointed out are fixed in v2.
Patch 2 and 3 has been merged as they both modify the new fsl.yaml.

This patch series is an effort to get mainline kernel support for TQ
boards with i.MX7 SoCs and is based on the vendor BSP. Products based
on these TQMa7 modules such as smart grid infrastructure will be sold
for many years with an expected field lifetime of up to two decades.
During this long lifetime they need to receive security patches and
new in-kernel security features. The typical device of this type does
not use audio and video, so it has not been a focus area. This however
could be worked on in the future if someone desire that functionality.

The following device trees has been tested on a 4.19 mainline kernel
together with U-Boot 2019.01.

Working/tested:
USB
Ethernet
SD
Console
Buttons

Not working/tested:
Sound
Video

No changes to MAINTAINERS are needed IMHO.

Bruno Thomsen (5):
  dt-bindings: add vendor prefix for TQ Systems GmbH
  dt-bindings: arm: add TQ boards
  ARM: dts: tq imx7 common board support
  ARM: dts: tq imx7s board support
  ARM: dts: tq imx7d board support

 .../devicetree/bindings/arm/fsl.yaml          |   7 +
 .../devicetree/bindings/vendor-prefixes.txt   |   1 +
 arch/arm/boot/dts/Makefile                    |   2 +
 arch/arm/boot/dts/imx7-mba7.dtsi              | 563 ++++++++++++++++++
 arch/arm/boot/dts/imx7-tqma7.dtsi             | 233 ++++++++
 arch/arm/boot/dts/imx7d-mba7.dts              | 119 ++++
 arch/arm/boot/dts/imx7d-tqma7.dtsi            |  11 +
 arch/arm/boot/dts/imx7s-mba7.dts              |  18 +
 arch/arm/boot/dts/imx7s-tqma7.dtsi            |  11 +
 9 files changed, 965 insertions(+)
 create mode 100644 arch/arm/boot/dts/imx7-mba7.dtsi
 create mode 100644 arch/arm/boot/dts/imx7-tqma7.dtsi
 create mode 100644 arch/arm/boot/dts/imx7d-mba7.dts
 create mode 100644 arch/arm/boot/dts/imx7d-tqma7.dtsi
 create mode 100644 arch/arm/boot/dts/imx7s-mba7.dts
 create mode 100644 arch/arm/boot/dts/imx7s-tqma7.dtsi

Comments

Shawn Guo April 11, 2019, 1:21 a.m. UTC | #1 
On Wed, Apr 03, 2019 at 08:44:22AM +0200, Bruno Thomsen wrote:
> Bruno Thomsen (5):
>   dt-bindings: add vendor prefix for TQ Systems GmbH
>   dt-bindings: arm: add TQ boards
>   ARM: dts: tq imx7 common board support
>   ARM: dts: tq imx7s board support
>   ARM: dts: tq imx7d board support

Applied all with a small fix-up on patch #3 (will reply to it).

Shawn
Shawn Guo April 11, 2019, 1:24 a.m. UTC | #2 
On Wed, Apr 03, 2019 at 08:44:25AM +0200, Bruno Thomsen wrote:
> diff --git a/arch/arm/boot/dts/imx7-mba7.dtsi b/arch/arm/boot/dts/imx7-mba7.dtsi
> new file mode 100644
> index 000000000000..a880a698bdaf
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx7-mba7.dtsi
> @@ -0,0 +1,563 @@
> +// SPDX-License-Identifier: GPL-2.0 OR X11
> +/*
> + * Device Tree Include file for TQ Systems MBa7 carrier board.
> + *
> + * Copyright (C) 2016 TQ Systems GmbH
> + * Author: Markus Niebel <Markus.Niebel@tq-group.com>
> + * Copyright (C) 2019 Bruno Thomsen <bruno.thomsen@gmail.com>
> + *
> + * Note: This file does not include nodes for all peripheral devices.
> + * As device driver coverage increases additional nodes can be added.
> + */
> +
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/net/ti-dp83867.h>
> +
> +/ {
> +	beeper {
> +		compatible = "gpio-beeper";
> +		gpios = <&pca9555 0 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	chosen {
> +		bootargs="ttymxc5,115200";

I do not think this is really needed.

> +		linux,stdout-path = &uart6;

The property is deprecated and unnecessary since we have stdout-path
just below.

I dropped these two lines when applying.

Shawn

> +		stdout-path = &uart6;
> +	};