[PATCH] dmaengine: bestcomm: Enable compile testing

[PATCH] dmaengine: bestcomm: Enable compile testing

[Rosen Penev]

Allow the BestComm DMA engine to be selected for PowerPC
compile-test builds.

The freescale ethernet and sound drivers now need a dependency on
PPC_MPC52xx as they use headers/functions from that platform.

Assisted-by: LLM
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 drivers/dma/bestcomm/Kconfig           | 3 +--
 drivers/net/ethernet/freescale/Kconfig | 2 +-
 sound/soc/fsl/Kconfig                  | 2 +-
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/dma/bestcomm/Kconfig b/drivers/dma/bestcomm/Kconfig
index 5dd437295964..153b5492c93c 100644
--- a/drivers/dma/bestcomm/Kconfig
+++ b/drivers/dma/bestcomm/Kconfig
@@ -5,7 +5,7 @@
 
 config PPC_BESTCOMM
 	tristate "Bestcomm DMA engine support"
-	depends on PPC_MPC52xx
+	depends on PPC_MPC52xx || (PPC && COMPILE_TEST)
 	default n
 	select PPC_LIB_RHEAP
 	help
@@ -34,4 +34,3 @@ config PPC_BESTCOMM_GEN_BD
 	depends on PPC_BESTCOMM
 	help
 	  This option enables the support for the GenBD tasks.
-
diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig
index b34c0fcfddea..d81984fcec64 100644
--- a/drivers/net/ethernet/freescale/Kconfig
+++ b/drivers/net/ethernet/freescale/Kconfig
@@ -38,7 +38,7 @@ config FEC
 
 config FEC_MPC52xx
 	tristate "FEC MPC52xx driver"
-	depends on PPC_BESTCOMM
+	depends on PPC_BESTCOMM && PPC_MPC52xx
 	select CRC32
 	select PHYLIB
 	select PPC_BESTCOMM_FEC
diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig
index 04940879dfd8..101bc952aa14 100644
--- a/sound/soc/fsl/Kconfig
+++ b/sound/soc/fsl/Kconfig
@@ -234,7 +234,7 @@ config SND_SOC_MPC5200_I2S
 
 config SND_SOC_MPC5200_AC97
 	tristate "Freescale MPC5200 PSC in AC97 mode driver"
-	depends on PPC_BESTCOMM
+	depends on PPC_BESTCOMM && PPC_MPC52xx
 	select SND_SOC_AC97_BUS
 	select SND_MPC52xx_DMA
 	select PPC_BESTCOMM_GEN_BD
-- 
2.55.0

Re: [PATCH] dmaengine: bestcomm: Enable compile testing

[Frank Li]

On Tue, Sep 15, 2026 at 11:17:09AM -0700, Rosen Penev wrote:
> Allow the BestComm DMA engine to be selected for PowerPC
> compile-test builds.
>
> The freescale ethernet and sound drivers now need a dependency on
> PPC_MPC52xx as they use headers/functions from that platform.
>
> Assisted-by: LLM
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---
>  drivers/dma/bestcomm/Kconfig           | 3 +--
>  drivers/net/ethernet/freescale/Kconfig | 2 +-
>  sound/soc/fsl/Kconfig                  | 2 +-
>  3 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/dma/bestcomm/Kconfig b/drivers/dma/bestcomm/Kconfig
> index 5dd437295964..153b5492c93c 100644
> --- a/drivers/dma/bestcomm/Kconfig
> +++ b/drivers/dma/bestcomm/Kconfig
> @@ -5,7 +5,7 @@
>
>  config PPC_BESTCOMM
>  	tristate "Bestcomm DMA engine support"
> -	depends on PPC_MPC52xx
> +	depends on PPC_MPC52xx || (PPC && COMPILE_TEST)
>  	default n
>  	select PPC_LIB_RHEAP
>  	help
> @@ -34,4 +34,3 @@ config PPC_BESTCOMM_GEN_BD
>  	depends on PPC_BESTCOMM
>  	help
>  	  This option enables the support for the GenBD tasks.
> -
> diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig
> index b34c0fcfddea..d81984fcec64 100644
> --- a/drivers/net/ethernet/freescale/Kconfig
> +++ b/drivers/net/ethernet/freescale/Kconfig
> @@ -38,7 +38,7 @@ config FEC
>
>  config FEC_MPC52xx
>  	tristate "FEC MPC52xx driver"
> -	depends on PPC_BESTCOMM
> +	depends on PPC_BESTCOMM && PPC_MPC52xx

use seperate patch for difference subsystem.

Frank

