[Pdbg] [PATCH 03/14] libpdbg/target: add a release operation
Nicholas Piggin
npiggin at gmail.com
Thu Apr 12 18:48:02 AEST 2018
On Thu, 12 Apr 2018 14:10:57 +1000
Alistair Popple <alistair at popple.id.au> wrote:
> On Monday, 9 April 2018 5:35:11 PM AEST Nicholas Piggin wrote:
> > This will be used to release special wakeup.
>
> Looks good, I've been meaning to add release as it's also needed for the FSI
> backend. One comment below though.
>
> > +void pdbg_target_release(void)
> > +{
> > + struct dt_node *dn;
> > +
> > + dt_for_each_node(dt_root, dn)
>
> What order does this happen in? We need the release to happen depth first as a
> child node should be released before its parent is.
I think it's a depth first, but something like pre-order whereas you
want post-order? Anyway I get the idea. I could try to make a new
iterator for it.
Thanks,
Nick
More information about the Pdbg
mailing list