[Intel-gfx] [PATCH 43/67] drm/i915: Use HAS_CSR instead of gen number on DMC load.

[Animesh Manna]

On 4/7/2017 12:45 AM, Rodrigo Vivi wrote:
> Since we have HAS_CSR tied to the platform definition
> let's use this instead of checking per platform.
>
> One less thing to worry when adding support to new platforms.
>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
>   drivers/gpu/drm/i915/intel_csr.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
> index 496a965..eecee65 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -241,7 +241,7 @@ void intel_csr_load_program(struct drm_i915_private *dev_priv)
>   	u32 *payload = dev_priv->csr.dmc_payload;
>   	uint32_t i, fw_size;
>   
> -	if (!IS_GEN9(dev_priv)) {
> +	if (!HAS_CSR(dev_priv)) {
>   		DRM_ERROR("No CSR support available for this platform\n");
>   		return;
>   	}

Changes looks good to me.

Reviewed-by: Animesh Manna<animesh.manna at intel.com>

Regards,

Animesh


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20170522/dcca2562/attachment-0001.html>