Hi, On Thu, Aug 31, 2023 at 1:10 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > Having conditional around the of_node pointers turns out to make driver > code use ugly #ifdef and #if blocks. So drop the conditionals. > > Suggested-by: Douglas Anderson <dianders@xxxxxxxxxxxx> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> > --- > v5->v6: > * Added Rb tag from Douglas Anderson. > * Dropped conditionals from remaining drm/bridge drivers. > v5: > * Split from patch#2 > --- > drivers/gpu/drm/bridge/analogix/analogix-anx78xx.c | 2 -- > drivers/gpu/drm/bridge/panel.c | 2 -- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 -- > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 -- > 4 files changed, 8 deletions(-) I think this has had enough time to marinate, so landed to drm-misc-next: 481fc9e7e11d drm/bridge: Drop CONFIG_OF conditionals around of_node pointers