From: Larry Finger <Larry.Finger@lwfinger.net>
To: Randy Dunlap <rdunlap@infradead.org>,
LKML <linux-kernel@vger.kernel.org>
Cc: Paul Bolle <pebolle@tiscali.nl>
Subject: Re: Kconfig circular dependency question
Date: Sun, 06 Apr 2014 16:03:01 -0500 [thread overview]
Message-ID: <5341C105.3060608@lwfinger.net> (raw)
In-Reply-To: <5341B999.3050206@infradead.org>
On 04/06/2014 03:31 PM, Randy Dunlap wrote:
> On 04/06/2014 12:49 PM, Larry Finger wrote:
>> A kernel build is reporting the following:
>>
>> net/rfkill/Kconfig:4:error: recursive dependency detected!
>> net/rfkill/Kconfig:4: symbol RFKILL is selected by R8723AU
>> drivers/staging/rtl8723au/Kconfig:1: symbol R8723AU depends on USB
>> drivers/usb/Kconfig:41: symbol USB is selected by MOUSE_APPLETOUCH
>> drivers/input/mouse/Kconfig:162: symbol MOUSE_APPLETOUCH depends on INPUT
>> drivers/input/Kconfig:8: symbol INPUT is selected by ACPI_CMPC
>> drivers/platform/x86/Kconfig:635: symbol ACPI_CMPC depends on RFKILL
>>
>> What is the recommended way to avoid this circularity? Of course, R8723AU could select USB rather than depending on it, but that does not seem quite right.
>
> I don't know if it will fix the problem, but Linus has spoken (written)
> against selecting subsystems, like INPUT or USB.
>
> Does changing ACPI_CMPC to depend on INPUT instead of select INPUT
> help?
> and/or changing MOUSE_APPLETOUCH to depend on USB instead of select it?
This case seems to prove why Linus has so spoken!
Changing ACPI_CMPC to depend on INPUT does fix the build. Making the change to
MOUSE_APPLETOUCH ends up getting a different recursive error.
Thanks for your suggestion. As I do not "own" the x86 platform drivers, I will
have to post a patch to fix the ACPI_CMPC problem, but at least I know what to
say in the commit message that adds the RFKILL selection to the R8723AU Kconfig.
Larry
prev parent reply other threads:[~2014-04-06 21:03 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-06 19:49 Kconfig circular dependency question Larry Finger
2014-04-06 20:10 ` Paul Bolle
2014-04-06 20:21 ` Paul Bolle
2014-04-06 20:31 ` Randy Dunlap
2014-04-06 20:52 ` Paul Bolle
2014-04-06 21:07 ` Larry Finger
2014-04-06 21:14 ` Paul Bolle
2014-04-06 22:51 ` Larry Finger
2014-04-07 17:10 ` Paul Bolle
2014-04-07 0:02 ` Gene Heskett
2014-04-06 21:03 ` Larry Finger [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5341C105.3060608@lwfinger.net \
--to=larry.finger@lwfinger.net \
--cc=linux-kernel@vger.kernel.org \
--cc=pebolle@tiscali.nl \
--cc=rdunlap@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.