From mboxrd@z Thu Jan 1 00:00:00 1970 From: arno@natisbad.org (Arnaud Ebalard) Date: Wed, 06 Nov 2013 19:08:04 +0100 Subject: [PATCH 1/2] ARM: mvebu: second PCIe unit of Armada XP mv78230 is only x1 capable In-Reply-To: <20131106154326.27a4cc27@skate> (Thomas Petazzoni's message of "Wed, 6 Nov 2013 15:43:26 +0100") References: <20131106154326.27a4cc27@skate> Message-ID: <877gclcsjf.fsf@natisbad.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Thomas Petazzoni writes: > On Tue, 05 Nov 2013 21:45:48 +0100, Arnaud Ebalard wrote: >> Various Marvell datasheets advertise second PCIe unit of mv78230 >> flavour of Armada XP as x4/quad x1 capable. This second unit is in >> fact only x1 capable. This patch fixes current mv78230 .dtsi to >> reflect that, i.e. makes 1.0 the second interface (instead of 2.0 >> at the moment). This was successfully tested on a mv78230-based >> ReadyNAS 2120 platform with a x1 device (FL1009 XHCI controller) >> connected to this second interface. >> >> Signed-off-by: Arnaud Ebalard >> --- >> arch/arm/boot/dts/armada-xp-mv78230.dtsi | 24 ++++++++++++------------ >> 1 file changed, 12 insertions(+), 12 deletions(-) > > Acked-by: Thomas Petazzoni Thanks for the review of both patches, Thomas. And no relation, but thanks for 714086029116b6 ;-) > This is actually a bug fix, and the problem exists since commit > 9d8f44f02d4a5f6e7b8d138ea8f8c6e30ae6e1a3, and therefore in 3.10, 3.11 > and 3.12. > > However, while the PCIe DT stuff was merged in 3.10, the actual driver > itself was only merged in 3.11, so in 3.10, there is no chance to hit > the bug. > > Therefore, having this fix pushed to the stable 3.11.x and 3.12.x trees > would be good. Jason, any action required on my side regarding the relay of the two patches to stable team or will you handle that once they are in your tree (or Linus one)? Cheers, a+