leds: ncp5623: Add MS suffix to time defines

To make the time macro defines clearer, add MS as a suffix.

Signed-off-by: Abdel Alkuor <alkuor@gmail.com>
Link: https://lore.kernel.org/r/20240305133824.1551809-1-alkuor@gmail.com
Signed-off-by: Lee Jones <lee@kernel.org>

diff --git a/drivers/leds/rgb/leds-ncp5623.c b/drivers/leds/rgb/leds-ncp5623.c
index b669c55c5483a67caef699d8387380c971742513..2be4ff9185169c87931e7942cf064a0c8c3cd66e 100644 (file)
--- a/drivers/leds/rgb/leds-ncp5623.c
+++ b/drivers/leds/rgb/leds-ncp5623.c
@@ -22,8 +22,8 @@
 #define NCP5623_DIMMING_TIME_REG       NCP5623_REG(0x7)
 
 #define NCP5623_MAX_BRIGHTNESS         0x1f
-#define NCP5623_MAX_DIM_TIME           240 /* ms */
-#define NCP5623_DIM_STEP               8   /* ms */
+#define NCP5623_MAX_DIM_TIME_MS                240
+#define NCP5623_DIM_STEP_MS            8
 
 struct ncp5623 {
        struct i2c_client *client;
@@ -92,8 +92,8 @@ static int ncp5623_pattern_set(struct led_classdev *cdev,
 
        ncp->delay = 0;
 
-       if (pattern[0].delta_t > NCP5623_MAX_DIM_TIME ||
-          (pattern[0].delta_t % NCP5623_DIM_STEP) != 0)
+       if (pattern[0].delta_t > NCP5623_MAX_DIM_TIME_MS ||
+          (pattern[0].delta_t % NCP5623_DIM_STEP_MS) != 0)
                return -EINVAL;
 
        brightness_diff = pattern[0].brightness - ncp->current_brightness;
@@ -117,7 +117,7 @@ static int ncp5623_pattern_set(struct led_classdev *cdev,
 
        ret = ncp5623_write(ncp->client,
                            NCP5623_DIMMING_TIME_REG,
-                           pattern[0].delta_t / NCP5623_DIM_STEP);
+                           pattern[0].delta_t / NCP5623_DIM_STEP_MS);
        if (ret)
                return ret;