diff mbox

clk: lpc32xx: allow peripheral clock selection in device tree

Message ID 1464982475-24738-1-git-send-email-slemieux.tyco@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sylvain Lemieux June 3, 2016, 7:34 p.m. UTC 
From: Sylvain Lemieux <slemieux@tycoint.com>

This patch add the support to select the peripheral clock (PERIPH)
as a parent clock source using the "assigned-clock-parents"
parameter in the device tree.

Signed-off-by: Sylvain Lemieux <slemieux@tycoint.com>
---
 drivers/clk/nxp/clk-lpc32xx.c             | 3 +--
 include/dt-bindings/clock/lpc32xx-clock.h | 1 +
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Sylvain Lemieux June 21, 2016, 12:35 p.m. UTC | #1 
Hi Vladimir,

On Tue, 2016-06-21 at 07:28 +0300, Vladimir Zapolskiy wrote:
> Hi Sylvain,
> 
> On 03.06.2016 22:34, Sylvain Lemieux wrote:
> > From: Sylvain Lemieux <slemieux@tycoint.com>
> > 
> > This patch add the support to select the peripheral clock (PERIPH)
> > as a parent clock source using the "assigned-clock-parents"
> > parameter in the device tree.
> > 
> > Signed-off-by: Sylvain Lemieux <slemieux@tycoint.com>
> 
> Acked-by: Vladimir Zapolskiy <vz@mleia.com>
> 
> The change looks good, is it done to support a selection of PWM parent clocks?

Yes, this is the case.
> 
> I think it makes sense to set PERIPH clock as a default parent for both
> PWM controllers, please consider to send a DTS change.

I will do.

Thanks for the feedback,
Sylvain
Sylvain Lemieux July 4, 2016, 1:43 p.m. UTC | #2 
Hi Stephan,

On Tue, 2016-06-21 at 08:35 -0400, Sylvain Lemieux wrote:
> Hi Vladimir,
> 
> On Tue, 2016-06-21 at 07:28 +0300, Vladimir Zapolskiy wrote:
> > Hi Sylvain,
> > 
> > On 03.06.2016 22:34, Sylvain Lemieux wrote:
> > > From: Sylvain Lemieux <slemieux@tycoint.com>
> > > 
> > > This patch add the support to select the peripheral clock (PERIPH)
> > > as a parent clock source using the "assigned-clock-parents"
> > > parameter in the device tree.
> > > 
> > > Signed-off-by: Sylvain Lemieux <slemieux@tycoint.com>
> > 
> > Acked-by: Vladimir Zapolskiy <vz@mleia.com>
> > 
> > The change looks good, is it done to support a selection of PWM parent clocks?
> 
> Yes, this is the case.
> > 
> > I think it makes sense to set PERIPH clock as a default parent for both
> > PWM controllers, please consider to send a DTS change.
> 
> I will do.
> 
> Thanks for the feedback,
> Sylvain
> 
do you think you can take this change for 4.8?


Regards,
Sylvain
diff mbox

Patch

diff --git a/drivers/clk/nxp/clk-lpc32xx.c b/drivers/clk/nxp/clk-lpc32xx.c
index 481b264..90d740a 100644
--- a/drivers/clk/nxp/clk-lpc32xx.c
+++ b/drivers/clk/nxp/clk-lpc32xx.c
@@ -87,7 +87,7 @@  enum {
 
 enum {
 	/* Start from the last defined clock in dt bindings */
-	LPC32XX_CLK_ADC_DIV = LPC32XX_CLK_HCLK_PLL + 1,
+	LPC32XX_CLK_ADC_DIV = LPC32XX_CLK_PERIPH + 1,
 	LPC32XX_CLK_ADC_RTC,
 	LPC32XX_CLK_TEST1,
 	LPC32XX_CLK_TEST2,
@@ -99,7 +99,6 @@  enum {
 	LPC32XX_CLK_HCLK_DIV_PERIPH,
 	LPC32XX_CLK_HCLK_DIV,
 	LPC32XX_CLK_HCLK,
-	LPC32XX_CLK_PERIPH,
 	LPC32XX_CLK_ARM,
 	LPC32XX_CLK_ARM_VFP,
 
diff --git a/include/dt-bindings/clock/lpc32xx-clock.h b/include/dt-bindings/clock/lpc32xx-clock.h
index d41b6fe..e624d3a 100644
--- a/include/dt-bindings/clock/lpc32xx-clock.h
+++ b/include/dt-bindings/clock/lpc32xx-clock.h
@@ -48,6 +48,7 @@ 
 #define LPC32XX_CLK_PWM2	33
 #define LPC32XX_CLK_ADC		34
 #define LPC32XX_CLK_HCLK_PLL	35
+#define LPC32XX_CLK_PERIPH	36
 
 /* LPC32XX USB clocks */
 #define LPC32XX_USB_CLK_I2C	1