On Tue, Sep 29, 2015 at 12:03:10AM +0800, Gabriele Paoloni wrote: > From: gabriele paoloni <gabriele.paoloni@xxxxxxxxxx> > > Currently num-lanes is read in dw_pcie_host_init(). > For a SoC that performs the link-up operation in UEFI, num-lanes > is not needed in the DTS; however currently it has to be specified > to some random value otherwise dw_pcie_host_init will fail. > If the link is brought up in BIOS dw_pcie_setup_rc should not > be called. > This patch reworks dw_pcie_host_init() so that, if num-lanes is > not specified, it sets pp->lanes = 0. If later on dw_pcie_setup_rc > is called and pp->num_lanes is not set to a valid value, > dw_pcie_setup_rc will send an error message and return. > > This patch is a follow-up of > http://www.spinics.net/lists/linux-pci/msg44394.html > > Signed-off-by: Gabriele Paoloni <gabriele.paoloni@xxxxxxxxxx> Applied to pci/host-designware for v4.4 with changelog as follows, thanks, Gabriele! commit efe52075227729aa2a1ed55e34dd599b3bb0dbf1 Author: Gabriele Paoloni <gabriele.paoloni@xxxxxxxxxx> Date: Tue Sep 29 00:03:10 2015 +0800 PCI: designware: Make num-lanes an optional DT property Currently "num-lanes" is read in dw_pcie_host_init(), but it is only used if we call dw_pcie_setup_rc() while bringing up the link. If the link has already been brought up by firmware, we need not call dw_pcie_setup_rc(), and "num-lanes" is unnecessary. Only complain about "num-lanes" if we actually need it and we didn't find a valid value. [bhelgaas: changelog] Signed-off-by: Gabriele Paoloni <gabriele.paoloni@xxxxxxxxxx> Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
