From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753359AbcAOXOY (ORCPT ); Fri, 15 Jan 2016 18:14:24 -0500 Received: from mail-pa0-f65.google.com ([209.85.220.65]:35565 "EHLO mail-pa0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753070AbcAOXOU (ORCPT ); Fri, 15 Jan 2016 18:14:20 -0500 Message-ID: <56997D49.6000203@gmail.com> Date: Fri, 15 Jan 2016 15:14:17 -0800 From: Frank Rowand Reply-To: frowand.list@gmail.com User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 MIME-Version: 1.0 To: Rob Herring CC: Grant Likely , david@gibson.dropbear.id.au, "devicetree@vger.kernel.org" , Linux Kernel list Subject: Re: [PATCH v2] dtc: create tool to diff device trees References: <568EB672.6060300@gmail.com> In-Reply-To: <568EB672.6060300@gmail.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On 1/7/2016 11:03 AM, Frank Rowand wrote: > From: Frank Rowand > > Create script to diff device trees. > > The device tree can be in any of the forms recognized by the dtc compiler: > - source > - binary blob > - file system tree (from /proc/devicetree) > > If the device tree is a source file, then it is pre-processed in the > same way as it would be when built in the linux kernel source tree > before diffing. > > Signed-off-by: Frank Rowand > --- > > Tools to develop and debug device tree are somewhat inadequate. This is a > small step in improving the situation. > > Rationale for and examples of using the script are provided in slides > 1 - 78 of the elce 2015 presentation "Solving Device Tree Issues", > which can be found at: > > http://elinux.org/images/0/04/Dt_debugging_elce_2015_151006_0421.pdf > > (The script was named dtdiff instead of dtx_diff in the presentation.) > > Changes in v2: > - Remove dt-bindings from list of includes in cpp_flags > - Remove arch_dtc_flags, which were generated from the arch specific > dts makefile > - Reformat to 8 character tabs > - compile_to_dts(): added back missing return for binary blob > > > scripts/dtc/dtx_diff | 343 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 343 insertions(+) < snip > Did v2 resolve your issues, or would you like some additional changes? Thanks, Frank