diff mbox series

[U-Boot,RFC,06/29] drivers: pci-uclass: add multi entry support for pci regions

Message ID 20191029210821.1954-7-suneelglinux@gmail.com
State RFC
Delegated to: Tom Rini
Headers show
Series arm: Introduce Marvell/Cavium OcteonTX | expand

Commit Message

Suneel Garapati Oct. 29, 2019, 9:07 p.m. UTC 
From: Suneel Garapati <sgarapati@marvell.com>

Enable PCI memory regions to be of multiple entry. This helps to
add support for SoC's like OcteonTX/TX2 where every peripheral is
on PCI bus.

Signed-off-by: Suneel Garapati <sgarapati@marvell.com>
---
 drivers/pci/Kconfig      | 9 +++++++++
 drivers/pci/pci-uclass.c | 2 ++
 2 files changed, 11 insertions(+)

Comments

Simon Glass Nov. 20, 2019, 3 a.m. UTC | #1 
Hi Suneel,

On Tue, 29 Oct 2019 at 14:08, Suneel Garapati <suneelglinux@gmail.com> wrote:
>
> From: Suneel Garapati <sgarapati@marvell.com>
>
> Enable PCI memory regions to be of multiple entry. This helps to
> add support for SoC's like OcteonTX/TX2 where every peripheral is
> on PCI bus.
>
> Signed-off-by: Suneel Garapati <sgarapati@marvell.com>
> ---
>  drivers/pci/Kconfig      | 9 +++++++++
>  drivers/pci/pci-uclass.c | 2 ++
>  2 files changed, 11 insertions(+)
>
> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
> index 19e7b50046..752439afcf 100644
> --- a/drivers/pci/Kconfig
> +++ b/drivers/pci/Kconfig
> @@ -43,6 +43,15 @@ config PCI_PNP
>         help
>           Enable PCI memory and I/O space resource allocation and assignment.
>
> +config PCI_REGION_MULTI_ENTRY
> +       bool "Enable Multiple entries of MEMORY region for PCI"
> +       depends on PCI || DM_PCI
> +       default n
> +       help
> +         Enable PCI memory regions to be of multiple entry. This helps to
> +         add support for SoC's like OcteonTX/TX2 where every peripheral is
> +         on PCI bus.

This isn't clear enough. Please expand to explain what 'multiple entry' means.

> +
>  config PCIE_ECAM_GENERIC
>         bool "Generic ECAM-based PCI host controller support"
>         default n
> diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c
> index 007b7472c6..f17e83565c 100644
> --- a/drivers/pci/pci-uclass.c
> +++ b/drivers/pci/pci-uclass.c
> @@ -902,10 +902,12 @@ static void decode_regions(struct pci_controller *hose, ofnode parent_node,
>                 }
>
>                 pos = -1;
> +#if !CONFIG_IS_ENABLED(PCI_REGION_MULTI_ENTRY)

Please use if()

>                 for (i = 0; i < hose->region_count; i++) {
>                         if (hose->regions[i].flags == type)
>                                 pos = i;
>                 }
> +#endif
>                 if (pos == -1)
>                         pos = hose->region_count++;
>                 debug(" - type=%d, pos=%d\n", type, pos);
> --
> 2.23.0
>

Regards,
Simon
diff mbox series

Patch

diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
index 19e7b50046..752439afcf 100644
--- a/drivers/pci/Kconfig
+++ b/drivers/pci/Kconfig
@@ -43,6 +43,15 @@  config PCI_PNP
 	help
 	  Enable PCI memory and I/O space resource allocation and assignment.
 
+config PCI_REGION_MULTI_ENTRY
+	bool "Enable Multiple entries of MEMORY region for PCI"
+	depends on PCI || DM_PCI
+	default n
+	help
+	  Enable PCI memory regions to be of multiple entry. This helps to
+	  add support for SoC's like OcteonTX/TX2 where every peripheral is
+	  on PCI bus.
+
 config PCIE_ECAM_GENERIC
 	bool "Generic ECAM-based PCI host controller support"
 	default n
diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c
index 007b7472c6..f17e83565c 100644
--- a/drivers/pci/pci-uclass.c
+++ b/drivers/pci/pci-uclass.c
@@ -902,10 +902,12 @@  static void decode_regions(struct pci_controller *hose, ofnode parent_node,
 		}
 
 		pos = -1;
+#if !CONFIG_IS_ENABLED(PCI_REGION_MULTI_ENTRY)
 		for (i = 0; i < hose->region_count; i++) {
 			if (hose->regions[i].flags == type)
 				pos = i;
 		}
+#endif
 		if (pos == -1)
 			pos = hose->region_count++;
 		debug(" - type=%d, pos=%d\n", type, pos);