drm/i915/display: Move lvds_channel_mode module parameter under display
authorJouni Högander <jouni.hogander@intel.com>
Tue, 24 Oct 2023 12:40:52 +0000 (15:40 +0300)
committerJouni Högander <jouni.hogander@intel.com>
Thu, 26 Oct 2023 09:14:24 +0000 (12:14 +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-7-jouni.hogander@intel.com
drivers/gpu/drm/i915/display/intel_display_params.c
drivers/gpu/drm/i915/display/intel_display_params.h
drivers/gpu/drm/i915/display/intel_lvds.c
drivers/gpu/drm/i915/i915_params.c
drivers/gpu/drm/i915/i915_params.h

index d00ea5cda4329e45d4874147b874c3d89acb570b..91e2b5056d6365b7b9609baf8a8a25bcb03da7b9 100644 (file)
@@ -30,6 +30,10 @@ static struct intel_display_params intel_display_modparams __read_mostly = {
 intel_display_param_named_unsafe(vbt_firmware, charp, 0400,
        "Load VBT from specified file under /lib/firmware");
 
+intel_display_param_named_unsafe(lvds_channel_mode, int, 0400,
+        "Specify LVDS channel mode "
+        "(0=probe BIOS [default], 1=single-channel, 2=dual-channel)");
+
 intel_display_param_named_unsafe(enable_fbc, int, 0400,
        "Enable frame buffer compression for power savings "
        "(default: -1 (use per-chip default))");
index e51e8a1960ebbde004f527141274f50980e626fc..09bd6614b8133ddafce5d972352dfc974ad218b0 100644 (file)
@@ -25,6 +25,7 @@ struct drm_i915_private;
  */
 #define INTEL_DISPLAY_PARAMS_FOR_EACH(param) \
        param(char *, vbt_firmware, NULL, 0400) \
+       param(int, lvds_channel_mode, 0, 0400) \
        param(int, enable_fbc, -1, 0600) \
        param(int, enable_psr, -1, 0600) \
        param(bool, psr_safest_params, false, 0400) \
index 2a4ca7e65775e5723077add76bab7bad1507752a..4b114fde57b1828c6b45d67594ba360c22b04439 100644 (file)
@@ -794,8 +794,8 @@ static bool compute_is_dual_link_lvds(struct intel_lvds_encoder *lvds_encoder)
        unsigned int val;
 
        /* use the module option value if specified */
-       if (i915->params.lvds_channel_mode > 0)
-               return i915->params.lvds_channel_mode == 2;
+       if (i915->display.params.lvds_channel_mode > 0)
+               return i915->display.params.lvds_channel_mode == 2;
 
        /* single channel LVDS is limited to 112 MHz */
        if (fixed_mode->clock > 112999)
index 9d0535d774c9377f04c7cc6ab46e58ff29096cfb..ea55cc2c48547add78415e56fad2cc4b459d5ba7 100644 (file)
@@ -72,10 +72,6 @@ i915_param_named_unsafe(enable_dc, int, 0400,
        "(-1=auto [default]; 0=disable; 1=up to DC5; 2=up to DC6; "
        "3=up to DC5 with DC3CO; 4=up to DC6 with DC3CO)");
 
-i915_param_named_unsafe(lvds_channel_mode, int, 0400,
-        "Specify LVDS channel mode "
-        "(0=probe BIOS [default], 1=single-channel, 2=dual-channel)");
-
 i915_param_named_unsafe(panel_use_ssc, int, 0400,
        "Use Spread Spectrum Clock with panels [LVDS/eDP] "
        "(default: auto from VBT)");
index 37a1d31a233c1f7b81be4d49bbc1e5c2d8cb83c6..03ec2c2b589d89e68716db5bed63d6c9fc45cee4 100644 (file)
@@ -47,7 +47,6 @@ struct drm_printer;
  */
 #define I915_PARAMS_FOR_EACH(param) \
        param(int, modeset, -1, 0400) \
-       param(int, lvds_channel_mode, 0, 0400) \
        param(int, panel_use_ssc, -1, 0600) \
        param(int, vbt_sdvo_panel_type, -1, 0400) \
        param(int, enable_dc, -1, 0400) \