* [radeon-alex:amd-staging-drm-next 164/1021] drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon
@ 2017-10-12 23:35 kbuild test robot
2017-10-12 23:35 ` [PATCH] drm/amd/dc: fix semicolon.cocci warnings kbuild test robot
0 siblings, 1 reply; 7+ messages in thread
From: kbuild test robot @ 2017-10-12 23:35 UTC (permalink / raw)
To: Harry Wentland; +Cc: Alex Deucher, kbuild-all, dri-devel
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head: 43dd6fde5df450938568885249b836eb376e2ad6
commit: 80be23c578680168a0c710be9c0bddcf65268fcb [164/1021] drm/amd/dc: Add dc display driver (v2)
coccinelle warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] drm/amd/dc: fix semicolon.cocci warnings
2017-10-12 23:35 [radeon-alex:amd-staging-drm-next 164/1021] drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon kbuild test robot
@ 2017-10-12 23:35 ` kbuild test robot
2017-10-13 19:55 ` Harry Wentland
0 siblings, 1 reply; 7+ messages in thread
From: kbuild test robot @ 2017-10-12 23:35 UTC (permalink / raw)
To: Harry Wentland; +Cc: Alex Deucher, kbuild-all, dri-devel
drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon
Remove unneeded semicolon.
Generated by: scripts/coccinelle/misc/semicolon.cocci
Fixes: 80be23c57868 ("drm/amd/dc: Add dc display driver (v2)")
CC: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
gpio_service.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c
+++ b/drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c
@@ -131,7 +131,7 @@ failure_2:
if (slot)
dm_free(slot);
- };
+ }
failure_1:
dm_free(service);
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] drm/amd/dc: fix semicolon.cocci warnings
2017-10-12 23:35 ` [PATCH] drm/amd/dc: fix semicolon.cocci warnings kbuild test robot
@ 2017-10-13 19:55 ` Harry Wentland
0 siblings, 0 replies; 7+ messages in thread
From: Harry Wentland @ 2017-10-13 19:55 UTC (permalink / raw)
To: kbuild test robot; +Cc: Alex Deucher, kbuild-all, dri-devel
On 2017-10-12 07:35 PM, kbuild test robot wrote:
> drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> Fixes: 80be23c57868 ("drm/amd/dc: Add dc display driver (v2)")
> CC: Harry Wentland <harry.wentland@amd.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Harry
> ---
>
> gpio_service.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c
> +++ b/drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c
> @@ -131,7 +131,7 @@ failure_2:
>
> if (slot)
> dm_free(slot);
> - };
> + }
>
> failure_1:
> dm_free(service);
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] drm/amd/dc: fix semicolon.cocci warnings
@ 2017-02-14 6:14 Julia Lawall
2017-02-14 17:07 ` Harry Wentland
0 siblings, 1 reply; 7+ messages in thread
From: Julia Lawall @ 2017-02-14 6:14 UTC (permalink / raw)
To: Harry Wentland
Cc: linux-kernel, dri-devel, kbuild-all, Alex Deucher,
Christian König
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>
---
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(
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] drm/amd/dc: fix semicolon.cocci warnings
2017-02-14 6:14 Julia Lawall
@ 2017-02-14 17:07 ` Harry Wentland
0 siblings, 0 replies; 7+ messages in thread
From: Harry Wentland @ 2017-02-14 17:07 UTC (permalink / raw)
To: Julia Lawall
Cc: linux-kernel, dri-devel, kbuild-all, Alex Deucher,
Christian König
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Harry
On 2017-02-14 01:14 AM, Julia Lawall wrote:
> 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>
> ---
>
> 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(
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] drm/amd/dc: fix semicolon.cocci warnings
@ 2017-02-14 6:13 Julia Lawall
2017-02-14 17:08 ` Harry Wentland
0 siblings, 1 reply; 7+ messages in thread
From: Julia Lawall @ 2017-02-14 6:13 UTC (permalink / raw)
To: Harry Wentland
Cc: linux-kernel, dri-devel, kbuild-all, Alex Deucher,
Christian König
Remove unneeded semicolons.
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>
---
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
amdgpu_dm_types.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
--- a/drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_types.c
+++ b/drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_types.c
@@ -890,11 +890,11 @@ static void copy_crtc_timing_for_drm_dis
dst_mode->crtc_hsync_end = src_mode->crtc_hsync_end;
dst_mode->crtc_htotal = src_mode->crtc_htotal;
dst_mode->crtc_hskew = src_mode->crtc_hskew;
- dst_mode->crtc_vblank_start = src_mode->crtc_vblank_start;;
- dst_mode->crtc_vblank_end = src_mode->crtc_vblank_end;;
- dst_mode->crtc_vsync_start = src_mode->crtc_vsync_start;;
- dst_mode->crtc_vsync_end = src_mode->crtc_vsync_end;;
- dst_mode->crtc_vtotal = src_mode->crtc_vtotal;;
+ dst_mode->crtc_vblank_start = src_mode->crtc_vblank_start;
+ dst_mode->crtc_vblank_end = src_mode->crtc_vblank_end;
+ dst_mode->crtc_vsync_start = src_mode->crtc_vsync_start;
+ dst_mode->crtc_vsync_end = src_mode->crtc_vsync_end;
+ dst_mode->crtc_vtotal = src_mode->crtc_vtotal;
}
static void decide_crtc_timing_for_drm_display_mode(
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/amd/dc: fix semicolon.cocci warnings
2017-02-14 6:13 Julia Lawall
@ 2017-02-14 17:08 ` Harry Wentland
0 siblings, 0 replies; 7+ messages in thread
From: Harry Wentland @ 2017-02-14 17:08 UTC (permalink / raw)
To: Julia Lawall
Cc: Alex Deucher, Christian König, David Airlie, dri-devel,
kbuild-all, linux-kernel
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Harry
On 2017-02-14 01:13 AM, Julia Lawall wrote:
> Remove unneeded semicolons.
>
> 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>
> ---
>
> 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
>
> amdgpu_dm_types.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> --- a/drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_types.c
> +++ b/drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_types.c
> @@ -890,11 +890,11 @@ static void copy_crtc_timing_for_drm_dis
> dst_mode->crtc_hsync_end = src_mode->crtc_hsync_end;
> dst_mode->crtc_htotal = src_mode->crtc_htotal;
> dst_mode->crtc_hskew = src_mode->crtc_hskew;
> - dst_mode->crtc_vblank_start = src_mode->crtc_vblank_start;;
> - dst_mode->crtc_vblank_end = src_mode->crtc_vblank_end;;
> - dst_mode->crtc_vsync_start = src_mode->crtc_vsync_start;;
> - dst_mode->crtc_vsync_end = src_mode->crtc_vsync_end;;
> - dst_mode->crtc_vtotal = src_mode->crtc_vtotal;;
> + dst_mode->crtc_vblank_start = src_mode->crtc_vblank_start;
> + dst_mode->crtc_vblank_end = src_mode->crtc_vblank_end;
> + dst_mode->crtc_vsync_start = src_mode->crtc_vsync_start;
> + dst_mode->crtc_vsync_end = src_mode->crtc_vsync_end;
> + dst_mode->crtc_vtotal = src_mode->crtc_vtotal;
> }
>
> static void decide_crtc_timing_for_drm_display_mode(
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-10-13 19:55 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-12 23:35 [radeon-alex:amd-staging-drm-next 164/1021] drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon kbuild test robot
2017-10-12 23:35 ` [PATCH] drm/amd/dc: fix semicolon.cocci warnings kbuild test robot
2017-10-13 19:55 ` Harry Wentland
-- strict thread matches above, loose matches on Subject: below --
2017-02-14 6:14 Julia Lawall
2017-02-14 17:07 ` Harry Wentland
2017-02-14 6:13 Julia Lawall
2017-02-14 17:08 ` Harry Wentland
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox