On Fri, Nov 18, 2016 at 06:21:41PM +0200, Kalle Valo wrote: > Amitkumar Karwar <akarwar@xxxxxxxxxxx> writes: > > > From: Xinming Hu <huxm@xxxxxxxxxxx> > > > > This patch derives device tree node from pcie bus layer framework. > > Device tree bindings file has been renamed(marvell-sd8xxx.txt -> > > marvell-8xxx.txt) to accommodate PCIe changes. > > > > Signed-off-by: Xinming Hu <huxm@xxxxxxxxxxx> > > Signed-off-by: Rajat Jain <rajatja@xxxxxxxxxx> > > Reviewed-by: Brian Norris <briannorris@xxxxxxxxxxxx> > > Signed-off-by: Amitkumar Karwar <akarwar@xxxxxxxxxxx> > > [...] > > > @@ -10,6 +10,8 @@ Required properties: > > - compatible : should be one of the following: > > * "marvell,sd8897" > > * "marvell,sd8997" > > + * "pci11ab,2b42" > > + * "pci1b4b,2b42" > > Are these correct? In v6 Rob said: Yes, Marvell has 2 vendor IDs it turns out. > > "I think I already said this, but you have the vendor and product IDs > reversed." > > https://patchwork.kernel.org/patch/9390225/ > > -- > Kalle Valo > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html