2019-07-21 21:31:08

by Christophe JAILLET

[permalink] [raw]

Subject: [PATCH] leds: ktd2692: Fix a typo in the name of a constant

---

There is a typo in KTD2962_MM_MIN_CURR_THRESHOLD_SCALE. 6 and 9 are
switched in 2962.

Define and use KTD2692_MM_MIN_CURR_THRESHOLD_SCALE instead.

Signed-off-by: Christophe JAILLET <[email protected]>
---
drivers/leds/leds-ktd2692.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/leds/leds-ktd2692.c b/drivers/leds/leds-ktd2692.c
index 45296aaca9da..44db748c570e 100644
--- a/drivers/leds/leds-ktd2692.c
+++ b/drivers/leds/leds-ktd2692.c
@@ -22,7 +22,7 @@
/* Value related the movie mode */
#define KTD2692_MOVIE_MODE_CURRENT_LEVELS 16
#define KTD2692_MM_TO_FL_RATIO(x) ((x) / 3)
-#define KTD2962_MM_MIN_CURR_THRESHOLD_SCALE 8
+#define KTD2692_MM_MIN_CURR_THRESHOLD_SCALE 8

/* Value related the flash mode */
#define KTD2692_FLASH_MODE_TIMEOUT_LEVELS 8
@@ -253,7 +253,7 @@ static void ktd2692_setup(struct ktd2692_context *led)
ktd2692_expresswire_reset(led);
gpiod_direction_output(led->aux_gpio, KTD2692_LOW);

- ktd2692_expresswire_write(led, (KTD2962_MM_MIN_CURR_THRESHOLD_SCALE - 1)
+ ktd2692_expresswire_write(led, (KTD2692_MM_MIN_CURR_THRESHOLD_SCALE - 1)
| KTD2692_REG_MM_MIN_CURR_THRESHOLD_BASE);
ktd2692_expresswire_write(led, KTD2692_FLASH_MODE_CURR_PERCENT(45)
| KTD2692_REG_FLASH_CURRENT_BASE);
--
2.20.1

2019-07-21 22:56:23

by Pavel Machek

[permalink] [raw]

Subject: Re: [PATCH] leds: ktd2692: Fix a typo in the name of a constant

---

On Sun 2019-07-21 23:05:39, Christophe JAILLET wrote:
> There is a typo in KTD2962_MM_MIN_CURR_THRESHOLD_SCALE. 6 and 9 are
> switched in 2962.
>
> Define and use KTD2692_MM_MIN_CURR_THRESHOLD_SCALE instead.
>
> Signed-off-by: Christophe JAILLET <[email protected]>

Acked-by: Pavel Machek <[email protected]>
Pavel

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

---

Attachments:

(No filename) (485.00 B)
signature.asc (188.00 B)
Digital signature Download all attachments

2019-07-22 09:37:19

by Enrico Weigelt, metux IT consult

[permalink] [raw]

Subject: Re: [PATCH] leds: ktd2692: Fix a typo in the name of a constant

---

On 21.07.19 23:05, Christophe JAILLET wrote:
> There is a typo in KTD2962_MM_MIN_CURR_THRESHOLD_SCALE. 6 and 9 are
> switched in 2962.
>
> Define and use KTD2692_MM_MIN_CURR_THRESHOLD_SCALE instead.
>
> Signed-off-by: Christophe JAILLET <[email protected]>
> ---
> drivers/leds/leds-ktd2692.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/leds/leds-ktd2692.c b/drivers/leds/leds-ktd2692.c
> index 45296aaca9da..44db748c570e 100644
> --- a/drivers/leds/leds-ktd2692.c
> +++ b/drivers/leds/leds-ktd2692.c
> @@ -22,7 +22,7 @@
> /* Value related the movie mode */
> #define KTD2692_MOVIE_MODE_CURRENT_LEVELS 16
> #define KTD2692_MM_TO_FL_RATIO(x) ((x) / 3)
> -#define KTD2962_MM_MIN_CURR_THRESHOLD_SCALE 8
> +#define KTD2692_MM_MIN_CURR_THRESHOLD_SCALE 8
>
> /* Value related the flash mode */
> #define KTD2692_FLASH_MODE_TIMEOUT_LEVELS 8
> @@ -253,7 +253,7 @@ static void ktd2692_setup(struct ktd2692_context *led)
> ktd2692_expresswire_reset(led);
> gpiod_direction_output(led->aux_gpio, KTD2692_LOW);
>
> - ktd2692_expresswire_write(led, (KTD2962_MM_MIN_CURR_THRESHOLD_SCALE - 1)
> + ktd2692_expresswire_write(led, (KTD2692_MM_MIN_CURR_THRESHOLD_SCALE - 1)
> | KTD2692_REG_MM_MIN_CURR_THRESHOLD_BASE);
> ktd2692_expresswire_write(led, KTD2692_FLASH_MODE_CURR_PERCENT(45)
> | KTD2692_REG_FLASH_CURRENT_BASE);
>

Reviewed-By: Enrico Weigelt <[email protected]>

--
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
[email protected] -- +49-151-27565287

2019-07-23 03:10:42

by Jacek Anaszewski

[permalink] [raw]

Subject: Re: [PATCH] leds: ktd2692: Fix a typo in the name of a constant

---

Hi Christophe,

Thank you for the patch.

On 7/21/19 11:05 PM, Christophe JAILLET wrote:
> There is a typo in KTD2962_MM_MIN_CURR_THRESHOLD_SCALE. 6 and 9 are
> switched in 2962.
>
> Define and use KTD2692_MM_MIN_CURR_THRESHOLD_SCALE instead.
>
> Signed-off-by: Christophe JAILLET <[email protected]>
> ---
> drivers/leds/leds-ktd2692.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/leds/leds-ktd2692.c b/drivers/leds/leds-ktd2692.c
> index 45296aaca9da..44db748c570e 100644
> --- a/drivers/leds/leds-ktd2692.c
> +++ b/drivers/leds/leds-ktd2692.c
> @@ -22,7 +22,7 @@
> /* Value related the movie mode */
> #define KTD2692_MOVIE_MODE_CURRENT_LEVELS 16
> #define KTD2692_MM_TO_FL_RATIO(x) ((x) / 3)
> -#define KTD2962_MM_MIN_CURR_THRESHOLD_SCALE 8
> +#define KTD2692_MM_MIN_CURR_THRESHOLD_SCALE 8
>
> /* Value related the flash mode */
> #define KTD2692_FLASH_MODE_TIMEOUT_LEVELS 8
> @@ -253,7 +253,7 @@ static void ktd2692_setup(struct ktd2692_context *led)
> ktd2692_expresswire_reset(led);
> gpiod_direction_output(led->aux_gpio, KTD2692_LOW);
>
> - ktd2692_expresswire_write(led, (KTD2962_MM_MIN_CURR_THRESHOLD_SCALE - 1)
> + ktd2692_expresswire_write(led, (KTD2692_MM_MIN_CURR_THRESHOLD_SCALE - 1)
> | KTD2692_REG_MM_MIN_CURR_THRESHOLD_BASE);
> ktd2692_expresswire_write(led, KTD2692_FLASH_MODE_CURR_PERCENT(45)
> | KTD2692_REG_FLASH_CURRENT_BASE);
>

Applied.

--
Best regards,
Jacek Anaszewski