Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH][next] drm/amdgpu: sdma: clean up identation
To:     Colin King <colin.king@canonical.com>,
        Alex Deucher <alexander.deucher@amd.com>, Xinhui.Pan@amd.com,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>, amd-gfx@lists.freedesktop.org,
        dri-devel@lists.freedesktop.org
Cc:     kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20210902215127.55330-1-colin.king@canonical.com>
From:   =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
Message-ID: <bb5105f8-2ec6-e494-8764-3dcc5fc0681b@amd.com>
Date:   Fri, 3 Sep 2021 08:31:33 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.13.0
In-Reply-To: <20210902215127.55330-1-colin.king@canonical.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?c3IyUFYwT2NDQk5ISlJ1V2N0aUZ5OTYvZVNnRjAwWU5kNUFPdGxHQ1BBR3pp?=
 =?utf-8?B?YUt3ZkJOVGJJV0ZuVG5BVzRWVjNEMGgyNklvVlhDU21zRXpZeDZDVjlQQmtz?=
 =?utf-8?B?OHNEVHBjYmJIZ05uRFdHSEF0V1ppNzVtcUFWdlhKMWl4eVRkU0YwV2Ixb0w1?=
 =?utf-8?B?Ylg2bktqTUNWc2lDbGdsMHpUUXk3NThvQklteEVkRUpFUGZhdWowNjFpYm00?=
 =?utf-8?B?eStmUXZHYUdIV2Rvc1FLU1VPSU5ma3Q4RU1IbkJESlF6M1BhYmNwSFozTDg5?=
 =?utf-8?B?SGpHMDU3VEFzSGoxVXhwT2pyTlJuYWo3cGdMZTcvRVdZcldhV1BsYS9RSUR1?=
 =?utf-8?B?cXFNaFJWdjJ5ZW1CNFJmU0poR09hT2RpaHZDSjUxdjk0YlBlSFdKeFE3RDRH?=
 =?utf-8?B?THBMaEJnWUlWbWpNRXBhclRXSlVub2w1V0FVUDhwVTBxVnc1ZVVxeWhuekRU?=
 =?utf-8?B?QUJjeDVrVkwzbnVYeGl2T1l4MFlJczM0b0N4eGs1dFBSZWx3ZzNIdGZ5U00r?=
 =?utf-8?B?ZTNHNk1sL1EwVXdmVlpodTZobG5yTy9YOWtnWU9KcjgraCtsckhhZ29JdExL?=
 =?utf-8?B?NUtqVWp0TGNtbnhVNlFwZ1FqQzg0aHdDbmVVRVpncXd4S0N2WnpUa2JTT2xl?=
 =?utf-8?B?TExHM3diWSs0ZTZoTWQwUkxpeE9qTnNQSkNFNzhJRHV1UHE0NVB0WUN0VFNS?=
 =?utf-8?B?MjlrL3d0dXlqK3V5SFhNUkFQbjFJM3Jmeis3VERycGx3Slh2VWxJdGVTY0pu?=
 =?utf-8?B?NXJnVFhxeFZNanh4ZTl5ZTF6cEFYc2lMRVZtWjV5d2Y1MHRldUV4QTBIb3pD?=
 =?utf-8?B?enRhRUtFQmhrZ1VEUXN3S3J6azk2YngzOStsd3BXak5mMmM1SDVlbVFIdTVr?=
 =?utf-8?B?MndDeCtpQU1KZkVPdkxqN2pmaG9oK3VGZjU0bCtQampyM1ZrSm1LRlB1anJI?=
 =?utf-8?B?NDdtSllxUWgxeldOVnNPM0U0WFg2aGNNU0RsbFJVZXJPVEpOWDhIMnV4Q2hD?=
 =?utf-8?B?UkQrZGYrSnFod3VzdzBXM0w1TVlqY3Q5SGt4bXpGdldEL0gveVRZUWtVdjcx?=
 =?utf-8?B?QWw4WHRQQk9MeEJZcEFERnI4OFRLWncyU1NpV2E5d0JScWJxVWRlb1ZSZW9Q?=
 =?utf-8?B?eDFLbUQ2MWNDajM4bUFEMEpoYitucTU1dHFQb2pKV3VmUC92blhWU0lSMjlj?=
 =?utf-8?B?ai9QaHBVakJNdDQwUmhqVzhqN0lwSndSUFBKT2twOFZUWmx1cXg1VTVvSXc0?=
 =?utf-8?B?SCt3WGpmQ3hWZlRQQU5kWDNtRmFvLy9iQnZLNko3ZEEwdllGREcwaG91NUd4?=
 =?utf-8?B?ZE42YmNVREpSYjQ5WVNrM1BQTHVERnRyMFR1TktFc1ZNN2J0a2FHNldSOFYw?=
 =?utf-8?B?RytYNGduejFJSHVsSEFabktaaDAwUFhvVzloV3pwdXBZNm5Ic3RQSWw0aFo1?=
 =?utf-8?B?aDRZcjUxUjhtdURvUmFCaUsrYWc0VjNYcmJMUHhlMW0rS1RtZHdnZHVubG9X?=
 =?utf-8?B?eEt5d3JObEQ5ak1YZUt2ZUE0MlVld3llcEJKbnBWalpoVGVNZy9pckdnUVE5?=
 =?utf-8?B?aVVRazNlamJ0b2l6Z1BaVzJqaEFhcEgvd2N3RE02N3Q5NkIzNW85VXoydFdR?=
 =?utf-8?B?MkFVZlNuR2xyaTd4MUZDL3dvTkRTeXhScnM2UFRPUmVzME8xUmlaZlBaTHdv?=
 =?utf-8?B?QlgzSzdlQXJpaCtaalFnc0V0YmxCSkRFeHdSY3h6Z0NpN2xKbjdkNnRnRHVV?=
 =?utf-8?Q?dbNqovR4j4rQpPmrDZKy8BurYQ8BSJ4Rdhif23S?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a2beca05-0d84-4ac7-1fdd-08d96ea47c77
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB3775.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2021 06:31:39.3338
 (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: 44PpZztBFGEHK4xtooMi4soPHG/M3SmOJQ0ZRC3+kcXPjwvosEheanktpTGjmSWV
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4550
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Am 02.09.21 um 23:51 schrieb Colin King:
> From: Colin Ian King <colin.king@canonical.com>
>
> There is a statement that is indented incorrectly. Clean it up.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

> ---
>   drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
> index 779f5c911e11..e4a96e7e386d 100644
> --- a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
> +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
> @@ -375,10 +375,10 @@ static void sdma_v5_2_ring_emit_ib(struct amdgpu_ring *ring,
>    */
>   static void sdma_v5_2_ring_emit_mem_sync(struct amdgpu_ring *ring)
>   {
> -    uint32_t gcr_cntl =
> -		    SDMA_GCR_GL2_INV | SDMA_GCR_GL2_WB | SDMA_GCR_GLM_INV |
> -			SDMA_GCR_GL1_INV | SDMA_GCR_GLV_INV | SDMA_GCR_GLK_INV |
> -			SDMA_GCR_GLI_INV(1);
> +	uint32_t gcr_cntl = SDMA_GCR_GL2_INV | SDMA_GCR_GL2_WB |
> +			    SDMA_GCR_GLM_INV | SDMA_GCR_GL1_INV |
> +			    SDMA_GCR_GLV_INV | SDMA_GCR_GLK_INV |
> +			    SDMA_GCR_GLI_INV(1);
>   
>   	/* flush entire cache L0/L1/L2, this can be optimized by performance requirement */
>   	amdgpu_ring_write(ring, SDMA_PKT_HEADER_OP(SDMA_OP_GCR_REQ));