From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753333AbbFEKpm (ORCPT ); Fri, 5 Jun 2015 06:45:42 -0400 Received: from mailgw01.mediatek.com ([210.61.82.183]:48605 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751653AbbFEKpj (ORCPT ); Fri, 5 Jun 2015 06:45:39 -0400 X-Listener-Flag: 11101 Message-ID: <1433501134.17376.23.camel@mtksdaap41> Subject: Re: [PATCH v2 0/2] Add MediaTek display PWM driver From: YH Huang To: Matthias Brugger CC: Yingjoe Chen , Thierry Reding , Mark Rutland , "linux-pwm@vger.kernel.org" , srv_heupstream , Pawel Moll , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Rob Herring , "linux-mediatek@lists.infradead.org" , Sascha Hauer , "linux-arm-kernel@lists.infradead.org" Date: Fri, 5 Jun 2015 18:45:34 +0800 In-Reply-To: References: <1432214964-40644-1-git-send-email-yh.huang@mediatek.com> <1432520091.23308.5.camel@mtksdaap41> <1433484929.17376.17.camel@mtksdaap41> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2015-06-05 at 10:39 +0200, Matthias Brugger wrote: > Hi YH, > > 2015-06-05 8:15 GMT+02:00 YH Huang : > > On Mon, 2015-05-25 at 10:14 +0800, Yingjoe Chen wrote: > >> On Thu, 2015-05-21 at 21:29 +0800, YH Huang wrote: > > > > This patch series add the use of display PWM driver and documentation > > for Mediatek SoCs. The driver is used to support the backlight of > > the panel. This is based on v4.1-rc1. Change in v2: 1. Rewrite descriptions for driver in kconfig to make it much clear. 2. Rename the driver from "pwm-mediatek-disp" to "pwm-mtk-disp". 3. Disable clocks in the error path. 4. Change the shift values in decimal instead of hex. 5. Rename and add some variables or function name for consistency and readability. 6. Drop unnecessary parentheses, spaces, variables and add newlines to make the code easiler to read. > > > >> > YH Huang (2): > >> > dt-bindings: pwm: add MediaTek display PWM bindings > >> > pwm: add MediaTek display PWM driver support > >> > > >> > .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 25 +++ > >> > drivers/pwm/Kconfig | 10 + > >> > drivers/pwm/Makefile | 1 + > >> > drivers/pwm/pwm-mtk-disp.c | 228 +++++++++++++++++++++ > >> > 4 files changed, 264 insertions(+) > >> > create mode 100644 Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt > >> > create mode 100644 drivers/pwm/pwm-mtk-disp.c > >> > >> Hi YH, > >> > >> It would be easier for reviewer if you have a summary here on what you > >> have changed compare to last version. > >> Also, please add patch series summary even for v2, it remind reviewer > >> what this series is about. > >> > >> Joe.C > >> > > > > The patch v2 is refined with everybody's suggestions. > > It is much readable and consistent. > > I suppose that Yingjoe wanted to emphasis that for the reviewer it is > much easier if you describe the changes you made from v1 to v2. > You can find an example (random pick from the mailing list) here: > http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000572.html > > Thanks, > Matthias > I have added the change lists. Thank you. Regards, YH Huang From mboxrd@z Thu Jan 1 00:00:00 1970 From: yh.huang@mediatek.com (YH Huang) Date: Fri, 5 Jun 2015 18:45:34 +0800 Subject: [PATCH v2 0/2] Add MediaTek display PWM driver In-Reply-To: References: <1432214964-40644-1-git-send-email-yh.huang@mediatek.com> <1432520091.23308.5.camel@mtksdaap41> <1433484929.17376.17.camel@mtksdaap41> Message-ID: <1433501134.17376.23.camel@mtksdaap41> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, 2015-06-05 at 10:39 +0200, Matthias Brugger wrote: > Hi YH, > > 2015-06-05 8:15 GMT+02:00 YH Huang : > > On Mon, 2015-05-25 at 10:14 +0800, Yingjoe Chen wrote: > >> On Thu, 2015-05-21 at 21:29 +0800, YH Huang wrote: > > > > This patch series add the use of display PWM driver and documentation > > for Mediatek SoCs. The driver is used to support the backlight of > > the panel. This is based on v4.1-rc1. Change in v2: 1. Rewrite descriptions for driver in kconfig to make it much clear. 2. Rename the driver from "pwm-mediatek-disp" to "pwm-mtk-disp". 3. Disable clocks in the error path. 4. Change the shift values in decimal instead of hex. 5. Rename and add some variables or function name for consistency and readability. 6. Drop unnecessary parentheses, spaces, variables and add newlines to make the code easiler to read. > > > >> > YH Huang (2): > >> > dt-bindings: pwm: add MediaTek display PWM bindings > >> > pwm: add MediaTek display PWM driver support > >> > > >> > .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 25 +++ > >> > drivers/pwm/Kconfig | 10 + > >> > drivers/pwm/Makefile | 1 + > >> > drivers/pwm/pwm-mtk-disp.c | 228 +++++++++++++++++++++ > >> > 4 files changed, 264 insertions(+) > >> > create mode 100644 Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt > >> > create mode 100644 drivers/pwm/pwm-mtk-disp.c > >> > >> Hi YH, > >> > >> It would be easier for reviewer if you have a summary here on what you > >> have changed compare to last version. > >> Also, please add patch series summary even for v2, it remind reviewer > >> what this series is about. > >> > >> Joe.C > >> > > > > The patch v2 is refined with everybody's suggestions. > > It is much readable and consistent. > > I suppose that Yingjoe wanted to emphasis that for the reviewer it is > much easier if you describe the changes you made from v1 to v2. > You can find an example (random pick from the mailing list) here: > http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000572.html > > Thanks, > Matthias > I have added the change lists. Thank you. Regards, YH Huang