Thread (4 messages) 4 messages, 4 authors, 2022-09-23

Re: [PATCH v9 09/10] leds: flash: mt6370: Add MediaTek MT6370 flashlight support

From: Joe Perches <joe@perches.com>
Date: 2022-09-23 22:02:49
Also in: dri-devel, linux-arm-kernel, linux-devicetree, linux-iio, linux-leds, linux-mediatek, linux-pm, lkml

Possibly related (same subject, not in this thread)

On Fri, 2022-09-23 at 14:34 -0700, Han Jingoo wrote:
On Wed, Sep 21, 2022 Andy Shevchenko [off-list ref] wrote:
quoted
On Wed, Sep 21, 2022 at 4:48 AM ChiaEn Wu [off-list ref] wrote:
quoted
On Sun, Sep 18, 2022 at 3:22 AM Han Jingoo [off-list ref] wrote:
quoted
On Mon, Aug 29, 2022 ChiaEn Wu [off-list ref] wrote:
quoted
quoted
quoted
+#define MT6370_ITORCH_MIN_uA           25000
+#define MT6370_ITORCH_STEP_uA          12500
+#define MT6370_ITORCH_MAX_uA           400000
+#define MT6370_ITORCH_DOUBLE_MAX_uA    800000
+#define MT6370_ISTRB_MIN_uA            50000
+#define MT6370_ISTRB_STEP_uA           12500
+#define MT6370_ISTRB_MAX_uA            1500000
+#define MT6370_ISTRB_DOUBLE_MAX_uA     3000000
Use upper letters as below:
For microseconds (and other -seconds) the common practice (I assume
historically) is to use upper letters, indeed. But for current it's
more natural to use small letters for unit multiplier as it's easier
to read and understand.
I think it's fine. see:

commit 22735ce857a2d9f4e6eec37c36be3fcf9d21d154
Author: Joe Perches [off-list ref]
Date:   Wed Jul 3 15:05:33 2013 -0700

    checkpatch: ignore SI unit CamelCase variants like "_uV"
    
    Many existing variable names use SI like variants that should be otherwise
    obvious and acceptable.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help