* [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig
@ 2016-03-31 0:35 Simon Horman
2016-04-05 18:10 ` Bjorn Helgaas
0 siblings, 1 reply; 4+ messages in thread
From: Simon Horman @ 2016-03-31 0:35 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Phil Edworthy, Magnus Damm, linux-pci, linux-renesas-soc,
Geert Uytterhoeven, Simon Horman
It appears that Gen2 is a misnomer for the R-Car PCIE driver
which also supports Gen 1 and Gen 3 SoCs. Accordingly, drop Gen 2
from the help text and Kconfig symbol.
Also, re-arange the Kconfig symbol name to use PCIE as the prefix.
This appears to be in keeping with other PCIE Kconfig symbols.
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
Bjorn, if this patch is acceptable I would like to request that we
co-ordinate things so that I can queue-up corresponding updates for
arm and arm64 defconfigs in the same release. From my point of view
v4.7 would make sense.
v2
* As suggested by Geert Uytterhoeven:
- Use PCIE as prefix in Kconfig symbol
---
drivers/pci/host/Kconfig | 4 ++--
drivers/pci/host/Makefile | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
index 7a0780d56d2d..8fb1cf54617d 100644
--- a/drivers/pci/host/Kconfig
+++ b/drivers/pci/host/Kconfig
@@ -69,11 +69,11 @@ config PCI_RCAR_GEN2
There are 3 internal PCI controllers available with a single
built-in EHCI/OHCI host controller present on each one.
-config PCI_RCAR_GEN2_PCIE
+config PCIE_RCAR
bool "Renesas R-Car PCIe controller"
depends on ARCH_RENESAS || (ARM && COMPILE_TEST)
help
- Say Y here if you want PCIe controller support on R-Car Gen2 SoCs.
+ Say Y here if you want PCIe controller support on R-Car SoCs.
config PCI_HOST_COMMON
bool
diff --git a/drivers/pci/host/Makefile b/drivers/pci/host/Makefile
index d85b5faf9bbc..d3d8e1b36fb9 100644
--- a/drivers/pci/host/Makefile
+++ b/drivers/pci/host/Makefile
@@ -7,7 +7,7 @@ obj-$(CONFIG_PCI_HYPERV) += pci-hyperv.o
obj-$(CONFIG_PCI_MVEBU) += pci-mvebu.o
obj-$(CONFIG_PCI_TEGRA) += pci-tegra.o
obj-$(CONFIG_PCI_RCAR_GEN2) += pci-rcar-gen2.o
-obj-$(CONFIG_PCI_RCAR_GEN2_PCIE) += pcie-rcar.o
+obj-$(CONFIG_PCIE_RCAR) += pcie-rcar.o
obj-$(CONFIG_PCI_HOST_COMMON) += pci-host-common.o
obj-$(CONFIG_PCI_HOST_GENERIC) += pci-host-generic.o
obj-$(CONFIG_PCIE_SPEAR13XX) += pcie-spear13xx.o
--
2.1.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig
2016-03-31 0:35 [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig Simon Horman
@ 2016-04-05 18:10 ` Bjorn Helgaas
2016-04-19 21:21 ` Bjorn Helgaas
0 siblings, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2016-04-05 18:10 UTC (permalink / raw)
To: Simon Horman
Cc: Bjorn Helgaas, Phil Edworthy, Magnus Damm, linux-pci,
linux-renesas-soc, Geert Uytterhoeven
Hi Simon,
On Thu, Mar 31, 2016 at 09:35:23AM +0900, Simon Horman wrote:
> It appears that Gen2 is a misnomer for the R-Car PCIE driver
> which also supports Gen 1 and Gen 3 SoCs. Accordingly, drop Gen 2
> from the help text and Kconfig symbol.
>
> Also, re-arange the Kconfig symbol name to use PCIE as the prefix.
> This appears to be in keeping with other PCIE Kconfig symbols.
>
> Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> Bjorn, if this patch is acceptable I would like to request that we
> co-ordinate things so that I can queue-up corresponding updates for
> arm and arm64 defconfigs in the same release. From my point of view
> v4.7 would make sense.
I'm fine with this. What sort of coordination do you want to do?
It would probably be nice to combine this patch with the defconfig updates
so we don't have a bisection problem. If you want to include this patch
and merge through another tree,
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
> v2
> * As suggested by Geert Uytterhoeven:
> - Use PCIE as prefix in Kconfig symbol
> ---
> drivers/pci/host/Kconfig | 4 ++--
> drivers/pci/host/Makefile | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
> index 7a0780d56d2d..8fb1cf54617d 100644
> --- a/drivers/pci/host/Kconfig
> +++ b/drivers/pci/host/Kconfig
> @@ -69,11 +69,11 @@ config PCI_RCAR_GEN2
> There are 3 internal PCI controllers available with a single
> built-in EHCI/OHCI host controller present on each one.
>
> -config PCI_RCAR_GEN2_PCIE
> +config PCIE_RCAR
> bool "Renesas R-Car PCIe controller"
> depends on ARCH_RENESAS || (ARM && COMPILE_TEST)
> help
> - Say Y here if you want PCIe controller support on R-Car Gen2 SoCs.
> + Say Y here if you want PCIe controller support on R-Car SoCs.
>
> config PCI_HOST_COMMON
> bool
> diff --git a/drivers/pci/host/Makefile b/drivers/pci/host/Makefile
> index d85b5faf9bbc..d3d8e1b36fb9 100644
> --- a/drivers/pci/host/Makefile
> +++ b/drivers/pci/host/Makefile
> @@ -7,7 +7,7 @@ obj-$(CONFIG_PCI_HYPERV) += pci-hyperv.o
> obj-$(CONFIG_PCI_MVEBU) += pci-mvebu.o
> obj-$(CONFIG_PCI_TEGRA) += pci-tegra.o
> obj-$(CONFIG_PCI_RCAR_GEN2) += pci-rcar-gen2.o
> -obj-$(CONFIG_PCI_RCAR_GEN2_PCIE) += pcie-rcar.o
> +obj-$(CONFIG_PCIE_RCAR) += pcie-rcar.o
> obj-$(CONFIG_PCI_HOST_COMMON) += pci-host-common.o
> obj-$(CONFIG_PCI_HOST_GENERIC) += pci-host-generic.o
> obj-$(CONFIG_PCIE_SPEAR13XX) += pcie-spear13xx.o
> --
> 2.1.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig
2016-04-05 18:10 ` Bjorn Helgaas
@ 2016-04-19 21:21 ` Bjorn Helgaas
2016-04-21 3:53 ` Simon Horman
0 siblings, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2016-04-19 21:21 UTC (permalink / raw)
To: Simon Horman
Cc: Bjorn Helgaas, Phil Edworthy, Magnus Damm, linux-pci,
linux-renesas-soc, Geert Uytterhoeven
On Tue, Apr 05, 2016 at 01:10:54PM -0500, Bjorn Helgaas wrote:
> Hi Simon,
>
> On Thu, Mar 31, 2016 at 09:35:23AM +0900, Simon Horman wrote:
> > It appears that Gen2 is a misnomer for the R-Car PCIE driver
> > which also supports Gen 1 and Gen 3 SoCs. Accordingly, drop Gen 2
> > from the help text and Kconfig symbol.
> >
> > Also, re-arange the Kconfig symbol name to use PCIE as the prefix.
> > This appears to be in keeping with other PCIE Kconfig symbols.
> >
> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > Bjorn, if this patch is acceptable I would like to request that we
> > co-ordinate things so that I can queue-up corresponding updates for
> > arm and arm64 defconfigs in the same release. From my point of view
> > v4.7 would make sense.
>
> I'm fine with this. What sort of coordination do you want to do?
> It would probably be nice to combine this patch with the defconfig updates
> so we don't have a bisection problem. If you want to include this patch
> and merge through another tree,
>
> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
Ping, do you want me to do anything with this?
> > v2
> > * As suggested by Geert Uytterhoeven:
> > - Use PCIE as prefix in Kconfig symbol
> > ---
> > drivers/pci/host/Kconfig | 4 ++--
> > drivers/pci/host/Makefile | 2 +-
> > 2 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
> > index 7a0780d56d2d..8fb1cf54617d 100644
> > --- a/drivers/pci/host/Kconfig
> > +++ b/drivers/pci/host/Kconfig
> > @@ -69,11 +69,11 @@ config PCI_RCAR_GEN2
> > There are 3 internal PCI controllers available with a single
> > built-in EHCI/OHCI host controller present on each one.
> >
> > -config PCI_RCAR_GEN2_PCIE
> > +config PCIE_RCAR
> > bool "Renesas R-Car PCIe controller"
> > depends on ARCH_RENESAS || (ARM && COMPILE_TEST)
> > help
> > - Say Y here if you want PCIe controller support on R-Car Gen2 SoCs.
> > + Say Y here if you want PCIe controller support on R-Car SoCs.
> >
> > config PCI_HOST_COMMON
> > bool
> > diff --git a/drivers/pci/host/Makefile b/drivers/pci/host/Makefile
> > index d85b5faf9bbc..d3d8e1b36fb9 100644
> > --- a/drivers/pci/host/Makefile
> > +++ b/drivers/pci/host/Makefile
> > @@ -7,7 +7,7 @@ obj-$(CONFIG_PCI_HYPERV) += pci-hyperv.o
> > obj-$(CONFIG_PCI_MVEBU) += pci-mvebu.o
> > obj-$(CONFIG_PCI_TEGRA) += pci-tegra.o
> > obj-$(CONFIG_PCI_RCAR_GEN2) += pci-rcar-gen2.o
> > -obj-$(CONFIG_PCI_RCAR_GEN2_PCIE) += pcie-rcar.o
> > +obj-$(CONFIG_PCIE_RCAR) += pcie-rcar.o
> > obj-$(CONFIG_PCI_HOST_COMMON) += pci-host-common.o
> > obj-$(CONFIG_PCI_HOST_GENERIC) += pci-host-generic.o
> > obj-$(CONFIG_PCIE_SPEAR13XX) += pcie-spear13xx.o
> > --
> > 2.1.4
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig
2016-04-19 21:21 ` Bjorn Helgaas
@ 2016-04-21 3:53 ` Simon Horman
0 siblings, 0 replies; 4+ messages in thread
From: Simon Horman @ 2016-04-21 3:53 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Bjorn Helgaas, Phil Edworthy, Magnus Damm, linux-pci,
linux-renesas-soc, Geert Uytterhoeven
On Tue, Apr 19, 2016 at 04:21:14PM -0500, Bjorn Helgaas wrote:
> On Tue, Apr 05, 2016 at 01:10:54PM -0500, Bjorn Helgaas wrote:
> > Hi Simon,
> >
> > On Thu, Mar 31, 2016 at 09:35:23AM +0900, Simon Horman wrote:
> > > It appears that Gen2 is a misnomer for the R-Car PCIE driver
> > > which also supports Gen 1 and Gen 3 SoCs. Accordingly, drop Gen 2
> > > from the help text and Kconfig symbol.
> > >
> > > Also, re-arange the Kconfig symbol name to use PCIE as the prefix.
> > > This appears to be in keeping with other PCIE Kconfig symbols.
> > >
> > > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > > ---
> > > Bjorn, if this patch is acceptable I would like to request that we
> > > co-ordinate things so that I can queue-up corresponding updates for
> > > arm and arm64 defconfigs in the same release. From my point of view
> > > v4.7 would make sense.
> >
> > I'm fine with this. What sort of coordination do you want to do?
> > It would probably be nice to combine this patch with the defconfig updates
> > so we don't have a bisection problem. If you want to include this patch
> > and merge through another tree,
> >
> > Acked-by: Bjorn Helgaas <bhelgaas@google.com>
>
> Ping, do you want me to do anything with this?
Sorry, this had completely slipped my mind.
I have just posted
"[PATCH v3] PCI: rcar-pcie: Remove Gen2 designation from Kconfig"
which I am planning to take through the renesas tree.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-04-21 3:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-31 0:35 [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig Simon Horman
2016-04-05 18:10 ` Bjorn Helgaas
2016-04-19 21:21 ` Bjorn Helgaas
2016-04-21 3:53 ` Simon Horman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).