Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936257Ab3DILgE (ORCPT ); Tue, 9 Apr 2013 07:36:04 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:60748 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935371Ab3DILgB (ORCPT ); Tue, 9 Apr 2013 07:36:01 -0400 Message-ID: <5163FCFD.6070606@ti.com> Date: Tue, 9 Apr 2013 17:05:25 +0530 From: Sekhar Nori User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130328 Thunderbird/17.0.5 MIME-Version: 1.0 To: "Philip, Avinash" CC: "linux@arm.linux.org.uk" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "davinci-linux-open-source@linux.davincidsp.com" , "Manjunathappa, Prakash" , "jacmet@sunsite.dk" Subject: Re: [PATCH v3 3/3] ARM: davinci: da850: add EHRPWM & ECAP DT node References: <1364197789-16783-1-git-send-email-avinashphilip@ti.com> <1364197789-16783-4-git-send-email-avinashphilip@ti.com> <515A97DE.1080403@ti.com> <518397C60809E147AF5323E0420B992E3EAA91B6@DBDE01.ent.ti.com> <5162C1A5.5060808@ti.com> <518397C60809E147AF5323E0420B992E3EAA9DA3@DBDE01.ent.ti.com> In-Reply-To: <518397C60809E147AF5323E0420B992E3EAA9DA3@DBDE01.ent.ti.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1367 Lines: 36 On 4/9/2013 2:12 PM, Philip, Avinash wrote: > On Mon, Apr 08, 2013 at 18:39:57, Nori, Sekhar wrote: >> >> On 4/8/2013 2:39 PM, Philip, Avinash wrote: >>> On Tue, Apr 02, 2013 at 14:03:34, Nori, Sekhar wrote: >>>> On 3/25/2013 1:19 PM, Philip Avinash wrote: >>>>> Add da850 EHRPWM & ECAP DT node. >>>>> Also adds OF_DEV_AUXDATA for EHRPWM & ECAP driver to use EHRPWM & ECAP >>>>> clock. >>>> >>>> This looks fine to me but I will wait for the bindings to get accepted >>>> before taking this one. >>> >>> Sekhar, >>> >>> Binding document got accepted in PWM tree [1]. >>> Can you accept this patch? >> >> Can you also add the pinmux definitions and resend just this patch? >> Sorry I did not notice those were missing earlier. > > According to latest schematics, ECAP instance 2 being used for PWM backlight > control. Should I add pin-mux only for ECAP2 or for all PWM instances? I meant add definitions in .dtsi. Since there is only one pin a given functionality can be present on in DaVinci, it can be done in a board independent manner. See examples for other peripherals in existing da850.dtsi file. Thanks, Sekhar -- 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/