Thomas, please fix the sparse/build warnings. BR, Jani.
On Mon, 20 Jan 2020, Patchwork <patchw...@emeril.freedesktop.org> wrote: > == Series Details == > > Series: drm: Clean up VBLANK callbacks in struct drm_driver (rev8) > URL : https://patchwork.freedesktop.org/series/71873/ > State : warning > > == Summary == > > $ dim sparse origin/drm-tip > Sparse version: v0.6.0 > Commit: drm: Remove internal setup of struct > drm_device.vblank_disable_immediate > Okay! > > Commit: drm: Add get_scanout_position() to struct drm_crtc_helper_funcs > Okay! > > Commit: drm: Add get_vblank_timestamp() to struct drm_crtc_funcs > + ~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > +drivers/gpu/drm/drm_vblank.c:1869:19: warning: suggest parentheses around > ‘&&’ within ‘||’ [-Wparentheses] > +drivers/gpu/drm/drm_vblank.c: In function ‘drm_handle_vblank_events’: > + high_prec = crtc && crtc->funcs->get_vblank_timestamp || > > Commit: drm/amdgpu: Convert to struct > drm_crtc_helper_funcs.get_scanout_position() > -drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5057:6: warning: > symbol 'dm_drm_plane_destroy_state' was not declared. Should it be static? > +drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5058:6: warning: > symbol 'dm_drm_plane_destroy_state' was not declared. Should it be static? > -drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8393:6: warning: > symbol 'amdgpu_dm_psr_enable' was not declared. Should it be static? > -drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8397:50: > warning: missing braces around initializer > +drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8394:6: warning: > symbol 'amdgpu_dm_psr_enable' was not declared. Should it be static? > +drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8398:50: > warning: missing braces around initializer > > Commit: drm/amdgpu: Convert to CRTC VBLANK callbacks > -drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5058:6: warning: > symbol 'dm_drm_plane_destroy_state' was not declared. Should it be static? > +drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5060:6: warning: > symbol 'dm_drm_plane_destroy_state' was not declared. Should it be static? > -drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8394:6: warning: > symbol 'amdgpu_dm_psr_enable' was not declared. Should it be static? > -drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8398:50: > warning: missing braces around initializer > +drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8396:6: warning: > symbol 'amdgpu_dm_psr_enable' was not declared. Should it be static? > +drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:8400:50: > warning: missing braces around initializer > > Commit: drm/gma500: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/i915: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/nouveau: Convert to struct > drm_crtc_helper_funcs.get_scanout_position() > Okay! > > Commit: drm/nouveau: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/radeon: Convert to struct > drm_crtc_helper_funcs.get_scanout_position() > Okay! > > Commit: drm/radeon: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/msm: Convert to struct > drm_crtc_helper_funcs.get_scanout_position() > Okay! > > Commit: drm/msm: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/stm: Convert to struct > drm_crtc_helper_funcs.get_scanout_position() > Okay! > > Commit: drm/stm: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/sti: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/vc4: Convert to struct > drm_crtc_helper_funcs.get_scanout_position() > Okay! > > Commit: drm/vc4: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/vkms: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm/vmwgfx: Convert to CRTC VBLANK callbacks > Okay! > > Commit: drm: Clean-up VBLANK-related callbacks in struct drm_driver > - ~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > -drivers/gpu/drm/drm_vblank.c: In function ‘drm_handle_vblank_events’: > - high_prec = crtc && crtc->funcs->get_vblank_timestamp || > -O:drivers/gpu/drm/drm_vblank.c:1869:19: warning: suggest parentheses around > ‘&&’ within ‘||’ [-Wparentheses] > > Commit: drm: Remove legacy version of get_scanout_position() > Okay! > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx