Re: [PATCH] drm/i915/xe3lpd: Load DMC

["Taylor, Clinton A" <clinton.a.taylor@intel.com>]

Reviewed-by: Clint Taylor <Clinton.A.Taylor@intel.com>

-Clint

On Tue, 2024-10-22 at 12:50 -0300, Gustavo Sousa wrote:
> Load the DMC for Xe3LPD.
> 
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_dmc.c | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c
> b/drivers/gpu/drm/i915/display/intel_dmc.c
> index 48bbbf8f312c..5392b68627ae 100644
> --- a/drivers/gpu/drm/i915/display/intel_dmc.c
> +++ b/drivers/gpu/drm/i915/display/intel_dmc.c
> @@ -113,6 +113,9 @@ static bool dmc_firmware_param_disabled(struct intel_display
> *display)
>  #define DISPLAY_VER13_DMC_MAX_FW_SIZE	0x20000
>  #define DISPLAY_VER12_DMC_MAX_FW_SIZE	ICL_DMC_MAX_FW_SIZE
>  
> +#define XE3LPD_DMC_PATH			DMC_PATH(xe3lpd)
> +MODULE_FIRMWARE(XE3LPD_DMC_PATH);
> +
>  #define XE2LPD_DMC_PATH			DMC_PATH(xe2lpd)
>  MODULE_FIRMWARE(XE2LPD_DMC_PATH);
>  
> @@ -168,7 +171,10 @@ static const char *dmc_firmware_default(struct intel_display
> *display, u32 *size
>  	const char *fw_path = NULL;
>  	u32 max_fw_size = 0;
>  
> -	if (DISPLAY_VER_FULL(display) == IP_VER(20, 0)) {
> +	if (DISPLAY_VER_FULL(display) == IP_VER(30, 0)) {
> +		fw_path = XE3LPD_DMC_PATH;
> +		max_fw_size = XE2LPD_DMC_MAX_FW_SIZE;
> +	} else if (DISPLAY_VER_FULL(display) == IP_VER(20, 0)) {
>  		fw_path = XE2LPD_DMC_PATH;
>  		max_fw_size = XE2LPD_DMC_MAX_FW_SIZE;
>  	} else if (DISPLAY_VER_FULL(display) == IP_VER(14, 1)) {