From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jassi Brar Subject: Re: [PATCH v3 5/8] gpio: Add Fujitsu MB86S7x GPIO driver Date: Fri, 9 Jan 2015 18:50:34 +0530 Message-ID: References: <1420802369-3840-1-git-send-email-Vincent.Yang@tw.fujitsu.com> <1420803212-4350-1-git-send-email-Vincent.Yang@tw.fujitsu.com> <20150109125243.GM12302@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150109125243.GM12302@n2100.arm.linux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Russell King - ARM Linux Cc: Mark Rutland , Alexandre Courbot , Andy Green , =?UTF-8?Q?Pawe=C5=82_Moll?= , Arnd Bergmann , Devicetree List , Olof Johansson , Vincent Yang , Linus Walleij , "ijc+devicetree@hellion.org.uk" , Vincent Yang , "linux-gpio@vger.kernel.org" , Rob Herring , Patch Tracking , Kumar Gala , Grant Likely , Tetsuya Nuriya , "linux-arm-kernel@lists.infradead.org" List-Id: linux-gpio@vger.kernel.org On 9 January 2015 at 18:22, Russell King - ARM Linux wrote: > On Fri, Jan 09, 2015 at 07:33:32PM +0800, Vincent Yang wrote: >> +static int mb86s70_gpio_remove(struct platform_device *pdev) >> +{ >> + struct mb86s70_gpio_chip *gchip = platform_get_drvdata(pdev); >> + >> + gpiochip_remove(&gchip->gc); > > This doesn't disable and unprepare the clock. > Will do. Thanks, Jassi From mboxrd@z Thu Jan 1 00:00:00 1970 From: jaswinder.singh@linaro.org (Jassi Brar) Date: Fri, 9 Jan 2015 18:50:34 +0530 Subject: [PATCH v3 5/8] gpio: Add Fujitsu MB86S7x GPIO driver In-Reply-To: <20150109125243.GM12302@n2100.arm.linux.org.uk> References: <1420802369-3840-1-git-send-email-Vincent.Yang@tw.fujitsu.com> <1420803212-4350-1-git-send-email-Vincent.Yang@tw.fujitsu.com> <20150109125243.GM12302@n2100.arm.linux.org.uk> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 9 January 2015 at 18:22, Russell King - ARM Linux wrote: > On Fri, Jan 09, 2015 at 07:33:32PM +0800, Vincent Yang wrote: >> +static int mb86s70_gpio_remove(struct platform_device *pdev) >> +{ >> + struct mb86s70_gpio_chip *gchip = platform_get_drvdata(pdev); >> + >> + gpiochip_remove(&gchip->gc); > > This doesn't disable and unprepare the clock. > Will do. Thanks, Jassi