* [PATCH] PCI: imx6: Simplify Kconfig depends on
@ 2019-03-11 18:59 Leonard Crestez
2019-03-26 18:19 ` Lorenzo Pieralisi
2019-07-05 15:51 ` Lorenzo Pieralisi
0 siblings, 2 replies; 8+ messages in thread
From: Leonard Crestez @ 2019-03-11 18:59 UTC (permalink / raw)
To: Lucas Stach, Lorenzo Pieralisi
Cc: Trent Piepho, Andrey Smirnov, Stefan Agner, Richard Zhu,
dl-linux-imx, Shawn Guo, linux-pci
In theory this driver can be used on imx6sx without enabling support for
imx6q or imx7d but the "depends on" condition doesn't allow that.
Instead of making the condition even longer just make it depend on
"ARCH_MXC || COMPILE_TEST" instead.
Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
---
drivers/pci/controller/dwc/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
index 6ea74b1c0d94..21747fd0e799 100644
--- a/drivers/pci/controller/dwc/Kconfig
+++ b/drivers/pci/controller/dwc/Kconfig
@@ -88,11 +88,11 @@ config PCI_EXYNOS
depends on PCI_MSI_IRQ_DOMAIN
select PCIE_DW_HOST
config PCI_IMX6
bool "Freescale i.MX6/7/8 PCIe controller"
- depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
+ depends on ARCH_MXC || COMPILE_TEST
depends on PCI_MSI_IRQ_DOMAIN
select PCIE_DW_HOST
config PCIE_SPEAR13XX
bool "STMicroelectronics SPEAr PCIe controller"
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-03-11 18:59 [PATCH] PCI: imx6: Simplify Kconfig depends on Leonard Crestez
@ 2019-03-26 18:19 ` Lorenzo Pieralisi
2019-04-12 17:56 ` Leonard Crestez
2019-07-05 15:51 ` Lorenzo Pieralisi
1 sibling, 1 reply; 8+ messages in thread
From: Lorenzo Pieralisi @ 2019-03-26 18:19 UTC (permalink / raw)
To: Leonard Crestez
Cc: Lucas Stach, Trent Piepho, Andrey Smirnov, Stefan Agner,
Richard Zhu, dl-linux-imx, Shawn Guo, linux-pci
On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
> In theory this driver can be used on imx6sx without enabling support for
> imx6q or imx7d but the "depends on" condition doesn't allow that.
>
> Instead of making the condition even longer just make it depend on
> "ARCH_MXC || COMPILE_TEST" instead.
>
> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
> ---
> drivers/pci/controller/dwc/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> index 6ea74b1c0d94..21747fd0e799 100644
> --- a/drivers/pci/controller/dwc/Kconfig
> +++ b/drivers/pci/controller/dwc/Kconfig
> @@ -88,11 +88,11 @@ config PCI_EXYNOS
> depends on PCI_MSI_IRQ_DOMAIN
> select PCIE_DW_HOST
>
> config PCI_IMX6
> bool "Freescale i.MX6/7/8 PCIe controller"
> - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
> + depends on ARCH_MXC || COMPILE_TEST
> depends on PCI_MSI_IRQ_DOMAIN
> select PCIE_DW_HOST
If Lucas does not spot any problem with this patch I would request his
ACK to merge it, thanks.
Lorenzo
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-03-26 18:19 ` Lorenzo Pieralisi
@ 2019-04-12 17:56 ` Leonard Crestez
2019-04-12 18:03 ` Trent Piepho
0 siblings, 1 reply; 8+ messages in thread
From: Leonard Crestez @ 2019-04-12 17:56 UTC (permalink / raw)
To: Lucas Stach
Cc: Lorenzo Pieralisi, Trent Piepho, Andrey Smirnov, Stefan Agner,
Richard Zhu, dl-linux-imx, Shawn Guo, linux-pci
On 3/26/19 8:19 PM, Lorenzo Pieralisi wrote:
> On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
>> In theory this driver can be used on imx6sx without enabling support for
>> imx6q or imx7d but the "depends on" condition doesn't allow that.
>>
>> Instead of making the condition even longer just make it depend on
>> "ARCH_MXC || COMPILE_TEST" instead.
>>
>> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
>> ---
>> drivers/pci/controller/dwc/Kconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
>> index 6ea74b1c0d94..21747fd0e799 100644
>> --- a/drivers/pci/controller/dwc/Kconfig
>> +++ b/drivers/pci/controller/dwc/Kconfig
>> @@ -88,11 +88,11 @@ config PCI_EXYNOS
>> depends on PCI_MSI_IRQ_DOMAIN
>> select PCIE_DW_HOST
>>
>> config PCI_IMX6
>> bool "Freescale i.MX6/7/8 PCIe controller"
>> - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
>> + depends on ARCH_MXC || COMPILE_TEST
>> depends on PCI_MSI_IRQ_DOMAIN
>> select PCIE_DW_HOST
>
> If Lucas does not spot any problem with this patch I would request his
> ACK to merge it, thanks.
Lucas: Can you please ack this? It's a bit old but applies cleanly and
resend shouldn't be required.
--
Regards,
Leonard
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-04-12 17:56 ` Leonard Crestez
@ 2019-04-12 18:03 ` Trent Piepho
2019-04-12 18:20 ` Leonard Crestez
0 siblings, 1 reply; 8+ messages in thread
From: Trent Piepho @ 2019-04-12 18:03 UTC (permalink / raw)
To: l.stach, leonard.crestez
Cc: stefan, shawnguo, lorenzo.pieralisi, andrew.smirnov,
hongxing.zhu, linux-pci, linux-imx
On Fri, 2019-04-12 at 17:56 +0000, Leonard Crestez wrote:
> On 3/26/19 8:19 PM, Lorenzo Pieralisi wrote:
> > On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
> > > In theory this driver can be used on imx6sx without enabling support for
> > > imx6q or imx7d but the "depends on" condition doesn't allow that.
> > >
> > > Instead of making the condition even longer just make it depend on
> > > "ARCH_MXC || COMPILE_TEST" instead.
> > >
> > > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
> > > ---
> > > drivers/pci/controller/dwc/Kconfig | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> > > index 6ea74b1c0d94..21747fd0e799 100644
> > > --- a/drivers/pci/controller/dwc/Kconfig
> > > +++ b/drivers/pci/controller/dwc/Kconfig
> > > @@ -88,11 +88,11 @@ config PCI_EXYNOS
> > > depends on PCI_MSI_IRQ_DOMAIN
> > > select PCIE_DW_HOST
> > >
> > > config PCI_IMX6
> > > bool "Freescale i.MX6/7/8 PCIe controller"
> > > - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
> > > + depends on ARCH_MXC || COMPILE_TEST
> > > depends on PCI_MSI_IRQ_DOMAIN
> > > select PCIE_DW_HOST
> >
> > If Lucas does not spot any problem with this patch I would request his
> > ACK to merge it, thanks.
>
> Lucas: Can you please ack this? It's a bit old but applies cleanly and
> resend shouldn't be required.
The IMX7s does not have a PCI-e controller. The IMX7d does have one,
but not the "s".
Enabling the controller driver on an IMX SoC that doesn't have it is
not exactly the end of the world and having to keep adding new IMX SOCs
to the list here is annoying. So IMHO, it's ok to make this change
even if the controller is allowed on a superset of the SoCs that have
it.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-04-12 18:03 ` Trent Piepho
@ 2019-04-12 18:20 ` Leonard Crestez
2019-04-30 14:22 ` Lorenzo Pieralisi
0 siblings, 1 reply; 8+ messages in thread
From: Leonard Crestez @ 2019-04-12 18:20 UTC (permalink / raw)
To: Trent Piepho, l.stach
Cc: stefan, shawnguo, lorenzo.pieralisi, andrew.smirnov, Richard Zhu,
linux-pci, dl-linux-imx
On 4/12/2019 9:03 PM, Trent Piepho wrote:
> On Fri, 2019-04-12 at 17:56 +0000, Leonard Crestez wrote:
>> On 3/26/19 8:19 PM, Lorenzo Pieralisi wrote:
>>> On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
>>>> In theory this driver can be used on imx6sx without enabling support for
>>>> imx6q or imx7d but the "depends on" condition doesn't allow that.
>>>>
>>>> Instead of making the condition even longer just make it depend on
>>>> "ARCH_MXC || COMPILE_TEST" instead.
>>>>
>>>> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
>>>> ---
>>>> drivers/pci/controller/dwc/Kconfig | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
>>>> index 6ea74b1c0d94..21747fd0e799 100644
>>>> --- a/drivers/pci/controller/dwc/Kconfig
>>>> +++ b/drivers/pci/controller/dwc/Kconfig
>>>> @@ -88,11 +88,11 @@ config PCI_EXYNOS
>>>> depends on PCI_MSI_IRQ_DOMAIN
>>>> select PCIE_DW_HOST
>>>>
>>>> config PCI_IMX6
>>>> bool "Freescale i.MX6/7/8 PCIe controller"
>>>> - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
>>>> + depends on ARCH_MXC || COMPILE_TEST
>>>> depends on PCI_MSI_IRQ_DOMAIN
>>>> select PCIE_DW_HOST
>>>
>>> If Lucas does not spot any problem with this patch I would request his
>>> ACK to merge it, thanks.
>>
>> Lucas: Can you please ack this? It's a bit old but applies cleanly and
>> resend shouldn't be required.
>
> The IMX7s does not have a PCI-e controller. The IMX7d does have one,
> but not the "s".
>
> Enabling the controller driver on an IMX SoC that doesn't have it is
> not exactly the end of the world and having to keep adding new IMX SOCs
> to the list here is annoying. So IMHO, it's ok to make this change
> even if the controller is allowed on a superset of the SoCs that have
> it.
There is no separate config option for IMX7S and I don't think it will
be ever added. There is however a CONFIG_SOC_IMX6SX.
There were attempts to add "depends on"/"selected by" IMX8MQ to various
drivers but arm64 maintainers disagreed. This is why this patch fixes
the "depends on" list for 6sx pcie by dropping per-SOC logic on 32-bit
arm as well.
People who want very small kernels for their boards will likely fiddle
with their specific defconfig anyway so maybe complex logic in Kconfig
is not very helpful.
--
Regards,
Leonard
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-04-12 18:20 ` Leonard Crestez
@ 2019-04-30 14:22 ` Lorenzo Pieralisi
2019-07-05 15:41 ` Lucas Stach
0 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Pieralisi @ 2019-04-30 14:22 UTC (permalink / raw)
To: Leonard Crestez
Cc: Trent Piepho, l.stach, stefan, shawnguo, andrew.smirnov,
Richard Zhu, linux-pci, dl-linux-imx
On Fri, Apr 12, 2019 at 06:20:53PM +0000, Leonard Crestez wrote:
> On 4/12/2019 9:03 PM, Trent Piepho wrote:
> > On Fri, 2019-04-12 at 17:56 +0000, Leonard Crestez wrote:
> >> On 3/26/19 8:19 PM, Lorenzo Pieralisi wrote:
> >>> On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
> >>>> In theory this driver can be used on imx6sx without enabling support for
> >>>> imx6q or imx7d but the "depends on" condition doesn't allow that.
> >>>>
> >>>> Instead of making the condition even longer just make it depend on
> >>>> "ARCH_MXC || COMPILE_TEST" instead.
> >>>>
> >>>> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
> >>>> ---
> >>>> drivers/pci/controller/dwc/Kconfig | 2 +-
> >>>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> >>>> index 6ea74b1c0d94..21747fd0e799 100644
> >>>> --- a/drivers/pci/controller/dwc/Kconfig
> >>>> +++ b/drivers/pci/controller/dwc/Kconfig
> >>>> @@ -88,11 +88,11 @@ config PCI_EXYNOS
> >>>> depends on PCI_MSI_IRQ_DOMAIN
> >>>> select PCIE_DW_HOST
> >>>>
> >>>> config PCI_IMX6
> >>>> bool "Freescale i.MX6/7/8 PCIe controller"
> >>>> - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
> >>>> + depends on ARCH_MXC || COMPILE_TEST
> >>>> depends on PCI_MSI_IRQ_DOMAIN
> >>>> select PCIE_DW_HOST
> >>>
> >>> If Lucas does not spot any problem with this patch I would request his
> >>> ACK to merge it, thanks.
> >>
> >> Lucas: Can you please ack this? It's a bit old but applies cleanly and
> >> resend shouldn't be required.
> >
> > The IMX7s does not have a PCI-e controller. The IMX7d does have one,
> > but not the "s".
> >
> > Enabling the controller driver on an IMX SoC that doesn't have it is
> > not exactly the end of the world and having to keep adding new IMX SOCs
> > to the list here is annoying. So IMHO, it's ok to make this change
> > even if the controller is allowed on a superset of the SoCs that have
> > it.
>
> There is no separate config option for IMX7S and I don't think it will
> be ever added. There is however a CONFIG_SOC_IMX6SX.
>
> There were attempts to add "depends on"/"selected by" IMX8MQ to various
> drivers but arm64 maintainers disagreed. This is why this patch fixes
> the "depends on" list for 6sx pcie by dropping per-SOC logic on 32-bit
> arm as well.
>
> People who want very small kernels for their boards will likely fiddle
> with their specific defconfig anyway so maybe complex logic in Kconfig
> is not very helpful.
My request still stands, please let me know.
Thanks,
Lorenzo
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-04-30 14:22 ` Lorenzo Pieralisi
@ 2019-07-05 15:41 ` Lucas Stach
0 siblings, 0 replies; 8+ messages in thread
From: Lucas Stach @ 2019-07-05 15:41 UTC (permalink / raw)
To: Lorenzo Pieralisi, Leonard Crestez
Cc: Trent Piepho, stefan, shawnguo, andrew.smirnov, Richard Zhu,
linux-pci, dl-linux-imx
Am Dienstag, den 30.04.2019, 15:22 +0100 schrieb Lorenzo Pieralisi:
> On Fri, Apr 12, 2019 at 06:20:53PM +0000, Leonard Crestez wrote:
> > On 4/12/2019 9:03 PM, Trent Piepho wrote:
> > > On Fri, 2019-04-12 at 17:56 +0000, Leonard Crestez wrote:
> > > > On 3/26/19 8:19 PM, Lorenzo Pieralisi wrote:
> > > > > On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
> > > > > > In theory this driver can be used on imx6sx without enabling support for
> > > > > > imx6q or imx7d but the "depends on" condition doesn't allow that.
> > > > > >
> > > > > > Instead of making the condition even longer just make it depend on
> > > > > > "ARCH_MXC || COMPILE_TEST" instead.
> > > > > >
> > > > > > > > > > > > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
> > > > > > ---
> > > > > > drivers/pci/controller/dwc/Kconfig | 2 +-
> > > > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > > >
> > > > > > diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> > > > > > index 6ea74b1c0d94..21747fd0e799 100644
> > > > > > --- a/drivers/pci/controller/dwc/Kconfig
> > > > > > +++ b/drivers/pci/controller/dwc/Kconfig
> > > > > > @@ -88,11 +88,11 @@ config PCI_EXYNOS
> > > > > > depends on PCI_MSI_IRQ_DOMAIN
> > > > > > select PCIE_DW_HOST
> > > > > >
> > > > > > config PCI_IMX6
> > > > > > bool "Freescale i.MX6/7/8 PCIe controller"
> > > > > > - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
> > > > > > + depends on ARCH_MXC || COMPILE_TEST
> > > > > > depends on PCI_MSI_IRQ_DOMAIN
> > > > > > select PCIE_DW_HOST
> > > > >
> > > > > If Lucas does not spot any problem with this patch I would request his
> > > > > ACK to merge it, thanks.
> > > >
> > > > Lucas: Can you please ack this? It's a bit old but applies cleanly and
> > > > resend shouldn't be required.
> > >
> > > The IMX7s does not have a PCI-e controller. The IMX7d does have one,
> > > but not the "s".
> > >
> > > Enabling the controller driver on an IMX SoC that doesn't have it is
> > > not exactly the end of the world and having to keep adding new IMX SOCs
> > > to the list here is annoying. So IMHO, it's ok to make this change
> > > even if the controller is allowed on a superset of the SoCs that have
> > > it.
> >
> > There is no separate config option for IMX7S and I don't think it will
> > be ever added. There is however a CONFIG_SOC_IMX6SX.
> >
> > There were attempts to add "depends on"/"selected by" IMX8MQ to various
> > drivers but arm64 maintainers disagreed. This is why this patch fixes
> > the "depends on" list for 6sx pcie by dropping per-SOC logic on 32-bit
> > arm as well.
> >
> > People who want very small kernels for their boards will likely fiddle
> > with their specific defconfig anyway so maybe complex logic in Kconfig
> > is not very helpful.
>
> My request still stands, please let me know.
I've acked the resend, but that one doesn't show up in patchwork for
some reason, so:
Acked-by: Lucas Stach <l.stach@pengutronix.de>
Regards,
Lucas
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] PCI: imx6: Simplify Kconfig depends on
2019-03-11 18:59 [PATCH] PCI: imx6: Simplify Kconfig depends on Leonard Crestez
2019-03-26 18:19 ` Lorenzo Pieralisi
@ 2019-07-05 15:51 ` Lorenzo Pieralisi
1 sibling, 0 replies; 8+ messages in thread
From: Lorenzo Pieralisi @ 2019-07-05 15:51 UTC (permalink / raw)
To: Leonard Crestez
Cc: Lucas Stach, Trent Piepho, Andrey Smirnov, Stefan Agner,
Richard Zhu, dl-linux-imx, Shawn Guo, linux-pci
On Mon, Mar 11, 2019 at 06:59:28PM +0000, Leonard Crestez wrote:
> In theory this driver can be used on imx6sx without enabling support for
> imx6q or imx7d but the "depends on" condition doesn't allow that.
>
> Instead of making the condition even longer just make it depend on
> "ARCH_MXC || COMPILE_TEST" instead.
>
> Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
> ---
> drivers/pci/controller/dwc/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied to pci/dwc for v5.3, thanks.
Lorenzo
> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> index 6ea74b1c0d94..21747fd0e799 100644
> --- a/drivers/pci/controller/dwc/Kconfig
> +++ b/drivers/pci/controller/dwc/Kconfig
> @@ -88,11 +88,11 @@ config PCI_EXYNOS
> depends on PCI_MSI_IRQ_DOMAIN
> select PCIE_DW_HOST
>
> config PCI_IMX6
> bool "Freescale i.MX6/7/8 PCIe controller"
> - depends on SOC_IMX6Q || SOC_IMX7D || (ARM64 && ARCH_MXC) || COMPILE_TEST
> + depends on ARCH_MXC || COMPILE_TEST
> depends on PCI_MSI_IRQ_DOMAIN
> select PCIE_DW_HOST
>
> config PCIE_SPEAR13XX
> bool "STMicroelectronics SPEAr PCIe controller"
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-07-05 15:51 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-11 18:59 [PATCH] PCI: imx6: Simplify Kconfig depends on Leonard Crestez
2019-03-26 18:19 ` Lorenzo Pieralisi
2019-04-12 17:56 ` Leonard Crestez
2019-04-12 18:03 ` Trent Piepho
2019-04-12 18:20 ` Leonard Crestez
2019-04-30 14:22 ` Lorenzo Pieralisi
2019-07-05 15:41 ` Lucas Stach
2019-07-05 15:51 ` Lorenzo Pieralisi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.