linux-kernel - Re: [PATCH] dtc: create tool to diff device trees

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 07 Jan 2016 10:40:29 -0800
From:	Frank Rowand <frowand.list@...il.com>
To:	Rob Herring <robh+dt@...nel.org>
CC:	Grant Likely <grant.likely@...aro.org>,
	David Gibson <david@...son.dropbear.id.au>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	Linux Kernel list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] dtc: create tool to diff device trees

On 1/4/2016 1:29 PM, Rob Herring wrote:
> On Thu, Dec 31, 2015 at 3:12 AM, Frank Rowand <frowand.list@...il.com> wrote:
>> From: Frank Rowand <frank.rowand@...ymobile.com>
>>
>> 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.
> 
> In general, I'd like to see some of this move to dtc and the kernel
> dependencies minimized.
> 

I didn't address this comment in my first reply.  Didn't mean to gloss
over it.

I agree with the sentiment, but I'm not sure how to act on it.  I don't
think that dtc should be changed to be Linux specific (but if the dtc
maintainers think differently I would follow their lead).  You did
provide at least one example of minimizing kernel dependencies
(removing the arch specific Makefile dtc flags), but beyond that
I'm not sure what can be done.

On the positive side, we can avoid adding more kernel dependencies.

< snip >

-Frank