video: fbdev: nvidia: deprecate pci_get_bus_and_slot()
authorSinan Kaya <okaya@codeaurora.org>
Tue, 19 Dec 2017 05:37:59 +0000 (00:37 -0500)
committerBjorn Helgaas <bhelgaas@google.com>
Wed, 17 Jan 2018 14:16:46 +0000 (08:16 -0600)
pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as
where a PCI device is present. This restricts the device drivers to be
reused for other domain numbers.

Getting ready to remove pci_get_bus_and_slot() function in favor of
pci_get_domain_bus_and_slot().

struct nvidia_par has a pointer to struct pci_dev. Use the pci_dev
member to extract the domain information and pass it to
pci_get_domain_bus_and_slot() function.

Signed-off-by: Sinan Kaya <okaya@codeaurora.org>
Signed-off-by: Bjorn Helgaas <helgaas@kernel.org>
Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
drivers/video/fbdev/nvidia/nv_hw.c
drivers/video/fbdev/nvidia/nv_setup.c

index 81c80ac3c76f5c8780332621cd1703e7dca29716..8335da4ca30e28dbcd5fa7fb96bdbd91db26de04 100644 (file)
@@ -683,10 +683,11 @@ static void nForceUpdateArbitrationSettings(unsigned VClk,
        nv10_sim_state sim_data;
        unsigned int M, N, P, pll, MClk, NVClk, memctrl;
        struct pci_dev *dev;
+       int domain = pci_domain_nr(par->pci_dev->bus);
 
        if ((par->Chipset & 0x0FF0) == 0x01A0) {
                unsigned int uMClkPostDiv;
-               dev = pci_get_bus_and_slot(0, 3);
+               dev = pci_get_domain_bus_and_slot(domain, 0, 3);
                pci_read_config_dword(dev, 0x6C, &uMClkPostDiv);
                uMClkPostDiv = (uMClkPostDiv >> 8) & 0xf;
 
@@ -694,7 +695,7 @@ static void nForceUpdateArbitrationSettings(unsigned VClk,
                        uMClkPostDiv = 4;
                MClk = 400000 / uMClkPostDiv;
        } else {
-               dev = pci_get_bus_and_slot(0, 5);
+               dev = pci_get_domain_bus_and_slot(domain, 0, 5);
                pci_read_config_dword(dev, 0x4c, &MClk);
                MClk /= 1000;
        }
@@ -707,13 +708,13 @@ static void nForceUpdateArbitrationSettings(unsigned VClk,
        sim_data.pix_bpp = (char)pixelDepth;
        sim_data.enable_video = 0;
        sim_data.enable_mp = 0;
-       dev = pci_get_bus_and_slot(0, 1);
+       dev = pci_get_domain_bus_and_slot(domain, 0, 1);
        pci_read_config_dword(dev, 0x7C, &sim_data.memory_type);
        pci_dev_put(dev);
        sim_data.memory_type = (sim_data.memory_type >> 12) & 1;
        sim_data.memory_width = 64;
 
-       dev = pci_get_bus_and_slot(0, 3);
+       dev = pci_get_domain_bus_and_slot(domain, 0, 3);
        pci_read_config_dword(dev, 0, &memctrl);
        pci_dev_put(dev);
        memctrl >>= 16;
@@ -721,7 +722,7 @@ static void nForceUpdateArbitrationSettings(unsigned VClk,
        if ((memctrl == 0x1A9) || (memctrl == 0x1AB) || (memctrl == 0x1ED)) {
                u32 dimm[3];
 
-               dev = pci_get_bus_and_slot(0, 2);
+               dev = pci_get_domain_bus_and_slot(domain, 0, 2);
                pci_read_config_dword(dev, 0x40, &dimm[0]);
                dimm[0] = (dimm[0] >> 8) & 0x4f;
                pci_read_config_dword(dev, 0x44, &dimm[1]);
index 2f2e162134fabe193add5d945cfc69a725aa6f03..b17acd2903608528410011c9b44fd4996153136e 100644 (file)
@@ -264,7 +264,8 @@ static void nv10GetConfig(struct nvidia_par *par)
        }
 #endif
 
-       dev = pci_get_bus_and_slot(0, 1);
+       dev = pci_get_domain_bus_and_slot(pci_domain_nr(par->pci_dev->bus),
+                                         0, 1);
        if ((par->Chipset & 0xffff) == 0x01a0) {
                u32 amt;