drm/armada: redo CRTC debugfs files
authorRussell King <rmk+kernel@armlinux.org.uk>
Sat, 7 Jul 2018 15:53:39 +0000 (16:53 +0100)
committerRussell King <rmk+kernel@armlinux.org.uk>
Fri, 28 Jun 2019 13:50:07 +0000 (14:50 +0100)
Move the CRTC debugfs files into the CRTC specific directory.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
drivers/gpu/drm/armada/armada_crtc.c
drivers/gpu/drm/armada/armada_debugfs.c
drivers/gpu/drm/armada/armada_drm.h

index 5de7ae7c0d765868992f855226994ca155f3b2af..2e17d7e72eeb9e0faf3a30b9aec3543eafed4c86 100644 (file)
@@ -774,6 +774,14 @@ static void armada_drm_crtc_destroy(struct drm_crtc *crtc)
        kfree(dcrtc);
 }
 
+static int armada_drm_crtc_late_register(struct drm_crtc *crtc)
+{
+       if (IS_ENABLED(CONFIG_DEBUG_FS))
+               armada_drm_crtc_debugfs_init(drm_to_armada_crtc(crtc));
+
+       return 0;
+}
+
 /* These are called under the vbl_lock. */
 static int armada_drm_crtc_enable_vblank(struct drm_crtc *crtc)
 {
@@ -806,6 +814,7 @@ static const struct drm_crtc_funcs armada_crtc_funcs = {
        .page_flip      = drm_atomic_helper_page_flip,
        .atomic_duplicate_state = drm_atomic_helper_crtc_duplicate_state,
        .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state,
+       .late_register  = armada_drm_crtc_late_register,
        .enable_vblank  = armada_drm_crtc_enable_vblank,
        .disable_vblank = armada_drm_crtc_disable_vblank,
 };
index 6758c3a83de24bde1d7201a2babfa70dc45bbf56..4dcce002ea2a77bda3be77383de90e8c7efcf1a0 100644 (file)
@@ -28,50 +28,33 @@ static int armada_debugfs_gem_linear_show(struct seq_file *m, void *data)
        return 0;
 }
 
