Comment 4 for bug 889179

Revision history for this message
Mattias Backman (mabac) wrote : Re: [Bug 889179] [NEW] Add vexpress_a9 platform to linaro-android-media-create

On Mon, Nov 14, 2011 at 8:27 AM, Tixy (Jon Medhurst)
<email address hidden> wrote:
> On Fri, 2011-11-11 at 17:09 +0000, Dave Martin wrote:
>> >   Add vexpress_a9 platform to linaro-android-media-create
>>
>> Should the name "vexpress" be used instead, for consistency with
>> linaro-media-create?  Or does l-a-m-c use an independent set of board
>> names?
>>
>> Other platforms based on vexpress would need to be more fully qualified
>> in any case.
>
> For linaro-media-create we can specify all of the required board
> parameters in the hardware pack so the tool doesn't need a command-line
> board parameter to distinguish between vexpress flavours.
>
> linaro-android-media-create however needs everything hardcoded into the
> tools, so we need to distinguish between vexpress core-tiles to pass the
> correct memory addresses to U-Boot.

Still, I agree that we should keep the naming consistent with the
names in l-m-c for code maintenance reasons and for not confusing the
end user with different names for the same board. If it's important to
change the name of the BoardConfig and --dev option we should also
make the same change in l-m-c. The --dev option is left in l-m-c since
it will be needed when there is support for multiple boards in one
hwpack.

> - boot_script = None
> + boot_script = 'boot.scr'

This change should go in the AndroidVexpressConfig so it does not
break support for existing Vexpress hwpacks. That line is still in
boards.py only to maintain support for old hwpacks. and will not take
effect for new hwpacks anyway.

Would you push your branch to launchpad and create a merge proposal so
we can keep track of this change?

Thanks,

Mattias