Re: latest list of unused Kconfig variables

[Robert P. J. Day]

On Wed, 13 Feb 2008, Josh Boyer wrote:

> OK.  Well all of your hits for 405EX, 440GRX, 440SPe, and
> WANT_DEVICE_TREE in arch/powerpc seem bogus.  I dunno if you prune
> those when reported or not.

not normally, but i don't know what you mean by "bogus":

  $ grep -r "config 405EX" *
  arch/powerpc/platforms/40x/Kconfig:config 405EX
  $

  $ grep -r CONFIG_405EX *
  arch/powerpc/configs/makalu_defconfig:CONFIG_405EX=y
  arch/powerpc/configs/kilauea_defconfig:CONFIG_405EX=y
  $

so, AFAICT, there exists a definition of the Kconfig variable 405EX,
which is subsequently not referenced *anywhere* in the tree except in
a couple defconfig files, which don't count.  how, then, does that
Kconfig variable have any practical value?  or am i missing something
painfully obvious?

> > > And you have false positives on several CPU variables, as they
> > > are used within Kconfig files themselves to select different
> > > sets of options.
> >
> > yes, i know ... perhaps there's a simple way to filter those out
> > but, at the moment, it's pure brute force.  i'm guessing i could
> > make that script a bit smarter but it probably isn't worth the
> > investment in time.  law of diminishing returns and all that.
>
> Maybe.  If the same false positives keep showing up people might
> ignore it.  I won't, but others might.

fair enough, but since this scanning happens only once every release,
it's not like it's a burden.  and if people want to ignore it, that's
fine, too.

rday
--

========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry:
    Have classroom, will lecture.

http://crashcourse.ca                          Waterloo, Ontario, CANADA
========================================================================
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/