* [Patch v5 0/1] leds: pca9532: Add device tree binding
@ 2016-06-14 1:56 Phil Reid
[not found] ` <1465869369-13449-1-git-send-email-preid-qgqNFa1JUf/o2iN0hyhwsIdd74u8MsAO@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Phil Reid @ 2016-06-14 1:56 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
riku.voipio, rpurdie, j.anaszewski, devicetree, linux-leds
Cc: Phil Reid
This patch adds very basic device tree binding support to the
pca9532 driver. Sufficent to load driver and enable LEDs.
Changes for v5:
- Address warning on 64bit builds. Change devid cast to unitptr_t.
- Make pca9532_of_populate_pdata static.
Changes for v4:
- Reference leds/common.txt in documentation.
- Check return value of of_match_device()
Changes for v3:
- Remove state from device tree binding.
This needs more thought on how to configure which PWM is used.
And is not essential for a first pass.
Changes for v2:
- Fixup device tree documentation
- Use label propert and fall back to node name,
- Call of_node_put when breaking loop iterating leds.
- Increment the loop counter.
- Add initial state and default-trigger.
Phil Reid (1):
leds: pca9532: Add device tree binding
.../devicetree/bindings/leds/leds-pca9532.txt | 39 +++++++++++
drivers/leds/leds-pca9532.c | 76 ++++++++++++++++++++--
include/dt-bindings/leds/leds-pca9532.h | 18 +++++
include/linux/leds-pca9532.h | 9 ++-
4 files changed, 133 insertions(+), 9 deletions(-)
create mode 100644 Documentation/devicetree/bindings/leds/leds-pca9532.txt
create mode 100644 include/dt-bindings/leds/leds-pca9532.h
--
1.8.3.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Patch v5 1/1] leds: pca9532: Add device tree binding
[not found] ` <1465869369-13449-1-git-send-email-preid-qgqNFa1JUf/o2iN0hyhwsIdd74u8MsAO@public.gmane.org>
@ 2016-06-14 1:56 ` Phil Reid
2016-06-14 7:30 ` Jacek Anaszewski
0 siblings, 1 reply; 3+ messages in thread
From: Phil Reid @ 2016-06-14 1:56 UTC (permalink / raw)
To: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, riku.voipio-X3B1VOXEql0,
rpurdie-Fm38FmjxZ/leoWH0uzbU5w,
j.anaszewski-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-leds-u79uwXL29TY76Z2rM5mHXA
Cc: Phil Reid
This patch adds basic device tree support for the pca9532 LEDs.
Signed-off-by: Phil Reid <preid-qgqNFa1JUf/o2iN0hyhwsIdd74u8MsAO@public.gmane.org>
---
.../devicetree/bindings/leds/leds-pca9532.txt | 39 +++++++++++
drivers/leds/leds-pca9532.c | 76 ++++++++++++++++++++--
include/dt-bindings/leds/leds-pca9532.h | 18 +++++
include/linux/leds-pca9532.h | 9 ++-
4 files changed, 133 insertions(+), 9 deletions(-)
create mode 100644 Documentation/devicetree/bindings/leds/leds-pca9532.txt
create mode 100644 include/dt-bindings/leds/leds-pca9532.h
diff --git a/Documentation/devicetree/bindings/leds/leds-pca9532.txt b/Documentation/devicetree/bindings/leds/leds-pca9532.txt
new file mode 100644
index 0000000..198f3ba
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/leds-pca9532.txt
@@ -0,0 +1,39 @@
+*NXP - pca9532 PWM LED Driver
+
+The PCA9532 family is SMBus I/O expander optimized for dimming LEDs.
+The PWM support 256 steps.
+
+Required properties:
+ - compatible:
+ "nxp,pca9530"
+ "nxp,pca9531"
+ "nxp,pca9532"
+ "nxp,pca9533"
+ - reg - I2C slave address
+
+Each led is represented as a sub-node of the nxp,pca9530.
+
+Optional sub-node properties:
+ - label: see Documentation/devicetree/bindings/leds/common.txt
+ - type: Output configuration, see dt-bindings/leds/leds-pca9532.h (default NONE)
+ - linux,default-trigger: see Documentation/devicetree/bindings/leds/common.txt
+
+Example:
+ #include <dt-bindings/leds/leds-pca9532.h>
+
+ leds: pca9530@60 {
+ compatible = "nxp,pca9530";
+ reg = <0x60>;
+
+ red-power {
+ label = "pca:red:power";
+ type = <PCA9532_TYPE_LED>;
+ };
+ green-power {
+ label = "pca:green:power";
+ type = <PCA9532_TYPE_LED>;
+ };
+ };
+
+For more product information please see the link below:
+http://nxp.com/documents/data_sheet/PCA9532.pdf
diff --git a/drivers/leds/leds-pca9532.c b/drivers/leds/leds-pca9532.c
index e3d3b1a..49930a3 100644
--- a/drivers/leds/leds-pca9532.c
+++ b/drivers/leds/leds-pca9532.c
@@ -21,6 +21,8 @@
#include <linux/workqueue.h>
#include <linux/leds-pca9532.h>
#include <linux/gpio.h>
+#include <linux/of.h>
+#include <linux/of_device.h>
/* m = num_leds*/
#define PCA9532_REG_INPUT(i) ((i) >> 3)
@@ -86,9 +88,22 @@ static const struct pca9532_chip_info pca9532_chip_info_tbl[] = {
},
};
+#ifdef CONFIG_OF
+static const struct of_device_id of_pca9532_leds_match[] = {
+ { .compatible = "nxp,pca9530", .data = (void *)pca9530 },
+ { .compatible = "nxp,pca9531", .data = (void *)pca9531 },
+ { .compatible = "nxp,pca9532", .data = (void *)pca9532 },
+ { .compatible = "nxp,pca9533", .data = (void *)pca9533 },
+ {},
+};
+
+MODULE_DEVICE_TABLE(of, of_pca9532_leds_match);
+#endif
+
static struct i2c_driver pca9532_driver = {
.driver = {
.name = "leds-pca953x",
+ .of_match_table = of_match_ptr(of_pca9532_leds_match),
},
.probe = pca9532_probe,
.remove = pca9532_remove,
@@ -354,6 +369,7 @@ static int pca9532_configure(struct i2c_client *client,
led->state = pled->state;
led->name = pled->name;
led->ldev.name = led->name;
+ led->ldev.default_trigger = led->default_trigger;
led->ldev.brightness = LED_OFF;
led->ldev.brightness_set_blocking =
pca9532_set_brightness;
@@ -432,15 +448,67 @@ exit:
return err;
}
+static struct pca9532_platform_data *
+pca9532_of_populate_pdata(struct device *dev, struct device_node *np)
+{
+ struct pca9532_platform_data *pdata;
+ struct device_node *child;
+ const struct of_device_id *match;
+ uintptr_t devid;
+ int maxleds;
+ int i = 0;
+
+ match = of_match_device(of_pca9532_leds_match, dev);
+ if (!match)
+ return ERR_PTR(-ENODEV);
+
+ devid = (uintptr_t)match->data;
+ maxleds = pca9532_chip_info_tbl[devid].num_leds;
+
+ pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL);
+ if (!pdata)
+ return ERR_PTR(-ENOMEM);
+
+ for_each_child_of_node(np, child) {
+ if (of_property_read_string(child, "label",
+ &pdata->leds[i].name))
+ pdata->leds[i].name = child->name;
+ of_property_read_u32(child, "type", &pdata->leds[i].type);
+ of_property_read_string(child, "linux,default-trigger",
+ &pdata->leds[i].default_trigger);
+ if (++i >= maxleds) {
+ of_node_put(child);
+ break;
+ }
+ }
+
+ return pdata;
+}
+
static int pca9532_probe(struct i2c_client *client,
const struct i2c_device_id *id)
{
+ uintptr_t devid;
struct pca9532_data *data = i2c_get_clientdata(client);
struct pca9532_platform_data *pca9532_pdata =
dev_get_platdata(&client->dev);
-
- if (!pca9532_pdata)
- return -EIO;
+ struct device_node *np = client->dev.of_node;
+
+ if (!pca9532_pdata) {
+ if (np) {
+ pca9532_pdata =
+ pca9532_of_populate_pdata(&client->dev, np);
+ if (IS_ERR(pca9532_pdata))
+ return PTR_ERR(pca9532_pdata);
+ } else {
+ dev_err(&client->dev, "no platform data\n");
+ return -EINVAL;
+ }
+ devid = (uintptr_t)of_match_device(
+ of_pca9532_leds_match, &client->dev)->data;
+ } else {
+ devid = id->driver_data;
+ }
if (!i2c_check_functionality(client->adapter,
I2C_FUNC_SMBUS_BYTE_DATA))
@@ -450,7 +518,7 @@ static int pca9532_probe(struct i2c_client *client,
if (!data)
return -ENOMEM;
- data->chip_info = &pca9532_chip_info_tbl[id->driver_data];
+ data->chip_info = &pca9532_chip_info_tbl[devid];
dev_info(&client->dev, "setting platform data\n");
i2c_set_clientdata(client, data);
diff --git a/include/dt-bindings/leds/leds-pca9532.h b/include/dt-bindings/leds/leds-pca9532.h
new file mode 100644
index 0000000..4d917aa
--- /dev/null
+++ b/include/dt-bindings/leds/leds-pca9532.h
@@ -0,0 +1,18 @@
+/*
+ * This header provides constants for pca9532 LED bindings.
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+#ifndef _DT_BINDINGS_LEDS_PCA9532_H
+#define _DT_BINDINGS_LEDS_PCA9532_H
+
+#define PCA9532_TYPE_NONE 0
+#define PCA9532_TYPE_LED 1
+#define PCA9532_TYPE_N2100_BEEP 2
+#define PCA9532_TYPE_GPIO 3
+#define PCA9532_LED_TIMER2 4
+
+#endif /* _DT_BINDINGS_LEDS_PCA9532_H */
diff --git a/include/linux/leds-pca9532.h b/include/linux/leds-pca9532.h
index b8d6fff..d215b45 100644
--- a/include/linux/leds-pca9532.h
+++ b/include/linux/leds-pca9532.h
@@ -16,6 +16,7 @@
#include <linux/leds.h>
#include <linux/workqueue.h>
+#include <dt-bindings/leds/leds-pca9532.h>
enum pca9532_state {
PCA9532_OFF = 0x0,
@@ -24,16 +25,14 @@ enum pca9532_state {
PCA9532_PWM1 = 0x3
};
-enum pca9532_type { PCA9532_TYPE_NONE, PCA9532_TYPE_LED,
- PCA9532_TYPE_N2100_BEEP, PCA9532_TYPE_GPIO };
-
struct pca9532_led {
u8 id;
struct i2c_client *client;
- char *name;
+ const char *name;
+ const char *default_trigger;
struct led_classdev ldev;
struct work_struct work;
- enum pca9532_type type;
+ u32 type;
enum pca9532_state state;
};
--
1.8.3.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Patch v5 1/1] leds: pca9532: Add device tree binding
2016-06-14 1:56 ` [Patch v5 1/1] " Phil Reid
@ 2016-06-14 7:30 ` Jacek Anaszewski
0 siblings, 0 replies; 3+ messages in thread
From: Jacek Anaszewski @ 2016-06-14 7:30 UTC (permalink / raw)
To: Phil Reid
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
riku.voipio, rpurdie, devicetree, linux-leds
Hi Phil
On 06/14/2016 03:56 AM, Phil Reid wrote:
> This patch adds basic device tree support for the pca9532 LEDs.
>
> Signed-off-by: Phil Reid <preid@electromag.com.au>
> ---
> .../devicetree/bindings/leds/leds-pca9532.txt | 39 +++++++++++
> drivers/leds/leds-pca9532.c | 76 ++++++++++++++++++++--
> include/dt-bindings/leds/leds-pca9532.h | 18 +++++
> include/linux/leds-pca9532.h | 9 ++-
> 4 files changed, 133 insertions(+), 9 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/leds/leds-pca9532.txt
> create mode 100644 include/dt-bindings/leds/leds-pca9532.h
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-pca9532.txt b/Documentation/devicetree/bindings/leds/leds-pca9532.txt
> new file mode 100644
> index 0000000..198f3ba
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/leds/leds-pca9532.txt
> @@ -0,0 +1,39 @@
> +*NXP - pca9532 PWM LED Driver
> +
> +The PCA9532 family is SMBus I/O expander optimized for dimming LEDs.
> +The PWM support 256 steps.
> +
> +Required properties:
> + - compatible:
> + "nxp,pca9530"
> + "nxp,pca9531"
> + "nxp,pca9532"
> + "nxp,pca9533"
> + - reg - I2C slave address
> +
> +Each led is represented as a sub-node of the nxp,pca9530.
> +
> +Optional sub-node properties:
> + - label: see Documentation/devicetree/bindings/leds/common.txt
> + - type: Output configuration, see dt-bindings/leds/leds-pca9532.h (default NONE)
> + - linux,default-trigger: see Documentation/devicetree/bindings/leds/common.txt
> +
> +Example:
> + #include <dt-bindings/leds/leds-pca9532.h>
> +
> + leds: pca9530@60 {
> + compatible = "nxp,pca9530";
> + reg = <0x60>;
> +
> + red-power {
> + label = "pca:red:power";
> + type = <PCA9532_TYPE_LED>;
> + };
> + green-power {
> + label = "pca:green:power";
> + type = <PCA9532_TYPE_LED>;
> + };
> + };
> +
> +For more product information please see the link below:
> +http://nxp.com/documents/data_sheet/PCA9532.pdf
> diff --git a/drivers/leds/leds-pca9532.c b/drivers/leds/leds-pca9532.c
> index e3d3b1a..49930a3 100644
> --- a/drivers/leds/leds-pca9532.c
> +++ b/drivers/leds/leds-pca9532.c
> @@ -21,6 +21,8 @@
> #include <linux/workqueue.h>
> #include <linux/leds-pca9532.h>
> #include <linux/gpio.h>
> +#include <linux/of.h>
> +#include <linux/of_device.h>
>
> /* m = num_leds*/
> #define PCA9532_REG_INPUT(i) ((i) >> 3)
> @@ -86,9 +88,22 @@ static const struct pca9532_chip_info pca9532_chip_info_tbl[] = {
> },
> };
>
> +#ifdef CONFIG_OF
> +static const struct of_device_id of_pca9532_leds_match[] = {
> + { .compatible = "nxp,pca9530", .data = (void *)pca9530 },
> + { .compatible = "nxp,pca9531", .data = (void *)pca9531 },
> + { .compatible = "nxp,pca9532", .data = (void *)pca9532 },
> + { .compatible = "nxp,pca9533", .data = (void *)pca9533 },
> + {},
> +};
> +
> +MODULE_DEVICE_TABLE(of, of_pca9532_leds_match);
> +#endif
> +
> static struct i2c_driver pca9532_driver = {
> .driver = {
> .name = "leds-pca953x",
> + .of_match_table = of_match_ptr(of_pca9532_leds_match),
> },
> .probe = pca9532_probe,
> .remove = pca9532_remove,
> @@ -354,6 +369,7 @@ static int pca9532_configure(struct i2c_client *client,
> led->state = pled->state;
> led->name = pled->name;
> led->ldev.name = led->name;
> + led->ldev.default_trigger = led->default_trigger;
> led->ldev.brightness = LED_OFF;
> led->ldev.brightness_set_blocking =
> pca9532_set_brightness;
> @@ -432,15 +448,67 @@ exit:
> return err;
> }
>
> +static struct pca9532_platform_data *
> +pca9532_of_populate_pdata(struct device *dev, struct device_node *np)
> +{
> + struct pca9532_platform_data *pdata;
> + struct device_node *child;
> + const struct of_device_id *match;
> + uintptr_t devid;
If devid is int the warning also disappears.
Please make it int and to be entirely consistent apply
double casting: (int)(uintptr_t).
> + int maxleds;
> + int i = 0;
> +
> + match = of_match_device(of_pca9532_leds_match, dev);
> + if (!match)
> + return ERR_PTR(-ENODEV);
> +
> + devid = (uintptr_t)match->data;
devid = (int)(uintptr_t)match->data
> + maxleds = pca9532_chip_info_tbl[devid].num_leds;
> +
> + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL);
> + if (!pdata)
> + return ERR_PTR(-ENOMEM);
> +
> + for_each_child_of_node(np, child) {
> + if (of_property_read_string(child, "label",
> + &pdata->leds[i].name))
> + pdata->leds[i].name = child->name;
> + of_property_read_u32(child, "type", &pdata->leds[i].type);
> + of_property_read_string(child, "linux,default-trigger",
> + &pdata->leds[i].default_trigger);
> + if (++i >= maxleds) {
> + of_node_put(child);
> + break;
> + }
> + }
> +
> + return pdata;
> +}
> +
> static int pca9532_probe(struct i2c_client *client,
> const struct i2c_device_id *id)
> {
> + uintptr_t devid;
int devid;
>
> struct pca9532_data *data = i2c_get_clientdata(client);
> struct pca9532_platform_data *pca9532_pdata =
> dev_get_platdata(&client->dev);
> -
> - if (!pca9532_pdata)
> - return -EIO;
> + struct device_node *np = client->dev.of_node;
> +
> + if (!pca9532_pdata) {
> + if (np) {
> + pca9532_pdata =
> + pca9532_of_populate_pdata(&client->dev, np);
> + if (IS_ERR(pca9532_pdata))
> + return PTR_ERR(pca9532_pdata);
> + } else {
> + dev_err(&client->dev, "no platform data\n");
> + return -EINVAL;
> + }
> + devid = (uintptr_t)of_match_device(
devid = (int)(uintptr_t)of_match_device(
> + of_pca9532_leds_match, &client->dev)->data;
> + } else {
> + devid = id->driver_data;
> + }
>
> if (!i2c_check_functionality(client->adapter,
> I2C_FUNC_SMBUS_BYTE_DATA))
> @@ -450,7 +518,7 @@ static int pca9532_probe(struct i2c_client *client,
> if (!data)
> return -ENOMEM;
>
> - data->chip_info = &pca9532_chip_info_tbl[id->driver_data];
> + data->chip_info = &pca9532_chip_info_tbl[devid];
>
> dev_info(&client->dev, "setting platform data\n");
> i2c_set_clientdata(client, data);
> diff --git a/include/dt-bindings/leds/leds-pca9532.h b/include/dt-bindings/leds/leds-pca9532.h
> new file mode 100644
> index 0000000..4d917aa
> --- /dev/null
> +++ b/include/dt-bindings/leds/leds-pca9532.h
> @@ -0,0 +1,18 @@
> +/*
> + * This header provides constants for pca9532 LED bindings.
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +#ifndef _DT_BINDINGS_LEDS_PCA9532_H
> +#define _DT_BINDINGS_LEDS_PCA9532_H
> +
> +#define PCA9532_TYPE_NONE 0
> +#define PCA9532_TYPE_LED 1
> +#define PCA9532_TYPE_N2100_BEEP 2
> +#define PCA9532_TYPE_GPIO 3
> +#define PCA9532_LED_TIMER2 4
> +
> +#endif /* _DT_BINDINGS_LEDS_PCA9532_H */
> diff --git a/include/linux/leds-pca9532.h b/include/linux/leds-pca9532.h
> index b8d6fff..d215b45 100644
> --- a/include/linux/leds-pca9532.h
> +++ b/include/linux/leds-pca9532.h
> @@ -16,6 +16,7 @@
>
> #include <linux/leds.h>
> #include <linux/workqueue.h>
> +#include <dt-bindings/leds/leds-pca9532.h>
>
> enum pca9532_state {
> PCA9532_OFF = 0x0,
> @@ -24,16 +25,14 @@ enum pca9532_state {
> PCA9532_PWM1 = 0x3
> };
>
> -enum pca9532_type { PCA9532_TYPE_NONE, PCA9532_TYPE_LED,
> - PCA9532_TYPE_N2100_BEEP, PCA9532_TYPE_GPIO };
> -
> struct pca9532_led {
> u8 id;
> struct i2c_client *client;
> - char *name;
> + const char *name;
> + const char *default_trigger;
> struct led_classdev ldev;
> struct work_struct work;
> - enum pca9532_type type;
> + u32 type;
> enum pca9532_state state;
> };
>
>
--
Best regards,
Jacek Anaszewski
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-06-14 7:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-14 1:56 [Patch v5 0/1] leds: pca9532: Add device tree binding Phil Reid
[not found] ` <1465869369-13449-1-git-send-email-preid-qgqNFa1JUf/o2iN0hyhwsIdd74u8MsAO@public.gmane.org>
2016-06-14 1:56 ` [Patch v5 1/1] " Phil Reid
2016-06-14 7:30 ` Jacek Anaszewski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).