[U-Boot] [PATCH 12/12 v2] arm: at91: Add gardena-gateway-at91sam support

[Eugen.Hristev at microchip.com]

On 26.03.2019 16:01, Stefan Roese wrote:

> 
> On 26.03.19 14:58, Eugen.Hristev at microchip.com wrote:
>>
>>
>> On 26.03.2019 14:16, Stefan Roese wrote:
>>
>>> The GARDENA smart Gateway boards are equipped with an Atmel / Microchip
>>> AT91SAM9G25 SoC and with 128 MiB of RAM and 256 MiB of NAND storage.
>>> This patch adds support for this board including SPL support. Therefore
>>> the AT91Boostrap is not needed on this platform any more.
>>>
>>> Signed-off-by: Stefan Roese <sr at denx.de>
>>> Cc: Heiko Schocher <hs at denx.de>
>>> Cc: Andreas Bießmann <andreas at biessmann.org>
>>> Cc: Eugen Hristev <eugen.hristev at microchip.com>
>>> ---
>>> v2:
>>> - Add dtb build target in dts Makefile
>>> - Add gardena-smart-gateway-at91sam-u-boot.dtsi file with U-Boot
>>>     specific DT properties
>>> - at91sam9x5ek.dtsi is not included in the dts file any more as this
>>>     did enable too much unused devices
>>>
>>>    arch/arm/dts/Makefile                         |   3 +
>>>    .../gardena-smart-gateway-at91sam-u-boot.dtsi |   5 +
>>>    .../arm/dts/gardena-smart-gateway-at91sam.dts | 119 +++++++++++++++
>>>    arch/arm/mach-at91/Kconfig                    |   8 ++
>>>    board/gardena/smart-gateway-at91sam/Kconfig   |  12 ++
>>>    .../gardena/smart-gateway-at91sam/MAINTAINERS |   7 +
>>>    board/gardena/smart-gateway-at91sam/Makefile  |   7 +
>>>    board/gardena/smart-gateway-at91sam/board.c   |  59 ++++++++
>>>    board/gardena/smart-gateway-at91sam/spl.c     | 135 
>>> ++++++++++++++++++
>>>    .../gardena-smart-gateway-at91sam_defconfig   |  84 +++++++++++
>>>    .../configs/gardena-smart-gateway-at91sam.h   |  88 ++++++++++++
>>>    11 files changed, 527 insertions(+)
>>>    create mode 100644 
>>> arch/arm/dts/gardena-smart-gateway-at91sam-u-boot.dtsi
>>>    create mode 100644 arch/arm/dts/gardena-smart-gateway-at91sam.dts
>>>    create mode 100644 board/gardena/smart-gateway-at91sam/Kconfig
>>>    create mode 100644 board/gardena/smart-gateway-at91sam/MAINTAINERS
>>>    create mode 100644 board/gardena/smart-gateway-at91sam/Makefile
>>>    create mode 100644 board/gardena/smart-gateway-at91sam/board.c
>>>    create mode 100644 board/gardena/smart-gateway-at91sam/spl.c
>>>    create mode 100644 configs/gardena-smart-gateway-at91sam_defconfig
>>>    create mode 100644 include/configs/gardena-smart-gateway-at91sam.h
>>>
>>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
>>> index 0e2ffdb87f..e890492456 100644
>>> --- a/arch/arm/dts/Makefile
>>> +++ b/arch/arm/dts/Makefile
>>> @@ -607,6 +607,9 @@ dtb-$(CONFIG_TARGET_AT91SAM9X5EK) += \
>>>    dtb-$(CONFIG_TARGET_AT91SAM9N12EK) += at91sam9n12ek.dtb
>>> +dtb-$(CONFIG_TARGET_GARDENA_SMART_GATWAY_AT91SAM) += \
>>
>> Hi Stefan,
>>
>> Is the "GATWAY" intentional? would be difficult to find if someone
>> searches for it...
> 
> Ups. Definitely not! Thanks for catching. I'll fix it in the next
> version.
> 
> I also thought about adding "at91sam9g25-" to the beginning of the
> dts file to better include it in the dts directory to the SoC
> family. What is your thinking about this?
> 

I do not have a strong opinion. As you like.

> Thanks,
> Stefan