Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
Message-ID: <6bd4f6d9-07c9-9ac1-2f3f-bb083b9c8085@amd.com>
Date:   Tue, 7 Nov 2023 10:15:24 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.13.1
Subject: Re: [PATCH v2] drm: amd: Resolve Sphinx unexpected indentation
 warning
Content-Language: en-US
To:     Hunter Chasens <hunter.chasens18@ncf.edu>,
        linux-kernel@vger.kernel.org
Cc:     linux-doc@vger.kernel.org, Xinhui.Pan@amd.com,
        amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
        daniel@ffwll.ch, alexander.deucher@amd.com, airlied@gmail.com,
        christian.koenig@amd.com
References: <20231106201739.29507-1-hunter.chasens18@ncf.edu>
From:   "Lazar, Lijo" <lijo.lazar@amd.com>
In-Reply-To: <20231106201739.29507-1-hunter.chasens18@ncf.edu>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?L2oyS0I3alZ6aDREeDJNc2YzdkJPTU5qcUszWUtKRHd2ZlpMSTNRc3dXWml3?=
 =?utf-8?B?Zm1lVzV0VHYxRU9VczdJcTFCM1ZiUVRQUjZpcnlFNkx6NVRrRndRaXVqdVcy?=
 =?utf-8?B?RGM5YzFmYkhQREhpaUQ3TWNialZtc0FGd3l3ZVI2dHVrOWtWYjJUUVRON1Iy?=
 =?utf-8?B?U0VQb3ZQL3dpMjhHTSsyZkVGazI2TE9CaU93ZXM4TVp4d0dsZFZVUHR3VTRW?=
 =?utf-8?B?a3lxVHBxeWpxc3JLN1dNRFRrL0xZOXNmaXI3U2JQMTBzamhZaklwR3N5OFBS?=
 =?utf-8?B?eGVOcUNqam5ZTTYxSGJweU45N2xSR0htbnFNK2szdXM2N0xzaFJZcHZqZFZq?=
 =?utf-8?B?OEhrNGtDVm04OVRrbUhsUk45WEMxd09vUXBUSUZCcmNKS04wbVBZR0NLK0tQ?=
 =?utf-8?B?aTdaLysyWWFudXRlUEVsQVcrUUFGci9wN3MwUFlZRkIzMlI0RHpnR3kzeXpB?=
 =?utf-8?B?WWJsSGx6RVRhWGtjNFBIMEVIbHhsU3lrb2d1YWlsRGNwcTdPRG1jRlYwWEVv?=
 =?utf-8?B?aThlSXVQam85K21MYllnVUViNkpIZmVlMUh1Q2J2K2YxclZ0bmQ3OHRycWkr?=
 =?utf-8?B?SFVnejR1SGY0YXZBblJUVzNrejJERXRZMXZqU09Ua04yTW95dEpVNElHSG1j?=
 =?utf-8?B?N21EVSt6SGpJYUZBcW4vMlNIaS9zcnZ6N2tDbHRCaTU3ejBhT29ZNVQveWVh?=
 =?utf-8?B?RFAvd2JxUWVCZWRTRmZEcWJydmV1MkFMVW9VaGhxcmU0RjZ4UXJWWFN5UEVv?=
 =?utf-8?B?MEhhTjAxNUpvSHVtbjZXZGlnQzFrUi80dUxlMThYM3R6SzI1eXd3WUFFZjdP?=
 =?utf-8?B?bHRWMG5WQVhqWTRUdFpxeW5ZWldBbjdjUzVBQmkxZnlBc1RJL2FLeWhRUFBh?=
 =?utf-8?B?bkdmWjhFL1VmNE1KK2RsajdFWk94cnBGT0RLN1ZBRHhkamNHVjhmVzZYSFFi?=
 =?utf-8?B?MkYyUDdnNFUrWTBIelFkaDFJSnhicHBsZnZvS2NhM3ZPSEdPeklTYk92YTc0?=
 =?utf-8?B?dnNUeko1aXRvYkVsUVprc2NJR0szYWhPdVA4aUEvdGxQTytXQ3pWejdOc2hk?=
 =?utf-8?B?WkY4V2RnU0xuME1ubXc0cWNIY1lSNTFIVWsvU01QY0xoaGlRdkd5TjVCT0hi?=
 =?utf-8?B?SHpHUTdITTZQaTAwOGtCUGhuODFqWUcyT1R0elEyZVI4emJUL2ZqS1ZPL0lR?=
 =?utf-8?B?QUplODRmVFBiYloyTWwrR0ZYTjVuNFM5Q2JRWU1PRE9pR3lkRXdoYWk1dkor?=
 =?utf-8?B?N0ZFMXFJdm9VNEhaOThmNlpRb29DVU9kM1o5RWRUS3VDNWkwSG9KS3FGOHNG?=
 =?utf-8?B?bmFHbmNxTGpJV1REZXRTdFpZdHJqdm1yeFBONG9wRjVqUk83bzY5R1BLSUVU?=
 =?utf-8?B?czE4L25xZ0tEKzNpRHRIMlErVmZkVjRiWmFUL1l6bW1nOTRwTGtkeS9PcGlr?=
 =?utf-8?B?YWpGcWlOVkdkVjNJNmJrckpSUlEvWnpHbnh4S1JQVHVEREtYSFk5Ym13OE56?=
 =?utf-8?B?aVAzSENHY0NyZ3kvbENuT1ZYd25uazIwT0hDOW9tcC80RHA0cHprV09CMVN4?=
 =?utf-8?B?TVFud3pkN0hZc2RERDNMV09BOWJRNG8rb2RDalBuWVZjb2o5S0x0SjB3TWh1?=
 =?utf-8?B?VkRhN0hwY1ZsMVByWmc5b05qRHQzaVZTVTkxSGRXemhtcWpicDdRcHQycDFW?=
 =?utf-8?B?NnBKYitGd3lTaVMzODhUYnoyL0Z6SEErR1RhYUlSRnEzdmpRaDdHQWZ3cjNW?=
 =?utf-8?B?a1pTbzhJVjJBQXBFUm5QL3BmL0oyRTRDbG5VWVZkU0RwK1ZrTERHU0NHVlhM?=
 =?utf-8?B?UnVwMUFITGF1c2tOdi9OTjhRQXhGWFJrV3oySWhOa1RLbUQvMzkyVTFuSk1z?=
 =?utf-8?B?YjBhd2FuOVNrUXBFd2dvR0xTUHprWDRwYzdMWUlwWjZsamd1dU1xNGZXNW5Z?=
 =?utf-8?B?bWtvd2U3UGtwNmhleTVnVmlBdzJIa1Zkc3B6akJYNS9WUjBoanpLZHUwVmw3?=
 =?utf-8?B?WEtJd2kzTmJHOE5jYkZuUDRxZXU4QXN2cXprRXlWZVd3QXg4NktHTktTczgy?=
 =?utf-8?B?MXgyaXNDa3d4cm9jUkdMa3FMd0JXTmtIT0owUkVLMUZqUG5JcG1KZlNOcGdT?=
 =?utf-8?Q?1vNrlLYl21vPcJ0KZB1PNbD6y?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 920f9335-4f17-46ac-b2a0-08dbdf4c60db
