On Wed, Jul 03, 2019 at 04:03:30PM +0100, Liviu Dudau wrote: > drm_debugfs_crtc_crc_add() function checks that both .set_crc_source and > .verify_crc_source hooks are provided before enabling debugfs support for > reading per-frame CRC data. Make that explicit in the documentation. > > Cc: Daniel Vetter <daniel@xxxxxxxx> > Signed-off-by: Liviu Dudau <liviu.dudau@xxxxxxx> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> I think we have some refactoring room here if we make verify_crc_source optional (and only allow "auto" for that case). But would only remove like 3-4 implementations, so I guess that's for when the next trivial one shows up. -Daniel > --- > drivers/gpu/drm/drm_debugfs_crc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/drm_debugfs_crc.c b/drivers/gpu/drm/drm_debugfs_crc.c > index 7ca486d750e9..6604ed223160 100644 > --- a/drivers/gpu/drm/drm_debugfs_crc.c > +++ b/drivers/gpu/drm/drm_debugfs_crc.c > @@ -66,9 +66,9 @@ > * the reported CRCs of frames that should have the same contents. > * > * On the driver side the implementation effort is minimal, drivers only need to > - * implement &drm_crtc_funcs.set_crc_source. The debugfs files are automatically > - * set up if that vfunc is set. CRC samples need to be captured in the driver by > - * calling drm_crtc_add_crc_entry(). > + * implement &drm_crtc_funcs.set_crc_source and &drm_crtc_funcs.verify_crc_source. > + * The debugfs files are automatically set up if those vfuncs are set. CRC samples > + * need to be captured in the driver by calling drm_crtc_add_crc_entry(). > */ > > static int crc_control_show(struct seq_file *m, void *data) > -- > 2.21.0 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel