[MIR] uboot-imx

Bug #504723 reported by Oliver Grawert
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
uboot-imx (Ubuntu)
Undecided
Alexander Sack
Lucid
Undecided
Alexander Sack

Bug Description

uboot-imx is the new bootloader of choice for the freescale imx51 architecture. it will replace redboot on our images soon (see https://blueprints.launchpad.net/ubuntu/+spec/mobile-lucid-imx51-debian-cd-to-uboot )

this specific package is in ubuntu only and is fully maintained by the ubuntu-mobile team, it doesnt have any executables and just puts the bootloader .bin file in a package specific subdirectory of /usr/lib/.

please promote it to main.

Martin Pitt (pitti)
Changed in uboot-imx (Ubuntu):
assignee: nobody → Alexander Sack (asac)
Revision history for this message
Alexander Sack (asac) wrote :

approving with some nits; those can be dealt after Main inclusion. Just ensure that the bugs are filed and assigned.

Please file bugs for and fix a couple of compiler warnings:

fat.c: In function 'read_bootsectandvi':
fat.c:707: warning: array subscript is above array bounds

mxc_fec.c: In function 'fec_init':
mxc_fec.c:625: warning: cast from pointer to integer of different size

xyzModem.c: In function 'xyzModem_stream_open':
xyzModem.c:564: warning: 'dummy' is used uninitialized in this function

bmp_logo.c: In function 'main':
bmp_logo.c:71: warning: ignoring return value of 'fread', declared with attribute warn_unused_result

a few more nice to have fixes:

various warnings of type:
   ... warning: function declaration isn't a prototype

also:
  fsl_mmc.c:1154: warning: 'mmc_send_scr' defined but not used
  fsl_mmc.c:1216: warning: 'mmc_read_switch' defined but not used

  imx_spi_nor_atmel.c: In function 'spi_nor_erase_page':
  imx_spi_nor_atmel.c:109: warning: unused variable 'cmd'
  imx_spi_nor_atmel.c: At top level:
  imx_spi_nor_atmel.c:106: warning: 'spi_nor_erase_page' defined but not used

  cmd_flash.c:471: warning: unused variable 'addr_last'
  cmd_flash.c:471: warning: unused variable 'addr_first'

Changed in uboot-imx (Ubuntu):
status: New → Fix Committed
Alexander Sack (asac)
Changed in uboot-imx (Ubuntu Lucid):
status: Fix Committed → Won't Fix
Changed in uboot-imx (Ubuntu):
milestone: none → later
Revision history for this message
Martin Pitt (pitti) wrote :

Do you still need this? We won't promote it until it gets seeded or depended on (and appears in component-mismatches)

Revision history for this message
Oliver Grawert (ogra) wrote :

not for lucid

Changed in uboot-imx (Ubuntu):
status: Fix Committed → Invalid
Revision history for this message
Oliver Grawert (ogra) wrote :

for L+1 we'll have a more generic uboot package that covers more different binaries than only imx51, if time comes i'll file a new MIR

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers