Re: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2 output on SoC pad

2013-06-03 Thread Benoit Cousson
Salut Paul,

On 05/29/2013 09:06 PM, Paul Walmsley wrote:
 Salut Benoît,
 
 On Mon, 27 May 2013, Benoit Cousson wrote:
 
 On 05/20/2013 04:47 AM, Paul Walmsley wrote:
 On Fri, 17 May 2013, Hiremath, Vaibhav wrote:

 Tony, Paul and Benoit,

 Any update on this series?

 Most of the changes are DT-related, so it should probably go in via 
 Benoît.  I'll ack the clock side, but have a comment on patch 2, to be 
 sent shortly.

 I've tried to apply that patch, but unfortunately, it does depend on
 that one:

 https://patchwork.kernel.org/patch/2212081/

 If you did not already pulled the first one I can take both with your
 acked-by.
 Otherwise, I can take only the DTS ones.

 Is that OK?
 
 Will ack that first patch now.  

Cool, Thanks Paul.

Regards,
Benoit


--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2 output on SoC pad

2013-05-29 Thread Paul Walmsley
Salut Benoît,

On Mon, 27 May 2013, Benoit Cousson wrote:

 On 05/20/2013 04:47 AM, Paul Walmsley wrote:
  On Fri, 17 May 2013, Hiremath, Vaibhav wrote:
  
  Tony, Paul and Benoit,
 
  Any update on this series?
  
  Most of the changes are DT-related, so it should probably go in via 
  Benoît.  I'll ack the clock side, but have a comment on patch 2, to be 
  sent shortly.
 
 I've tried to apply that patch, but unfortunately, it does depend on
 that one:
 
 https://patchwork.kernel.org/patch/2212081/

 If you did not already pulled the first one I can take both with your
 acked-by.
 Otherwise, I can take only the DTS ones.
 
 Is that OK?

Will ack that first patch now.  

Vaibhav, looks like that first series is still in RFC form; you might want 
to repost it as a non-RFC series.


- Paul

Re: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2 output on SoC pad

2013-05-27 Thread Benoit Cousson
Salut Paul,

On 05/20/2013 04:47 AM, Paul Walmsley wrote:
 On Fri, 17 May 2013, Hiremath, Vaibhav wrote:
 
 Tony, Paul and Benoit,

 Any update on this series?
 
 Most of the changes are DT-related, so it should probably go in via 
 Benoît.  I'll ack the clock side, but have a comment on patch 2, to be 
 sent shortly.

I've tried to apply that patch, but unfortunately, it does depend on
that one:

https://patchwork.kernel.org/patch/2212081/

If you did not already pulled the first one I can take both with your
acked-by.
Otherwise, I can take only the DTS ones.

Is that OK?

Thanks,
Benoit


--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2 output on SoC pad

2013-05-19 Thread Paul Walmsley
On Fri, 17 May 2013, Hiremath, Vaibhav wrote:

 Tony, Paul and Benoit,
 
 Any update on this series?

Most of the changes are DT-related, so it should probably go in via 
Benoît.  I'll ack the clock side, but have a comment on patch 2, to be 
sent shortly.


- Paul

RE: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2 output on SoC pad

2013-05-19 Thread Hiremath, Vaibhav

 -Original Message-
 From: Paul Walmsley [mailto:p...@pwsan.com]
 Sent: Monday, May 20, 2013 8:17 AM
 To: Hiremath, Vaibhav
 Cc: linux-omap@vger.kernel.org; t...@atomide.com; Cousson, Benoit;
 linux-arm-ker...@lists.infradead.org
 Subject: RE: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2
 output on SoC pad
 
 On Fri, 17 May 2013, Hiremath, Vaibhav wrote:
 
  Tony, Paul and Benoit,
 
  Any update on this series?
 
 Most of the changes are DT-related, so it should probably go in via
 Benoît.  I'll ack the clock side, but have a comment on patch 2, to be
 sent shortly.
 
 

Thanks Paul for the review, I will next version shortly with your ack.


Thanks,
Vaibhav
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2 output on SoC pad

2013-05-17 Thread Hiremath, Vaibhav

 -Original Message-
 From: Hiremath, Vaibhav
 Sent: Tuesday, May 07, 2013 3:10 PM
 To: linux-omap@vger.kernel.org
 Cc: t...@atomide.com; Cousson, Benoit; p...@pwsan.com; linux-arm-
 ker...@lists.infradead.org; Hiremath, Vaibhav
 Subject: [PATCH 0/2] ARM: OMAP AM33XX: clock data: Enable clkout2
 output on SoC pad
 
 'clkout2' comes out on the device pad and is being used by various
 external on-board peripherals like, Audio codecs, Wilink and stuff.
 So enable the clkout2 by default during init sequence itself
 along with right pinmux configuration clkout2 output.
 
 Also, add the missing entry of clkout2_ck to the AM33xx clock table.
 
 As far as enabling of clkout2 during init is concerned,
 we can argue that it can be handled using DT property and enable
 the clock only when specified; but not until OMAP clock-tree
 migration to DT.
 
 Vaibhav Hiremath (2):
   ARM: OMAP AM33XX: clock data: Enable clkout2 as part of init
   ARM: dts: AM33XX: Set pinmux for clkout2 pad used for clock output
 
  arch/arm/boot/dts/am335x-bone.dts |8 +++-
  arch/arm/boot/dts/am335x-evm.dts  |8 +++-
  arch/arm/boot/dts/am335x-evmsk.dts|8 +++-
  arch/arm/mach-omap2/cclock33xx_data.c |2 ++
  4 files changed, 23 insertions(+), 3 deletions(-)

Tony, Paul and Benoit,

Any update on this series?

Thanks,
Vaibhav

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html