[PATCH v1 01/35] drm/atomic-helper: Rename drm_atomic_helper_connector_tv_reset to avoid ambiguity
Noralf Trønnes
noralf at tronnes.org
Mon Aug 8 05:25:34 PDT 2022
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 at cerno.tech>
>
Reviewed-by: Noralf Trønnes <noralf at tronnes.org>
More information about the linux-amlogic
mailing list