Hi Peter, On 06/22/2018 01:52 PM, Peter Maydell wrote: > 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... OK
Thanks Eric > > thanks > -- PMM >
