devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Jon Hunter <jon-hunter@ti.com>
Cc: Pratik Patel <pratikp@codeaurora.org>,
	linux-arm-msm@vger.kernel.org,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>,
	Will Deacon <will.deacon@arm.com>,
	linux-kernel@vger.kernel.org,
	"magnus.p.persson@stericsson.com"
	<magnus.p.persson@stericsson.com>,
	"david.rusling@linaro.org" <david.rusling@linaro.org>,
	"arve@android.com" <arve@android.com>,
	"dsaxena@linaro.org" <dsaxena@linaro.org>,
	"john.stultz@linaro.org" <john.stultz@linaro.org>,
	"d-deao@ti.com" <d-deao@ti.com>,
	"christian.bejram@stericsson.com"
	<christian.bejram@stericsson.com>,
	"devicetree-discuss@lists.ozlabs.org"
	<devicetree-discuss@lists.ozlabs.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: CoreSight framework and drivers
Date: Thu, 20 Dec 2012 23:40:11 +0000	[thread overview]
Message-ID: <20121220234011.GE14363@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <50D3972E.4030607@ti.com>

On Thu, Dec 20, 2012 at 04:54:38PM -0600, Jon Hunter wrote:
> On 12/20/2012 01:51 PM, Pratik Patel wrote:
> > Ok, so are you referring to making CoreSight devices register
> > with AMBA bus instead of platform bus keeping everything else
> > intact?
> 
> Yes exactly. However, please note I am not saying that we should do
> this, and I asking what direction does the community want us to take
> here? Platform bus or AMBA bus?

One of the issues which worries me about mixing peripheral drivers on
random different buses is... what happens when we end up with a SoC
which gates the APB clock at bus level (there are SoCs which gate the
APB clock at peripheral level.)  In other words, an APB bus only gets
clocked upon request.

We can deal with that with the infrastructure we have in place in the
AMBA bus layer, but not with the platform bus - we'd have to teach the
platform bus driver about the special APB clock instead of having it
handled primerily at the bus layer.

At least the coresight ETM peripherals make use of the APB bus.  They
have a whole pile of registers on the APB bus, and they have the
primecell IDs stored in the last words of the peripheral, again just
like the other primecell devices we have using the AMBA bus layer.

What I'd say is... why stick it on a different bus type from the other
peripherals which might make things harder in the future?

  reply	other threads:[~2012-12-20 23:40 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-18 19:19 CoreSight framework and drivers pratikp
     [not found] ` <1355858365-11849-1-git-send-email-pratikp-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2012-12-18 19:19   ` [RFC 1/8] coresight: add CoreSight core layer framework pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2012-12-18 19:19   ` [RFC 2/8] coresight: add CoreSight TMC driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2013-02-21 14:20     ` Robert MARKLUND
2012-12-18 19:19   ` [RFC 3/8] coresight: add CoreSight TPIU driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2012-12-18 19:19   ` [RFC 4/8] coresight: add CoreSight ETB driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
     [not found]     ` <1355858365-11849-5-git-send-email-pratikp-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2012-12-20 17:49       ` Jon Hunter
     [not found]         ` <50D34F8F.5050304-l0cyMroinI0@public.gmane.org>
2012-12-20 19:54           ` Pratik Patel
2012-12-18 19:19   ` [RFC 5/8] coresight: add CoreSight Funnel driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2012-12-18 19:19   ` [RFC 6/8] coresight: add CoreSight Replicator driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2012-12-18 19:19   ` [RFC 7/8] coresight: add CoreSight STM driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2012-12-18 19:19   ` [RFC 8/8] coresight: add CoreSight ETM driver pratikp-sgV2jX0FEOL9JmXXK+q4OQ
2012-12-19 11:23 ` CoreSight framework and drivers Will Deacon
     [not found]   ` <20121219112314.GA26329-MRww78TxoiP5vMa5CHWGZ34zcgK1vI+I0E9HWUfgJXw@public.gmane.org>
2012-12-19 17:03     ` Jon Hunter
2012-12-19 21:24       ` Pratik Patel
2012-12-20 17:46         ` Jon Hunter
2012-12-20 19:51           ` Pratik Patel
2012-12-20 20:16             ` Jean Pihet
2012-12-21 22:12               ` Pratik Patel
2012-12-20 22:54             ` Jon Hunter
2012-12-20 23:40               ` Russell King - ARM Linux [this message]
2012-12-21 22:17                 ` Pratik Patel
2012-12-21 22:18               ` Pratik Patel
     [not found]                 ` <20121221221828.GA2658-9H3kiKr51/JHGpnEYu0o6d07Fnf2crXX0E9HWUfgJXw@public.gmane.org>
2012-12-23 11:32                   ` Will Deacon
2013-01-03 18:06                     ` Pratik Patel
     [not found]                       ` <20130103180643.GA30277-9H3kiKr51/JHGpnEYu0o6d07Fnf2crXX0E9HWUfgJXw@public.gmane.org>
2013-01-07 11:58                         ` Will Deacon
2013-01-16  0:14                           ` Pratik Patel
2013-01-17 10:55                             ` Will Deacon
2013-01-02 20:00                   ` Jon Hunter
2013-01-03 19:32                     ` Pratik Patel
2012-12-19 21:06   ` Pratik Patel
2013-02-21 14:32 ` Robert MARKLUND

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=20121220234011.GE14363@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=arve@android.com \
    --cc=christian.bejram@stericsson.com \
    --cc=d-deao@ti.com \
    --cc=david.rusling@linaro.org \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=dsaxena@linaro.org \
    --cc=john.stultz@linaro.org \
    --cc=jon-hunter@ti.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=magnus.p.persson@stericsson.com \
    --cc=pratikp@codeaurora.org \
    --cc=will.deacon@arm.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).