From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Thu, 13 Feb 2020 10:40:13 -0700
Subject: [BUG] binman: Add a library to access binman entries
In-Reply-To: <08CDF651-9FC4-4D4A-9E61-10174C2A1886@public-files.de>
References: <20191207044315.51770-1-sjg@chromium.org>
 <20191206213936.v6.1.I8c03ae7027508705f6f535ba5137b5e6c5dea840@changeid>
 <trinity-d20dc487-67de-4677-976b-cfbe1a13e46e-1579708143145@3c-app-gmx-bap41>
 <trinity-d299106e-0083-466e-bb69-52242da63a08-1579889748716@3c-app-gmx-bap68>
 <CAPnjgZ3YYv9OwE=Q=O6y=zHpOGdL-G5JWm=8A5n4CVsKu3h_Zg@mail.gmail.com>
 <3D6666EE-43E9-49C4-AC18-CFDBEE53B2D7@public-files.de>
 <CAPnjgZ0UYp5wvZds=jzzW0D7mVjtP9gbP1O6dTFgRTy9mJ_1qA@mail.gmail.com>
 <08CDF651-9FC4-4D4A-9E61-10174C2A1886@public-files.de>
Message-ID: <CAPnjgZ1zWOCsP83BR5YhhDBcmO8oM9QOKdXCKYjaGgo9k87zJg@mail.gmail.com>
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

Hi Frank,

On Wed, 12 Feb 2020 at 11:02, Frank Wunderlich <frank-w@public-files.de> wrote:
>
> Hi
>
> Why not just disable binman_fdt (or not default y)? Your way sounds more complex. As i do not understand it and see no benefit for this board, i would leave it disabled. This leave time for a thoughtful solution
> Maybe mtk knows a better way...
>
> Sunxi sounds wrong because board is not with allwinner soc. Maybe a more generic name as target for boards not yet supported by binman_fdt?

Oh so now I am confused. What U-Boot board name are you referring to?
I assumed it was bananapi_m64.

Regards,
Simon


>
> Regards Frank
>
> Am 12. Februar 2020 18:14:29 MEZ schrieb Simon Glass <sjg@chromium.org>:
> >
> >This code in the Makefile should do the same thing for ARM64 and 32:
> >
> >ifneq ($(CONFIG_ARCH_SUNXI),)
> >ifeq ($(CONFIG_ARM64),)
> >u-boot-sunxi-with-spl.bin: spl/sunxi-spl.bin u-boot.img u-boot.dtb
> >FORCE
> >$(call if_changed,binman)
> >else
> >u-boot-sunxi-with-spl.bin: spl/sunxi-spl.bin u-boot.itb FORCE
> >$(call if_changed,cat)
> >endif
> >endif
> >
> >To make this work I think you'll need to add a new 'sunxi-itb' entry
> >type into binman as a first step. That is ugly but it will work.
> >
> >Then mksunxi_fit_atf.sh should move into binman, with 'sunxi-itb'
> >becoming a new method that generates the .its from source. I can help
> >with that bit.
> >
> >Regards,
> >Simon