All of lore.kernel.org
 help / color / mirror / Atom feed
From: iivanov@mm-sol.com (Ivan T. Ivanov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V2 1/1] ARM: dts: imx7d: add cortex-a7 coresight component
Date: Tue, 02 Jun 2015 15:15:47 +0300	[thread overview]
Message-ID: <1433247347.6844.25.camel@mm-sol.com> (raw)
In-Reply-To: <20150602105407.GB2067@n2100.arm.linux.org.uk>


On Tue, 2015-06-02 at 11:54 +0100, Russell King - ARM Linux wrote:
> On Tue, Jun 02, 2015 at 12:26:48PM +0300, Ivan T. Ivanov wrote:
> > Hi,
> > 
> > On Tue, 2015-06-02 at 04:54 +0800, Frank.Li at freescale.com wrote:
> > > +
> > > +???????etm at 3007d000 {
> > > +               compatible = "arm,coresight-etm3x", "arm,primecell";
> > > +               reg = <0x3007d000 0x1000>;
> > > +
> > > +               /* System will hang if added nosmp in kernel command line
> > > +                       * without arm,primecell-periphid because amba bus try to
> > > +                       * read id and core1 power off at this time.
> > > +                       */
> > > +               arm,primecell-periphid =  <0xbb956>;
> > 
> > I do have similar problem, but in my case it is APB clock which is
> > missing during device creation time. And I don't know how to fix it.
> > 
> > Adding Russell King as he's the maintainer of the arm,primecell bus.
> 
> If it was possible to postpone reading the ID until probe time, then
> that's how I'd suggest to fix this problem, but unfortunately it isn't.
> We need to know the device's ID by the time the bus ->match function
> is called.

I have tried to read device ID again and again on every bus ->match until
succeed. with hope that driver which provide APB clock will be probed
in between, and this solve problem for me. But as Stephen Boyd point me,
this way uevent used for module auto loading will be broken. So no joy.

> Short of modifying the driver core (which isn't going to be popular)
> I don't see an easy solution to this problem.
> 

Thanks,
Ivan

  reply	other threads:[~2015-06-02 12:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-01 20:54 [PATCH V2 1/1] ARM: dts: imx7d: add cortex-a7 coresight component Frank.Li at freescale.com
2015-06-02  9:26 ` Ivan T. Ivanov
2015-06-02 10:54   ` Russell King - ARM Linux
2015-06-02 12:15     ` Ivan T. Ivanov [this message]
2015-06-02 14:36 ` Mathieu Poirier
2015-06-02 15:58   ` Zhi Li
2015-06-02 18:17     ` Mathieu Poirier
2015-06-28 13:39 ` Shawn Guo
2015-06-29 14:36   ` Mathieu Poirier

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=1433247347.6844.25.camel@mm-sol.com \
    --to=iivanov@mm-sol.com \
    --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.