[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <50192CBF.6040903@metafoo.de>
Date: Wed, 01 Aug 2012 15:18:55 +0200
From: Lars-Peter Clausen <lars@...afoo.de>
To: Steven Rostedt <rostedt@...dmis.org>
CC: Borislav Petkov <bp@...en8.de>,
Thierry Reding <thierry.reding@...onic-design.de>,
Dave Jones <davej@...hat.com>,
Linux Kernel <linux-kernel@...r.kernel.org>
Subject: Re: awful kconfig help texts.
On 08/01/2012 02:46 PM, Steven Rostedt wrote:
> On Wed, 2012-08-01 at 12:56 +0200, Lars-Peter Clausen wrote:
>
>> You could do that by using a construct like below, but well...
>>
>> config HAS_PWM
>> bool
>>
>> config PWM
>> bool "PWM
>> depends on HAS_PWM
>>
>> config PWM_CAN_BUILD_DRIVER_X
>> defbool I2C
>> select HAS_PWM
>>
>> config PWM_DRIVER_X
>> tristate "PWM chip X support"
>> depends on PWM && PWM_CAN_BUILD_DRIVER_X
>>
>> config PWM_CAN_BUILD_DRIVER_Y
>> defbool ARCH_Y
>> select HAS_PWM
>>
>> config PWM_DRIVER_Y
>> tristate "PWM chip Y support"
>> depends on PWM && PWM_CAN_BUILD_DRIVER_Y
>
> What selects the 'PWM_CAN_BUILD_DRIVER_FOO'?
>
Its def_bool statement, which lists the prerequisites to build the driver.
E.g. for a I2C PWM expander I2S support, for a on-SoC PWM core support for
the SoC family, etc.. So it will be true if the driver can actually be built
and false otherwise. If one of the PWM_CAN_BUILD_DRIVER_FOO symbols is true
also HAS_PWM will be true and PWM becomes selectable.
But it seems to be a bid tedious to have these extra lines for each driver
and I guess it is not a PWM subsystem specific issue. There are other
subsystems where this probably applies as well, e.g. the MFD subsystem.
Also such a solution would rule out out-of-tree PWM driver modules, since it
is not possible to get CONFIG_PWM selected.
- Lars
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists