iommu/omap: Fix debugfs_create_*() usage
authorGeert Uytterhoeven <geert+renesas@glider.be>
Tue, 2 Jan 2018 15:29:45 +0000 (16:29 +0100)
committerJoerg Roedel <jroedel@suse.de>
Wed, 17 Jan 2018 13:23:33 +0000 (14:23 +0100)
When exposing data access through debugfs, the correct
debugfs_create_*() functions must be used, depending on data type.

Remove all casts from data pointers passed to debugfs_create_*()
functions, as such casts prevent the compiler from flagging bugs.

omap_iommu.nr_tlb_entries is "int", hence casting to "u8 *" exposes only
a part of it.  Fix this by using debugfs_create_u32() instead.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/omap-iommu-debug.c

index 505548aafeff2276c82c64de4702aae37ddfe15b..50217548c3b8e7afbb906a57b28705d2c1df7cb7 100644 (file)
@@ -274,8 +274,8 @@ void omap_iommu_debugfs_add(struct omap_iommu *obj)
        if (!obj->debug_dir)
                return;
 
-       d = debugfs_create_u8("nr_tlb_entries", 0400, obj->debug_dir,
-                             (u8 *)&obj->nr_tlb_entries);
+       d = debugfs_create_u32("nr_tlb_entries", 0400, obj->debug_dir,
+                              &obj->nr_tlb_entries);
        if (!d)
                return;