Den 29.07.2022 18.34, skrev Maxime Ripard: > We currently have two sets of TV properties. > > The first one is there to deal with analog TV properties, creating > properties such as the TV mode, subconnectors, saturation, hue and so on. > It's created by calling the drm_mode_create_tv_properties() function. > > The second one is there to deal with properties that might be useful on a > TV, creating the overscan margins for example. It's created by calling the > drm_mode_create_tv_margin_properties(). > > However, we also have a drm_atomic_helper_connector_tv_reset() function > that will reset the TV margin properties to their default values, and thus > is supposed to be called for the latter set. This creates an ambiguity due > to the inconsistent naming. > > We can thus rename the drm_atomic_helper_connector_tv_reset() function to > drm_atomic_helper_connector_tv_margins_reset() to remove that ambiguity > and hopefully make it more obvious. > > Signed-off-by: Maxime Ripard <maxime@xxxxxxxxxx> > Reviewed-by: Noralf Trønnes <noralf@xxxxxxxxxxx>
