From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@konsulko.com>
Date: Mon, 22 Jan 2018 10:15:43 -0500
Subject: [U-Boot] [PATCH v2] arm: add support for PDU001
In-Reply-To: <1516619269-27243-1-git-send-email-fb@ltec.ch>
References: <1516619269-27243-1-git-send-email-fb@ltec.ch>
Message-ID: <20180122151543.GG32220@bill-the-cat>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Mon, Jan 22, 2018 at 12:07:49PM +0100, Felix Brack wrote:

> This patch adds support for the PDU001 board.
> 
> Signed-off-by: Felix Brack <fb@ltec.ch>
[snip]
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index f6d57f5..15349b3 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1339,6 +1339,7 @@ source "board/vscom/baltos/Kconfig"
>  source "board/woodburn/Kconfig"
>  source "board/work-microwave/work_92105/Kconfig"
>  source "board/zipitz2/Kconfig"
> +source "board/eets/pdu001/Kconfig"

This should stay alphabetized.

[snip]
> new file mode 100644
> index 0000000..0967e82
> --- /dev/null
> +++ b/arch/arm/dts/am335x-pdu001.dts

So, have you pushed this platform upstream to Linux?

> @@ -0,0 +1,612 @@
> +/*
> + * pdu001.dts
> + *
> + * EETS GmbH PDU001 board device tree file
> + *
> + * Copyright (C) 2018 EETS GmbH - http://www.eets.ch/
> + *
> + * Copyright (C) 2011, Texas Instruments, Incorporated - http://www.ti.com/
> + *
> + * SPDX-License-Identifier:  GPL-2.0-or-later

This is not a free-form tag.  Here and elsewhere you want "GPL-2.0+".

[snip]
> +	ocp {
> +		u-boot,dm-pre-reloc;
> +	};
> +};
> +
> +&l4_wkup {
> +	u-boot,dm-pre-reloc;
> +};
> +
> +&scm {
> +	u-boot,dm-pre-reloc;
> +};
> +

For long term maintenance you'll want to move these U-Boot specific
parts to a am335x-pdu001-u-boot.dtsi file so that you can re-sync the
main DTS file with Linux and not have any important U-Boot changes be
dropped.

Thanks!

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