Re: [PATCH v1 3/3] drm/i915: Invert if/else ladder for stolen init

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

 



On Thu, Sep 08, 2022 at 01:08:16PM -0700, Lucas De Marchi wrote:
> Continue converting the driver to the convention of last version first,
> extending it to the future platforms. Now, any GRAPHICS_VER >= 11 will
> be handled by the first branch.
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>

Reviewed-by: Gustavo Sousa <gustavo.sousa@xxxxxxxxx>

> 
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> index 4f4c9461a23b..acc561c0f0aa 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> @@ -430,48 +430,29 @@ static int i915_gem_init_stolen(struct intel_memory_region *mem)
>  	reserved_base = stolen_top;
>  	reserved_size = 0;
>  
> -	switch (GRAPHICS_VER(i915)) {
> -	case 2:
> -	case 3:
> -		break;
> -	case 4:
> -		if (!IS_G4X(i915))
> -			break;
> -		fallthrough;
> -	case 5:
> -		g4x_get_stolen_reserved(i915, uncore,
> +	if (GRAPHICS_VER(i915) >= 11) {
> +		icl_get_stolen_reserved(i915, uncore,
>  					&reserved_base, &reserved_size);
> -		break;
> -	case 6:
> -		gen6_get_stolen_reserved(i915, uncore,
> -					 &reserved_base, &reserved_size);
> -		break;
> -	case 7:
> -		if (IS_VALLEYVIEW(i915))
> -			vlv_get_stolen_reserved(i915, uncore,
> -						&reserved_base, &reserved_size);
> -		else
> -			gen7_get_stolen_reserved(i915, uncore,
> -						 &reserved_base, &reserved_size);
> -		break;
> -	case 8:
> -	case 9:
> +	} else if (GRAPHICS_VER(i915) >= 8) {
>  		if (IS_LP(i915))
>  			chv_get_stolen_reserved(i915, uncore,
>  						&reserved_base, &reserved_size);
>  		else
>  			bdw_get_stolen_reserved(i915, uncore,
>  						&reserved_base, &reserved_size);
> -		break;
> -	default:
> -		MISSING_CASE(GRAPHICS_VER(i915));
> -		fallthrough;
> -	case 11:
> -	case 12:
> -		icl_get_stolen_reserved(i915, uncore,
> -					&reserved_base,
> -					&reserved_size);
> -		break;
> +	} else if (GRAPHICS_VER(i915) >= 7) {
> +		if (IS_VALLEYVIEW(i915))
> +			vlv_get_stolen_reserved(i915, uncore,
> +						&reserved_base, &reserved_size);
> +		else
> +			gen7_get_stolen_reserved(i915, uncore,
> +						 &reserved_base, &reserved_size);
> +	} else if (GRAPHICS_VER(i915) >= 6) {
> +		gen6_get_stolen_reserved(i915, uncore,
> +					 &reserved_base, &reserved_size);
> +	} else if (GRAPHICS_VER(i915) >= 5 || IS_G4X(i915)) {
> +		g4x_get_stolen_reserved(i915, uncore,
> +					&reserved_base, &reserved_size);
>  	}
>  
>  	/*
> 
> -- 
> b4 0.10.0-dev-df873



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

  Powered by Linux