IGEP default bootcmd broken and not as useful as beagle's

Bug #716540 reported by Loïc Minier
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro U-Boot
Fix Released
Medium
John Rigby
u-boot-linaro (Ubuntu)
Fix Released
Medium
John Rigby

Bug Description

Hey

#define CONFIG_BOOTCOMMAND \
        "mmc init 0 ; fatload mmc 0 0x80000000 setup.ini ; source \0"

should be mmc rescan...!!

also, not terribly useful to locate and use a boot.scr...

Cheers,

Tags: patch
Revision history for this message
Loïc Minier (lool) wrote :

What I meant is that it would be a good time to change the default bootcmd to look for boot.scr (additionally to setup.ini), and perhaps even boot.txt as well, if the text script patch goes in.

Revision history for this message
Enric Balletbo i Serra (eballetbo) wrote :

This is the first proposal patch to solve the issue.

tags: added: patch
Revision history for this message
Loïc Minier (lool) wrote :

From a quick look, this seems much more modern than what we have now; something like the beagle script is fine, if it was inspired by beagle!

You don't seem to look for boot.ini/setup.ini; it doesn't matter for Linaro, but maybe it would be nice for people who upgrade their u-boot to have their boot.ini/setup.ini still picked up?

I see you're the upstream u-boot maintainer for this board, if the patch works for you would you mind sending this on the upstream u-boot list? Thanks a lot! Whatever the upstream fate of this patch, we can take it in the Linaro tree, but it would be best to get upstream ack before we merge it

Changed in u-boot-linaro (Ubuntu):
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
Loïc Minier (lool) wrote :

I've sent this patch to the upstream u-boot list

John Rigby (jcrigby)
Changed in u-boot-linaro (Ubuntu):
assignee: nobody → John Rigby (jcrigby)
Changed in u-boot-linaro:
assignee: nobody → John Rigby (jcrigby)
John Rigby (jcrigby)
Changed in u-boot-linaro:
status: New → In Progress
Revision history for this message
Mounir Bsaibes (mounir-bsaibes) wrote :

Setting Importance to Medium as patch has been submitted upstream per comment #4.

Changed in u-boot-linaro:
importance: Undecided → Medium
Revision history for this message
Jani Monoses (jani) wrote :

Can upstream ml-thread or issue tracker link be put here as well to more easily follow what happens?

Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Patch already applied upstream.

Changed in u-boot-linaro:
status: In Progress → Fix Released
Changed in u-boot-linaro (Ubuntu):
status: In Progress → Fix Released
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.