From: Andrew Lunn <andrew@lunn.ch>
To: Jason Cooper <jason@lakedaemon.net>
Cc: Simon Guinot <simon.guinot@sequanux.org>,
Jeff Garzik <jgarzik@redhat.com>, Tejun Heo <tj@kernel.org>,
linux-ide@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
Lior Amsalem <alior@marvell.com>,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Andrew Lunn <andrew@lunn.ch>,
Gregory Clement <gregory.clement@free-electrons.com>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
stable@vger.kernel.org
Subject: Re: [PATCH] ata: sata_mv: setting PHY speed according to SControl speed
Date: Wed, 25 Dec 2013 17:41:19 +0100 [thread overview]
Message-ID: <20131225164119.GC31584@lunn.ch> (raw)
In-Reply-To: <20131224194603.GE19878@titan.lakedaemon.net>
On Tue, Dec 24, 2013 at 02:46:03PM -0500, Jason Cooper wrote:
> On Mon, Dec 23, 2013 at 01:07:35PM +0100, Simon Guinot wrote:
> > From: Lior Amsalem <alior@marvell.com>
> >
> > From: Lior Amsalem <alior@marvell.com>
> >
> > This patch fixes a SATA hotplug issue on the Armada 370 and Armada XP
> > SoCs. Without it, if a disk is unplugged from a SATA port, then further
> > hotplug notification are now longer received on this port.
> >
> > This should be applied to every -stable kernel supporting Armada SoCs.
>
> Could we get a little more specific here? Please determine which commit
> introduced the regression and note it with 'Fixes: <commitish> "oneline"'
>
> It's really needed here since the sata_mv driver predates the Armada
> SoCs introduction. Is it possible Kirkwood et al also experience this
> problem?
Hi Jason
It is on my TODO list to try this out on Kirkwood and Dove. I'm pretty
sure replug works on Dove without this, not tried Kirkwood yet.
Andrew
WARNING: multiple messages have this Message-ID (diff)
From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ata: sata_mv: setting PHY speed according to SControl speed
Date: Wed, 25 Dec 2013 17:41:19 +0100 [thread overview]
Message-ID: <20131225164119.GC31584@lunn.ch> (raw)
In-Reply-To: <20131224194603.GE19878@titan.lakedaemon.net>
On Tue, Dec 24, 2013 at 02:46:03PM -0500, Jason Cooper wrote:
> On Mon, Dec 23, 2013 at 01:07:35PM +0100, Simon Guinot wrote:
> > From: Lior Amsalem <alior@marvell.com>
> >
> > From: Lior Amsalem <alior@marvell.com>
> >
> > This patch fixes a SATA hotplug issue on the Armada 370 and Armada XP
> > SoCs. Without it, if a disk is unplugged from a SATA port, then further
> > hotplug notification are now longer received on this port.
> >
> > This should be applied to every -stable kernel supporting Armada SoCs.
>
> Could we get a little more specific here? Please determine which commit
> introduced the regression and note it with 'Fixes: <commitish> "oneline"'
>
> It's really needed here since the sata_mv driver predates the Armada
> SoCs introduction. Is it possible Kirkwood et al also experience this
> problem?
Hi Jason
It is on my TODO list to try this out on Kirkwood and Dove. I'm pretty
sure replug works on Dove without this, not tried Kirkwood yet.
Andrew
next prev parent reply other threads:[~2013-12-25 16:41 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-23 12:07 [PATCH] ata: sata_mv: setting PHY speed according to SControl speed Simon Guinot
2013-12-23 12:07 ` Simon Guinot
2013-12-24 19:46 ` Jason Cooper
2013-12-24 19:46 ` Jason Cooper
2013-12-25 16:41 ` Andrew Lunn [this message]
2013-12-25 16:41 ` Andrew Lunn
2013-12-25 22:40 ` Simon Guinot
2013-12-25 22:40 ` Simon Guinot
2013-12-26 7:53 ` Thomas Petazzoni
2013-12-26 7:53 ` Thomas Petazzoni
2013-12-26 11:54 ` Simon Guinot
2013-12-26 11:54 ` Simon Guinot
2013-12-26 13:34 ` Thomas Petazzoni
2013-12-26 13:34 ` Thomas Petazzoni
2013-12-27 15:49 ` Jason Cooper
2013-12-27 15:49 ` Jason Cooper
2013-12-26 18:01 ` Andrew Lunn
2013-12-26 18:01 ` Andrew Lunn
2013-12-27 17:37 ` Simon Guinot
2013-12-27 17:37 ` Simon Guinot
2013-12-31 12:12 ` Tejun Heo
2013-12-31 12:12 ` Tejun Heo
2013-12-31 17:05 ` Andrew Lunn
2013-12-31 17:05 ` Andrew Lunn
2014-01-08 13:45 ` Lior Amsalem
2014-01-08 13:45 ` Lior Amsalem
2014-01-10 17:44 ` Jason Cooper
2014-01-10 17:44 ` Jason Cooper
2014-01-10 23:55 ` Thomas Petazzoni
2014-01-10 23:55 ` Thomas Petazzoni
2014-01-13 14:36 ` Jason Cooper
2014-01-13 14:36 ` Jason Cooper
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=20131225164119.GC31584@lunn.ch \
--to=andrew@lunn.ch \
--cc=alior@marvell.com \
--cc=gregory.clement@free-electrons.com \
--cc=jason@lakedaemon.net \
--cc=jgarzik@redhat.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-ide@vger.kernel.org \
--cc=sebastian.hesselbarth@gmail.com \
--cc=simon.guinot@sequanux.org \
--cc=stable@vger.kernel.org \
--cc=thomas.petazzoni@free-electrons.com \
--cc=tj@kernel.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.