From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH 0/4] [v7] pinctrl: qcom: add support for sparse GPIOs Date: Wed, 15 Nov 2017 09:06:15 -0600 Message-ID: <8a0fe5e6-b27c-1e23-b9ec-5fc18a0ae27b@codeaurora.org> References: <1510096056-13765-1-git-send-email-timur@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:36148 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757778AbdKOPGS (ORCPT ); Wed, 15 Nov 2017 10:06:18 -0500 In-Reply-To: Content-Language: en-US Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Linus Walleij , Andy Shevchenko Cc: Varadarajan Narayanan , Neil Armstrong , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, Mika Westerberg , "thierry.reding@gmail.com" , Stephen Boyd , David Brown , Andy Gross , Bjorn Andersson On 11/14/2017 04:03 AM, Linus Walleij wrote: > The more intrusive design is on my request so I will look at it in detail > ASAP. I took a quick look and liked what I saw, I just need to make > sure about the details. Also it'd be nice to have a nod from Björn. > > The main reason for some of the work is ACPI, am I right? Yes. We don't have a nice hierarchical tree that lists the individual pins and their purposes. We just have a dumb list of GPIOs. > Timur could you look quickly at the series posted by Andy Shevchenko > (6 patches prefixed gpiolib: acpi: ) for augmenting ACPI GPIOs in the core? > Especially patch 5 and 6 which introduce the ability to add quirks in the > core. So this is something that I've never really understood with gpiolib-acpi. I don't think this file is used with the pin control driver. We do have GpioInt and GpioIo statements in our GPIO table, but they don't appear to be using the TLMM (our GPIO / pin control device) For example: GpioInt(Level, ActiveHigh, Exclusive, PullNone, , "\\_SB.TCS0.QIC5") {10} The \\_SB.TCS0 device is NOT the TLMM. It's our "irq combiner" which has something to do with performance measurement. I think the I don't think Andy's patch set has anything to do with my patches. My patches just try to register a GPIO device, whereas the gpiolib-acpi patches are about ACPI nodes performing gpio-like operations. > I'd like to get some ARM-based people to look at it to make sure we > those systems can also quirk their GPIOs if need be. My patches aren't about a quirk. All I'm trying to do is registers specific GPIOs rather than one giant block of them. -- Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.