From: "Timo Sigurdsson" <public_timo.s@silentcreek.de>
To: maxime.ripard@free-electrons.com
Cc: robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com,
ijc+devicetree@hellion.org.uk, galak@codeaurora.org,
linux@arm.linux.org.uk, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com,
hdegoede@redhat.com, wens@csie.org
Subject: Re: [linux-sunxi] [RFC] ARM: dts: sunxi: Add regulators and board-specific operating points for LeMaker BananaPi
Date: Tue, 28 Jul 2015 16:57:52 +0200 (CEST) [thread overview]
Message-ID: <20150728145752.43F676C81B65@dd34104.kasserver.com> (raw)
In-Reply-To: <20150728125500.GB2564@lukather>
Hi,
Maxime Ripard schrieb am 28.07.2015 14:55:
>> > I plan to submit this for 4.3.
>>
>> Ok, then I guess we can drop my patch.
>
> Please don't.
Ok.
> It was used in mainline, and reverted because it was not stable
> enough.
Well, the explanation given was, it was not stable because of the missing
regulator support. I don't know whether anyboady actually ever reported
it wouldn't run fine at 1008MHz with 1.45V. So the actual point should be
whether we wan't voltages that are out of the specified range in the
official kernel. The consensus seems to be no, with good reasons for that.
So, I won't object this.
> There's a lot of things we do differently in mainline, it's one of
> them. If someone can provide an OPP for 1008MHz that is stable for all
> the boards and within the operating limits of the SoC, I'd be totally
> fine with that, but we didn't find it so far.
>
>> For those who don't want to use that setting, it's easier to
>> limit the maximum in userspace compared to compiling a new device
>> tree blob.
>
> Except that the kernel should not rely on the userspace to be stable
> and harmless for the hardware. It should just work reliably by itself.
Same as above.
Regards,
Timo
next prev parent reply other threads:[~2015-07-28 14:57 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-27 1:28 Timo Sigurdsson
2015-07-27 8:07 ` [linux-sunxi] " Hans de Goede
2015-07-27 12:09 ` public_timo.s
2015-07-27 12:43 ` Hans de Goede
2015-07-28 9:02 ` Timo Sigurdsson
2015-07-28 12:55 ` Maxime Ripard
2015-07-28 14:57 ` Timo Sigurdsson [this message]
2015-07-28 12:49 ` Maxime Ripard
2015-07-28 14:24 ` Hans de Goede
2015-07-28 15:09 ` Timo Sigurdsson
2015-07-28 15:29 ` Hans de Goede
2015-08-02 22:00 ` Timo Sigurdsson
2015-07-28 14:45 ` Timo Sigurdsson
2015-07-27 12:36 ` public_timo.s
2015-07-27 12:54 ` Hans de Goede
[not found] ` <CAGb2v65ApKvrj6K+kw43u=0q6=auTsmQjCXhibgZkW+vd5nDqA@mail.gmail.com>
2015-07-28 9:02 ` Timo Sigurdsson
2015-07-28 12:55 ` Maxime Ripard
2015-07-28 15:01 ` Timo Sigurdsson
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=20150728145752.43F676C81B65@dd34104.kasserver.com \
--to=public_timo.s@silentcreek.de \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=hdegoede@redhat.com \
--cc=ijc+devicetree@hellion.org.uk \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sunxi@googlegroups.com \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=maxime.ripard@free-electrons.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=wens@csie.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
Powered by JetHome