Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1702144 > unrolled thread
| Started by | "H. Nikolaus Schaller" <hns@goldelico.com> |
|---|---|
| First post | 2017-08-02 16:10 +0200 |
| Last post | 2017-08-02 21:00 +0200 |
| Articles | 7 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D "H. Nikolaus Schaller" <hns@goldelico.com> - 2017-08-02 16:10 +0200
Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D Arnd Bergmann <arnd@arndb.de> - 2017-08-02 17:50 +0200
Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-08-02 18:00 +0200
Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D "H. Nikolaus Schaller" <hns@goldelico.com> - 2017-08-02 18:10 +0200
Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D Arnd Bergmann <arnd@arndb.de> - 2017-08-02 18:10 +0200
Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D "H. Nikolaus Schaller" <hns@goldelico.com> - 2017-08-02 18:20 +0200
Re: [Letux-kernel] [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D "H. Nikolaus Schaller" <hns@goldelico.com> - 2017-08-02 21:00 +0200
| From | "H. Nikolaus Schaller" <hns@goldelico.com> |
|---|---|
| Date | 2017-08-02 16:10 +0200 |
| Subject | [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D |
| Message-ID | <ua2vw-2qm-15@gated-at.bofh.it> |
fix missing tristate prompt and add help for LIS3LV02D Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com> --- drivers/misc/Kconfig | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig index 8136dc7e863d..41781d61eac7 100644 --- a/drivers/misc/Kconfig +++ b/drivers/misc/Kconfig @@ -5,10 +5,13 @@ menu "Misc devices" config SENSORS_LIS3LV02D - tristate + tristate "support STMicroelectronics Sensor LIS3LV02D" depends on INPUT select INPUT_POLLDEV default n + help + If you say yes here, you get support for the STM + LIS3LV02D sensor. config AD525X_DPOT tristate "Analog Devices Digital Potentiometers" -- 2.12.2
[toc] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2017-08-02 17:50 +0200 |
| Subject | Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D |
| Message-ID | <ua44i-3hU-15@gated-at.bofh.it> |
| In reply to | #1702144 |
On Wed, Aug 2, 2017 at 4:05 PM, H. Nikolaus Schaller <hns@goldelico.com> wrote:
> fix missing tristate prompt and add help for LIS3LV02D
Should there be another patch to change the 'select SENSORS_LIS3LV02D'
statements to 'depends on'? Generally speaking, we should avoid selecting
user visible symbols.
Arnd
[toc] | [prev] | [next] | [standalone]
| From | Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
|---|---|
| Date | 2017-08-02 18:00 +0200 |
| Subject | Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D |
| Message-ID | <ua4dY-3lq-3@gated-at.bofh.it> |
| In reply to | #1702144 |
On Wed, Aug 02, 2017 at 04:05:36PM +0200, H. Nikolaus Schaller wrote: > fix missing tristate prompt and add help for LIS3LV02D > > Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com> > --- > drivers/misc/Kconfig | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig > index 8136dc7e863d..41781d61eac7 100644 > --- a/drivers/misc/Kconfig > +++ b/drivers/misc/Kconfig > @@ -5,10 +5,13 @@ > menu "Misc devices" > > config SENSORS_LIS3LV02D > - tristate > + tristate "support STMicroelectronics Sensor LIS3LV02D" > depends on INPUT > select INPUT_POLLDEV > default n "default n" can be removed as that is the deafule. > + help > + If you say yes here, you get support for the STM > + LIS3LV02D sensor. No name for what the module is called? thanks, greg k-h
[toc] | [prev] | [next] | [standalone]
| From | "H. Nikolaus Schaller" <hns@goldelico.com> |
|---|---|
| Date | 2017-08-02 18:10 +0200 |
| Message-ID | <ua4nG-3Eo-49@gated-at.bofh.it> |
| In reply to | #1702209 |
Hi, > Am 02.08.2017 um 17:57 schrieb Greg Kroah-Hartman <gregkh@linuxfoundation.org>: > > On Wed, Aug 02, 2017 at 04:05:36PM +0200, H. Nikolaus Schaller wrote: >> fix missing tristate prompt and add help for LIS3LV02D >> >> Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com> >> --- >> drivers/misc/Kconfig | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig >> index 8136dc7e863d..41781d61eac7 100644 >> --- a/drivers/misc/Kconfig >> +++ b/drivers/misc/Kconfig >> @@ -5,10 +5,13 @@ >> menu "Misc devices" >> >> config SENSORS_LIS3LV02D >> - tristate >> + tristate "support STMicroelectronics Sensor LIS3LV02D" >> depends on INPUT >> select INPUT_POLLDEV >> default n > > "default n" can be removed as that is the deafule. > >> + help >> + If you say yes here, you get support for the STM >> + LIS3LV02D sensor. > > No name for what the module is called? Well, I am not the author of the driver. I just noticed that the Kconfig record is incomplete. But I will research. BR and thanks, Nikolaus
[toc] | [prev] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2017-08-02 18:10 +0200 |
| Subject | Re: [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D |
| Message-ID | <ua4nH-3Eo-65@gated-at.bofh.it> |
| In reply to | #1702209 |
On Wed, Aug 2, 2017 at 5:57 PM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Wed, Aug 02, 2017 at 04:05:36PM +0200, H. Nikolaus Schaller wrote:
>> + help
>> + If you say yes here, you get support for the STM
>> + LIS3LV02D sensor.
>
> No name for what the module is called?
Not sure if that's even relevant here, as the symbol only controls the
base driver that doesn't do anything by itself, while the LIS3_SPI/LIS3_I2C
Kconfig symbols are the interesting ones.
The help text should also reflect that I think and explain why you would
enable the symbol in the first place (which the patch description doesn't
either).
Arnd
[toc] | [prev] | [next] | [standalone]
| From | "H. Nikolaus Schaller" <hns@goldelico.com> |
|---|---|
| Date | 2017-08-02 18:20 +0200 |
| Message-ID | <ua4xk-3Io-15@gated-at.bofh.it> |
| In reply to | #1702233 |
Hi, > Am 02.08.2017 um 18:06 schrieb Arnd Bergmann <arnd@arndb.de>: > > On Wed, Aug 2, 2017 at 5:57 PM, Greg Kroah-Hartman > <gregkh@linuxfoundation.org> wrote: >> On Wed, Aug 02, 2017 at 04:05:36PM +0200, H. Nikolaus Schaller wrote: >>> + help >>> + If you say yes here, you get support for the STM >>> + LIS3LV02D sensor. >> >> No name for what the module is called? > > Not sure if that's even relevant here, as the symbol only controls the > base driver that doesn't do anything by itself, while the LIS3_SPI/LIS3_I2C > Kconfig symbols are the interesting ones. > > The help text should also reflect that I think and explain why you would > enable the symbol in the first place (which the patch description doesn't > either). Hm. I start to no longer understand how this LIS302 is configured at all and what SENSORS_LIS3LV02D is good for. I just noticed while preparing some other driver/misc that there is no description in the drivers/misc/Kconfig file while all other CONFIGs have some. But configuring LIS302 works (we have some GTA04 boards where one is installed). BR and thanks, Nikolaus
[toc] | [prev] | [next] | [standalone]
| From | "H. Nikolaus Schaller" <hns@goldelico.com> |
|---|---|
| Date | 2017-08-02 21:00 +0200 |
| Subject | Re: [Letux-kernel] [PATCH] misc: Kconfig: fix missing tristate prompt and add help for LIS3LV02D |
| Message-ID | <ua72a-5hv-19@gated-at.bofh.it> |
| In reply to | #1702245 |
> Am 02.08.2017 um 18:13 schrieb H. Nikolaus Schaller <hns@goldelico.com>: > > Hi, > >> Am 02.08.2017 um 18:06 schrieb Arnd Bergmann <arnd@arndb.de>: >> >> On Wed, Aug 2, 2017 at 5:57 PM, Greg Kroah-Hartman >> <gregkh@linuxfoundation.org> wrote: >>> On Wed, Aug 02, 2017 at 04:05:36PM +0200, H. Nikolaus Schaller wrote: >>>> + help >>>> + If you say yes here, you get support for the STM >>>> + LIS3LV02D sensor. >>> >>> No name for what the module is called? >> >> Not sure if that's even relevant here, as the symbol only controls the >> base driver that doesn't do anything by itself, while the LIS3_SPI/LIS3_I2C >> Kconfig symbols are the interesting ones. >> >> The help text should also reflect that I think and explain why you would >> enable the symbol in the first place (which the patch description doesn't >> either). > > Hm. I start to no longer understand how this LIS302 is configured at all > and what SENSORS_LIS3LV02D is good for. > > I just noticed while preparing some other driver/misc that there is no > description in the drivers/misc/Kconfig file while all other CONFIGs have some. > > But configuring LIS302 works (we have some GTA04 boards where one is installed). Ok, I got it. The config SENSORS_LIS3LV02D is like a temporary variable to build and link the lis3lv02d.o core if either SPI or I2C is enabled. And there is a not very obvious feature that if there is no tristate prompt, the whole variable is hidden. Quite confusing but correct and therefore we should not add one. And if it is not shown, we do not need a help. If we apply my proposed patch, the variable becomes separately configurable and you can e.g. set SENSORS_LIS3LV02D=m and CONFIG_SENSORS_LIS3_I2C=y which confuses the linker. So we should leave it as it is and ignore this patch. BR and thanks for enlightening me, Nikolaus
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web