Hi Mark, this series is a follow-up for my previous patch. The first commit renames the empty locking routine introduced in it - I agree with Andy that it just looks better.
The second patch disables the debugfs entries when locking is disabled as suggested by Lars-Peter. The last patch is optional - feel free to drop it if you think it's not necessary. I just noticed that most frameworks duplicate any name strings, if it's possible that an unaware user gives us a pointer to a temporary buffer. While I haven't found any such instance in the current kernel, I believe it would make regmap safer for the future. Bartosz Golaszewski (3): regmap: rename regmap_lock_unlock_empty() to regmap_lock_unlock_none() regmap: don't create the debugfs entries if locking is disabled regmap: duplicate the name string stored in regmap drivers/base/regmap/internal.h | 2 ++ drivers/base/regmap/regmap-debugfs.c | 5 +++++ drivers/base/regmap/regmap.c | 15 ++++++++++++--- 3 files changed, 19 insertions(+), 3 deletions(-) -- 2.15.1
