From: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
To: Gregory CLEMENT
<gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
Cc: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>,
linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>,
Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
Thomas Petazzoni
<thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Ezequiel Garcia
<ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Sebastian Hesselbarth
<sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [RFC PATCH 0/3] Fix i2c bus hang on A0 version of the Armada XP SoCs
Date: Wed, 1 Jan 2014 20:28:16 +0100 [thread overview]
Message-ID: <20140101192816.GP32537@lunn.ch> (raw)
In-Reply-To: <1388508293-9141-1-git-send-email-gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
On Tue, Dec 31, 2013 at 05:44:50PM +0100, Gregory CLEMENT wrote:
> Hi,
>
> This series is an RFC to fix the i2c bus hang on A0 version of the
> Armada XP SoCs. It occured on the the eraly release of the OpenBlocks
> AX3-4 bords. Indeed the first variants of Armada XP SoCs (A0 stepping)
> have issues related to the i2c controller which prevent to use the
> offload mechanism and lead to a kernel hang during boot.
>
> The best solution will be to detect the SoCs version at runtime and
> use it in the driver, but I chosed to submit this version in case CPU
> version detection feature takes more time than expected (and we
> already reached the rc6 kernel).
>
> Thanks,
>
> Gregory
Hi Gregory
Thanks for these RFC patches. I can confirm they work on my Openblocks
device with an A0 stepping SoC. I can now access the RTC.
You missed adding the new DTS file to the Makefile. Depending on how
the discussion goes about auto detecting stepping or not, i would of
used different filenames, probably:
armada-xp-openblocks-ax3-4.dts
armada-xp-openblocks-ax3-4-a0.dts
armada-xp-openblocks-ax3-4-common.dts
Keeps all the files together when listed alphabetically
Thanks
Andrew
WARNING: multiple messages have this Message-ID (diff)
From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 0/3] Fix i2c bus hang on A0 version of the Armada XP SoCs
Date: Wed, 1 Jan 2014 20:28:16 +0100 [thread overview]
Message-ID: <20140101192816.GP32537@lunn.ch> (raw)
In-Reply-To: <1388508293-9141-1-git-send-email-gregory.clement@free-electrons.com>
On Tue, Dec 31, 2013 at 05:44:50PM +0100, Gregory CLEMENT wrote:
> Hi,
>
> This series is an RFC to fix the i2c bus hang on A0 version of the
> Armada XP SoCs. It occured on the the eraly release of the OpenBlocks
> AX3-4 bords. Indeed the first variants of Armada XP SoCs (A0 stepping)
> have issues related to the i2c controller which prevent to use the
> offload mechanism and lead to a kernel hang during boot.
>
> The best solution will be to detect the SoCs version at runtime and
> use it in the driver, but I chosed to submit this version in case CPU
> version detection feature takes more time than expected (and we
> already reached the rc6 kernel).
>
> Thanks,
>
> Gregory
Hi Gregory
Thanks for these RFC patches. I can confirm they work on my Openblocks
device with an A0 stepping SoC. I can now access the RTC.
You missed adding the new DTS file to the Makefile. Depending on how
the discussion goes about auto detecting stepping or not, i would of
used different filenames, probably:
armada-xp-openblocks-ax3-4.dts
armada-xp-openblocks-ax3-4-a0.dts
armada-xp-openblocks-ax3-4-common.dts
Keeps all the files together when listed alphabetically
Thanks
Andrew
next prev parent reply other threads:[~2014-01-01 19:28 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-31 16:44 [RFC PATCH 0/3] Fix i2c bus hang on A0 version of the Armada XP SoCs Gregory CLEMENT
2013-12-31 16:44 ` Gregory CLEMENT
[not found] ` <1388508293-9141-1-git-send-email-gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-12-31 16:44 ` [RFC PATCH 1/3] arm: mvebu: add new dts file for old variant of Openblocks AX3-4 Gregory CLEMENT
2013-12-31 16:44 ` Gregory CLEMENT
[not found] ` <1388508293-9141-2-git-send-email-gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-01-01 21:41 ` Jason Cooper
2014-01-01 21:41 ` Jason Cooper
[not found] ` <20140101214116.GY19878-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2014-01-01 21:42 ` Jason Cooper
2014-01-01 21:42 ` Jason Cooper
2014-01-02 16:17 ` Gregory CLEMENT
2014-01-02 16:17 ` Gregory CLEMENT
2013-12-31 16:44 ` [RFC PATCH 2/3] i2c: mv64xxx: Fix bus hang on A0 version of the Armada XP SoCs Gregory CLEMENT
2013-12-31 16:44 ` Gregory CLEMENT
2013-12-31 16:44 ` [RFC PATCH 3/3] i2c: mv64xxx: Document the newly introduced Armada XP A0 compatible Gregory CLEMENT
2013-12-31 16:44 ` Gregory CLEMENT
[not found] ` <1388508293-9141-4-git-send-email-gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-01-01 21:44 ` Jason Cooper
2014-01-01 21:44 ` Jason Cooper
2014-01-01 21:45 ` Jason Cooper
2014-01-01 21:45 ` Jason Cooper
[not found] ` <20140101214552.GB19878-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2014-01-02 16:34 ` Gregory CLEMENT
2014-01-02 16:34 ` Gregory CLEMENT
2014-01-01 19:28 ` Andrew Lunn [this message]
2014-01-01 19:28 ` [RFC PATCH 0/3] Fix i2c bus hang on A0 version of the Armada XP SoCs Andrew Lunn
[not found] ` <20140101192816.GP32537-g2DYL2Zd6BY@public.gmane.org>
2014-01-02 16:22 ` Gregory CLEMENT
2014-01-02 16:22 ` Gregory CLEMENT
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=20140101192816.GP32537@lunn.ch \
--to=andrew-g2dyl2zd6by@public.gmane.org \
--cc=ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org \
/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.