From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH 2/2] ARM: dts: MSM8974: Add pinctrl node Date: Tue, 08 Apr 2014 09:18:44 -0500 Message-ID: <53440544.9040502@codeaurora.org> References: <1391700529-11816-1-git-send-email-iivanov@mm-sol.com> <1391700529-11816-2-git-send-email-iivanov@mm-sol.com> <1396959284.28420.45.camel@iivanov-dev> <5343EC87.700@codeaurora.org> <1396964805.28420.53.camel@iivanov-dev> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1396964805.28420.53.camel@iivanov-dev> Sender: linux-arm-msm-owner@vger.kernel.org To: "Ivan T. Ivanov" Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , devicetree@vger.kernel.org, lkml , linux-arm-msm@vger.kernel.org List-Id: devicetree@vger.kernel.org On 04/08/2014 08:46 AM, Ivan T. Ivanov wrote: >> >This patch adds that same exact information into the device tree. Why >> >are we duplicating that information? Why add it to the device tree when >> >it's already in the driver (and already working). > Probably. It was my natural way of thinking. Pin have a functions. > It is easier if I measure signals to just look at the device > tree file. What are you suggesting? Back in July, Qualcomm submitted a patch that added this information into the device tree: http://marc.info/?t=137185166100003&r=1&w=2 However, this was rejected. Now it appears that this information is again being added to the device tree, but it's being accepted. What's different now? Another problem is that these device tree changes makes it difficult to support ACPI. It's easy to move information between the drivers and the device tree, because they're kept together. It's not so easy with ACPI. I'm trying to add ACPI support to the 8x74 pinctrl driver, but it's a moving target. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by the Linux Foundation.