release: Add arm_install_boot to install the commit boot bits

[manu]

Summary

This reduce the per-board arm_install_uboot to just install u-boot.
While here remove the installation of rpi.dtb and rpi2.dtb as we load
them from the UFS partition via ubldr.

[imp]

This is a good first step...
I wonder, though about the one item I highlighted.

release/tools/arm.subr

189 ↗

(On Diff #45196)

${FATMOUNT}/EFI/BOOT/$(efi_boot_name ${EMBEDDED_TARGET})

might be better, then you don't need the if/then/else here.

This might be useful since it would allow any target.

195 ↗

(On Diff #45196)

why are we touching firstboot here?

196 ↗

(On Diff #45196)

strictly speaking, this should not be needed...

[gjb]

release/tools/arm.subr

195 ↗

(On Diff #45196)

The growfs(8) rc script depends on it.

[manu]

release/tools/arm.subr

189 ↗

(On Diff #45196)

for arm64 the name is aa64 so this won't work.
I choose this because we need a if/then/else anyway for bootarm/bootaa64 etc ...

196 ↗

(On Diff #45196)

I didn't add it.
And I have the old-timer need to do it from time to time but never checked if it's still needed.

[imp]

release/tools/arm.subr

195 ↗

(On Diff #45196)

that's not my question, why is it needed *HERE*.

It's got nothing to do with booting, and is the only reason we mount the UFS fileysstem in this routine.

[imp]

release/tools/arm.subr

189 ↗

(On Diff #45196)

Yes it does. the efi_boot_name fuction would return bootaa64.efi for the first arg being arm64. It would returm bootarm.efi if the first arg is arm.

[gjb]

release/tools/arm.subr

195 ↗

(On Diff #45196)

I do not recall why I did it this way. I'm sure I had a reason to do it this way, but it escapes me at the moment.

[manu]

Add efi_boot_name function and use it to find the correct name for the efi loader.

[manu]

You requested changes but did not say what changes.

[gjb]

In D16239#347318, @manu wrote:

You requested changes but did not say what changes.

Sigh. Because Phabricator's UI sucks.

release/arm/RPI-B.conf

26 ↗

(On Diff #45365)

UFSMOUNT declaration was removed here, but the mkdir(1) call still has the variable.

release/arm/RPI2.conf

27 ↗

(On Diff #45365)

Same here.

[manu]

Remove UFSMOUNT for RPI-B and RPI2

[manu]

Meh, I guess you're right about the UI, I just accepted the review when I just wanted to mark comment as done :)