PCI: dwc: Drop the .set_num_vectors() host op
authorRob Herring <robh@kernel.org>
Thu, 5 Nov 2020 21:11:50 +0000 (15:11 -0600)
committerLorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Thu, 19 Nov 2020 10:51:41 +0000 (10:51 +0000)
There's no reason for the .set_num_vectors() host op. Drivers needing a
non-default value can just initialize pcie_port.num_vectors directly.

Link: https://lore.kernel.org/r/20201105211159.1814485-8-robh@kernel.org
Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Acked-by: Jingoo Han <jingoohan1@gmail.com>
Cc: Gustavo Pimentel <gustavo.pimentel@synopsys.com>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Jonathan Hunter <jonathanh@nvidia.com>
Cc: linux-tegra@vger.kernel.org
drivers/pci/controller/dwc/pcie-designware-host.c
drivers/pci/controller/dwc/pcie-designware-plat.c
drivers/pci/controller/dwc/pcie-designware.h
drivers/pci/controller/dwc/pcie-tegra194.c

index 265a48f1a0aec5c6b83444a25fd56b98ec72d9ac..1bd6a9762426183a2cbc27337c877813e24862dc 100644 (file)
@@ -365,22 +365,11 @@ int dw_pcie_host_init(struct pcie_port *pp)
                pci->link_gen = of_pci_get_max_link_speed(np);
 
        if (pci_msi_enabled()) {
-               /*
-                * If a specific SoC driver needs to change the
-                * default number of vectors, it needs to implement
-                * the set_num_vectors callback.
-                */
-               if (!pp->ops->set_num_vectors) {
+               if (!pp->num_vectors) {
                        pp->num_vectors = MSI_DEF_NUM_VECTORS;
-               } else {
-                       pp->ops->set_num_vectors(pp);
-
-                       if (pp->num_vectors > MAX_MSI_IRQS ||
-                           pp->num_vectors == 0) {
-                               dev_err(dev,
-                                       "Invalid number of vectors\n");
-                               return -EINVAL;
-                       }
+               } else if (pp->num_vectors > MAX_MSI_IRQS) {
+                       dev_err(dev, "Invalid number of vectors\n");
+                       return -EINVAL;
                }
 
                if (!pp->ops->msi_host_init) {
index 562a05e07b1d5961045394bc903a3905782fbefb..13fede1d41572d2c6a4855db30a494f234caaa80 100644 (file)
@@ -44,14 +44,8 @@ static int dw_plat_pcie_host_init(struct pcie_port *pp)
        return 0;
 }
 
-static void dw_plat_set_num_vectors(struct pcie_port *pp)
-{
-       pp->num_vectors = MAX_MSI_IRQS;
-}
-
 static const struct dw_pcie_host_ops dw_plat_pcie_host_ops = {
        .host_init = dw_plat_pcie_host_init,
-       .set_num_vectors = dw_plat_set_num_vectors,
 };
 
 static int dw_plat_pcie_establish_link(struct dw_pcie *pci)
@@ -128,6 +122,7 @@ static int dw_plat_add_pcie_port(struct dw_plat_pcie *dw_plat_pcie,
                        return pp->msi_irq;
        }
 
+       pp->num_vectors = MAX_MSI_IRQS;
        pp->ops = &dw_plat_pcie_host_ops;
 
        ret = dw_pcie_host_init(pp);
index f9a20ce9ab9ab4ab53fd55f9ac48b6ab41402871..41e739a73fbd6f3eb2c79a71e874b692171cbbcd 100644 (file)
@@ -172,7 +172,6 @@ enum dw_pcie_device_mode {
 
 struct dw_pcie_host_ops {
        int (*host_init)(struct pcie_port *pp);
-       void (*set_num_vectors)(struct pcie_port *pp);
        int (*msi_host_init)(struct pcie_port *pp);
 };
 
index f8fca6794282215d852af27ab87203848f3986a0..5e2841f58700f63a791c4bf5646cdf907ceef7ab 100644 (file)
@@ -990,11 +990,6 @@ static int tegra_pcie_dw_link_up(struct dw_pcie *pci)
        return !!(val & PCI_EXP_LNKSTA_DLLLA);
 }
 
-static void tegra_pcie_set_msi_vec_num(struct pcie_port *pp)
-{
-       pp->num_vectors = MAX_MSI_IRQS;
-}
-
 static int tegra_pcie_dw_start_link(struct dw_pcie *pci)
 {
        struct tegra_pcie_dw *pcie = to_tegra_pcie(pci);
@@ -1019,7 +1014,6 @@ static const struct dw_pcie_ops tegra_dw_pcie_ops = {
 
 static struct dw_pcie_host_ops tegra_pcie_dw_host_ops = {
        .host_init = tegra_pcie_dw_host_init,
-       .set_num_vectors = tegra_pcie_set_msi_vec_num,
 };
 
 static void tegra_pcie_disable_phy(struct tegra_pcie_dw *pcie)
@@ -1995,6 +1989,7 @@ static int tegra_pcie_dw_probe(struct platform_device *pdev)
        pci->n_fts[1] = FTS_VAL;
 
        pp = &pci->pp;
+       pp->num_vectors = MAX_MSI_IRQS;
        pcie->dev = &pdev->dev;
        pcie->mode = (enum dw_pcie_device_mode)data->mode;