Received-SPF: pass (google.com: domain of linux-kernel+bounces-164609-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <c1d339ef-7ba9-4ef9-ae46-1b6b61743a92@amd.com>
Date: Tue, 30 Apr 2024 14:43:58 -0400
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2] drm/amd/display: re-indent dc_power_down_on_boot()
To: Dan Carpenter <dan.carpenter@linaro.org>
Cc: Leo Li <sunpeng.li@amd.com>, Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>,
 Alex Deucher <alexander.deucher@amd.com>,
 =?UTF-8?Q?Christian_K=C3=B6nig?= <christian.koenig@amd.com>,
 "Pan, Xinhui" <Xinhui.Pan@amd.com>, David Airlie <airlied@gmail.com>,
 Daniel Vetter <daniel@ffwll.ch>, Alvin Lee <alvin.lee2@amd.com>,
 Wenjing Liu <wenjing.liu@amd.com>, Jun Lei <jun.lei@amd.com>,
 Tom Chung <chiahsuan.chung@amd.com>,
 Aurabindo Pillai <aurabindo.pillai@amd.com>,
 Dillon Varone <dillon.varone@amd.com>, amd-gfx@lists.freedesktop.org,
 dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
 kernel-janitors@vger.kernel.org
References: <ad54bf29-cb70-49bd-a89f-c51ef7abb3ee@moroto.mountain>
Content-Language: en-US
From: Harry Wentland <harry.wentland@amd.com>
In-Reply-To: <ad54bf29-cb70-49bd-a89f-c51ef7abb3ee@moroto.mountain>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?U1FLT3FHUHZBWlllaENWMEhJQzg4Wk1xbjBleVZWcHdZZlY3c05DSXlRcmhE?=
 =?utf-8?B?ZHhIM2Y5dHpLbkhTNDJ0UnJKWHQvbEV1cDBFTlRnVDVaa2J5S1ZjKzJ5dnNn?=
 =?utf-8?B?QkNLdUZqRVFlSXJ2OXgrK1N6WHZhZlBMeTZZTXd3cDNrMjZSUkc5Q3IvR2Fl?=
 =?utf-8?B?NVlCNG9HelZ4T2hNdHlUMGJPR25UVlVSeDZlS21mRk0yTWl0L1pzMFlBQ0tw?=
 =?utf-8?B?Mm5pTFVPdkdUZEEvVXpPdW1abm45RlB2am9FQjVSSnB5ZWRmUE84TWVqNzQ5?=
 =?utf-8?B?UlhiMjBLQVU2TzdwamZ2cVVUTlVSTnU5TXJoNVo0TlMycTZaMStIdFJwSC9l?=
 =?utf-8?B?QlFCMENmREdSMFdXRW1UYmkrWHFWQlZYSDhlQklHSnZZL3RMaDg4KzVKTFpv?=
 =?utf-8?B?QjlSV1gwd3BuUGx2TFhFMGh0ZnVHL2I5bG5INlBrVyt3NE9LdUtBaWphY3N3?=
 =?utf-8?B?MlJUdFIvTVRFY0JrSEFEcjV2ZktvNE15aG5uYjdmb3U0UUY0aFRFQUdVN2w5?=
 =?utf-8?B?R2E4eXpCa2NacDBvYmFmVTlMdzMvaDNrRVlMRTZoM1RDYUdMcWhlQitnRDg3?=
 =?utf-8?B?a3dxSzI5YVZrenZ5R2s5blVWMU5FTTMwRGhINHNIbjM3TkJNemxMb3VJRmZo?=
 =?utf-8?B?Y1ZWbGpzN3hxTzVLOFRIRTVWRThoZ0wvRHBFekVhQTQ4TXAySjZIN0c2MnZO?=
 =?utf-8?B?RjF2YzVKNE12Tit3SmtYMUZvbytjZkg5eVJkK2hGWk5KbEVuNms2RldKYmsr?=
 =?utf-8?B?VlA4cmF3V3NEZmlmQlZkelk4ckZnM1VSLzlHYm96elVLaUVlK3l2MFppRGlM?=
 =?utf-8?B?MDFhNTZUN1dnc0wyR2tzcGxuWjh4Z0tLY0RlcFNMN3RaZHZhNVk4Nzh6Mk9i?=
 =?utf-8?B?WUZTZkxla1NObVNydVptZitQRnZEdGpBZFhwR1l1a1V3M1krQTQ5bHVWY1Zp?=
 =?utf-8?B?ZElOQjN2Y2NpamVyMnlqdGtlRVM0Zll0ZGszeUxzNmswUUs5emVDUyt0ZVYx?=
 =?utf-8?B?U3ZiT3NqcUNkQUs5NTZlWUpaU0pPY3NGdXFZWFdmM3ZaSjk5SlVDVXljYjNR?=
 =?utf-8?B?QXkxMnBkZFZyV3Z1TEpRb2Q4bUlYclZYaThhNFNPN0N2UVFUSkZMZ0lwRHU1?=
 =?utf-8?B?T25hTFNseFBQWTJPVWVnd0Y1Q0ZnZWJkcVA4WHVDYkZUQU43UVp3dVJScUhH?=
 =?utf-8?B?dGtySENXSHdkTUtGOW41UEtheTlsZmpSbWcrdHZMZm1TQndoM0ZEWGZRTVho?=
 =?utf-8?B?WHRaNERvWU9sYS9ENllJSTNDUVErUElvNHZCaUQ3OEVOYlFSMktrbFpvZ3dh?=
 =?utf-8?B?TlNqUUZpT0g5MlVtY05QZS9wd25kci9uOHRmRXBQb1EySlN4Y3FvMmwvcmtr?=
 =?utf-8?B?RnNDM3YzUzF4ODRmd2xqTXVSQmp5SlloT2hyZTJvNkI3QkhkNXl6UDl0MktP?=
 =?utf-8?B?VXczOWFsMy9zaUxRVk9vL1Z2OEsxbkxvUS90Q2FEalhPcE5iemViVGx2b1gy?=
 =?utf-8?B?MUdWdEd2bkRmM3dkRGFqVHRGaDZnK21HY2JsRFhaT0ZmQUEveFNPdUhhOEVi?=
 =?utf-8?B?c2xHMXhBcXN6MXN0STlRK293WDI2d0Y3RVE5RGZhMENHZXJ5VTh3N1BWOHNE?=
 =?utf-8?B?OTgrYWZUK1VVdEZ4L0lZd1NiR0NEQlhNSVlrRUNPVXJZVzh1YWlZZEw0SDk4?=
 =?utf-8?B?eGJmYlhNYStzc3A3bzRMWHZSZnFSQXZsNzVGaHVFczRHdGxqOTluM2ZLZFRM?=
 =?utf-8?B?d0RuUitRaStVb3BLL0luWExickorTytFK0xjSXFLVTgwUXBadnU0bFB1WlBv?=
 =?utf-8?B?ZDMvWW5CMExleTBRalJLNUJua2trSkhXWVpRQ1MxU2pBckIxc2NYWlNkYTVl?=
 =?utf-8?B?ZDAyUzJZaUJsYkY4dStvY3JOamY5dC9PVFpMTjJNRllwUjRlN0lvQ1h3Z1JZ?=
 =?utf-8?B?TllucUtlR2E5d0pCTk81UDBVdnV2RXAyRzBIRXlEVzhVRGhBVW95a0MvSUx1?=
 =?utf-8?B?dDluSHg2Z21yeUkreUFYd0IwdXRPZCtMNDhIYVZUZm9sQy85R3VFYnlhbi9C?=
 =?utf-8?B?V2lxSkRiaUNOQTJOUnpiZ0FMazFXUUFrNjdEei9JZkl2YkNXU0ltSWIxaUd4?=
 =?utf-8?Q?IpeIE80kHvymHL9yyWggg/5Nc?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 773fe295-66e0-477a-93c0-08dc6945816b
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5427.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Apr 2024 18:44:02.6560
 (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: OrtWn11ltGpHIdiOy4c/sMu3WLYPKS5mFFciKslc0VHGO/BHRC5P9riN2CtVawQ2yX0zCQ0Q91rVtv1BWCfRxg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL3PR12MB6619


On 2024-04-25 07:21, Dan Carpenter wrote:
> These lines are indented too far.  Clean the whitespace.
> 

Thanks.

Reviewed-by: Harry Wentland <harry.wentland@amd.com>

In the process of merging it into amd-staging-drm-next.

Harry

> Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
> ---
> v2: Delete another blank line (checkpatch.pl --strict).
> 
>  drivers/gpu/drm/amd/display/dc/core/dc.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> index 3e16041bf4f9..5a0835f884a8 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> @@ -5192,11 +5192,9 @@ void dc_interrupt_ack(struct dc *dc, enum dc_irq_source src)
>  void dc_power_down_on_boot(struct dc *dc)
>  {
>  	if (dc->ctx->dce_environment != DCE_ENV_VIRTUAL_HW &&
> -			dc->hwss.power_down_on_boot) {
> -
> -			if (dc->caps.ips_support)
> -				dc_exit_ips_for_hw_access(dc);
> -
> +	    dc->hwss.power_down_on_boot) {
> +		if (dc->caps.ips_support)
> +			dc_exit_ips_for_hw_access(dc);
>  		dc->hwss.power_down_on_boot(dc);
>  	}
>  }