Re: [PATCH 1/5] drivers/regulator: Remove dead CONFIG_TWL4030_ALLOW_UNSUPPORTED

From: Christian Dietrich
Date: Wed Jul 14 2010 - 10:42:24 EST


Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> writes:

> On Wed, Jul 14, 2010 at 04:04:28PM +0200, Christian Dietrich wrote:
>> CONFIG_TWL4030_ALLOW_UNSUPPORTED doesn't exist in Kconfig and is never
>> defined anywhere else, therefore removing all references for it from
>> the source code.
>>
>> Signed-off-by: Christian Dietrich <qy03fugy@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
>
> Nack, you're missing the point of having this code here.

So if it isn't a config option, then perhaps it should be named CONFIG_?

--
(Î x . x x) (Î x . x x) -- See how beatiful the lambda is
No documentation is better than bad documentation

--
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/