Re: [PATCH] of: add dtc annotations functionality to dtx_diff

2018-12-11 Thread Frank Rowand
Hi Rob,


On 11/26/18 4:01 AM, frowand.l...@gmail.com wrote:
> From: Frank Rowand 
> 
> 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 
> ---
> 
> This feature depends upon commit 5667e7ef9a9a ("annotations: add the
> annotation functionality") in the dtc git repository.  To use the

Ping. Your subsequent patch on 11/30, "[PATCH 1/3] scripts/dtc:
Update to upstream version v1.4.7-57-gf267e674d145" adds the required
commit.

-Frank


> 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
>  
>  
> +  --annotatesynonym for -T
> -f   print full dts in diff (--unified=9)
> -h   synonym for --help
> -helpsynonym 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
> 



Re: [PATCH] of: add dtc annotations functionality to dtx_diff

2018-11-26 Thread Julia Lawall



On Mon, 26 Nov 2018, frowand.l...@gmail.com wrote:

> From: Frank Rowand 
>
> 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 

Tested-by: Julia Lawall 

> ---
>
> 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
>
>
> +  --annotatesynonym for -T
> -f   print full dts in diff (--unified=9)
> -h   synonym for --help
> -helpsynonym 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 
>
>


Re: [PATCH] of: add dtc annotations functionality to dtx_diff

2018-11-26 Thread Julia Lawall



On Mon, 26 Nov 2018, frowand.l...@gmail.com wrote:

> From: Frank Rowand 
>
> 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 

Tested-by: Julia Lawall 

> ---
>
> 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
>
>
> +  --annotatesynonym for -T
> -f   print full dts in diff (--unified=9)
> -h   synonym for --help
> -helpsynonym 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 
>
>


[PATCH] of: add dtc annotations functionality to dtx_diff

2018-11-26 Thread frowand . list
From: Frank Rowand 

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 
---

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
 
 
+  --annotatesynonym for -T
-f   print full dts in diff (--unified=9)
-h   synonym for --help
-helpsynonym 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 



[PATCH] of: add dtc annotations functionality to dtx_diff

2018-11-26 Thread frowand . list
From: Frank Rowand 

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 
---

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
 
 
+  --annotatesynonym for -T
-f   print full dts in diff (--unified=9)
-h   synonym for --help
-helpsynonym 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