On Fri, Mar 22, 2024 at 10:48:43AM +0200, Jani Nikula wrote:
On Thu, 21 Mar 2024, Lucas De Marchi <lucas.demar...@intel.com> wrote:
On Thu, Mar 21, 2024 at 06:20:57PM +0200, Jani Nikula wrote:
On Thu, 21 Mar 2024, Jani Nikula <jani.nik...@intel.com> wrote:
The dmc_firmware_path parameter is clearly a display parameter. Move it
there. This also cleans up the ugly member in struct xe_device.
Hmm, does this actually fix the parameter on xe?!
not sure what you meant here. Patches makes sense to me.
Unless I'm mistaken, the parameter currently does not work on xe at
all. I only realized after posting the patch that this probably fixes
that issue too.
+Gustavo
I never really tried the dmc_firmware_path param, but current code
contains:
drivers/gpu/drm/i915/display/intel_dmc.c: if (err == -ENOENT &&
!i915->params.dmc_firmware_path) {
drivers/gpu/drm/i915/display/intel_dmc.c: if
(i915->params.dmc_firmware_path) {
drivers/gpu/drm/i915/display/intel_dmc.c: if
(strlen(i915->params.dmc_firmware_path) == 0) {
drivers/gpu/drm/i915/display/intel_dmc.c: dmc->fw_path =
i915->params.dmc_firmware_path;
drivers/gpu/drm/i915/i915_params.c:i915_param_named_unsafe(dmc_firmware_path,
charp, 0400,
drivers/gpu/drm/i915/i915_params.h: param(char *, dmc_firmware_path, NULL,
0400) \
drivers/gpu/drm/xe/xe_device_types.h: const char *dmc_firmware_path;
So dmc_firmware_path is only dealt with inside display... it would be
odd if it was not working. I'll double check.
Lucas De Marchi
