From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934964AbbGHOPp (ORCPT ); Wed, 8 Jul 2015 10:15:45 -0400 Received: from lists.s-osg.org ([54.187.51.154]:58155 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933960AbbGHOPn (ORCPT ); Wed, 8 Jul 2015 10:15:43 -0400 Subject: Re: [PATCH 0/4] ARM: dts: Use labels instead of full paths for Exynos5 pinctrl nodes To: Krzysztof Kozlowski , Kukjin Kim References: <1436333790-18219-1-git-send-email-javier@osg.samsung.com> <559CDEDA.50608@samsung.com> Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: Javier Martinez Canillas Message-ID: <559D308E.3020809@osg.samsung.com> Date: Wed, 8 Jul 2015 07:15:42 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 MIME-Version: 1.0 In-Reply-To: <559CDEDA.50608@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Krzysztof, On 07/08/2015 01:27 AM, Krzysztof Kozlowski wrote: > On 08.07.2015 14:36, Javier Martinez Canillas wrote: >> Hello Kukjin and Krzysztof, >> >> This is a trivial series that changes Exynos5 pinctrl dtsi files to extend >> the pinctrl nodes using labels instead of full paths. >> >> Using labels is less error prone since a misstyped label leads to a build >> error while full paths can lead to the creation of a new node instead of >> overriding the original which makes the mistake harder to detect. >> >> I tested the changes by decompiling the Exynos5* dtbs and comparing before >> and after. The resulting dts were identical modulo some node ordering due >> the pinctrl include change. > > Is the DTB before and after the same? It should not change by re-ordering. > The DTB differ but I'm not that familiar with how dtc represents the FDT to know what is changing in the binary. But as mentioned, the decompiled DTS only changes in which position the pinctrl are defined and both DTS are identical if I do: $ cat decompiled.dts | sort > Best regards, > Krzysztof > -- Javier Martinez Canillas Sr. Linux Kernel Developer Open Source Group Samsung Research America