From patchwork Mon Oct 16 11:16:34 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Jouni_H=C3=B6gander?= X-Patchwork-Id: 13423049 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C1897CDB465 for ; Mon, 16 Oct 2023 11:17:16 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 205FF10E04A; Mon, 16 Oct 2023 11:17:16 +0000 (UTC) Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id C6A0E10E04A for ; Mon, 16 Oct 2023 11:17:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1697455034; x=1728991034; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=2UWn1CuX5LRL1RaquQ7B9Xcf/CNSZxpXyVdIAfMI8uQ=; b=MqKV20Ls3ggAOLZ6nVXh5XNw1qtLxIcCMxwkdu9z8mLDWYj7b6e+Q3D6 O8bdWK+DDBlfZOURiWgVNor26YUeUCt5xzhbottjgciquVrbUeVGTxNA8 5xIED/Tnb6IChxCjXgSS00f91GqQkJWRc6Hk/itb2f83DZYVT7ZKgzY43 abwCoIaAghVO1QptOdbkcRXxObajMeirQu7xvK3y9jw2qoqn25HBbtjDP NZGqT4441KQKXAOAVwXEfTXzLU7YRCZO9iFe+NKweX4kcdDozOHwOWtNu winmldu3J1dNEdH76uaC3MInPB/y8u8zJCNojOuZWVcoJshgh+Y6jvuaS A==; X-IronPort-AV: E=McAfee;i="6600,9927,10863"; a="385341334" X-IronPort-AV: E=Sophos;i="6.03,229,1694761200"; d="scan'208";a="385341334" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Oct 2023 04:17:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10863"; a="821532513" X-IronPort-AV: E=Sophos;i="6.03,229,1694761200"; d="scan'208";a="821532513" Received: from dleve-mobl.ger.corp.intel.com (HELO jhogande-mobl1.intel.com) ([10.252.41.143]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Oct 2023 04:17:11 -0700 From: =?utf-8?q?Jouni_H=C3=B6gander?= To: intel-gfx@lists.freedesktop.org Date: Mon, 16 Oct 2023 14:16:34 +0300 Message-Id: <20231016111658.3432581-1-jouni.hogander@intel.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Subject: [Intel-gfx] [PATCH v2 00/24] Framework for display parameters X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jani Nikula , Rodrigo Vivi Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" Currently all module parameters are handled by i915_param.c/h. This is a problem for display parameters when Xe driver is used. This patch set adds a mechanism to add parameters specific to the display. This is mainly copied from existing i915 parameters implementation with some naming changes and taking into account varying driver name. Also all display specific module parameters are moved under display and the module parameter are all converted as non-writable. This should be ok as we have writable device parameters under debugfs. v2: - Drop fastboot parameter - Include display parameters into i915_capabilities debugfs interface Cc: Jani Nikula Cc: Uma Shankar Cc: Maarten Lankhorst Cc: Ville Syrjälä Cc: Rodrigo Vivi Cc: Tvrtko Ursulin Cc: Joonas Lahtinen Jouni Högander (24): drm/i915/display: Add framework to add parameters specific to display drm/i915/display: Dump also display parameters drm/i915/display: Move enable_fbc module parameter under display drm/i915/display: Move psr related module parameters under display drm/i915/display: Move vbt_firmware module parameter under display drm/i915/display: Move lvds_channel_mode module parameter under display drm/i915/display: Move panel_use_ssc module parameter under display drm/i915/display: Move vbt_sdvo_panel_type module parameter under display drm/i915/display: Move enable_dc module parameter under display drm/i915/display: Move enable_dpt module parameter under display drm/i915/display: Move enable_sagv module parameter under display drm/i915/display: Move disable_power_well module parameter under display drm/i915/display: Move enable_ips module parameter under display drm/i915/display: Move invert_brightness module parameter under display drm/i915/display: Move edp_vswing module parameter under display drm/i915/display: Move enable_dpcd_backlightmodule parameter under display drm/i915/display: Move load_detect_test parameter under display drm/i915/display: Move force_reset_modeset_test parameter under display drm/i915/display: Move disable_display parameter under display drm/i915/display: Use device parameters instead of module in I915_STATE_WARN drm/i915/display: Move verbose_state_checks under display drm/i915/display: Move nuclear_pageflip under display drm/i915/display: Move enable_dp_mst under display drm/i915/display: Use same permissions for enable_sagv as for rest drivers/gpu/drm/i915/Makefile | 2 + drivers/gpu/drm/i915/display/hsw_ips.c | 4 +- drivers/gpu/drm/i915/display/i9xx_wm.c | 2 +- .../gpu/drm/i915/display/intel_backlight.c | 9 +- drivers/gpu/drm/i915/display/intel_bios.c | 6 +- drivers/gpu/drm/i915/display/intel_crt.c | 4 +- drivers/gpu/drm/i915/display/intel_display.h | 2 +- .../gpu/drm/i915/display/intel_display_core.h | 2 + .../drm/i915/display/intel_display_debugfs.c | 2 + .../display/intel_display_debugfs_params.c | 176 ++++++++++++++ .../display/intel_display_debugfs_params.h | 14 ++ .../drm/i915/display/intel_display_device.c | 13 +- .../drm/i915/display/intel_display_device.h | 1 + .../drm/i915/display/intel_display_params.c | 216 ++++++++++++++++++ .../drm/i915/display/intel_display_params.h | 61 +++++ .../drm/i915/display/intel_display_power.c | 14 +- .../drm/i915/display/intel_display_reset.c | 2 +- drivers/gpu/drm/i915/display/intel_dp.c | 6 +- .../drm/i915/display/intel_dp_aux_backlight.c | 4 +- drivers/gpu/drm/i915/display/intel_dpt.c | 6 +- drivers/gpu/drm/i915/display/intel_fb.c | 2 +- drivers/gpu/drm/i915/display/intel_fbc.c | 10 +- drivers/gpu/drm/i915/display/intel_lvds.c | 4 +- drivers/gpu/drm/i915/display/intel_opregion.c | 2 +- drivers/gpu/drm/i915/display/intel_panel.c | 4 +- drivers/gpu/drm/i915/display/intel_psr.c | 14 +- .../drm/i915/display/skl_universal_plane.c | 2 +- drivers/gpu/drm/i915/display/skl_watermark.c | 5 +- drivers/gpu/drm/i915/i915_debugfs.c | 3 + drivers/gpu/drm/i915/i915_driver.c | 2 + drivers/gpu/drm/i915/i915_gpu_error.c | 3 + drivers/gpu/drm/i915/i915_gpu_error.h | 2 + drivers/gpu/drm/i915/i915_params.c | 89 -------- drivers/gpu/drm/i915/i915_params.h | 22 -- 34 files changed, 548 insertions(+), 162 deletions(-) create mode 100644 drivers/gpu/drm/i915/display/intel_display_debugfs_params.c create mode 100644 drivers/gpu/drm/i915/display/intel_display_debugfs_params.h create mode 100644 drivers/gpu/drm/i915/display/intel_display_params.c create mode 100644 drivers/gpu/drm/i915/display/intel_display_params.h