Re: [PATCH v2] PCI: rcar-pcie: Remove Gen2 designation from Kconfig

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
> > ---
> > 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@xxxxxxxxxx>

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@xxxxxxxxxxxxxxx
> > 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@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux