Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1524417
| From | Paul Bolle <pebolle@tiscali.nl> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: i2c: undefined option I2C_ALGO_BUSCLEAR |
| Date | 2016-11-17 13:00 +0100 |
| Message-ID | <sEtwd-b3-5@gated-at.bofh.it> (permalink) |
| References | <sE2Z3-8iI-1@gated-at.bofh.it> <sEqI1-6MO-11@gated-at.bofh.it> <sEtcR-8vW-1@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
Hi Valentin,
On Thu, 2016-11-17 at 12:33 +0100, Valentin Rothberg wrote:
> I tested your patch and it works properly for me. It even still applies
> on linux-next.
Cool.
(We should probably compile a list of current invalid selects somewhere
publicly, and team up to submit patches for the low hanging fruit (ie, obvious
typos) and for the arches that actually matter before I resubmit.)
> Would it be possible to extend your patch to also check
> symbols in other statements (depends on, if, etc.)?
No. Depending on an unknown symbol is common and correct. Think
depends on SOME_OBSCURE_ARM_BOARD
somewhere in drivers. Only arm builds will ever know about the
SOME_OBSCURE_ARM_BOARD symbol.
What could be done is starting any "make *config" target with a treewide pass
over all Kconfig files to collect all valid symbols and so spot typos and
other obviously incorrect. The example that comes to mind is
depends on SH
that I have spotted a few times in the past years. But this treewide pass will
incur some runtime cost and might not be easy to implement cleanly. Perhaps
we're better of with using your script for that.
Thanks,
Paul Bolle
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
i2c: undefined option I2C_ALGO_BUSCLEAR Valentin Rothberg <valentinrothberg@gmail.com> - 2016-11-16 08:40 +0100
Re: i2c: undefined option I2C_ALGO_BUSCLEAR Thierry Reding <treding@nvidia.com> - 2016-11-16 16:50 +0100
Re: i2c: undefined option I2C_ALGO_BUSCLEAR Thierry Reding <treding@nvidia.com> - 2016-11-16 16:50 +0100
RE: i2c: undefined option I2C_ALGO_BUSCLEAR Shardar Mohammed <smohammed@nvidia.com> - 2016-11-17 09:50 +0100
Re: i2c: undefined option I2C_ALGO_BUSCLEAR Paul Bolle <pebolle@tiscali.nl> - 2016-11-17 10:00 +0100
Re: i2c: undefined option I2C_ALGO_BUSCLEAR Valentin Rothberg <valentinrothberg@gmail.com> - 2016-11-17 12:40 +0100
Re: i2c: undefined option I2C_ALGO_BUSCLEAR Paul Bolle <pebolle@tiscali.nl> - 2016-11-17 13:00 +0100
Re: i2c: undefined option I2C_ALGO_BUSCLEAR Valentin Rothberg <valentinrothberg@gmail.com> - 2016-11-17 18:50 +0100
csiph-web