Return-path: Received: from eddie.linux-mips.org ([78.24.191.182]:42858 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932189AbaE3Pt2 (ORCPT ); Fri, 30 May 2014 11:49:28 -0400 Date: Fri, 30 May 2014 17:48:59 +0200 From: Ralf Baechle To: Geert Uytterhoeven Cc: abdoulaye berthe , Linus Walleij , Alexandre Courbot , m@bues.ch, "linux-gpio@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Linux MIPS Mailing List , "linuxppc-dev@lists.ozlabs.org" , Linux-sh list , linux-wireless , patches@opensource.wolfsonmicro.com, "linux-input@vger.kernel.org" , "linux-leds@vger.kernel.org" , Linux Media Mailing List , linux-samsungsoc@vger.kernel.org, spear-devel@list.st.com, platform-driver-x86@vger.kernel.org, "netdev@vger.kernel.org" , driverdevel Subject: Re: [PATCH 2/2] gpio: gpiolib: set gpiochip_remove retval to void Message-ID: <20140530154859.GK17197@linux-mips.org> (sfid-20140530_174946_179538_F8D2B24F) References: <20140530094025.3b78301e@canb.auug.org.au> <1401449454-30895-1-git-send-email-berthe.ab@gmail.com> <1401449454-30895-2-git-send-email-berthe.ab@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-wireless-owner@vger.kernel.org List-ID: On Fri, May 30, 2014 at 01:39:15PM +0200, Geert Uytterhoeven wrote: > > + if (test_bit(FLAG_REQUESTED, &chip->desc[id].flags)) > > + panic("gpio: removing gpiochip with gpios still requested\n"); > > panic? > > Is this likely to happen? And while we're at it - panic() is going to add a \n itself so don't pass a string ending in \n to panic(). Ralf