Re: [PATCH 1/1] v4l: Kconfig: Drop subdev API option help text

[Sakari Ailus <sakari.ailus@iki.fi>]

On Wed, Jul 13, 2022 at 10:47:02PM +0300, Laurent Pinchart wrote:
> On Wed, Jul 13, 2022 at 10:41:01PM +0300, Sakari Ailus wrote:
> > On Wed, Jul 13, 2022 at 10:32:55PM +0300, Laurent Pinchart wrote:
> > > On Wed, Jul 13, 2022 at 08:54:12PM +0300, Laurent Pinchart wrote:
> > > > Hi Sakari,
> > > > 
> > > > Thank you for the patch.
> > > > 
> > > > On Wed, Jul 13, 2022 at 08:39:20PM +0300, Sakari Ailus wrote:
> > > > > The config option text was recently removed, also remove the help text.
> > > > > 
> > > > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > > > 
> > > > Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > > 
> > > Actually, on second thought, isn't it useful to keep the help text as
> > > documentation ?
> > 
> > We haven't generally done that for other such options either. This was
> > intended for end users IMO.
> 
> I don't have a strong opinion, but this came to my attention when
> reviewing your patch that adds selection of GENERIC_PHY_MIPI_DPHY for
> sun6i. The GENERIC_PHY_MIPI_DPHY symbol isn't user-selectable but has a
> help text. A short documentation is useful in my opinion.

I'll leave it ouf of the next pull request then. We can later on decide to
merge it if we like.

-- 
Sakari Ailus