[U-Boot] [PATCH v3 016/108] fdt: Show the preprocessed .dts file on error

Bin Meng bmeng.cn at gmail.com
Mon Oct 28 06:47:07 UTC 2019


Hi Simon,

On Mon, Oct 21, 2019 at 11:33 AM Simon Glass <sjg at chromium.org> wrote:
>
> When device-tree compilation fails it is sometimes tricky to see which
> line is broken, since the input file to dtc is a pre-processed version
> of the device tree.
>
> Add a line that points to the file that needs to be checked:
>
> Output is something like this:
>
> Error: arch/x86/dts/u-boot.dtsi:137.14-15 syntax error

To me this already provides enough information for people to look at.
I don't think we need another line to duplicate the report.

> FATAL ERROR: Unable to parse input tree
> Check /tmp/b/chromebook_coral/arch/x86/dts/.chromebook_coral.dtb.pre.tmp

Another reason is that if the error does not happen in the dtsi file
but the main dts file, the line you added here is a duplicates of the
error message coming from the dtc.

>    for errors
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v3:
> - Update example error message to better show the intended purpose
>
> Changes in v2: None
>
>  scripts/Makefile.lib | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>

Regards,
Bin


More information about the U-Boot mailing list