>  	select CRC32
>  	select PHYLIB
>  	select PPC_BESTCOMM_FEC
> diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig
> index 04940879dfd8..101bc952aa14 100644
> --- a/sound/soc/fsl/Kconfig
> +++ b/sound/soc/fsl/Kconfig
> @@ -234,7 +234,7 @@ config SND_SOC_MPC5200_I2S
>
>  config SND_SOC_MPC5200_AC97
>  	tristate "Freescale MPC5200 PSC in AC97 mode driver"
> -	depends on PPC_BESTCOMM
> +	depends on PPC_BESTCOMM && PPC_MPC52xx
>  	select SND_SOC_AC97_BUS
>  	select SND_MPC52xx_DMA
>  	select PPC_BESTCOMM_GEN_BD
> --
> 2.55.0
>

Re: [PATCH] dmaengine: bestcomm: Enable compile testing

[Rosen Penev]

On Wed, Sep 16, 2026 at 12:42 PM Frank Li <Frank.li@oss.nxp.com> wrote:
>
> On Tue, Sep 15, 2026 at 11:17:09AM -0700, Rosen Penev wrote:
> > Allow the BestComm DMA engine to be selected for PowerPC
> > compile-test builds.
> >
> > The freescale ethernet and sound drivers now need a dependency on
> > PPC_MPC52xx as they use headers/functions from that platform.
> >
> > Assisted-by: LLM
> > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > ---
> >  drivers/dma/bestcomm/Kconfig           | 3 +--
> >  drivers/net/ethernet/freescale/Kconfig | 2 +-
> >  sound/soc/fsl/Kconfig                  | 2 +-
> >  3 files changed, 3 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/dma/bestcomm/Kconfig b/drivers/dma/bestcomm/Kconfig
> > index 5dd437295964..153b5492c93c 100644
> > --- a/drivers/dma/bestcomm/Kconfig
> > +++ b/drivers/dma/bestcomm/Kconfig
> > @@ -5,7 +5,7 @@
> >
> >  config PPC_BESTCOMM
> >       tristate "Bestcomm DMA engine support"
> > -     depends on PPC_MPC52xx
> > +     depends on PPC_MPC52xx || (PPC && COMPILE_TEST)
> >       default n
> >       select PPC_LIB_RHEAP
> >       help
> > @@ -34,4 +34,3 @@ config PPC_BESTCOMM_GEN_BD
> >       depends on PPC_BESTCOMM
> >       help
> >         This option enables the support for the GenBD tasks.
> > -
> > diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig
> > index b34c0fcfddea..d81984fcec64 100644
> > --- a/drivers/net/ethernet/freescale/Kconfig
> > +++ b/drivers/net/ethernet/freescale/Kconfig
> > @@ -38,7 +38,7 @@ config FEC
> >
> >  config FEC_MPC52xx
> >       tristate "FEC MPC52xx driver"
> > -     depends on PPC_BESTCOMM
> > +     depends on PPC_BESTCOMM && PPC_MPC52xx
>
> use seperate patch for difference subsystem.
The issue is this patch exposes the issue. It's also extremely
difficult to get anything into net without the maintainers claiming
this is pointless churn.

>
> Frank
>
> >       select CRC32
> >       select PHYLIB
> >       select PPC_BESTCOMM_FEC
> > diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig
> > index 04940879dfd8..101bc952aa14 100644
> > --- a/sound/soc/fsl/Kconfig
> > +++ b/sound/soc/fsl/Kconfig
> > @@ -234,7 +234,7 @@ config SND_SOC_MPC5200_I2S
> >
> >  config SND_SOC_MPC5200_AC97
> >       tristate "Freescale MPC5200 PSC in AC97 mode driver"
> > -     depends on PPC_BESTCOMM
> > +     depends on PPC_BESTCOMM && PPC_MPC52xx
> >       select SND_SOC_AC97_BUS
> >       select SND_MPC52xx_DMA
> >       select PPC_BESTCOMM_GEN_BD
> > --
> > 2.55.0
> >

Re: [PATCH] dmaengine: bestcomm: Enable compile testing

[Frank Li]

