Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932322AbbFEIjP (ORCPT ); Fri, 5 Jun 2015 04:39:15 -0400 Received: from mail-la0-f46.google.com ([209.85.215.46]:32829 "EHLO mail-la0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750831AbbFEIjN (ORCPT ); Fri, 5 Jun 2015 04:39:13 -0400 MIME-Version: 1.0 In-Reply-To: <1433484929.17376.17.camel@mtksdaap41> References: <1432214964-40644-1-git-send-email-yh.huang@mediatek.com> <1432520091.23308.5.camel@mtksdaap41> <1433484929.17376.17.camel@mtksdaap41> Date: Fri, 5 Jun 2015 10:39:11 +0200 Message-ID: Subject: Re: [PATCH v2 0/2] Add MediaTek display PWM driver From: Matthias Brugger To: YH Huang 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" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1863 Lines: 50 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. > >> > 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 -- motzblog.wordpress.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/