-static int armada_debugfs_reg_show(struct seq_file *m, void *data)
+static int armada_debugfs_crtc_reg_show(struct seq_file *m, void *data)
 {
-       struct drm_device *dev = m->private;
-       struct armada_private *priv = dev->dev_private;
-       int n, i;
-
-       if (priv) {
-               for (n = 0; n < ARRAY_SIZE(priv->dcrtc); n++) {
-                       struct armada_crtc *dcrtc = priv->dcrtc[n];
-                       if (!dcrtc)
-                               continue;
-
-                       for (i = 0x84; i <= 0x1c4; i += 4) {
-                               uint32_t v = readl_relaxed(dcrtc->base + i);
-                               seq_printf(m, "%u: 0x%04x: 0x%08x\n", n, i, v);
-                       }
-               }
+       struct armada_crtc *dcrtc = m->private;
+       int i;
+
+       for (i = 0x84; i <= 0x1c4; i += 4) {
+               u32 v = readl_relaxed(dcrtc->base + i);
+               seq_printf(m, "0x%04x: 0x%08x\n", i, v);
        }
 
        return 0;
 }
 
-static int armada_debugfs_reg_r_open(struct inode *inode, struct file *file)
+static int armada_debugfs_crtc_reg_open(struct inode *inode, struct file *file)
 {
-       return single_open(file, armada_debugfs_reg_show, inode->i_private);
+       return single_open(file, armada_debugfs_crtc_reg_show,
+                          inode->i_private);
 }
 
-static const struct file_operations fops_reg_r = {
-       .owner = THIS_MODULE,
-       .open = armada_debugfs_reg_r_open,
-       .read = seq_read,
-       .llseek = seq_lseek,
-       .release = single_release,
-};
-
-static int armada_debugfs_write(struct file *file, const char __user *ptr,
-       size_t len, loff_t *off)
+static int armada_debugfs_crtc_reg_write(struct file *file,
+       const char __user *ptr, size_t len, loff_t *off)
 {
-       struct drm_device *dev = file->private_data;
-       struct armada_private *priv = dev->dev_private;
-       struct armada_crtc *dcrtc = priv->dcrtc[0];
-       char buf[32], *p;
-       uint32_t reg, val;
+       struct armada_crtc *dcrtc;
+       unsigned long reg, mask, val;
+       char buf[32];
        int ret;
+       u32 v;
 
        if (*off != 0)
                return 0;
@@ -84,24 +67,35 @@ static int armada_debugfs_write(struct file *file, const char __user *ptr,
                return ret;
        buf[len] = '\0';
 
-       reg = simple_strtoul(buf, &p, 16);
-       if (!isspace(*p))
+       if (sscanf(buf, "%lx %lx %lx", &reg, &mask, &val) != 3)
                return -EINVAL;
-       val = simple_strtoul(p + 1, NULL, 16);
+       if (reg < 0x84 || reg > 0x1c4 || reg & 3)
+               return -ERANGE;
 
-       if (reg >= 0x84 && reg <= 0x1c4)
-               writel(val, dcrtc->base + reg);
+       dcrtc = ((struct seq_file *)file->private_data)->private;
+       v = readl(dcrtc->base + reg);
+       v &= ~mask;
+       v |= val & mask;
+       writel(v, dcrtc->base + reg);
 
        return len;
 }
 
-static const struct file_operations fops_reg_w = {
+static const struct file_operations armada_debugfs_crtc_reg_fops = {
        .owner = THIS_MODULE,
-       .open = simple_open,
-       .write = armada_debugfs_write,
-       .llseek = noop_llseek,
+       .open = armada_debugfs_crtc_reg_open,
+       .read = seq_read,
+       .write = armada_debugfs_crtc_reg_write,
+       .llseek = seq_lseek,
+       .release = single_release,
 };
 
+void armada_drm_crtc_debugfs_init(struct armada_crtc *dcrtc)
+{
+       debugfs_create_file("armada-regs", 0600, dcrtc->crtc.debugfs_entry,
+                           dcrtc, &armada_debugfs_crtc_reg_fops);
+}
+
 static struct drm_info_list armada_debugfs_list[] = {
        { "gem_linear", armada_debugfs_gem_linear_show, 0 },
 };
@@ -109,24 +103,8 @@ static struct drm_info_list armada_debugfs_list[] = {
 
 int armada_drm_debugfs_init(struct drm_minor *minor)
 {
-       struct dentry *de;
-       int ret;
-
-       ret = drm_debugfs_create_files(armada_debugfs_list,
-                                      ARMADA_DEBUGFS_ENTRIES,
-                                      minor->debugfs_root, minor);
-       if (ret)
-               return ret;
-
-       de = debugfs_create_file("reg", S_IFREG | S_IRUSR,
-                                minor->debugfs_root, minor->dev, &fops_reg_r);
-       if (!de)
-               return -ENOMEM;
-
-       de = debugfs_create_file("reg_wr", S_IFREG | S_IWUSR,
-                                minor->debugfs_root, minor->dev, &fops_reg_w);
-       if (!de)
-               return -ENOMEM;
+       drm_debugfs_create_files(armada_debugfs_list, ARMADA_DEBUGFS_ENTRIES,
+                                minor->debugfs_root, minor);
 
        return 0;
 }
index f09083ff15d3b678ab41d811f2813034411793cd..b6307235f320560823c6d01f82cf42f5654eb214 100644 (file)
@@ -78,6 +78,7 @@ void armada_fbdev_fini(struct drm_device *);
 
 int armada_overlay_plane_create(struct drm_device *, unsigned long);
 
+void armada_drm_crtc_debugfs_init(struct armada_crtc *dcrtc);
 int armada_drm_debugfs_init(struct drm_minor *);
 
 #endif