All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] clocksource: sh_cmt: 32-bit control register support
Date: Mon, 05 Aug 2013 01:40:55 +0000	[thread overview]
Message-ID: <20130805014055.GC28465@verge.net.au> (raw)
In-Reply-To: <20130804192312.GE10876@quad.lixom.net>

On Sun, Aug 04, 2013 at 12:23:12PM -0700, Olof Johansson wrote:
> On Wed, Jul 24, 2013 at 09:26:55AM +0900, Simon Horman wrote:
> > On Mon, Jul 22, 2013 at 06:04:19AM +0200, Daniel Lezcano wrote:
> > > On 07/19/2013 06:36 AM, Simon Horman wrote:
> > > > From: Magnus Damm <damm@opensource.se>
> > > > 
> > > > Add support for CMT hardware with 32-bit control and counter
> > > > registers, as found on r8a73a4 and r8a7790. To use the CMT
> > > > with 32-bit hardware a second I/O memory resource needs to
> > > > point out the CMSTR register and it needs to be 32 bit wide.
> > > > 
> > > > Signed-off-by: Magnus Damm <damm@opensource.se>
> > > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > > > ---
> > > 
> > > In the future, can you Cc drivers/clocksource maintainers please ?
> > 
> > Yes of course, sorry for that oversight.
> > 
> > > Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> > 
> > Olof, Arnd, please let me know if you wish me to re-send the pull
> > request with Daniel's Reviewed-by added to this patch.
> 
> This patch should just go in through Daniel, there's no reason to take it
> through arm-soc at all, unless you have platform code that relies on it for
> bisectability.

I'll double-check but I am reasonably sure that there are no bisection
issues. If so I'll send it through Daniel as you suggest.

WARNING: multiple messages have this Message-ID (diff)
From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] clocksource: sh_cmt: 32-bit control register support
Date: Mon, 5 Aug 2013 10:40:55 +0900	[thread overview]
Message-ID: <20130805014055.GC28465@verge.net.au> (raw)
In-Reply-To: <20130804192312.GE10876@quad.lixom.net>

On Sun, Aug 04, 2013 at 12:23:12PM -0700, Olof Johansson wrote:
> On Wed, Jul 24, 2013 at 09:26:55AM +0900, Simon Horman wrote:
> > On Mon, Jul 22, 2013 at 06:04:19AM +0200, Daniel Lezcano wrote:
> > > On 07/19/2013 06:36 AM, Simon Horman wrote:
> > > > From: Magnus Damm <damm@opensource.se>
> > > > 
> > > > Add support for CMT hardware with 32-bit control and counter
> > > > registers, as found on r8a73a4 and r8a7790. To use the CMT
> > > > with 32-bit hardware a second I/O memory resource needs to
> > > > point out the CMSTR register and it needs to be 32 bit wide.
> > > > 
> > > > Signed-off-by: Magnus Damm <damm@opensource.se>
> > > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > > > ---
> > > 
> > > In the future, can you Cc drivers/clocksource maintainers please ?
> > 
> > Yes of course, sorry for that oversight.
> > 
> > > Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> > 
> > Olof, Arnd, please let me know if you wish me to re-send the pull
> > request with Daniel's Reviewed-by added to this patch.
> 
> This patch should just go in through Daniel, there's no reason to take it
> through arm-soc at all, unless you have platform code that relies on it for
> bisectability.

I'll double-check but I am reasonably sure that there are no bisection
issues. If so I'll send it through Daniel as you suggest.

  reply	other threads:[~2013-08-05  1:40 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-19  4:35 [GIT] Renesas ARM based clocksource updates for v3.12 Simon Horman
2013-07-19  4:35 ` Simon Horman
2013-07-19  4:36 ` [PATCH] clocksource: sh_cmt: 32-bit control register support Simon Horman
2013-07-19  4:36   ` Simon Horman
2013-07-22  4:04   ` Daniel Lezcano
2013-07-22  4:04     ` Daniel Lezcano
2013-07-24  0:26     ` Simon Horman
2013-07-24  0:26       ` Simon Horman
2013-08-04 19:23       ` Olof Johansson
2013-08-04 19:23         ` Olof Johansson
2013-08-05  1:40         ` Simon Horman [this message]
2013-08-05  1:40           ` Simon Horman
  -- strict thread matches above, loose matches on Subject: below --
2012-12-14  5:55 [PATCH] clocksource: sh_cmt: 32-bit control register access prototype Magnus Damm
2013-06-17  6:40 ` [PATCH] clocksource: sh_cmt: 32-bit control register support Magnus Damm
2013-06-17  6:40   ` Magnus Damm
2013-06-17 18:37   ` Laurent Pinchart
2013-06-17 18:37     ` Laurent Pinchart
2013-06-18  5:39     ` Magnus Damm
2013-06-18  5:39       ` Magnus Damm
2013-06-18 10:35       ` Laurent Pinchart
2013-06-18 10:35         ` Laurent Pinchart
2013-06-18 11:54         ` Magnus Damm
2013-06-18 11:54           ` Magnus Damm
2013-06-18 12:30           ` Laurent Pinchart
2013-06-18 12:30             ` Laurent Pinchart
2013-06-18 13:27             ` Magnus Damm
2013-06-18 13:27               ` Magnus Damm
2013-06-19 12:31               ` Simon Horman
2013-06-19 12:31                 ` Simon Horman
2013-06-19 12:58                 ` Laurent Pinchart
2013-06-19 12:58                   ` Laurent Pinchart
2013-06-20 12:30                   ` Simon Horman
2013-06-20 12:30                     ` Simon Horman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130805014055.GC28465@verge.net.au \
    --to=horms@verge.net.au \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.