On 22 June 2018 at 12:48, Auger Eric <eric.au...@redhat.com> wrote: > Hi Peter, > > On 06/22/2018 01:02 PM, Peter Maydell wrote: >> This removes all nodes named "memory" or "memory@*", rather than >> just the ones in the root, right? > > correct > > Is that definitely what we want to >> do? The old code only deleted /memory, and the comment implies >> that we only look in the root. >> >> (I'm wondering if there are DTs that use memory nodes for random >> odd things other than system memory...) > > I don't know the exact use case. Removing the exact /memory@40000000 > node is simpler and does not require 2/3 I think. So if this is what is > expected I can stick to that node.
Well, I think we should remove the wildcard /memory@*, but not /foo/bar/memory@xyz and other things that aren't in the root... thanks -- PMM
