From: Arnd Bergmann <arnd@arndb.de>
To: Bjorn Andersson <bjorn@kryo.se>
Cc: Jassi Brar <jaswinder.singh@linaro.org>,
Jassi Brar <jassisinghbrar@gmail.com>,
lkml <linux-kernel@vger.kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"Anna, Suman" <s-anna@ti.com>,
Loic Pallardy <loic.pallardy@st.com>,
LeyFoon Tan <lftan.linux@gmail.com>,
Craig McGeachie <slapdau@yahoo.com.au>,
Courtney Cavin <courtney.cavin@sonymobile.com>,
Rob Herring <robherring2@gmail.com>,
Josh Cartwright <joshc@codeaurora.org>,
Linus Walleij <linus.walleij@linaro.org>,
Kumar Gala <galak@codeaurora.org>,
"ks.giri@samsung.com" <ks.giri@samsung.com>
Subject: Re: [PATCHv5 2/4] mailbox: Introduce framework for mailbox
Date: Mon, 19 May 2014 22:01:36 +0200 [thread overview]
Message-ID: <5908149.zGas12hmDq@wuerfel> (raw)
In-Reply-To: <CAJAp7OjUEkG9Wxg_ZhgAqRWQ9QE5sCnUHXW=Qsggq=OSprWLCQ@mail.gmail.com>
On Monday 19 May 2014 12:55:51 Bjorn Andersson wrote:
> On Mon, May 19, 2014 at 11:03 AM, Jassi Brar <jaswinder.singh@linaro.org> wrote:
> > On 19 May 2014 18:38, Arnd Bergmann <arnd@arndb.de> wrote:
> >> On Friday 16 May 2014 19:03:25 Jassi Brar wrote:
> [...]
> >> Do you have an example for a platform like that? I'd expect that normally
> >> we can have a boot loader that sets up the system timer to work good
> >> enough for us to get into normal driver initialization.
> >>
> > My platform. We choose to keep bootloader to the minimum and make
> > kernel not depend upon any goodies provided.
> > Second, which I don't think can be helped by a bootloader, the remote
> > master has gate & rate control of clocks to peripheral IPs. The
> > clk-api driver simply maps Linux requests onto mailbox commands. So
> > the mailbox is needed as early as CLK_OF_DECLARE (when kernel reads
> > the rate of every registered clock). Any suggestions?
>
> Hi Jessi
>
> On the newer Qualcomm platform the root clocks are controlled by a "resource
> power manager" system; all clocks and regulators accesses are done over a fifo
> based communication link, over shared memory, with dependencies on e.g.
> hwspinlocks.
>
> There's nothing strange with this, you just have to bring up (probe) all those
> drivers before you can initialize the clock driver so that you can
> initialize any
> peripheral device drivers. That's what you have EPROBE_DEFER for.
>
>
> Even in your sparsely initialized system, you must have enough clocks to
> bring up the mailbox driver, so that you can bring up your clock driver and
> from that you're good to initialize the rest.
The one dependency that is really hard to avoid is usually the clocksource.
If you can't enable that early, you don't get into the normal driver probe
and need some hack instead.
Arnd
next prev parent reply other threads:[~2014-05-19 20:02 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-15 6:08 [PATCHv5 0/4] Common Mailbox Framework Jassi Brar
2014-05-15 6:10 ` [PATCHv5 1/4] mailbox: rename pl320-ipc specific mailbox.h Jassi Brar
2014-05-15 6:11 ` [PATCHv5 2/4] mailbox: Introduce framework for mailbox Jassi Brar
2014-05-15 14:27 ` Arnd Bergmann
2014-05-16 13:33 ` Jassi Brar
2014-05-19 13:08 ` Arnd Bergmann
2014-05-19 18:03 ` Jassi Brar
2014-05-19 19:55 ` Bjorn Andersson
2014-05-19 20:01 ` Arnd Bergmann [this message]
2014-05-20 18:11 ` Jassi Brar
2014-05-29 15:43 ` Matt Porter
2014-05-30 5:31 ` Jassi Brar
2014-06-02 15:14 ` Matt Porter
2014-06-02 17:11 ` Jassi Brar
[not found] ` <CABb+yY3ZYqtT+R0PwZDtpW0O0SsbxTyiYmXaseZHoj4Nr6UBPQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-02 22:04 ` Matt Porter
2014-06-02 22:04 ` Matt Porter
2014-06-03 9:35 ` Sudeep Holla
2014-06-03 10:21 ` Jassi Brar
2014-06-03 15:06 ` Sudeep Holla
[not found] ` <CAJe_ZheA_2PwzFGwx2rdba0oVsAKRnwK02XE-8nPY6K5NKpdTw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-05 11:12 ` Matt Porter
2014-06-05 11:12 ` Matt Porter
2014-06-05 11:39 ` Jassi Brar
2014-06-05 11:39 ` Jassi Brar
2014-06-11 16:07 ` Mark Brown
2014-05-21 17:27 ` Mark Brown
2014-05-21 18:14 ` Arnd Bergmann
2014-05-28 4:20 ` Jassi Brar
2014-05-28 15:50 ` Suman Anna
2014-06-11 15:37 ` Mark Brown
2014-05-15 6:11 ` [PATCHv5 3/4] mailbox: Fix TX completion init Jassi Brar
2014-05-15 6:12 ` [PATCHv5 4/4] mailbox: Fix deleteing poll timer Jassi Brar
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=5908149.zGas12hmDq@wuerfel \
--to=arnd@arndb.de \
--cc=bjorn@kryo.se \
--cc=courtney.cavin@sonymobile.com \
--cc=galak@codeaurora.org \
--cc=gregkh@linuxfoundation.org \
--cc=jassisinghbrar@gmail.com \
--cc=jaswinder.singh@linaro.org \
--cc=joshc@codeaurora.org \
--cc=ks.giri@samsung.com \
--cc=lftan.linux@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=loic.pallardy@st.com \
--cc=robherring2@gmail.com \
--cc=s-anna@ti.com \
--cc=slapdau@yahoo.com.au \
/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.