[MIR] uboot-imx

Bug #504723 reported by Oliver Grawert
This bug affects 1 person
Affects Status Importance Assigned to Milestone
uboot-imx (Ubuntu)
Alexander Sack
Won't Fix
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

  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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.