mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-12-28 11:18:45 +07:00
regulator: core: Allow multiple requests of a single supply mapping
Sometimes it may be useful to allow a device to request a supply multiple times, for example in order to allow framework management of some uses of the supply with some additional driver specific management or in order to allow multiple children of an MFD to work with the supply. Currently this is not possible due to the creation of Solve this by removing the requested_uA entry (we have no current users of this feature anyway) and ignoring errors creating the symlink to the consumer. We should do something nicer than this as this causes sysfs to spew enormous warnings but it allows users to run for now. Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com> Acked-by: Liam Girdwood <lrg@ti.com>
This commit is contained in:
parent
5aff3a8b20
commit
222cc7b1ce
@ -303,18 +303,6 @@ static int regulator_check_drms(struct regulator_dev *rdev)
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static ssize_t device_requested_uA_show(struct device *dev,
|
|
||||||
struct device_attribute *attr, char *buf)
|
|
||||||
{
|
|
||||||
struct regulator *regulator;
|
|
||||||
|
|
||||||
regulator = get_device_regulator(dev);
|
|
||||||
if (regulator == NULL)
|
|
||||||
return 0;
|
|
||||||
|
|
||||||
return sprintf(buf, "%d\n", regulator->uA_load);
|
|
||||||
}
|
|
||||||
|
|
||||||
static ssize_t regulator_uV_show(struct device *dev,
|
static ssize_t regulator_uV_show(struct device *dev,
|
||||||
struct device_attribute *attr, char *buf)
|
struct device_attribute *attr, char *buf)
|
||||||
{
|
{
|
||||||
@ -1097,48 +1085,27 @@ static struct regulator *create_regulator(struct regulator_dev *rdev,
|
|||||||
list_add(®ulator->list, &rdev->consumer_list);
|
list_add(®ulator->list, &rdev->consumer_list);
|
||||||
|
|
||||||
if (dev) {
|
if (dev) {
|
||||||
/* create a 'requested_microamps_name' sysfs entry */
|
/* Add a link to the device sysfs entry */
|
||||||
size = scnprintf(buf, REG_STR_SIZE,
|
|
||||||
"microamps_requested_%s-%s",
|
|
||||||
dev_name(dev), supply_name);
|
|
||||||
if (size >= REG_STR_SIZE)
|
|
||||||
goto overflow_err;
|
|
||||||
|
|
||||||
regulator->dev = dev;
|
|
||||||
sysfs_attr_init(®ulator->dev_attr.attr);
|
|
||||||
regulator->dev_attr.attr.name = kstrdup(buf, GFP_KERNEL);
|
|
||||||
if (regulator->dev_attr.attr.name == NULL)
|
|
||||||
goto attr_name_err;
|
|
||||||
|
|
||||||
regulator->dev_attr.attr.mode = 0444;
|
|
||||||
regulator->dev_attr.show = device_requested_uA_show;
|
|
||||||
err = device_create_file(dev, ®ulator->dev_attr);
|
|
||||||
if (err < 0) {
|
|
||||||
rdev_warn(rdev, "could not add regulator_dev requested microamps sysfs entry\n");
|
|
||||||
goto attr_name_err;
|
|
||||||
}
|
|
||||||
|
|
||||||
/* also add a link to the device sysfs entry */
|
|
||||||
size = scnprintf(buf, REG_STR_SIZE, "%s-%s",
|
size = scnprintf(buf, REG_STR_SIZE, "%s-%s",
|
||||||
dev->kobj.name, supply_name);
|
dev->kobj.name, supply_name);
|
||||||
if (size >= REG_STR_SIZE)
|
if (size >= REG_STR_SIZE)
|
||||||
goto attr_err;
|
goto overflow_err;
|
||||||
|
|
||||||
regulator->supply_name = kstrdup(buf, GFP_KERNEL);
|
regulator->supply_name = kstrdup(buf, GFP_KERNEL);
|
||||||
if (regulator->supply_name == NULL)
|
if (regulator->supply_name == NULL)
|
||||||
goto attr_err;
|
goto overflow_err;
|
||||||
|
|
||||||
err = sysfs_create_link(&rdev->dev.kobj, &dev->kobj,
|
err = sysfs_create_link(&rdev->dev.kobj, &dev->kobj,
|
||||||
buf);
|
buf);
|
||||||
if (err) {
|
if (err) {
|
||||||
rdev_warn(rdev, "could not add device link %s err %d\n",
|
rdev_warn(rdev, "could not add device link %s err %d\n",
|
||||||
dev->kobj.name, err);
|
dev->kobj.name, err);
|
||||||
goto link_name_err;
|
/* non-fatal */
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
regulator->supply_name = kstrdup(supply_name, GFP_KERNEL);
|
regulator->supply_name = kstrdup(supply_name, GFP_KERNEL);
|
||||||
if (regulator->supply_name == NULL)
|
if (regulator->supply_name == NULL)
|
||||||
goto attr_err;
|
goto overflow_err;
|
||||||
}
|
}
|
||||||
|
|
||||||
regulator->debugfs = debugfs_create_dir(regulator->supply_name,
|
regulator->debugfs = debugfs_create_dir(regulator->supply_name,
|
||||||
@ -1165,12 +1132,6 @@ static struct regulator *create_regulator(struct regulator_dev *rdev,
|
|||||||
|
|
||||||
mutex_unlock(&rdev->mutex);
|
mutex_unlock(&rdev->mutex);
|
||||||
return regulator;
|
return regulator;
|
||||||
link_name_err:
|
|
||||||
kfree(regulator->supply_name);
|
|
||||||
attr_err:
|
|
||||||
device_remove_file(regulator->dev, ®ulator->dev_attr);
|
|
||||||
attr_name_err:
|
|
||||||
kfree(regulator->dev_attr.attr.name);
|
|
||||||
overflow_err:
|
overflow_err:
|
||||||
list_del(®ulator->list);
|
list_del(®ulator->list);
|
||||||
kfree(regulator);
|
kfree(regulator);
|
||||||
|
Loading…
Reference in New Issue
Block a user