On Wed, Sep 16, 2026 at 12:47:52PM -0700, Rosen Penev wrote:
> On Wed, Sep 16, 2026 at 12:42 PM Frank Li <Frank.li@oss.nxp.com> wrote:
> >
> > On Tue, Sep 15, 2026 at 11:17:09AM -0700, Rosen Penev wrote:
> > > Allow the BestComm DMA engine to be selected for PowerPC
> > > compile-test builds.
> > >
> > > The freescale ethernet and sound drivers now need a dependency on
> > > PPC_MPC52xx as they use headers/functions from that platform.
> > >
> > > Assisted-by: LLM
> > > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > > ---
> > >  drivers/dma/bestcomm/Kconfig           | 3 +--
> > >  drivers/net/ethernet/freescale/Kconfig | 2 +-
> > >  sound/soc/fsl/Kconfig                  | 2 +-
> > >  3 files changed, 3 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/drivers/dma/bestcomm/Kconfig b/drivers/dma/bestcomm/Kconfig
> > > index 5dd437295964..153b5492c93c 100644
> > > --- a/drivers/dma/bestcomm/Kconfig
> > > +++ b/drivers/dma/bestcomm/Kconfig
> > > @@ -5,7 +5,7 @@
> > >
> > >  config PPC_BESTCOMM
> > >       tristate "Bestcomm DMA engine support"
> > > -     depends on PPC_MPC52xx
> > > +     depends on PPC_MPC52xx || (PPC && COMPILE_TEST)
> > >       default n
> > >       select PPC_LIB_RHEAP
> > >       help
> > > @@ -34,4 +34,3 @@ config PPC_BESTCOMM_GEN_BD
> > >       depends on PPC_BESTCOMM
> > >       help
> > >         This option enables the support for the GenBD tasks.
> > > -
> > > diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig
> > > index b34c0fcfddea..d81984fcec64 100644
> > > --- a/drivers/net/ethernet/freescale/Kconfig
> > > +++ b/drivers/net/ethernet/freescale/Kconfig
> > > @@ -38,7 +38,7 @@ config FEC
> > >
> > >  config FEC_MPC52xx
> > >       tristate "FEC MPC52xx driver"
> > > -     depends on PPC_BESTCOMM
> > > +     depends on PPC_BESTCOMM && PPC_MPC52xx
> >
> > use seperate patch for difference subsystem.
> The issue is this patch exposes the issue. It's also extremely
> difficult to get anything into net without the maintainers claiming
> this is pointless churn.

Maybe you try enable COMPILE_TEST for FEC_MPC52xx also

Frank

>
> >
> > Frank
> >
> > >       select CRC32
> > >       select PHYLIB
> > >       select PPC_BESTCOMM_FEC
> > > diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig
> > > index 04940879dfd8..101bc952aa14 100644
> > > --- a/sound/soc/fsl/Kconfig
> > > +++ b/sound/soc/fsl/Kconfig
> > > @@ -234,7 +234,7 @@ config SND_SOC_MPC5200_I2S
> > >
> > >  config SND_SOC_MPC5200_AC97
> > >       tristate "Freescale MPC5200 PSC in AC97 mode driver"
> > > -     depends on PPC_BESTCOMM
> > > +     depends on PPC_BESTCOMM && PPC_MPC52xx
> > >       select SND_SOC_AC97_BUS
> > >       select SND_MPC52xx_DMA
> > >       select PPC_BESTCOMM_GEN_BD
> > > --
> > > 2.55.0
> > >

Re: [PATCH] dmaengine: bestcomm: Enable compile testing

[Rosen Penev]

On Wed, Sep 16, 2026 at 1:25 PM Frank Li <Frank.li@oss.nxp.com> wrote:
>
> On Wed, Sep 16, 2026 at 12:47:52PM -0700, Rosen Penev wrote:
> > On Wed, Sep 16, 2026 at 12:42 PM Frank Li <Frank.li@oss.nxp.com> wrote:
> > >
> > > On Tue, Sep 15, 2026 at 11:17:09AM -0700, Rosen Penev wrote:
> > > > Allow the BestComm DMA engine to be selected for PowerPC
> > > > compile-test builds.
> > > >
> > > > The freescale ethernet and sound drivers now need a dependency on
> > > > PPC_MPC52xx as they use headers/functions from that platform.
> > > >
> > > > Assisted-by: LLM
> > > > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > > > ---
> > > >  drivers/dma/bestcomm/Kconfig           | 3 +--
> > > >  drivers/net/ethernet/freescale/Kconfig | 2 +-
> > > >  sound/soc/fsl/Kconfig                  | 2 +-
> > > >  3 files changed, 3 insertions(+), 4 deletions(-)
> > > >
> > > > diff --git a/drivers/dma/bestcomm/Kconfig b/drivers/dma/bestcomm/Kconfig
> > > > index 5dd437295964..153b5492c93c 100644
> > > > --- a/drivers/dma/bestcomm/Kconfig
> > > > +++ b/drivers/dma/bestcomm/Kconfig
> > > > @@ -5,7 +5,7 @@
> > > >
> > > >  config PPC_BESTCOMM
> > > >       tristate "Bestcomm DMA engine support"
> > > > -     depends on PPC_MPC52xx
> > > > +     depends on PPC_MPC52xx || (PPC && COMPILE_TEST)
> > > >       default n
> > > >       select PPC_LIB_RHEAP
> > > >       help
> > > > @@ -34,4 +34,3 @@ config PPC_BESTCOMM_GEN_BD
> > > >       depends on PPC_BESTCOMM
> > > >       help
> > > >         This option enables the support for the GenBD tasks.
> > > > -
> > > > diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig
> > > > index b34c0fcfddea..d81984fcec64 100644
> > > > --- a/drivers/net/ethernet/freescale/Kconfig
> > > > +++ b/drivers/net/ethernet/freescale/Kconfig
> > > > @@ -38,7 +38,7 @@ config FEC
> > > >
> > > >  config FEC_MPC52xx
> > > >       tristate "FEC MPC52xx driver"
> > > > -     depends on PPC_BESTCOMM
> > > > +     depends on PPC_BESTCOMM && PPC_MPC52xx
> > >
> > > use seperate patch for difference subsystem.
> > The issue is this patch exposes the issue. It's also extremely
> > difficult to get anything into net without the maintainers claiming
> > this is pointless churn.
>
> Maybe you try enable COMPILE_TEST for FEC_MPC52xx also
IIRC it compiles but does not link.
>
> Frank
>
> >
> > >
> > > Frank
> > >
> > > >       select CRC32
> > > >       select PHYLIB
> > > >       select PPC_BESTCOMM_FEC
> > > > diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig
> > > > index 04940879dfd8..101bc952aa14 100644
> > > > --- a/sound/soc/fsl/Kconfig
> > > > +++ b/sound/soc/fsl/Kconfig
> > > > @@ -234,7 +234,7 @@ config SND_SOC_MPC5200_I2S
> > > >
> > > >  config SND_SOC_MPC5200_AC97
> > > >       tristate "Freescale MPC5200 PSC in AC97 mode driver"
> > > > -     depends on PPC_BESTCOMM
> > > > +     depends on PPC_BESTCOMM && PPC_MPC52xx
> > > >       select SND_SOC_AC97_BUS
> > > >       select SND_MPC52xx_DMA
> > > >       select PPC_BESTCOMM_GEN_BD
> > > > --
> > > > 2.55.0
> > > >

