On Wed, 24 Sep 2025, Marek Vasut <[email protected]> wrote: > Convert register bits to BIT() macro and bitfields to GENMASK()/FIELD_PREP() > macros. > Most of this patchset is boring mechanical conversion.
I suggest using the relatively new sized BIT_U32() and GENMASK_U32(), or other sizes if your register sizes are different. You'll get better compile time protection against mistakes, and sometimes the unsigned long type of plain BIT() and GENMASK() can be annoying. BR, Jani. > > Noteworthy patches are 6 and 7 , those introduce handling of DSI mode flags > and convert use of DRM_MODE_FLAG_P.SYNC into DRM_MODE_FLAG_N.SYNC, but that > should not have any adverse effect on existing hardware. > > Marek Vasut (10): > drm/rcar-du: dsi: Fix missing parameter in RXSETR_...EN macros > drm/rcar-du: dsi: Document TXVMSETR PIXWDTH as bitfield > drm/rcar-du: dsi: Deduplicate mipi_dsi_pixel_format_to_bpp() usage > drm/rcar-du: dsi: Clean up VCLKSET register macros > drm/rcar-du: dsi: Clean up CLOCKSET1 CLKINSEL macros > drm/rcar-du: dsi: Clean up TXVMPSPHSETR DT macros > drm/rcar-du: dsi: Respect DSI mode flags > drm/rcar-du: dsi: Clean up handling of DRM mode flags > drm/rcar-du: dsi: Convert register bits to BIT() macro > drm/rcar-du: dsi: Convert register bitfields to GENMASK() macro > > .../gpu/drm/renesas/rcar-du/rcar_mipi_dsi.c | 89 +++-- > .../drm/renesas/rcar-du/rcar_mipi_dsi_regs.h | 363 +++++++++--------- > 2 files changed, 232 insertions(+), 220 deletions(-) > > --- > Cc: David Airlie <[email protected]> > Cc: Geert Uytterhoeven <[email protected]> > Cc: Kieran Bingham <[email protected]> > Cc: Laurent Pinchart <[email protected]> > Cc: Maarten Lankhorst <[email protected]> > Cc: Magnus Damm <[email protected]> > Cc: Maxime Ripard <[email protected]> > Cc: Simona Vetter <[email protected]> > Cc: Thomas Zimmermann <[email protected]> > Cc: Tomi Valkeinen <[email protected]> > Cc: [email protected] > Cc: [email protected] -- Jani Nikula, Intel
