Add a helper to get the EDID property for sysfs property show. This
hides all the edid_blob_ptr usage within drm_edid.c.

Signed-off-by: Jani Nikula <jani.nik...@intel.com>
---
 drivers/gpu/drm/drm_crtc_internal.h |  2 ++
 drivers/gpu/drm/drm_edid.c          | 33 +++++++++++++++++++++++++++++
 drivers/gpu/drm/drm_sysfs.c         | 24 ++-------------------
 3 files changed, 37 insertions(+), 22 deletions(-)

diff --git a/drivers/gpu/drm/drm_crtc_internal.h 
b/drivers/gpu/drm/drm_crtc_internal.h
index 25aaae937ceb..20e9d7b206a2 100644
--- a/drivers/gpu/drm/drm_crtc_internal.h
+++ b/drivers/gpu/drm/drm_crtc_internal.h
@@ -303,6 +303,8 @@ const u8 *drm_edid_find_extension(const struct drm_edid 
*drm_edid,
                                  int ext_id, int *ext_index);
 void drm_edid_cta_sad_get(const struct cea_sad *cta_sad, u8 *sad);
 void drm_edid_cta_sad_set(struct cea_sad *cta_sad, const u8 *sad);
+ssize_t drm_edid_connector_property_show(struct drm_connector *connector,
+                                        char *buf, loff_t off, size_t count);
 
 /* drm_edid_load.c */
 #ifdef CONFIG_DRM_LOAD_EDID_FIRMWARE
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 4f54c91b31b2..97362dd2330b 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -6969,6 +6969,39 @@ static int _drm_edid_connector_property_update(struct 
drm_connector *connector,
        return ret;
 }
 
+/* For sysfs edid show implementation */
+ssize_t drm_edid_connector_property_show(struct drm_connector *connector,
+                                        char *buf, loff_t off, size_t count)
+{
+       const void *edid;
+       size_t size;
+       ssize_t ret = 0;
+
+       mutex_lock(&connector->dev->mode_config.mutex);
+
+       if (!connector->edid_blob_ptr)
+               goto unlock;
+
+       edid = connector->edid_blob_ptr->data;
+       size = connector->edid_blob_ptr->length;
+       if (!edid)
+               goto unlock;
+
+       if (off >= size)
+               goto unlock;
+
+       if (off + count > size)
+               count = size - off;
+
+       memcpy(buf, edid + off, count);
+
+       ret = count;
+unlock:
+       mutex_unlock(&connector->dev->mode_config.mutex);
+
+       return ret;
+}
+
 /**
  * drm_edid_connector_update - Update connector information from EDID
  * @connector: Connector
diff --git a/drivers/gpu/drm/drm_sysfs.c b/drivers/gpu/drm/drm_sysfs.c
index bd9b8ab4f82b..fb3bbb6adcd1 100644
--- a/drivers/gpu/drm/drm_sysfs.c
+++ b/drivers/gpu/drm/drm_sysfs.c
@@ -266,29 +266,9 @@ static ssize_t edid_show(struct file *filp, struct kobject 
*kobj,
 {
        struct device *connector_dev = kobj_to_dev(kobj);
        struct drm_connector *connector = to_drm_connector(connector_dev);
-       unsigned char *edid;
-       size_t size;
-       ssize_t ret = 0;
+       ssize_t ret;
 
-       mutex_lock(&connector->dev->mode_config.mutex);
-       if (!connector->edid_blob_ptr)
-               goto unlock;
-
-       edid = connector->edid_blob_ptr->data;
-       size = connector->edid_blob_ptr->length;
-       if (!edid)
-               goto unlock;
-
-       if (off >= size)
-               goto unlock;
-
-       if (off + count > size)
-               count = size - off;
-       memcpy(buf, edid + off, count);
-
-       ret = count;
-unlock:
-       mutex_unlock(&connector->dev->mode_config.mutex);
+       ret = drm_edid_connector_property_show(connector, buf, off, count);
 
        return ret;
 }
-- 
2.39.2

Reply via email to