drm/i915/display: Move edp_vswing module parameter under display
authorJouni Högander <jouni.hogander@intel.com>
Tue, 24 Oct 2023 12:41:01 +0000 (15:41 +0300)
committerJouni Högander <jouni.hogander@intel.com>
Thu, 26 Oct 2023 09:15:31 +0000 (12:15 +0300)
Signed-off-by: Jouni Högander <jouni.hogander@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Luca Coelho <luciano.coelho@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231024124109.384973-16-jouni.hogander@intel.com
drivers/gpu/drm/i915/display/intel_bios.c
drivers/gpu/drm/i915/display/intel_display_params.c
drivers/gpu/drm/i915/display/intel_display_params.h
drivers/gpu/drm/i915/i915_params.c
drivers/gpu/drm/i915/i915_params.h

index 70c0491aac42248d91184fd6aea4c0e8279e1fc0..69db1a3a1499e30d23c975f59a90f17f5d4f8e13 100644 (file)
@@ -1514,9 +1514,9 @@ parse_edp(struct drm_i915_private *i915,
                u8 vswing;
 
                /* Don't read from VBT if module parameter has valid value*/
-               if (i915->params.edp_vswing) {
+               if (i915->display.params.edp_vswing) {
                        panel->vbt.edp.low_vswing =
-                               i915->params.edp_vswing == 1;
+                               i915->display.params.edp_vswing == 1;
                } else {
                        vswing = (edp->edp_vswing_preemph >> (panel_type * 4)) & 0xF;
                        panel->vbt.edp.low_vswing = vswing == 0;
index 2d721afaba3026288c059dd4e29a926cad187ec1..70a4752235129223cb17831c577fa5bad4318adf 100644 (file)
@@ -66,6 +66,12 @@ intel_display_param_named_unsafe(invert_brightness, int, 0400,
        "to dri-devel@lists.freedesktop.org, if your machine needs it. "
        "It will then be included in an upcoming module version.");
 
+/* WA to get away with the default setting in VBT for early platforms.Will be removed */
+intel_display_param_named_unsafe(edp_vswing, int, 0400,
+       "Ignore/Override vswing pre-emph table selection from VBT "
+       "(0=use value from vbt [default], 1=low power swing(200mV),"
+       "2=default swing(400mV))");
+
 intel_display_param_named_unsafe(enable_fbc, int, 0400,
        "Enable frame buffer compression for power savings "
        "(default: -1 (use per-chip default))");
index 4723b45224134d6c6b4f1ea48a0995987ed672b7..10c79ce23063055500b121cfa647b48be9e9fb5b 100644 (file)
@@ -34,6 +34,7 @@ struct drm_i915_private;
        param(int, disable_power_well, -1, 0400) \
        param(bool, enable_ips, true, 0600) \
        param(int, invert_brightness, 0, 0600) \
+       param(int, edp_vswing, 0, 0400) \
        param(int, enable_fbc, -1, 0600) \
        param(int, enable_psr, -1, 0600) \
        param(bool, psr_safest_params, false, 0400) \
index 423fe54484e10bad0b5ef072b60b5529d2e56ee3..6b9df9f9d84277353278f9c032c6baf85e4bfeee 100644 (file)
@@ -111,12 +111,6 @@ i915_param_named(verbose_state_checks, bool, 0600,
 i915_param_named_unsafe(nuclear_pageflip, bool, 0400,
        "Force enable atomic functionality on platforms that don't have full support yet.");
 
-/* WA to get away with the default setting in VBT for early platforms.Will be removed */
-i915_param_named_unsafe(edp_vswing, int, 0400,
-       "Ignore/Override vswing pre-emph table selection from VBT "
-       "(0=use value from vbt [default], 1=low power swing(200mV),"
-       "2=default swing(400mV))");
-
 i915_param_named_unsafe(enable_guc, int, 0400,
        "Enable GuC load for GuC submission and/or HuC load. "
        "Required functionality can be selected using bitmask values. "
index ae0873443a655976a079f393ebb39e28364107bf..c33edaee503210b23d856c42b5c59b3ff339f313 100644 (file)
@@ -55,7 +55,6 @@ struct drm_printer;
        param(char *, gsc_firmware_path, NULL, 0400) \
        param(bool, memtest, false, 0400) \
        param(int, mmio_debug, -IS_ENABLED(CONFIG_DRM_I915_DEBUG_MMIO), 0600) \
-       param(int, edp_vswing, 0, 0400) \
        param(unsigned int, reset, 3, 0600) \
        param(unsigned int, inject_probe_failure, 0, 0) \
        param(int, enable_dpcd_backlight, -1, 0600) \