Re: [PATCH 2/5] annotations: Add position information to various calls

[Frank Rowand <frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

On 01/08/18 14:29, Rob Herring wrote:
> On Mon, Jan 8, 2018 at 7:36 AM, Julia Lawall <Julia.Lawall-L2FTfq7BK8M@public.gmane.org> wrote:
>> Builds on a patch proposed by Frank Rowand:
>>
>> https://www.mail-archive.com/devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org/msg00372.html
> 
> Better to just put the details here and keep Frank's S-o-b if
> significant parts he wrote.

If you keep my S-o-b, please update it with my current email:

Signed-off-by: Frank Rowand <frank.rowand-7U/KSKJipcs@public.gmane.org>

> 
>> Added NULL position argument in a few new places in dtc-parser.tab.c (1)
>> and livetree.c (3).
> 
> I think this patch should probably be split into what's needed to keep
> track of the source position and the annotating dts files part. Mainly
> I say this because I'm interested in changing the checks to print the
> dts position (if available) instead of the output file.
> 
> The changes themselves look good to me.
> 
> Rob
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree-compiler" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>