mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Roberto Nibali <rnibali@gmail.com>
Cc: barebox@lists.infradead.org
Subject: Re: [BUG: barebox-git] failure at common/block.c:248/block_put()!
Date: Wed, 30 May 2012 07:33:42 +0200	[thread overview]
Message-ID: <20120530053342.GE30400@pengutronix.de> (raw)
In-Reply-To: <CAONxwYMJdybK9AgnOp5pVNPNFbDmb_-To=w0+S2P1098_mhGoQ@mail.gmail.com>

On Tue, May 29, 2012 at 09:57:38PM +0200, Roberto Nibali wrote:
> Hi
> 
> > Data Write Failed in PIO Mode.imx-esdhc@imx-esdhc0: timeout 2
> > > imx-esdhc@imx-esdhc0: timeout 1
> > > imx-esdhc@imx-esdhc0: timeout 1
> > > imx-esdhc@imx-esdhc0: timeout 1
> > > block_cache: blk->ops->read returned -110
> >
> > Ok, so your problem is inside the esdhc driver. Sorry, I don't know what
> > quirks we may need here. Might be worth looking at the kernel or U-Boot
> > to find out.
> 
> 
> I don't know where this driver comes from, however I have been revising the
> i.MX25 Chipset Errata and there are heaps of problems with regard to eSDHC.
> I will try to address some of them (for example WML related) and see if
> this fixes anything.

The driver comes from U-Boot, but the kernel might also be a good source
to see where the i.MX25 gets special handling.

> 
> 
> > > BUG: failure at common/block.c:249/block_put()!
> > > BUG!
> >
> > Nevertheless we'll need a fix for this. I'll prepare a patch for this if
> > noone else is faster ;)
> 
> 
> Well, the BUG() is legit, I just seem to be the first to hit it. I am not
> sure how you want to fix it, because if it's an issue with regard to
> chipset errata, at least a warning has to be issues. Maybe BUG() is a bit
> excessive ;).

I only want to get rid of the BUG() (see the patches I just sent). Of
course this won't fix your problems with the driver.

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

      reply	other threads:[~2012-05-30  5:33 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-29  9:49 Roberto Nibali
2012-05-29 10:14 ` Sascha Hauer
2012-05-29 10:25   ` Roberto Nibali
2012-05-29 11:47     ` Roberto Nibali
2012-05-29 13:33       ` Roberto Nibali
2012-05-29 18:57       ` Sascha Hauer
2012-05-29 19:57         ` Roberto Nibali
2012-05-30  5:33           ` Sascha Hauer [this message]

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=20120530053342.GE30400@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=rnibali@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox