Messages in this thread | | | Date | Sun, 8 Oct 2023 09:35:20 +0200 | From | Michael Büsch <> | Subject | Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies |
| |
Hi Randy,
thanks for the patch.
On Sat, 7 Oct 2023 11:24:43 -0700 Randy Dunlap <rdunlap@infradead.org> wrote:
> This is a kconfig warning in a randconfig when CONFIG_PCI is not set: > > WARNING: unmet direct dependencies detected for SSB_EMBEDDED > Depends on [n]: SSB [=y] && SSB_DRIVER_MIPS [=y] && > SSB_PCICORE_HOSTMODE [=n] Selected by [y]: > - BCM47XX_SSB [=y] && BCM47XX [=y] > > This is caused by arch/mips/bcm47xx/Kconfig's symbol BCM47XX_SSB > selecting SSB_EMBEDDED when CONFIG_PCI is not set. > > This warning can be prevented by having SSB_EMBEDDED not depend on > SSB_PCICORE_HOSTMODE, although some parts of SSB use PCI.
> diff -- a/drivers/ssb/Kconfig b/drivers/ssb/Kconfig > --- a/drivers/ssb/Kconfig > +++ b/drivers/ssb/Kconfig > @@ -134,7 +134,7 @@ config SSB_SFLASH > # Assumption: We are on embedded, if we compile the MIPS core. > config SSB_EMBEDDED > bool > - depends on SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE > + depends on SSB_DRIVER_MIPS > default y > > config SSB_DRIVER_EXTIF
Could we instead make SSB_EMBEDDED depend on SSB_PCICORE_HOSTMODE if PCI!=n. Wouldn't that also solve the problem?
I don't fully remember how all this ssb config stuff works, but to me adding a PCICORE->PCI dependency sounds safer against build regressions in some other configurations.
What do you think?
-- Michael Büsch https://bues.ch/ [unhandled content-type:application/pgp-signature] | |