Re: [PATCH v1 01/35] drm/atomic-helper: Rename drm_atomic_helper_connector_tv_reset to avoid ambiguity
From: Noralf Trønnes
Date: Mon Aug 08 2022 - 08:25:49 EST
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>