On Thu, Jun 15, 2023 at 10:16:35PM +0200, Marek Vasut wrote: > None of these four bits are bitfields, use BIT() macro and treat > them as bits. No functional change. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> I always find FLD_VAL() hard to follow. Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > --- > Cc: Andrzej Hajda <andrzej.hajda@xxxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx> > Cc: David Airlie <airlied@xxxxxxxxx> > Cc: Jernej Skrabec <jernej.skrabec@xxxxxxxxx> > Cc: Jonas Karlman <jonas@xxxxxxxxx> > Cc: Laurent Pinchart <Laurent.pinchart@xxxxxxxxxxxxxxxx> > Cc: Neil Armstrong <neil.armstrong@xxxxxxxxxx> > Cc: Robert Foss <rfoss@xxxxxxxxxx> > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx > --- > drivers/gpu/drm/bridge/tc358764.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/tc358764.c b/drivers/gpu/drm/bridge/tc358764.c > index f85654f1b1045..6a4cd313f5281 100644 > --- a/drivers/gpu/drm/bridge/tc358764.c > +++ b/drivers/gpu/drm/bridge/tc358764.c > @@ -42,10 +42,10 @@ > > /* Video path registers */ > #define VP_CTRL 0x0450 /* Video Path Control */ > -#define VP_CTRL_MSF(v) FLD_VAL(v, 0, 0) /* Magic square in RGB666 */ > -#define VP_CTRL_VTGEN(v) FLD_VAL(v, 4, 4) /* Use chip clock for timing */ > -#define VP_CTRL_EVTMODE(v) FLD_VAL(v, 5, 5) /* Event mode */ > -#define VP_CTRL_RGB888(v) FLD_VAL(v, 8, 8) /* RGB888 mode */ > +#define VP_CTRL_MSF BIT(0) /* Magic square in RGB666 */ > +#define VP_CTRL_VTGEN BIT(4) /* Use chip clock for timing */ > +#define VP_CTRL_EVTMODE BIT(5) /* Event mode */ > +#define VP_CTRL_RGB888 BIT(8) /* RGB888 mode */ > #define VP_CTRL_VSDELAY(v) FLD_VAL(v, 31, 20) /* VSYNC delay */ > #define VP_CTRL_HSPOL BIT(17) /* Polarity of HSYNC signal */ > #define VP_CTRL_DEPOL BIT(18) /* Polarity of DE signal */ > @@ -233,8 +233,8 @@ static int tc358764_init(struct tc358764 *ctx) > tc358764_write(ctx, DSI_STARTDSI, DSI_RX_START); > > /* configure video path */ > - tc358764_write(ctx, VP_CTRL, VP_CTRL_VSDELAY(15) | VP_CTRL_RGB888(1) | > - VP_CTRL_EVTMODE(1) | VP_CTRL_HSPOL | VP_CTRL_VSPOL); > + tc358764_write(ctx, VP_CTRL, VP_CTRL_VSDELAY(15) | VP_CTRL_RGB888 | > + VP_CTRL_EVTMODE | VP_CTRL_HSPOL | VP_CTRL_VSPOL); > > /* reset PHY */ > tc358764_write(ctx, LV_PHY0, LV_PHY0_RST(1) | > -- > 2.39.2