Re: PATCH: Create new LED trigger for CPU activity (ledtrig-cpu) (UPDATED)

[Thomas Tuttle]

On July 05 at 23:01 EDT, Randy.Dunlap hastily scribbled:
> On Wed, 5 Jul 2006 22:48:17 -0400 Thomas Tuttle wrote:
> 
> > Here is a new version of the patch, incorporating code style tips from
> > Randy Dunlap <rdunlap@xxxxxxxxxxxx>, and based on 2.6.17-git25, rather
> > than 2.6.17.1.
> > 
> > I noticed that there's a Heartbeat LED trigger in the git version.  I
> > hope this isn't too similar.
> 
> I missed at least one thing:
> 
> Don't #include <linux/config.h>
> That is done automatically now by Kbuild.  Source files
> should not do it.  (you could wait to see if there are other comments. :)
> 
> ---
> ~Randy

Are you sure?  Will it rebuild a file if a config entry is changed that
is simply mentioned in an #ifdef?

Is this a recent change?  It wasn't working this way in 2.6.17.1--it
automatically noticed changes to the config of the file itself, but not
to config symbols tested using #ifdef.

Attachment: pgp00000.pgp
Description: PGP signature