Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933892AbbGUWTT (ORCPT ); Tue, 21 Jul 2015 18:19:19 -0400 Received: from bh-25.webhostbox.net ([208.91.199.152]:59341 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932984AbbGUWTQ (ORCPT ); Tue, 21 Jul 2015 18:19:16 -0400 Message-ID: <55AEC50E.3030202@roeck-us.net> Date: Tue, 21 Jul 2015 15:17:50 -0700 From: Guenter Roeck User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: =?UTF-8?B?VXdlIEtsZWluZS1Lw7ZuaWc=?= CC: Greg Kroah-Hartman , Jiri Slaby , linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, linux-cris-kernel@axis.com, Jesper Nilsson Subject: Re: [PATCH -next] serial: etraxfs-uart: Update gpiod API References: <1437510892-30376-1-git-send-email-linux@roeck-us.net> <20150721210115.GJ9981@pengutronix.de> In-Reply-To: <20150721210115.GJ9981@pengutronix.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Authenticated_sender: linux@roeck-us.net X-OutGoing-Spam-Status: No, score=-1.0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: authenticated_id: linux@roeck-us.net X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2282 Lines: 59 On 07/21/2015 02:01 PM, Uwe Kleine-König 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") >> Cc: Uwe Kleine-König >> Signed-off-by: Guenter Roeck > We discussed this driver while I worked on updating the gpiod API. I > don't care much about this driver, but if you want to do something about > the build failure, I'd prefer: > > diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig > index 76e65b714471..3a6a7c235f09 100644 > --- a/drivers/tty/serial/Kconfig > +++ b/drivers/tty/serial/Kconfig > @@ -1066,6 +1066,7 @@ config SERIAL_VT8500_CONSOLE > config SERIAL_ETRAXFS > bool "ETRAX FS serial port support" > depends on ETRAX_ARCH_V32 && OF > + depends on BROKEN > select SERIAL_CORE > > config SERIAL_ETRAXFS_CONSOLE > > Best regards > Uwe > Uwe, I understand the value of BROKEN, but I do not think it would be appropriate here. It would be appropriate for a driver which is really broken, not one that doesn't build anymore because an API it uses was updated and the driver code wasn't updated for some reason. With your suggested patch I would no longer be able to run my qemu tests for this architecture, which I would not entirely be happy about. You may not care, but maybe others do. Makes me wonder: It used to be that API changes were handled by those making the API changes, not by those responsible for the code using the API. Has this changed recently, ie is it now acceptable to not update all callers of a modified API (on purpose, as it looks like it was done here) ? Thanks, Guenter -- 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/