On 2020-12-22 at 20:19:43 +0530, Jani Nikula wrote:
> Follow the usual naming pattern for functions.
>
> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
Looks good to me.
Reviewed-by: Anshuman Gupta <anshuman.gupta@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 6 +++---
> drivers/gpu/drm/i915/display/intel_pps.c | 10 +++++-----
> drivers/gpu/drm/i915/display/intel_pps.h | 6 +++---
> 3 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
> index 0870872fb594..9813fb7e109c 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -2536,7 +2536,7 @@ void intel_edp_backlight_on(const struct intel_crtc_state *crtc_state,
> drm_dbg_kms(&i915->drm, "\n");
>
> intel_panel_enable_backlight(crtc_state, conn_state);
> - _intel_edp_backlight_on(intel_dp);
> + intel_pps_backlight_on(intel_dp);
> }
>
> /* Disable backlight PP control and backlight PWM. */
> @@ -2550,7 +2550,7 @@ void intel_edp_backlight_off(const struct drm_connector_state *old_conn_state)
>
> drm_dbg_kms(&i915->drm, "\n");
>
> - _intel_edp_backlight_off(intel_dp);
> + intel_pps_backlight_off(intel_dp);
> intel_panel_disable_backlight(old_conn_state);
> }
>
> @@ -6688,7 +6688,7 @@ static bool intel_edp_init_connector(struct intel_dp *intel_dp,
> }
>
> intel_panel_init(&intel_connector->panel, fixed_mode, downclock_mode);
> - intel_connector->panel.backlight.power = intel_edp_backlight_power;
> + intel_connector->panel.backlight.power = intel_pps_backlight_power;
> intel_panel_setup_backlight(connector, pipe);
>
> if (fixed_mode) {
> diff --git a/drivers/gpu/drm/i915/display/intel_pps.c b/drivers/gpu/drm/i915/display/intel_pps.c
> index 9b0c432552b7..0edda87dee94 100644
> --- a/drivers/gpu/drm/i915/display/intel_pps.c
> +++ b/drivers/gpu/drm/i915/display/intel_pps.c
> @@ -847,7 +847,7 @@ void intel_edp_panel_off(struct intel_dp *intel_dp)
> }
>
> /* Enable backlight in the panel power control. */
> -void _intel_edp_backlight_on(struct intel_dp *intel_dp)
> +void intel_pps_backlight_on(struct intel_dp *intel_dp)
> {
> struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
> intel_wakeref_t wakeref;
> @@ -873,7 +873,7 @@ void _intel_edp_backlight_on(struct intel_dp *intel_dp)
> }
>
> /* Disable backlight in the panel power control. */
> -void _intel_edp_backlight_off(struct intel_dp *intel_dp)
> +void intel_pps_backlight_off(struct intel_dp *intel_dp)
> {
> struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
> intel_wakeref_t wakeref;
> @@ -900,7 +900,7 @@ void _intel_edp_backlight_off(struct intel_dp *intel_dp)
> * Hook for controlling the panel power control backlight through the bl_power
> * sysfs attribute. Take care to handle multiple calls.
> */
> -void intel_edp_backlight_power(struct intel_connector *connector, bool enable)
> +void intel_pps_backlight_power(struct intel_connector *connector, bool enable)
> {
> struct drm_i915_private *i915 = to_i915(connector->base.dev);
> struct intel_dp *intel_dp = intel_attached_dp(connector);
> @@ -917,9 +917,9 @@ void intel_edp_backlight_power(struct intel_connector *connector, bool enable)
> enable ? "enable" : "disable");
>
> if (enable)
> - _intel_edp_backlight_on(intel_dp);
> + intel_pps_backlight_on(intel_dp);
> else
> - _intel_edp_backlight_off(intel_dp);
> + intel_pps_backlight_off(intel_dp);
> }
>
> static void vlv_detach_power_sequencer(struct intel_dp *intel_dp)
> diff --git a/drivers/gpu/drm/i915/display/intel_pps.h b/drivers/gpu/drm/i915/display/intel_pps.h
> index f44e6ce9e8c1..81e4e9fc3cf5 100644
> --- a/drivers/gpu/drm/i915/display/intel_pps.h
> +++ b/drivers/gpu/drm/i915/display/intel_pps.h
> @@ -23,9 +23,9 @@ intel_wakeref_t intel_pps_unlock(struct intel_dp *intel_dp, intel_wakeref_t wake
> for ((wf) = intel_pps_lock(dp); (wf); (wf) = intel_pps_unlock((dp), (wf)))
>
> void intel_dp_check_edp(struct intel_dp *intel_dp);
> -void _intel_edp_backlight_on(struct intel_dp *intel_dp);
> -void _intel_edp_backlight_off(struct intel_dp *intel_dp);
> -void intel_edp_backlight_power(struct intel_connector *connector, bool enable);
> +void intel_pps_backlight_on(struct intel_dp *intel_dp);
> +void intel_pps_backlight_off(struct intel_dp *intel_dp);
> +void intel_pps_backlight_power(struct intel_connector *connector, bool enable);
>
> bool edp_panel_vdd_on(struct intel_dp *intel_dp);
> void edp_panel_vdd_off(struct intel_dp *intel_dp, bool sync);
> --
> 2.20.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
