Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 3/4] configs: bpi-m64: Use Mainline ARM ATF
Date: Wed, 5 Sep 2018 23:48:37 +0200	[thread overview]
Message-ID: <20180905234837.395a8e2d@windsurf> (raw)
In-Reply-To: <5ab06b35-1627-5669-6fc8-1f8ecbedabb7@arm.com>

Hello,

On Mon, 3 Sep 2018 10:06:40 +0100, Andre Przywara wrote:
> Hi,
> 
> On 03/09/18 09:26, Jagan Teki wrote:
> > Use Mainline ARM ATF for bananapi-m64 board instead
> > Andre's github.  
> 
> Have you tested it? The mainline ATF is not feature-complete yet, so in
> comparison to the old port you are missing the PMIC setup. This breaks
> power off and, more importantly, Ethernet support in U-Boot. I have some
> code to introduce proper (even better) PMIC support, just didn't come
> around to clean it up and post it.
> Meanwhile I would suggest to stick to the old port - actually this was
> the reason I didn't advertise this more widely yet.

So I guess this comment also applied to the Orangepi One Plus patch
(PATCH 2/4) in this series, since it also uses mainline ATF:

+# 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/ARM-software/arm-trusted-firmware.git"
+BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="sun50i_h6"
+BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_REPO_VERSION="cffb00342897ed73a2bb9fc7a1860102bd927f2e"
+BR2_TARGET_ARM_TRUSTED_FIRMWARE_BL31=y

I've marked patches 2 to 4 in this series as Changes Requested.

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

  parent reply	other threads:[~2018-09-05 21:48 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-03  8:26 [Buildroot] [PATCH 1/4] DEVELOPERS: Fix file order list maintained by me Jagan Teki
2018-09-03  8:26 ` [Buildroot] [PATCH 2/4] board: orangepi: Add Orangepi One Plus support Jagan Teki
2018-09-03  8:26 ` [Buildroot] [PATCH 3/4] configs: bpi-m64: Use Mainline ARM ATF Jagan Teki
2018-09-03  9:06   ` Andre Przywara
2018-09-03 10:37     ` Jagan Teki
2018-09-05 21:48     ` Thomas Petazzoni [this message]
2018-09-03  8:26 ` [Buildroot] [PATCH 4/4] configs: bpi-m64: bump linux, uboot Jagan Teki
2018-09-03 14:43   ` Luca Ceresoli
2018-09-04  8:21     ` Thomas Petazzoni
2018-09-05 21:46 ` [Buildroot] [PATCH 1/4] DEVELOPERS: Fix file order list maintained by me Thomas Petazzoni
2018-09-06 15:48   ` Jagan Teki
2018-09-06 15:51     ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180905234837.395a8e2d@windsurf \
    --to=thomas.petazzoni@bootlin.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox