Re: [PATCH 03/11] drm/i915: Clean up some struct/array initializers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 01 Mar 2022, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Use the simple '= {}' form to initialize empty arrays/structs.
> Also add some missing whitespace.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>

> ---
>  drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> index 4e06c8203aca..bc26ebacae12 100644
> --- a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> +++ b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> @@ -832,7 +832,7 @@ hsw_ddi_calculate_wrpll(int clock /* in Hz */,
>  {
>  	u64 freq2k;
>  	unsigned p, n2, r2;
> -	struct hsw_wrpll_rnp best = { 0, 0, 0 };
> +	struct hsw_wrpll_rnp best = {};
>  	unsigned budget;
>  
>  	freq2k = clock / 100;
> @@ -1567,8 +1567,8 @@ skl_ddi_calculate_wrpll(int clock /* in Hz */,
>  static bool skl_ddi_hdmi_pll_dividers(struct intel_crtc_state *crtc_state)
>  {
>  	struct drm_i915_private *i915 = to_i915(crtc_state->uapi.crtc->dev);
> +	struct skl_wrpll_params wrpll_params = {};
>  	u32 ctrl1, cfgcr1, cfgcr2;
> -	struct skl_wrpll_params wrpll_params = { 0, };
>  
>  	/*
>  	 * See comment in intel_dpll_hw_state to understand why we always use 0
> @@ -2095,13 +2095,13 @@ struct bxt_clk_div {
>  
>  /* pre-calculated values for DP linkrates */
>  static const struct bxt_clk_div bxt_dp_clk_val[] = {
> -	{162000, 4, 2, 32, 1677722, 1, 1},
> -	{270000, 4, 1, 27,       0, 0, 1},
> -	{540000, 2, 1, 27,       0, 0, 1},
> -	{216000, 3, 2, 32, 1677722, 1, 1},
> -	{243000, 4, 1, 24, 1258291, 1, 1},
> -	{324000, 4, 1, 32, 1677722, 1, 1},
> -	{432000, 3, 1, 32, 1677722, 1, 1}
> +	{ 162000, 4, 2, 32, 1677722, 1, 1 },
> +	{ 270000, 4, 1, 27,       0, 0, 1 },
> +	{ 540000, 2, 1, 27,       0, 0, 1 },
> +	{ 216000, 3, 2, 32, 1677722, 1, 1 },
> +	{ 243000, 4, 1, 24, 1258291, 1, 1 },
> +	{ 324000, 4, 1, 32, 1677722, 1, 1 },
> +	{ 432000, 3, 1, 32, 1677722, 1, 1 }
>  };
>  
>  static bool

-- 
Jani Nikula, Intel Open Source Graphics Center




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux