From patchwork Fri Mar 7 23:02:23 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Dr. David Alan Gilbert" X-Patchwork-Id: 871797 Received: from mx.treblig.org (mx.treblig.org [46.235.229.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1B9A019006F; Fri, 7 Mar 2025 23:02:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=46.235.229.95 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741388550; cv=none; b=LI/HaCO4SAicwB2PuJTDEeySURT6oEDE6lfh7pu7HtS/E8X6jUW+H4ODLZBLAQFmViAm4LJ+MTLvINgsfYsTUCJnCbPICyTsaUMFIiBEF33nGAMhgbNTFMYs31w/FEL7HBeIzQTlyWvwgAI0wwhGr9j8Eay6xo9Ug1CZ8mvNBVQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741388550; c=relaxed/simple; bh=/vuVUlwiJC1vlB8rYoBU/EU5I/W3M2F4MRh4xrG4JIo=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=IeCoyMZ5/hP26BAM2wvt2zuo5jVZxWCkQHZomohUMa3cdoFXCUcq0lsU8VSaU0x/SPzd/eneXJFIv2jjw8q/SI/K3sNkJ2s80PL3TEBLk8+njUmAI37J//99AMLIVsxknSOV+IAaGBo4jgrGosPeWQcFIAgZNb73MxWAuFN4w3c= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=treblig.org; spf=pass smtp.mailfrom=treblig.org; dkim=pass (2048-bit key) header.d=treblig.org header.i=@treblig.org header.b=VGhZ72qm; arc=none smtp.client-ip=46.235.229.95 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=treblig.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=treblig.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=treblig.org header.i=@treblig.org header.b="VGhZ72qm" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=treblig.org ; s=bytemarkmx; h=MIME-Version:Message-ID:Date:Subject:From:Content-Type:From :Subject; bh=OvaaM5K1iTaNPtb1WzCdYsTgO1bTNP7eXeSy1JzDb1I=; b=VGhZ72qm6JhBDu9g cPlTJtGfHTeNsmBwdbyiJCtuNA1hYKGPlmjR1tW7f3+mbEF3libN2GaKc1Z259kiOZu5wKsXq31MC nHlRt45pwcFrVj/YUSgCcZCqY+XjL1MkrxDkhvB3TYPjwOu2NGbrO0YZrsq3IwBtbJsAykKI6daro 9Oy3J4B3iRvH2hDi0rY6kAKi7RuBHaAf2cKb8gS6WIVpxscvN0arU8VPqCejVSFBrALgyTuX5ooeT m2RHt3O/JZ5BhGl2vEVQKy6K+3Nu813XQeXDrVXneEr2mX+fTnqdXF+YC1TXeC/61RBFtExFEuXM1 /TrCjpZcEiiYKltPAw==; Received: from localhost ([127.0.0.1] helo=dalek.home.treblig.org) by mx.treblig.org with esmtp (Exim 4.96) (envelope-from ) id 1tqgi2-003XFb-2b; Fri, 07 Mar 2025 23:02:26 +0000 From: linux@treblig.org To: sre@kernel.org, linux-pm@vger.kernel.org Cc: linux-kernel@vger.kernel.org, "Dr. David Alan Gilbert" Subject: [PATCH 1/3] power: supply: core: Remove unused power_supply_set_battery_charged Date: Fri, 7 Mar 2025 23:02:23 +0000 Message-ID: <20250307230225.128775-2-linux@treblig.org> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250307230225.128775-1-linux@treblig.org> References: <20250307230225.128775-1-linux@treblig.org> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Dr. David Alan Gilbert" power_supply_set_battery_charged() has been unused since 2019's commit 0f884f8a090e ("ARM: pxa: remove raumfeld board files and defconfig") Remove it. Signed-off-by: Dr. David Alan Gilbert --- drivers/power/supply/power_supply_core.c | 13 ------------- include/linux/power_supply.h | 1 - 2 files changed, 14 deletions(-) diff --git a/drivers/power/supply/power_supply_core.c b/drivers/power/supply/power_supply_core.c index 76c340b38015..47e0fea84917 100644 --- a/drivers/power/supply/power_supply_core.c +++ b/drivers/power/supply/power_supply_core.c @@ -449,19 +449,6 @@ int power_supply_get_property_from_supplier(struct power_supply *psy, } EXPORT_SYMBOL_GPL(power_supply_get_property_from_supplier); -int power_supply_set_battery_charged(struct power_supply *psy) -{ - if (atomic_read(&psy->use_cnt) >= 0 && - psy->desc->type == POWER_SUPPLY_TYPE_BATTERY && - psy->desc->set_charged) { - psy->desc->set_charged(psy); - return 0; - } - - return -EINVAL; -} -EXPORT_SYMBOL_GPL(power_supply_set_battery_charged); - static int power_supply_match_device_by_name(struct device *dev, const void *data) { const char *name = data; diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h index 6ed53b292162..5afc5946eb03 100644 --- a/include/linux/power_supply.h +++ b/include/linux/power_supply.h @@ -852,7 +852,6 @@ extern int power_supply_am_i_supplied(struct power_supply *psy); int power_supply_get_property_from_supplier(struct power_supply *psy, enum power_supply_property psp, union power_supply_propval *val); -extern int power_supply_set_battery_charged(struct power_supply *psy); static inline bool power_supply_supports_maintenance_charging(struct power_supply_battery_info *info) From patchwork Fri Mar 7 23:02:24 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Dr. David Alan Gilbert" X-Patchwork-Id: 871430 Received: from mx.treblig.org (mx.treblig.org [46.235.229.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6C12D1B0F11; Fri, 7 Mar 2025 23:02:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=46.235.229.95 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741388550; cv=none; b=pDxChcQaPYB09eR8ceP6I9bb0rgZRQwnJIqwnpsSGvxe8HxDbOXP/KvbhdffG9MCxiqWhdYVw8+IdrBTyEwSKA0/oUCN8FafFp6OHkLZtkOOdHDI/NsfXzc089nQEAWUgxzOl4ib5QBlXGmxxH8XBQ2s0Kn4Zdv34h+atQxiDCk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741388550; c=relaxed/simple; bh=SoNEziyvGgKhUQ6TOuU1ybgmFsUuCAx7PBtGLbxk0Qk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=J8tCvQaccZ8Bxz1iJkRcR0PNHRRXSnBN6FMZ1hUHiVR15r4uBC6Tiu8DF35P0S6P2Yi1HTNXHfa8ZYb3eZEEDEVtEezJDtTYgFm8e1i2KEUaOcXS38wXttl8vNkq9YtdTIVsEj33GQybNmrw+vdGUGAUn637fLkwJOmetK8fqDU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=treblig.org; spf=pass smtp.mailfrom=treblig.org; dkim=pass (2048-bit key) header.d=treblig.org header.i=@treblig.org header.b=jZFybx2L; arc=none smtp.client-ip=46.235.229.95 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=treblig.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=treblig.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=treblig.org header.i=@treblig.org header.b="jZFybx2L" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=treblig.org ; s=bytemarkmx; h=MIME-Version:Message-ID:Date:Subject:From:Content-Type:From :Subject; bh=R+yDZqBmrY7R81jPy3BAIpyzLTLF7d2CCpAj0+tuUBU=; b=jZFybx2L5mF/hKUN gS0YYuXAyvd0zPjjkomBFGCmB4i8vTt3b1a0vvexWRbqVRTEY9fISL6v3FmdF4AW8qQO4uWn7OOgj bofL4YS+CL8+IlnTYHmCwvMMhcmiQk1BgQncqhsZyygtgrphXf6un0UwhmDQ3t20sARtPnEf8FdZu vTEDbU8ykFG9cXm8MBYR27CFE/9crQDE9tCgMVcx4yRBsUJj3WXUpGKEy202g4kBzZUFz7zT+sT2m f95v0hgfJ4ij78BpWxjO/xhu8foLEnQZbDCr0+3i6u3ivALosP1jtet13PquEWr+/v3tn9mw7kguN 3NpxVHCXwF99bnAvUg==; Received: from localhost ([127.0.0.1] helo=dalek.home.treblig.org) by mx.treblig.org with esmtp (Exim 4.96) (envelope-from ) id 1tqgi3-003XFb-0o; Fri, 07 Mar 2025 23:02:27 +0000 From: linux@treblig.org To: sre@kernel.org, linux-pm@vger.kernel.org Cc: linux-kernel@vger.kernel.org, "Dr. David Alan Gilbert" Subject: [PATCH 2/3] power: supply: ds2760: Remove unused ds2760_battery_set_charged Date: Fri, 7 Mar 2025 23:02:24 +0000 Message-ID: <20250307230225.128775-3-linux@treblig.org> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250307230225.128775-1-linux@treblig.org> References: <20250307230225.128775-1-linux@treblig.org> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Dr. David Alan Gilbert" With power_supply_set_battery_charged() deleted in the previous patch, there's no one left to call the set_charged method. The only implemented is in ds2760, unwire it and delete the associated functions and data. (Arguably it might be time to delete ds2760 since I don't think there are any users left) Signed-off-by: Dr. David Alan Gilbert --- drivers/power/supply/ds2760_battery.c | 49 --------------------------- 1 file changed, 49 deletions(-) diff --git a/drivers/power/supply/ds2760_battery.c b/drivers/power/supply/ds2760_battery.c index 83bdec5a2bda..a7f7d4c3f1ee 100644 --- a/drivers/power/supply/ds2760_battery.c +++ b/drivers/power/supply/ds2760_battery.c @@ -112,7 +112,6 @@ struct ds2760_device_info { struct power_supply_desc bat_desc; struct workqueue_struct *monitor_wqueue; struct delayed_work monitor_work; - struct delayed_work set_charged_work; struct notifier_block pm_notifier; }; @@ -489,50 +488,6 @@ static void ds2760_battery_external_power_changed(struct power_supply *psy) } -static void ds2760_battery_set_charged_work(struct work_struct *work) -{ - char bias; - struct ds2760_device_info *di = container_of(work, - struct ds2760_device_info, set_charged_work.work); - - dev_dbg(di->dev, "%s\n", __func__); - - ds2760_battery_read_status(di); - - /* When we get notified by external circuitry that the battery is - * considered fully charged now, we know that there is no current - * flow any more. However, the ds2760's internal current meter is - * too inaccurate to rely on - spec say something ~15% failure. - * Hence, we use the current offset bias register to compensate - * that error. - */ - - if (!power_supply_am_i_supplied(di->bat)) - return; - - bias = (signed char) di->current_raw + - (signed char) di->raw[DS2760_CURRENT_OFFSET_BIAS]; - - dev_dbg(di->dev, "%s: bias = %d\n", __func__, bias); - - w1_ds2760_write(di->dev, &bias, DS2760_CURRENT_OFFSET_BIAS, 1); - w1_ds2760_store_eeprom(di->dev, DS2760_EEPROM_BLOCK1); - w1_ds2760_recall_eeprom(di->dev, DS2760_EEPROM_BLOCK1); - - /* Write to the di->raw[] buffer directly - the CURRENT_OFFSET_BIAS - * value won't be read back by ds2760_battery_read_status() */ - di->raw[DS2760_CURRENT_OFFSET_BIAS] = bias; -} - -static void ds2760_battery_set_charged(struct power_supply *psy) -{ - struct ds2760_device_info *di = power_supply_get_drvdata(psy); - - /* postpone the actual work by 20 secs. This is for debouncing GPIO - * signals and to let the current value settle. See AN4188. */ - mod_delayed_work(di->monitor_wqueue, &di->set_charged_work, HZ * 20); -} - static int ds2760_battery_get_property(struct power_supply *psy, enum power_supply_property psp, union power_supply_propval *val) @@ -692,7 +647,6 @@ static int w1_ds2760_add_slave(struct w1_slave *sl) di->bat_desc.set_property = ds2760_battery_set_property; di->bat_desc.property_is_writeable = ds2760_battery_property_is_writeable; - di->bat_desc.set_charged = ds2760_battery_set_charged; di->bat_desc.external_power_changed = ds2760_battery_external_power_changed; @@ -747,8 +701,6 @@ static int w1_ds2760_add_slave(struct w1_slave *sl) } INIT_DELAYED_WORK(&di->monitor_work, ds2760_battery_work); - INIT_DELAYED_WORK(&di->set_charged_work, - ds2760_battery_set_charged_work); di->monitor_wqueue = alloc_ordered_workqueue(name, WQ_MEM_RECLAIM); if (!di->monitor_wqueue) { retval = -ESRCH; @@ -774,7 +726,6 @@ static void w1_ds2760_remove_slave(struct w1_slave *sl) unregister_pm_notifier(&di->pm_notifier); cancel_delayed_work_sync(&di->monitor_work); - cancel_delayed_work_sync(&di->set_charged_work); destroy_workqueue(di->monitor_wqueue); } From patchwork Fri Mar 7 23:02:25 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Dr. David Alan Gilbert" X-Patchwork-Id: 871429 Received: from mx.treblig.org (mx.treblig.org [46.235.229.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A2F501EBFE6; Fri, 7 Mar 2025 23:02:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=46.235.229.95 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741388551; cv=none; b=gR4HhB0MirhZowXHja1q/Y1vL+g2h1072pEo/m9Uait3sDxALKZWlCc3VdNFsykqKgsMRF1EgL7ASvsTIcu/Rj1t/ouq5kwTtPtKdZ0AM5ifo+YH4TZH6FnOjHXz2gp7kNkuc2wU7GyFhswHBFNAPM4vwIrp33nj+mODaAq+U3U= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741388551; c=relaxed/simple; bh=mdMVobuWepIr7PV03x7xbJHm7C/7keEOmWcHH70aAcU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ousHqFLuovtUED5KiJzPd8LZOe3u3p0HQ7L04aIdIeP083gMGDsf4G9BUb0nIOXHc8Jh+nYZQ1vf2/2MGM0CQIETQ11f7mLVRFz7i9Mmh+F3KHs2SzxQcRunU073i3Yew6uPm00JvD6ZIJ5kiHcRUGIRRsu0CKbnzAm6Z8R5VQg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=treblig.org; spf=pass smtp.mailfrom=treblig.org; dkim=pass (2048-bit key) header.d=treblig.org header.i=@treblig.org header.b=F7v69lV1; arc=none smtp.client-ip=46.235.229.95 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=treblig.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=treblig.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=treblig.org header.i=@treblig.org header.b="F7v69lV1" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=treblig.org ; s=bytemarkmx; h=MIME-Version:Message-ID:Date:Subject:From:Content-Type:From :Subject; bh=nqjQmn5zuk4Kwr62lIq32KHD5DpJuH6RLBYrL+pZlHA=; b=F7v69lV1camihRet BEi7CVqtRE/GMGTZhLkw47xmKG1pwuPPUT074TETZ0YxB/o9UNJXC5afdJXirFLKEE5HmXQZGnrGa DntZEaBVqGH3iavPcq/zSQRvQSCpxunGSvMNOTi0Qozz+JsysXARVQCIAPHTuSk22sb+6l5Jro+FI TZz2MvJhPJF/4eJbRGHuoeLtDwOHG19rSslapHz2qoBynbOZt9U7bo1V2wX0obYwZpDW3wEYtTQst Y8Yr5c9Ud0DxSNUoGRrb+d/gAWN1NOPy4Vilf4hpf3ZYZphVPwBHKf1vt2Q62n1R2b/blqmURXnnN IeKabVeW+2Z/XfUlTg==; Received: from localhost ([127.0.0.1] helo=dalek.home.treblig.org) by mx.treblig.org with esmtp (Exim 4.96) (envelope-from ) id 1tqgi3-003XFb-2A; Fri, 07 Mar 2025 23:02:27 +0000 From: linux@treblig.org To: sre@kernel.org, linux-pm@vger.kernel.org Cc: linux-kernel@vger.kernel.org, "Dr. David Alan Gilbert" Subject: [PATCH 3/3] power: supply: Remove unused set_charged method Date: Fri, 7 Mar 2025 23:02:25 +0000 Message-ID: <20250307230225.128775-4-linux@treblig.org> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250307230225.128775-1-linux@treblig.org> References: <20250307230225.128775-1-linux@treblig.org> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Dr. David Alan Gilbert" The previous patches in this series removed the only caller and only setter of this method. Remove it. Signed-off-by: Dr. David Alan Gilbert --- include/linux/power_supply.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h index 5afc5946eb03..92da3deba342 100644 --- a/include/linux/power_supply.h +++ b/include/linux/power_supply.h @@ -274,7 +274,6 @@ struct power_supply_desc { int (*property_is_writeable)(struct power_supply *psy, enum power_supply_property psp); void (*external_power_changed)(struct power_supply *psy); - void (*set_charged)(struct power_supply *psy); /* * Set if thermal zone should not be created for this power supply.