All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonas Jensen <jonas.jensen@gmail.com>
To: Mark Rutland <mark.rutland@arm.com>
Cc: "linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"arm@kernel.org" <arm@kernel.org>,
	"mturquette@linaro.org" <mturquette@linaro.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3] clk: add MOXA ART SoCs clock driver
Date: Thu, 18 Jul 2013 16:25:27 +0200	[thread overview]
Message-ID: <CACmBeS2rBTtyfzUgB41zXkDC5-DRcvArgeyjbqHQHXO5+QLvmA@mail.gmail.com> (raw)
In-Reply-To: <20130718135650.GD29153@e106331-lin.cambridge.arm.com>

On 18 July 2013 15:56, Mark Rutland <mark.rutland@arm.com> wrote:
> When you say look up the apb-clock node, I assume they find it via their
> clocks property with of_clk_get or similar, rather than scanning the
> tree for a "moxart,moxa-apb-clock" node specifically?

Yes, all use of_clk_get(node, 0), which is fortunate because e.g.
clocksource is already merged with linaro's clockevent drivers tree.

That should mean I can get away with only changing the DT files :)

> Given the above, I think it would make more sense to have core-clock
> registered, and have it passed to the devices currently passed
> apb-clock.

OK, I'll return with an updated version.


Best regards,
Jonas

WARNING: multiple messages have this Message-ID (diff)
From: jonas.jensen@gmail.com (Jonas Jensen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3] clk: add MOXA ART SoCs clock driver
Date: Thu, 18 Jul 2013 16:25:27 +0200	[thread overview]
Message-ID: <CACmBeS2rBTtyfzUgB41zXkDC5-DRcvArgeyjbqHQHXO5+QLvmA@mail.gmail.com> (raw)
In-Reply-To: <20130718135650.GD29153@e106331-lin.cambridge.arm.com>

On 18 July 2013 15:56, Mark Rutland <mark.rutland@arm.com> wrote:
> When you say look up the apb-clock node, I assume they find it via their
> clocks property with of_clk_get or similar, rather than scanning the
> tree for a "moxart,moxa-apb-clock" node specifically?

Yes, all use of_clk_get(node, 0), which is fortunate because e.g.
clocksource is already merged with linaro's clockevent drivers tree.

That should mean I can get away with only changing the DT files :)

> Given the above, I think it would make more sense to have core-clock
> registered, and have it passed to the devices currently passed
> apb-clock.

OK, I'll return with an updated version.


Best regards,
Jonas

  reply	other threads:[~2013-07-18 14:25 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-27 14:03 [PATCH] clk: add MOXA ART SoCs clock driver Jonas Jensen
2013-06-27 14:03 ` Jonas Jensen
2013-07-04 13:08 ` [PATCH v2] " Jonas Jensen
2013-07-04 13:08   ` Jonas Jensen
2013-07-17 13:23   ` [PATCH v3] " Jonas Jensen
2013-07-17 13:23     ` Jonas Jensen
2013-07-18  9:50     ` Mark Rutland
2013-07-18  9:50       ` Mark Rutland
2013-07-18 10:36       ` Jonas Jensen
2013-07-18 10:36         ` Jonas Jensen
2013-07-18 11:02         ` Mark Rutland
2013-07-18 11:02           ` Mark Rutland
2013-07-18 11:55           ` Jonas Jensen
2013-07-18 11:55             ` Jonas Jensen
2013-07-18 13:56             ` Mark Rutland
2013-07-18 13:56               ` Mark Rutland
2013-07-18 14:25               ` Jonas Jensen [this message]
2013-07-18 14:25                 ` Jonas Jensen
2013-07-19  8:07     ` [PATCH v4] " Jonas Jensen
2013-07-19  8:07       ` Jonas Jensen
2013-07-19  8:17       ` [PATCH v5] " Jonas Jensen
2013-07-19  8:17         ` Jonas Jensen
2013-07-22  9:21         ` Mark Rutland
2013-07-22  9:21           ` Mark Rutland
2013-07-23  8:09           ` Tomasz Figa
2013-07-23  8:09             ` Tomasz Figa
2013-07-26 22:32             ` Mike Turquette
2013-07-26 22:32               ` Mike Turquette
2013-07-29  9:44         ` [PATCH v6] " Jonas Jensen
2013-07-29  9:44           ` Jonas Jensen
2013-10-07  4:47           ` Mike Turquette
2013-10-07  4:47             ` Mike Turquette
2013-10-09 14:54           ` [PATCH v7] " Jonas Jensen
2013-10-09 14:54             ` Jonas Jensen
2013-11-01 18:13             ` Sylwester Nawrocki
2013-11-01 18:13               ` Sylwester Nawrocki
2013-12-09 15:16             ` [PATCH v8] " Jonas Jensen
2013-12-09 15:16               ` Jonas Jensen
2014-01-17 15:03               ` [PATCH v9] " Jonas Jensen
2014-01-17 15:03                 ` Jonas Jensen
2014-01-17 15:17                 ` Sudeep Holla
2014-01-17 15:17                   ` Sudeep Holla
2014-01-17 15:17                   ` Sudeep Holla
2014-01-21 12:44                 ` [PATCH v10] " Jonas Jensen
2014-01-21 12:44                   ` Jonas Jensen
2014-01-21 12:44                   ` Jonas Jensen
2014-01-27 10:20                   ` Mark Rutland
2014-01-27 10:20                     ` Mark Rutland
2014-01-27 10:20                     ` Mark Rutland
2014-01-28 11:09                   ` [PATCH v11] " Jonas Jensen
2014-01-28 11:09                     ` Jonas Jensen
2014-01-28 11:09                     ` Jonas Jensen
2014-03-13 20:27                     ` Mike Turquette
2014-03-13 20:27                       ` Mike Turquette

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=CACmBeS2rBTtyfzUgB41zXkDC5-DRcvArgeyjbqHQHXO5+QLvmA@mail.gmail.com \
    --to=jonas.jensen@gmail.com \
    --cc=arm@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=mturquette@linaro.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.