From mboxrd@z Thu Jan 1 00:00:00 1970 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 54281C369B2 for ; Thu, 17 Apr 2025 08:55:40 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id F3A7210E193; Thu, 17 Apr 2025 08:55:39 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="iByu2/Xf"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) by gabe.freedesktop.org (Postfix) with ESMTPS id 706DE10E193; Thu, 17 Apr 2025 08:55:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1744880138; x=1776416138; h=from:to:cc:subject:in-reply-to:references:date: message-id:mime-version; bh=HY0IJdZ0xUrLxc/f5sk/9tuJaU9s69AQTdtLNO8zLPo=; b=iByu2/Xfp64H9NVKEI5EEwANB6wfIjto6Os0PTowU10Rax3dVSsDwVas 1+lJQg9InDuujtDFffuu3S4qsEfeKegps5tvUgylQjBbFEkuZWMhPKL7x 8USFoxaT9tGx/o8q1J25SXMlayfRQvaqHnwDMRwAAd1+XVCKuiyJO9Ma6 iVHU4yGa4ffdu/M4JCc6wtSGYmtYqFFZdUkUm5is8SMhlM6xNLToDw7vj M2NuF9WxjBgNNGSAmqEYMlFo0MYEeKmL+0c8Ky5y63eSRRMreI18cSvgj aq5uz7k8TCXE5ucYfxtV5rKYmrI5tm3rygtniDhyfLnUsuIKib5PS7HDN A==; X-CSE-ConnectionGUID: 6PJv+KEETxGnNBwVOIFcnQ== X-CSE-MsgGUID: uUKUraUXQGmZki4VtzZJzA== X-IronPort-AV: E=McAfee;i="6700,10204,11405"; a="45691687" X-IronPort-AV: E=Sophos;i="6.15,218,1739865600"; d="scan'208";a="45691687" Received: from orviesa007.jf.intel.com ([10.64.159.147]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Apr 2025 01:55:35 -0700 X-CSE-ConnectionGUID: MRFAzu9HQV21dmpc4DID/Q== X-CSE-MsgGUID: FfP6mRDuQm+XTQKAEhNA5Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.15,218,1739865600"; d="scan'208";a="131284750" Received: from dprybysh-mobl.ger.corp.intel.com (HELO localhost) ([10.245.246.139]) by orviesa007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Apr 2025 01:55:30 -0700 From: Jani Nikula To: Luke Hofstetter , Alex Deucher , Christian =?utf-8?Q?K=C3=B6nig?= Cc: David Airlie , Simona Vetter , Sunil Khatri , Mario Limonciello , Aurabindo Pillai , Boyuan Zhang , Leo Li , Dominik Kaszewski , amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Luke Hofstetter Subject: Re: [PATCH] drm/amd/include: fix kernel-doc formatting in amd_shared.h In-Reply-To: <20250416091143.67704-1-ldhofstetter@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20250416091143.67704-1-ldhofstetter@gmail.com> Date: Thu, 17 Apr 2025 11:55:26 +0300 Message-ID: <877c3jp3pd.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" On Wed, 16 Apr 2025, Luke Hofstetter wrote: > when doing make htmldocs, Sphinx complained about in-line documentation > in enum DC_DEBUG_MASK, so reformatted documentation to define each > member in kernel-doc comment above the enum instead. Why? What's the error message exactly? I think what you have now is the preferred style. If it really doesn't work for enums, kernel-doc should be fixed. BR, Jani. > > Signed-off-by: Luke Hofstetter > --- > drivers/gpu/drm/amd/include/amd_shared.h | 124 ++++++----------------- > 1 file changed, 32 insertions(+), 92 deletions(-) > > diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h > index 4c95b885d1d0..7074ec3b467f 100644 > --- a/drivers/gpu/drm/amd/include/amd_shared.h > +++ b/drivers/gpu/drm/amd/include/amd_shared.h > @@ -255,120 +255,60 @@ enum DC_FEATURE_MASK { > > /** > * enum DC_DEBUG_MASK - Bits that are useful for debugging the Display Core IP > + * @DC_DISABLE_PIPE_SPLIT: If set, disable pipe-splitting > + * @DC_DISABLE_STUTTER: If set, disable memory stutter mode > + * @DC_DISABLE_DSC: If set, disable display stream compression > + * @DC_DISABLE_CLOCK_GATING: If set, disable clock gating optimizations > + * @DC_DISABLE_PSR: If set, disable Panel self refresh v1 and PSR-SU > + * @DC_FORCE_SUBVP_MCLK_SWITCH: If set, force mclk switch in subvp, even > + * if mclk switch in vblank is possible > + * @DC_DISABLE_MPO: If set, disable multi-plane offloading > + * @DC_ENABLE_DPIA_TRACE: If set, enable trace logging for DPIA > + * @DC_ENABLE_DML2: If set, force usage of DML2, even if the DCN version > + * does not default to it. > + * @DC_DISABLE_PSR_SU: If set, disable PSR SU > + * @DC_DISABLE_REPLAY: If set, disable Panel Replay > + * @DC_DISABLE_IPS: If set, disable all Idle Power States, all the time. > + * If more than one IPS debug bit is set, the lowest bit takes > + * precedence. For example, if DC_FORCE_IPS_ENABLE and > + * DC_DISABLE_IPS_DYNAMIC are set, then DC_DISABLE_IPS_DYNAMIC takes > + * precedence. > + * @DC_DISABLE_IPS_DYNAMIC: If set, disable all IPS, all the time, > + * *except* when driver goes into suspend. > + * @DC_DISABLE_IPS2_DYNAMIC: If set, disable IPS2 (IPS1 allowed) if > + * there is an enabled display. Otherwise, enable all IPS. > + * @DC_FORCE_IPS_ENABLE: If set, force enable all IPS, all the time. > + * @DC_DISABLE_ACPI_EDID: If set, don't attempt to fetch EDID for > + * eDP display from ACPI _DDC method. > + * @DC_DISABLE_HDMI_CEC: If set, disable HDMI-CEC feature in amdgpu driver. > + * @DC_DISABLE_SUBVP: If set, disable DCN Sub-Viewport feature in amdgpu driver. > + * @DC_DISABLE_CUSTOM_BRIGHTNESS_CURVE: If set, disable support for custom brightness curves > + * @DC_HDCP_LC_FORCE_FW_ENABLE: If set, use HDCP Locality Check FW > + * path regardless of reported HW capabilities. > + * @DC_HDCP_LC_ENABLE_SW_FALLBACK: If set, upon HDCP Locality Check FW > + * path failure, retry using legacy SW path. > */ > enum DC_DEBUG_MASK { > - /** > - * @DC_DISABLE_PIPE_SPLIT: If set, disable pipe-splitting > - */ > DC_DISABLE_PIPE_SPLIT = 0x1, > - > - /** > - * @DC_DISABLE_STUTTER: If set, disable memory stutter mode > - */ > DC_DISABLE_STUTTER = 0x2, > - > - /** > - * @DC_DISABLE_DSC: If set, disable display stream compression > - */ > DC_DISABLE_DSC = 0x4, > - > - /** > - * @DC_DISABLE_CLOCK_GATING: If set, disable clock gating optimizations > - */ > DC_DISABLE_CLOCK_GATING = 0x8, > - > - /** > - * @DC_DISABLE_PSR: If set, disable Panel self refresh v1 and PSR-SU > - */ > DC_DISABLE_PSR = 0x10, > - > - /** > - * @DC_FORCE_SUBVP_MCLK_SWITCH: If set, force mclk switch in subvp, even > - * if mclk switch in vblank is possible > - */ > DC_FORCE_SUBVP_MCLK_SWITCH = 0x20, > - > - /** > - * @DC_DISABLE_MPO: If set, disable multi-plane offloading > - */ > DC_DISABLE_MPO = 0x40, > - > - /** > - * @DC_ENABLE_DPIA_TRACE: If set, enable trace logging for DPIA > - */ > DC_ENABLE_DPIA_TRACE = 0x80, > - > - /** > - * @DC_ENABLE_DML2: If set, force usage of DML2, even if the DCN version > - * does not default to it. > - */ > DC_ENABLE_DML2 = 0x100, > - > - /** > - * @DC_DISABLE_PSR_SU: If set, disable PSR SU > - */ > DC_DISABLE_PSR_SU = 0x200, > - > - /** > - * @DC_DISABLE_REPLAY: If set, disable Panel Replay > - */ > DC_DISABLE_REPLAY = 0x400, > - > - /** > - * @DC_DISABLE_IPS: If set, disable all Idle Power States, all the time. > - * If more than one IPS debug bit is set, the lowest bit takes > - * precedence. For example, if DC_FORCE_IPS_ENABLE and > - * DC_DISABLE_IPS_DYNAMIC are set, then DC_DISABLE_IPS_DYNAMIC takes > - * precedence. > - */ > DC_DISABLE_IPS = 0x800, > - > - /** > - * @DC_DISABLE_IPS_DYNAMIC: If set, disable all IPS, all the time, > - * *except* when driver goes into suspend. > - */ > DC_DISABLE_IPS_DYNAMIC = 0x1000, > - > - /** > - * @DC_DISABLE_IPS2_DYNAMIC: If set, disable IPS2 (IPS1 allowed) if > - * there is an enabled display. Otherwise, enable all IPS. > - */ > DC_DISABLE_IPS2_DYNAMIC = 0x2000, > - > - /** > - * @DC_FORCE_IPS_ENABLE: If set, force enable all IPS, all the time. > - */ > DC_FORCE_IPS_ENABLE = 0x4000, > - /** > - * @DC_DISABLE_ACPI_EDID: If set, don't attempt to fetch EDID for > - * eDP display from ACPI _DDC method. > - */ > DC_DISABLE_ACPI_EDID = 0x8000, > - > - /** > - * @DC_DISABLE_HDMI_CEC: If set, disable HDMI-CEC feature in amdgpu driver. > - */ > DC_DISABLE_HDMI_CEC = 0x10000, > - > - /** > - * @DC_DISABLE_SUBVP: If set, disable DCN Sub-Viewport feature in amdgpu driver. > - */ > DC_DISABLE_SUBVP = 0x20000, > - /** > - * @DC_DISABLE_CUSTOM_BRIGHTNESS_CURVE: If set, disable support for custom brightness curves > - */ > DC_DISABLE_CUSTOM_BRIGHTNESS_CURVE = 0x40000, > - > - /** > - * @DC_HDCP_LC_FORCE_FW_ENABLE: If set, use HDCP Locality Check FW > - * path regardless of reported HW capabilities. > - */ > DC_HDCP_LC_FORCE_FW_ENABLE = 0x80000, > - > - /** > - * @DC_HDCP_LC_ENABLE_SW_FALLBACK If set, upon HDCP Locality Check FW > - * path failure, retry using legacy SW path. > - */ > DC_HDCP_LC_ENABLE_SW_FALLBACK = 0x100000, > }; -- Jani Nikula, Intel