On Tue, Oct 06, 2015 at 12:38:19AM -0700, Frank Rowand wrote:
> On 10/5/2015 9:56 PM, David Gibson wrote:
> > On Fri, Oct 02, 2015 at 09:52:48PM -0700, Frank Rowand wrote:
> >> From: Frank Rowand <frank.rowand@xxxxxxxxxxxxxx>
> >>
> >> Proof of concept patch.
> >>
> >> Annotates input source file and line number of nodes and properties
> >> as comments in output .dts file when --annotate flag is supplied.
>
> < snip >
>
> > The two branches here aren't quite consistent - the first doesn't
> > include the '/', the second does. You could either change the second
> > to just &@3, or use &@$ for the first.
>
> < snip >
>
> > This one doesn't include the name/label again
>
> < snip>
>
> >
> > .. and this one does.
> >
> > Looking at all of these it's probably going to be simplest not to
> > include the label/name (i.e. just use the srcpos from the nodedef).
>
> This also shows how much I need to create a better "make check" test
> suite to exercise all of the relevant paths (as is on my todo list).
>
> I've been trying to capture the line that the name is on, since it
> might be different than the line that the "{" is on, but maybe I'm
> just trying too hard and should accept the simpler solution. The
> result will be obvious to the person using the output.
Hm, yes I see your point, it probably is slightly better to include
the name. I'm fine with that, as long as it's done consistently.
> I'll fix this all up after I get back from the conference.
>
> Thanks for checking the patch over at that level of detail!
>
> -frank
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
Attachment:
signature.asc
Description: PGP signature
