All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Johannes Schauer <j.schauer@email.de>
Cc: gta04-owner@goldelico.com, barebox@lists.infradead.org
Subject: Re: barebox on gta04
Date: Tue, 25 Oct 2011 10:36:47 +0200	[thread overview]
Message-ID: <20111025083647.GS23421@pengutronix.de> (raw)
In-Reply-To: <20111025063850.GA19390@hoothoot>

On Tue, Oct 25, 2011 at 08:38:50AM +0200, Johannes Schauer wrote:
> Hi Sascha,
> 
> On Mon, Oct 24, 2011 at 11:34:13PM +0200, Sascha Hauer wrote:
> > You could start barebox from U-Boot (just load the barebox binary
> > image to RAM and jump to it using the U-Boot 'go' command). Then you
> > could comment out the lowlevel code from barebox (arch_init_lowlevel,
> > pin_mux) and see if the problem persists. Maybe there is something
> > different in the lowlevel code.
> 
> This is a great idea! So first I loaded the unmodified barebox binary
> using:
> 
> mmc rescan 0
> fatload mmc 0 0x82000000 barebox.bin
> go 0x82000000
> 
> It started executing and the error was the same as before.
> 
> Then I tried doing what you suggested above and commented
> arch_init_lowlevel in ./arch/arm/mach-omap/omap3_core.S and I was not
> sure what you meant by `pin_mux` as the string appeared nowhere in the
> code but I thought you might mean the mux_config in
> ./arch/arm/boards/omap/board-beagle.c ?

Yes, that's what I meant

> But after doing that, barebox
> would just not start anymore when loaded by uboot.

Strange. Should not be the case, no idea what's happening here.

> 
> So can you give me some more specific advices on what to comment?
> 
> > Another thing: Where does this string come from? We have this twice in
> > the driver and one of them should really be "timeout waiting for reset
> > done"
> 
> If I read the code correctly, then my error comes from line
> 224 of ./drivers/mci/omap_hsmmc.c :
> 
> dev_dbg(hsmmc->dev, "timedout waiting for cc2!\n");

Look again, we accidently have the same string twice in the driver:

./drivers/mci/omap_hsmmc.c:201:                 dev_dbg(hsmmc->dev, "timedout waiting for cc2!\n");
./drivers/mci/omap_hsmmc.c:224:                 dev_dbg(hsmmc->dev, "timedout waiting for cc2!\n");

The one in line 224 should really be "timeout waiting for reset done".

So what I wanted to know is whether it's the error message in line 201
or 224 that we see. So please change one of them

> 
> I activated this debug message by using "#define DEBUG" on top of the
> file. Initially I only got the message "Cannot reset the SD/MMC
> interface" from line 1185 in ./drivers/mci/mci-core.c after it tried to
> call the init code from omap_hsmmc.c.
> 
> But where would you find "timeout waiting for reset done"? I tried to
> recursively grep my barebox sources without finding this string?
> 
> thanks!
> 
> cheers, josch
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
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

  reply	other threads:[~2011-10-25  8:36 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-24 21:11 barebox on gta04 Johannes Schauer
2011-10-24 21:34 ` Sascha Hauer
2011-10-25  6:38   ` Johannes Schauer
2011-10-25  8:36     ` Sascha Hauer [this message]
2011-10-25 10:05       ` Johannes Schauer
2011-10-31 11:57         ` Robert Schwebel
2011-10-31 14:05           ` Johannes Schauer
2011-10-31 14:18             ` Robert Schwebel
  -- strict thread matches above, loose matches on Subject: below --
2011-10-24 21:01 Johannes Schauer
2011-10-24 21:07 ` Robert Schwebel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20111025083647.GS23421@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=gta04-owner@goldelico.com \
    --cc=j.schauer@email.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.