Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c9546932 authored by Michal Wajdeczko's avatar Michal Wajdeczko Committed by Jani Nikula
Browse files

drm/i915: Introduce custom variant of module_param_named macro 



As we now use same name for public module param and its local
representation we can simplify param definition macro.

Changes done with Coccinelle:

@@
declarer name module_param_named;
declarer name module_param_named_unsafe;
declarer name i915_param_named;
declarer name i915_param_named_unsafe;
identifier n;
@@

(
-module_param_named(n, i915.n,
+i915_module_param_named(n,
  ...);
|
-module_param_named_unsafe(n, i915.n,
+i915_module_param_named_unsafe(n,
 ...);
)

Suggested-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarMichal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170914150805.28376-3-michal.wajdeczko@intel.com
parent 17533bf9

drivers/gpu/drm/i915/i915_params.c

+45 −40
Original line number Diff line number Diff line
@@ -25,6 +25,11 @@ 
#include "i915_params.h"

#include "i915_drv.h"



#define i915_param_named(name, T, perm) \

	module_param_named(name, i915.name, T, perm)

#define i915_param_named_unsafe(name, T, perm) \

	module_param_named_unsafe(name, i915.name, T, perm)



struct i915_params i915 __read_mostly = {

	.modeset = -1,

	.panel_ignore_lid = 1,
@@ -67,22 +72,22 @@ struct i915_params i915 __read_mostly = { 
	.enable_gvt = false,

};



module_param_named(modeset, i915.modeset, int, 0400);

i915_param_named(modeset, int, 0400);

MODULE_PARM_DESC(modeset,

	"Use kernel modesetting [KMS] (0=disable, "

	"1=on, -1=force vga console preference [default])");



module_param_named_unsafe(panel_ignore_lid, i915.panel_ignore_lid, int, 0600);

i915_param_named_unsafe(panel_ignore_lid, int, 0600);

MODULE_PARM_DESC(panel_ignore_lid,

	"Override lid status (0=autodetect, 1=autodetect disabled [default], "

	"-1=force lid closed, -2=force lid open)");



module_param_named_unsafe(semaphores, i915.semaphores, int, 0400);

i915_param_named_unsafe(semaphores, int, 0400);

MODULE_PARM_DESC(semaphores,

	"Use semaphores for inter-ring sync "

	"(default: -1 (use per-chip defaults))");



module_param_named_unsafe(enable_rc6, i915.enable_rc6, int, 0400);

i915_param_named_unsafe(enable_rc6, int, 0400);

MODULE_PARM_DESC(enable_rc6,

	"Enable power-saving render C-state 6. "

	"Different stages can be selected via bitmask values "
@@ -90,100 +95,100 @@ MODULE_PARM_DESC(enable_rc6, 
	"For example, 3 would enable rc6 and deep rc6, and 7 would enable everything. "

	"default: -1 (use per-chip default)");



module_param_named_unsafe(enable_dc, i915.enable_dc, int, 0400);

i915_param_named_unsafe(enable_dc, int, 0400);

MODULE_PARM_DESC(enable_dc,

	"Enable power-saving display C-states. "

	"(-1=auto [default]; 0=disable; 1=up to DC5; 2=up to DC6)");



module_param_named_unsafe(enable_fbc, i915.enable_fbc, int, 0600);

i915_param_named_unsafe(enable_fbc, int, 0600);

MODULE_PARM_DESC(enable_fbc,

	"Enable frame buffer compression for power savings "

	"(default: -1 (use per-chip default))");



module_param_named_unsafe(lvds_channel_mode, i915.lvds_channel_mode, int, 0400);

i915_param_named_unsafe(lvds_channel_mode, int, 0400);

MODULE_PARM_DESC(lvds_channel_mode,

	 "Specify LVDS channel mode "

	 "(0=probe BIOS [default], 1=single-channel, 2=dual-channel)");



module_param_named_unsafe(panel_use_ssc, i915.panel_use_ssc, int, 0600);

i915_param_named_unsafe(panel_use_ssc, int, 0600);

MODULE_PARM_DESC(panel_use_ssc,

	"Use Spread Spectrum Clock with panels [LVDS/eDP] "

	"(default: auto from VBT)");



module_param_named_unsafe(vbt_sdvo_panel_type, i915.vbt_sdvo_panel_type, int, 0400);

i915_param_named_unsafe(vbt_sdvo_panel_type, int, 0400);

MODULE_PARM_DESC(vbt_sdvo_panel_type,

	"Override/Ignore selection of SDVO panel mode in the VBT "

	"(-2=ignore, -1=auto [default], index in VBT BIOS table)");



module_param_named_unsafe(reset, i915.reset, int, 0600);

i915_param_named_unsafe(reset, int, 0600);

MODULE_PARM_DESC(reset, "Attempt GPU resets (0=disabled, 1=full gpu reset, 2=engine reset [default])");



module_param_named_unsafe(vbt_firmware, i915.vbt_firmware, charp, 0400);

i915_param_named_unsafe(vbt_firmware, charp, 0400);

MODULE_PARM_DESC(vbt_firmware,

		 "Load VBT from specified file under /lib/firmware");



#if IS_ENABLED(CONFIG_DRM_I915_CAPTURE_ERROR)

module_param_named(error_capture, i915.error_capture, bool, 0600);

i915_param_named(error_capture, bool, 0600);

MODULE_PARM_DESC(error_capture,

	"Record the GPU state following a hang. "

	"This information in /sys/class/drm/card<N>/error is vital for "

	"triaging and debugging hangs.");

#endif



module_param_named_unsafe(enable_hangcheck, i915.enable_hangcheck, bool, 0644);

i915_param_named_unsafe(enable_hangcheck, bool, 0644);

MODULE_PARM_DESC(enable_hangcheck,

	"Periodically check GPU activity for detecting hangs. "

	"WARNING: Disabling this can cause system wide hangs. "

	"(default: true)");



module_param_named_unsafe(enable_ppgtt, i915.enable_ppgtt, int, 0400);

i915_param_named_unsafe(enable_ppgtt, int, 0400);

MODULE_PARM_DESC(enable_ppgtt,

	"Override PPGTT usage. "

	"(-1=auto [default], 0=disabled, 1=aliasing, 2=full, 3=full with extended address space)");



module_param_named_unsafe(enable_execlists, i915.enable_execlists, int, 0400);

i915_param_named_unsafe(enable_execlists, int, 0400);

MODULE_PARM_DESC(enable_execlists,

	"Override execlists usage. "

	"(-1=auto [default], 0=disabled, 1=enabled)");



module_param_named_unsafe(enable_psr, i915.enable_psr, int, 0600);

i915_param_named_unsafe(enable_psr, int, 0600);

MODULE_PARM_DESC(enable_psr, "Enable PSR "

		 "(0=disabled, 1=enabled - link mode chosen per-platform, 2=force link-standby mode, 3=force link-off mode) "

		 "Default: -1 (use per-chip default)");



module_param_named_unsafe(alpha_support, i915.alpha_support, bool, 0400);

i915_param_named_unsafe(alpha_support, bool, 0400);

MODULE_PARM_DESC(alpha_support,

	"Enable alpha quality driver support for latest hardware. "

	"See also CONFIG_DRM_I915_ALPHA_SUPPORT.");



module_param_named_unsafe(disable_power_well, i915.disable_power_well, int, 0400);

i915_param_named_unsafe(disable_power_well, int, 0400);

MODULE_PARM_DESC(disable_power_well,

	"Disable display power wells when possible "

	"(-1=auto [default], 0=power wells always on, 1=power wells disabled when possible)");



module_param_named_unsafe(enable_ips, i915.enable_ips, int, 0600);

i915_param_named_unsafe(enable_ips, int, 0600);

MODULE_PARM_DESC(enable_ips, "Enable IPS (default: true)");



module_param_named(fastboot, i915.fastboot, bool, 0600);

i915_param_named(fastboot, bool, 0600);

MODULE_PARM_DESC(fastboot,

	"Try to skip unnecessary mode sets at boot time (default: false)");



module_param_named_unsafe(prefault_disable, i915.prefault_disable, bool, 0600);

i915_param_named_unsafe(prefault_disable, bool, 0600);

MODULE_PARM_DESC(prefault_disable,

	"Disable page prefaulting for pread/pwrite/reloc (default:false). "

	"For developers only.");



module_param_named_unsafe(load_detect_test, i915.load_detect_test, bool, 0600);

i915_param_named_unsafe(load_detect_test, bool, 0600);

MODULE_PARM_DESC(load_detect_test,

	"Force-enable the VGA load detect code for testing (default:false). "

	"For developers only.");



module_param_named_unsafe(force_reset_modeset_test, i915.force_reset_modeset_test, bool, 0600);

i915_param_named_unsafe(force_reset_modeset_test, bool, 0600);

MODULE_PARM_DESC(force_reset_modeset_test,

	"Force a modeset during gpu reset for testing (default:false). "

	"For developers only.");



module_param_named_unsafe(invert_brightness, i915.invert_brightness, int, 0600);

i915_param_named_unsafe(invert_brightness, int, 0600);

MODULE_PARM_DESC(invert_brightness,

	"Invert backlight brightness "

	"(-1 force normal, 0 machine defaults, 1 force inversion), please "
@@ -191,69 +196,69 @@ MODULE_PARM_DESC(invert_brightness, 
	"to dri-devel@lists.freedesktop.org, if your machine needs it. "

	"It will then be included in an upcoming module version.");



module_param_named(disable_display, i915.disable_display, bool, 0400);

i915_param_named(disable_display, bool, 0400);

MODULE_PARM_DESC(disable_display, "Disable display (default: false)");



module_param_named_unsafe(enable_cmd_parser, i915.enable_cmd_parser, bool, 0400);

i915_param_named_unsafe(enable_cmd_parser, bool, 0400);

MODULE_PARM_DESC(enable_cmd_parser,

		 "Enable command parsing (true=enabled [default], false=disabled)");



module_param_named_unsafe(use_mmio_flip, i915.use_mmio_flip, int, 0600);

i915_param_named_unsafe(use_mmio_flip, int, 0600);

MODULE_PARM_DESC(use_mmio_flip,

		 "use MMIO flips (-1=never, 0=driver discretion [default], 1=always)");



module_param_named(mmio_debug, i915.mmio_debug, int, 0600);

i915_param_named(mmio_debug, int, 0600);

MODULE_PARM_DESC(mmio_debug,

	"Enable the MMIO debug code for the first N failures (default: off). "

	"This may negatively affect performance.");



module_param_named(verbose_state_checks, i915.verbose_state_checks, bool, 0600);

i915_param_named(verbose_state_checks, bool, 0600);

MODULE_PARM_DESC(verbose_state_checks,

	"Enable verbose logs (ie. WARN_ON()) in case of unexpected hw state conditions.");



module_param_named_unsafe(nuclear_pageflip, i915.nuclear_pageflip, bool, 0400);

i915_param_named_unsafe(nuclear_pageflip, bool, 0400);

MODULE_PARM_DESC(nuclear_pageflip,

		 "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 */

module_param_named_unsafe(edp_vswing, i915.edp_vswing, int, 0400);

i915_param_named_unsafe(edp_vswing, int, 0400);

MODULE_PARM_DESC(edp_vswing,

		 "Ignore/Override vswing pre-emph table selection from VBT "

		 "(0=use value from vbt [default], 1=low power swing(200mV),"

		 "2=default swing(400mV))");



module_param_named_unsafe(enable_guc_loading, i915.enable_guc_loading, int, 0400);

i915_param_named_unsafe(enable_guc_loading, int, 0400);

MODULE_PARM_DESC(enable_guc_loading,

		"Enable GuC firmware loading "

		"(-1=auto, 0=never [default], 1=if available, 2=required)");



module_param_named_unsafe(enable_guc_submission, i915.enable_guc_submission, int, 0400);

i915_param_named_unsafe(enable_guc_submission, int, 0400);

MODULE_PARM_DESC(enable_guc_submission,

		"Enable GuC submission "

		"(-1=auto, 0=never [default], 1=if available, 2=required)");



module_param_named(guc_log_level, i915.guc_log_level, int, 0400);

i915_param_named(guc_log_level, int, 0400);

MODULE_PARM_DESC(guc_log_level,

	"GuC firmware logging level (-1:disabled (default), 0-3:enabled)");



module_param_named_unsafe(guc_firmware_path, i915.guc_firmware_path, charp, 0400);

i915_param_named_unsafe(guc_firmware_path, charp, 0400);

MODULE_PARM_DESC(guc_firmware_path,

	"GuC firmware path to use instead of the default one");



module_param_named_unsafe(huc_firmware_path, i915.huc_firmware_path, charp, 0400);

i915_param_named_unsafe(huc_firmware_path, charp, 0400);

MODULE_PARM_DESC(huc_firmware_path,

	"HuC firmware path to use instead of the default one");



module_param_named_unsafe(enable_dp_mst, i915.enable_dp_mst, bool, 0600);

i915_param_named_unsafe(enable_dp_mst, bool, 0600);

MODULE_PARM_DESC(enable_dp_mst,

	"Enable multi-stream transport (MST) for new DisplayPort sinks. (default: true)");

module_param_named_unsafe(inject_load_failure, i915.inject_load_failure, uint, 0400);

i915_param_named_unsafe(inject_load_failure, uint, 0400);

MODULE_PARM_DESC(inject_load_failure,

	"Force an error after a number of failure check points (0:disabled (default), N:force failure at the Nth failure check point)");

module_param_named(enable_dpcd_backlight, i915.enable_dpcd_backlight, bool, 0600);

i915_param_named(enable_dpcd_backlight, bool, 0600);

MODULE_PARM_DESC(enable_dpcd_backlight,

	"Enable support for DPCD backlight control (default:false)");



module_param_named(enable_gvt, i915.enable_gvt, bool, 0400);

i915_param_named(enable_gvt, bool, 0400);

MODULE_PARM_DESC(enable_gvt,

	"Enable support for Intel GVT-g graphics virtualization host support(default:false)");