From: ChiaEn Wu <peterwu.pub@gmail.com> To: Lee Jones <lee.jones@linaro.org>, Daniel Thompson <daniel.thompson@linaro.org>, Jingoo Han <jingoohan1@gmail.com>, Pavel Machek <pavel@ucw.cz>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Matthias Brugger <matthias.bgg@gmail.com>, Sebastian Reichel <sre@kernel.org>, Chunfeng Yun <chunfeng.yun@mediatek.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Jonathan Cameron <jic23@kernel.org>, Lars-Peter Clausen <lars@metafoo.de>, Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>, Guenter Roeck <linux@roeck-us.net>, "Krogerus, Heikki" <heikki.krogerus@linux.intel.com>, Helge Deller <deller@gmx.de>, Andy Shevchenko <andy.shevchenko@gmail.com> Cc: ChiaEn Wu <chiaen_wu@richtek.com>, Alice Chen <alice_chen@richtek.com>, ChiYuan Huang <cy_huang@richtek.com>, dri-devel <dri-devel@lists.freedesktop.org>, 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>, Linux PM <linux-pm@vger.kernel.org>, USB <linux-usb@vger.kernel.org>, linux-iio <linux-iio@vger.kernel.org>, "open list:FRAMEBUFFER LAYER" <linux-fbdev@vger.kernel.org>, szuni chen <szunichen@gmail.com>, Rob Herring <robh@kernel.org> Subject: Re: [PATCH v6 05/13] dt-bindings: backlight: Add MediaTek MT6370 backlight Date: Mon, 1 Aug 2022 14:47:36 +0800 [thread overview] Message-ID: <CABtFH5L83d5Di6O9TC-L3UX2ma5J3PE47ihfJFfPD5YGJ43NxQ@mail.gmail.com> (raw) In-Reply-To: <20220722102407.2205-6-peterwu.pub@gmail.com> On Fri, Jul 22, 2022 at 6:24 PM ChiaEn Wu <peterwu.pub@gmail.com> wrote: > > Signed-off-by: ChiYuan Huang <cy_huang@richtek.com> > Reviewed-by: Rob Herring <robh@kernel.org> > --- [snip] > + compatible: > + const: mediatek,mt6370-backlight > + > + default-brightness: > + minimum: 0 > + maximum: 2048 > + > + max-brightness: > + minimum: 0 > + maximum: 2048 > + Hi Rob, Krzysztof, First, I'm so sorry for our SoB writing wrong, I'll fix it in the next patch. Because of this mail thread (https://lore.kernel.org/all/20220728113109.7gf3b36mqjxlhcq3@maple.lan/), I would like to discuss with you that I may change the following in this DT document, I am not sure whether you agree or not. 1. Add some descriptions about MT6372, which supports 16384 steps (14 bits) and is different from MT6370/MT6371 (2048 steps, 11 bits) 2. Modify the format of 'compatible' as follows to distinguish between MT6370/MT6371 or MT6372 ------------------------------------------ properties: compatible: enum: - mediatek,mt6370-backlight - mediatek,mt6372-backlight ------------------------------------------ 3. Remove the 'maximum' value of 'default-brightness' and 'max-brightness', and add "if-else" to determine the "maximum" value as follows ------------------------------------------ default-brightness: minimum: 0 max-brightness: minimum: 0 ... if: properties: compatible: contains: const: mediatek,mt6372-backlight then: properties: default-brightness: maximum: 16384 max-brightness: maximum: 16384 else: properties: default-brightness: maximum: 2048 max-brightness: maximum: 2048 ------------------------------------------ 4. Add a new boolean property to allow the user to determine whether to enable the exponential mode of backlight brightness or not. Like this, ------------------------------------------ mediatek,bled-exponential-mode-enable: description: | Enable the exponential mode of backlight brightness. If this property is not enabled, the default is to use linear mode. type: boolean ------------------------------------------ I hope these changes I have added will meet your expectations. And I'm not sure if I should remove 'Reviewed-By' first until it passes your review. Thanks. -- Best Regards, ChiaEn Wu
WARNING: multiple messages have this Message-ID (diff)
From: ChiaEn Wu <peterwu.pub@gmail.com> To: Lee Jones <lee.jones@linaro.org>, Daniel Thompson <daniel.thompson@linaro.org>, Jingoo Han <jingoohan1@gmail.com>, Pavel Machek <pavel@ucw.cz>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Matthias Brugger <matthias.bgg@gmail.com>, Sebastian Reichel <sre@kernel.org>, Chunfeng Yun <chunfeng.yun@mediatek.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Jonathan Cameron <jic23@kernel.org>, Lars-Peter Clausen <lars@metafoo.de>, Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>, Guenter Roeck <linux@roeck-us.net>, "Krogerus, Heikki" <heikki.krogerus@linux.intel.com>, Helge Deller <deller@gmx.de>, Andy Shevchenko <andy.shevchenko@gmail.com> Cc: ChiaEn Wu <chiaen_wu@richtek.com>, Alice Chen <alice_chen@richtek.com>, ChiYuan Huang <cy_huang@richtek.com>, dri-devel <dri-devel@lists.freedesktop.org>, 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>, Linux PM <linux-pm@vger.kernel.org>, USB <linux-usb@vger.kernel.org>, linux-iio <linux-iio@vger.kernel.org>, "open list:FRAMEBUFFER LAYER" <linux-fbdev@vger.kernel.org>, szuni chen <szunichen@gmail.com>, Rob Herring <robh@kernel.org> Subject: Re: [PATCH v6 05/13] dt-bindings: backlight: Add MediaTek MT6370 backlight Date: Mon, 1 Aug 2022 14:47:36 +0800 [thread overview] Message-ID: <CABtFH5L83d5Di6O9TC-L3UX2ma5J3PE47ihfJFfPD5YGJ43NxQ@mail.gmail.com> (raw) In-Reply-To: <20220722102407.2205-6-peterwu.pub@gmail.com> On Fri, Jul 22, 2022 at 6:24 PM ChiaEn Wu <peterwu.pub@gmail.com> wrote: > > Signed-off-by: ChiYuan Huang <cy_huang@richtek.com> > Reviewed-by: Rob Herring <robh@kernel.org> > --- [snip] > + compatible: > + const: mediatek,mt6370-backlight > + > + default-brightness: > + minimum: 0 > + maximum: 2048 > + > + max-brightness: > + minimum: 0 > + maximum: 2048 > + Hi Rob, Krzysztof, First, I'm so sorry for our SoB writing wrong, I'll fix it in the next patch. Because of this mail thread (https://lore.kernel.org/all/20220728113109.7gf3b36mqjxlhcq3@maple.lan/), I would like to discuss with you that I may change the following in this DT document, I am not sure whether you agree or not. 1. Add some descriptions about MT6372, which supports 16384 steps (14 bits) and is different from MT6370/MT6371 (2048 steps, 11 bits) 2. Modify the format of 'compatible' as follows to distinguish between MT6370/MT6371 or MT6372 ------------------------------------------ properties: compatible: enum: - mediatek,mt6370-backlight - mediatek,mt6372-backlight ------------------------------------------ 3. Remove the 'maximum' value of 'default-brightness' and 'max-brightness', and add "if-else" to determine the "maximum" value as follows ------------------------------------------ default-brightness: minimum: 0 max-brightness: minimum: 0 ... if: properties: compatible: contains: const: mediatek,mt6372-backlight then: properties: default-brightness: maximum: 16384 max-brightness: maximum: 16384 else: properties: default-brightness: maximum: 2048 max-brightness: maximum: 2048 ------------------------------------------ 4. Add a new boolean property to allow the user to determine whether to enable the exponential mode of backlight brightness or not. Like this, ------------------------------------------ mediatek,bled-exponential-mode-enable: description: | Enable the exponential mode of backlight brightness. If this property is not enabled, the default is to use linear mode. type: boolean ------------------------------------------ I hope these changes I have added will meet your expectations. And I'm not sure if I should remove 'Reviewed-By' first until it passes your review. Thanks. -- Best Regards, ChiaEn Wu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-08-01 6:48 UTC|newest] Thread overview: 154+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-07-22 10:23 [PATCH v6 00/13] Add MediaTek MT6370 PMIC support ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` [PATCH v6 01/13] dt-bindings: usb: Add MediaTek MT6370 TCPC ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` [PATCH v6 02/13] dt-bindings: power: supply: Add MediaTek MT6370 Charger ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` [PATCH v6 03/13] dt-bindings: leds: mt6370: Add MediaTek MT6370 current sink type LED indicator ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` [PATCH v6 04/13] dt-bindings: leds: Add MediaTek MT6370 flashlight ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-30 21:30 ` Pavel Machek 2022-07-30 21:30 ` Pavel Machek 2022-07-30 21:30 ` Pavel Machek 2022-07-22 10:23 ` [PATCH v6 05/13] dt-bindings: backlight: Add MediaTek MT6370 backlight ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-07-22 10:23 ` ChiaEn Wu 2022-08-01 6:47 ` ChiaEn Wu [this message] 2022-08-01 6:47 ` ChiaEn Wu 2022-07-22 10:24 ` [PATCH v6 06/13] dt-bindings: mfd: Add MediaTek MT6370 ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` [PATCH v6 07/13] mfd: mt6370: Add MediaTek MT6370 support ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-25 7:59 ` Andy Shevchenko 2022-07-25 7:59 ` Andy Shevchenko 2022-07-25 7:59 ` Andy Shevchenko 2022-07-25 8:29 ` ChiaEn Wu 2022-07-25 8:29 ` ChiaEn Wu 2022-07-25 8:29 ` ChiaEn Wu 2022-07-25 8:43 ` Andy Shevchenko 2022-07-25 8:43 ` Andy Shevchenko 2022-07-25 8:43 ` Andy Shevchenko 2022-07-25 9:06 ` ChiaEn Wu 2022-07-25 9:06 ` ChiaEn Wu 2022-07-25 9:06 ` ChiaEn Wu 2022-07-25 9:09 ` Andy Shevchenko 2022-07-25 9:09 ` Andy Shevchenko 2022-07-25 9:09 ` Andy Shevchenko 2022-07-22 10:24 ` [PATCH v6 08/13] usb: typec: tcpci_mt6370: Add MediaTek MT6370 tcpci driver ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 14:19 ` Guenter Roeck 2022-07-22 14:19 ` Guenter Roeck 2022-07-22 14:19 ` Guenter Roeck 2022-07-25 7:06 ` Chunfeng Yun 2022-07-25 7:06 ` Chunfeng Yun 2022-07-25 7:06 ` Chunfeng Yun 2022-07-25 7:17 ` ChiaEn Wu 2022-07-25 7:17 ` ChiaEn Wu 2022-07-25 7:17 ` ChiaEn Wu 2022-07-25 8:03 ` Andy Shevchenko 2022-07-25 8:03 ` Andy Shevchenko 2022-07-25 8:03 ` Andy Shevchenko 2022-07-22 10:24 ` [PATCH v6 09/13] iio: adc: mt6370: Add MediaTek MT6370 support ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-25 8:10 ` Andy Shevchenko 2022-07-25 8:10 ` Andy Shevchenko 2022-07-25 8:10 ` Andy Shevchenko 2022-07-22 10:24 ` [PATCH v6 10/13] power: supply: mt6370: Add MediaTek MT6370 charger driver ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-25 8:25 ` Andy Shevchenko 2022-07-25 8:25 ` Andy Shevchenko 2022-07-25 8:25 ` Andy Shevchenko 2022-07-22 10:24 ` [PATCH v6 11/13] leds: rgb: mt6370: Add MediaTek MT6370 current sink type LED Indicator support ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:52 ` AngeloGioacchino Del Regno 2022-07-22 10:52 ` AngeloGioacchino Del Regno 2022-07-22 10:52 ` AngeloGioacchino Del Regno 2022-07-25 8:40 ` Andy Shevchenko 2022-07-25 8:40 ` Andy Shevchenko 2022-07-25 8:40 ` Andy Shevchenko 2022-07-26 11:45 ` ChiaEn Wu 2022-07-26 11:45 ` ChiaEn Wu 2022-07-26 11:45 ` ChiaEn Wu 2022-07-26 12:17 ` Andy Shevchenko 2022-07-26 12:17 ` Andy Shevchenko 2022-07-26 12:17 ` Andy Shevchenko 2022-07-27 7:36 ` ChiaEn Wu 2022-07-27 7:36 ` ChiaEn Wu 2022-07-27 7:36 ` ChiaEn Wu 2022-07-27 10:03 ` Andy Shevchenko 2022-07-27 10:03 ` Andy Shevchenko 2022-07-27 10:03 ` Andy Shevchenko 2022-07-30 21:39 ` Pavel Machek 2022-07-30 21:39 ` Pavel Machek 2022-07-30 21:39 ` Pavel Machek 2022-08-01 7:55 ` szuni chen 2022-08-01 7:55 ` szuni chen 2022-07-22 10:24 ` [PATCH v6 12/13] leds: flash: mt6370: Add MediaTek MT6370 flashlight support ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:48 ` AngeloGioacchino Del Regno 2022-07-22 10:48 ` AngeloGioacchino Del Regno 2022-07-22 10:48 ` AngeloGioacchino Del Regno 2022-07-25 8:51 ` Andy Shevchenko 2022-07-25 8:51 ` Andy Shevchenko 2022-07-25 8:51 ` Andy Shevchenko 2022-07-29 6:17 ` szuni chen 2022-07-29 6:17 ` szuni chen 2022-07-29 6:17 ` szuni chen 2022-07-29 10:34 ` Andy Shevchenko 2022-07-29 10:34 ` Andy Shevchenko 2022-07-29 10:34 ` Andy Shevchenko 2022-07-25 8:55 ` Andy Shevchenko 2022-07-25 8:55 ` Andy Shevchenko 2022-07-25 8:55 ` Andy Shevchenko 2022-07-26 4:15 ` szuni chen 2022-07-26 4:15 ` szuni chen 2022-07-26 4:15 ` szuni chen 2022-07-26 6:10 ` Andy Shevchenko 2022-07-26 6:10 ` Andy Shevchenko 2022-07-26 6:10 ` Andy Shevchenko 2022-07-30 21:42 ` Pavel Machek 2022-07-30 21:42 ` Pavel Machek 2022-07-30 21:42 ` Pavel Machek 2022-08-04 9:53 ` Alice Chen 2022-08-04 9:53 ` Alice Chen 2022-08-04 9:53 ` Alice Chen 2022-07-22 10:24 ` [PATCH v6 13/13] video: backlight: mt6370: Add MediaTek MT6370 support ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-22 10:24 ` ChiaEn Wu 2022-07-25 9:07 ` Andy Shevchenko 2022-07-25 9:07 ` Andy Shevchenko 2022-07-25 9:07 ` Andy Shevchenko 2022-07-25 10:31 ` Daniel Thompson 2022-07-25 10:31 ` Daniel Thompson 2022-07-25 10:31 ` Daniel Thompson 2022-07-26 2:20 ` ChiaEn Wu 2022-07-26 2:20 ` ChiaEn Wu 2022-07-26 2:20 ` ChiaEn Wu 2022-07-26 9:30 ` Daniel Thompson 2022-07-26 9:30 ` Daniel Thompson 2022-07-26 9:30 ` Daniel Thompson 2022-07-26 11:28 ` ChiaEn Wu 2022-07-26 11:28 ` ChiaEn Wu 2022-07-26 11:28 ` ChiaEn Wu 2022-07-26 11:59 ` Daniel Thompson 2022-07-26 11:59 ` Daniel Thompson 2022-07-26 11:59 ` Daniel Thompson 2022-07-27 6:24 ` ChiaEn Wu 2022-07-27 6:24 ` ChiaEn Wu 2022-07-27 6:24 ` ChiaEn Wu 2022-07-28 11:31 ` Daniel Thompson 2022-07-28 11:31 ` Daniel Thompson 2022-07-28 11:31 ` Daniel Thompson
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CABtFH5L83d5Di6O9TC-L3UX2ma5J3PE47ihfJFfPD5YGJ43NxQ@mail.gmail.com \ --to=peterwu.pub@gmail.com \ --cc=alice_chen@richtek.com \ --cc=andy.shevchenko@gmail.com \ --cc=broonie@kernel.org \ --cc=chiaen_wu@richtek.com \ --cc=chunfeng.yun@mediatek.com \ --cc=cy_huang@richtek.com \ --cc=daniel.thompson@linaro.org \ --cc=deller@gmx.de \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=gregkh@linuxfoundation.org \ --cc=heikki.krogerus@linux.intel.com \ --cc=jic23@kernel.org \ --cc=jingoohan1@gmail.com \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=lars@metafoo.de \ --cc=lee.jones@linaro.org \ --cc=lgirdwood@gmail.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-fbdev@vger.kernel.org \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-leds@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=matthias.bgg@gmail.com \ --cc=pavel@ucw.cz \ --cc=robh+dt@kernel.org \ --cc=robh@kernel.org \ --cc=sre@kernel.org \ --cc=szunichen@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.