From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id CD97EC6FA99 for ; Tue, 7 Mar 2023 04:08:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229559AbjCGEIf (ORCPT ); Mon, 6 Mar 2023 23:08:35 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54016 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229525AbjCGEIe (ORCPT ); Mon, 6 Mar 2023 23:08:34 -0500 Received: from mg.richtek.com (mg.richtek.com [220.130.44.152]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id C87E22821E; Mon, 6 Mar 2023 20:08:31 -0800 (PST) X-MailGates: (flag:4,DYNAMIC,BADHELO,RELAY,NOHOST:PASS)(compute_score:DE LIVER,40,3) Received: from 192.168.10.46 by mg.richtek.com with MailGates ESMTP Server V5.0(22240:0:AUTH_RELAY) (envelope-from ); Tue, 07 Mar 2023 12:08:05 +0800 (CST) Received: from ex3.rt.l (192.168.10.46) by ex3.rt.l (192.168.10.46) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.25; Tue, 7 Mar 2023 12:08:04 +0800 Received: from linuxcarl2.richtek.com (192.168.10.154) by ex3.rt.l (192.168.10.45) with Microsoft SMTP Server id 15.2.1118.25 via Frontend Transport; Tue, 7 Mar 2023 12:08:04 +0800 Date: Tue, 7 Mar 2023 12:08:04 +0800 From: ChiYuan Huang To: Lee Jones CC: ChiaEn Wu , , , , , , , , , , , , , Subject: Re: [PATCH v17 RESEND 3/3] docs: leds: Add MT6370 RGB LED pattern document Message-ID: <20230307040804.GA15766@linuxcarl2.richtek.com> References: <20230305101824.GE2574592@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230305101824.GE2574592@google.com> User-Agent: Mutt/1.5.21 (2010-09-15) Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org Hi, Lee: Some question about the comment. On Sun, Mar 05, 2023 at 10:18:24AM +0000, Lee Jones wrote: > On Thu, 23 Feb 2023, ChiaEn Wu wrote: > > > From: ChiYuan Huang > > > > Document the MT6370 RGB LED pattern trigger. > > > > This simply describe how the pattern works, each timing period, and the > > pattern diagram for MT6370 RGB LED. > > > > Signed-off-by: ChiYuan Huang > > Signed-off-by: ChiaEn Wu > > --- > > Documentation/leds/leds-mt6370-rgb.rst | 64 ++++++++++++++++++++++++++++++++++ > > 1 file changed, 64 insertions(+) > > create mode 100644 Documentation/leds/leds-mt6370-rgb.rst > > > > diff --git a/Documentation/leds/leds-mt6370-rgb.rst b/Documentation/leds/leds-mt6370-rgb.rst > > new file mode 100644 > > index 00000000..d1b2e4f > > --- /dev/null > > +++ b/Documentation/leds/leds-mt6370-rgb.rst > > @@ -0,0 +1,64 @@ > > +.. SPDX-License-Identifier: GPL-2.0 > > + > > +========================================= > > +Kernel driver for Mediatek MT6370 RGB LED > > Are you describing the device or the kernel driver? > But referring to others, it seems each one also write this as 'Kernel driver'. Sorry, I cannot catch the point. Did you mean to modify this description as 'The device for MT6370 RGB LED'? > > +========================================= > > + > > +Description > > +----------- > > + > > +The MT6370 integrates a four-channel RGB LED driver, designed to provide a > > +variety of lighting effect for mobile device applications. The RGB LED driver > > "devices" > > > +includes a smart LED string controller and it can drive 3 channels of LEDs with > > +a sink current up to 24mA and a CHG_VIN power good indicator LED with sink > > +current up to 6mA. It provides three operation modes for RGB LEDs: > > +PWM Dimming mode, breath pattern mode, and constant current mode. The device > > +can increase or decrease the brightness of the RGB LED via I2C interface. > > "an I2C interface" > > > +The breath pattern for a channel can be programmed using the "pattern" trigger, > > +using the hw_pattern attribute. > > + > > +/sys/class/leds//hw_pattern > > +-------------------------------- > > + > > +Specify a hardware breath pattern for a MT6370 RGB LED. > > + > > +The breath pattern is a series of timing pairs, with the hold-time expressed in > > +milliseconds. And the brightness is controlled by > > +'/sys/class/leds//brightness'. The pattern doesn't include the brightness > > +setting. Hardware pattern only controls the timing for each pattern stage > > +depending on the current brightness setting. > > + > > +Pattern diagram:: > > + > > + "0 Tr1 0 Tr2 0 Tf1 0 Tf2 0 Ton 0 Toff" --> '0' for dummy brightness code > > + > > + ^ > > + | ============ > > + | / \ / > > +Icurr | / \ / > > + | / \ / > > + | / \ / .....repeat > > + | / \ / > > + | --- --- --- > > + |--- --- --- > > + +----------------------------------============------------> Time > > + < Tr1>< Ton >< Toff >< Tr1> > > + > > +Timing description:: > > + > > +Tr1: First rising time for duty 0 to 30%. > > +Tr2: Second rising time for duty 31% to 100%. > > +Ton: On time for duty 100%. > > +Tf1: First falling time for duty 100% to 31%. > > +Tf2: Second falling time for duty 30% to 0%. > > +Toff: Off time for duty 0%. > > + > > +Tr1/Tr2/Tf1/Tf2/Ton: 125ms to 3125ms, 200ms per step. > > +Toff: 250ms to 6250ms, 400ms per step. > > + > > +Pattern example:: > > + > > + "0 125 0 125 0 125 0 125 0 625 0 1050" > > + > > +This Will configure Tr1/Tr2/Tf1/Tf2 to 125m, Ton to 625ms, and Toff to 1050ms. > > -- > > 2.7.4 > > > > -- > Lee Jones [李琼斯] From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C5BAEC678DB for ; Tue, 7 Mar 2023 04:09:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=6G/JC8nSqaWxsq97EDiEk+WvmnjjfzGS+veRzF8dDgc=; b=PWtZXhLcxQc6U9 k/As0Stbb0dkxfgAL059pzJ5VYXj7sSW4TVRxHHYBDazZGWD8+dGOrjNgNKGT/PWs5Il80X40rurH V/MiPg+ZMFe1jiu6YsAkQldDpXPiPWKi584D7E5/Bq83xSqx5+XGYCdQFySjODGaJ4Y6JClZXQvnJ klTuFXVaQzuinaQYBUhCbexUimRdnAuo91kBMUy7qqVC0qxvMir39mClggYpMcHVtNNi+AFyoVQ99 va49cqsNoB7Lo6UKSL2sVOpFYIOJDinuJaiHviivPn8MYNCp61wwPiO8nw8WwuA7j40yaD1m0avVL gtI/vn4h79VdiC5LmMTw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pZOd3-00G6v9-B0; Tue, 07 Mar 2023 04:08:45 +0000 Received: from mg.richtek.com ([220.130.44.152]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pZOcx-00G6tP-Vr; Tue, 07 Mar 2023 04:08:42 +0000 X-MailGates: (flag:4,DYNAMIC,BADHELO,RELAY,NOHOST:PASS)(compute_score:DE LIVER,40,3) Received: from 192.168.10.46 by mg.richtek.com with MailGates ESMTP Server V5.0(22240:0:AUTH_RELAY) (envelope-from ); Tue, 07 Mar 2023 12:08:05 +0800 (CST) Received: from ex3.rt.l (192.168.10.46) by ex3.rt.l (192.168.10.46) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.25; Tue, 7 Mar 2023 12:08:04 +0800 Received: from linuxcarl2.richtek.com (192.168.10.154) by ex3.rt.l (192.168.10.45) with Microsoft SMTP Server id 15.2.1118.25 via Frontend Transport; Tue, 7 Mar 2023 12:08:04 +0800 Date: Tue, 7 Mar 2023 12:08:04 +0800 From: ChiYuan Huang To: Lee Jones CC: ChiaEn Wu , , , , , , , , , , , , , Subject: Re: [PATCH v17 RESEND 3/3] docs: leds: Add MT6370 RGB LED pattern document Message-ID: <20230307040804.GA15766@linuxcarl2.richtek.com> References: <20230305101824.GE2574592@google.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230305101824.GE2574592@google.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230306_200840_475288_64ACA8D4 X-CRM114-Status: GOOD ( 25.73 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGksIExlZToKCiAgICBTb21lIHF1ZXN0aW9uIGFib3V0IHRoZSBjb21tZW50LgpPbiBTdW4sIE1h ciAwNSwgMjAyMyBhdCAxMDoxODoyNEFNICswMDAwLCBMZWUgSm9uZXMgd3JvdGU6Cj4gT24gVGh1 LCAyMyBGZWIgMjAyMywgQ2hpYUVuIFd1IHdyb3RlOgo+IAo+ID4gRnJvbTogQ2hpWXVhbiBIdWFu ZyA8Y3lfaHVhbmdAcmljaHRlay5jb20+Cj4gPiAKPiA+IERvY3VtZW50IHRoZSBNVDYzNzAgUkdC IExFRCBwYXR0ZXJuIHRyaWdnZXIuCj4gPiAKPiA+IFRoaXMgc2ltcGx5IGRlc2NyaWJlIGhvdyB0 aGUgcGF0dGVybiB3b3JrcywgZWFjaCB0aW1pbmcgcGVyaW9kLCBhbmQgdGhlCj4gPiBwYXR0ZXJu IGRpYWdyYW0gZm9yIE1UNjM3MCBSR0IgTEVELgo+ID4gCj4gPiBTaWduZWQtb2ZmLWJ5OiBDaGlZ dWFuIEh1YW5nIDxjeV9odWFuZ0ByaWNodGVrLmNvbT4KPiA+IFNpZ25lZC1vZmYtYnk6IENoaWFF biBXdSA8Y2hpYWVuX3d1QHJpY2h0ZWsuY29tPgo+ID4gLS0tCj4gPiAgRG9jdW1lbnRhdGlvbi9s ZWRzL2xlZHMtbXQ2MzcwLXJnYi5yc3QgfCA2NCArKysrKysrKysrKysrKysrKysrKysrKysrKysr KysrKysrCj4gPiAgMSBmaWxlIGNoYW5nZWQsIDY0IGluc2VydGlvbnMoKykKPiA+ICBjcmVhdGUg bW9kZSAxMDA2NDQgRG9jdW1lbnRhdGlvbi9sZWRzL2xlZHMtbXQ2MzcwLXJnYi5yc3QKPiA+IAo+ ID4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vbGVkcy9sZWRzLW10NjM3MC1yZ2IucnN0IGIv RG9jdW1lbnRhdGlvbi9sZWRzL2xlZHMtbXQ2MzcwLXJnYi5yc3QKPiA+IG5ldyBmaWxlIG1vZGUg MTAwNjQ0Cj4gPiBpbmRleCAwMDAwMDAwMC4uZDFiMmU0Zgo+ID4gLS0tIC9kZXYvbnVsbAo+ID4g KysrIGIvRG9jdW1lbnRhdGlvbi9sZWRzL2xlZHMtbXQ2MzcwLXJnYi5yc3QKPiA+IEBAIC0wLDAg KzEsNjQgQEAKPiA+ICsuLiBTUERYLUxpY2Vuc2UtSWRlbnRpZmllcjogR1BMLTIuMAo+ID4gKwo+ ID4gKz09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Cj4gPiArS2VybmVs IGRyaXZlciBmb3IgTWVkaWF0ZWsgTVQ2MzcwIFJHQiBMRUQKPiAKPiBBcmUgeW91IGRlc2NyaWJp bmcgdGhlIGRldmljZSBvciB0aGUga2VybmVsIGRyaXZlcj8KPiAKQnV0IHJlZmVycmluZyB0byBv dGhlcnMsIGl0IHNlZW1zIGVhY2ggb25lIGFsc28gd3JpdGUgdGhpcyBhcyAnS2VybmVsIGRyaXZl cicuClNvcnJ5LCBJIGNhbm5vdCBjYXRjaCB0aGUgcG9pbnQuIERpZCB5b3UgbWVhbiB0byBtb2Rp ZnkgdGhpcyBkZXNjcmlwdGlvbiBhcwonVGhlIGRldmljZSBmb3IgTVQ2MzcwIFJHQiBMRUQnPwo+ ID4gKz09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Cj4gPiArCj4gPiAr RGVzY3JpcHRpb24KPiA+ICstLS0tLS0tLS0tLQo+ID4gKwo+ID4gK1RoZSBNVDYzNzAgaW50ZWdy YXRlcyBhIGZvdXItY2hhbm5lbCBSR0IgTEVEIGRyaXZlciwgZGVzaWduZWQgdG8gcHJvdmlkZSBh Cj4gPiArdmFyaWV0eSBvZiBsaWdodGluZyBlZmZlY3QgZm9yIG1vYmlsZSBkZXZpY2UgYXBwbGlj YXRpb25zLiBUaGUgUkdCIExFRCBkcml2ZXIKPiAKPiAiZGV2aWNlcyIKPiAKPiA+ICtpbmNsdWRl cyBhIHNtYXJ0IExFRCBzdHJpbmcgY29udHJvbGxlciBhbmQgaXQgY2FuIGRyaXZlIDMgY2hhbm5l bHMgb2YgTEVEcyB3aXRoCj4gPiArYSBzaW5rIGN1cnJlbnQgdXAgdG8gMjRtQSBhbmQgYSBDSEdf VklOIHBvd2VyIGdvb2QgaW5kaWNhdG9yIExFRCB3aXRoIHNpbmsKPiA+ICtjdXJyZW50IHVwIHRv IDZtQS4gSXQgcHJvdmlkZXMgdGhyZWUgb3BlcmF0aW9uIG1vZGVzIGZvciBSR0IgTEVEczoKPiA+ ICtQV00gRGltbWluZyBtb2RlLCBicmVhdGggcGF0dGVybiBtb2RlLCBhbmQgY29uc3RhbnQgY3Vy cmVudCBtb2RlLiBUaGUgZGV2aWNlCj4gPiArY2FuIGluY3JlYXNlIG9yIGRlY3JlYXNlIHRoZSBi cmlnaHRuZXNzIG9mIHRoZSBSR0IgTEVEIHZpYSBJMkMgaW50ZXJmYWNlLgo+IAo+ICJhbiBJMkMg aW50ZXJmYWNlIgo+IAo+ID4gK1RoZSBicmVhdGggcGF0dGVybiBmb3IgYSBjaGFubmVsIGNhbiBi ZSBwcm9ncmFtbWVkIHVzaW5nIHRoZSAicGF0dGVybiIgdHJpZ2dlciwKPiA+ICt1c2luZyB0aGUg aHdfcGF0dGVybiBhdHRyaWJ1dGUuCj4gPiArCj4gPiArL3N5cy9jbGFzcy9sZWRzLzxsZWQ+L2h3 X3BhdHRlcm4KPiA+ICstLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+ID4gKwo+ID4g K1NwZWNpZnkgYSBoYXJkd2FyZSBicmVhdGggcGF0dGVybiBmb3IgYSBNVDYzNzAgUkdCIExFRC4K PiA+ICsKPiA+ICtUaGUgYnJlYXRoIHBhdHRlcm4gaXMgYSBzZXJpZXMgb2YgdGltaW5nIHBhaXJz LCB3aXRoIHRoZSBob2xkLXRpbWUgZXhwcmVzc2VkIGluCj4gPiArbWlsbGlzZWNvbmRzLiBBbmQg dGhlIGJyaWdodG5lc3MgaXMgY29udHJvbGxlZCBieQo+ID4gKycvc3lzL2NsYXNzL2xlZHMvPGxl ZD4vYnJpZ2h0bmVzcycuIFRoZSBwYXR0ZXJuIGRvZXNuJ3QgaW5jbHVkZSB0aGUgYnJpZ2h0bmVz cwo+ID4gK3NldHRpbmcuIEhhcmR3YXJlIHBhdHRlcm4gb25seSBjb250cm9scyB0aGUgdGltaW5n IGZvciBlYWNoIHBhdHRlcm4gc3RhZ2UKPiA+ICtkZXBlbmRpbmcgb24gdGhlIGN1cnJlbnQgYnJp Z2h0bmVzcyBzZXR0aW5nLgo+ID4gKwo+ID4gK1BhdHRlcm4gZGlhZ3JhbTo6Cj4gPiArCj4gPiAr ICAgICAiMCBUcjEgMCBUcjIgMCBUZjEgMCBUZjIgMCBUb24gMCBUb2ZmIiAtLT4gJzAnIGZvciBk dW1teSBicmlnaHRuZXNzIGNvZGUKPiA+ICsKPiA+ICsgICAgICBeCj4gPiArICAgICAgfCAgICAg ICAgICAgPT09PT09PT09PT09Cj4gPiArICAgICAgfCAgICAgICAgICAvICAgICAgICAgICAgXCAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgLwo+ID4gK0ljdXJyIHwgICAgICAgICAvICAg ICAgICAgICAgICBcICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgLwo+ID4gKyAgICAgIHwg ICAgICAgIC8gICAgICAgICAgICAgICAgXCAgICAgICAgICAgICAgICAgICAgICAgICAgICAvCj4g PiArICAgICAgfCAgICAgICAvICAgICAgICAgICAgICAgICAgXCAgICAgICAgICAgICAgICAgICAg ICAgICAgLyAgIC4uLi4ucmVwZWF0Cj4gPiArICAgICAgfCAgICAgIC8gICAgICAgICAgICAgICAg ICAgIFwgICAgICAgICAgICAgICAgICAgICAgICAvCj4gPiArICAgICAgfCAgIC0tLSAgICAgICAg ICAgICAgICAgICAgICAtLS0gICAgICAgICAgICAgICAgICAtLS0KPiA+ICsgICAgICB8LS0tICAg ICAgICAgICAgICAgICAgICAgICAgICAgIC0tLSAgICAgICAgICAgIC0tLQo+ID4gKyAgICAgICst LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tPT09PT09PT09PT09LS0tLS0tLS0tLS0t PiBUaW1lCj4gPiArICAgICAgIDwgVHIxPjxUcjI+PCAgIFRvbiAgICA+PFRmMT48VGYyID48ICBU b2ZmICAgID48IFRyMT48VHIyPgo+ID4gKwo+ID4gK1RpbWluZyBkZXNjcmlwdGlvbjo6Cj4gPiAr Cj4gPiArVHIxOiAgICBGaXJzdCByaXNpbmcgdGltZSBmb3IgZHV0eSAwIHRvIDMwJS4KPiA+ICtU cjI6ICAgIFNlY29uZCByaXNpbmcgdGltZSBmb3IgZHV0eSAzMSUgdG8gMTAwJS4KPiA+ICtUb246 ICAgIE9uIHRpbWUgZm9yIGR1dHkgMTAwJS4KPiA+ICtUZjE6ICAgIEZpcnN0IGZhbGxpbmcgdGlt ZSBmb3IgZHV0eSAxMDAlIHRvIDMxJS4KPiA+ICtUZjI6ICAgIFNlY29uZCBmYWxsaW5nIHRpbWUg Zm9yIGR1dHkgMzAlIHRvIDAlLgo+ID4gK1RvZmY6ICAgT2ZmIHRpbWUgZm9yIGR1dHkgMCUuCj4g PiArCj4gPiArVHIxL1RyMi9UZjEvVGYyL1RvbjogMTI1bXMgdG8gMzEyNW1zLCAyMDBtcyBwZXIg c3RlcC4KPiA+ICtUb2ZmOiAyNTBtcyB0byA2MjUwbXMsIDQwMG1zIHBlciBzdGVwLgo+ID4gKwo+ ID4gK1BhdHRlcm4gZXhhbXBsZTo6Cj4gPiArCj4gPiArICAgICAgICIwIDEyNSAwIDEyNSAwIDEy NSAwIDEyNSAwIDYyNSAwIDEwNTAiCj4gPiArCj4gPiArVGhpcyBXaWxsIGNvbmZpZ3VyZSBUcjEv VHIyL1RmMS9UZjIgdG8gMTI1bSwgVG9uIHRvIDYyNW1zLCBhbmQgVG9mZiB0byAxMDUwbXMuCj4g PiAtLSAKPiA+IDIuNy40Cj4gPiAKPiAKPiAtLSAKPiBMZWUgSm9uZXMgW+adjueQvOaWr10KCl9f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1r ZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpo dHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJu ZWwK