On Fri, Mar 20, 2026 at 07:19:06AM +0000, cristian_ci wrote: > On Thursday, March 19th, 2026 at 03:27, Dmitry Baryshkov > <[email protected]> wrote: > > > On Wed, Mar 18, 2026 at 11:28:10PM +0100, Cristian Cozzolino via B4 Relay > > wrote: > > > From: Cristian Cozzolino <[email protected]> > > > > > > Add support for Novatek NT35532-based 1080p video mode DSI panel. > > > > > > Signed-off-by: Cristian Cozzolino <[email protected]> > > > --- > > > MAINTAINERS | 1 + > > > drivers/gpu/drm/panel/Kconfig | 11 + > > > drivers/gpu/drm/panel/Makefile | 1 + > > > drivers/gpu/drm/panel/panel-novatek-nt35532.c | 779 > > > ++++++++++++++++++++++++++ > > > 4 files changed, 792 insertions(+) > > > > > > > Reviewed-by: Dmitry Baryshkov <[email protected]> > > I'm doing a test: since I'm not sure if: > > select VIDEOMODE_HELPERS > > is actually needed in Kconfig nt35532 entry, I've temporarily > removed that select line and regenerated .config file. Though, > I've noticed CONFIG_VIDEOMODE_HELPERS was still set to 'y' > nevetheless, so I decided to not going forward since that > code would have been built anyway so that there was no > point for doing that. I wonder: is that select line really > necessary, at this point?
No, it is not used (you don't use corresponding API functions). You can drop it. -- With best wishes Dmitry
