[OE-core] [PATCH] kernel-fitimage.bbclass: Allow unset load/entry addresses for ramdisks

[Nathan Rossi]

On 23 November 2016 at 15:29, Nathan Rossi <nathan at nathanrossi.com> wrote:
> Just pinging these patches, to see if there were any issues or
> comments. I realize I sent them just before a release so it was very
> likely they would get lost in the noise.

Pinging these three patches again.

https://patchwork.openembedded.org/patch/133134/
https://patchwork.openembedded.org/patch/133135/ (another patch was
just sent by someone else which does the same change,
https://lists.yoctoproject.org/pipermail/yocto/2017-January/033840.html)

And I've updated the third one, sent a v2 so that it applies cleanly to master.

https://patchwork.openembedded.org/patch/135805/

Thanks,
Nathan

>
> Thanks,
> Nathan
>
> On 21 October 2016 at 22:07, Nathan Rossi <nathan at nathanrossi.com> wrote:
>> Allow the load and entry addresses to remain unset if the UBOOT_RD_*
>> variables are also unset for ramdisk entries in the image tree. This
>> allows for U-Boot to decide dynamically where to load the ramdisk.
>>
>> Signed-off-by: Nathan Rossi <nathan at nathanrossi.com>
>> ---
>>  meta/classes/kernel-fitimage.bbclass | 13 +++++++++++--
>>  1 file changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/classes/kernel-fitimage.bbclass b/meta/classes/kernel-fitimage.bbclass
>> index 3bbf91f65a..5c9e76b9ac 100644
>> --- a/meta/classes/kernel-fitimage.bbclass
>> +++ b/meta/classes/kernel-fitimage.bbclass
>> @@ -178,6 +178,15 @@ EOF
>>  fitimage_emit_section_ramdisk() {
>>
>>         ramdisk_csum="sha1"
>> +       ramdisk_loadline=""
>> +       ramdisk_entryline=""
>> +
>> +       if [ -n "${UBOOT_RD_LOADADDRESS}" ]; then
>> +               ramdisk_loadline="load = <${UBOOT_RD_LOADADDRESS}>;"
>> +       fi
>> +       if [ -n "${UBOOT_RD_ENTRYPOINT}" ]; then
>> +               ramdisk_entryline="entry = <${UBOOT_RD_ENTRYPOINT}>;"
>> +       fi
>>
>>         cat << EOF >> ${1}
>>                  ramdisk@${2} {
>> @@ -187,8 +196,8 @@ fitimage_emit_section_ramdisk() {
>>                          arch = "${UBOOT_ARCH}";
>>                          os = "linux";
>>                          compression = "none";
>> -                        load = <${UBOOT_RD_LOADADDRESS}>;
>> -                        entry = <${UBOOT_RD_ENTRYPOINT}>;
>> +                        ${ramdisk_loadline}
>> +                        ${ramdisk_entryline}
>>                          hash at 1 {
>>                                  algo = "${ramdisk_csum}";
>>                          };
>> --
>> 2.9.3