Re: Checkpatch problem with Kconfig help symbol ?
From: Andi Kleen
Date: Mon Jan 31 2011 - 18:00:30 EST
Tomas Winkler <tomasw@xxxxxxxxx> writes:
>> Yeah that is a little better. ÂI do remind you that if the description
>> is good enough shorter than 4 lines then you can ignore checkpatch. ÂIt
>> is a style guide not a style enforcer. ÂYou are allowed to ignore things
>> if you can justify it.
> Please consider patch I've posted
I originally didn't put in the number intentionally to make
it harder to game it. Most likely for a whole driver
or whole subsystems 4 lines are not enough to describe
But yes if it's just for a debug feature you can ignore it.
But then most likely your debug feature shouldn't be
in Kconfig in the first place, but some runtime setting
(rule of thumb: if it controls less than 1KB of code it's
likely a bad idea in Kconfig)
And more complex debug features that do actually carry
significant code should have proper description.
I think a better change would be to describe this
more fully, not add the number.
ak@xxxxxxxxxxxxxxx -- Speaking for myself only
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/