Re: [PATCH] dmaengine: bestcomm: Enable compile testing

[kernel test robot]

Hi Rosen,

kernel test robot noticed the following build errors:

[auto build test ERROR on vkoul-dmaengine/next]
[also build test ERROR on linus/master v7.1-rc4 next-20260521]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Rosen-Penev/dmaengine-bestcomm-Enable-compile-testing/20260521-063851
base:   https://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git next
patch link:    https://lore.kernel.org/r/20260520223704.39320-1-rosenp%40gmail.com
patch subject: [PATCH] dmaengine: bestcomm: Enable compile testing
config: powerpc-allmodconfig (https://download.01.org/0day-ci/archive/20260522/202605222111.yK5iSsqD-lkp@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 15.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260522/202605222111.yK5iSsqD-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202605222111.yK5iSsqD-lkp@intel.com/

All errors (new ones prefixed by >>, old ones prefixed by <<):

>> ERROR: modpost: "mpc5xxx_fwnode_get_bus_frequency" [drivers/ata/pata_mpc52xx.ko] undefined!

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

[PATCH] dmaengine: bestcomm: Enable compile testing

[Rosen Penev]

Allow the BestComm DMA engine to be selected for PowerPC
compile-test builds.

Make PATA_MPC52xx depend on PPC_BESTCOMM directly so selecting
PPC_BESTCOMM_ATA does not bypass the helper dependency.

Assisted-by: Codex:GPT-5.5
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 drivers/ata/Kconfig          | 2 +-
 drivers/dma/bestcomm/Kconfig | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
index 28ca856ecc75..6e25e9fcadb2 100644
--- a/drivers/ata/Kconfig
+++ b/drivers/ata/Kconfig
@@ -823,7 +823,7 @@ config PATA_MARVELL
 
 config PATA_MPC52xx
 	tristate "Freescale MPC52xx SoC internal IDE"
-	depends on PPC_MPC52xx && PPC_BESTCOMM
+	depends on PPC_BESTCOMM
 	select PPC_BESTCOMM_ATA
 	help
 	  This option enables support for integrated IDE controller
diff --git a/drivers/dma/bestcomm/Kconfig b/drivers/dma/bestcomm/Kconfig
index 5dd437295964..153b5492c93c 100644
--- a/drivers/dma/bestcomm/Kconfig
+++ b/drivers/dma/bestcomm/Kconfig
@@ -5,7 +5,7 @@
 
 config PPC_BESTCOMM
 	tristate "Bestcomm DMA engine support"
-	depends on PPC_MPC52xx
+	depends on PPC_MPC52xx || (PPC && COMPILE_TEST)
 	default n
 	select PPC_LIB_RHEAP
 	help
@@ -34,4 +34,3 @@ config PPC_BESTCOMM_GEN_BD
 	depends on PPC_BESTCOMM
 	help
 	  This option enables the support for the GenBD tasks.
-
-- 
2.54.0