* Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:21 ` Lukas Bulwahn 0 siblings, 0 replies; 14+ messages in thread From: Lukas Bulwahn @ 2023-03-14 7:21 UTC (permalink / raw) To: Herve Codina, Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao Cc: linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Dear Herve, In your patch below, you added the config CPM_QMC which depends on the non-existing config SOC_FSL: https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com Up to my knowledge, the config SOC_FSL never existed in the mainline tree. Is this dependency really required or can the expression simply be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? Note: This patch has now shown up in linux-next with commit 3178d58e0b97. Currently, it would not be possible to compile test this driver, as the dependency on SOC_FSL is never met. Best regards, Lukas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:21 ` Lukas Bulwahn 0 siblings, 0 replies; 14+ messages in thread From: Lukas Bulwahn @ 2023-03-14 7:21 UTC (permalink / raw) To: Herve Codina, Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao Cc: linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Dear Herve, In your patch below, you added the config CPM_QMC which depends on the non-existing config SOC_FSL: https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com Up to my knowledge, the config SOC_FSL never existed in the mainline tree. Is this dependency really required or can the expression simply be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? Note: This patch has now shown up in linux-next with commit 3178d58e0b97. Currently, it would not be possible to compile test this driver, as the dependency on SOC_FSL is never met. Best regards, Lukas _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:21 ` Lukas Bulwahn 0 siblings, 0 replies; 14+ messages in thread From: Lukas Bulwahn @ 2023-03-14 7:21 UTC (permalink / raw) To: Herve Codina, Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao Cc: kernel-janitors, linuxppc-dev, Linux Kernel Mailing List, Linux ARM Dear Herve, In your patch below, you added the config CPM_QMC which depends on the non-existing config SOC_FSL: https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com Up to my knowledge, the config SOC_FSL never existed in the mainline tree. Is this dependency really required or can the expression simply be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? Note: This patch has now shown up in linux-next with commit 3178d58e0b97. Currently, it would not be possible to compile test this driver, as the dependency on SOC_FSL is never met. Best regards, Lukas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC 2023-03-14 7:21 ` Lukas Bulwahn (?) @ 2023-03-14 7:57 ` Herve Codina -1 siblings, 0 replies; 14+ messages in thread From: Herve Codina @ 2023-03-14 7:57 UTC (permalink / raw) To: Lukas Bulwahn Cc: Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao, linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Hi Lukas, On Tue, 14 Mar 2023 08:21:50 +0100 Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > Dear Herve, > > In your patch below, you added the config CPM_QMC which depends on the > non-existing config SOC_FSL: > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > Up to my knowledge, the config SOC_FSL never existed in the mainline > tree. Is this dependency really required or can the expression simply > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > Note: This patch has now shown up in linux-next with commit > 3178d58e0b97. Currently, it would not be possible to compile test this > driver, as the dependency on SOC_FSL is never met. > > > Best regards, > > Lukas My bad :( The dependency must be FSL_SOC instead of SOC_FSL. I mean: diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig index f90cfdf0c763..7268c2fbcbc1 100644 --- a/drivers/soc/fsl/qe/Kconfig +++ b/drivers/soc/fsl/qe/Kconfig @@ -47,7 +47,7 @@ config CPM_TSA config CPM_QMC tristate "CPM QMC support" depends on OF && HAS_IOMEM - depends on CPM1 || (SOC_FSL && COMPILE_TEST) + depends on CPM1 || (FSL_SOC && COMPILE_TEST) depends on CPM_TSA help Freescale CPM QUICC Multichannel Controller -- Hervé Codina, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:57 ` Herve Codina 0 siblings, 0 replies; 14+ messages in thread From: Herve Codina @ 2023-03-14 7:57 UTC (permalink / raw) To: Lukas Bulwahn Cc: Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao, linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Hi Lukas, On Tue, 14 Mar 2023 08:21:50 +0100 Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > Dear Herve, > > In your patch below, you added the config CPM_QMC which depends on the > non-existing config SOC_FSL: > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > Up to my knowledge, the config SOC_FSL never existed in the mainline > tree. Is this dependency really required or can the expression simply > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > Note: This patch has now shown up in linux-next with commit > 3178d58e0b97. Currently, it would not be possible to compile test this > driver, as the dependency on SOC_FSL is never met. > > > Best regards, > > Lukas My bad :( The dependency must be FSL_SOC instead of SOC_FSL. I mean: diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig index f90cfdf0c763..7268c2fbcbc1 100644 --- a/drivers/soc/fsl/qe/Kconfig +++ b/drivers/soc/fsl/qe/Kconfig @@ -47,7 +47,7 @@ config CPM_TSA config CPM_QMC tristate "CPM QMC support" depends on OF && HAS_IOMEM - depends on CPM1 || (SOC_FSL && COMPILE_TEST) + depends on CPM1 || (FSL_SOC && COMPILE_TEST) depends on CPM_TSA help Freescale CPM QUICC Multichannel Controller -- Hervé Codina, Bootlin Embedded Linux and Kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:57 ` Herve Codina 0 siblings, 0 replies; 14+ messages in thread From: Herve Codina @ 2023-03-14 7:57 UTC (permalink / raw) To: Lukas Bulwahn Cc: kernel-janitors, Linux Kernel Mailing List, Li Yang, Mark Brown, linuxppc-dev, Linux ARM, Qiang Zhao Hi Lukas, On Tue, 14 Mar 2023 08:21:50 +0100 Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > Dear Herve, > > In your patch below, you added the config CPM_QMC which depends on the > non-existing config SOC_FSL: > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > Up to my knowledge, the config SOC_FSL never existed in the mainline > tree. Is this dependency really required or can the expression simply > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > Note: This patch has now shown up in linux-next with commit > 3178d58e0b97. Currently, it would not be possible to compile test this > driver, as the dependency on SOC_FSL is never met. > > > Best regards, > > Lukas My bad :( The dependency must be FSL_SOC instead of SOC_FSL. I mean: diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig index f90cfdf0c763..7268c2fbcbc1 100644 --- a/drivers/soc/fsl/qe/Kconfig +++ b/drivers/soc/fsl/qe/Kconfig @@ -47,7 +47,7 @@ config CPM_TSA config CPM_QMC tristate "CPM QMC support" depends on OF && HAS_IOMEM - depends on CPM1 || (SOC_FSL && COMPILE_TEST) + depends on CPM1 || (FSL_SOC && COMPILE_TEST) depends on CPM_TSA help Freescale CPM QUICC Multichannel Controller -- Hervé Codina, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC 2023-03-14 7:57 ` Herve Codina (?) @ 2023-03-14 8:17 ` Lukas Bulwahn -1 siblings, 0 replies; 14+ messages in thread From: Lukas Bulwahn @ 2023-03-14 8:17 UTC (permalink / raw) To: Herve Codina Cc: Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao, linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors On Tue, Mar 14, 2023 at 8:57 AM Herve Codina <herve.codina@bootlin.com> wrote: > > Hi Lukas, > > On Tue, 14 Mar 2023 08:21:50 +0100 > Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > > Dear Herve, > > > > In your patch below, you added the config CPM_QMC which depends on the > > non-existing config SOC_FSL: > > > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > > > Up to my knowledge, the config SOC_FSL never existed in the mainline > > tree. Is this dependency really required or can the expression simply > > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > > > Note: This patch has now shown up in linux-next with commit > > 3178d58e0b97. Currently, it would not be possible to compile test this > > driver, as the dependency on SOC_FSL is never met. > > > > > > Best regards, > > > > Lukas > > My bad :( > > The dependency must be FSL_SOC instead of SOC_FSL. Herve, are you going to send a quick fix to your patch or ask Mark if your original patch can simply be replaced with a new one with this change added? Lukas > I mean: > diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig > index f90cfdf0c763..7268c2fbcbc1 100644 > --- a/drivers/soc/fsl/qe/Kconfig > +++ b/drivers/soc/fsl/qe/Kconfig > @@ -47,7 +47,7 @@ config CPM_TSA > config CPM_QMC > tristate "CPM QMC support" > depends on OF && HAS_IOMEM > - depends on CPM1 || (SOC_FSL && COMPILE_TEST) > + depends on CPM1 || (FSL_SOC && COMPILE_TEST) > depends on CPM_TSA > help > Freescale CPM QUICC Multichannel Controller > > > > -- > Hervé Codina, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 8:17 ` Lukas Bulwahn 0 siblings, 0 replies; 14+ messages in thread From: Lukas Bulwahn @ 2023-03-14 8:17 UTC (permalink / raw) To: Herve Codina Cc: Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao, linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors On Tue, Mar 14, 2023 at 8:57 AM Herve Codina <herve.codina@bootlin.com> wrote: > > Hi Lukas, > > On Tue, 14 Mar 2023 08:21:50 +0100 > Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > > Dear Herve, > > > > In your patch below, you added the config CPM_QMC which depends on the > > non-existing config SOC_FSL: > > > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > > > Up to my knowledge, the config SOC_FSL never existed in the mainline > > tree. Is this dependency really required or can the expression simply > > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > > > Note: This patch has now shown up in linux-next with commit > > 3178d58e0b97. Currently, it would not be possible to compile test this > > driver, as the dependency on SOC_FSL is never met. > > > > > > Best regards, > > > > Lukas > > My bad :( > > The dependency must be FSL_SOC instead of SOC_FSL. Herve, are you going to send a quick fix to your patch or ask Mark if your original patch can simply be replaced with a new one with this change added? Lukas > I mean: > diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig > index f90cfdf0c763..7268c2fbcbc1 100644 > --- a/drivers/soc/fsl/qe/Kconfig > +++ b/drivers/soc/fsl/qe/Kconfig > @@ -47,7 +47,7 @@ config CPM_TSA > config CPM_QMC > tristate "CPM QMC support" > depends on OF && HAS_IOMEM > - depends on CPM1 || (SOC_FSL && COMPILE_TEST) > + depends on CPM1 || (FSL_SOC && COMPILE_TEST) > depends on CPM_TSA > help > Freescale CPM QUICC Multichannel Controller > > > > -- > Hervé Codina, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 8:17 ` Lukas Bulwahn 0 siblings, 0 replies; 14+ messages in thread From: Lukas Bulwahn @ 2023-03-14 8:17 UTC (permalink / raw) To: Herve Codina Cc: kernel-janitors, Linux Kernel Mailing List, Li Yang, Mark Brown, linuxppc-dev, Linux ARM, Qiang Zhao On Tue, Mar 14, 2023 at 8:57 AM Herve Codina <herve.codina@bootlin.com> wrote: > > Hi Lukas, > > On Tue, 14 Mar 2023 08:21:50 +0100 > Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > > Dear Herve, > > > > In your patch below, you added the config CPM_QMC which depends on the > > non-existing config SOC_FSL: > > > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > > > Up to my knowledge, the config SOC_FSL never existed in the mainline > > tree. Is this dependency really required or can the expression simply > > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > > > Note: This patch has now shown up in linux-next with commit > > 3178d58e0b97. Currently, it would not be possible to compile test this > > driver, as the dependency on SOC_FSL is never met. > > > > > > Best regards, > > > > Lukas > > My bad :( > > The dependency must be FSL_SOC instead of SOC_FSL. Herve, are you going to send a quick fix to your patch or ask Mark if your original patch can simply be replaced with a new one with this change added? Lukas > I mean: > diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig > index f90cfdf0c763..7268c2fbcbc1 100644 > --- a/drivers/soc/fsl/qe/Kconfig > +++ b/drivers/soc/fsl/qe/Kconfig > @@ -47,7 +47,7 @@ config CPM_TSA > config CPM_QMC > tristate "CPM QMC support" > depends on OF && HAS_IOMEM > - depends on CPM1 || (SOC_FSL && COMPILE_TEST) > + depends on CPM1 || (FSL_SOC && COMPILE_TEST) > depends on CPM_TSA > help > Freescale CPM QUICC Multichannel Controller > > > > -- > Hervé Codina, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC 2023-03-14 8:17 ` Lukas Bulwahn @ 2023-03-14 8:31 ` Herve Codina -1 siblings, 0 replies; 14+ messages in thread From: Herve Codina @ 2023-03-14 8:31 UTC (permalink / raw) To: Lukas Bulwahn Cc: Li Yang, Christophe Leroy, Mark Brown, Qiang Zhao, linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Hi Lukas, Mark, On Tue, 14 Mar 2023 09:17:18 +0100 Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > On Tue, Mar 14, 2023 at 8:57 AM Herve Codina <herve.codina@bootlin.com> wrote: > > > > Hi Lukas, > > > > On Tue, 14 Mar 2023 08:21:50 +0100 > > Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > > > > Dear Herve, > > > > > > In your patch below, you added the config CPM_QMC which depends on the > > > non-existing config SOC_FSL: > > > > > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > > > > > Up to my knowledge, the config SOC_FSL never existed in the mainline > > > tree. Is this dependency really required or can the expression simply > > > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > > > > > Note: This patch has now shown up in linux-next with commit > > > 3178d58e0b97. Currently, it would not be possible to compile test this > > > driver, as the dependency on SOC_FSL is never met. > > > > > > > > > Best regards, > > > > > > Lukas > > > > My bad :( > > > > The dependency must be FSL_SOC instead of SOC_FSL. > > Herve, are you going to send a quick fix to your patch or ask Mark if > your original patch can simply be replaced with a new one with this > change added? I have just sent a quick fix. https://lore.kernel.org/linux-kernel/20230314082157.137176-1-herve.codina@bootlin.com/ Mark, it can be squashed with 3178d58e0b97 ("soc: fsl: cpm1: Add support for QMC") Regards Hervé > > Lukas > > > I mean: > > diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig > > index f90cfdf0c763..7268c2fbcbc1 100644 > > --- a/drivers/soc/fsl/qe/Kconfig > > +++ b/drivers/soc/fsl/qe/Kconfig > > @@ -47,7 +47,7 @@ config CPM_TSA > > config CPM_QMC > > tristate "CPM QMC support" > > depends on OF && HAS_IOMEM > > - depends on CPM1 || (SOC_FSL && COMPILE_TEST) > > + depends on CPM1 || (FSL_SOC && COMPILE_TEST) > > depends on CPM_TSA > > help > > Freescale CPM QUICC Multichannel Controller > > > > > > > > -- > > Hervé Codina, Bootlin > > Embedded Linux and Kernel engineering > > https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 8:31 ` Herve Codina 0 siblings, 0 replies; 14+ messages in thread From: Herve Codina @ 2023-03-14 8:31 UTC (permalink / raw) To: Lukas Bulwahn Cc: kernel-janitors, Linux Kernel Mailing List, Li Yang, Mark Brown, linuxppc-dev, Linux ARM, Qiang Zhao Hi Lukas, Mark, On Tue, 14 Mar 2023 09:17:18 +0100 Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > On Tue, Mar 14, 2023 at 8:57 AM Herve Codina <herve.codina@bootlin.com> wrote: > > > > Hi Lukas, > > > > On Tue, 14 Mar 2023 08:21:50 +0100 > > Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote: > > > > > Dear Herve, > > > > > > In your patch below, you added the config CPM_QMC which depends on the > > > non-existing config SOC_FSL: > > > > > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > > > > > Up to my knowledge, the config SOC_FSL never existed in the mainline > > > tree. Is this dependency really required or can the expression simply > > > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? > > > > > > Note: This patch has now shown up in linux-next with commit > > > 3178d58e0b97. Currently, it would not be possible to compile test this > > > driver, as the dependency on SOC_FSL is never met. > > > > > > > > > Best regards, > > > > > > Lukas > > > > My bad :( > > > > The dependency must be FSL_SOC instead of SOC_FSL. > > Herve, are you going to send a quick fix to your patch or ask Mark if > your original patch can simply be replaced with a new one with this > change added? I have just sent a quick fix. https://lore.kernel.org/linux-kernel/20230314082157.137176-1-herve.codina@bootlin.com/ Mark, it can be squashed with 3178d58e0b97 ("soc: fsl: cpm1: Add support for QMC") Regards Hervé > > Lukas > > > I mean: > > diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig > > index f90cfdf0c763..7268c2fbcbc1 100644 > > --- a/drivers/soc/fsl/qe/Kconfig > > +++ b/drivers/soc/fsl/qe/Kconfig > > @@ -47,7 +47,7 @@ config CPM_TSA > > config CPM_QMC > > tristate "CPM QMC support" > > depends on OF && HAS_IOMEM > > - depends on CPM1 || (SOC_FSL && COMPILE_TEST) > > + depends on CPM1 || (FSL_SOC && COMPILE_TEST) > > depends on CPM_TSA > > help > > Freescale CPM QUICC Multichannel Controller > > > > > > > > -- > > Hervé Codina, Bootlin > > Embedded Linux and Kernel engineering > > https://bootlin.com ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC 2023-03-14 7:21 ` Lukas Bulwahn (?) @ 2023-03-14 7:57 ` Christophe Leroy -1 siblings, 0 replies; 14+ messages in thread From: Christophe Leroy @ 2023-03-14 7:57 UTC (permalink / raw) To: Lukas Bulwahn, Herve Codina, Li Yang, Mark Brown, Qiang Zhao Cc: linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Hi Lukas Le 14/03/2023 à 08:21, Lukas Bulwahn a écrit : > Dear Herve, > > In your patch below, you added the config CPM_QMC which depends on the > non-existing config SOC_FSL: > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > Up to my knowledge, the config SOC_FSL never existed in the mainline > tree. Is this dependency really required or can the expression simply > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? That's a mistake, should be FSL_SOC. See https://patchwork.ozlabs.org/project/linuxppc-dev/patch/20230126083222.374243-7-herve.codina@bootlin.com/#3058690 Christophe ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:57 ` Christophe Leroy 0 siblings, 0 replies; 14+ messages in thread From: Christophe Leroy @ 2023-03-14 7:57 UTC (permalink / raw) To: Lukas Bulwahn, Herve Codina, Li Yang, Mark Brown, Qiang Zhao Cc: linuxppc-dev, Linux ARM, Linux Kernel Mailing List, kernel-janitors Hi Lukas Le 14/03/2023 à 08:21, Lukas Bulwahn a écrit : > Dear Herve, > > In your patch below, you added the config CPM_QMC which depends on the > non-existing config SOC_FSL: > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > Up to my knowledge, the config SOC_FSL never existed in the mainline > tree. Is this dependency really required or can the expression simply > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? That's a mistake, should be FSL_SOC. See https://patchwork.ozlabs.org/project/linuxppc-dev/patch/20230126083222.374243-7-herve.codina@bootlin.com/#3058690 Christophe _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Question about the dependency on the config SOC_FSL in CPM_QMC @ 2023-03-14 7:57 ` Christophe Leroy 0 siblings, 0 replies; 14+ messages in thread From: Christophe Leroy @ 2023-03-14 7:57 UTC (permalink / raw) To: Lukas Bulwahn, Herve Codina, Li Yang, Mark Brown, Qiang Zhao Cc: kernel-janitors, linuxppc-dev, Linux Kernel Mailing List, Linux ARM Hi Lukas Le 14/03/2023 à 08:21, Lukas Bulwahn a écrit : > Dear Herve, > > In your patch below, you added the config CPM_QMC which depends on the > non-existing config SOC_FSL: > > https://lore.kernel.org/r/20230217145645.1768659-7-herve.codina@bootlin.com > > Up to my knowledge, the config SOC_FSL never existed in the mainline > tree. Is this dependency really required or can the expression simply > be reduced to COMPILE_TEST and we drop the dependency to SOC_FSL? That's a mistake, should be FSL_SOC. See https://patchwork.ozlabs.org/project/linuxppc-dev/patch/20230126083222.374243-7-herve.codina@bootlin.com/#3058690 Christophe ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2023-03-14 8:32 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-03-14 7:21 Question about the dependency on the config SOC_FSL in CPM_QMC Lukas Bulwahn 2023-03-14 7:21 ` Lukas Bulwahn 2023-03-14 7:21 ` Lukas Bulwahn 2023-03-14 7:57 ` Herve Codina 2023-03-14 7:57 ` Herve Codina 2023-03-14 7:57 ` Herve Codina 2023-03-14 8:17 ` Lukas Bulwahn 2023-03-14 8:17 ` Lukas Bulwahn 2023-03-14 8:17 ` Lukas Bulwahn 2023-03-14 8:31 ` Herve Codina 2023-03-14 8:31 ` Herve Codina 2023-03-14 7:57 ` Christophe Leroy 2023-03-14 7:57 ` Christophe Leroy 2023-03-14 7:57 ` Christophe Leroy
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.