Quoting Taylor, Clinton A (2024-10-22 14:43:52-03:00) >Reviewed-by: Clint Taylor <Clinton.A.Taylor@xxxxxxxxx> Pushed to drm-intel-next. Thanks for the review! -- Gustavo Sousa > >-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@xxxxxxxxx> >> --- >> 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)) {