On Thu, Dec 26, 2013 at 08:53:55AM +0100, Thomas Petazzoni wrote: > Dear Simon Guinot, > > On Wed, 25 Dec 2013 23:40:34 +0100, Simon Guinot wrote: > > > > > 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"' > > > > Well, since the DT support for the sata_mv driver precedes SATA > > support for Armada SoCs, I'd say that the bug has been introduced by: > > > > a6a6de1a "arm: mvebu: SATA support: SoC-level DT data for Armada > > 370/XP" > > > > Let me know if you agree with that. I will update the commit message > > accorgingly. > > In some sense, we could say that this is not a regression. According to > what you mean, SATA hotplug has *never* been working on Armada 370/XP. > So technically, it could be seen as a new feature for this platform, > and is therefore not a regression (i.e something that used to work, and > that no longer works). There has been no kernel release for which SATA > hotplug was working for Armada 370/XP. I agree with that. It is not correct to call this issue a regression. But, it would also be nice to have this patch applied in the stable branches 3.10 and onwards. Maybe, I could just add this information while CC'ing Linux stable ? Regards, Simon
Attachment:
signature.asc
Description: Digital signature