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

[Stefan Roese]

On 26.03.19 15:12, Eugen.Hristev at microchip.com wrote:
> 
> 
> 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.

I've changed it locally and will send v3 in a few days (waiting for some
further review comments).

Thanks,
Stefan