From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Franklin S Cooper Jr." Subject: Re: [Patch v4 6/6] ARM: dts: DRA7: Add dt nodes for PWMSS Date: Mon, 7 Mar 2016 18:20:42 -0600 Message-ID: <56DE1ADA.5040903@ti.com> References: <1457383187-17166-1-git-send-email-fcooper@ti.com> <1457383187-17166-7-git-send-email-fcooper@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Paul Walmsley Cc: vigneshr@ti.com, thierry.reding@gmail.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, bcousson@baylibre.com, tony@atomide.com, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: linux-omap@vger.kernel.org On 03/07/2016 05:53 PM, Paul Walmsley wrote: > On Mon, 7 Mar 2016, Franklin S Cooper Jr wrote: > >> From: Vignesh R >> >> Add PWMSS device tree nodes for DRA7 SoC family and add documentation >> for dt bindings. >> >> Signed-off-by: Vignesh R >> [fcooper@ti.com: Use updated bindings for PWMSS and ePWM] >> Signed-off-by: Franklin S Cooper Jr >> --- >> Version 4 changes: >> Remove hwmod entries. >> Add clocks/clock-names to pwm nodes. > Hmm, what about moving the eqep, ecap DT nodes under the epwmss nodes? > > > - Paul We don't have a driver for eQEP which is why there is no DT entry for it. For eCAP, the original patch series didn't include the eCAP DT nodes and I never verified why this is the case. In our TI tree we have eCAP0 only in the DT. The TRM doesn't state that eCAP0-2 are unsupported so there should be entries for all 3 eCAPs. For AM335x and AM437x, eCAP nodes are already under the ePWMSS nodes. From mboxrd@z Thu Jan 1 00:00:00 1970 From: fcooper@ti.com (Franklin S Cooper Jr.) Date: Mon, 7 Mar 2016 18:20:42 -0600 Subject: [Patch v4 6/6] ARM: dts: DRA7: Add dt nodes for PWMSS In-Reply-To: References: <1457383187-17166-1-git-send-email-fcooper@ti.com> <1457383187-17166-7-git-send-email-fcooper@ti.com> Message-ID: <56DE1ADA.5040903@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 03/07/2016 05:53 PM, Paul Walmsley wrote: > On Mon, 7 Mar 2016, Franklin S Cooper Jr wrote: > >> From: Vignesh R >> >> Add PWMSS device tree nodes for DRA7 SoC family and add documentation >> for dt bindings. >> >> Signed-off-by: Vignesh R >> [fcooper at ti.com: Use updated bindings for PWMSS and ePWM] >> Signed-off-by: Franklin S Cooper Jr >> --- >> Version 4 changes: >> Remove hwmod entries. >> Add clocks/clock-names to pwm nodes. > Hmm, what about moving the eqep, ecap DT nodes under the epwmss nodes? > > > - Paul We don't have a driver for eQEP which is why there is no DT entry for it. For eCAP, the original patch series didn't include the eCAP DT nodes and I never verified why this is the case. In our TI tree we have eCAP0 only in the DT. The TRM doesn't state that eCAP0-2 are unsupported so there should be entries for all 3 eCAPs. For AM335x and AM437x, eCAP nodes are already under the ePWMSS nodes. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934216AbcCHAVd (ORCPT ); Mon, 7 Mar 2016 19:21:33 -0500 Received: from arroyo.ext.ti.com ([192.94.94.40]:60826 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934076AbcCHAV1 (ORCPT ); Mon, 7 Mar 2016 19:21:27 -0500 From: "Franklin S Cooper Jr." To: Paul Walmsley CC: , , , , , , , , , , , , , Subject: Re: [Patch v4 6/6] ARM: dts: DRA7: Add dt nodes for PWMSS References: <1457383187-17166-1-git-send-email-fcooper@ti.com> <1457383187-17166-7-git-send-email-fcooper@ti.com> Message-ID: <56DE1ADA.5040903@ti.com> Date: Mon, 7 Mar 2016 18:20:42 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: 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 On 03/07/2016 05:53 PM, Paul Walmsley wrote: > On Mon, 7 Mar 2016, Franklin S Cooper Jr wrote: > >> From: Vignesh R >> >> Add PWMSS device tree nodes for DRA7 SoC family and add documentation >> for dt bindings. >> >> Signed-off-by: Vignesh R >> [fcooper@ti.com: Use updated bindings for PWMSS and ePWM] >> Signed-off-by: Franklin S Cooper Jr >> --- >> Version 4 changes: >> Remove hwmod entries. >> Add clocks/clock-names to pwm nodes. > Hmm, what about moving the eqep, ecap DT nodes under the epwmss nodes? > > > - Paul We don't have a driver for eQEP which is why there is no DT entry for it. For eCAP, the original patch series didn't include the eCAP DT nodes and I never verified why this is the case. In our TI tree we have eCAP0 only in the DT. The TRM doesn't state that eCAP0-2 are unsupported so there should be entries for all 3 eCAPs. For AM335x and AM437x, eCAP nodes are already under the ePWMSS nodes.