* [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
@ 2023-10-07 18:24 Randy Dunlap
2023-10-08 7:35 ` Michael Büsch
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2023-10-07 18:24 UTC (permalink / raw)
To: linux-kernel; +Cc: Randy Dunlap, Michael Buesch, linux-wireless
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.
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Michael Buesch <m@bues.ch>
Cc: linux-wireless@vger.kernel.org
---
Comments, thoughts?
I also tried changing arch/mips/bcm47xx/Kconfig's BCM47XX_SSB to
select SSB_EMBEDDED if PCI
but that results in build errors.
drivers/ssb/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
2023-10-07 18:24 [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies Randy Dunlap
@ 2023-10-08 7:35 ` Michael Büsch
2023-10-08 14:08 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Michael Büsch @ 2023-10-08 7:35 UTC (permalink / raw)
To: Randy Dunlap; +Cc: linux-kernel, linux-wireless
[-- Attachment #1: Type: text/plain, Size: 1418 bytes --]
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/
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
2023-10-08 7:35 ` Michael Büsch
@ 2023-10-08 14:08 ` Randy Dunlap
2023-10-09 6:12 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2023-10-08 14:08 UTC (permalink / raw)
To: Michael Büsch; +Cc: linux-kernel, linux-wireless
On 10/8/23 00:35, Michael Büsch wrote:
> 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?
I'll test it some and see how it works out.
Thanks.
--
~Randy
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
2023-10-08 14:08 ` Randy Dunlap
@ 2023-10-09 6:12 ` Randy Dunlap
2023-10-10 18:43 ` Michael Büsch
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2023-10-09 6:12 UTC (permalink / raw)
To: Michael Büsch; +Cc: linux-kernel, linux-wireless
Hi Michael,
On 10/8/23 07:08, Randy Dunlap wrote:
>
>
> On 10/8/23 00:35, Michael Büsch wrote:
>> 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?
>
> I'll test it some and see how it works out.
> Thanks.
The following change seems to work and passes some testing.
Is this what you had in mind?
Thanks.
---
---
drivers/ssb/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
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 PCI=n || SSB_PCICORE_HOSTMODE
default y
config SSB_DRIVER_EXTIF
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
2023-10-09 6:12 ` Randy Dunlap
@ 2023-10-10 18:43 ` Michael Büsch
2023-10-12 3:44 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Michael Büsch @ 2023-10-10 18:43 UTC (permalink / raw)
To: Randy Dunlap; +Cc: linux-kernel, linux-wireless
[-- Attachment #1: Type: text/plain, Size: 384 bytes --]
On Sun, 8 Oct 2023 23:12:49 -0700
Randy Dunlap <rdunlap@infradead.org> wrote:
> - depends on SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE
> + depends on PCI=n || SSB_PCICORE_HOSTMODE
I thought about something like this:
depends on (SSB_DRIVER_MIPS && PCI=n) || (SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE)
Would that solve the warning?
--
Michael Büsch
https://bues.ch/
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
2023-10-10 18:43 ` Michael Büsch
@ 2023-10-12 3:44 ` Randy Dunlap
2023-10-12 17:39 ` Michael Büsch
0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2023-10-12 3:44 UTC (permalink / raw)
To: Michael Büsch; +Cc: linux-kernel, linux-wireless
Hi Michael,
On 10/10/23 11:43, Michael Büsch wrote:
> On Sun, 8 Oct 2023 23:12:49 -0700
> Randy Dunlap <rdunlap@infradead.org> wrote:
>
>> - depends on SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE
>> + depends on PCI=n || SSB_PCICORE_HOSTMODE
>
> I thought about something like this:
>
> depends on (SSB_DRIVER_MIPS && PCI=n) || (SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE)
>
> Would that solve the warning?
Yes, that works as well, although I prefer to express it like this:
config SSB_EMBEDDED
bool
- depends on SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE
+ depends on SSB_DRIVER_MIPS
+ depends on PCI=n || SSB_PCICORE_HOSTMODE
default y
although if you prefer the way that you have it above,
I'm OK with that also.
Thanks for your help.
--
~Randy
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies
2023-10-12 3:44 ` Randy Dunlap
@ 2023-10-12 17:39 ` Michael Büsch
0 siblings, 0 replies; 7+ messages in thread
From: Michael Büsch @ 2023-10-12 17:39 UTC (permalink / raw)
To: Randy Dunlap; +Cc: linux-kernel, linux-wireless
[-- Attachment #1: Type: text/plain, Size: 612 bytes --]
On Wed, 11 Oct 2023 20:44:49 -0700
Randy Dunlap <rdunlap@infradead.org> wrote:
> > I thought about something like this:
> >
> > depends on (SSB_DRIVER_MIPS && PCI=n) || (SSB_DRIVER_MIPS &&
> > SSB_PCICORE_HOSTMODE)
> >
> > Would that solve the warning?
>
> Yes, that works as well, although I prefer to express it like this:
>
> config SSB_EMBEDDED
> bool
> - depends on SSB_DRIVER_MIPS && SSB_PCICORE_HOSTMODE
> + depends on SSB_DRIVER_MIPS
> + depends on PCI=n || SSB_PCICORE_HOSTMODE
> default y
Acked-by: Michael Büsch <m@bues.ch>
--
Michael Büsch
https://bues.ch/
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-10-12 17:40 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-07 18:24 [RFC PATCH] ssb: relax SSB_EMBEDDED dependencies Randy Dunlap
2023-10-08 7:35 ` Michael Büsch
2023-10-08 14:08 ` Randy Dunlap
2023-10-09 6:12 ` Randy Dunlap
2023-10-10 18:43 ` Michael Büsch
2023-10-12 3:44 ` Randy Dunlap
2023-10-12 17:39 ` Michael Büsch
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).