[PATCH 1/1] mcb: Remove requesting memory region to avoid memory overlapping

From: Rodríguez Barbarin, José Javier
Date: Thu Mar 23 2023 - 08:50:05 EST


mcb-pci requests a fixed-size memory region to parse the chameleon
table, however, if the chameleon table is smaller that the allocated
region, it could overlaps with the IP Cores' memory regions.

There is no need to request the memory region for only parsing the
chameleon table as it is read-only. It is sufficient to only remaps
it when parsing.

Co-developed-by: Jorge Sanjuan Garcia <jorge.sanjuangarcia@xxxxxxxxxx>
Signed-off-by: Jorge Sanjuan Garcia <jorge.sanjuangarcia@xxxxxxxxxx>
Signed-off-by: Javier Rodriguez <josejavier.rodriguez@xxxxxxxxxx>
---
drivers/mcb/mcb-pci.c | 10 ----------
1 file changed, 10 deletions(-)

diff --git a/drivers/mcb/mcb-pci.c b/drivers/mcb/mcb-pci.c
index dc88232d9af8..e98f626020a7 100644
--- a/drivers/mcb/mcb-pci.c
+++ b/drivers/mcb/mcb-pci.c
@@ -29,7 +29,6 @@ static int mcb_pci_get_irq(struct mcb_device *mdev)

static int mcb_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id)
{
- struct resource *res;
struct priv *priv;
int ret;
unsigned long flags;
@@ -52,15 +51,6 @@ static int mcb_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id)
goto out_disable;
}

- res = devm_request_mem_region(&pdev->dev, priv->mapbase,
- CHAM_HEADER_SIZE,
- KBUILD_MODNAME);
- if (!res) {
- dev_err(&pdev->dev, "Failed to request PCI memory\n");
- ret = -EBUSY;
- goto out_disable;
- }
-
priv->base = devm_ioremap(&pdev->dev, priv->mapbase, CHAM_HEADER_SIZE);
if (!priv->base) {
dev_err(&pdev->dev, "Cannot ioremap\n");
--
2.34.1