* [PATCH v3 1/4] as3645a: Use ams,input-max-microamp as documented in DT bindings
2017-09-22 9:32 [PATCH v3 0/4] AS3645A fixes Sakari Ailus
@ 2017-09-22 9:32 ` Sakari Ailus
2017-09-22 9:32 ` [PATCH v3 2/4] dt: bindings: as3645a: Use LED number to refer to LEDs Sakari Ailus
` (3 subsequent siblings)
4 siblings, 0 replies; 11+ messages in thread
From: Sakari Ailus @ 2017-09-22 9:32 UTC (permalink / raw)
To: linux-leds, jacek.anaszewski; +Cc: linux-media, devicetree
DT bindings document the property "ams,input-max-microamp" that limits the
chip's maximum input current. The driver and the DTS however used
"peak-current-limit" property. Fix this by using the property documented
in DT binding documentation.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
Acked-by: Jacek Anaszewski <jacek.anaszewski@gmail.com>
---
arch/arm/boot/dts/omap3-n950-n9.dtsi | 2 +-
drivers/leds/leds-as3645a.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/omap3-n950-n9.dtsi b/arch/arm/boot/dts/omap3-n950-n9.dtsi
index cb47ae79a5f9..b86fc83a5a65 100644
--- a/arch/arm/boot/dts/omap3-n950-n9.dtsi
+++ b/arch/arm/boot/dts/omap3-n950-n9.dtsi
@@ -273,7 +273,7 @@
flash-timeout-us = <150000>;
flash-max-microamp = <320000>;
led-max-microamp = <60000>;
- peak-current-limit = <1750000>;
+ ams,input-max-microamp = <1750000>;
};
indicator {
led-max-microamp = <10000>;
diff --git a/drivers/leds/leds-as3645a.c b/drivers/leds/leds-as3645a.c
index bbbbe0898233..e3f89c6130d2 100644
--- a/drivers/leds/leds-as3645a.c
+++ b/drivers/leds/leds-as3645a.c
@@ -534,7 +534,7 @@ static int as3645a_parse_node(struct as3645a *flash,
of_property_read_u32(flash->flash_node, "voltage-reference",
&cfg->voltage_reference);
- of_property_read_u32(flash->flash_node, "peak-current-limit",
+ of_property_read_u32(flash->flash_node, "ams,input-max-microamp",
&cfg->peak);
cfg->peak = AS_PEAK_mA_TO_REG(cfg->peak);
--
2.11.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v3 2/4] dt: bindings: as3645a: Use LED number to refer to LEDs
2017-09-22 9:32 [PATCH v3 0/4] AS3645A fixes Sakari Ailus
2017-09-22 9:32 ` [PATCH v3 1/4] as3645a: Use ams,input-max-microamp as documented in DT bindings Sakari Ailus
@ 2017-09-22 9:32 ` Sakari Ailus
2017-09-22 9:32 ` [PATCH v3 3/4] as3645a: Use integer numbers for parsing LEDs Sakari Ailus
` (2 subsequent siblings)
4 siblings, 0 replies; 11+ messages in thread
From: Sakari Ailus @ 2017-09-22 9:32 UTC (permalink / raw)
To: linux-leds, jacek.anaszewski; +Cc: linux-media, devicetree
Use integers (reg property) to tell the number of the LED to the driver
instead of the node name. While both of these approaches are currently
used by the LED bindings, using integers will require less driver changes
for ACPI support. Additionally, it will make possible LED naming using
chip and LED node names, effectively making the label property most useful
for human-readable names only.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Jacek Anaszewski <jacek.anaszewski@gmail.com>
Acked-by: Rob Herring <robh@kernel.org>
---
.../devicetree/bindings/leds/ams,as3645a.txt | 28 ++++++++++++++--------
1 file changed, 18 insertions(+), 10 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/ams,as3645a.txt b/Documentation/devicetree/bindings/leds/ams,as3645a.txt
index 12c5ef26ec73..fdc40e354a64 100644
--- a/Documentation/devicetree/bindings/leds/ams,as3645a.txt
+++ b/Documentation/devicetree/bindings/leds/ams,as3645a.txt
@@ -15,11 +15,14 @@ Required properties
compatible : Must be "ams,as3645a".
reg : The I2C address of the device. Typically 0x30.
+#address-cells : 1
+#size-cells : 0
-Required properties of the "flash" child node
-=============================================
+Required properties of the flash child node (0)
+===============================================
+reg: 0
flash-timeout-us: Flash timeout in microseconds. The value must be in
the range [100000, 850000] and divisible by 50000.
flash-max-microamp: Maximum flash current in microamperes. Has to be
@@ -33,20 +36,21 @@ ams,input-max-microamp: Maximum flash controller input current. The
and divisible by 50000.
-Optional properties of the "flash" child node
-=============================================
+Optional properties of the flash child node
+===========================================
label : The label of the flash LED.
-Required properties of the "indicator" child node
-=================================================
+Required properties of the indicator child node (1)
+===================================================
+reg: 1
led-max-microamp: Maximum indicator current. The allowed values are
2500, 5000, 7500 and 10000.
-Optional properties of the "indicator" child node
-=================================================
+Optional properties of the indicator child node
+===============================================
label : The label of the indicator LED.
@@ -55,16 +59,20 @@ Example
=======
as3645a@30 {
+ #address-cells: 1
+ #size-cells: 0
reg = <0x30>;
compatible = "ams,as3645a";
- flash {
+ flash@0 {
+ reg = <0x0>;
flash-timeout-us = <150000>;
flash-max-microamp = <320000>;
led-max-microamp = <60000>;
ams,input-max-microamp = <1750000>;
label = "as3645a:flash";
};
- indicator {
+ indicator@1 {
+ reg = <0x1>;
led-max-microamp = <10000>;
label = "as3645a:indicator";
};
--
2.11.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v3 3/4] as3645a: Use integer numbers for parsing LEDs
2017-09-22 9:32 [PATCH v3 0/4] AS3645A fixes Sakari Ailus
2017-09-22 9:32 ` [PATCH v3 1/4] as3645a: Use ams,input-max-microamp as documented in DT bindings Sakari Ailus
2017-09-22 9:32 ` [PATCH v3 2/4] dt: bindings: as3645a: Use LED number to refer to LEDs Sakari Ailus
@ 2017-09-22 9:32 ` Sakari Ailus
2017-09-22 9:32 ` [PATCH v3 4/4] as3645a: Unregister indicator LED on device unbind Sakari Ailus
2017-09-22 19:37 ` [PATCH v3 0/4] AS3645A fixes Jacek Anaszewski
4 siblings, 0 replies; 11+ messages in thread
From: Sakari Ailus @ 2017-09-22 9:32 UTC (permalink / raw)
To: linux-leds, jacek.anaszewski; +Cc: linux-media, devicetree
Use integer numbers for LEDs, 0 is the flash and 1 is the indicator.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
Acked-by: Jacek Anaszewski <jacek.anaszewski@gmail.com>
---
arch/arm/boot/dts/omap3-n950-n9.dtsi | 8 ++++++--
drivers/leds/leds-as3645a.c | 26 ++++++++++++++++++++++++--
2 files changed, 30 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/omap3-n950-n9.dtsi b/arch/arm/boot/dts/omap3-n950-n9.dtsi
index b86fc83a5a65..1b0bd72945f2 100644
--- a/arch/arm/boot/dts/omap3-n950-n9.dtsi
+++ b/arch/arm/boot/dts/omap3-n950-n9.dtsi
@@ -267,15 +267,19 @@
clock-frequency = <400000>;
as3645a@30 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <0x30>;
compatible = "ams,as3645a";
- flash {
+ flash@0 {
+ reg = <0x0>;
flash-timeout-us = <150000>;
flash-max-microamp = <320000>;
led-max-microamp = <60000>;
ams,input-max-microamp = <1750000>;
};
- indicator {
+ indicator@1 {
+ reg = <0x1>;
led-max-microamp = <10000>;
};
};
diff --git a/drivers/leds/leds-as3645a.c b/drivers/leds/leds-as3645a.c
index e3f89c6130d2..605e0c64e974 100644
--- a/drivers/leds/leds-as3645a.c
+++ b/drivers/leds/leds-as3645a.c
@@ -112,6 +112,10 @@
#define AS_PEAK_mA_TO_REG(a) \
((min_t(u32, AS_PEAK_mA_MAX, a) - 1250) / 250)
+/* LED numbers for Devicetree */
+#define AS_LED_FLASH 0
+#define AS_LED_INDICATOR 1
+
enum as_mode {
AS_MODE_EXT_TORCH = 0 << AS_CONTROL_MODE_SETTING_SHIFT,
AS_MODE_INDICATOR = 1 << AS_CONTROL_MODE_SETTING_SHIFT,
@@ -491,10 +495,29 @@ static int as3645a_parse_node(struct as3645a *flash,
struct device_node *node)
{
struct as3645a_config *cfg = &flash->cfg;
+ struct device_node *child;
const char *name;
int rval;
- flash->flash_node = of_get_child_by_name(node, "flash");
+ for_each_child_of_node(node, child) {
+ u32 id = 0;
+
+ of_property_read_u32(child, "reg", &id);
+
+ switch (id) {
+ case AS_LED_FLASH:
+ flash->flash_node = of_node_get(child);
+ break;
+ case AS_LED_INDICATOR:
+ flash->indicator_node = of_node_get(child);
+ break;
+ default:
+ dev_warn(&flash->client->dev,
+ "unknown LED %u encountered, ignoring\n", id);
+ break;
+ }
+ }
+
if (!flash->flash_node) {
dev_err(&flash->client->dev, "can't find flash node\n");
return -ENODEV;
@@ -538,7 +561,6 @@ static int as3645a_parse_node(struct as3645a *flash,
&cfg->peak);
cfg->peak = AS_PEAK_mA_TO_REG(cfg->peak);
- flash->indicator_node = of_get_child_by_name(node, "indicator");
if (!flash->indicator_node) {
dev_warn(&flash->client->dev,
"can't find indicator node\n");
--
2.11.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v3 4/4] as3645a: Unregister indicator LED on device unbind
2017-09-22 9:32 [PATCH v3 0/4] AS3645A fixes Sakari Ailus
` (2 preceding siblings ...)
2017-09-22 9:32 ` [PATCH v3 3/4] as3645a: Use integer numbers for parsing LEDs Sakari Ailus
@ 2017-09-22 9:32 ` Sakari Ailus
2017-09-22 19:37 ` [PATCH v3 0/4] AS3645A fixes Jacek Anaszewski
4 siblings, 0 replies; 11+ messages in thread
From: Sakari Ailus @ 2017-09-22 9:32 UTC (permalink / raw)
To: linux-leds, jacek.anaszewski; +Cc: linux-media, devicetree
The indicator LED was registered in probe but was not removed in driver
remove callback. Fix this.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
drivers/leds/leds-as3645a.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/leds/leds-as3645a.c b/drivers/leds/leds-as3645a.c
index 605e0c64e974..9a257f969300 100644
--- a/drivers/leds/leds-as3645a.c
+++ b/drivers/leds/leds-as3645a.c
@@ -743,6 +743,7 @@ static int as3645a_remove(struct i2c_client *client)
as3645a_set_control(flash, AS_MODE_EXT_TORCH, false);
v4l2_flash_release(flash->vf);
+ v4l2_flash_release(flash->vfind);
led_classdev_flash_unregister(&flash->fled);
led_classdev_unregister(&flash->iled_cdev);
--
2.11.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v3 0/4] AS3645A fixes
2017-09-22 9:32 [PATCH v3 0/4] AS3645A fixes Sakari Ailus
` (3 preceding siblings ...)
2017-09-22 9:32 ` [PATCH v3 4/4] as3645a: Unregister indicator LED on device unbind Sakari Ailus
@ 2017-09-22 19:37 ` Jacek Anaszewski
[not found] ` <650b7cb3-f7dd-5959-3147-df7284415521-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
4 siblings, 1 reply; 11+ messages in thread
From: Jacek Anaszewski @ 2017-09-22 19:37 UTC (permalink / raw)
To: Sakari Ailus, linux-leds; +Cc: linux-media, devicetree
Hi Sakari,
On 09/22/2017 11:32 AM, Sakari Ailus wrote:
> Hi Jacek and others,
>
> Here are a few fixes for the as3645a DTS as well as changes in bindings.
> The driver is not in a release yet.
>
> Jacek: Could you take these to your fixes branch so we don't get faulty DT
> bindings to a release? I've dropped the patches related to LED naming and
> label property as the discusion appears to continue on that.
No problem. One question - isn't patch 3/4 missing?
Best regards,
Jacek Anaszewski
> Thanks.
>
>
> since v2:
>
> - Drop patches related to LED naming.
>
> - No other changes.
>
> since v1:
>
> - Add LED colour to the name of the LED, this adds two patches to the set.
>
> - Add a patch to unregister the indicator LED in driver remove function.
>
> - No changes to v1 patches.
>
> Sakari Ailus (4):
> as3645a: Use ams,input-max-microamp as documented in DT bindings
> dt: bindings: as3645a: Use LED number to refer to LEDs
> as3645a: Use integer numbers for parsing LEDs
> as3645a: Unregister indicator LED on device unbind
>
> .../devicetree/bindings/leds/ams,as3645a.txt | 28 +++++++++++++--------
> arch/arm/boot/dts/omap3-n950-n9.dtsi | 10 +++++---
> drivers/leds/leds-as3645a.c | 29 +++++++++++++++++++---
> 3 files changed, 51 insertions(+), 16 deletions(-)
>
^ permalink raw reply [flat|nested] 11+ messages in thread