Received: by 2002:a05:6a10:d5a5:0:0:0:0 with SMTP id gn37csp717728pxb; Thu, 30 Sep 2021 15:58:13 -0700 (PDT) X-Google-Smtp-Source: ABdhPJywZH3afU+DT/Nl4//WTsX0dY4AcOcrgDXYAtu/0cvyAKXuJ1z60f8RIrhzkJsNgJGaA3xT X-Received: by 2002:a63:b00e:: with SMTP id h14mr7012189pgf.135.1633042693509; Thu, 30 Sep 2021 15:58:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633042693; cv=none; d=google.com; s=arc-20160816; b=q1fd/3g207IeKbRERJgd3WeBD07D0T4Vhlj5fkLhNtwD7xurwcrURIkCQ1Cp/y2L6m kUKZcDt79ytsainPIBMkzRYa8TDjhJj8erA6ZSy/Q4NmqQCvsgDeUVTGVtftHA31XTMc A5vGemBy20whSFJmRgeAnvr1LOoxrsJO1ME7Uk5C/rTCn96y8uIcYvJS88uK+X1JEOLT B/tHxn2vIe0BTPOMoQvJBeDQcJ2XJZWyG5AIvA1HMewFdsUVzAhUOUvp0prAKEXsUsoQ s44PBInNO4iwMGanB6fYIDJB9/qTaJMtNXBil2EV8SiUkM79uhoXwRdMa1iwgHtfxR99 fN8w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=NZx052EPHOOevPQyJczIlENjaY5Qs9gObt70oqC8Hkc=; b=vEnKfEF4OTajq6tRso6KQUU8WBQZKlj/lPBQCNKPbdeYNyuwaY8CiLhSm21wQwaXB1 WKYqPjY8RrucKXJQCydZrMnoTOEfp+QJyc8dTE+us3joweTTafXUE4kMdZcD9QCi64e+ 2SKYn3jN2EJvVE+/NaHiEMje875q19/X7AIofqb0v842RYmnem3ccJajQhVVHay7Dn/B 7ixiie2TYz+HfDQPWTgNX6ea3uz4luVZSnUf7cXTJlBMTwxQqUQSYp+d1zTAN0VthSgU 0Zpqags8g3cp/VwNDw/hwU76BWgywa7v0kjZHA38OteaxLjqcfuAr8vWIlzGjz+n3mbJ Sdcg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id l22si4759194pff.260.2021.09.30.15.58.00; Thu, 30 Sep 2021 15:58:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344078AbhI3WEV (ORCPT + 99 others); Thu, 30 Sep 2021 18:04:21 -0400 Received: from relay10.mail.gandi.net ([217.70.178.230]:38109 "EHLO relay10.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229644AbhI3WEU (ORCPT ); Thu, 30 Sep 2021 18:04:20 -0400 Received: (Authenticated sender: alexandre.belloni@bootlin.com) by relay10.mail.gandi.net (Postfix) with ESMTPSA id 7798C240007; Thu, 30 Sep 2021 22:02:35 +0000 (UTC) Date: Fri, 1 Oct 2021 00:02:34 +0200 From: Alexandre Belloni To: Jonathan Cameron Cc: Alexandru Ardelean , linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, Maxime Ripard Subject: Re: [PATCH] iio: adc: nau7802: convert probe to full device-managed Message-ID: References: <20210926154932.3287590-1-aardelean@deviqon.com> <20210930171023.2bf5a414@jic23-huawei> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210930171023.2bf5a414@jic23-huawei> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 30/09/2021 17:10:23+0100, Jonathan Cameron wrote: > On Sun, 26 Sep 2021 18:49:32 +0300 > Alexandru Ardelean wrote: > > > This is a trivial conversion to device-managed functions. > > The mutex_destroy() calls are redundant, as the data will be free'd anyway. > > > > And the IRQ and IIO register functions both have device-managed > > equivalents. > > > > Signed-off-by: Alexandru Ardelean > > +CC Maxime and Alexandre. > > Looks simple enough to me that I won't wait on their replies to queue it up. > Obviously feedback welcome though! > > Applied to the togreg branch of iio.git and pushed out as testing for 0-day > to see if it can find stuff we missed. > This seems good to me too Reviewed-by: Alexandre Belloni > Thanks, > > Jonathan > > > > --- > > drivers/iio/adc/nau7802.c | 50 +++++++-------------------------------- > > 1 file changed, 9 insertions(+), 41 deletions(-) > > > > diff --git a/drivers/iio/adc/nau7802.c b/drivers/iio/adc/nau7802.c > > index bb70b51d25b1..976c235f3079 100644 > > --- a/drivers/iio/adc/nau7802.c > > +++ b/drivers/iio/adc/nau7802.c > > @@ -428,8 +428,6 @@ static int nau7802_probe(struct i2c_client *client, > > > > st = iio_priv(indio_dev); > > > > - i2c_set_clientdata(client, indio_dev); > > - > > indio_dev->name = dev_name(&client->dev); > > indio_dev->modes = INDIO_DIRECT_MODE; > > indio_dev->info = &nau7802_info; > > @@ -495,13 +493,13 @@ static int nau7802_probe(struct i2c_client *client, > > * will enable them back when we will need them.. > > */ > > if (client->irq) { > > - ret = request_threaded_irq(client->irq, > > - NULL, > > - nau7802_eoc_trigger, > > - IRQF_TRIGGER_HIGH | IRQF_ONESHOT | > > - IRQF_NO_AUTOEN, > > - client->dev.driver->name, > > - indio_dev); > > + ret = devm_request_threaded_irq(&client->dev, client->irq, > > + NULL, > > + nau7802_eoc_trigger, > > + IRQF_TRIGGER_HIGH | IRQF_ONESHOT | > > + IRQF_NO_AUTOEN, > > + client->dev.driver->name, > > + indio_dev); > > if (ret) { > > /* > > * What may happen here is that our IRQ controller is > > @@ -526,7 +524,7 @@ static int nau7802_probe(struct i2c_client *client, > > ret = i2c_smbus_write_byte_data(st->client, NAU7802_REG_CTRL2, > > NAU7802_CTRL2_CRS(st->sample_rate)); > > if (ret) > > - goto error_free_irq; > > + return ret; > > } > > > > /* Setup the ADC channels available on the board */ > > @@ -536,36 +534,7 @@ static int nau7802_probe(struct i2c_client *client, > > mutex_init(&st->lock); > > mutex_init(&st->data_lock); > > > > - ret = iio_device_register(indio_dev); > > - if (ret < 0) { > > - dev_err(&client->dev, "Couldn't register the device.\n"); > > - goto error_device_register; > > - } > > - > > - return 0; > > - > > -error_device_register: > > - mutex_destroy(&st->lock); > > - mutex_destroy(&st->data_lock); > > -error_free_irq: > > - if (client->irq) > > - free_irq(client->irq, indio_dev); > > - > > - return ret; > > -} > > - > > -static int nau7802_remove(struct i2c_client *client) > > -{ > > - struct iio_dev *indio_dev = i2c_get_clientdata(client); > > - struct nau7802_state *st = iio_priv(indio_dev); > > - > > - iio_device_unregister(indio_dev); > > - mutex_destroy(&st->lock); > > - mutex_destroy(&st->data_lock); > > - if (client->irq) > > - free_irq(client->irq, indio_dev); > > - > > - return 0; > > + return devm_iio_device_register(&client->dev, indio_dev); > > } > > > > static const struct i2c_device_id nau7802_i2c_id[] = { > > @@ -582,7 +551,6 @@ MODULE_DEVICE_TABLE(of, nau7802_dt_ids); > > > > static struct i2c_driver nau7802_driver = { > > .probe = nau7802_probe, > > - .remove = nau7802_remove, > > .id_table = nau7802_i2c_id, > > .driver = { > > .name = "nau7802", > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com