lkml.org 
[lkml]   [2022]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v9 09/10] leds: flash: mt6370: Add MediaTek MT6370 flashlight support
On Sun, Sep 18, 2022 at 3:22 AM Han Jingoo <jingoohan1@gmail.com> wrote:
>
> On Mon, Aug 29, 2022 ChiaEn Wu <peterwu.pub@gmail.com> wrote:

...

> > +#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:
>
> #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
>
>
> > +#define MT6370_STRBTO_MIN_US 64000
> > +#define MT6370_STRBTO_STEP_US 32000
> > +#define MT6370_STRBTO_MAX_US 2432000
> > +

Hi Jingoo,

This coding style is in accordance with Andy's opinion in this mail:
https://lore.kernel.org/linux-arm-kernel/CAHp75Vciq4M4kVrabNV9vTLLcd1vR=bMe8JLEdAF9MkRTpcB5g@mail.gmail.com/

And I will revise other parts in v12.
Thanks for your review!

--
Best Regards,
ChiaEn Wu

\
 
 \ /
  Last update: 2022-09-21 03:49    [W:0.114 / U:0.248 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site