Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932481Ab3J1Oz7 (ORCPT ); Mon, 28 Oct 2013 10:55:59 -0400 Received: from eusmtp01.atmel.com ([212.144.249.242]:18966 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932324Ab3J1Ote (ORCPT ); Mon, 28 Oct 2013 10:49:34 -0400 Message-ID: <526E797A.9010509@atmel.com> Date: Mon, 28 Oct 2013 15:49:30 +0100 From: Nicolas Ferre Organization: atmel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 MIME-Version: 1.0 To: Bo Shen CC: , , , , , Thierry Reding Subject: Re: [PATCH v5] PWM: atmel-pwm: add PWM controller driver References: <1380532240-18650-1-git-send-email-voice.shen@atmel.com> <20131008131727.GB12839@ulmo.nvidia.com> <52664FB6.4000004@atmel.com> In-Reply-To: <52664FB6.4000004@atmel.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.161.30.18] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2269 Lines: 58 On 22/10/2013 12:13, Bo Shen : > Hi All, > > On 10/8/2013 21:17, Thierry Reding wrote: >> On Mon, Sep 30, 2013 at 05:10:40PM +0800, Bo Shen wrote: >>> Add Atmel PWM controller driver based on PWM framework. >>> >>> This is the basic function implementation of Atmel PWM controller. >>> It can work with PWM based led and backlight. >>> >>> Signed-off-by: Bo Shen >>> >>> --- >>> Changes in v5: >>> - call clk_disable directly, if so, it won't cause more than one channel >>> enabled, the clock can not be disabled. >>> >>> Changes in v4: >>> - check the return value of clk_prepare() >>> - change channel register size as constant >>> >>> Changes in v3: >>> - change compatible string from "atmel,sama5-pwm" to "atmel,sama5d3-pwm" >>> - Add PWM led example in binding documentation >>> - Using micro replace hard code >>> >>> Changes in v2: >>> - Address the comments from Thierry Reding >>> >>> .../devicetree/bindings/pwm/atmel-pwm.txt | 41 +++ >>> drivers/pwm/Kconfig | 9 + >>> drivers/pwm/Makefile | 1 + >>> drivers/pwm/pwm-atmel.c | 344 ++++++++++++++++++++ >>> 4 files changed, 395 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/pwm/atmel-pwm.txt >>> create mode 100644 drivers/pwm/pwm-atmel.c >> >> I haven't seen an Acked-by from the device tree bindings maintainers on >> this. I've Cc'ed them, so hopefully one of them has time to review. > > Any comments? As a recommendation from the recent Kernel Summit stated, you will have to separate the binding itself from the driver and associated code. Simply send the binding documentation to the DT maintainers but still keep the two patches linked together in a series (but without putting DT maintainers nor the DT mailing-list in CC of the non-documentation patches). This way you may have more chances to get feedback. Thanks, bye, -- Nicolas Ferre -- 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/