Skip to content

rcar_flash.yaml: rename some salvator-x related files#38

Open
mourogurt wants to merge 1 commit intoxen-troops:mainfrom
mourogurt:rcar3-rename
Open

rcar_flash.yaml: rename some salvator-x related files#38
mourogurt wants to merge 1 commit intoxen-troops:mainfrom
mourogurt:rcar3-rename

Conversation

@mourogurt
Copy link
Copy Markdown

@mourogurt mourogurt commented Dec 23, 2025

Some spec files were renamed in the Yocto v5.9.7. Rename them in a config as well to make easier flash upstream firmware.

See: elinux.org/R-Car/Boards/H3SK#Flashing_firmware

@mourogurt
Copy link
Copy Markdown
Author

Showing file structure after building core image for salvator-x (H3SK).

> ls -1 build_salvator-x/build_mmp/tmp/deploy/images/salvator-x/
bl2-salvator-x-2x2g.bin
bl2-salvator-x-2x2g.elf
bl2-salvator-x-2x2g.srec
bl2-salvator-x-4x1g.bin
bl2-salvator-x-4x1g.elf
bl2-salvator-x-4x1g.srec
bl2-salvator-x-4x2g.bin
bl2-salvator-x-4x2g.elf
bl2-salvator-x-4x2g.srec
bl31-salvator-x-2x2g.bin
bl31-salvator-x-2x2g.elf
bl31-salvator-x-2x2g.srec
bl31-salvator-x-4x1g.bin
bl31-salvator-x-4x1g.elf
bl31-salvator-x-4x1g.srec
bl31-salvator-x-4x2g.bin
bl31-salvator-x-4x2g.elf
bl31-salvator-x-4x2g.srec
bootparam_sa0-2x2g.srec
bootparam_sa0-4x1g.srec
bootparam_sa0-4x2g.srec
cert_header_sa6-2x2g.srec
cert_header_sa6-4x1g.srec
cert_header_sa6-4x2g.srec
core-image-minimal.env
core-image-minimal-salvator-x.rootfs-20251129222219.ext4
core-image-minimal-salvator-x.rootfs-20251129222219.manifest
core-image-minimal-salvator-x.rootfs-20251129222219.tar.bz2
core-image-minimal-salvator-x.rootfs-20251129222219.testdata.json
core-image-minimal-salvator-x.rootfs-20251129222219.wic.bmap
core-image-minimal-salvator-x.rootfs-20251129222219.wic.xz
core-image-minimal-salvator-x.rootfs-20251129222219.wic.xz.sha256sum
core-image-minimal-salvator-x.rootfs.ext4
core-image-minimal-salvator-x.rootfs.manifest
core-image-minimal-salvator-x.rootfs.tar.bz2
core-image-minimal-salvator-x.rootfs.testdata.json
core-image-minimal-salvator-x.rootfs.wic.bmap
core-image-minimal-salvator-x.rootfs.wic.xz
core-image-minimal-salvator-x.rootfs.wic.xz.sha256sum
Image
Image--5.10.194+git0+6b085f452e-r1-salvator-x-20251129222219.bin
Image-salvator-x.bin
modules--5.10.194+git0+6b085f452e-r1-salvator-x-20251129222219.tgz
modules-salvator-x.tgz
r8a77950-salvator-x--5.10.194+git0+6b085f452e-r1-salvator-x-20251129222219.dtb
r8a77950-salvator-x.dtb
r8a77951-salvator-x--5.10.194+git0+6b085f452e-r1-salvator-x-20251129222219.dtb
r8a77951-salvator-x.dtb
r8a77951-salvator-xs--5.10.194+git0+6b085f452e-r1-salvator-x-20251129222219.dtb
r8a77951-salvator-xs.dtb
r8a779m1-salvator-xs--5.10.194+git0+6b085f452e-r1-salvator-x-20251129222219.dtb
r8a779m1-salvator-xs.dtb
tee-salvator-x.bin
tee-salvator-x.elf
tee-salvator-x.srec
u-boot.bin
u-boot.bin-r8a7795_salvator-x
u-boot-elf-salvator-x.srec
u-boot-elf-salvator-x-v2020.10+git-r0.srec
u-boot-initial-env-r8a7795_salvator-x
u-boot-initial-env-salvator-x-r8a7795_salvator-x
u-boot-initial-env-salvator-x-r8a7795_salvator-x-v2020.10+git-r0
u-boot-r8a7795_salvator-x-v2020.10+git-r0.bin
u-boot-salvator-x.bin
u-boot-salvator-x.bin-r8a7795_salvator-x

Some spec files were renamed in the Yocto v5.9.7. Rename them in a config as well to make easier flash upstream firmware.

See: elinux.org/R-Car/Boards/H3SK#Flashing_firmware
@mourogurt mourogurt changed the title rcar_flash.toml: rename some salvator-x related files rcar_flash.yaml: rename some salvator-x related files Dec 23, 2025
@lorc lorc requested a review from rshym December 24, 2025 18:13
@lorc
Copy link
Copy Markdown
Collaborator

lorc commented Dec 24, 2025

Ah, so Renesas strikes again....

@mourogurt, thank you for your contribution. We'll take a look at it. Problem is that we use this tool mostly with our own xen-troops based layers and we are using an older Yocto. What I am trying to say, that if we merge your PR, it probably would break some our READMEs and release notes.

@rshym, I know that you are busy RN, but probably you can take a look and asses how this PR will impact our products?

@rshym
Copy link
Copy Markdown
Contributor

rshym commented Jan 7, 2026

@mourogurt
Sorry for the late response.
Thanks a lot for your PR! We appreciate your contribution, especially taking into account that we were not aware of this change on Renesas side.
The only comment I have right now is that renaming will make it impossible to use this script for loaders generated with previous releases of BSP.
We are considering some changes to the script to allow flashing loaders with different names.
Let's keep your PR for now, until we implement and test additional changes.

@mourogurt
Copy link
Copy Markdown
Author

Are there any news regarding to this PR?

@rshym
Copy link
Copy Markdown
Contributor

rshym commented Feb 26, 2026

@mourogurt
Sorry for the delay. Unfortunately, I totally forgot about this PR, thanks for reminding.
I have one simple idea for the changes we need to make so we can merge your PR.
I'll be back in a few days.

@rshym
Copy link
Copy Markdown
Contributor

rshym commented Mar 9, 2026

@mourogurt
To keep you informed.
The PR I was talking about was submitted as a draft and is under testing/reworking.
#40

@mourogurt
Copy link
Copy Markdown
Author

Ok, let me know when PR will be merged, please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants