diff mbox series

[1/2] roms/u-boot: Bump ppce500 u-boot to v2021.07 to add eTSEC support

Message ID 20210706043124.1871866-1-bmeng.cn@gmail.com
State New
Headers show
Series [1/2] roms/u-boot: Bump ppce500 u-boot to v2021.07 to add eTSEC support | expand

Commit Message

Bin Meng July 6, 2021, 4:31 a.m. UTC
Update the QEMU shipped u-boot.e500 image built from U-Boot mainline
v2021.07 release, which added eTSEC support to the QEMU ppce500 target,
via the following U-Boot series:

  http://patchwork.ozlabs.org/project/uboot/list/?series=233875&state=*

The cross-compilation toolchain used to build the U-Boot image is:
https://mirrors.edge.kernel.org/pub/tools/crosstool/files/bin/x86_64/10.1.0/x86_64-gcc-10.1.0-nolibc-powerpc-linux.tar.xz

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---

Please pull the full contents (the u-boot.e500 binary) from
https://github.com/lbmeng/qemu, branch qemu-ppc.

 pc-bios/u-boot.e500 | Bin 406920 -> 421720 bytes
 roms/u-boot         |   2 +-
 2 files changed, 1 insertion(+), 1 deletion(-)

Comments

Philippe Mathieu-Daudé July 6, 2021, 7:41 a.m. UTC | #1
Hi Bin,

On 7/6/21 6:31 AM, Bin Meng wrote:
> Update the QEMU shipped u-boot.e500 image built from U-Boot mainline
> v2021.07 release, which added eTSEC support to the QEMU ppce500 target,
> via the following U-Boot series:
> 
>   http://patchwork.ozlabs.org/project/uboot/list/?series=233875&state=*
> 
> The cross-compilation toolchain used to build the U-Boot image is:
> https://mirrors.edge.kernel.org/pub/tools/crosstool/files/bin/x86_64/10.1.0/x86_64-gcc-10.1.0-nolibc-powerpc-linux.tar.xz

Ideally the process to update a firmware blob is (*):

- Add a CI job (as you did for .gitlab-ci.d/opensbi.yml)
- Commit the submodule new hash including "git shortlog current..new"
  in description
- Run the CI, download the almost-reproducible blob
- Test the blob locally
- Amend to the previous commit the CI blob

Since soft freeze is in 1 week, you might won't have enough time
to add a CI job, so this current patch is acceptable, but please
include the "git shortlog current..new" output.

Regards,

Phil.

(*) Yes we should document this somewhere. Wiki or docs?

> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> ---
> 
> Please pull the full contents (the u-boot.e500 binary) from
> https://github.com/lbmeng/qemu, branch qemu-ppc.
> 
>  pc-bios/u-boot.e500 | Bin 406920 -> 421720 bytes
>  roms/u-boot         |   2 +-
>  2 files changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/pc-bios/u-boot.e500 b/pc-bios/u-boot.e500
> index d2e29f81d6..8e635c8a5c 100644
> Binary files a/pc-bios/u-boot.e500 and b/pc-bios/u-boot.e500 differ
> diff --git a/roms/u-boot b/roms/u-boot
> index b46dd116ce..840658b093 160000
> --- a/roms/u-boot
> +++ b/roms/u-boot
> @@ -1 +1 @@
> -Subproject commit b46dd116ce03e235f2a7d4843c6278e1da44b5e1
> +Subproject commit 840658b093976390e9537724f802281c9c8439f5
>
Bin Meng July 6, 2021, 7:51 a.m. UTC | #2
Hi Philippe,

On Tue, Jul 6, 2021 at 3:41 PM Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>
> Hi Bin,
>
> On 7/6/21 6:31 AM, Bin Meng wrote:
> > Update the QEMU shipped u-boot.e500 image built from U-Boot mainline
> > v2021.07 release, which added eTSEC support to the QEMU ppce500 target,
> > via the following U-Boot series:
> >
> >   http://patchwork.ozlabs.org/project/uboot/list/?series=233875&state=*
> >
> > The cross-compilation toolchain used to build the U-Boot image is:
> > https://mirrors.edge.kernel.org/pub/tools/crosstool/files/bin/x86_64/10.1.0/x86_64-gcc-10.1.0-nolibc-powerpc-linux.tar.xz
>
> Ideally the process to update a firmware blob is (*):
>
> - Add a CI job (as you did for .gitlab-ci.d/opensbi.yml)
> - Commit the submodule new hash including "git shortlog current..new"
>   in description
> - Run the CI, download the almost-reproducible blob
> - Test the blob locally
> - Amend to the previous commit the CI blob

Thanks for the guideline!

>
> Since soft freeze is in 1 week, you might won't have enough time
> to add a CI job, so this current patch is acceptable, but please
> include the "git shortlog current..new" output.

I did consider including the shotlogs here, just like what I did for
OpenSBI update. But I am afraid it is a toooo long list because unlike
OpenSBI U-Boot is a very active community. Hence I chose not to
mention it in the commit message.

>
> Regards,
>
> Phil.
>
> (*) Yes we should document this somewhere. Wiki or docs?

Either should be okay. If we put in the docs, we should probably
collect some other development processes (e.g.: submitting patches)
and put there, like what is done in the Linux kernel.

Regards,
Bin
diff mbox series

Patch

diff --git a/pc-bios/u-boot.e500 b/pc-bios/u-boot.e500
index d2e29f81d6..8e635c8a5c 100644
Binary files a/pc-bios/u-boot.e500 and b/pc-bios/u-boot.e500 differ
diff --git a/roms/u-boot b/roms/u-boot
index b46dd116ce..840658b093 160000
--- a/roms/u-boot
+++ b/roms/u-boot
@@ -1 +1 @@ 
-Subproject commit b46dd116ce03e235f2a7d4843c6278e1da44b5e1
+Subproject commit 840658b093976390e9537724f802281c9c8439f5