All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10
Date: Wed, 27 Nov 2013 15:11:21 +0100	[thread overview]
Message-ID: <20131127151121.51f124e6@skate> (raw)
In-Reply-To: <CAFqH_539GUrz34anUp4d-kFarcGnxfeCZiufg8ZjkF+G_s6dUA@mail.gmail.com>

Dear Enric Balletbo Serra,

On Wed, 27 Nov 2013 14:56:15 +0100, Enric Balletbo Serra wrote:

> 2013/11/27 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>:
> > Hello,
> >
> > We've recently updated from u-boot 2013.04 to u-boot 2013.10 on our
> > IGEP boards (OMAP3 based, U-Boot shows "OMAP36XX/37XX-GP ES1.2"), and
> > we're seeing random I2C communication problems at startup.
> 
> Right, I've reproduced the issue. Any OMAP3-based board affected for
> this issue ?

Not sure to understand your question: my paragraph above mentions the
IGEP board as being the platform on which I'm seeing this. So indeed, a
OMAP3-based board is affected. But maybe I misunderstood your question.

> > I see that 960187ffa125b3938fec4b827bd9e8c04a204af8 ("ARM: OMAP: I2C:
> > New read, write and probe functions") has changed significantly the
> > OMAP I2C driver. And it turns out that reverting this commit actually
> > fixes the problem. No more error messages, no more hang at boot. The
> > commit message says that it was tested on OMAP4, OMAP5 and AM335x, but
> > apparently OMAP3 isn't working all that well with this commit.
> >
> > Best regards,
> 
> I'll try to investigate more.

Thanks! In the mean time, I'll just keep this commit reverted.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2013-11-27 14:11 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-27 12:19 [U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10 Thomas Petazzoni
2013-11-27 13:56 ` Enric Balletbo Serra
2013-11-27 14:11   ` Thomas Petazzoni [this message]
2013-11-27 14:22     ` Enric Balletbo Serra
2013-11-27 14:50       ` Andreas Bießmann
2013-11-27 14:52       ` Nikita Kiryanov
2013-11-27 15:28         ` Thomas Petazzoni
2013-11-27 15:52           ` Nikita Kiryanov
2013-11-27 16:01             ` Lubomir Popov
2013-11-27 16:13               ` Tom Rini
2013-11-27 21:16                 ` Lubomir Popov
2013-11-27 16:10             ` Thomas Petazzoni
2013-11-27 16:31               ` Nikita Kiryanov
2013-11-27 17:12                 ` Enric Balletbo Serra
2013-11-28 13:06                   ` Nikita Kiryanov
2013-11-27 14:58       ` Nikita Kiryanov
2013-11-27 14:45 ` Tom Rini
2013-11-27 15:35   ` Thomas Petazzoni
2013-11-27 15:45     ` Michael Trimarchi

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=20131127151121.51f124e6@skate \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=u-boot@lists.denx.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.