[U-Boot] [PATCH v4 2/9] arm: imx6ul: Add Engicam Is.IoT MX6UL Starter Kit initial support

[Tom Rini]

On Sat, Jan 28, 2017 at 01:25:05PM +0100, Stefano Babic wrote:
> Hi Jagan, Tom,
> 
> On 27/01/2017 16:55, Jagan Teki wrote:
> > n Fri, Jan 27, 2017 at 1:38 PM, Stefano Babic <sbabic at denx.de> wrote:
> >> Hi Jagan,
> >>
> >> On 27/01/2017 09:54, Jagan Teki wrote:
> >>> From: Jagan Teki <jagan at amarulasolutions.com>
> >>>
> >>> Boot from MMC:
> >>> -------------
> >>> U-Boot SPL 2017.01-rc2-gba3c151-dirty (Jan 02 2017 - 16:59:33)
> >>> Trying to boot from MMC1
> >>>
> >>> U-Boot 2017.01-rc2-gba3c151-dirty (Jan 02 2017 - 16:59:33 +0100)
> >>>
> >>> CPU:   Freescale i.MX6UL rev1.1 528 MHz (running at 396 MHz)
> >>> CPU:   Industrial temperature grade (-40C to 105C) at 33C
> >>> Reset cause: POR
> >>> Model: Engicam Is.IoT MX6UL Starterkit
> >>> DRAM:  512 MiB
> >>> MMC:   FSL_SDHC: 0
> >>> *** Warning - bad CRC, using default environment
> >>>
> >>> In:    serial
> >>> Out:   serial
> >>> Err:   serial
> >>> Net:   CPU Net Initialization Failed
> >>> No ethernet found.
> >>> Hit any key to stop autoboot:  0
> >>> isiotmx6ul>
> >>>
> >>> Cc: Stefano Babic <sbabic at denx.de>
> >>> Cc: Matteo Lisi <matteo.lisi at engicam.com>
> >>> Cc: Michael Trimarchi <michael at amarulasolutions.com>
> >>> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
> >>> ---
> >>>  arch/arm/cpu/armv7/mx6/Kconfig        |  11 ++
> >>>  arch/arm/dts/Makefile                 |   3 +-
> >>>  arch/arm/dts/imx6ul-isiot-mmc.dts     |  50 +++++++
> >>>  arch/arm/dts/imx6ul-isiot.dtsi        |  92 +++++++++++++
> >>>  board/engicam/isiotmx6ul/Kconfig      |  12 ++
> >>>  board/engicam/isiotmx6ul/MAINTAINERS  |   6 +
> >>>  board/engicam/isiotmx6ul/Makefile     |   6 +
> >>>  board/engicam/isiotmx6ul/README       |  28 ++++
> >>>  board/engicam/isiotmx6ul/isiotmx6ul.c | 247 ++++++++++++++++++++++++++++++++++
> >>>  configs/imx6ul_isiot_mmc_defconfig    |  36 +++++
> >>>  include/configs/imx6ul_isiot.h        | 142 +++++++++++++++++++
> >>>  11 files changed, 632 insertions(+), 1 deletion(-)
> >>>  create mode 100644 arch/arm/dts/imx6ul-isiot-mmc.dts
> >>>  create mode 100644 arch/arm/dts/imx6ul-isiot.dtsi
> >>>  create mode 100644 board/engicam/isiotmx6ul/Kconfig
> >>>  create mode 100644 board/engicam/isiotmx6ul/MAINTAINERS
> >>>  create mode 100644 board/engicam/isiotmx6ul/Makefile
> >>>  create mode 100644 board/engicam/isiotmx6ul/README
> >>>  create mode 100644 board/engicam/isiotmx6ul/isiotmx6ul.c
> >>>  create mode 100644 configs/imx6ul_isiot_mmc_defconfig
> >>>  create mode 100644 include/configs/imx6ul_isiot.h
> >>>
> >>> diff --git a/arch/arm/cpu/armv7/mx6/Kconfig b/arch/arm/cpu/armv7/mx6/Kconfig
> >>> index c646966..72780d7 100644
> >>> --- a/arch/arm/cpu/armv7/mx6/Kconfig
> >>> +++ b/arch/arm/cpu/armv7/mx6/Kconfig
> >>> @@ -205,6 +205,16 @@ config TARGET_MX6UL_GEAM
> >>>       select DM_THERMAL
> >>>       select SUPPORT_SPL
> >>>
> >>> +config TARGET_MX6UL_ISIOT
> >>> +     bool "Support Engicam Is.IoT MX6UL"
> >>> +     select MX6UL
> >>> +     select OF_CONTROL
> >>> +     select DM
> >>> +     select DM_GPIO
> >>> +     select DM_MMC
> >>> +     select DM_THERMAL
> >>> +     select SUPPORT_SPL
> >>> +
> >>>  config TARGET_MX6ULL_14X14_EVK
> >>>       bool "Support mx6ull_14x14_evk"
> >>>       select MX6ULL
> >>> @@ -313,6 +323,7 @@ source "board/embest/mx6boards/Kconfig"
> >>>  source "board/engicam/geam6ul/Kconfig"
> >>>  source "board/engicam/icorem6/Kconfig"
> >>>  source "board/engicam/icorem6_rqs/Kconfig"
> >>> +source "board/engicam/isiotmx6ul/Kconfig"
> >>>  source "board/freescale/mx6qarm2/Kconfig"
> >>>  source "board/freescale/mx6qsabreauto/Kconfig"
> >>>  source "board/freescale/mx6sabresd/Kconfig"
> >>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> >>> index 6a7924e..e83308c 100644
> >>> --- a/arch/arm/dts/Makefile
> >>> +++ b/arch/arm/dts/Makefile
> >>> @@ -313,7 +313,8 @@ dtb-$(CONFIG_MX6) += imx6ull-14x14-evk.dtb \
> >>>       imx6dl-icore-rqs.dtb \
> >>>       imx6q-icore.dtb \
> >>>       imx6q-icore-rqs.dtb \
> >>> -     imx6ul-geam-kit.dtb
> >>> +     imx6ul-geam-kit.dtb \
> >>> +     imx6ul-isiot-mmc.dtb
> >>>
> >>>  dtb-$(CONFIG_MX7) += imx7-colibri.dtb
> >>>
> >>> diff --git a/arch/arm/dts/imx6ul-isiot-mmc.dts b/arch/arm/dts/imx6ul-isiot-mmc.dts
> >>> new file mode 100644
> >>> index 0000000..bb5086a
> >>> --- /dev/null
> >>> +++ b/arch/arm/dts/imx6ul-isiot-mmc.dts
> >>> @@ -0,0 +1,50 @@
> >>> +/*
> >>> + * Copyright (C) 2016 Amarula Solutions B.V.
> >>> + * Copyright (C) 2016 Engicam S.r.l.
> >>> + *
> >>> + * This file is dual-licensed: you can use it either under the terms
> >>> + * of the GPL or the X11 license, at your option. Note that this dual
> >>> + * licensing only applies to this file, and not this project as a
> >>> + * whole.
> >>> + *
> >>
> >>
> >> I have seen that a lot of files slipped into mainline, and I have myself
> >> not paid enough attention befeore (see your imx6ul-geam-kit.dts for
> >> example).
> >>
> >> But we moved some years ago to SPDX-License-Identifier. Other DTS files
> >> are using this as well without copying the whole license. Could you do
> >> this here, too ? It would be also nice if you plan to fix this for all
> >> Engicam's boards, thanks !
> > 
> > Yes for boards, I will prepare patch.
> > 
> > For devicetress, since the base dtsi files copied from Linux, we keep
> > the license as it is. I am not sure all other dts(i) files use the
> > SPDX-License-Identifier. example imx6sll-evk, imx6ull-14x14-evk and
> > all sunxi .
> 
> It is true that some files slipped into mainline copying again all
> license stuff - and several files ported from Linux was adjusting for SPDX.
> 
> Tom, what do you think ?

The DT files we copy from the kernel are the exception to the rule, they
should just keep the usual full license header they come in with.  This
is even "easier" to do now, given that we can append any U-Boot centric
bits into a -uboot.dtsi file and keep the upstream DT files unmodified
entirely.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170128/eddf6801/attachment.sig>