[PATCH v2] drm/amd/amdgpu: Fix trivial style errors

[Mario Limonciello]

Also the changelog should be below the cutlist (---).

On 4/5/23 09:12, Russell, Kent wrote:
> [AMD Official Use Only - General]
>
> Probably worth changing the commit message and headline now that you're removing the parameter instead of just fixing the code style.
>
>   Kent
>
>> -----Original Message-----
>> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of
>> Srinivasan Shanmugam
>> Sent: Wednesday, April 5, 2023 9:24 AM
>> To: Koenig, Christian <Christian.Koenig at amd.com>; Deucher, Alexander
>> <Alexander.Deucher at amd.com>; Limonciello, Mario
>> <Mario.Limonciello at amd.com>
>> Cc: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM at amd.com>; amd-
>> gfx at lists.freedesktop.org
>> Subject: [PATCH v2] drm/amd/amdgpu: Fix trivial style errors
>>
>> Fix coding style errors reported by checkpatch, specifically:
>>
>> ERROR: space prohibited before that ',' (ctx:WxV)
>> +module_param_named(job_hang_limit, amdgpu_job_hang_limit, int ,0444);
>>                                                                 ^
>>
>> ERROR: space required after that ',' (ctx:WxV)
>> +module_param_named(job_hang_limit, amdgpu_job_hang_limit, int ,0444);
>>
>> This patch gets rid of all above type of  "ERROR" messages in
>> drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c.
>>
>> v2: The driver doesn't resubmit jobs on hangs any more, hence drop
>> the hang limit module parameter. (Christian)
>>
>> Suggested-by: Christian König <christian.koenig at amd.com>
>> Cc: Alex Deucher <alexander.deucher at amd.com>
>> Cc: Mario Limonciello <mario.limonciello at amd.com>
>> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 7 -------
>>   1 file changed, 7 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> index e652ffb2c68e..25606af348b8 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> @@ -521,13 +521,6 @@ MODULE_PARM_DESC(virtual_display,
>>   		 "Enable virtual display feature (the virtual_display will be set
>> like xxxx:xx:xx.x,x;xxxx:xx:xx.x,x)");
>>   module_param_named(virtual_display, amdgpu_virtual_display, charp, 0444);
>>
>> -/**
>> - * DOC: job_hang_limit (int)
>> - * Set how much time allow a job hang and not drop it. The default is 0.
>> - */
>> -MODULE_PARM_DESC(job_hang_limit, "how much time allow a job hang and
>> not drop it (default 0)");
>> -module_param_named(job_hang_limit, amdgpu_job_hang_limit, int ,0444);
>> -
>>   /**
>>    * DOC: lbpw (int)
>>    * Override Load Balancing Per Watt (LBPW) support (1 = enable, 0 = disable).
>> The default is -1 (auto, enabled).
>> --
>> 2.25.1