Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753993AbbG0N7Z (ORCPT ); Mon, 27 Jul 2015 09:59:25 -0400 Received: from metis.ext.pengutronix.de ([92.198.50.35]:44113 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753499AbbG0N7X (ORCPT ); Mon, 27 Jul 2015 09:59:23 -0400 Date: Mon, 27 Jul 2015 15:59:15 +0200 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= To: Niklas Cassel Cc: Linus Walleij , Greg Kroah-Hartman , Niklas Cassel , Guenter Roeck , Jiri Slaby , "linux-serial@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-cris-kernel , Jesper Nilsson Subject: Re: [PATCH -next] serial: etraxfs-uart: Update gpiod API Message-ID: <20150727135915.GL15360@pengutronix.de> References: <1437510892-30376-1-git-send-email-linux@roeck-us.net> <20150723222349.GB9973@kroah.com> <55B63241.7080409@axis.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <55B63241.7080409@axis.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c0 X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1906 Lines: 43 On Mon, Jul 27, 2015 at 03:29:37PM +0200, Niklas Cassel wrote: > On 07/27/2015 03:16 PM, Linus Walleij wrote: > > On Fri, Jul 24, 2015 at 12:23 AM, Greg Kroah-Hartman > > wrote: > >> On Tue, Jul 21, 2015 at 01:34:52PM -0700, Guenter Roeck wrote: > >>> Commit b17d1bf16cc7 ("gpio: make flags mandatory for gpiod_get functions") > >>> makes the flags argument to devm_gpiod_get_optional mandatory but does not > >>> update all users. This results in the following build error. > >>> > >>> drivers/tty/serial/etraxfs-uart.c:933:16: error: > >>> too few arguments to function ‘devm_gpiod_get_optional’ > >>> > >>> Fixes: b17d1bf16cc7 ("gpio: make flags mandatory for gpiod_get functions") > >> > >> This patch isn't in Linus's tree, so whatever tree this commit is in, > >> needs to also take this fix. > > > > OK I'll take it. It's a bit messy with this breakage but it is for > > a good cause. > > > > Niklas: send me your patch when finished. > > I've already sent them to the linux-serial and linux-kernel mailing lists. > > I'm sorry that I didn't think of CC:ing you Linus. > It might be better if you take them, since you got > the devm_gpiod_get_optional patches in your tree. If you use devm_gpiod_get_optional with the additional flags parameter it should already now work for you. So if your change goes in before the gpio change (which only makes the flags parameter mandatory) everything is fine. Not sure this makes it easier though. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/