Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] Raspberry PI 3 kernel, firmware and dtb update
Date: Tue, 19 Apr 2016 23:33:21 +0200	[thread overview]
Message-ID: <20160419213321.GG3505@free.fr> (raw)
In-Reply-To: <87a8kpe39z.fsf@dell.be.48ers.dk>

Ubaldo, Peter, All,

On 2016-04-19 22:45 +0200, Peter Korsgaard spake thusly:
> >>>>> "Ubaldo" == Ubaldo Porcheddu <ubaldo@eja.it> writes:
> 
>  > Signed-off-by: Ubaldo Porcheddu <ubaldo@eja.it>
>  > ---
>  >  board/raspberrypi/genimage-raspberrypi2.cfg | 1 +
>  >  configs/raspberrypi2_defconfig              | 4 ++--
>  >  package/rpi-firmware/rpi-firmware.mk        | 2 +-
>  >  3 files changed, 4 insertions(+), 3 deletions(-)
> 
> Thanks for the patch, but can you instead split it up 2 seperate
> patches (with sensible descriptions) - One bumping the rpi-firmware
> version and one updating the rpi2 config to also support rpi3.
> 
> The rpi3 afaik uses a different serial port than rpi2. How will that
> work if we use the same defconfig for both boards? Perhaps it makes more
> sense to have a seperate raspberrypi3_defconfig?

Yes, expecially since the CPU is not the same, andthat we may^Wwill one
day have it run in 64-bit mode (one can dream, no?).

So yes, make it a separate defconfig, please.

Regards,
Yann E. MORIN.

>  > diff --git a/board/raspberrypi/genimage-raspberrypi2.cfg b/board/raspberrypi/genimage-raspberrypi2.cfg
>  > index 443c821..453fb00 100644
>  > --- a/board/raspberrypi/genimage-raspberrypi2.cfg
>  > +++ b/board/raspberrypi/genimage-raspberrypi2.cfg
>  > @@ -2,6 +2,7 @@ image boot.vfat {
>  >    vfat {
>  >      files = {
>  >        "bcm2709-rpi-2-b.dtb",
>  > +      "bcm2710-rpi-3-b.dtb",
>  >        "rpi-firmware/bootcode.bin",
>  >        "rpi-firmware/cmdline.txt",
>  >        "rpi-firmware/config.txt",
>  > diff --git a/configs/raspberrypi2_defconfig b/configs/raspberrypi2_defconfig
>  > index ce41ee3..b77db14 100644
>  > --- a/configs/raspberrypi2_defconfig
>  > +++ b/configs/raspberrypi2_defconfig
>  > @@ -13,13 +13,13 @@ BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_1=y
>  >  BR2_LINUX_KERNEL=y
>  >  BR2_LINUX_KERNEL_CUSTOM_GIT=y
>  >  BR2_LINUX_KERNEL_CUSTOM_REPO_URL="https://github.com/raspberrypi/linux.git"
>  > -BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="d33d0293e245badc4ca6ede3984d8bb8ea63cb1a"
>  > +BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="20fe468af4bb40fec0f81753da4b20a8bfc259c9"
>  >  BR2_LINUX_KERNEL_DEFCONFIG="bcm2709"
>  >  BR2_LINUX_KERNEL_ZIMAGE=y
>  
>  >  # Build the DTB from the kernel sources
>  >  BR2_LINUX_KERNEL_DTS_SUPPORT=y
>  > -BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2709-rpi-2-b"
>  > +BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2709-rpi-2-b bcm2710-rpi-3-b"
>  
>  >  BR2_PACKAGE_RPI_FIRMWARE=y
>  >  # BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTB_OVERLAYS is not set
>  > diff --git a/package/rpi-firmware/rpi-firmware.mk b/package/rpi-firmware/rpi-firmware.mk
>  > index 291da3a..8476678 100644
>  > --- a/package/rpi-firmware/rpi-firmware.mk
>  > +++ b/package/rpi-firmware/rpi-firmware.mk
>  > @@ -4,7 +4,7 @@
>  >  #
>  >  ################################################################################
>  
>  > -RPI_FIRMWARE_VERSION = 8b4e5482b52e6fb438dddc0d88ba0ba8d44af54b
>  > +RPI_FIRMWARE_VERSION = 951799bbcd795ddf27769d14acf4813fdcbe53dc
>  >  RPI_FIRMWARE_SITE = $(call github,raspberrypi,firmware,$(RPI_FIRMWARE_VERSION))
>  >  RPI_FIRMWARE_LICENSE = BSD-3c
>  >  RPI_FIRMWARE_LICENSE_FILES = boot/LICENCE.broadcom
>  > -- 
>  > 1.9.1
> 
>  > _______________________________________________
>  > buildroot mailing list
>  > buildroot at busybox.net
>  > http://lists.busybox.net/mailman/listinfo/buildroot
> 
> 
> -- 
> Bye, Peter Korsgaard
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2016-04-19 21:33 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-31 10:51 [Buildroot] [PATCH 1/1] Raspberry PI 3 kernel, firmware and dtb update Ubaldo Porcheddu
2016-04-19 20:45 ` Peter Korsgaard
2016-04-19 21:33   ` Yann E. MORIN [this message]
2016-04-20  7:08     ` Peter Korsgaard
2016-04-20  8:37       ` Ubaldo Porcheddu
2016-04-20 14:25       ` Gustavo Zacarias
2016-04-23  8:36         ` Peter Korsgaard
2016-04-23 16:53       ` Yann E. MORIN

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=20160419213321.GG3505@free.fr \
    --to=yann.morin.1998@free.fr \
    --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