* [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies
@ 2023-03-08 12:49 Geert Uytterhoeven
2023-03-14 12:12 ` Damien Le Moal
2023-03-15 1:27 ` Damien Le Moal
0 siblings, 2 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2023-03-08 12:49 UTC (permalink / raw)
To: Damien Le Moal
Cc: linux-ide, linux-arm-kernel, linux-kernel, Geert Uytterhoeven,
Arnd Bergmann, Li Yang
The Freescale QorIQ AHCI SATA controller is only present on Freescale
Layerscape SoCs. Add platform dependencies to the AHCI_QORIQ config
symbol, to avoid asking the user about it when configuring a kernel
without Layerscape support.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Li Yang <leoyang.li@nxp.com>
---
v2:
- Add Acked-by.
---
drivers/ata/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
index b56fba76b43f48f7..42b51c9812a0ebab 100644
--- a/drivers/ata/Kconfig
+++ b/drivers/ata/Kconfig
@@ -276,6 +276,7 @@ config AHCI_XGENE
config AHCI_QORIQ
tristate "Freescale QorIQ AHCI SATA support"
depends on OF
+ depends on SOC_LS1021A || ARCH_LAYERSCAPE || COMPILE_TEST
select SATA_HOST
help
This option enables support for the Freescale QorIQ AHCI SoC's
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies
2023-03-08 12:49 [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies Geert Uytterhoeven
@ 2023-03-14 12:12 ` Damien Le Moal
2023-03-14 12:50 ` Geert Uytterhoeven
2023-03-15 1:27 ` Damien Le Moal
1 sibling, 1 reply; 4+ messages in thread
From: Damien Le Moal @ 2023-03-14 12:12 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: linux-ide, linux-arm-kernel, linux-kernel, Arnd Bergmann, Li Yang
On 3/8/23 21:49, Geert Uytterhoeven wrote:
> The Freescale QorIQ AHCI SATA controller is only present on Freescale
> Layerscape SoCs. Add platform dependencies to the AHCI_QORIQ config
> symbol, to avoid asking the user about it when configuring a kernel
> without Layerscape support.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Arnd Bergmann <arnd@arndb.de>
> Acked-by: Li Yang <leoyang.li@nxp.com>
> ---
> v2:
> - Add Acked-by.
> ---
> drivers/ata/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
> index b56fba76b43f48f7..42b51c9812a0ebab 100644
> --- a/drivers/ata/Kconfig
> +++ b/drivers/ata/Kconfig
> @@ -276,6 +276,7 @@ config AHCI_XGENE
> config AHCI_QORIQ
> tristate "Freescale QorIQ AHCI SATA support"
> depends on OF
> + depends on SOC_LS1021A || ARCH_LAYERSCAPE || COMPILE_TEST
> select SATA_HOST
> help
> This option enables support for the Freescale QorIQ AHCI SoC's
Do you want this in fixes or queuing this for 6.4 is fine ?
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies
2023-03-14 12:12 ` Damien Le Moal
@ 2023-03-14 12:50 ` Geert Uytterhoeven
0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2023-03-14 12:50 UTC (permalink / raw)
To: Damien Le Moal
Cc: linux-ide, linux-arm-kernel, linux-kernel, Arnd Bergmann, Li Yang
Hi Damien,
On Tue, Mar 14, 2023 at 1:12 PM Damien Le Moal
<damien.lemoal@opensource.wdc.com> wrote:
> On 3/8/23 21:49, Geert Uytterhoeven wrote:
> > The Freescale QorIQ AHCI SATA controller is only present on Freescale
> > Layerscape SoCs. Add platform dependencies to the AHCI_QORIQ config
> > symbol, to avoid asking the user about it when configuring a kernel
> > without Layerscape support.
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > Acked-by: Arnd Bergmann <arnd@arndb.de>
> > Acked-by: Li Yang <leoyang.li@nxp.com>
> > ---
> > v2:
> > - Add Acked-by.
> > ---
> > drivers/ata/Kconfig | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
> > index b56fba76b43f48f7..42b51c9812a0ebab 100644
> > --- a/drivers/ata/Kconfig
> > +++ b/drivers/ata/Kconfig
> > @@ -276,6 +276,7 @@ config AHCI_XGENE
> > config AHCI_QORIQ
> > tristate "Freescale QorIQ AHCI SATA support"
> > depends on OF
> > + depends on SOC_LS1021A || ARCH_LAYERSCAPE || COMPILE_TEST
> > select SATA_HOST
> > help
> > This option enables support for the Freescale QorIQ AHCI SoC's
>
> Do you want this in fixes or queuing this for 6.4 is fine ?
I guess 6.4 is fine. We've been seeing this message while configuring
our kernels for years, so a few extra weeks won't hurt ;-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies
2023-03-08 12:49 [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies Geert Uytterhoeven
2023-03-14 12:12 ` Damien Le Moal
@ 2023-03-15 1:27 ` Damien Le Moal
1 sibling, 0 replies; 4+ messages in thread
From: Damien Le Moal @ 2023-03-15 1:27 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: linux-ide, linux-arm-kernel, linux-kernel, Arnd Bergmann, Li Yang
On 3/8/23 21:49, Geert Uytterhoeven wrote:
> The Freescale QorIQ AHCI SATA controller is only present on Freescale
> Layerscape SoCs. Add platform dependencies to the AHCI_QORIQ config
> symbol, to avoid asking the user about it when configuring a kernel
> without Layerscape support.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Arnd Bergmann <arnd@arndb.de>
> Acked-by: Li Yang <leoyang.li@nxp.com>
Applied to for-6.4. Thanks !
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-03-15 1:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-08 12:49 [PATCH v2 resend 3] ahci: qoriq: Add platform dependencies Geert Uytterhoeven
2023-03-14 12:12 ` Damien Le Moal
2023-03-14 12:50 ` Geert Uytterhoeven
2023-03-15 1:27 ` Damien Le Moal
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).