summaryrefslogtreecommitdiffstats
path: root/include/linux/debugfs.h
diff options
context:
space:
mode:
authorViresh Kumar <viresh.kumar@linaro.org>2015-09-26 15:04:07 -0700
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-10-04 11:36:07 +0100
commit621a5f7ad9cd1ce7933f1d302067cbd58354173c (patch)
treea9cd51d4b4e70286a339ffc463618d0f33279c67 /include/linux/debugfs.h
parent6e58f752a6502b43e039fd7df2c7c5cde8dde437 (diff)
downloadop-kernel-dev-621a5f7ad9cd1ce7933f1d302067cbd58354173c.zip
op-kernel-dev-621a5f7ad9cd1ce7933f1d302067cbd58354173c.tar.gz
debugfs: Pass bool pointer to debugfs_create_bool()
Its a bit odd that debugfs_create_bool() takes 'u32 *' as an argument, when all it needs is a boolean pointer. It would be better to update this API to make it accept 'bool *' instead, as that will make it more consistent and often more convenient. Over that bool takes just a byte. That required updates to all user sites as well, in the same commit updating the API. regmap core was also using debugfs_{read|write}_file_bool(), directly and variable types were updated for that to be bool as well. Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> Acked-by: Mark Brown <broonie@kernel.org> Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux/debugfs.h')
-rw-r--r--include/linux/debugfs.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/debugfs.h b/include/linux/debugfs.h
index 9beb636..8321fe3 100644
--- a/include/linux/debugfs.h
+++ b/include/linux/debugfs.h
@@ -92,7 +92,7 @@ struct dentry *debugfs_create_size_t(const char *name, umode_t mode,
struct dentry *debugfs_create_atomic_t(const char *name, umode_t mode,
struct dentry *parent, atomic_t *value);
struct dentry *debugfs_create_bool(const char *name, umode_t mode,
- struct dentry *parent, u32 *value);
+ struct dentry *parent, bool *value);
struct dentry *debugfs_create_blob(const char *name, umode_t mode,
struct dentry *parent,
@@ -243,7 +243,7 @@ static inline struct dentry *debugfs_create_atomic_t(const char *name, umode_t m
static inline struct dentry *debugfs_create_bool(const char *name, umode_t mode,
struct dentry *parent,
- u32 *value)
+ bool *value)
{
return ERR_PTR(-ENODEV);
}
OpenPOWER on IntegriCloud