drm: crc: Introduce verify_crc_source callback
authorMahesh Kumar <mahesh1.kumar@intel.com>
Fri, 13 Jul 2018 13:59:33 +0000 (19:29 +0530)
committerMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Mon, 13 Aug 2018 12:00:20 +0000 (14:00 +0200)
This patch adds a new callback function "verify_crc_source" which will
be used during setting the crc source in control node. This will help
in avoiding setting of wrong string for source.

Changes since V1:
 - do not yet verify_crc_source during open.
Changes since V1:
 - improve callback description

Signed-off-by: Mahesh Kumar <mahesh1.kumar@intel.com>
Cc: dri-devel@lists.freedesktop.org
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180713135942.25061-2-mahesh1.kumar@intel.com
drivers/gpu/drm/drm_debugfs_crc.c
include/drm/drm_crtc.h

index 99961192bf034f893cbac5521c996dc98aa49887..72bfd8af3f7a2d0bb373da3ba3358d3eb1c7ba7d 100644 (file)
@@ -87,6 +87,8 @@ static ssize_t crc_control_write(struct file *file, const char __user *ubuf,
        struct drm_crtc *crtc = m->private;
        struct drm_crtc_crc *crc = &crtc->crc;
        char *source;
+       size_t values_cnt;
+       int ret;
 
        if (len == 0)
                return 0;
@@ -104,6 +106,12 @@ static ssize_t crc_control_write(struct file *file, const char __user *ubuf,
        if (source[len] == '\n')
                source[len] = '\0';
 
+       if (crtc->funcs->verify_crc_source) {
+               ret = crtc->funcs->verify_crc_source(crtc, source, &values_cnt);
+               if (ret)
+                       return ret;
+       }
+
        spin_lock_irq(&crc->lock);
 
        if (crc->opened) {
index 92e7fc7f05a4486746d19cfc4399dc0ecb8094f5..9dcbce93aeaea07bcff67bd4c949a23bddfac801 100644 (file)
@@ -746,6 +746,22 @@ struct drm_crtc_funcs {
         */
        int (*set_crc_source)(struct drm_crtc *crtc, const char *source,
                              size_t *values_cnt);
+       /**
+        * @verify_crc_source:
+        *
+        * verifies the source of CRC checksums of frames before setting the
+        * source for CRC and during crc open. Source parameter can be NULL
+        * while disabling crc source.
+        *
+        * This callback is optional if the driver does not support any CRC
+        * generation functionality.
+        *
+        * RETURNS:
+        *
+        * 0 on success or a negative error code on failure.
+        */
+       int (*verify_crc_source)(struct drm_crtc *crtc, const char *source,
+                                size_t *values_cnt);
 
        /**
         * @atomic_print_state: