On 10/28/20, 4:47 PM, Rob Herring wrote: > > All the DWC drivers do link setup and checks at roughly the same time. > Let's use the existing .start_link() hook (currently only used in EP > mode) and move the link handling to the core code. > > The behavior for a link down was inconsistent as some drivers would fail > probe in that case while others succeed. Let's standardize this to > succeed as there are usecases where devices (and the link) appear later > even without hotplug. For example, a reconfigured FPGA device. > > Cc: Kishon Vijay Abraham I <kis...@ti.com> > Cc: Lorenzo Pieralisi <lorenzo.pieral...@arm.com> > Cc: Bjorn Helgaas <bhelg...@google.com> > Cc: Jingoo Han <jingooh...@gmail.com>
Acked-by: Jingoo Han <jingooh...@gmail.com> Best regards, Jingoo Han > Cc: Kukjin Kim <kg...@kernel.org> > Cc: Krzysztof Kozlowski <k...@kernel.org> > Cc: Richard Zhu <hongxing....@nxp.com> > Cc: Lucas Stach <l.st...@pengutronix.de> > Cc: Shawn Guo <shawn...@kernel.org> > Cc: Sascha Hauer <s.ha...@pengutronix.de> > Cc: Pengutronix Kernel Team <ker...@pengutronix.de> > Cc: Fabio Estevam <feste...@gmail.com> > Cc: NXP Linux Team <linux-...@nxp.com> > Cc: Murali Karicheri <m-kariche...@ti.com> > Cc: Yue Wang <yue.w...@amlogic.com> > Cc: Kevin Hilman <khil...@baylibre.com> > Cc: Neil Armstrong <narmstr...@baylibre.com> > Cc: Jerome Brunet <jbru...@baylibre.com> > Cc: Martin Blumenstingl <martin.blumensti...@googlemail.com> > Cc: Thomas Petazzoni <thomas.petazz...@bootlin.com> > Cc: Jesper Nilsson <jesper.nils...@axis.com> > Cc: Gustavo Pimentel <gustavo.pimen...@synopsys.com> > Cc: Xiaowei Song <songxiao...@hisilicon.com> > Cc: Binghui Wang <wangbing...@hisilicon.com> > Cc: Andy Gross <agr...@kernel.org> > Cc: Bjorn Andersson <bjorn.anders...@linaro.org> > Cc: Stanimir Varbanov <svarba...@mm-sol.com> > Cc: Pratyush Anand <pratyush.an...@gmail.com> > Cc: Thierry Reding <thierry.red...@gmail.com> > Cc: Jonathan Hunter <jonath...@nvidia.com> > Cc: Kunihiko Hayashi <hayashi.kunih...@socionext.com> > Cc: Masahiro Yamada <yamada.masah...@socionext.com> > Cc: linux-o...@vger.kernel.org > Cc: linux-samsung-...@vger.kernel.org > Cc: linux-amlo...@lists.infradead.org > Cc: linux-arm-ker...@axis.com > Cc: linux-arm-...@vger.kernel.org > Cc: linux-te...@vger.kernel.org > Signed-off-by: Rob Herring <r...@kernel.org> > --- > drivers/pci/controller/dwc/pci-dra7xx.c | 2 - > drivers/pci/controller/dwc/pci-exynos.c | 41 +++++++---------- > drivers/pci/controller/dwc/pci-imx6.c | 9 ++-- > drivers/pci/controller/dwc/pci-keystone.c | 9 ---- > drivers/pci/controller/dwc/pci-meson.c | 24 ++++------ > drivers/pci/controller/dwc/pcie-armada8k.c | 39 +++++++--------- > drivers/pci/controller/dwc/pcie-artpec6.c | 2 - > .../pci/controller/dwc/pcie-designware-host.c | 9 ++++ > .../pci/controller/dwc/pcie-designware-plat.c | 3 -- > drivers/pci/controller/dwc/pcie-histb.c | 34 +++++++------- > drivers/pci/controller/dwc/pcie-kirin.c | 23 ++-------- > drivers/pci/controller/dwc/pcie-qcom.c | 19 ++------ > drivers/pci/controller/dwc/pcie-spear13xx.c | 46 ++++++++----------- > drivers/pci/controller/dwc/pcie-tegra194.c | 1 - > drivers/pci/controller/dwc/pcie-uniphier.c | 13 ++---- > 15 files changed, 103 insertions(+), 171 deletions(-) [...]
