[Buildroot] [EXT] Re: [PATCH v4 5/9] boot/arm-trusted-firmware: introduce two new options

[Buildroot] [EXT] Re: [PATCH v4 5/9] boot/arm-trusted-firmware: introduce two new options

[Gervais, Francois]

> > two new options for arm-trusted-firmware:
> > 1. BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_TARGETS
> >?????? - for additional targets, E.G. "pbl".
> 
> This option was already added in commit
> 05b8c7da6de863561d66717735a4870f2766454c, which is part of Buildroot
> master.

Somebody reported a problem with my patch last week. I'm still investigating with
him but it seems that the qstrip wrapping might be required which
05b8c7da6de863561d66717735a4870f2766454c is missing.

Jerry maybe you can try on your side? You could rebase your v5 on top
of master and try to build arm-trusted-firmware from scratch with 
BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_TARGETS empty.

You might see this error message:

make: *** empty string invalid as file name.  Stop.

[Buildroot] [EXT] Re: [PATCH v4 5/9] boot/arm-trusted-firmware: introduce two new options

[Thomas Petazzoni]

On Mon, 10 Feb 2020 14:30:13 +0000
"Gervais, Francois" <FGervais@distech-controls.com> wrote:

> Somebody reported a problem with my patch last week. I'm still investigating with
> him but it seems that the qstrip wrapping might be required which
> 05b8c7da6de863561d66717735a4870f2766454c is missing.

Indeed, a $(call qstrip,...) is missing, it should be added.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [EXT] Re: [PATCH v4 5/9] boot/arm-trusted-firmware: introduce two new options

[Jerry Huang]

Best Regards
Jerry Huang

> -----Original Message-----
> From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Sent: Wednesday, February 5, 2020 8:29 PM
> To: Jerry Huang <jerry.huang@nxp.com>
> Cc: buildroot at busybox.net; olteanv at gmail.com; michael at walle.cc;
> geomatsi at gmail.com; matthew.weber at collins.com
> Subject: [EXT] Re: [Buildroot] [PATCH v4 5/9] boot/arm-trusted-firmware:
> introduce two new options
> 
> Caution: EXT Email
> 
> Hello,
> 
> On Wed,  5 Feb 2020 18:52:23 +0800
> Changming Huang <jerry.huang@nxp.com> wrote:
> 
> > two new options for arm-trusted-firmware:
> > 1. BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_TARGETS
> >       - for additional targets, E.G. "pbl".
> 
> This option was already added in commit
> 05b8c7da6de863561d66717735a4870f2766454c, which is part of Buildroot
> master.
> 
> > 2. BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_DEPENDENCIES
> >       - for additional dependencies, E.G. "host-qoriq-rcw"
> >
> > Signed-off-by: Changming Huang <jerry.huang@nxp.com>
> 
> No need to resend just for that, though, it can be fixed when applying.
Sure, I will remove this options from next version.

> Best regards,
> 
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbootlin.c
> om&amp;data=02%7C01%7Cjerry.huang%40nxp.com%7Cc92f05de22194ab439
> ff08d7aa3709bc%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6371
> 65025674223306&amp;sdata=wsI3e76EHN6NNtKknPiZpBnWAn1FyALHUgbjp4
> LKAdw%3D&amp;reserved=0