mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] OMAP4: fix omap4_bootsource regression
@ 2013-06-21 13:47 Jan Weitzel
  2013-06-23 19:30 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Weitzel @ 2013-06-21 13:47 UTC (permalink / raw)
  To: barebox; +Cc: vicencb

commit d7a913469c34553e96c887f8a9934bacd794e81c breaks boot source detection
for pcm049. The tracing vectors shows all tested boot sources, so order is
important. By not returning but overwriting src we effectively reversed the
order if more than one flag is set.

Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
---
 arch/arm/mach-omap/omap4_generic.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-omap/omap4_generic.c b/arch/arm/mach-omap/omap4_generic.c
index bf1f1a7..4878aec 100644
--- a/arch/arm/mach-omap/omap4_generic.c
+++ b/arch/arm/mach-omap/omap4_generic.c
@@ -508,9 +508,9 @@ static int omap4_bootsource(void)
 
 	if (bootsrc & (1 << 5))
 		src = BOOTSOURCE_MMC;
-	if (bootsrc & (1 << 3))
+	else if (bootsrc & (1 << 3))
 		src = BOOTSOURCE_NAND;
-	if (bootsrc & (1<<20))
+	else if (bootsrc & (1<<20))
 		src = BOOTSOURCE_USB;
 	bootsource_set(src);
 	bootsource_set_instance(0);
-- 
1.7.0.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] OMAP4: fix omap4_bootsource regression
  2013-06-21 13:47 [PATCH] OMAP4: fix omap4_bootsource regression Jan Weitzel
@ 2013-06-23 19:30 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2013-06-23 19:30 UTC (permalink / raw)
  To: Jan Weitzel; +Cc: barebox, vicencb

On Fri, Jun 21, 2013 at 03:47:42PM +0200, Jan Weitzel wrote:
> commit d7a913469c34553e96c887f8a9934bacd794e81c breaks boot source detection
> for pcm049. The tracing vectors shows all tested boot sources, so order is
> important. By not returning but overwriting src we effectively reversed the
> order if more than one flag is set.
> 
> Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>

Applied to master and stable/v2013.06

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-06-23 19:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-21 13:47 [PATCH] OMAP4: fix omap4_bootsource regression Jan Weitzel
2013-06-23 19:30 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox