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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E81D2F513E3 for ; Thu, 5 Mar 2026 21:46:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Cc:To:In-Reply-To:References :Message-Id:Content-Transfer-Encoding:Content-Type:MIME-Version:Subject:Date: From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=AH50fw9QTtdpPbt41EGIzTuypwuiZyQQHwG7B4PGWqg=; b=emAvAJAuFvwa7kpVZB2BuJfRWu 4bOlb/TkpZpuWNPZwCAp20h4TDi+0/GeLDDcKWzEE+L3RwPXllXEpsYyziC1YiIoTUb5hmvgkfkAN Emqtd4yJONFuxdC02fqghd3WYHDHh0x/c/e94Eor9wRiXcvV4FWiRD7c9L+LMEnTIgKEv9SauJbbK gpEjdbyJlffnuNZLh3WKE+/4MsMoyr3NclCXOC56izl3MAvP5Jflrd3t0F/J2VO0QtrpYo1jyZHgJ RekIY+mZP6RldwN0pT+Bj48Etn0tcaHhdQbOm21u5FF405lbjoxli6JbQYTAlBWYozwhfSPTsddtT 9f3mTjjQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vyGWh-00000002bJN-1ZQf; Thu, 05 Mar 2026 21:46:35 +0000 Received: from mx0b-0031df01.pphosted.com ([205.220.180.131]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vyGWd-00000002bCU-0ZTW for linux-mediatek@lists.infradead.org; Thu, 05 Mar 2026 21:46:32 +0000 Received: from pps.filterd (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 625KmwXM1810101 for ; Thu, 5 Mar 2026 21:46:30 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= AH50fw9QTtdpPbt41EGIzTuypwuiZyQQHwG7B4PGWqg=; b=WFSxdaViIVPBNC+X 9bqAQeOpl9vGXSUUioaKpINGyFSaBj1czQ0kQkCk8MAiZaNmcE7wHL7+fV2sSzvQ 6+x6p53n81WDEWEXo0Nv8GVfz/Bq6O3kyYYQTrRB5tyQjxBDwKMW+0/9IXoQz2Y6 d4BwliZ31/MTimweCNxGsU/+OJhDIJazRYQ9e0jeS9F19EGb9kvDa4WRvV90AKH6 2VVrNet5cmq2/CEVpkYGWwoO80ZawsyODj/naisE4DyT4Ns4KkPWvMc1KeARrEeT q79E+dOAggE0iGVpm+VqfUmpaIxDya2l8W/S/L/z/yRp6GtTChefieGrRO/pVVkG 7QkkbQ== Received: from mail-qk1-f200.google.com (mail-qk1-f200.google.com [209.85.222.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4cqgp407pc-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Thu, 05 Mar 2026 21:46:30 +0000 (GMT) Received: by mail-qk1-f200.google.com with SMTP id af79cd13be357-8cb706313beso1085814085a.3 for ; Thu, 05 Mar 2026 13:46:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1772747189; x=1773351989; darn=lists.infradead.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=AH50fw9QTtdpPbt41EGIzTuypwuiZyQQHwG7B4PGWqg=; b=amfyNzaC9uLpfJ0KzDy9K6n9KWjGSU7CKGFzTtr9eqE8QFV4zKIYJ36vl6GrRwnG8w KspIDNa03CpTrmPTgh5x5oVmwmH4mDXOU4ldlX6H0GBjrtrDeVJLYdCPZvIRoMXj4FGc y6EF0WFZmzrrdoIMpZAWurqqJXlMkTm2m0mpCt4dvs0Ksw+qU1bFqc35D/clyZIeRYLX 2x0MY4H7ikph/Z+74cx239jS5xTDDY/c29QGZCMZp+HV9sn8WtL/p5Pt9MyHJ/9CQpgZ LWwN3TRt43edDc1/KLoyXnyC9k0ZsH4Of2aZCoiIJZa68dhvYu3yODRk6ke541LjPBSV gWFQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772747189; x=1773351989; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=AH50fw9QTtdpPbt41EGIzTuypwuiZyQQHwG7B4PGWqg=; b=fED9eGxbiAXGWqfbKDJ0/YsRuLtE6Xo0jo9Jt8BCGlwm2GfuQiblLRM01qFvVAHM5G XTYkuMWLlciZ60h0He0UL5DaKrzCvpvW3sgNurLbtxpX308QbVqZmMOH1MRD8kOuIiwr xMPQkedcJ5a5pYYZtvIG2MLQi3PK3zrN3BYBUt9Zv4hEqTCo4qI8uPwNAaUhcrOahIs1 7mre5OVyAsiQ/xXVN1r5KDVvBP2iaS8pv/w3bJs4TVnWZhV2OO0oXenMroXck8blw4R9 h2jl2in1mh4XZw+E4Nl9oYc8gtSeJ1h2lsmCczGPyp6/aSNhb9JSQHyuFXW6TW07ayzo dXZg== X-Forwarded-Encrypted: i=1; AJvYcCXNxKjfVP37xN/uOgdg+P2pR/oAlxi7MivKodZrmmWoQWUQNi2kmmrHtttlj8SYyG/Wq3kTXkmKdQoklywOKA==@lists.infradead.org X-Gm-Message-State: AOJu0YwrMxyzJmm2auZC0MGRkDilTAllwwJxZTuD5jczAvnk47lRdxPu zIZS7K+dUucxTPCYSUGIIg7CBGky8ULm+hy8ojs7w32WnRcGdc6gd/EDINiWn1k/GRl5cRIpngZ krRehuPD/jafioH/d3qpyLY24OrNd1PHzZMqY+W4xyElIPIiSGKoghkJJRVeYv4HFBqf7tybi0v M= X-Gm-Gg: ATEYQzwUHttaMmwUj8W0tTd2fRkI9f994DfNF6vtXg3UJ2tcwJClbENTjxtL/2jHN2s 2EP+bXMo2PO/0Vl8yqqODuXu6G5ZGfANE1hC08gD18qCgs3SxVR5ZN79i4NSpyuSmlVy8iEdju3 l2YViwgC4rTBBB5ncnd2XKuzDVtGCaNAKj5DFjf8/AMYkMiHUcRscUvYSBUDHlHo6Ghri8YTeoJ TeR7NdpKYtHs/JB/ayTGyUachBvSeOPoFXR4C2W2Fzj1aBPirNd5xKAnt1RmOsq42PQ7k1U7fK/ GxcJDuXcDuwPFVsmW5LCa1kxtaPfaHZebIjbk9PvghRKuZfLveBIebIyxoNDIdfpg5v7SKTS5dl z40Z+KVh3JUTc6QipiKxq3P30ZhXUFiFTNMDmhuPFnRnB X-Received: by 2002:a05:620a:1726:b0:8cb:4fa5:d47a with SMTP id af79cd13be357-8cd6d494e49mr2329185a.77.1772747189106; Thu, 05 Mar 2026 13:46:29 -0800 (PST) X-Received: by 2002:a05:620a:1726:b0:8cb:4fa5:d47a with SMTP id af79cd13be357-8cd6d494e49mr2326485a.77.1772747188622; Thu, 05 Mar 2026 13:46:28 -0800 (PST) Received: from [127.0.1.1] ([178.197.219.94]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-439ba2a58dasm28571575f8f.27.2026.03.05.13.46.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 05 Mar 2026 13:46:27 -0800 (PST) From: Krzysztof Kozlowski Date: Thu, 05 Mar 2026 22:45:45 +0100 Subject: [PATCH v2 06/10] power: supply: ipaq_micro: Simplify with devm MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260305-workqueue-devm-v2-6-66a38741c652@oss.qualcomm.com> References: <20260305-workqueue-devm-v2-0-66a38741c652@oss.qualcomm.com> In-Reply-To: <20260305-workqueue-devm-v2-0-66a38741c652@oss.qualcomm.com> To: Greg Kroah-Hartman , "Rafael J. Wysocki" , Danilo Krummrich , Jonathan Corbet , Shuah Khan , Tejun Heo , Lai Jiangshan , Tobias Schrammm , Sebastian Reichel , Andy Shevchenko , Dan Carpenter , Krzysztof Kozlowski , Lee Jones , Dzmitry Sankouski , Matthias Brugger , AngeloGioacchino Del Regno , Benson Leung , Tzung-Bi Shih Cc: Matti Vaittinen , driver-core@lists.linux.dev, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Sebastian Reichel , linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, chrome-platform@lists.linux.dev, Krzysztof Kozlowski X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=3021; i=krzysztof.kozlowski@oss.qualcomm.com; h=from:subject:message-id; bh=G2Qaq1IYTDtA8nYq8hb67ozAR45iYXNAROMWtKf03Uw=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBpqfmdS1zMtD2UaoQRfaiq5mLP0QlOTpilY/8TD J4ztu2IWgOJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaan5nQAKCRDBN2bmhouD 13BxD/0YZqo0vRusnBrySJI4j7j0qBvMSBLl9ERMwbH5zZQFXisFq0V0/yim2m6aeuWG3sue54q cQyLvmbMOtlHEjhHsJW0gAP7otcp1O0SfR3ykSEKToa0wZY7+zGXm7wnrXKcTsXhcddZj9Ywk7G k8pxhlb6R/iKHlXjSedxxBM2U3iicVf9s043yF+9ADEZTF2SsoruyLdMui8TCO60ILt39gQjuqx 9inYM98ZePPGSQtHGfvmONO7Gu1D+EvkseMIaO+YIhDzEMAJUAmew+WPrK567ES5pxsJ/GIg9ul ND9JhEpU12ZAB7EKrHhCcxcFQOn6vDEvL5ZJ4rtl/Ts+C8ufpzL1pVEuldmRhFHjG17dv3Mm1hU qG4oztSBnPshts2d6Nc/Y4uayRlpNpeK+7SrP2f/PrWGQdd5KdFo5Iu1uJnwGMsiExW0MLOT2eI 43o7eDO7M+jC3NfI7za2SVUY8POdUyx/ba5BPcXUmJd5RpRiyOgPbEIZeRhYhkxSm6zNITQjBIQ 7tVjG3QlahgAPAcbxe0FiplwIwhYBRcso/cJGY5XD8IFB9T8+OH6y3pOed4UM7DNe3E2zT03P08 8/si+s74GY7hTw3HR+7C9YKGWQeAGDNHcOFKDrfcLEjPqmN4Od/VOYDbfizGIxxbX4sM4VQDDfi JSVXiCJBbZxkykw== X-Developer-Key: i=krzysztof.kozlowski@oss.qualcomm.com; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B X-Proofpoint-ORIG-GUID: A1ZkpFMD6NOaiVzdlPlRsSnllRI37zLY X-Authority-Analysis: v=2.4 cv=LegxKzfi c=1 sm=1 tr=0 ts=69a9f9b6 cx=c_pps a=hnmNkyzTK/kJ09Xio7VxxA==:117 a=gOEeR9iKwsj33Yj5oN/cWg==:17 a=IkcTkHD0fZMA:10 a=Yq5XynenixoA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=ZpdpYltYx_vBUK5n70dp:22 a=EUspDBNiAAAA:8 a=KKAkSRfTAAAA:8 a=PkD0MmKtpR7JdVUpc-MA:9 a=QEXdDO2ut3YA:10 a=PEH46H7Ffwr30OY-TuGO:22 a=cvBusfyB2V15izCimMoJ:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwMzA1MDE4NSBTYWx0ZWRfXxX7Ms20W2Hxs qQ3wxK+2m4Irde4ks76ycjXJGsIWCxglcLEHyBWdD14av5g0jsc0ec4NUtGVtMmjQc+wECyaBJn Ny43hJKsLGWghnQ2j41gIPqG7h02WKhtaKwQa/YkFcf3ZIZ4xJcZArZQLd/g8Mqe05D04banp5u xXWviALMmUpwtotFKMTFi8jvc0tgMzU+NJBq/38Juxrmtdsj16o/9Js77sGxNxY5hWa99TDedir FHXRbDnXPArkCVfLxloZA1+Y1BPv2FojfhwxmcpPhtJZHSot9NnIu+Htc4yIItZtLOSDUpXng2m fZXHUPdIRI29wEFLEsjMxinu+2jXNWI6UQT7lYXb/O9EqmAX/oIimGhQxtWC6Ws9vgR9XJhpamd adCalhW62/ezn9hJYx2nOVw1gOPgSvn+XIZqQBmtMrAlN1z3WNwort2VWTQkYBaNvy0euNzKvE3 dVlg6y8dRj2+vGV7hug== X-Proofpoint-GUID: A1ZkpFMD6NOaiVzdlPlRsSnllRI37zLY X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-03-05_06,2026-03-04_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 impostorscore=0 malwarescore=0 adultscore=0 lowpriorityscore=0 priorityscore=1501 phishscore=0 bulkscore=0 spamscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2602130000 definitions=main-2603050185 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260305_134631_346624_9A54BCEE X-CRM114-Status: GOOD ( 16.95 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Simplify the driver by using devm interfaces, which allow to drop probe() error paths and the remove() callback. Signed-off-by: Krzysztof Kozlowski --- drivers/power/supply/ipaq_micro_battery.c | 50 ++++++++++--------------------- 1 file changed, 16 insertions(+), 34 deletions(-) diff --git a/drivers/power/supply/ipaq_micro_battery.c b/drivers/power/supply/ipaq_micro_battery.c index ff8573a5ca6d..5e3fe3852d0e 100644 --- a/drivers/power/supply/ipaq_micro_battery.c +++ b/drivers/power/supply/ipaq_micro_battery.c @@ -7,6 +7,7 @@ * Author : Linus Walleij */ +#include #include #include #include @@ -232,49 +233,31 @@ static int micro_batt_probe(struct platform_device *pdev) return -ENOMEM; mb->micro = dev_get_drvdata(pdev->dev.parent); - mb->wq = alloc_workqueue("ipaq-battery-wq", - WQ_MEM_RECLAIM | WQ_PERCPU, 0); + mb->wq = devm_alloc_workqueue(&pdev->dev, "ipaq-battery-wq", + WQ_MEM_RECLAIM | WQ_PERCPU, 0); if (!mb->wq) return -ENOMEM; - INIT_DELAYED_WORK(&mb->update, micro_battery_work); + ret = devm_delayed_work_autocancel(&pdev->dev, &mb->update, micro_battery_work); + if (ret) + return ret; + platform_set_drvdata(pdev, mb); queue_delayed_work(mb->wq, &mb->update, 1); - micro_batt_power = power_supply_register(&pdev->dev, - µ_batt_power_desc, NULL); - if (IS_ERR(micro_batt_power)) { - ret = PTR_ERR(micro_batt_power); - goto batt_err; - } + micro_batt_power = devm_power_supply_register(&pdev->dev, + µ_batt_power_desc, + NULL); + if (IS_ERR(micro_batt_power)) + return PTR_ERR(micro_batt_power); - micro_ac_power = power_supply_register(&pdev->dev, - µ_ac_power_desc, NULL); - if (IS_ERR(micro_ac_power)) { - ret = PTR_ERR(micro_ac_power); - goto ac_err; - } + micro_ac_power = devm_power_supply_register(&pdev->dev, + µ_ac_power_desc, NULL); + if (IS_ERR(micro_ac_power)) + return PTR_ERR(micro_ac_power); dev_info(&pdev->dev, "iPAQ micro battery driver\n"); return 0; - -ac_err: - power_supply_unregister(micro_batt_power); -batt_err: - cancel_delayed_work_sync(&mb->update); - destroy_workqueue(mb->wq); - return ret; -} - -static void micro_batt_remove(struct platform_device *pdev) - -{ - struct micro_battery *mb = platform_get_drvdata(pdev); - - power_supply_unregister(micro_ac_power); - power_supply_unregister(micro_batt_power); - cancel_delayed_work_sync(&mb->update); - destroy_workqueue(mb->wq); } static int __maybe_unused micro_batt_suspend(struct device *dev) @@ -303,7 +286,6 @@ static struct platform_driver micro_batt_device_driver = { .pm = µ_batt_dev_pm_ops, }, .probe = micro_batt_probe, - .remove = micro_batt_remove, }; module_platform_driver(micro_batt_device_driver); -- 2.51.0