Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753265AbbFVC0A (ORCPT ); Sun, 21 Jun 2015 22:26:00 -0400 Received: from mail-oi0-f46.google.com ([209.85.218.46]:35367 "EHLO mail-oi0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751674AbbFVCZz (ORCPT ); Sun, 21 Jun 2015 22:25:55 -0400 MIME-Version: 1.0 In-Reply-To: <1434936985.20199.8.camel@ubuntu> References: <1434542383-21314-1-git-send-email-yamada.masahiro@socionext.com> <1434542383-21314-2-git-send-email-yamada.masahiro@socionext.com> <1434936985.20199.8.camel@ubuntu> From: Alexandre Courbot Date: Mon, 22 Jun 2015 11:25:34 +0900 Message-ID: Subject: Re: [PATCH 2/2] gpio: altera: fix return value of altera_gpio_remove() To: Tien Hock Loh Cc: Masahiro Yamada , "linux-gpio@vger.kernel.org" , Linus Walleij , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1516 Lines: 44 On Mon, Jun 22, 2015 at 10:36 AM, Tien Hock Loh wrote: > Sorry I was away from my mail for the past few weeks. > This isn't intentional, should be a bug I overlook. The fix is correct. Interestingly that has never been caught by reviewers despite 10 respins of your series! Thanks for confirming. Reviewed-by: Alexandre Courbot > > On Sun, 2015-06-21 at 16:25 +0900, Alexandre Courbot wrote: >> On Wed, Jun 17, 2015 at 8:59 PM, Masahiro Yamada >> wrote: >> > The remove callback never succeeds, which seems odd. >> > >> > Signed-off-by: Masahiro Yamada >> > --- >> > >> > I wonder why nobody has pointed this out before me. >> > I am suspecting -EIO might be intentional. >> > I hope some Altera guys will give me comments. >> > >> > >> > drivers/gpio/gpio-altera.c | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/drivers/gpio/gpio-altera.c b/drivers/gpio/gpio-altera.c >> > index c653c83..5861550 100644 >> > --- a/drivers/gpio/gpio-altera.c >> > +++ b/drivers/gpio/gpio-altera.c >> > @@ -339,7 +339,7 @@ static int altera_gpio_remove(struct platform_device *pdev) >> > >> > of_mm_gpiochip_remove(&altera_gc->mmchip); >> > >> > - return -EIO; >> > + return 0; >> >> That looks weird indeed. Tien, can you comment on this? > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/