Hi
Am 29.04.20 um 16:10 schrieb Jason Yan:
The struct member 'set_config' was assigned twice:
static const struct drm_crtc_funcs ast_crtc_funcs = { .reset = ast_crtc_reset, .set_config = drm_crtc_helper_set_config, ...... .set_config = drm_atomic_helper_set_config, ...... };
Since the second one is which we use now in fact, we can remove the first one.
This fixes the following coccicheck warning:
drivers/gpu/drm/ast/ast_mode.c:932:50-51: set_config: first occurrence line 934, second occurrence line 937
Signed-off-by: Jason Yan yanaijie@huawei.com
Reviewed-by: Thomas Zimmermann tzimmermann@suse.de
Thanks! I'll apply the patch to drm-misc-next soonish.
Best regards Thomas
drivers/gpu/drm/ast/ast_mode.c | 1 - 1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c index d2ab81f9c498..7062bcd78740 100644 --- a/drivers/gpu/drm/ast/ast_mode.c +++ b/drivers/gpu/drm/ast/ast_mode.c @@ -931,7 +931,6 @@ static void ast_crtc_atomic_destroy_state(struct drm_crtc *crtc,
static const struct drm_crtc_funcs ast_crtc_funcs = { .reset = ast_crtc_reset,
- .set_config = drm_crtc_helper_set_config, .gamma_set = drm_atomic_helper_legacy_gamma_set, .destroy = ast_crtc_destroy, .set_config = drm_atomic_helper_set_config,