Merge lp://qastaging/~berolinux/linaro-image-tools/mx6qsabrelite-bootargs into lp://qastaging/linaro-image-tools/11.11
Proposed by
Bernhard Rosenkraenzer
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Данило Шеган | ||||
Proposed branch: | lp://qastaging/~berolinux/linaro-image-tools/mx6qsabrelite-bootargs | ||||
Merge into: | lp://qastaging/linaro-image-tools/11.11 | ||||
Diff against target: |
41 lines (+24/-1) 1 file modified
linaro_image_tools/media_create/android_boards.py (+24/-1) |
||||
To merge this branch: | bzr merge lp://qastaging/~berolinux/linaro-image-tools/mx6qsabrelite-bootargs | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mattias Backman (community) | Needs Fixing | ||
Zach Pfeffer | Pending | ||
Review via email: mp+95181@code.qastaging.launchpad.net |
Description of the change
New version of the Mx6QSabreLite bootargs patch - this time implemented completely inside android_boards.py
To post a comment you must log in.
Unmerged revisions
- 497. By Bernhard Rosenkraenzer
-
Use the right bootargs for MX6QSabrelite
Thanks for the update. I have a couple of more requests.
On Wed, Feb 29, 2012 at 3:34 PM, Bernhard Rosenkraenzer image-tools) /code.launchpad .net/~berolinux /linaro- image-tools/ mx6qsabrelite- bootargs/ +merge/ 95181 /code.launchpad .net/~berolinux /linaro- image-tools/ mx6qsabrelite- bootargs/ +merge/ 95181 image_tools/ media_create/ android_ boards. py' image_tools/ media_create/ android_ boards. py 2012-02-21 08:01:08 +0000 image_tools/ media_create/ android_ boards. py 2012-02-29 14:33:21 +0000 mx5_boot_ loader( os.path. join(boot_ device_ or_file, "u-boot.imx"), boot_partition, cls.LOADER_ MIN_SIZE_ S) eliteConfig( AndroidMx53LoCo Config) : eliteConfig( AndroidBoardCon fig, Mx53LoCoConfig):
<email address hidden> wrote:
> Bernhard Rosenkraenzer has proposed merging lp:~berolinux/linaro-image-tools/mx6qsabrelite-bootargs into lp:linaro-image-tools.
>
> Requested reviews:
> Zach Pfeffer (pfefferz)
> linaro-image-tools maintainers (linaro-
>
> For more details, see:
> https:/
>
> New version of the Mx6QSabreLite bootargs patch - this time implemented completely inside android_boards.py
> --
> https:/
> You are subscribed to branch lp:linaro-image-tools.
>
> === modified file 'linaro_
> --- linaro_
> +++ linaro_
> @@ -296,13 +296,36 @@
> install_
>
>
> -class AndroidMx6QSabr
> +class AndroidMx6QSabr
> + serial_tty = 'ttymxc1'
This fails the test, could you please update that too? Please run
testr init
testr run
to make sure that the test suite succeeds before we merge this.
> uboot_flavor = 'mx6qsabrelite' args_options = ( %s,115200n8' % ( specific_ args = 'init=/init androidboot. console= %s' % ( align_boot_ part=False) : MIN_SIZE_ S, 1, PART_ALIGN_S) 6QSabreliteConf ig, cls).get_ sfdisk_ cmd( align_boot_ part=True, start_addr= loader_ end, boot_loader( cls, boot_partition, boot_device_ or_file) : mx5_boot_ loader( os.path. join(boot_ device_ or_file, "u-boot.imx"), boot_partition, cls.LOADER_ MIN_SIZE_ S)
> kernel_addr = '0x10000000'
> initrd_addr = '0x12000000'
> load_addr = '0x10008000'
> dtb_addr = '0x11ff0000'
> dtb_name = 'board.dtb'
> + extra_boot_
> + 'earlyprintk rootdelay=1 fixrtc nocompcache di1_primary tve')
> + _extra_serial_opts = 'console=
> + serial_tty)
> + android_
> + serial_tty)
> +
> + @classmethod
> + def get_sfdisk_cmd(cls, should_
> + loader_start, loader_end, loader_len = align_partition(
> + 1, cls.LOADER_
> +
> + command = super(AndroidMx
> + should_
> + extra_part=True)
> +
> + return '%s,%s,0xDA\n%s' % (
> + loader_start, loader_len, command)
> +
> + @classmethod
> + def install_
> + install_
Are these new functions identical to those on AndroidMx53LoCo Config?
Could you find a way to avoid duplicating them, like ?
> onfig(AndroidBo ardConfig) :
>
> class AndroidSamsungC
>
>