From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH v2 1/3] mfd: upboard: Add UP2 platform controller driver Date: Thu, 25 Oct 2018 16:15:23 +0300 Message-ID: <20181025131523.GY10650@smile.fi.intel.com> References: <20180421085009.28773-1-javier@emutex.com> <1539969334-24577-1-git-send-email-dan@emutex.com> <1539969334-24577-2-git-send-email-dan@emutex.com> <20181025110505.GA4870@dell> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20181025110505.GA4870@dell> Sender: linux-kernel-owner@vger.kernel.org To: Lee Jones Cc: Dan O'Donovan , Linux Kernel Mailing List , Mika Westerberg , "Krogerus, Heikki" , Linus Walleij , Jacek Anaszewski , Pavel Machek , "open list:GPIO SUBSYSTEM" , Linux LED Subsystem , carlos.iglesias@emutex.com, Javier Arteaga List-Id: linux-gpio@vger.kernel.org On Thu, Oct 25, 2018 at 12:05:05PM +0100, Lee Jones wrote: > On Sat, 20 Oct 2018, Andy Shevchenko wrote: > > On Fri, Oct 19, 2018 at 8:26 PM Dan O'Donovan wrote: > > > + ret = upboard_init_gpio(dev); > > > + if (ret) { > > > + if (ret != -EPROBE_DEFER) > > > + dev_err(dev, "failed to init GPIOs: %d", ret); > > > + return ret; > > > + } > > > > I don't know if probe_err() helper is going to be a part of v4.21 > > (which this series targets), it would be good to use it. > > Interesting. What does it do? The code will tell better than I https://lore.kernel.org/lkml/20181017132205.5204-1-a.hajda@samsung.com/ -- With Best Regards, Andy Shevchenko