[PATCH v4 19/21] drivers/of: Support adding sub-tree
benh at kernel.crashing.org
Thu May 14 17:47:32 AEST 2015
On Thu, 2015-05-14 at 10:34 +0300, Pantelis Antoniou wrote:
> >> What you describe is inserting a bunch of properties and nodes under
> >> a slot’s device node. Reverting the overlay removes them all just fine.
> > Except that still doesn't work for boot time :-)
> > So I would have to do a special case on unplug:
> > if (slot->dt_is_overlay) /* set to false at boot */
> > remove_subtree_myself();
> > else
> > undo_overlay(slot->overlay);
> OK, in that case you do require removal. But in any case it’s the ‘negative’
> of an already applied one, either at boot time or not.
Sort-of, unless we have a way in the overlay to simply specify node
removal statements so we don't have to explicitly remove all properties
(or even all children).
> Modifying the overlay code to apply a ‘negative’ property should do the trick.
> Is that correct?
I would do negatives node and let Linux imply the properties (or even
But yes, that would probably do.
> I will probably need that generation count myself for my PCI use case.
> > However that requires supporting removal of nodes/properties. So I'm
> > tempted to keep that feature on the back burner and go with an ad-hoc
> > interface for PCI for now.
> I see. Bonne chance :)
More information about the Linuxppc-dev