From: Viresh Kumar <viresh.kumar@linaro.org>
To: Mark Brown <broonie@kernel.org>
Cc: Liam Girdwood <lgirdwood@gmail.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"Nayak, Rajendra" <rnayak@codeaurora.org>,
Stephen Boyd <sboyd@codeaurora.org>,
Vincent Guittot <vincent.guittot@linaro.org>,
Serge Broslavsky <serge.broslavsky@linaro.org>
Subject: Re: [RFC] regulator: Shared regulators (configured by bootloader)
Date: Mon, 15 May 2017 16:17:13 +0530 [thread overview]
Message-ID: <20170515104713.GY17481@vireshk-i7> (raw)
In-Reply-To: <20170514093031.5pu3x7ugvjhqn34h@sirena.org.uk>
On 14-05-17, 18:30, Mark Brown wrote:
> On Mon, May 08, 2017 at 03:51:02PM +0530, Viresh Kumar wrote:
>
> > I am looking to solve a problem faced by some of the Qualcomm
> > platforms and want your suggestions on how should we fix it. One of my
> > ex-colleague tried to solve [1] this problem but that thread never
> > concluded (and I don't really agree with the solution it offered).
>
> Please engage with the feedback I offered then. I see no point in
> repeating myself here, I'm just going to provide the same feedback as
> before and if I'm going to be ignored (which appears to be the case) it
> doesn't seem like it's worth the effort.
I didn't ignore any of that. Most of the discussions then happened around the
solution which Pingbo tried to implement (boot-protection flag). And that had
obvious flaws as it depended on the state of the kernel to be known
(booting/booted). And that will never work, as you pointed out then, because of
the way kernel works (specially with modules).
You said this in one of the replies [1]:
> I think you need to be looking at some combination of getting the
> devices you're interested in started up early and more precisely
> describing the end result you're trying to achieve. The issues with
> probe deferral do seem related here, it's another symptom of not really
> making any decisions about init ordering and so sometimes making bad
> ones.
I don't think we should depend on the order in which the devices get added. This
is going to break for sure.
I have some idea about the end-result we want to achieve (will take LCD as an
example here):
- The regulator should always fulfill the requirement of the LCD device set from
the bootloader, until a point where the kernel driver has taken over.
- If the LCD kernel driver never turns up, then the regulator shall continue
satisfying the requirements set from the bootloader. Even if that means that
some devices may not get what they request for and perform badly. Someone
needs to fix the LCD driver and make sure it comes up.
I failed to find any other workable solution that anyone may have suggested in
those email threads. Can you please point me to those (if any)?
Thanks.
--
viresh
[1] https://marc.info/?l=linux-kernel&m=146540618625247
next prev parent reply other threads:[~2017-05-15 10:47 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-08 10:21 Viresh Kumar
2017-05-14 9:30 ` Mark Brown
2017-05-15 10:47 ` Viresh Kumar [this message]
2017-06-14 17:13 ` Mark Brown
2017-06-16 9:59 ` Viresh Kumar
2017-06-19 22:21 ` Mark Brown
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=20170515104713.GY17481@vireshk-i7 \
--to=viresh.kumar@linaro.org \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rnayak@codeaurora.org \
--cc=sboyd@codeaurora.org \
--cc=serge.broslavsky@linaro.org \
--cc=vincent.guittot@linaro.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
all inboxes | Powered by JetHome®