From nobody Mon Apr 6 19:44:45 2026 Received: from mail-pf1-f170.google.com (mail-pf1-f170.google.com [209.85.210.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F2A2E339870 for ; Fri, 3 Apr 2026 08:06:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.170 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775203616; cv=none; b=DOIUdelGnd+wqZx+ZlVwv+Wc4J6sdIxZNJmMgpwu7JAE+mKQihJMRneYgcl8nCBLownEqQJ9bTHZtQEKFzXHoYN9WjBWhNpUqHCxIo3bJDELrPCxPXUkLP1C8+BsS5KIk55rEynLf6gZfokfNqR9ZiXKFCwVU+F5SBKb1eioe0Q= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775203616; c=relaxed/simple; bh=m1KMSyTgNoTIjs0yfbu3OFUljzgRa7Od+ZK3UUmuaKc=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=PqYjOhunHpziWcOak9nEDpyHe/HTn4IYNS8w/3w9iz8Z1WsVt3lQtQOJ6zSIp41CHKf8WwQZ54alT59THCrRsFRE8GYaI8BPEg7l1n+dAUseDeA6zx63o3hBWNIGmodgUCk+I1ROzWTM0zYsYBNeo+GTpY1mw2BvyDF2CTZ+xmY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=HL2M4Okh; arc=none smtp.client-ip=209.85.210.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="HL2M4Okh" Received: by mail-pf1-f170.google.com with SMTP id d2e1a72fcca58-82cef263bedso827867b3a.0 for ; Fri, 03 Apr 2026 01:06:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775203610; x=1775808410; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=tTtWp6TMoxddZliac+SyJr7XIDYz1K5UHMGnb/koc6g=; b=HL2M4OkhlDKV39yQmT4yAMAASyq+ODmQqgc3DM1mpBYVZjXSm5Iu71C0ScB5wHEVBu 5zNsSXkSIB0bQxIVRe/EDguWIilGFZ0BqKB08FBVLs7xuZfMeKArTDomYexy88e+/+g6 Ny4j41jO+P5icKlwU8mCrqnnw62/W+DEf+rcmN7Mhc2ugwjMnhzF3aNEjlhbIOYFEjSc GAEWGQhatMpILB82qhbEYghesei97+bE6U+LLfNnpcNM5KXlNQlbtYFjQ5/NvBUvrEF0 pAU6IMeJnUsgcpMTJpe2ueb3Ko74FtfL1rJEfniO4cmyVon0R/gLjvy2/xChXJZSQGle OS9Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775203610; x=1775808410; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=tTtWp6TMoxddZliac+SyJr7XIDYz1K5UHMGnb/koc6g=; b=Ln6sV/Musp9R1dzXF2qGUQa+xpeAjPjoIvhwqigd07EdrNaeyfuGKkRpHwv9W4wEzx LDAyVnvnm7/u17UJPPVlDbGuvDbOzef/KFqUrnC9Gmr6mgKbbZCwfQlzXkYtmHANMvRP 5UZX7FoOf1beI0CWNGIfgn2py44+jGWjyen/ns9M7j+5t35ZkgALuzmCiR+gFjgEzg1B syourRC82iAEpm/4TkVFOHiOsYaEIGvFhERCHpMl76wWeXq5tlfThHfBIjDEDzNQQ9/y zUB00cPjRTxaV1BxMsk9jcxctFOTYA2EBJdyAufvctFAzZ5a1Fb6qpqmRSS4cCAgg4kw aMcA== X-Forwarded-Encrypted: i=1; AJvYcCWba8aY95+wsAiwrZqGVBYrOGvwULBajDCx+4vMXzY/u9GRRfmLbYkPqdwtGn3WhExNCsHr8X+d0VaXads=@vger.kernel.org X-Gm-Message-State: AOJu0Yx4mJ+3VlVdCbND0a/Amd5Squ80JZ6ES0OjIG/HrKmIsKn5xTeG 5/qcGC8vEQb08z1VYTottIA7hTIS2x6o9RSUhEo1g3DNhxXYDNL1OgQP X-Gm-Gg: AeBDiesJsWPq3Ep/yRDam5l3SYK2Gaohsh2lQ4s2PfxR7OrOBaaEGsTysYV7lHBVTK5 /ilhTLwyIPpntI7JLavtXqMAWkowtKnndA/6hkwrqm5sW8SJqDwYwL0q94B0Vlc5LAMBfag1D80 jiWainBGEGhnT+P2uam3XK4TCG4pCxYO73svDvs0QPpsfubjPU1kj1yCcrlj5gcpXP6/J1VsW5U 6+1VAykNO3E38Ryw/wEdYBiFPbdKyVAOVns40dVjPy+Mj6TRsgX367rEwt7oy1V0RdDUv5Z6gkx gT0pETWxZQBaADGIeRa9FG2A/BWOlRrSjqqf822SpZOgh7mFcj1Mp22RR4cBjm9dDgazFDMuFOE DaQBtpenpgFFWKFKb/NY273JpE6o0Bp+UYrZtwsbANyGxr0XfoU+20yS9Q2GC4hY52w+j8urdy2 hsw+vXD3aDZGw8TD7jS+zCYzPMJziXFHpveNfUbWSEroAhiVy8q69DSBvVxqrAtM9oYFF43kneN g== X-Received: by 2002:a05:6a00:2d15:b0:82c:9c47:fefb with SMTP id d2e1a72fcca58-82d0da4de31mr2178254b3a.20.1775203610142; Fri, 03 Apr 2026 01:06:50 -0700 (PDT) Received: from debian.ari ([152.58.181.253]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82cf9c6ad8asm5481093b3a.40.2026.04.03.01.06.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Apr 2026 01:06:49 -0700 (PDT) From: Archit Anant To: jic23@kernel.org, dlechner@baylibre.com Cc: lars@metafoo.de, Michael.Hennerich@analog.com, nuno.sa@analog.com, andy@kernel.org, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Archit Anant , Andy Shevchenko Subject: [PATCH v7 5/5] iio: adc: ad799x: convert to fully managed resources and drop remove() Date: Fri, 3 Apr 2026 13:36:14 +0530 Message-Id: <20260403080614.14213-6-architanant5@gmail.com> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20260403080614.14213-1-architanant5@gmail.com> References: <20260403080614.14213-1-architanant5@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Convert the driver's remaining manual resource management to use the devm_ infrastructure, allowing for the complete removal of the ad799x_remove() function and the simplification of the probe error paths. Specifically: - Initialize the mutex using devm_mutex_init() and move it to the top of probe() (before IRQ registration) to prevent a race condition where an interrupt could attempt to take an uninitialized lock. - Use devm_add_action_or_reset() to ensure that the VCC and VREF regulators are disabled safely and in the correct order during driver teardown or probe failure. - Refactor the optional VREF error handling path for better readability. - Convert iio_triggered_buffer_setup() and iio_device_register() to their devm_ variants. Because all resources are now managed by the devm core, the unwinding order is guaranteed to follow the reverse order of allocation. All manual error handling goto labels in ad799x_probe() have been removed. Suggested-by: Jonathan Cameron Suggested-by: David Lechner Suggested-by: Andy Shevchenko Signed-off-by: Archit Anant --- drivers/iio/adc/ad799x.c | 69 ++++++++++++++++------------------------ 1 file changed, 28 insertions(+), 41 deletions(-) diff --git a/drivers/iio/adc/ad799x.c b/drivers/iio/adc/ad799x.c index e37bb64edd2b..9170109d64ef 100644 --- a/drivers/iio/adc/ad799x.c +++ b/drivers/iio/adc/ad799x.c @@ -775,6 +775,11 @@ static const struct ad799x_chip_info ad799x_chip_info_= tbl[] =3D { }, }; =20 +static void ad799x_reg_disable(void *reg) +{ + regulator_disable(reg); +} + static int ad799x_probe(struct i2c_client *client) { struct device *dev =3D &client->dev; @@ -793,6 +798,9 @@ static int ad799x_probe(struct i2c_client *client) st =3D iio_priv(indio_dev); /* this is only used for device removal purposes */ i2c_set_clientdata(client, indio_dev); + ret =3D devm_mutex_init(dev, &st->lock); + if (ret) + return ret; =20 st->id =3D id->driver_data; if (client->irq > 0 && chip_info->irq_config.info) @@ -809,15 +817,19 @@ static int ad799x_probe(struct i2c_client *client) if (ret) return ret; =20 + ret =3D devm_add_action_or_reset(dev, ad799x_reg_disable, st->reg); + if (ret) + return ret; + /* check if an external reference is supplied */ if (chip_info->has_vref) { st->vref =3D devm_regulator_get_optional(dev, "vref"); ret =3D PTR_ERR_OR_ZERO(st->vref); - if (ret) { - if (ret !=3D -ENODEV) - goto error_disable_reg; + if (ret =3D=3D -ENODEV) { st->vref =3D NULL; dev_info(dev, "Using VCC reference voltage\n"); + } else if (ret) { + return ret; } =20 if (st->vref) { @@ -825,10 +837,15 @@ static int ad799x_probe(struct i2c_client *client) extra_config |=3D AD7991_REF_SEL; ret =3D regulator_enable(st->vref); if (ret) - goto error_disable_reg; + return ret; + + ret =3D devm_add_action_or_reset(dev, ad799x_reg_disable, st->vref); + if (ret) + return ret; + ret =3D regulator_get_voltage(st->vref); if (ret < 0) - goto error_disable_vref; + return ret; st->vref_uV =3D ret; } } @@ -836,7 +853,7 @@ static int ad799x_probe(struct i2c_client *client) if (!st->vref) { ret =3D regulator_get_voltage(st->reg); if (ret < 0) - goto error_disable_reg; + return ret; st->vref_uV =3D ret; } =20 @@ -851,12 +868,12 @@ static int ad799x_probe(struct i2c_client *client) =20 ret =3D ad799x_update_config(st, st->chip_config->default_config | extra_= config); if (ret) - goto error_disable_vref; + return ret; =20 - ret =3D iio_triggered_buffer_setup(indio_dev, NULL, + ret =3D devm_iio_triggered_buffer_setup(dev, indio_dev, NULL, &ad799x_trigger_handler, NULL); if (ret) - goto error_disable_vref; + return ret; =20 if (client->irq > 0) { ret =3D devm_request_threaded_irq(dev, @@ -868,39 +885,10 @@ static int ad799x_probe(struct i2c_client *client) client->name, indio_dev); if (ret) - goto error_cleanup_ring; + return ret; } =20 - mutex_init(&st->lock); - - ret =3D iio_device_register(indio_dev); - if (ret) - goto error_cleanup_ring; - - return 0; - -error_cleanup_ring: - iio_triggered_buffer_cleanup(indio_dev); -error_disable_vref: - if (st->vref) - regulator_disable(st->vref); -error_disable_reg: - regulator_disable(st->reg); - - return ret; -} - -static void ad799x_remove(struct i2c_client *client) -{ - struct iio_dev *indio_dev =3D i2c_get_clientdata(client); - struct ad799x_state *st =3D iio_priv(indio_dev); - - iio_device_unregister(indio_dev); - - iio_triggered_buffer_cleanup(indio_dev); - if (st->vref) - regulator_disable(st->vref); - regulator_disable(st->reg); + return devm_iio_device_register(dev, indio_dev); } =20 static int ad799x_suspend(struct device *dev) @@ -970,7 +958,6 @@ static struct i2c_driver ad799x_driver =3D { .pm =3D pm_sleep_ptr(&ad799x_pm_ops), }, .probe =3D ad799x_probe, - .remove =3D ad799x_remove, .id_table =3D ad799x_id, }; module_i2c_driver(ad799x_driver); --=20 2.39.5