Merge lp:~lag/linaro-image-tools/lt-ux500-support into lp:linaro-image-tools/11.11

[Lee Jones]

Include support for the new u8500 hardware pack

[Loïc Minier (lool)]

Could you explain why the kernel is suffixed u8500? Are there going to be both ux500 and u8500 kernels?

Is one a Landing Team kernel? Should it be lt-u8500?

[Lee Jones (lag)]

On 18/03/11 13:52, Loïc Minier wrote:
> Could you explain why the kernel is suffixed u8500? Are there going to be both ux500 and u8500 kernels?

Because this kernel does not support the u5500.

> Is one a Landing Team kernel? Should it be lt-u8500?

There is only one kernel at present, which is the BSP one.

[Loïc Minier (lool)]

So the current config never worked?

Will you eventually support u5500?

[Lee Jones (lag)]

On 18/03/11 14:33, Loïc Minier wrote:
> So the current config never worked?

Not for the new hwpack, no.

The old hwpack still works as well as it always did.

> Will you eventually support u5500?

Not that I'm aware of.

[Guilherme Salgado (salgado)]

Lee, do you foresee any issues if we are a bit more conservative and use 'u?500' as the suffix instead of 'u*500'?

[Lee Jones (lag)]

On 28/03/11 14:39, Guilherme Salgado wrote:
> Lee, do you foresee any issues if we are a bit more conservative and use 'u?500' as the suffix instead of 'u*500'?

What's the difference?

[Guilherme Salgado (salgado)]

On Mon, 2011-03-28 at 13:43 +0000, Lee Jones wrote:
> On 28/03/11 14:39, Guilherme Salgado wrote:
> > Lee, do you foresee any issues if we are a bit more conservative and use 'u?500' as the suffix instead of 'u*500'?
>
> What's the difference?

'*' is greedy: it will match any number of characters whereas '?' will
match only one.

[Lee Jones (lag)]

On 28/03/11 14:51, Guilherme Salgado wrote:
> On Mon, 2011-03-28 at 13:43 +0000, Lee Jones wrote:
>> On 28/03/11 14:39, Guilherme Salgado wrote:
>>> Lee, do you foresee any issues if we are a bit more conservative and use 'u?500' as the suffix instead of 'u*500'?
>>
>> What's the difference?
>
> '*' is greedy: it will match any number of characters whereas '?' will
> match only one.

'?' is perfect.

[Guilherme Salgado (salgado)]

I see we currently have only one hwpack with the BSP kernel but if there are plans to have a non-BSP kernel then we should take advantage of the support for multiple suffixes that is being proposed on bug 744857. Otherwise just changing 'u*500' to 'u?500' should be fine.

review: Needs Fixing

[Lee Jones (lag)]

I guess either would work.

There are already two hwpacks by the way, a BSP one and a LT one.

[Guilherme Salgado (salgado)]

But do they share the same suffix? 'cause if they don't then we'll need
to use a list of suffixes.

[Loïc Minier (lool)]

I've merged this with the change Guilherme proposed and added a note in https://code.launchpad.net/~mabac/linaro-image-tools/bug-744857/+merge/55492 that we should use kernel_flavors instead of a regexp.

=== modified file 'linaro_media_create/boards.py'
--- linaro_media_create/boards.py	2011-03-10 23:28:21 +0000
+++ linaro_media_create/boards.py	2011-03-18 10:36:54 +0000
@@ -395,7 +395,7 @@
     kernel_addr = '0x00100000'
     initrd_addr = '0x08000000'
     load_addr = '0x00008000'
-    kernel_suffix = 'ux500'
+    kernel_suffix = 'u*500'
     boot_script = 'flash.scr'
     extra_boot_args_options = (
         'earlyprintk rootdelay=1 fixrtc nocompcache '