diff mbox

[v2] drm/amd/dc: resource: fix semicolon.cocci warnings (fwd)

Message ID alpine.DEB.2.20.1702140720010.2033@hadrien (mailing list archive)
State New, archived
Headers show

Commit Message

Julia Lawall Feb. 14, 2017, 6:21 a.m. UTC 
Remove unneeded semicolon.

Generated by: scripts/coccinelle/misc/semicolon.cocci

CC: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

v2: make subject line unique

tree:   git://people.freedesktop.org/~agd5f/linux.git amd-staging-4.9
head:   79d2de1bcb650296adff1cb08bfbf1501a6e6e14
commit: bad4c165a6986a131cdd1455507ba3857baaa561 [201/657] drm/amd/dc: Add
dc display driver

 dc_resource.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Christian König Feb. 14, 2017, 9:47 a.m. UTC | #1 
Am 14.02.2017 um 07:21 schrieb Julia Lawall:
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> CC: Harry Wentland <harry.wentland@amd.com>
> Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: Christian König <christian.koenig@amd.com> for this one as 
well as the other similar patches.

Thanks for letting us know about those typos.

Regards,
Christian.

> ---
>
> v2: make subject line unique
>
> tree:   git://people.freedesktop.org/~agd5f/linux.git amd-staging-4.9
> head:   79d2de1bcb650296adff1cb08bfbf1501a6e6e14
> commit: bad4c165a6986a131cdd1455507ba3857baaa561 [201/657] drm/amd/dc: Add
> dc display driver
>
>   dc_resource.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c
> +++ b/drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c
> @@ -135,7 +135,7 @@ static void update_num_audio(
>   		break;
>   	default:
>   		DC_ERR("DC: unexpected audio fuse!\n");
> -	};
> +	}
>   }
>
>   bool resource_construct(
Harry Wentland Feb. 14, 2017, 5:07 p.m. UTC | #2 
Thanks for these fixes. I'll merge them.

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

Harry

On 2017-02-14 04:47 AM, Christian König wrote:
> Am 14.02.2017 um 07:21 schrieb Julia Lawall:
>> Remove unneeded semicolon.
>>
>> Generated by: scripts/coccinelle/misc/semicolon.cocci
>>
>> CC: Harry Wentland <harry.wentland@amd.com>
>> Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
>> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
>
> Acked-by: Christian König <christian.koenig@amd.com> for this one as 
> well as the other similar patches.
>
> Thanks for letting us know about those typos.
>
> Regards,
> Christian.
>
>> ---
>>
>> v2: make subject line unique
>>
>> tree:   git://people.freedesktop.org/~agd5f/linux.git amd-staging-4.9
>> head:   79d2de1bcb650296adff1cb08bfbf1501a6e6e14
>> commit: bad4c165a6986a131cdd1455507ba3857baaa561 [201/657] 
>> drm/amd/dc: Add
>> dc display driver
>>
>>   dc_resource.c |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> --- a/drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c
>> @@ -135,7 +135,7 @@ static void update_num_audio(
>>           break;
>>       default:
>>           DC_ERR("DC: unexpected audio fuse!\n");
>> -    };
>> +    }
>>   }
>>
>>   bool resource_construct(
>
>
diff mbox

Patch

--- a/drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c
+++ b/drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c
@@ -135,7 +135,7 @@  static void update_num_audio(
 		break;
 	default:
 		DC_ERR("DC: unexpected audio fuse!\n");
-	};
+	}
 }

 bool resource_construct(