X-MS-Exchange-CrossTenant-AuthSource: BYAPR12MB4614.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Nov 2023 04:45:34.5723
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: vbDNt8cnBRHIcp1R2DLXcohAcS1BiSGfJx2SdcywaDVCuVF/ACieSaEYREicAdSR
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6563
X-Spam-Status: No, score=-3.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Mon, 06 Nov 2023 20:46:09 -0800 (PST)


On 11/7/2023 1:47 AM, Hunter Chasens wrote:
> Resolves Sphinx unexpected indentation warning when compiling
> documentation (e.g. `make htmldocs`). Replaces tabs with spaces and adds
> a literal block to keep vertical formatting of the
> example power state list.
> 
> Signed-off-by: Hunter Chasens <hunter.chasens18@ncf.edu>
> ---
>   drivers/gpu/drm/amd/pm/amdgpu_pm.c | 13 +++++++------
>   1 file changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> index 517b9fb4624c..81b8ceb26890 100644
> --- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> @@ -989,12 +989,13 @@ static ssize_t amdgpu_get_pp_features(struct device *dev,
>    * Reading back the files will show you the available power levels within
>    * the power state and the clock information for those levels. If deep sleep is
>    * applied to a clock, the level will be denoted by a special level 'S:'
> - * E.g.,
> - *	S: 19Mhz *
> - *	0: 615Mhz
> - *	1: 800Mhz
> - *	2: 888Mhz
> - *	3: 1000Mhz
> + * E.g.::

This will be like E.g.: Could you keep it like E.g.,<space>:: so that :: 
is taken out?

Thanks,
Lijo

> + *
> + *  S: 19Mhz *
> + *  0: 615Mhz
> + *  1: 800Mhz
> + *  2: 888Mhz
> + *  3: 1000Mhz
>    *
>    *
>    * To manually adjust these states, first select manual using