diff options
| author | Arvind Yadav <arvind.yadav.cs@gmail.com> | 2017-09-21 11:15:36 +0530 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-10-11 01:49:17 +0200 |
| commit | d741029a2390406d4d94279ae5b346831a9e61e6 (patch) | |
| tree | 83ff522f4ee8e0ce726f33e9c7b8ed7574c91555 | |
| parent | 7813dd6fc75fb375d4caf002e7f80a826fc3153a (diff) | |
| download | blackbird-op-linux-d741029a2390406d4d94279ae5b346831a9e61e6.tar.gz blackbird-op-linux-d741029a2390406d4d94279ae5b346831a9e61e6.zip | |
PM / OPP: Use snprintf() to avoid kasprintf() and kfree()
Use snprintf() to avoid unnecessary initializations, avoid calling
kfree().
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | drivers/opp/debugfs.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/opp/debugfs.c b/drivers/opp/debugfs.c index 81cf120fcf43..9318848f3c67 100644 --- a/drivers/opp/debugfs.c +++ b/drivers/opp/debugfs.c @@ -41,16 +41,15 @@ static bool opp_debug_create_supplies(struct dev_pm_opp *opp, { struct dentry *d; int i; - char *name; for (i = 0; i < opp_table->regulator_count; i++) { - name = kasprintf(GFP_KERNEL, "supply-%d", i); + char name[15]; + + snprintf(name, sizeof(name), "supply-%d", i); /* Create per-opp directory */ d = debugfs_create_dir(name, pdentry); - kfree(name); - if (!d) return false; |
