On Fri, 2013-05-31 at 17:31 +0100, Grant Likely wrote:

On Fri, May 31, 2013 at 5:04 PM, Stephen Warren [off-list ref] wrote:

quoted

On 05/31/2013 05:48 AM, Grant Likely wrote:

quoted

---
commit d01dccdcb3ea8233b09efb9c24db9f057fbd3b37
Author: Grant Likely [off-list ref]
Date:   Fri May 31 12:45:18 2013 +0100

    dtc: Suppress cpp linemarker annotations

    DTC isn't able to parse cpp linemarker annotations, so suppress them in
    the cpp output by adding the -P flag to the cpp options.

That's not true; it explicitly does have code to parse the line markers.
I'll have to investigate why it isn't working in this case.

If you apply this patch, then anyone who has switched to #include rther
than /include/ will get incorrect line numbers in dtc error messages.
Admittedly that's a smaller population right now though. Perhaps we
should just do a kernel-wide conversion though.

My mistake. I tested the wrong thing. I've dropped the patch.

FWIW I've also dropped it from my device-tree.git tree.

Ian.