From: magnus.damm@gmail.com (Magnus Damm)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] clkdev: add support to lookup for early platform device
Date: Thu, 28 Apr 2011 12:44:05 +0900 [thread overview]
Message-ID: <BANLkTi=7FAR-KyL+ZEU4MwZpN6xgHE0hzw@mail.gmail.com> (raw)
In-Reply-To: <20110428032906.GN29103@game.jcrosoft.org>
On Thu, Apr 28, 2011 at 12:29 PM, Jean-Christophe PLAGNIOL-VILLARD
<plagnioj@jcrosoft.com> wrote:
> On 12:35 Thu 28 Apr ? ? , Magnus Damm wrote:
>> On Thu, Apr 28, 2011 at 12:20 PM, Jean-Christophe PLAGNIOL-VILLARD
>> <plagnioj@jcrosoft.com> wrote:
>> >> > > > >
>> >> > > > Are you willfully ignoring the init_name handling in
>> >> > > > early_platform_driver_probe_id() or something?
>> >> > > >
>> >> > > > See a636ee7fb35b731ba2b331f6294e809bb6be09c8 for example.
>> >> > > excatly but the issue is that the slab is not availlable yet
>> >> > > see 06fe53beb636294587d8e94ef83c06cef07c21fd
>> >> > > so init_name is still NULL :(
>> >> > >
>> >> > This is a non-argument until you demonstrate a use case where we have a
>> >> > reasonable expectation for the clock framework to be available and the
>> >> > slab allocator not.
>> >> >
>> >> > If you're thinking about the SH earlyprintk case then I suggest you look
>> >> > at the sh-sci driver to see how we presently deal with the situation
>> >> > there.
>> >> the sh-sci driver expect the clock will enable for him
>> >> which did IIRC Magnus on sh-mobile
>> >> I do not want to do so
>> >>
>> >> I've unfortunatly on at91 I use the early device during the map_io
>>
>> But so do we on SH-Mobile ARM.
>>
>> >> so the slab is available and bootmem either
>> > sorry none of theem is available yet
>>
>> Sounds like you are aiming to use earlyprintk a little bit too early,
>> or perhaps your idea of using clocks that early on is what is causing
>> you trouble?
>>
>> Why don't you move down your earlyprintk setup to a time when you have
>> slabs available? Or for clocks, in the sh-sci driver we don't deal
>> with clocks in the earlyprintk case.
>>
>> I understand you want output as early as possible, but there is always
>> going to be a time when you need to hack up some static serial output
>> code.
> I use it for gpio drivers that are init during map_io
>
> so not yet for earlyprintk
You may have explained this before, but why do you need to deal with
that early on?
Can't you simply setup the clocks statically for the early case?
/ magnus
next prev parent reply other threads:[~2011-04-28 3:44 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-20 14:05 [PATCH] clkdev: add support to lookup for early platform device Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 8:42 ` Russell King - ARM Linux
2011-04-27 10:31 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 10:47 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 15:00 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 22:12 ` Greg KH
2011-04-28 2:09 ` Paul Mundt
2011-04-28 2:45 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 3:08 ` Paul Mundt
2011-04-28 3:17 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 3:20 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 3:35 ` Magnus Damm
2011-04-28 3:29 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 3:44 ` Magnus Damm [this message]
2011-04-28 8:33 ` Russell King - ARM Linux
2011-04-28 9:28 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 3:38 ` Paul Mundt
2011-04-28 4:31 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 4:46 ` Saravana Kannan
2011-04-28 4:50 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 5:12 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 5:47 ` Magnus Damm
2011-04-28 8:35 ` Russell King - ARM Linux
2011-04-28 9:26 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 10:07 ` Russell King - ARM Linux
2011-04-28 11:19 ` Jean-Christophe PLAGNIOL-VILLARD
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='BANLkTi=7FAR-KyL+ZEU4MwZpN6xgHE0hzw@mail.gmail.com' \
--to=magnus.damm@gmail.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 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).