Hi Bjorn and Vidya,

[...]

quoted

+}
+
+struct pci_ecam_ops tegra194_pcie_ops = {
+	.bus_shift	= 20,

I think e7708f5b10e2 ("PCI: Unify ECAM constants in native PCI Express
drivers") means you don't need this .bus_shift.

[...]

Correct.  If this platform implements ECAM as per the specification,
then the .bus_shift initializer is no longer needed.

Krzysztof