Thread (269 messages) 269 messages, 18 authors, 2014-11-11

[linux-sunxi] Re: [PATCH 4/4] simplefb: add clock handling code

From: broonie@kernel.org (Mark Brown)
Date: 2014-10-01 17:17:04
Also in: linux-fbdev

On Wed, Oct 01, 2014 at 02:48:53PM +0200, Thierry Reding wrote:
On Wed, Oct 01, 2014 at 01:20:08PM +0100, Mark Brown wrote:
quoted
I think you're setting constraints on the implementation you want to see
which make it unworkable but I don't think those constraints are needed.
You're starting from the position that the DT needs to be updated without
the bootloader
No, what I'm saying is that what the simplefb driver expects and what
the bootloader sets up may diverge as resource drivers are added to the
kernel. The DT /could/ be updated without the bootloader. You may only
be able to replace the DTB but not the bootloader on a given platform.
Sure, but doing that and also having the bootloader write part of the DT
from scratch with no cooperation from the rest of the DT doesn't seem
like the way to robustness.
quoted
               and that the DT must not contain any hint of simplefb as
shipped separately.
Well, I don't think it should because it describes the same resources
that the device tree node for the real device already describes. But
perhaps this is one of the cases where duplication isn't all that bad?
If we were worried about this wecould also do it by referring to
those nodes and saying "get all the resources these things need" rather
than duplicating the references (this might make it easier to work out
when the system is ready to hand off to the real drivers).
quoted
                    That's never going to work well as far as I can see
but doesn't seem like an ABI stability issue, or at least not a
reasonable one.
It would work well under the assumption that the kernel wouldn't be
touching any of the resources that simplefb depends on. If that's not a
reasonable assumption then I think we can't make simplefb work the way
it's currently written.
I can't see how that's reasonable unless the kernel has some way of
figuring out what it shouldn't be touching.
quoted
Either the bootloader needs to be updated along with the DT
I thought we had decided that this was one of the big no-nos. But
perhaps I'm misremembering.
It makes things more fragile so it's not desirable, no.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20141001/f3dc3840/attachment.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help