Re: [ptxdist] [PATCH 1/2] kernel: pass loadaddr for building uImage

[Marc Kleine-Budde <mkl@pengutronix.de>]

[-- Attachment #1.1.1: Type: text/plain, Size: 2269 bytes --]

On 11/28/2016 05:46 PM, Michael Olbrich wrote:
> On Mon, Nov 28, 2016 at 11:21:31AM +0100, Alexander Dahl wrote:
>> On newer kernels (4.x ?) kernel build fails when building an uImage
>> unless you provide LOADADDR as kernel make var, which is put in the
>> resulting uImage header. You could append it to
>> PTXCONF_KERNEL_EXTRA_MAKEVARS but this would be just a workaround. This
>> patch extends the menu if uImage is selected so you can directly input
>> the desired address.
> 
> I don't have a lot of experience with uImages. Marc, what do you think?

According to rmk uimages are considered obsolete. However there are
probably some old bootloaders around that insist of that format.

>> Signed-off-by: Alexander Dahl <post@lespocky.de>
>> ---
>>  platforms/kernel.in | 7 +++++++
>>  rules/kernel.make   | 5 +++++
>>  2 files changed, 12 insertions(+)
>>
>> diff --git a/platforms/kernel.in b/platforms/kernel.in
>> index fd1c492..2a00122 100644
>> --- a/platforms/kernel.in
>> +++ b/platforms/kernel.in
>> @@ -213,6 +213,13 @@ config KERNEL_IMAGE
>>  config KERNEL_DTC
>>  	bool
>>  
>> +config KERNEL_LOADADDR
>> +	hex
>> +	prompt "uImage load address"
>> +	depends on KERNEL_IMAGE_U
>> +	help
>> +	  Newer kernels want this passed if you build an uImage.
>> +
> 
> This should be right after the image selection.
> 
> Michael
> 
>>  config KERNEL_XZ
>>  	prompt "build xz-utils hosttool"
>>  	bool
>> diff --git a/rules/kernel.make b/rules/kernel.make
>> index d02a69c..3270231 100644
>> --- a/rules/kernel.make
>> +++ b/rules/kernel.make
>> @@ -71,6 +71,11 @@ KERNEL_MAKEVARS += \
>>  	DEPMOD=$(PTXCONF_SYSROOT_HOST)/sbin/depmod
>>  endif
>>  
>> +ifdef PTXCONF_KERNEL_IMAGE_U
>> +KERNEL_MAKEVARS += \
>> +	LOADADDR=$(PTXCONF_KERNEL_LOADADDR)
>> +endif
>> +

What happens if you build a older kernel and don't configure
PTXCONF_KERNEL_LOADADDR via ptxdist? Does it break while it used to work
with an older ptxdist?

Marc
-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 91 bytes --]

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de