From: Julia Lawall <julia.lawall@lip6.fr>
To: frowand.list@gmail.com
Cc: Rob Herring <robh+dt@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] of: add dtc annotations functionality to dtx_diff
Date: Mon, 26 Nov 2018 13:26:24 +0100 (CET) [thread overview]
Message-ID: <alpine.DEB.2.20.1811261325430.3447@hadrien> (raw)
In-Reply-To: <1543233669-8696-1-git-send-email-frowand.list@gmail.com>
On Mon, 26 Nov 2018, frowand.list@gmail.com wrote:
> From: Frank Rowand <frank.rowand@sony.com>
>
> Add -T and --annotations command line arguments to dtx_diff. These
> arguments will be passed through to dtc. dtc will then add source
> location annotations to its output.
>
> Signed-off-by: Frank Rowand <frank.rowand@sony.com>
Tested-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
>
> This feature depends upon commit 5667e7ef9a9a ("annotations: add the
> annotation functionality") in the dtc git repository. To use the
> new flags before the new version of dtc is imported to the linux
> kernel, download the dtc repository, compile dtc with the make command,
> then add the path of the dtc repository to the shell PATH variable.
>
> scripts/dtc/dtx_diff | 13 ++++++++++++-
> 1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/dtc/dtx_diff b/scripts/dtc/dtx_diff
> index 8c4fbad2055e..0d8572008729 100755
> --- a/scripts/dtc/dtx_diff
> +++ b/scripts/dtc/dtx_diff
> @@ -21,6 +21,7 @@ Usage:
> diff DTx_1 and DTx_2
>
>
> + --annotate synonym for -T
> -f print full dts in diff (--unified=99999)
> -h synonym for --help
> -help synonym for --help
> @@ -28,6 +29,7 @@ Usage:
> -s SRCTREE linux kernel source tree is at path SRCTREE
> (default is current directory)
> -S linux kernel source tree is at root of current git repo
> + -T Annotate output .dts with input source file and line (-T -T for more details)
> -u unsorted, do not sort DTx
>
>
> @@ -174,6 +176,7 @@ compile_to_dts() {
>
> # ----- start of script
>
> +annotate=""
> cmd_diff=0
> diff_flags="-u"
> dtx_file_1=""
> @@ -208,6 +211,14 @@ while [ $# -gt 0 ] ; do
> shift
> ;;
>
> + -T | --annotate )
> + if [ "${annotate}" = "" ] ; then
> + annotate="-T"
> + elif [ "${annotate}" = "-T" ] ; then
> + annotate="-T -T"
> + fi
> + shift
> + ;;
> -u )
> dtc_sort=""
> shift
> @@ -327,7 +338,7 @@ cpp_flags="\
> DTC="\
> ${DTC} \
> -i ${srctree}/scripts/dtc/include-prefixes \
> - -O dts -qq -f ${dtc_sort} -o -"
> + -O dts -qq -f ${dtc_sort} ${annotate} -o -"
>
>
> # ----- do the diff or decompile
> --
> Frank Rowand <frank.rowand@sony.com>
>
>
next prev parent reply other threads:[~2018-11-26 12:26 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-26 12:01 [PATCH] of: add dtc annotations functionality to dtx_diff frowand.list
2018-11-26 12:26 ` Julia Lawall [this message]
2018-12-11 21:58 ` Frank Rowand
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.20.1811261325430.3447@hadrien \
--to=julia.lawall@lip6.fr \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).