Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v11 3/5] dt-bindings: leds: Add LED_FUNCTION_MOONLIGHT
 definitions
To:     Pavel Machek <pavel@ucw.cz>,
        Gene Chen <gene.chen.richtek@gmail.com>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Dan Murphy <dmurphy@ti.com>,
        Linux LED Subsystem <linux-leds@vger.kernel.org>,
        devicetree <devicetree@vger.kernel.org>,
        linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>,
        "moderated list:ARM/Mediatek SoC support" 
        <linux-mediatek@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Gene Chen <gene_chen@richtek.com>, Wilma.Wu@mediatek.com,
        shufan_lee@richtek.com, cy_huang@richtek.com,
        benjamin.chao@mediatek.com
References: <1606906011-25633-1-git-send-email-gene.chen.richtek@gmail.com>
 <1606906011-25633-4-git-send-email-gene.chen.richtek@gmail.com>
 <20201202122329.GA30929@duo.ucw.cz>
 <CAE+NS34D8a6Udt7ZZ0=U7oqm5POtihKvD3WjD_sAADamqQ=1AQ@mail.gmail.com>
 <20201203114044.GA9061@duo.ucw.cz>
From:   Jacek Anaszewski <jacek.anaszewski@gmail.com>
Message-ID: <87a4f0b9-1450-cd9f-70ea-dd5b408893f3@gmail.com>
Date:   Thu, 3 Dec 2020 21:06:42 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.5.0
MIME-Version: 1.0
In-Reply-To: <20201203114044.GA9061@duo.ucw.cz>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Pavel,

On 12/3/20 12:40 PM, Pavel Machek wrote:
> Hi!
> 
>>>> +++ b/include/dt-bindings/leds/common.h
>>>> @@ -78,6 +78,7 @@
>>>>   #define LED_FUNCTION_INDICATOR "indicator"
>>>>   #define LED_FUNCTION_LAN "lan"
>>>>   #define LED_FUNCTION_MAIL "mail"
>>>> +#define LED_FUNCTION_MOONLIGHT "moonlight"
>>>
>>> There's "torch" function that should be used for this. I guess comment
>>> should be added with explanation what exactly that is and how should
>>> the LED be named.
>>>
>>
>> According to mail, 11/25 "Re: [PATCH v7 2/5] dt-bindings: leds: Add
>> LED_COLOR_ID_MOONLIGHT definitions",
>> The Moonlight LED is LED which maximum current more than torch, but
>> less than flash. Such as front camera fill light.
>> I think our channel is moonlight, not torch.
>> I will add this description to comment.
>> We can't exactly define moonlight current level, because every vendor
>> has their own specification.
> 
> So... what is the timelimit on moonlight?
> 
> But if it is used for camera illumination, I believe it should be
> simply called flash.

Let's keep FLASH reserved for LED flash class devices.
This device has already two other flash iouts.

Also iouts amperage gives clue that they have three different
functions.

-- 
Best regards,
Jacek Anaszewski