Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1239666 > unrolled thread

Re: [PATCH v4 4/4] misc: atmel_tclib: get and use slow clock

Started byThierry Reding <thierry.reding@gmail.com>
First post2015-10-05 17:30 +0200
Last post2015-10-06 09:40 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v4 4/4] misc: atmel_tclib: get and use slow clock Thierry Reding <thierry.reding@gmail.com> - 2015-10-05 17:30 +0200
    Re: [PATCH v4 4/4] misc: atmel_tclib: get and use slow clock Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2015-10-05 19:00 +0200
      Re: [PATCH v4 4/4] misc: atmel_tclib: get and use slow clock Thierry Reding <thierry.reding@gmail.com> - 2015-10-06 09:40 +0200

#1239666 — Re: [PATCH v4 4/4] misc: atmel_tclib: get and use slow clock

FromThierry Reding <thierry.reding@gmail.com>
Date2015-10-05 17:30 +0200
SubjectRe: [PATCH v4 4/4] misc: atmel_tclib: get and use slow clock
Message-ID<qgfSa-2Pr-25@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

On Sun, Aug 16, 2015 at 11:23:46AM +0200, Alexandre Belloni wrote:
> From: Boris Brezillon <boris.brezillon@free-electrons.com>
> 
> Commit dca1a4b5ff6e ("clk: at91: keep slow clk enabled to prevent system
> hang") added a workaround for the slow clock as it is not properly handled
> by its users.
> 
> Get and use the slow clock as it is necessary for the timer counters.
> 
> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> ---
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: linux-pwm@vger.kernel.org
>  drivers/clocksource/tcb_clksrc.c | 10 +++++++++-
>  drivers/misc/atmel_tclib.c       |  4 ++++
>  drivers/pwm/pwm-atmel-tcb.c      | 26 +++++++++++++++++++-------
>  include/linux/atmel_tc.h         |  1 +
>  4 files changed, 33 insertions(+), 8 deletions(-)

This doesn't seem to apply anymore. Does it need to be rebased? Or do I
need some other patch as a dependency?

Thierry

[toc] | [next] | [standalone]


#1239746

FromAlexandre Belloni <alexandre.belloni@free-electrons.com>
Date2015-10-05 19:00 +0200
Message-ID<qghhf-4Jy-5@gated-at.bofh.it>
In reply to#1239666
On 05/10/2015 at 17:27:06 +0200, Thierry Reding wrote :
> On Sun, Aug 16, 2015 at 11:23:46AM +0200, Alexandre Belloni wrote:
> > From: Boris Brezillon <boris.brezillon@free-electrons.com>
> > 
> > Commit dca1a4b5ff6e ("clk: at91: keep slow clk enabled to prevent system
> > hang") added a workaround for the slow clock as it is not properly handled
> > by its users.
> > 
> > Get and use the slow clock as it is necessary for the timer counters.
> > 
> > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > ---
> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> > Cc: Thierry Reding <thierry.reding@gmail.com>
> > Cc: linux-pwm@vger.kernel.org
> >  drivers/clocksource/tcb_clksrc.c | 10 +++++++++-
> >  drivers/misc/atmel_tclib.c       |  4 ++++
> >  drivers/pwm/pwm-atmel-tcb.c      | 26 +++++++++++++++++++-------
> >  include/linux/atmel_tc.h         |  1 +
> >  4 files changed, 33 insertions(+), 8 deletions(-)
> 
> This doesn't seem to apply anymore. Does it need to be rebased? Or do I
> need some other patch as a dependency?
> 

It will apply cleanly after [PATCH v4 3/4] clocksource: tcb_clksrc: fix
setup_clkevents error path

It already has Daniel's ack so you can also take it if you want. Else,
we can take it through at91.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
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/

[toc] | [prev] | [next] | [standalone]


#1240209

FromThierry Reding <thierry.reding@gmail.com>
Date2015-10-06 09:40 +0200
Message-ID<qgv0S-7Vp-1@gated-at.bofh.it>
In reply to#1239746

[Multipart message — attachments visible in raw view] — view raw

On Mon, Oct 05, 2015 at 06:58:16PM +0200, Alexandre Belloni wrote:
> On 05/10/2015 at 17:27:06 +0200, Thierry Reding wrote :
> > On Sun, Aug 16, 2015 at 11:23:46AM +0200, Alexandre Belloni wrote:
> > > From: Boris Brezillon <boris.brezillon@free-electrons.com>
> > > 
> > > Commit dca1a4b5ff6e ("clk: at91: keep slow clk enabled to prevent system
> > > hang") added a workaround for the slow clock as it is not properly handled
> > > by its users.
> > > 
> > > Get and use the slow clock as it is necessary for the timer counters.
> > > 
> > > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > ---
> > > Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> > > Cc: Thierry Reding <thierry.reding@gmail.com>
> > > Cc: linux-pwm@vger.kernel.org
> > >  drivers/clocksource/tcb_clksrc.c | 10 +++++++++-
> > >  drivers/misc/atmel_tclib.c       |  4 ++++
> > >  drivers/pwm/pwm-atmel-tcb.c      | 26 +++++++++++++++++++-------
> > >  include/linux/atmel_tc.h         |  1 +
> > >  4 files changed, 33 insertions(+), 8 deletions(-)
> > 
> > This doesn't seem to apply anymore. Does it need to be rebased? Or do I
> > need some other patch as a dependency?
> > 
> 
> It will apply cleanly after [PATCH v4 3/4] clocksource: tcb_clksrc: fix
> setup_clkevents error path
> 
> It already has Daniel's ack so you can also take it if you want. Else,
> we can take it through at91.

Might be best to take it all into the at91 tree, so:

Acked-by: Thierry Reding <thierry.reding@gmail.com>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web