From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9718AC433E2 for ; Sat, 6 Jun 2020 15:13:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 764C3207BC for ; Sat, 6 Jun 2020 15:13:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1591456386; bh=yw1R8hsp7uFM58J1K/Vd5IWGmoqQ1p7Bcgwbu0B9E3U=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=S5isI2gGDNvnoKGvxAJE37vpinQcImyjyxby9dH8FTxPmQYT7qhUFRNhPEHin7Jim L71oP6CQobhafEh/3D4mJAfIF+7scjSOeV93X+cWbxQOJZJSK6WPrLpiTCDw8TuaQE GVAwsS1ud122yvJ9AmEhzlQ1YdDTtqyVCNxth15k= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728713AbgFFPNF (ORCPT ); Sat, 6 Jun 2020 11:13:05 -0400 Received: from mail.kernel.org ([198.145.29.99]:39188 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726089AbgFFPNE (ORCPT ); Sat, 6 Jun 2020 11:13:04 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 323572073E; Sat, 6 Jun 2020 15:13:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1591456383; bh=yw1R8hsp7uFM58J1K/Vd5IWGmoqQ1p7Bcgwbu0B9E3U=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=j2rrAzD0CWobFE0w5a7qiZVAG6DfRTL4AdpL+yIaESjoUAmlXS6vRmebV/dWskS/G wuFO3nbntLLZ5nhD/DX3HbzlaF/Y322UCUPyrQEYoM1QEdDgA1gLn05BVWWULdOQGg px9BV1/SwCrkrrhlIOksrbXxsZfkbDVWC/ccYVxg= Date: Sat, 6 Jun 2020 16:12:59 +0100 From: Jonathan Cameron To: "Ardelean, Alexandru" Cc: "hslester96@gmail.com" , "pmeerw@pmeerw.net" , "linux-iio@vger.kernel.org" , "lars@metafoo.de" , "linux-kernel@vger.kernel.org" , "paul.cercueil@analog.com" , "Hennerich, Michael" , "knaack.h@gmx.de" Subject: Re: [PATCH v3] iio: amplifiers: ad8366: Change devm_gpiod_get() to optional and add the missed check Message-ID: <20200606161259.146eb221@archlinux> In-Reply-To: <36f033adf9b53adb289e7569ea5fca647c0dbb7a.camel@analog.com> References: <20200603092610.1424489-1-hslester96@gmail.com> <36f033adf9b53adb289e7569ea5fca647c0dbb7a.camel@analog.com> X-Mailer: Claws Mail 3.17.5 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 3 Jun 2020 10:19:12 +0000 "Ardelean, Alexandru" wrote: > On Wed, 2020-06-03 at 17:26 +0800, Chuhong Yuan wrote: > > Since if there is no GPIO, nothing happens, replace devm_gpiod_get() > > with devm_gpiod_get_optional(). > > Also add IS_ERR() to fix the missing-check bug. > > > > Acked-by: Alexandru Ardelean Applied with layout tweaked a bit as per Markus' comment which made sense in this particular case. I also tweaked the description to say "missing-check" warning given as we've established the failure to check for an error wasn't a problem as such. On that basis I thought about dropping the fixes tag, but it's 'sort' of a fix so I'll leave it there. Not sending this the fast path however as is not urgent (shout if you disagree!) Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > > > Fixes: cee211f4e5a0 ("iio: amplifiers: ad8366: Add support for the ADA4961 > > DGA") > > Signed-off-by: Chuhong Yuan > > --- > > Changes in v3: > > - Change devm_gpiod_get() to optional. > > - Modify description. > > > > drivers/iio/amplifiers/ad8366.c | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/iio/amplifiers/ad8366.c b/drivers/iio/amplifiers/ad8366.c > > index 62167b87caea..8819e8997f76 100644 > > --- a/drivers/iio/amplifiers/ad8366.c > > +++ b/drivers/iio/amplifiers/ad8366.c > > @@ -262,8 +262,12 @@ static int ad8366_probe(struct spi_device *spi) > > case ID_ADA4961: > > case ID_ADL5240: > > case ID_HMC1119: > > - st->reset_gpio = devm_gpiod_get(&spi->dev, "reset", > > + st->reset_gpio = devm_gpiod_get_optional(&spi->dev, "reset", > > GPIOD_OUT_HIGH); > > + if (IS_ERR(st->reset_gpio)) { > > + ret = PTR_ERR(st->reset_gpio); > > + goto error_disable_reg; > > + } > > indio_dev->channels = ada4961_channels; > > indio_dev->num_channels = ARRAY_SIZE(ada4961_channels); > > break;