From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH 2/3] [v3] pinctrl: qcom: disable GPIO groups with no pins Date: Wed, 16 Aug 2017 13:32:44 -0500 Message-ID: <66bfbee6-245d-fddb-f6cb-a2c3c06ffc9b@codeaurora.org> References: <1501179565-26466-1-git-send-email-timur@codeaurora.org> <1501179565-26466-3-git-send-email-timur@codeaurora.org> <7c278367-bf3b-6a26-e31f-ee696cb5534e@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <7c278367-bf3b-6a26-e31f-ee696cb5534e@codeaurora.org> Content-Language: en-US Sender: linux-arm-msm-owner@vger.kernel.org To: Jiandi An , Bjorn Andersson Cc: Linus Walleij , Andy Gross , David Brown , "linux-gpio@vger.kernel.org" , "linux-arm-msm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" List-Id: linux-gpio@vger.kernel.org On 08/16/2017 01:10 PM, Jiandi An wrote: > > Technically the same check added in msm_gpio_irq_mask() and > msm_gpio_irq_unmask() should be added in msm_gpio_irq_ack(), > msm_gpio_irq_set_type(), and msm_gpio_irq_set_wake() if it's registered > with irq domain. I assume that if the GPIO is never unmasked, then msm_gpio_irq_ack() will never be called. msm_gpio_irq_set_type() and msm_gpio_irq_set_wake() might be called, so I can add checks for those functions. I'm hoping that won't be necessary, however. The GPIO and IRQ code is too entangled for me to figure out whether unclaimed GPIOs can still have their interrupts programmed. -- 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.