* [PATCH v7 1/2] dt-bindings: leds: convert ktd2692 bindings to yaml
2022-05-04 19:34 [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Markuss Broks
@ 2022-05-04 19:34 ` Markuss Broks
2022-05-04 19:34 ` [PATCH v7 2/2] leds: ktd2692: Make aux-gpios optional Markuss Broks
2022-05-04 19:44 ` [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Andy Shevchenko
2 siblings, 0 replies; 7+ messages in thread
From: Markuss Broks @ 2022-05-04 19:34 UTC (permalink / raw)
To: pavel
Cc: Markuss Broks, Rob Herring, Rob Herring, Krzysztof Kozlowski,
Lad Prabhakar, Linus Walleij, Christophe JAILLET, linux-leds,
devicetree, linux-kernel
This patch converts the leds-ktd2692.txt bindings to modern yaml
style device-tree bindings.
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Markuss Broks <markuss.broks@gmail.com>
---
.../bindings/leds/kinetic,ktd2692.yaml | 87 +++++++++++++++++++
.../devicetree/bindings/leds/leds-ktd2692.txt | 50 -----------
2 files changed, 87 insertions(+), 50 deletions(-)
create mode 100644 Documentation/devicetree/bindings/leds/kinetic,ktd2692.yaml
delete mode 100644 Documentation/devicetree/bindings/leds/leds-ktd2692.txt
diff --git a/Documentation/devicetree/bindings/leds/kinetic,ktd2692.yaml b/Documentation/devicetree/bindings/leds/kinetic,ktd2692.yaml
new file mode 100644
index 000000000000..bac95a51afa1
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/kinetic,ktd2692.yaml
@@ -0,0 +1,87 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/leds/kinetic,ktd2692.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: KTD2692 Flash LED Driver from Kinetic Technologies
+
+maintainers:
+ - Markuss Broks <markuss.broks@gmail.com>
+
+description: |
+ KTD2692 is the ideal power solution for high-power flash LEDs.
+ It uses ExpressWire single-wire programming for maximum flexibility.
+
+ The ExpressWire interface through CTRL pin can control LED on/off and
+ enable/disable the IC, Movie(max 1/3 of Flash current) / Flash mode current,
+ Flash timeout, LVP(low voltage protection).
+
+ Also, When the AUX pin is pulled high while CTRL pin is high,
+ LED current will be ramped up to the flash-mode current level.
+
+properties:
+ compatible:
+ const: kinetic,ktd2692
+
+ ctrl-gpios:
+ maxItems: 1
+ description: Specifier of the GPIO connected to CTRL pin.
+
+ aux-gpios:
+ maxItems: 1
+ description: Specifier of the GPIO connected to CTRL pin.
+
+ vin-supply:
+ description: LED supply (2.7V to 5.5V).
+
+ led:
+ type: object
+ $ref: common.yaml#
+ description: Properties for the LED.
+ properties:
+ function: true
+ color: true
+ flash-max-timeout-us:
+ description: Flash LED maximum timeout.
+
+ led-max-microamp:
+ maximum: 300000
+ description: Minimum Threshold for Timer protection
+ is defined internally (Maximum 300mA).
+
+ flash-max-microamp:
+ maximum: 300000
+ description: Flash LED maximum current
+ Formula - I(uA) = 15000000 / Rset.
+
+ additionalProperties: false
+
+required:
+ - compatible
+ - ctrl-gpios
+ - led
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/leds/common.h>
+
+ ktd2692 {
+ compatible = "kinetic,ktd2692";
+ ctrl-gpios = <&gpc0 1 0>;
+ aux-gpios = <&gpc0 2 0>;
+ vin-supply = <&vbat>;
+
+ led {
+ function = LED_FUNCTION_FLASH;
+ color = <LED_COLOR_ID_WHITE>;
+ flash-max-timeout-us = <250000>;
+ flash-max-microamp = <150000>;
+ led-max-microamp = <25000>;
+ };
+ };
+
+...
diff --git a/Documentation/devicetree/bindings/leds/leds-ktd2692.txt b/Documentation/devicetree/bindings/leds/leds-ktd2692.txt
deleted file mode 100644
index 853737452580..000000000000
--- a/Documentation/devicetree/bindings/leds/leds-ktd2692.txt
+++ /dev/null
@@ -1,50 +0,0 @@
-* Kinetic Technologies - KTD2692 Flash LED Driver
-
-KTD2692 is the ideal power solution for high-power flash LEDs.
-It uses ExpressWire single-wire programming for maximum flexibility.
-
-The ExpressWire interface through CTRL pin can control LED on/off and
-enable/disable the IC, Movie(max 1/3 of Flash current) / Flash mode current,
-Flash timeout, LVP(low voltage protection).
-
-Also, When the AUX pin is pulled high while CTRL pin is high,
-LED current will be ramped up to the flash-mode current level.
-
-Required properties:
-- compatible : Should be "kinetic,ktd2692".
-- ctrl-gpios : Specifier of the GPIO connected to CTRL pin.
-- aux-gpios : Specifier of the GPIO connected to AUX pin.
-
-Optional properties:
-- vin-supply : "vin" LED supply (2.7V to 5.5V).
- See Documentation/devicetree/bindings/regulator/regulator.txt
-
-A discrete LED element connected to the device must be represented by a child
-node - See Documentation/devicetree/bindings/leds/common.txt
-
-Required properties for flash LED child nodes:
- See Documentation/devicetree/bindings/leds/common.txt
-- led-max-microamp : Minimum Threshold for Timer protection
- is defined internally (Maximum 300mA).
-- flash-max-microamp : Flash LED maximum current
- Formula : I(mA) = 15000 / Rset.
-- flash-max-timeout-us : Flash LED maximum timeout.
-
-Optional properties for flash LED child nodes:
-- label : See Documentation/devicetree/bindings/leds/common.txt
-
-Example:
-
-ktd2692 {
- compatible = "kinetic,ktd2692";
- ctrl-gpios = <&gpc0 1 0>;
- aux-gpios = <&gpc0 2 0>;
- vin-supply = <&vbat>;
-
- flash-led {
- label = "ktd2692-flash";
- led-max-microamp = <300000>;
- flash-max-microamp = <1500000>;
- flash-max-timeout-us = <1835000>;
- };
-};
--
2.35.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v7 2/2] leds: ktd2692: Make aux-gpios optional
2022-05-04 19:34 [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Markuss Broks
2022-05-04 19:34 ` [PATCH v7 1/2] dt-bindings: leds: convert ktd2692 bindings to yaml Markuss Broks
@ 2022-05-04 19:34 ` Markuss Broks
2022-05-04 19:44 ` [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Andy Shevchenko
2 siblings, 0 replies; 7+ messages in thread
From: Markuss Broks @ 2022-05-04 19:34 UTC (permalink / raw)
To: pavel
Cc: Markuss Broks, Andy Shevchenko, Rob Herring, Krzysztof Kozlowski,
Christophe JAILLET, Linus Walleij, Lad Prabhakar, linux-leds,
devicetree, linux-kernel
Make the AUX pin optional, since it isn't a core part of functionality,
and the device is designed to be operational with only one CTRL pin.
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Markuss Broks <markuss.broks@gmail.com>
---
drivers/leds/flash/leds-ktd2692.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/leds/flash/leds-ktd2692.c b/drivers/leds/flash/leds-ktd2692.c
index ed1f20a58bf6..1736b3f6b899 100644
--- a/drivers/leds/flash/leds-ktd2692.c
+++ b/drivers/leds/flash/leds-ktd2692.c
@@ -284,9 +284,9 @@ static int ktd2692_parse_dt(struct ktd2692_context *led, struct device *dev,
return ret;
}
- led->aux_gpio = devm_gpiod_get(dev, "aux", GPIOD_ASIS);
- ret = PTR_ERR_OR_ZERO(led->aux_gpio);
- if (ret) {
+ led->aux_gpio = devm_gpiod_get_optional(dev, "aux", GPIOD_ASIS);
+ if (IS_ERR(led->aux_gpio)) {
+ ret = PTR_ERR(led->aux_gpio);
dev_err(dev, "cannot get aux-gpios %d\n", ret);
return ret;
}
--
2.35.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692
2022-05-04 19:34 [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Markuss Broks
2022-05-04 19:34 ` [PATCH v7 1/2] dt-bindings: leds: convert ktd2692 bindings to yaml Markuss Broks
2022-05-04 19:34 ` [PATCH v7 2/2] leds: ktd2692: Make aux-gpios optional Markuss Broks
@ 2022-05-04 19:44 ` Andy Shevchenko
2022-05-04 19:50 ` Markuss Broks
2022-05-04 19:50 ` Pavel Machek
2 siblings, 2 replies; 7+ messages in thread
From: Andy Shevchenko @ 2022-05-04 19:44 UTC (permalink / raw)
To: Markuss Broks
Cc: Pavel Machek, Rob Herring, Krzysztof Kozlowski,
Christophe JAILLET, Linus Walleij, Lad Prabhakar,
Linux LED Subsystem, devicetree, Linux Kernel Mailing List
On Wed, May 4, 2022 at 9:35 PM Markuss Broks <markuss.broks@gmail.com> wrote:
>
> Some appliances of ktd2692 don't have the AUX pin connected to
> a GPIO. Specifically, Samsung Galaxy J5 (2015), which uses ktd2692
> for driving the front flash LED, has the pin not connected anywhere on
> schematics. Make specifying the AUX pin optional, since it is additional
> functionality and only affects amount of current going through the LED.
the amount
> Also convert the txt device-tree bindings to yaml and pick up maintenance
> over the yaml binding and the driver itself.
...
> v7:
> - drop the MAINTAINERS part
I'm not sure why it happened.
What I mentioned is to create a series out of 4 (four) patches:
1) fix the potential issue with the repetitive message;
2) update DT;
3) switch to optional GPIO;
4) update MAINTAINERS.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692
2022-05-04 19:44 ` [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Andy Shevchenko
@ 2022-05-04 19:50 ` Markuss Broks
2022-05-04 19:50 ` Pavel Machek
1 sibling, 0 replies; 7+ messages in thread
From: Markuss Broks @ 2022-05-04 19:50 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Pavel Machek, Rob Herring, Krzysztof Kozlowski,
Christophe JAILLET, Linus Walleij, Lad Prabhakar,
Linux LED Subsystem, devicetree, Linux Kernel Mailing List
Hi Andy,
On 5/4/22 22:44, Andy Shevchenko wrote:
> On Wed, May 4, 2022 at 9:35 PM Markuss Broks <markuss.broks@gmail.com> wrote:
>> Some appliances of ktd2692 don't have the AUX pin connected to
>> a GPIO. Specifically, Samsung Galaxy J5 (2015), which uses ktd2692
>> for driving the front flash LED, has the pin not connected anywhere on
>> schematics. Make specifying the AUX pin optional, since it is additional
>> functionality and only affects amount of current going through the LED.
> the amount
>
>> Also convert the txt device-tree bindings to yaml and pick up maintenance
>> over the yaml binding and the driver itself.
> ...
>
>> v7:
>> - drop the MAINTAINERS part
> I'm not sure why it happened.
Pavel asked me to drop the MAINTAINERS part for now.
>
> What I mentioned is to create a series out of 4 (four) patches:
> 1) fix the potential issue with the repetitive message;
Bummer, totally forgot about that, thank you for a reminder!
> 2) update DT;
> 3) switch to optional GPIO;
> 4) update MAINTAINERS.
>
>
- Markuss
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692
2022-05-04 19:44 ` [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692 Andy Shevchenko
2022-05-04 19:50 ` Markuss Broks
@ 2022-05-04 19:50 ` Pavel Machek
2022-05-04 19:55 ` Andy Shevchenko
1 sibling, 1 reply; 7+ messages in thread
From: Pavel Machek @ 2022-05-04 19:50 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Markuss Broks, Rob Herring, Krzysztof Kozlowski,
Christophe JAILLET, Linus Walleij, Lad Prabhakar,
Linux LED Subsystem, devicetree, Linux Kernel Mailing List
[-- Attachment #1: Type: text/plain, Size: 301 bytes --]
Hi!
> > v7:
> > - drop the MAINTAINERS part
>
> I'm not sure why it happened.
>
> 4) update MAINTAINERS.
I asked for that one.
If there's no other problem, I can take the series.
Best regards,
Pavel
--
People of Russia, stop Putin before his war on Ukraine escalates.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 0/2] Make AUX gpio pin optional for ktd2692
2022-05-04 19:50 ` Pavel Machek
@ 2022-05-04 19:55 ` Andy Shevchenko
0 siblings, 0 replies; 7+ messages in thread
From: Andy Shevchenko @ 2022-05-04 19:55 UTC (permalink / raw)
To: Pavel Machek
Cc: Markuss Broks, Rob Herring, Krzysztof Kozlowski,
Christophe JAILLET, Linus Walleij, Lad Prabhakar,
Linux LED Subsystem, devicetree, Linux Kernel Mailing List
On Wed, May 4, 2022 at 9:50 PM Pavel Machek <pavel@ucw.cz> wrote:
> > > v7:
> > > - drop the MAINTAINERS part
> >
> > I'm not sure why it happened.
> >
> > 4) update MAINTAINERS.
>
> I asked for that one.
>
> If there's no other problem, I can take the series.
Ah, thanks for clarification.
As I mentioned before, it would be really nice to have a fix-patch
prepending this series. That said, this series needs to be rebased on
it.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread