On Fri, 06 Sep 2024, Ville Syrjala <[email protected]> wrote:
> From: Ville Syrjälä <[email protected]>
>
> CodingStyle says when one branch of an if ladder is braced
> then all of them should be. Make it so.
>
> Signed-off-by: Ville Syrjälä <[email protected]>

Reviewed-by: Jani Nikula <[email protected]>

> ---
>  drivers/gpu/drm/i915/display/intel_cdclk.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c 
> b/drivers/gpu/drm/i915/display/intel_cdclk.c
> index 66964c7d2a2c..9d870d15d888 100644
> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> @@ -2053,8 +2053,9 @@ static void _bxt_set_cdclk(struct drm_i915_private 
> *dev_priv,
>                       dg2_cdclk_squash_program(dev_priv, 0);
>  
>               icl_cdclk_pll_update(dev_priv, vco);
> -     } else
> +     } else {
>               bxt_cdclk_pll_update(dev_priv, vco);
> +     }
>  
>       if (HAS_CDCLK_SQUASH(dev_priv)) {
>               u16 waveform = cdclk_squash_waveform(dev_priv, cdclk);

-- 
Jani Nikula, Intel

Reply via email to