Thread (39 messages) 39 messages, 4 authors, 2015-09-15

[PATCH v3 02/18] of/platform: add of_platform_probe

From: broonie@kernel.org (Mark Brown)
Date: 2015-09-11 09:58:22
Also in: linux-acpi, linux-devicetree, lkml

On Mon, Sep 07, 2015 at 02:31:06PM +0200, Tomeu Vizoso wrote:
On 11 August 2015 at 11:37, Tomeu Vizoso [off-list ref] wrote:
quoted
On 7 August 2015 at 14:19, Mark Brown [off-list ref] wrote:
quoted
quoted
This sounds like it's going to break in the case where we have MFDs that
represent their functions in DT (not a pattern I'm a fan of but it's a
thing people do).  We'll walk back to the platform device for the MFD
function, try to probe it and then give up.  Perhaps that's good enough
anyway but it's not clear to me why we don't just try every parent we
find?
quoted
Agreed. In the attempt at probing dependencies before a device is
probed, I considered that a device's parent is also a dependency and
Actually I'm not sure how we could probe the ascendants on demand, as
currently the parent's device lock is taken when probing so trying to
probe a sibling from within a probe callback will cause a deadlock.
How do silbilings come into this?  There is an issue there but it's
going to happen anyway.
AFAICS this is only needed for USB interface devices and this
behaviour could be limited to them, but I don't like much assuming
that no USB device will ever have a dependency on a sibling (though
that probably won't happen ever).
I don't see the connection with USB here, sorry - my initial thought was
about MFDs?
-------------- 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/20150911/f70cb0ee/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