From: Kevin Hilman <khilman@deeprootsystems.com>
To: Jean Pihet <jean.pihet@newoldbits.com>
Cc: Paul Walmsley <paul@pwsan.com>,
balbi@ti.com, aaro.koskinen@iki.fi, linux-omap@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: OMAP baseline test results for v3.7-rc1
Date: Tue, 23 Oct 2012 12:17:23 -0700 [thread overview]
Message-ID: <87fw5580cc.fsf@deeprootsystems.com> (raw)
In-Reply-To: <CAORVsuUkU3iTTFcXfYJ=_FntjiTR33Gs4z9OsNEn2Cxq7Ef-qA@mail.gmail.com> (Jean Pihet's message of "Mon, 22 Oct 2012 18:12:06 +0200")
Jean Pihet <jean.pihet@newoldbits.com> writes:
> Hi,
>
> On Sat, Oct 20, 2012 at 8:14 AM, Paul Walmsley <paul@pwsan.com> wrote:
>> Hi Jean
>>
>> On Fri, 19 Oct 2012, Paul Walmsley wrote:
>>
>>> On Thu, 18 Oct 2012, Paul Walmsley wrote:
>>>
>>> > Here are some basic OMAP test results for Linux v3.7-rc1.
>>> > Logs and other details at http://www.pwsan.com/omap/testlogs/test_v3.7-rc1/
>>
>> ...
>>
>>> > Failing tests: needing investigation
>>> > ------------------------------------
>>> >
>>> > Boot tests:
>>>
>>> * 3530ES3 Beagle: I2C timeouts during userspace init
>>> - May be related to the threaded IRQ conversion of the I2C driver
>>> - Unknown cause
>>
>> This one turned out to be caused by:
>>
>> commit 3db11feffc1ad2ab9dea27789e6b5b3032827adc
>> Author: Jean Pihet <jean.pihet@newoldbits.com>
>> Date: Thu Sep 20 18:08:03 2012 +0200
>>
>> ARM: OMAP: convert I2C driver to PM QoS for MPU latency constraints
>>
>>
>> Reverting this commit causes the problem to go away, but since the OMAP PM
>> constraint code was removed as well, it's unlikely that a simple revert is
>> the right thing to do.
>>
>> Jean could you please investigate and fix this?
> I tried the latest l-o with omap2plus defconfig on my Beagleboard B5
> (ES2.1) and could not reproduce the problem.
> I do not have the I2C error messages at boot, nor at user space start
> up. I tried to read/write the TWL RTC, successfully.
>
> Another difference is the bootloader images. I have the following:
> - Texas Instruments X-Loader 1.4.2 (Feb 3 2009 - 15:34:17)
> - U-Boot 2009.01-dirty (Feb 19 2009 - 12:22:31)
> Could you send your bootloader images?
>
> I noticed you have I2C error messages in U-Boot, could that be the
> cause of the I2C lock-up?
FWIW, I have a relatively recent mainline u-boot+SPL:
U-Boot SPL 2012.04.01 (Jul 03 2012 - 07:07:04)
U-Boot 2012.04.01 (Jul 03 2012 - 07:07:04)
And I see I2C error messages from u-boot too:
timed out in wait_for_pin: I2C_STAT=0
I2C read: I/O error
I think these are normal/expected since u-boot may be looking for
expansion cards.
Kevin
WARNING: multiple messages have this Message-ID (diff)
From: khilman@deeprootsystems.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: OMAP baseline test results for v3.7-rc1
Date: Tue, 23 Oct 2012 12:17:23 -0700 [thread overview]
Message-ID: <87fw5580cc.fsf@deeprootsystems.com> (raw)
In-Reply-To: <CAORVsuUkU3iTTFcXfYJ=_FntjiTR33Gs4z9OsNEn2Cxq7Ef-qA@mail.gmail.com> (Jean Pihet's message of "Mon, 22 Oct 2012 18:12:06 +0200")
Jean Pihet <jean.pihet@newoldbits.com> writes:
> Hi,
>
> On Sat, Oct 20, 2012 at 8:14 AM, Paul Walmsley <paul@pwsan.com> wrote:
>> Hi Jean
>>
>> On Fri, 19 Oct 2012, Paul Walmsley wrote:
>>
>>> On Thu, 18 Oct 2012, Paul Walmsley wrote:
>>>
>>> > Here are some basic OMAP test results for Linux v3.7-rc1.
>>> > Logs and other details at http://www.pwsan.com/omap/testlogs/test_v3.7-rc1/
>>
>> ...
>>
>>> > Failing tests: needing investigation
>>> > ------------------------------------
>>> >
>>> > Boot tests:
>>>
>>> * 3530ES3 Beagle: I2C timeouts during userspace init
>>> - May be related to the threaded IRQ conversion of the I2C driver
>>> - Unknown cause
>>
>> This one turned out to be caused by:
>>
>> commit 3db11feffc1ad2ab9dea27789e6b5b3032827adc
>> Author: Jean Pihet <jean.pihet@newoldbits.com>
>> Date: Thu Sep 20 18:08:03 2012 +0200
>>
>> ARM: OMAP: convert I2C driver to PM QoS for MPU latency constraints
>>
>>
>> Reverting this commit causes the problem to go away, but since the OMAP PM
>> constraint code was removed as well, it's unlikely that a simple revert is
>> the right thing to do.
>>
>> Jean could you please investigate and fix this?
> I tried the latest l-o with omap2plus defconfig on my Beagleboard B5
> (ES2.1) and could not reproduce the problem.
> I do not have the I2C error messages at boot, nor at user space start
> up. I tried to read/write the TWL RTC, successfully.
>
> Another difference is the bootloader images. I have the following:
> - Texas Instruments X-Loader 1.4.2 (Feb 3 2009 - 15:34:17)
> - U-Boot 2009.01-dirty (Feb 19 2009 - 12:22:31)
> Could you send your bootloader images?
>
> I noticed you have I2C error messages in U-Boot, could that be the
> cause of the I2C lock-up?
FWIW, I have a relatively recent mainline u-boot+SPL:
U-Boot SPL 2012.04.01 (Jul 03 2012 - 07:07:04)
U-Boot 2012.04.01 (Jul 03 2012 - 07:07:04)
And I see I2C error messages from u-boot too:
timed out in wait_for_pin: I2C_STAT=0
I2C read: I/O error
I think these are normal/expected since u-boot may be looking for
expansion cards.
Kevin
next prev parent reply other threads:[~2012-10-23 19:16 UTC|newest]
Thread overview: 138+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-18 5:20 OMAP baseline test results for v3.7-rc1 Paul Walmsley
2012-10-18 5:20 ` Paul Walmsley
2012-10-18 6:48 ` Paul Walmsley
2012-10-18 6:48 ` Paul Walmsley
2012-10-18 8:37 ` Tero Kristo
2012-10-18 8:37 ` Tero Kristo
2012-10-18 8:48 ` Santosh Shilimkar
2012-10-18 8:48 ` Santosh Shilimkar
2012-10-20 17:30 ` Paul Walmsley
2012-10-20 17:30 ` Paul Walmsley
2012-10-19 16:55 ` Paul Walmsley
2012-10-19 16:55 ` Paul Walmsley
2012-10-19 17:01 ` Felipe Balbi
2012-10-19 17:01 ` Felipe Balbi
2012-10-19 17:56 ` Paul Walmsley
2012-10-19 17:56 ` Paul Walmsley
2012-10-19 18:10 ` Felipe Balbi
2012-10-19 18:10 ` Felipe Balbi
2012-10-19 19:03 ` Aaro Koskinen
2012-10-19 19:03 ` Aaro Koskinen
2012-10-19 19:01 ` Felipe Balbi
2012-10-19 19:01 ` Felipe Balbi
2012-10-19 19:38 ` Aaro Koskinen
2012-10-19 19:38 ` Aaro Koskinen
2012-10-22 17:21 ` Kevin Hilman
2012-10-22 17:21 ` Kevin Hilman
2012-10-22 20:43 ` Kevin Hilman
2012-10-22 20:43 ` Kevin Hilman
2012-10-20 6:14 ` Paul Walmsley
2012-10-20 6:14 ` Paul Walmsley
2012-10-22 16:12 ` Jean Pihet
2012-10-22 16:12 ` Jean Pihet
2012-10-22 17:26 ` Jean Pihet
2012-10-22 17:26 ` Jean Pihet
2012-10-23 19:19 ` Paul Walmsley
2012-10-23 19:19 ` Paul Walmsley
2012-10-23 19:23 ` Jean Pihet
2012-10-23 19:23 ` Jean Pihet
2012-10-25 10:12 ` Felipe Balbi
2012-10-25 10:12 ` Felipe Balbi
2012-10-26 20:15 ` Felipe Balbi
2012-10-26 20:15 ` Felipe Balbi
2012-10-26 22:03 ` Paul Walmsley
2012-10-26 22:03 ` Paul Walmsley
2012-10-29 20:00 ` Felipe Balbi
2012-10-29 20:00 ` Felipe Balbi
2012-10-30 12:17 ` Paul Walmsley
2012-10-30 12:17 ` Paul Walmsley
2012-10-30 12:32 ` Felipe Balbi
2012-10-30 12:32 ` Felipe Balbi
2012-10-30 12:50 ` Paul Walmsley
2012-10-30 12:50 ` Paul Walmsley
2012-10-30 12:54 ` Felipe Balbi
2012-10-30 12:54 ` Felipe Balbi
2012-10-30 13:17 ` Paul Walmsley
2012-10-30 13:17 ` Paul Walmsley
2012-10-30 14:04 ` Paul Walmsley
2012-10-30 14:04 ` Paul Walmsley
2012-10-31 8:34 ` Jean Pihet
2012-10-31 8:34 ` Jean Pihet
2012-10-31 9:05 ` Rafael J. Wysocki
2012-10-31 9:05 ` Rafael J. Wysocki
2012-10-31 8:22 ` Jean Pihet
2012-10-31 8:22 ` Jean Pihet
2012-10-31 10:49 ` Kevin Hilman
2012-10-31 10:49 ` Kevin Hilman
2012-10-31 21:11 ` Jean Pihet
2012-10-31 21:11 ` Jean Pihet
2012-11-01 2:44 ` Paul Walmsley
2012-11-01 2:44 ` Paul Walmsley
2012-11-01 7:51 ` Jean Pihet
2012-11-01 7:51 ` Jean Pihet
2012-11-03 21:39 ` Jean Pihet
2012-11-03 21:39 ` Jean Pihet
2012-11-05 3:15 ` Paul Walmsley
2012-11-05 3:15 ` Paul Walmsley
2012-11-05 9:26 ` Jean Pihet
2012-11-05 9:26 ` Jean Pihet
2012-11-06 0:01 ` Kevin Hilman
2012-11-06 0:01 ` Kevin Hilman
2012-11-06 9:27 ` Jean Pihet
2012-11-06 9:27 ` Jean Pihet
2012-11-06 16:13 ` Paul Walmsley
2012-11-06 16:13 ` Paul Walmsley
2012-11-06 10:06 ` Shubhrajyoti Datta
2012-11-06 10:06 ` Shubhrajyoti Datta
2012-10-30 4:16 ` Paul Walmsley
2012-10-30 4:16 ` Paul Walmsley
2012-10-30 8:54 ` Jean Pihet
2012-10-30 8:54 ` Jean Pihet
2012-10-30 11:23 ` Paul Walmsley
2012-10-30 11:23 ` Paul Walmsley
2012-10-31 8:18 ` Jean Pihet
2012-10-31 8:18 ` Jean Pihet
2012-10-23 19:17 ` Kevin Hilman [this message]
2012-10-23 19:17 ` Kevin Hilman
2012-10-23 19:18 ` Paul Walmsley
2012-10-23 19:18 ` Paul Walmsley
2012-10-20 6:24 ` Paul Walmsley
2012-10-20 6:24 ` Paul Walmsley
2012-10-22 17:53 ` Kevin Hilman
2012-10-22 17:53 ` Kevin Hilman
2012-10-22 20:44 ` Kevin Hilman
2012-10-22 20:44 ` Kevin Hilman
2012-10-20 14:11 ` Richard Cochran
2012-10-20 14:11 ` Richard Cochran
2012-10-20 16:27 ` Paul Walmsley
2012-10-20 16:27 ` Paul Walmsley
2012-10-20 18:01 ` Richard Cochran
2012-10-20 18:01 ` Richard Cochran
2012-10-20 18:12 ` Paul Walmsley
2012-10-20 18:12 ` Paul Walmsley
2012-10-20 18:37 ` Richard Cochran
2012-10-20 18:37 ` Richard Cochran
2012-10-20 18:58 ` Paul Walmsley
2012-10-20 18:58 ` Paul Walmsley
2012-10-21 13:51 ` Matt Porter
2012-10-21 13:51 ` Matt Porter
2012-10-21 16:26 ` Paul Walmsley
2012-10-21 16:26 ` Paul Walmsley
2012-10-22 21:56 ` Kevin Hilman
2012-10-22 21:56 ` Kevin Hilman
2012-10-21 7:35 ` Richard Cochran
2012-10-21 7:35 ` Richard Cochran
2012-10-21 8:23 ` Paul Walmsley
2012-10-21 8:23 ` Paul Walmsley
2012-10-21 11:44 ` Richard Cochran
2012-10-21 11:44 ` Richard Cochran
2012-10-21 16:24 ` Paul Walmsley
2012-10-21 16:24 ` Paul Walmsley
2012-10-21 12:29 ` Mohammed, Afzal
2012-10-21 12:29 ` Mohammed, Afzal
2012-10-21 12:36 ` Richard Cochran
2012-10-21 12:36 ` Richard Cochran
2012-10-20 17:20 ` Paul Walmsley
2012-10-20 17:20 ` Paul Walmsley
2012-10-22 16:13 ` Tero Kristo
2012-10-22 16:13 ` Tero Kristo
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=87fw5580cc.fsf@deeprootsystems.com \
--to=khilman@deeprootsystems.com \
--cc=aaro.koskinen@iki.fi \
--cc=balbi@ti.com \
--cc=jean.pihet@newoldbits.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.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 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.