| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <200811091920.06174.david-b@pacbell.net>
Date: Sun, 9 Nov 2008 19:20:06 -0800
From: David Brownell <david-b@...bell.net>
To: broonie@...nsource.wolfsonmicro.com, lrg@...mlogic.co.uk
Cc: lkml <linux-kernel@...r.kernel.org>
Subject: [patch 2.6.28-rc4 1/2] regulator: code shrink
From: David Brownell <dbrownell@...rs.sourceforge.net>
Shrink regulator core by removing duplication in attribute printing
and probe() cleanup paths. Saves about 340 bytes (object) on ARM.
Signed-off-by: David Brownell <dbrownell@...rs.sourceforge.net>
---
Goes on top of the previous REGULATOR_MODE_OFF patch, but that
only affects one obvious chunk of code.
drivers/regulator/core.c | 113 +++++++++++++++++----------------------------
1 file changed, 44 insertions(+), 69 deletions(-)
--- a/drivers/regulator/core.c
+++ b/drivers/regulator/core.c
@@ -257,12 +257,8 @@ static ssize_t regulator_name_show(struc
return sprintf(buf, "%s\n", name);
}
-static ssize_t regulator_opmode_show(struct device *dev,
- struct device_attribute *attr, char *buf)
+static ssize_t regulator_print_opmode(char *buf, int mode)
{
- struct regulator_dev *rdev = dev_get_drvdata(dev);
- int mode = _regulator_get_mode(rdev);
-
switch (mode) {
case REGULATOR_MODE_FAST:
return sprintf(buf, "fast\n");
@@ -278,12 +274,16 @@ static ssize_t regulator_opmode_show(str
return sprintf(buf, "unknown\n");
}
-static ssize_t regulator_state_show(struct device *dev,
- struct device_attribute *attr, char *buf)
+static ssize_t regulator_opmode_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
{
struct regulator_dev *rdev = dev_get_drvdata(dev);
- int state = _regulator_is_enabled(rdev);
+ return regulator_print_opmode(buf, _regulator_get_mode(rdev));
+}
+
+static ssize_t regulator_print_state(char *buf, int state)
+{
if (state > 0)
return sprintf(buf, "enabled\n");
else if (state == 0)
@@ -292,6 +292,14 @@ static ssize_t regulator_state_show(stru
return sprintf(buf, "unknown\n");
}
+static ssize_t regulator_state_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
+{
+ struct regulator_dev *rdev = dev_get_drvdata(dev);
+
+ return regulator_print_state(buf, _regulator_is_enabled(rdev));
+}
+
static ssize_t regulator_min_uA_show(struct device *dev,
struct device_attribute *attr, char *buf)
{
@@ -401,24 +409,6 @@ static ssize_t regulator_suspend_standby
return sprintf(buf, "%d\n", rdev->constraints->state_standby.uV);
}
-static ssize_t suspend_opmode_show(struct regulator_dev *rdev,
- unsigned int mode, char *buf)
-{
- switch (mode) {
- case REGULATOR_MODE_FAST:
- return sprintf(buf, "fast\n");
- case REGULATOR_MODE_NORMAL:
- return sprintf(buf, "normal\n");
- case REGULATOR_MODE_IDLE:
- return sprintf(buf, "idle\n");
- case REGULATOR_MODE_STANDBY:
- return sprintf(buf, "standby\n");
- case REGULATOR_MODE_OFF:
- return sprintf(buf, "off\n");
- }
- return sprintf(buf, "unknown\n");
-}
-
static ssize_t regulator_suspend_mem_mode_show(struct device *dev,
struct device_attribute *attr, char *buf)
{
@@ -426,8 +416,8 @@ static ssize_t regulator_suspend_mem_mod
if (!rdev->constraints)
return sprintf(buf, "not defined\n");
- return suspend_opmode_show(rdev,
- rdev->constraints->state_mem.mode, buf);
+ return regulator_print_opmode(buf,
+ rdev->constraints->state_mem.mode);
}
static ssize_t regulator_suspend_disk_mode_show(struct device *dev,
@@ -437,8 +427,8 @@ static ssize_t regulator_suspend_disk_mo
if (!rdev->constraints)
return sprintf(buf, "not defined\n");
- return suspend_opmode_show(rdev,
- rdev->constraints->state_disk.mode, buf);
+ return regulator_print_opmode(buf,
+ rdev->constraints->state_disk.mode);
}
static ssize_t regulator_suspend_standby_mode_show(struct device *dev,
@@ -448,8 +438,8 @@ static ssize_t regulator_suspend_standby
if (!rdev->constraints)
return sprintf(buf, "not defined\n");
- return suspend_opmode_show(rdev,
- rdev->constraints->state_standby.mode, buf);
+ return regulator_print_opmode(buf,
+ rdev->constraints->state_standby.mode);
}
static ssize_t regulator_suspend_mem_state_show(struct device *dev,
@@ -460,10 +450,8 @@ static ssize_t regulator_suspend_mem_sta
if (!rdev->constraints)
return sprintf(buf, "not defined\n");
- if (rdev->constraints->state_mem.enabled)
- return sprintf(buf, "enabled\n");
- else
- return sprintf(buf, "disabled\n");
+ return regulator_print_state(buf,
+ rdev->constraints->state_mem.enabled);
}
static ssize_t regulator_suspend_disk_state_show(struct device *dev,
@@ -474,10 +462,8 @@ static ssize_t regulator_suspend_disk_st
if (!rdev->constraints)
return sprintf(buf, "not defined\n");
- if (rdev->constraints->state_disk.enabled)
- return sprintf(buf, "enabled\n");
- else
- return sprintf(buf, "disabled\n");
+ return regulator_print_state(buf,
+ rdev->constraints->state_disk.enabled);
}
static ssize_t regulator_suspend_standby_state_show(struct device *dev,
@@ -488,10 +474,8 @@ static ssize_t regulator_suspend_standby
if (!rdev->constraints)
return sprintf(buf, "not defined\n");
- if (rdev->constraints->state_standby.enabled)
- return sprintf(buf, "enabled\n");
- else
- return sprintf(buf, "disabled\n");
+ return regulator_print_state(buf,
+ rdev->constraints->state_standby.enabled);
}
static struct device_attribute regulator_dev_attrs[] = {
@@ -1771,20 +1755,14 @@ struct regulator_dev *regulator_register
/* preform any regulator specific init */
if (init_data->regulator_init) {
ret = init_data->regulator_init(rdev->reg_data);
- if (ret < 0) {
- kfree(rdev);
- rdev = ERR_PTR(ret);
- goto out;
- }
+ if (ret < 0)
+ goto clean;
}
/* set regulator constraints */
ret = set_machine_constraints(rdev, &init_data->constraints);
- if (ret < 0) {
- kfree(rdev);
- rdev = ERR_PTR(ret);
- goto out;
- }
+ if (ret < 0)
+ goto clean;
/* register with sysfs */
rdev->dev.class = ®ulator_class;
@@ -1792,11 +1770,8 @@ struct regulator_dev *regulator_register
snprintf(rdev->dev.bus_id, sizeof(rdev->dev.bus_id),
"regulator.%d", atomic_inc_return(®ulator_no) - 1);
ret = device_register(&rdev->dev);
- if (ret != 0) {
- kfree(rdev);
- rdev = ERR_PTR(ret);
- goto out;
- }
+ if (ret != 0)
+ goto clean;
dev_set_drvdata(&rdev->dev, rdev);
@@ -1804,12 +1779,8 @@ struct regulator_dev *regulator_register
if (init_data->supply_regulator_dev) {
ret = set_supply(rdev,
dev_get_drvdata(init_data->supply_regulator_dev));
- if (ret < 0) {
- device_unregister(&rdev->dev);
- kfree(rdev);
- rdev = ERR_PTR(ret);
- goto out;
- }
+ if (ret < 0)
+ goto scrub;
}
/* add consumers devices */
@@ -1821,10 +1792,7 @@ struct regulator_dev *regulator_register
for (--i; i >= 0; i--)
unset_consumer_device_supply(rdev,
init_data->consumer_supplies[i].dev);
- device_unregister(&rdev->dev);
- kfree(rdev);
- rdev = ERR_PTR(ret);
- goto out;
+ goto scrub;
}
}
@@ -1832,6 +1800,13 @@ struct regulator_dev *regulator_register
out:
mutex_unlock(®ulator_list_mutex);
return rdev;
+
+scrub:
+ device_unregister(&rdev->dev);
+clean:
+ kfree(rdev);
+ rdev = ERR_PTR(ret);
+ goto out;
}
EXPORT_SYMBOL_GPL(regulator_register);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists