PCI: endpoint: pci-epf-vntb: Remove superfluous checks for pci_epf_alloc_space() API
authorNiklas Cassel <cassel@kernel.org>
Wed, 7 Feb 2024 21:39:17 +0000 (22:39 +0100)
committerManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Fri, 9 Feb 2024 17:10:23 +0000 (22:40 +0530)
Now that the checks are performed by the pci_epf_alloc_space() API, let's
remove the superfluous checks in this driver.

Signed-off-by: Niklas Cassel <cassel@kernel.org>
Reviewed-by: Frank Li <Frank.Li@nxp.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20240207213922.1796533-5-cassel@kernel.org
[mani: reworded the commit message]
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
drivers/pci/endpoint/functions/pci-epf-vntb.c

index ba509d6..eda4b90 100644 (file)
@@ -527,7 +527,6 @@ static int epf_ntb_configure_interrupt(struct epf_ntb *ntb)
 static int epf_ntb_db_bar_init(struct epf_ntb *ntb)
 {
        const struct pci_epc_features *epc_features;
-       u32 align;
        struct device *dev = &ntb->epf->dev;
        int ret;
        struct pci_epf_bar *epf_bar;
@@ -538,16 +537,6 @@ static int epf_ntb_db_bar_init(struct epf_ntb *ntb)
        epc_features = pci_epc_get_features(ntb->epf->epc,
                                            ntb->epf->func_no,
                                            ntb->epf->vfunc_no);
-       align = epc_features->align;
-
-       if (size < 128)
-               size = 128;
-
-       if (align)
-               size = ALIGN(size, align);
-       else
-               size = roundup_pow_of_two(size);
-
        barno = ntb->epf_ntb_bar[BAR_DB];
 
        mw_addr = pci_epf_alloc_space(ntb->epf, size, barno, epc_features, 0);