Am Freitag, dem 18.02.2022 um 02:00 +0100 schrieb Marek Vasut: > These functions are specific to (e)DP output initialization and > operation, add specific tc_edp_ prefix to those functions to > discern them from DPI output functions that will be added later > in this series. No functional change. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > Cc: Jonas Karlman <jonas@xxxxxxxxx> > Cc: Laurent Pinchart <Laurent.pinchart@xxxxxxxxxxxxxxxx> > Cc: Maxime Ripard <maxime@xxxxxxxxxx> > Cc: Neil Armstrong <narmstrong@xxxxxxxxxxxx> > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx> Reviewed-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx> > --- > V2: - New patch > --- > drivers/gpu/drm/bridge/tc358767.c | 35 ++++++++++++++++--------------- > 1 file changed, 18 insertions(+), 17 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c > index c23e0abc65e8f..4b8ea0db2a5e8 100644 > --- a/drivers/gpu/drm/bridge/tc358767.c > +++ b/drivers/gpu/drm/bridge/tc358767.c > @@ -1164,7 +1164,7 @@ static int tc_main_link_disable(struct tc_data *tc) > return regmap_write(tc->regmap, DP0CTL, 0); > } > > -static int tc_stream_enable(struct tc_data *tc) > +static int tc_edp_stream_enable(struct tc_data *tc) > { > int ret; > u32 value; > @@ -1219,7 +1219,7 @@ static int tc_stream_enable(struct tc_data *tc) > return 0; > } > > -static int tc_stream_disable(struct tc_data *tc) > +static int tc_edp_stream_disable(struct tc_data *tc) > { > int ret; > > @@ -1234,7 +1234,7 @@ static int tc_stream_disable(struct tc_data *tc) > return 0; > } > > -static void tc_bridge_enable(struct drm_bridge *bridge) > +static void tc_edp_bridge_enable(struct drm_bridge *bridge) > { > struct tc_data *tc = bridge_to_tc(bridge); > int ret; > @@ -1251,7 +1251,7 @@ static void tc_bridge_enable(struct drm_bridge *bridge) > return; > } > > - ret = tc_stream_enable(tc); > + ret = tc_edp_stream_enable(tc); > if (ret < 0) { > dev_err(tc->dev, "main link stream start error: %d\n", ret); > tc_main_link_disable(tc); > @@ -1259,12 +1259,12 @@ static void tc_bridge_enable(struct drm_bridge *bridge) > } > } > > -static void tc_bridge_disable(struct drm_bridge *bridge) > +static void tc_edp_bridge_disable(struct drm_bridge *bridge) > { > struct tc_data *tc = bridge_to_tc(bridge); > int ret; > > - ret = tc_stream_disable(tc); > + ret = tc_edp_stream_disable(tc); > if (ret < 0) > dev_err(tc->dev, "main link stream stop error: %d\n", ret); > > @@ -1285,9 +1285,10 @@ static bool tc_bridge_mode_fixup(struct drm_bridge *bridge, > return true; > } > > -static enum drm_mode_status tc_mode_valid(struct drm_bridge *bridge, > - const struct drm_display_info *info, > - const struct drm_display_mode *mode) > +static enum drm_mode_status > +tc_edp_mode_valid(struct drm_bridge *bridge, > + const struct drm_display_info *info, > + const struct drm_display_mode *mode) > { > struct tc_data *tc = bridge_to_tc(bridge); > u32 req, avail; > @@ -1395,8 +1396,8 @@ static const struct drm_connector_funcs tc_connector_funcs = { > .atomic_destroy_state = drm_atomic_helper_connector_destroy_state, > }; > > -static int tc_bridge_attach(struct drm_bridge *bridge, > - enum drm_bridge_attach_flags flags) > +static int tc_edp_bridge_attach(struct drm_bridge *bridge, > + enum drm_bridge_attach_flags flags) > { > u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24; > struct tc_data *tc = bridge_to_tc(bridge); > @@ -1448,18 +1449,18 @@ static int tc_bridge_attach(struct drm_bridge *bridge, > return ret; > } > > -static void tc_bridge_detach(struct drm_bridge *bridge) > +static void tc_edp_bridge_detach(struct drm_bridge *bridge) > { > drm_dp_aux_unregister(&bridge_to_tc(bridge)->aux); > } > > static const struct drm_bridge_funcs tc_bridge_funcs = { > - .attach = tc_bridge_attach, > - .detach = tc_bridge_detach, > - .mode_valid = tc_mode_valid, > + .attach = tc_edp_bridge_attach, > + .detach = tc_edp_bridge_detach, > + .mode_valid = tc_edp_mode_valid, > .mode_set = tc_bridge_mode_set, > - .enable = tc_bridge_enable, > - .disable = tc_bridge_disable, > + .enable = tc_edp_bridge_enable, > + .disable = tc_edp_bridge_disable, > .mode_fixup = tc_bridge_mode_fixup, > .detect = tc_bridge_detect, > .get_edid = tc_get_edid,