leds: as3645a: Switch to fwnode property API

Switch the as3645a from OF to the fwnode property API. Also add ACPI
support.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Jacek Anaszewski <jacek.anaszewski@gmail.com>
This commit is contained in:
Sakari Ailus 2019-02-25 14:23:41 +02:00 коммит произвёл Jacek Anaszewski
Родитель 9e98c678c2
Коммит 88b7e9ffe5
1 изменённых файлов: 52 добавлений и 41 удалений

Просмотреть файл

@ -25,7 +25,7 @@
#include <linux/leds.h> #include <linux/leds.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/mutex.h> #include <linux/mutex.h>
#include <linux/of.h> #include <linux/property.h>
#include <linux/slab.h> #include <linux/slab.h>
#include <media/v4l2-flash-led-class.h> #include <media/v4l2-flash-led-class.h>
@ -148,8 +148,8 @@ struct as3645a {
struct v4l2_flash *vf; struct v4l2_flash *vf;
struct v4l2_flash *vfind; struct v4l2_flash *vfind;
struct device_node *flash_node; struct fwnode_handle *flash_node;
struct device_node *indicator_node; struct fwnode_handle *indicator_node;
struct as3645a_config cfg; struct as3645a_config cfg;
@ -493,30 +493,31 @@ static int as3645a_detect(struct as3645a *flash)
static int as3645a_parse_node(struct as3645a *flash, static int as3645a_parse_node(struct as3645a *flash,
struct as3645a_names *names, struct as3645a_names *names,
struct device_node *node) struct fwnode_handle *fwnode)
{ {
struct as3645a_config *cfg = &flash->cfg; struct as3645a_config *cfg = &flash->cfg;
struct device_node *child; struct fwnode_handle *child;
const char *name; const char *name;
int rval; int rval;
for_each_child_of_node(node, child) { fwnode_for_each_child_node(fwnode, child) {
u32 id = 0; u32 id = 0;
of_property_read_u32(child, "reg", &id); fwnode_property_read_u32(child, "reg", &id);
switch (id) { switch (id) {
case AS_LED_FLASH: case AS_LED_FLASH:
flash->flash_node = of_node_get(child); flash->flash_node = child;
break; break;
case AS_LED_INDICATOR: case AS_LED_INDICATOR:
flash->indicator_node = of_node_get(child); flash->indicator_node = child;
break; break;
default: default:
dev_warn(&flash->client->dev, dev_warn(&flash->client->dev,
"unknown LED %u encountered, ignoring\n", id); "unknown LED %u encountered, ignoring\n", id);
break; break;
} }
fwnode_handle_get(child);
} }
if (!flash->flash_node) { if (!flash->flash_node) {
@ -524,14 +525,18 @@ static int as3645a_parse_node(struct as3645a *flash,
return -ENODEV; return -ENODEV;
} }
rval = of_property_read_string(flash->flash_node, "label", &name); rval = fwnode_property_read_string(flash->flash_node, "label", &name);
if (!rval) if (!rval) {
strlcpy(names->flash, name, sizeof(names->flash)); strlcpy(names->flash, name, sizeof(names->flash));
else } else if (is_of_node(fwnode)) {
snprintf(names->flash, sizeof(names->flash), snprintf(names->flash, sizeof(names->flash),
"%pOFn:flash", node); "%pOFn:flash", to_of_node(fwnode));
} else {
dev_err(&flash->client->dev, "flash node has no label!\n");
return -EINVAL;
}
rval = of_property_read_u32(flash->flash_node, "flash-timeout-us", rval = fwnode_property_read_u32(flash->flash_node, "flash-timeout-us",
&cfg->flash_timeout_us); &cfg->flash_timeout_us);
if (rval < 0) { if (rval < 0) {
dev_err(&flash->client->dev, dev_err(&flash->client->dev,
@ -539,7 +544,7 @@ static int as3645a_parse_node(struct as3645a *flash,
goto out_err; goto out_err;
} }
rval = of_property_read_u32(flash->flash_node, "flash-max-microamp", rval = fwnode_property_read_u32(flash->flash_node, "flash-max-microamp",
&cfg->flash_max_ua); &cfg->flash_max_ua);
if (rval < 0) { if (rval < 0) {
dev_err(&flash->client->dev, dev_err(&flash->client->dev,
@ -547,7 +552,7 @@ static int as3645a_parse_node(struct as3645a *flash,
goto out_err; goto out_err;
} }
rval = of_property_read_u32(flash->flash_node, "led-max-microamp", rval = fwnode_property_read_u32(flash->flash_node, "led-max-microamp",
&cfg->assist_max_ua); &cfg->assist_max_ua);
if (rval < 0) { if (rval < 0) {
dev_err(&flash->client->dev, dev_err(&flash->client->dev,
@ -555,10 +560,10 @@ static int as3645a_parse_node(struct as3645a *flash,
goto out_err; goto out_err;
} }
of_property_read_u32(flash->flash_node, "voltage-reference", fwnode_property_read_u32(flash->flash_node, "voltage-reference",
&cfg->voltage_reference); &cfg->voltage_reference);
of_property_read_u32(flash->flash_node, "ams,input-max-microamp", fwnode_property_read_u32(flash->flash_node, "ams,input-max-microamp",
&cfg->peak); &cfg->peak);
cfg->peak = AS_PEAK_mA_TO_REG(cfg->peak); cfg->peak = AS_PEAK_mA_TO_REG(cfg->peak);
@ -568,14 +573,20 @@ static int as3645a_parse_node(struct as3645a *flash,
goto out_err; goto out_err;
} }
rval = of_property_read_string(flash->indicator_node, "label", &name); rval = fwnode_property_read_string(flash->indicator_node, "label",
if (!rval) &name);
if (!rval) {
strlcpy(names->indicator, name, sizeof(names->indicator)); strlcpy(names->indicator, name, sizeof(names->indicator));
else } else if (is_of_node(fwnode)) {
snprintf(names->indicator, sizeof(names->indicator), snprintf(names->indicator, sizeof(names->indicator),
"%pOFn:indicator", node); "%pOFn:indicator", to_of_node(fwnode));
} else {
dev_err(&flash->client->dev, "indicator node has no label!\n");
return -EINVAL;
}
rval = of_property_read_u32(flash->indicator_node, "led-max-microamp", rval = fwnode_property_read_u32(flash->indicator_node,
"led-max-microamp",
&cfg->indicator_max_ua); &cfg->indicator_max_ua);
if (rval < 0) { if (rval < 0) {
dev_err(&flash->client->dev, dev_err(&flash->client->dev,
@ -586,8 +597,8 @@ static int as3645a_parse_node(struct as3645a *flash,
return 0; return 0;
out_err: out_err:
of_node_put(flash->flash_node); fwnode_handle_put(flash->flash_node);
of_node_put(flash->indicator_node); fwnode_handle_put(flash->indicator_node);
return rval; return rval;
} }
@ -668,14 +679,14 @@ static int as3645a_v4l2_setup(struct as3645a *flash)
strlcpy(cfgind.dev_name, flash->iled_cdev.name, sizeof(cfg.dev_name)); strlcpy(cfgind.dev_name, flash->iled_cdev.name, sizeof(cfg.dev_name));
flash->vf = v4l2_flash_init( flash->vf = v4l2_flash_init(
&flash->client->dev, of_fwnode_handle(flash->flash_node), &flash->client->dev, flash->flash_node, &flash->fled, NULL,
&flash->fled, NULL, &cfg); &cfg);
if (IS_ERR(flash->vf)) if (IS_ERR(flash->vf))
return PTR_ERR(flash->vf); return PTR_ERR(flash->vf);
flash->vfind = v4l2_flash_indicator_init( flash->vfind = v4l2_flash_indicator_init(
&flash->client->dev, of_fwnode_handle(flash->indicator_node), &flash->client->dev, flash->indicator_node, &flash->iled_cdev,
&flash->iled_cdev, &cfgind); &cfgind);
if (IS_ERR(flash->vfind)) { if (IS_ERR(flash->vfind)) {
v4l2_flash_release(flash->vf); v4l2_flash_release(flash->vf);
return PTR_ERR(flash->vfind); return PTR_ERR(flash->vfind);
@ -690,7 +701,7 @@ static int as3645a_probe(struct i2c_client *client)
struct as3645a *flash; struct as3645a *flash;
int rval; int rval;
if (client->dev.of_node == NULL) if (!dev_fwnode(&client->dev))
return -ENODEV; return -ENODEV;
flash = devm_kzalloc(&client->dev, sizeof(*flash), GFP_KERNEL); flash = devm_kzalloc(&client->dev, sizeof(*flash), GFP_KERNEL);
@ -699,7 +710,7 @@ static int as3645a_probe(struct i2c_client *client)
flash->client = client; flash->client = client;
rval = as3645a_parse_node(flash, &names, client->dev.of_node); rval = as3645a_parse_node(flash, &names, dev_fwnode(&client->dev));
if (rval < 0) if (rval < 0)
return rval; return rval;
@ -731,8 +742,8 @@ out_mutex_destroy:
mutex_destroy(&flash->mutex); mutex_destroy(&flash->mutex);
out_put_nodes: out_put_nodes:
of_node_put(flash->flash_node); fwnode_handle_put(flash->flash_node);
of_node_put(flash->indicator_node); fwnode_handle_put(flash->indicator_node);
return rval; return rval;
} }
@ -751,8 +762,8 @@ static int as3645a_remove(struct i2c_client *client)
mutex_destroy(&flash->mutex); mutex_destroy(&flash->mutex);
of_node_put(flash->flash_node); fwnode_handle_put(flash->flash_node);
of_node_put(flash->indicator_node); fwnode_handle_put(flash->indicator_node);
return 0; return 0;
} }