From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jagan Teki <jagan@amarulasolutions.com>
Date: Sat, 2 Dec 2017 20:40:13 +0530
Subject: [Buildroot] [PATCH v5 03/11] board: Add Bananapi M64 support
In-Reply-To: <20171202155818.0d04f798@windsurf.lan>
References: <1509559553-26544-1-git-send-email-jagan@amarulasolutions.com>
 <1509559553-26544-4-git-send-email-jagan@amarulasolutions.com>
 <20171202155818.0d04f798@windsurf.lan>
Message-ID: <CAMty3ZDAHW7OKSfrJXgLLncpOj8H_XCVDeBFSWaXtVqh5tGEWQ@mail.gmail.com>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

+ Andre

On Sat, Dec 2, 2017 at 8:28 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Wed,  1 Nov 2017 19:05:45 +0100, Jagan Teki wrote:
>
>> +# Firmware
>> +BR2_TARGET_ARM_TRUSTED_FIRMWARE=y
>> +BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_GIT=y
>> +BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_REPO_URL="https://github.com/apritzel/arm-trusted-firmware.git"
>> +BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="sun50iw1p1"
>> +BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_REPO_VERSION="aa75c8da415158a94b82a430b2b40000778e851f"
>> +BR2_TARGET_ARM_TRUSTED_FIRMWARE_ADDITIONAL_VARIABLES="bl31"
>
> What is this "bl31" additional variable doing?

Source build varies based on the exception levels switches, like bl31
we have bl31, bl2, bl1

>
> Aren't you hijacking the ADDITIONAL_VARIABLES thing to call the bl31
> make target ?

Since ADDITIONAL_VARIABLES resembles to use additional parameters for
ATF build, I've treated bl31 as additional variable for this ATF

thanks!
-- 
Jagan Teki
Senior Linux Kernel Engineer | Amarula Solutions
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.