Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-13 Thread Bartlomiej Zolnierkiewicz
On Thursday, February 01, 2018 05:56:18 PM Ulf Magnusson wrote: > On Thu, Feb 1, 2018 at 4:52 PM, Bartlomiej Zolnierkiewicz > wrote: > > > > Hi, > > > > On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: > >> Blank help texts are probably either a typo, a

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-13 Thread Bartlomiej Zolnierkiewicz
On Thursday, February 01, 2018 05:56:18 PM Ulf Magnusson wrote: > On Thu, Feb 1, 2018 at 4:52 PM, Bartlomiej Zolnierkiewicz > wrote: > > > > Hi, > > > > On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: > >> Blank help texts are probably either a typo, a Kconfig misunderstanding, >

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-01 Thread Masahiro Yamada
2018-02-02 1:56 GMT+09:00 Ulf Magnusson : > On Thu, Feb 1, 2018 at 4:52 PM, Bartlomiej Zolnierkiewicz > wrote: >> >> Hi, >> >> On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: >>> Blank help texts are probably either a typo, a Kconfig

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-01 Thread Masahiro Yamada
2018-02-02 1:56 GMT+09:00 Ulf Magnusson : > On Thu, Feb 1, 2018 at 4:52 PM, Bartlomiej Zolnierkiewicz > wrote: >> >> Hi, >> >> On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: >>> Blank help texts are probably either a typo, a Kconfig misunderstanding, >>> or some kind of

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-01 Thread Ulf Magnusson
On Thu, Feb 1, 2018 at 4:52 PM, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: >> Blank help texts are probably either a typo, a Kconfig misunderstanding, >> or some kind of half-committing to adding a help

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-01 Thread Ulf Magnusson
On Thu, Feb 1, 2018 at 4:52 PM, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: >> Blank help texts are probably either a typo, a Kconfig misunderstanding, >> or some kind of half-committing to adding a help text (in which case a >>

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-01 Thread Bartlomiej Zolnierkiewicz
Hi, On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right

Re: [PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-02-01 Thread Bartlomiej Zolnierkiewicz
Hi, On Wednesday, January 31, 2018 10:34:21 AM Ulf Magnusson wrote: > Blank help texts are probably either a typo, a Kconfig misunderstanding, > or some kind of half-committing to adding a help text (in which case a > TODO comment would be clearer, if the help text really can't be added > right

[PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-01-31 Thread Ulf Magnusson
Blank help texts are probably either a typo, a Kconfig misunderstanding, or some kind of half-committing to adding a help text (in which case a TODO comment would be clearer, if the help text really can't be added right away). Best to remove them, IMO. Signed-off-by: Ulf Magnusson

[PATCH 02/11] video: fbdev: kconfig: Remove blank help text

2018-01-31 Thread Ulf Magnusson
Blank help texts are probably either a typo, a Kconfig misunderstanding, or some kind of half-committing to adding a help text (in which case a TODO comment would be clearer, if the help text really can't be added right away). Best to remove them, IMO. Signed-off-by: Ulf Magnusson ---