Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752759AbcD0MtC (ORCPT ); Wed, 27 Apr 2016 08:49:02 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:48435 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752159AbcD0Ms7 (ORCPT ); Wed, 27 Apr 2016 08:48:59 -0400 Subject: Re: [PATCH v7 1/9] clk: ti: am335x/am4372: Add tbclk to pwm node To: Franklin S Cooper Jr , , , , , , , , , , , , References: <1461710209-6563-1-git-send-email-fcooper@ti.com> <1461710209-6563-2-git-send-email-fcooper@ti.com> From: Tero Kristo Message-ID: <5720B51E.6050106@ti.com> Date: Wed, 27 Apr 2016 15:48:30 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1461710209-6563-2-git-send-email-fcooper@ti.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2124 Lines: 53 On 27/04/16 01:36, Franklin S Cooper Jr wrote: > Add tblck to the pwm nodes. This insures that the ehrpwm driver has access > to the time-based clk. > > Do not remove similar entries for ehrpwm node. Later patches will switch > from using ehrpwm node name to pwm. But to maintain ABI compatibility we > shouldn't remove the old entries. > > Signed-off-by: Franklin S Cooper Jr > Acked-by: Stephen Boyd I don't like adding new clock aliases to the list as I am trying to get rid of them, but in this case we can't avoid it I believe. Thus, Acked-by: Tero Kristo > --- > drivers/clk/ti/clk-33xx.c | 3 +++ > drivers/clk/ti/clk-43xx.c | 6 ++++++ > 2 files changed, 9 insertions(+) > > diff --git a/drivers/clk/ti/clk-33xx.c b/drivers/clk/ti/clk-33xx.c > index ef2ec64..0e47d95 100644 > --- a/drivers/clk/ti/clk-33xx.c > +++ b/drivers/clk/ti/clk-33xx.c > @@ -108,6 +108,9 @@ static struct ti_dt_clk am33xx_clks[] = { > DT_CLK("48300200.ehrpwm", "tbclk", "ehrpwm0_tbclk"), > DT_CLK("48302200.ehrpwm", "tbclk", "ehrpwm1_tbclk"), > DT_CLK("48304200.ehrpwm", "tbclk", "ehrpwm2_tbclk"), > + DT_CLK("48300200.pwm", "tbclk", "ehrpwm0_tbclk"), > + DT_CLK("48302200.pwm", "tbclk", "ehrpwm1_tbclk"), > + DT_CLK("48304200.pwm", "tbclk", "ehrpwm2_tbclk"), > { .node_name = NULL }, > }; > > diff --git a/drivers/clk/ti/clk-43xx.c b/drivers/clk/ti/clk-43xx.c > index 097fc90..7255aa8 100644 > --- a/drivers/clk/ti/clk-43xx.c > +++ b/drivers/clk/ti/clk-43xx.c > @@ -115,6 +115,12 @@ static struct ti_dt_clk am43xx_clks[] = { > DT_CLK("48306200.ehrpwm", "tbclk", "ehrpwm3_tbclk"), > DT_CLK("48308200.ehrpwm", "tbclk", "ehrpwm4_tbclk"), > DT_CLK("4830a200.ehrpwm", "tbclk", "ehrpwm5_tbclk"), > + DT_CLK("48300200.pwm", "tbclk", "ehrpwm0_tbclk"), > + DT_CLK("48302200.pwm", "tbclk", "ehrpwm1_tbclk"), > + DT_CLK("48304200.pwm", "tbclk", "ehrpwm2_tbclk"), > + DT_CLK("48306200.pwm", "tbclk", "ehrpwm3_tbclk"), > + DT_CLK("48308200.pwm", "tbclk", "ehrpwm4_tbclk"), > + DT_CLK("4830a200.pwm", "tbclk", "ehrpwm5_tbclk"), > { .node_name = NULL }, > }; > >