From: Jon Hunter <jon-hunter@ti.com>
To: Ezequiel Garcia <elezegarcia@gmail.com>
Cc: Mark Jackson <mpfj-list@mimc.co.uk>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
Afzal Mohammed <afzal@ti.com>
Subject: Re: DT GPMC SRAM and NOR flash support ?
Date: Fri, 1 Mar 2013 16:23:22 -0600 [thread overview]
Message-ID: <51312A5A.8020609@ti.com> (raw)
In-Reply-To: <CALF0-+WqTwCErUdbTg-Spp6Z6BGXfZzPepgcAVTJwqup8mSdoQ@mail.gmail.com>
On 03/01/2013 03:42 PM, Ezequiel Garcia wrote:
> On Fri, Mar 1, 2013 at 6:08 PM, Ezequiel Garcia <elezegarcia@gmail.com> wrote:
>> Hi Jon,
>>
>> Yet more questions :-) See below...
>>
>> On Wed, Feb 13, 2013 at 7:07 PM, Jon Hunter <jon-hunter@ti.com> wrote:
>>>
>>>
>>> You don't need this extra entry if you add "simple-bus" to
>>> the gpmc node compatible string.
>>>
>>> + gpmc: gpmc@6800a000 {
>>> + compatible = "ti,omap2420-gpmc", "simple-bus";
>>> + ti,hwmods = "gpmc";
>>> + reg = <0x6800a000 0x1000>;
>>> + interrupts = <20>;
>>> +
>>
>> This "simple-bus" trick is great and very clean!
>>
>> I'm working on a new driver, similar to gpmc, called device-bus for Marvell SoC.
>> One remarkable difference is that I need to *setup* the address space
>> (sort of allocate the address region) for a child device, before I can
>> access it.
>>
>> Using simple-bus is a very clean solution. However I'm facing a strange issue:
>> the child (physmap driver) probe() is being called *before* the parent
>> (device-bus driver)
>> probe(), and so the flash device address space cannot get accessed because
>> it hasn't been setup yet.
>>
>
> Ok, now I think I understand that my problem has nothing to do with simple-bus,
> but instead related to the driver probing order.
>
> The physmap driver is just being probed before the device bus driver, and so it
> won't work.
>
> Now: is there any clean solution?
I can't say I understand why if it is a child device. I have just posted
a series this week to enable NOR support for OMAP with DT [1].
Cheers
Jon
[1] http://permalink.gmane.org/gmane.linux.ports.arm.omap/94378
next prev parent reply other threads:[~2013-03-01 22:23 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-01 16:56 DT GPMC SRAM and NOR flash support ? Mark Jackson
2013-02-01 17:12 ` Jon Hunter
2013-02-01 19:39 ` Mark Jackson
2013-02-05 16:16 ` Mark Jackson
2013-02-05 16:35 ` Jon Hunter
2013-02-05 16:48 ` Mark Jackson
2013-02-05 17:08 ` Jon Hunter
2013-02-07 9:51 ` Mark Jackson
2013-02-07 10:34 ` Mark Jackson
2013-02-09 0:43 ` Jon Hunter
2013-02-09 13:27 ` Ezequiel Garcia
2013-02-11 22:21 ` Jon Hunter
2013-02-12 14:36 ` Ezequiel Garcia
2013-02-13 21:07 ` Jon Hunter
2013-02-14 10:18 ` Ezequiel Garcia
2013-02-15 7:42 ` Jon Hunter
2013-03-01 21:08 ` Ezequiel Garcia
2013-03-01 21:42 ` Ezequiel Garcia
2013-03-01 22:23 ` Jon Hunter [this message]
2013-02-06 5:07 ` Mohammed, Afzal
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=51312A5A.8020609@ti.com \
--to=jon-hunter@ti.com \
--cc=afzal@ti.com \
--cc=elezegarcia@gmail.com \
--cc=linux-omap@vger.kernel.org \
--cc=mpfj-list@mimc.co.uk \
/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).