diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2015-09-26 15:04:07 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-10-04 11:36:07 +0100 |
commit | 621a5f7ad9cd1ce7933f1d302067cbd58354173c (patch) | |
tree | a9cd51d4b4e70286a339ffc463618d0f33279c67 /drivers/scsi/snic/snic_trc.h | |
parent | 6e58f752a6502b43e039fd7df2c7c5cde8dde437 (diff) | |
download | linux-621a5f7ad9cd1ce7933f1d302067cbd58354173c.tar.gz linux-621a5f7ad9cd1ce7933f1d302067cbd58354173c.tar.bz2 linux-621a5f7ad9cd1ce7933f1d302067cbd58354173c.zip |
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 'drivers/scsi/snic/snic_trc.h')
-rw-r--r-- | drivers/scsi/snic/snic_trc.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/snic/snic_trc.h b/drivers/scsi/snic/snic_trc.h index 427faee5f97e..b37f8867bfde 100644 --- a/drivers/scsi/snic/snic_trc.h +++ b/drivers/scsi/snic/snic_trc.h @@ -45,7 +45,7 @@ struct snic_trc { u32 max_idx; /* Max Index into trace buffer */ u32 rd_idx; u32 wr_idx; - u32 enable; /* Control Variable for Tracing */ + bool enable; /* Control Variable for Tracing */ struct dentry *trc_enable; /* debugfs file object */ struct dentry *trc_file; |