From nobody Sat Jul 25 00:16:11 2026 Received: from outbound.qs.icloud.com (qs-2002a-snip4-11.eps.apple.com [57.103.87.141]) (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 82DCE46E012 for ; Tue, 21 Jul 2026 18:59:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=57.103.87.141 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660379; cv=none; b=vFABy2is9Jmrx/sYA+MpwxJTHuVnhO5ZEj03djequJrGvRKJGYEBqN6cgQ/gVV8r/D/N5J7Nh5Uc/l8okA69Zyn67p1sq6qOAkD+05JRDF/B6ImBL0k9bvc588/tt8rIxa/tgnyAoXqygVycVlTko3loJZ/JTy9cqzNRU8Gf+30= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660379; c=relaxed/simple; bh=WuWCkEdvrb2h4vS91JHngedGnQPfvRM7UhLzbRwVLHM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ALpYEvlmG7mFPQtZ1njiAT0/FDg4WghN7h+3fgBdqLeOM7tYJ0kFAkQ/8+RMx2cqpMgvq1gcxGvrPbY5TT9qJ4wbT/25RInH84nvn0fcDG02BmuxRcfkr4c1STAtvreXMp+ux6GITwjXK7zjGShCHcxEuV6HPnN04Le3w5naR8Y= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com; spf=pass smtp.mailfrom=icloud.com; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b=yj20YK3+; arc=none smtp.client-ip=57.103.87.141 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=icloud.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b="yj20YK3+" Received: from outbound.qs.icloud.com (unknown [127.0.0.2]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPS id 63D9418000A1; Tue, 21 Jul 2026 18:59:35 +0000 (UTC) X-ICL-RepId: 019f860c-36e3-7c8e-9a03-49e6f4402c20 X-ICL-Out-Info: HUtFAUMEWwJACUgBTUQeDx5WFlZNRAJCTQhPAEMGXAVeC1YBXwFLVxQEDloDVA5cBBcbXwJCH1sVSzhaDlsERxQXG1wAFw1WTVAbXwJCDxwTVhUTH1RWA0IZEFYBWFZdBU0aXBhZDxwBSlZaDlsERxQXG1wAFxtGAgQjAl8ARQJeCVYBMBcPVk1QG18CQg8cE1YVEwBeDw9MC0gBWwdcAEEIQAZUB18cRAlPBlsaWR8GUBZQCFkfHBNVF0YZXg5AMFAbXwJCDxwTVhUNTUMSQhUEEVABWB5W Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1784660377; x=1787252377; bh=i8a/m1R9oqsx6t0nI7k2MPlYERLMh/6Ev5E/3vcDxuU=; h=From:To:Subject:Date:Message-ID:MIME-Version:x-icloud-hme; b=yj20YK3++VdZQzFgxdvpYJN7cGI8qC0No/DpR7eRErY4xuJ0kfnQ1HMaepUbmgvrqMVzGCpVBbdohoQFml/7LzZW806sPzTx2jUkJlj8wO1dDKBcvTa1p/UCJRKGUhlU7kDqn1t/JioOq8DE5K1lSWED8mzJ2vZVnNBw2jSjFHMhB8qgBRQcRHG5KNbrVkWpai0FGeNvtOrx40Xr9VeT8MwLFGFK/d47A2A7a3TjghpRGL1NLwx/C+R/g9Ldta70stjlmJwv4VeKdSSImv65eBwDEH1Nh0wc48rpBzLWjR21aJbfmajN71cxrn995FUtg/DDiTem+IOKVDi4wiBA8g== Received: from bigre.localdomain (unknown [17.57.155.37]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPSA id 0D7FA18004C0; Tue, 21 Jul 2026 18:59:34 +0000 (UTC) From: Vincent Cloutier To: Sebastian Reichel Cc: Hans de Goede , Krzysztof Kozlowski , Marek Szyprowski , Sebastian Krzyszkowiak , Purism Kernel Team , Rob Herring , Conor Dooley , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Vincent Cloutier Subject: [PATCH v3 1/4] power: supply: Add registration init callback Date: Tue, 21 Jul 2026 14:57:33 -0400 Message-ID: <20260721185904.40756-2-vincent.cloutier@icloud.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260721185904.40756-1-vincent.cloutier@icloud.com> References: <20260721185904.40756-1-vincent.cloutier@icloud.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 X-Proofpoint-ORIG-GUID: hd58LFtUmY4QWRoH42Dlk6pe886Zw-iV X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzIxMDE5OSBTYWx0ZWRfXxq0b0a6yrEe+ dTBvR8/RhoF3y4ji9W8cd34iHTrEMMt6Mv1zfgNiX2g3kbygFpC28vR2i7+H5o+QmhGf8Zb+hgY wLzeTfHq+BnE9HDPtEMTtuSa27aNTob3ma9lGcyAXi1Quq2wk72Ce86CDjXKQLvwVwupCJF3vKR 7HZzwRX89JSs6pth1S4UStR8k+6ldF4KUo8nQbLM9OyLMp9sECh1zqFpt/+rt8septrz9jY4GkK IyiIXT+RAPGUKRT2AZcI2LI2iGDKP9cvmGjg2zkRBMyWPNKUkDl3G52nX6jqxJy1TYMXD2AZMyY J9u1IR7xWn6+aZQ07WQ X-Proofpoint-GUID: hd58LFtUmY4QWRoH42Dlk6pe886Zw-iV Content-Type: text/plain; charset="utf-8" From: Vincent Cloutier Some battery drivers need to consume monitored-battery data before their power_supply is visible. That lets them prepare hardware configuration from parsed battery information without racing userspace exposure. power_supply_get_battery_info() already runs in __power_supply_register() for battery devices before device_add(). Add an optional descriptor init callback after driver data and battery info are available. The callback runs in sleepable process context while the power supply is still unpublished. Require callbacks to return zero or a negative errno. Defensively reject positive returns so registration cannot return an invalid error pointer. Assisted-by: OpenCode:gpt-5.6-sol Signed-off-by: Vincent Cloutier --- drivers/power/supply/power_supply_core.c | 8 ++++++++ include/linux/power_supply.h | 8 ++++++++ 2 files changed, 16 insertions(+) diff --git a/drivers/power/supply/power_supply_core.c b/drivers/power/suppl= y/power_supply_core.c index 2532e221b2e1..3ed4c253706f 100644 --- a/drivers/power/supply/power_supply_core.c +++ b/drivers/power/supply/power_supply_core.c @@ -1624,6 +1624,14 @@ __power_supply_register(struct device *parent, init_rwsem(&psy->extensions_sem); INIT_LIST_HEAD(&psy->extensions); =20 + if (desc->init) { + rc =3D desc->init(psy); + if (WARN_ON_ONCE(rc > 0)) + rc =3D -EINVAL; + if (rc) + goto check_supplies_failed; + } + rc =3D device_add(dev); if (rc) goto device_add_failed; diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h index 7a5e4c3242a0..6042f5e1fdc2 100644 --- a/include/linux/power_supply.h +++ b/include/linux/power_supply.h @@ -281,6 +281,14 @@ 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); + /* + * Optional registration-time initialization. This runs in sleepable + * process context after driver data and any battery info are available, + * but before the device is added. The callback must not publish changes + * because registration is not complete. Return 0 on success or a negative + * errno on failure. + */ + int (*init)(struct power_supply *psy); =20 /* * Set if thermal zone should not be created for this power supply. --=20 2.55.0 From nobody Sat Jul 25 00:16:11 2026 Received: from outbound.qs.icloud.com (qs-2002g-snip4-3.eps.apple.com [57.103.87.194]) (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 39ECA46E009 for ; Tue, 21 Jul 2026 18:59:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=57.103.87.194 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660380; cv=none; b=IC9t7R2BS0LVhBwWvTgkGU823B0fd81dCh+T7/5Ks8loFx8NNbQVOyZdW9+OChrbXBVvN5wNZzA1PcX5SxIxH3sZnSC63Kff7A1uLP/5lA8VDTNHEM+gGDyIQIbTlenrNgYczsq40eWZwn1rMxm0ycOjVlJyghLB1EI1lcdA1es= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660380; c=relaxed/simple; bh=MpCEGbVhBKIlAdc1p/tHkHq4S3ljcuQWEtg0SJcUGGk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Vogi7T/Y3/sHXudgUIG+0tualITxIQ0iYXTugwl6NSwqeD08uOb4bEcCaYEmmH4IgEBxfpMBwIzjkTYT9NhaHUb+N/vF14lsEN1VQPRNtWNwba2paV/DaCl4SP9gW7rCvrqDIMkDVIoH6OdD5l85EZcA3dZZExWSWuC810g8cNE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com; spf=pass smtp.mailfrom=icloud.com; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b=cCt/NFHV; arc=none smtp.client-ip=57.103.87.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=icloud.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b="cCt/NFHV" Received: from outbound.qs.icloud.com (unknown [127.0.0.2]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPS id 8FCC61800275; Tue, 21 Jul 2026 18:59:35 +0000 (UTC) X-ICL-RepId: 019f860c-37e2-7708-b1f4-59dcc3b191ba X-ICL-Out-Info: HUtFAUMEWwJACUgBTUQeDx5WFlZNRAJCTQhPAEMGXAVeC1YBXwFLVxQEDloDVA5cBBcbXwJCH1sVSzhaDlsERxQXG1wAFw1WTVAbXwJCDxwTVhUTH1RWA0IZEFYBWFZdBU0aXBhZDxwBSlZaDlsERxQXG1wAFxtGAgQjAl8ARQJeCVYBMBcPVk1QG18CQg8cE1YVEwBeDw9MC0gBWwdcAEEIQAZUB18cRAlPBlsaWB8GUBZQCFkfHBNVF0YZXg5AMFAbXwJCDxwTVhUNTUMSQhUEEVABWB5W Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1784660377; x=1787252377; bh=vAAou0ItuODHJ7VRRr9+GAd58QJ1e8SPQzYumIVrkD8=; h=From:To:Subject:Date:Message-ID:MIME-Version:Content-Type:x-icloud-hme; b=cCt/NFHVYfv340bzbTcNPgixa6azLjp533bLLKq74hwFFDedeV6bLOsiI+SeMXlJH5rgwzb8eA/hRQLwfvLvI3+k5lcteOuRBKV1PMbAiQ2OWeStr3zS/eXlK8uvXNxQBpmgrts+rUbK2mbl54W/0bLMsI54M6HO2RW6uzhIbyEI8ogo3DV4oMSMJZbVU+AHSOR46JcEnJ1T3Ia42xD33qJQPh44CDXTn95cR6sTBtLkEdi4pujdvivbjygt/ooro1w62lNJTLk0JN51+XZk/28t05b7BAMdk+tMa6cK2auRYP5ClNQrx5TPMzkMVYtO5LDIflUxkQmaOn0aoKx08w== Received: from bigre.localdomain (unknown [17.57.155.37]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPSA id 8F6C618004F2; Tue, 21 Jul 2026 18:59:34 +0000 (UTC) From: Vincent Cloutier To: Sebastian Reichel Cc: Hans de Goede , Krzysztof Kozlowski , Marek Szyprowski , Sebastian Krzyszkowiak , Purism Kernel Team , Rob Herring , Conor Dooley , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Vincent Cloutier Subject: [PATCH v3 2/4] power: supply: max17042_battery: Initialize MAX17055 from battery info Date: Tue, 21 Jul 2026 14:57:34 -0400 Message-ID: <20260721185904.40756-3-vincent.cloutier@icloud.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260721185904.40756-1-vincent.cloutier@icloud.com> References: <20260721185904.40756-1-vincent.cloutier@icloud.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Proofpoint-ORIG-GUID: MmWNXYdaDwmrDHVAFpGvbywHTmBTDA9G X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzIxMDE5OSBTYWx0ZWRfX/+RinmhL0l8+ nYYriQ5FCro48u1G8gvpSugO0nqKudlHFccpqmMoBcASRZQGZU2SoXCii2ZmyWjFdgkr+YGySR4 KJA91mLMHNvJfBK5zR8GgvXPejekjYlYag0wK3SFEPdrP3+YFCkqyF8B7MZ/5rWL5bXAQHd/c/j c5yUoDLd4UZg/hXfepcEZegtRtTTpXPjMi9KD0ASbxjLn0XzqB1i92rGfU7ecZKs+k+02FKpP0y Vk2cKJJ7WM5HNy2I1n6sPSwz0dRmF2io386f/EpCSbMNOEJwGVPx4YwUGEYBoabC5xMAkQlfCPQ F7MqiU171jDFZGvWZGG X-Proofpoint-GUID: MmWNXYdaDwmrDHVAFpGvbywHTmBTDA9G From: Vincent Cloutier MAX17055 can use charge-full-design-microamp-hours and charge-term-current-microamp from monitored-battery for its POR configuration. Prepare quantized DesignCap, IChgTerm, and EZ Config dQAcc values in the power-supply registration callback after the core has parsed battery information. Validate all supplied values before changing the sparse configuration. Reject zero, values that quantize to zero, capacities that cannot produce dQAcc, and positive termination currents outside the signed register range. Limit this path to MAX17055 because the other supported gauges require complete characterization data. Follow the documented POR sequence. Wait for FSTAT.DNR, save HibCfg, exit hibernate, write stable battery registers once and verify them after 1 ms, issue command writes directly, poll until refresh clears, and restore HibCfg. Clear POR only after every step succeeds. Keep driver-backed properties unavailable while initialization is incomplete. If an I/O error interrupts the transaction, restore hibernate before the next attempt and retry periodically so a transient startup failure does not become permanent. Assisted-by: OpenCode:gpt-5.6-sol Signed-off-by: Vincent Cloutier --- drivers/power/supply/max17042_battery.c | 335 +++++++++++++++++++++--- include/linux/power/max17042_battery.h | 2 + 2 files changed, 306 insertions(+), 31 deletions(-) diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply= /max17042_battery.c index d409d2f0d383..b25bf26135e0 100644 --- a/drivers/power/supply/max17042_battery.c +++ b/drivers/power/supply/max17042_battery.c @@ -62,18 +62,30 @@ #define MAX17042_RESISTANCE_LSB 1 / 4096 /* =CE=A9 */ #define MAX17042_TEMPERATURE_LSB 1 / 256 /* =C2=B0C */ =20 +#define MAX17055_DQACC_DIV 32 +#define MAX17055_DPACC_FACTOR 44138 +#define MAX17055_DPACC_VCHG_FACTOR 51200 +#define MAX17055_FSTAT_DNR_BIT BIT(0) +#define MAX17055_DNR_POLL_US 10000 +#define MAX17055_DNR_TIMEOUT_US 2000000 +#define MAX17055_INIT_RETRY_DELAY_MS 10000 +#define MAX17055_REFRESH_POLL_US 10000 +#define MAX17055_REFRESH_TIMEOUT_US 1000000 + struct max17042_chip { struct device *dev; struct regmap *regmap; struct power_supply *battery; enum max170xx_chip_type chip_type; struct max17042_config_data *config_data; - struct work_struct work; - int init_complete; + struct delayed_work work; int irq; int task_period; bool enable_current_sense; bool enable_por_init; + bool init_complete; + bool hib_restore_pending; + u16 hib_cfg; unsigned int r_sns; int vmin; /* in millivolts */ int vmax; /* in millivolts */ @@ -256,7 +268,7 @@ static int max17042_get_property(struct power_supply *p= sy, u32 data; u64 data64; =20 - if (!chip->init_complete) + if (!READ_ONCE(chip->init_complete)) return -EAGAIN; =20 switch (psp) { @@ -566,6 +578,24 @@ static int max17042_write_verify_reg(struct regmap *ma= p, u8 reg, u32 value) return ret; } =20 +static int max17055_write_verify_reg(struct regmap *map, u8 reg, u32 value) +{ + u32 read_value; + int ret; + + ret =3D regmap_write(map, reg, value); + if (ret) + return ret; + + usleep_range(1000, 2000); + + ret =3D regmap_read(map, reg, &read_value); + if (ret) + return ret; + + return read_value =3D=3D value ? 0 : -EIO; +} + static inline void max17042_override_por(struct regmap *map, u8 reg, u16 value) { @@ -801,8 +831,12 @@ static inline void max17042_override_por_values(struct= max17042_chip *chip) max17042_override_por(map, MAX17042_CONFIG, config->config); max17042_override_por(map, MAX17042_SHDNTIMER, config->shdntimer); =20 - max17042_override_por(map, MAX17042_DesignCap, config->design_cap); - max17042_override_por(map, MAX17042_ICHGTerm, config->ichgt_term); + if (chip->chip_type !=3D MAXIM_DEVICE_TYPE_MAX17055) { + max17042_override_por(map, MAX17042_DesignCap, + config->design_cap); + max17042_override_por(map, MAX17042_ICHGTerm, + config->ichgt_term); + } =20 max17042_override_por(map, MAX17042_AtRate, config->at_rate); max17042_override_por(map, MAX17042_LearnCFG, config->learn_cfg); @@ -812,8 +846,10 @@ static inline void max17042_override_por_values(struct= max17042_chip *chip) =20 max17042_override_por(map, MAX17042_FullCAP, config->fullcap); max17042_override_por(map, MAX17042_FullCAPNom, config->fullcapnom); - max17042_override_por(map, MAX17042_dQacc, config->dqacc); - max17042_override_por(map, MAX17042_dPacc, config->dpacc); + if (chip->chip_type !=3D MAXIM_DEVICE_TYPE_MAX17055) { + max17042_override_por(map, MAX17042_dQacc, config->dqacc); + max17042_override_por(map, MAX17042_dPacc, config->dpacc); + } =20 max17042_override_por(map, MAX17042_RCOMP0, config->rcomp0); max17042_override_por(map, MAX17042_TempCo, config->tcompc0); @@ -847,26 +883,162 @@ static inline void max17042_override_por_values(stru= ct max17042_chip *chip) max17042_override_por(map, MAX17055_ModelCfg, config->model_cfg); } =20 -static int max17042_init_chip(struct max17042_chip *chip) +static int max17055_override_battery_values(struct max17042_chip *chip) { + struct max17042_config_data *config =3D chip->config_data; struct regmap *map =3D chip->regmap; + unsigned int design_cap; + unsigned int model_cfg; + unsigned int dqacc; + u64 dpacc; int ret; =20 + if (config->design_cap) { + ret =3D max17055_write_verify_reg(map, MAX17042_DesignCap, + config->design_cap); + if (ret) + return ret; + } + + if (config->dqacc) { + ret =3D max17055_write_verify_reg(map, MAX17042_dQacc, + config->dqacc); + if (ret) + return ret; + } + + if (config->ichgt_term) { + ret =3D max17055_write_verify_reg(map, MAX17042_ICHGTerm, + config->ichgt_term); + if (ret) + return ret; + } + + if (!config->design_cap && !config->dqacc) + return 0; + + ret =3D regmap_read(map, MAX17042_DesignCap, &design_cap); + if (ret) + return ret; + + ret =3D regmap_read(map, MAX17042_dQacc, &dqacc); + if (ret) + return ret; + + ret =3D regmap_read(map, MAX17055_ModelCfg, &model_cfg); + if (ret) + return ret; + + if (!design_cap || !dqacc) + return -ERANGE; + + dpacc =3D (u64)dqacc * + (model_cfg & MAX17055_MODELCFG_VCHG_BIT ? + MAX17055_DPACC_VCHG_FACTOR : MAX17055_DPACC_FACTOR); + do_div(dpacc, design_cap); + if (dpacc > U16_MAX) + return -ERANGE; + + return max17055_write_verify_reg(map, MAX17042_dPacc, (u16)dpacc); +} + +static int max17055_restore_hibernate(struct max17042_chip *chip) +{ + int restore_hib_ret; + int soft_wakeup_ret; + + soft_wakeup_ret =3D regmap_write(chip->regmap, MAX17055_SoftWakeup, 0); + restore_hib_ret =3D max17055_write_verify_reg(chip->regmap, + MAX17055_HibCfg, + chip->hib_cfg); + if (!soft_wakeup_ret && !restore_hib_ret) + chip->hib_restore_pending =3D false; + + return soft_wakeup_ret ?: restore_hib_ret; +} + +static int max17055_init_chip(struct max17042_chip *chip) +{ + struct regmap *map =3D chip->regmap; + unsigned int hib_cfg; + unsigned int model_cfg; + unsigned int fstat; + int restore_ret; + int ret; + + if (chip->hib_restore_pending) { + ret =3D max17055_restore_hibernate(chip); + if (ret) + return ret; + } + + ret =3D regmap_read_poll_timeout(map, MAX17042_FSTAT, fstat, + !(fstat & MAX17055_FSTAT_DNR_BIT), + MAX17055_DNR_POLL_US, + MAX17055_DNR_TIMEOUT_US); + if (ret) + return ret; + + ret =3D regmap_read(map, MAX17055_HibCfg, &hib_cfg); + if (ret) + return ret; + + chip->hib_cfg =3D hib_cfg; + chip->hib_restore_pending =3D true; + + ret =3D regmap_write(map, MAX17055_SoftWakeup, 0x0090); + if (ret) + goto restore_hibernate; + + ret =3D max17055_write_verify_reg(map, MAX17055_HibCfg, 0); + if (ret) + goto restore_hibernate; + + ret =3D regmap_write(map, MAX17055_SoftWakeup, 0); + if (ret) + goto restore_hibernate; + max17042_override_por_values(chip); =20 + ret =3D max17055_override_battery_values(chip); + if (ret) + goto restore_hibernate; + + ret =3D regmap_write_bits(map, MAX17055_ModelCfg, + MAX17055_MODELCFG_REFRESH_BIT, + MAX17055_MODELCFG_REFRESH_BIT); + if (ret) + goto restore_hibernate; + + ret =3D regmap_read_poll_timeout(map, MAX17055_ModelCfg, model_cfg, + !(model_cfg & + MAX17055_MODELCFG_REFRESH_BIT), + MAX17055_REFRESH_POLL_US, + MAX17055_REFRESH_TIMEOUT_US); + +restore_hibernate: + restore_ret =3D max17055_restore_hibernate(chip); + if (restore_ret) + return restore_ret; + + return ret; +} + +static int max17042_init_chip(struct max17042_chip *chip) +{ + struct regmap *map =3D chip->regmap; + int ret; + if (chip->chip_type =3D=3D MAXIM_DEVICE_TYPE_MAX17055) { - regmap_write_bits(map, MAX17055_ModelCfg, - MAX17055_MODELCFG_REFRESH_BIT, - MAX17055_MODELCFG_REFRESH_BIT); - } + ret =3D max17055_init_chip(chip); + if (ret) + return ret; + } else { + max17042_override_por_values(chip); =20 - /* After Power up, the MAX17042 requires 500mS in order - * to perform signal debouncing and initial SOC reporting - */ - msleep(500); + /* Allow signal debouncing and initial SOC reporting. */ + msleep(500); =20 - if (chip->chip_type !=3D MAXIM_DEVICE_TYPE_MAX17055) { - /* Initialize configuration */ max17042_write_config_regs(chip); =20 /* write cell characterization data */ @@ -902,8 +1074,7 @@ static int max17042_init_chip(struct max17042_chip *ch= ip) } =20 /* Init complete, Clear the POR bit */ - regmap_update_bits(map, MAX17042_STATUS, STATUS_POR_BIT, 0x0); - return 0; + return regmap_clear_bits(map, MAX17042_STATUS, STATUS_POR_BIT); } =20 static void max17042_set_soc_threshold(struct max17042_chip *chip, u16 off) @@ -989,18 +1160,28 @@ static irqreturn_t max17042_thread_handler(int id, v= oid *dev) =20 static void max17042_init_worker(struct work_struct *work) { - struct max17042_chip *chip =3D container_of(work, + struct max17042_chip *chip =3D container_of(to_delayed_work(work), struct max17042_chip, work); - int ret; + int ret =3D 0; =20 /* Initialize registers according to values from config_data */ - if (chip->enable_por_init && chip->config_data) { + if (chip->enable_por_init && chip->config_data) ret =3D max17042_init_chip(chip); - if (ret) - return; + + if (ret) { + if (chip->chip_type =3D=3D MAXIM_DEVICE_TYPE_MAX17055) { + dev_warn_ratelimited(chip->dev, + "initialization failed: %d, retrying\n", ret); + queue_delayed_work(system_freezable_wq, &chip->work, + msecs_to_jiffies(MAX17055_INIT_RETRY_DELAY_MS)); + } else { + dev_err(chip->dev, "initialization failed: %d\n", ret); + } + return; } =20 - chip->init_complete =3D 1; + WRITE_ONCE(chip->init_complete, true); + power_supply_changed(chip->battery); } =20 #ifdef CONFIG_OF @@ -1060,6 +1241,93 @@ static int max17042_init_defaults(struct max17042_ch= ip *chip) return 0; } =20 +static int max17042_apply_battery_properties(struct max17042_chip *chip, + struct power_supply_battery_info *info) +{ + struct max17042_config_data *config; + struct device *dev =3D chip->dev; + bool have_design_cap; + bool have_ichgt_term; + u16 design_cap =3D 0; + u16 ichgt_term =3D 0; + u16 dqacc =3D 0; + u64 data64; + + if (!info || chip->chip_type !=3D MAXIM_DEVICE_TYPE_MAX17055) + return 0; + + have_design_cap =3D chip->enable_current_sense && + info->charge_full_design_uah >=3D 0; + have_ichgt_term =3D chip->enable_current_sense && + info->charge_term_current_ua >=3D 0; + if (!have_design_cap && !have_ichgt_term) + return 0; + + if (have_design_cap) { + if (!info->charge_full_design_uah) + return dev_err_probe(dev, -EINVAL, + "battery design capacity must be positive\n"); + + data64 =3D (u64)info->charge_full_design_uah * chip->r_sns; + do_div(data64, MAX17042_CAPACITY_LSB); + if (!data64) + return dev_err_probe(dev, -ERANGE, + "battery design capacity is too small for sense resistor\n"); + if (data64 > U16_MAX) + return dev_err_probe(dev, -ERANGE, + "battery design capacity exceeds register range\n"); + + design_cap =3D (u16)data64; + dqacc =3D design_cap / MAX17055_DQACC_DIV; + if (!dqacc) + return dev_err_probe(dev, -ERANGE, + "battery design capacity is too small for EZ config\n"); + } + + if (have_ichgt_term) { + if (!info->charge_term_current_ua) + return dev_err_probe(dev, -EINVAL, + "charge termination current must be positive\n"); + + data64 =3D (u64)info->charge_term_current_ua * chip->r_sns; + do_div(data64, MAX17042_CURRENT_LSB); + if (!data64) + return dev_err_probe(dev, -ERANGE, + "charge termination current is too small for sense resistor\n"); + if (data64 > S16_MAX) + return dev_err_probe(dev, -ERANGE, + "charge termination current exceeds positive register range\n"); + + ichgt_term =3D (u16)data64; + } + + config =3D chip->config_data; + if (!config) { + config =3D devm_kzalloc(dev, sizeof(*config), GFP_KERNEL); + if (!config) + return -ENOMEM; + } + + if (have_design_cap) { + config->design_cap =3D design_cap; + config->dqacc =3D dqacc; + } + if (have_ichgt_term) + config->ichgt_term =3D ichgt_term; + + chip->config_data =3D config; + chip->enable_por_init =3D true; + + return 0; +} + +static int max17042_init_battery(struct power_supply *psy) +{ + struct max17042_chip *chip =3D power_supply_get_drvdata(psy); + + return max17042_apply_battery_properties(chip, psy->battery_info); +} + static const struct regmap_config max17042_regmap_config =3D { .name =3D "max17042", .reg_bits =3D 8, @@ -1113,6 +1381,7 @@ static const struct power_supply_desc max17042_psy_de= sc =3D { .set_property =3D max17042_set_property, .property_is_writeable =3D max17042_property_is_writeable, .external_power_changed =3D power_supply_changed, + .init =3D max17042_init_battery, .properties =3D max17042_battery_props, .num_properties =3D ARRAY_SIZE(max17042_battery_props), }; @@ -1123,6 +1392,7 @@ static const struct power_supply_desc max17042_no_cur= rent_sense_psy_desc =3D { .get_property =3D max17042_get_property, .set_property =3D max17042_set_property, .property_is_writeable =3D max17042_property_is_writeable, + .init =3D max17042_init_battery, .properties =3D max17042_battery_props, .num_properties =3D ARRAY_SIZE(max17042_battery_props) - 2, }; @@ -1242,15 +1512,18 @@ static int max17042_probe(struct i2c_client *client= , struct device *dev, int irq =20 chip->irq =3D irq; =20 - regmap_read(chip->regmap, MAX17042_STATUS, &val); + ret =3D regmap_read(chip->regmap, MAX17042_STATUS, &val); + if (ret) + return dev_err_probe(dev, ret, "failed to read status\n"); + if (val & STATUS_POR_BIT) { - ret =3D devm_work_autocancel(dev, &chip->work, - max17042_init_worker); + ret =3D devm_delayed_work_autocancel(dev, &chip->work, + max17042_init_worker); if (ret) return ret; - schedule_work(&chip->work); + queue_delayed_work(system_freezable_wq, &chip->work, 0); } else { - chip->init_complete =3D 1; + WRITE_ONCE(chip->init_complete, true); } =20 return 0; diff --git a/include/linux/power/max17042_battery.h b/include/linux/power/m= ax17042_battery.h index 13aeab1597c6..810e068eafd2 100644 --- a/include/linux/power/max17042_battery.h +++ b/include/linux/power/max17042_battery.h @@ -25,6 +25,7 @@ #define MAX17042_CHARACTERIZATION_DATA_SIZE 48 =20 #define MAX17055_MODELCFG_REFRESH_BIT BIT(15) +#define MAX17055_MODELCFG_VCHG_BIT BIT(10) =20 enum max17042_register { MAX17042_STATUS =3D 0x00, @@ -124,6 +125,7 @@ enum max17055_register { =20 MAX17055_ConvgCfg =3D 0x49, MAX17055_VFRemCap =3D 0x4A, + MAX17055_SoftWakeup =3D 0x60, =20 MAX17055_STATUS2 =3D 0xB0, MAX17055_POWER =3D 0xB1, --=20 2.55.0 From nobody Sat Jul 25 00:16:12 2026 Received: from outbound.qs.icloud.com (qs-2002k-snip4-10.eps.apple.com [57.103.87.240]) (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 087B846EF6D for ; Tue, 21 Jul 2026 18:59:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=57.103.87.240 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660382; cv=none; b=lWQDvlmcXQQk7+EPwfMX2LVyFNofaqN9jk8btiD7zrwkmpAR5PwBtucF4mDSNfL6zhlcZVvdbavPepIbqE4egbMPKotLLrWqBFklg2uQhmR4Meqxp+9JRm5hXVAWdvLJZrEJyZmvDAAbjBDFaAUGNifNorU7P4LrlTEfMVaAAUc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660382; c=relaxed/simple; bh=zYPDk0dFuqI3SWYSbwOOATjyNJP19NdKD/Zid6+2nB0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=rcnPGPgTEpzZFTf/0ViZ1UlbcaYDZuRGoSoFV50D519kMyBSaHb2Sube4UG/q4z06hkw/QxO3mwWdHPPsP6zPzWAEEpN9ZiPlZ3AM6YI93KsA8WDK5edWbr124RzsHR8+LYql7CCu3czWpJpHT7dxehVeFTWLAmmcZ4IF2aVe5o= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com; spf=pass smtp.mailfrom=icloud.com; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b=s8NvcgRV; arc=none smtp.client-ip=57.103.87.240 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=icloud.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b="s8NvcgRV" Received: from outbound.qs.icloud.com (unknown [127.0.0.2]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPS id 7799618004D8; Tue, 21 Jul 2026 18:59:36 +0000 (UTC) X-ICL-RepId: 019f860c-3b17-7494-9120-83c21685520b X-ICL-Out-Info: HUtFAUMEWwJACUgBTUQeDx5WFlZNRAJCTQhPAEMGXAVeC1YBXwFLVxQEDloDVA5cBBcbXwJCH1sVSzhaDlsERxQXG1wAFw1WTVAbXwJCDxwTVhUTH1RWA0IZEFYBWFZdBU0aXBhZDxwBSlZaDlsERxQXG1wAFxtGAgQjAl8ARQJeCVYBMBcPVk1QG18CQg8cE1YVEwBeDw9MC0gBWwdcAEEIQAZUB18cRAlPBlsaXx8GUBZQCFkfHBNVF0YZXg5AMFAbXwJCDxwTVhUNTUMSQhUEEVABWB5W Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1784660380; x=1787252380; bh=xVEkIySqC2w1VTBoBLzRmkBJvvkPdFwntWG87jLmjgU=; h=From:To:Subject:Date:Message-ID:MIME-Version:x-icloud-hme; b=s8NvcgRVaH4oWAxO2ECYrcTt0vNoNYMheub05EhlPaCH7EkCpsrKMc7vJY9KBC0uttNBxJcL2ujJG+uSUW4kQ62yubESUzLItfsL4Roy2EAiCcquibFQBTO5t5ZxH5OZ8pX7REBEZYNYGn+6drB/nLfAnwqjsugzmRPa1GB7J5mgM2nIOvMj9nMUNqsaxt7hyXb+kl/+/urM2900K1VJtyLqlGqKxVDUIMPnyTGD9wrjb3tnAtqfN6BeDG7dqOARzZwR4+L/BUxq09H2KLVhzI7k1WLQM6QLlV3DMCi70AHA6g8wF6QPfob5z3w0+miWPjdf7y6nQIcDfyjpzMFhfQ== Received: from bigre.localdomain (unknown [17.57.155.37]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPSA id 13D5A1801822; Tue, 21 Jul 2026 18:59:35 +0000 (UTC) From: Vincent Cloutier To: Sebastian Reichel Cc: Hans de Goede , Krzysztof Kozlowski , Marek Szyprowski , Sebastian Krzyszkowiak , Purism Kernel Team , Rob Herring , Conor Dooley , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Vincent Cloutier Subject: [PATCH v3 3/4] power: supply: max17042_battery: Honor MAX17055 charge voltage Date: Tue, 21 Jul 2026 14:57:35 -0400 Message-ID: <20260721185904.40756-4-vincent.cloutier@icloud.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260721185904.40756-1-vincent.cloutier@icloud.com> References: <20260721185904.40756-1-vincent.cloutier@icloud.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 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzIxMDE5OSBTYWx0ZWRfX+Cx+dPXcv0XU g8V1LgJBEBL04zjys+Asq2Kp0ZIfIvaero0kefwMk6PZLYl5pRYv2RZW9PxpnO2tLhLb/G8DdUc 5pWVQV/6xT0EIFi7z3rzZZAxMpc33UkoYVJMB/kt01bFaGcg/7NPEroL4QhK1CiwWxAfBV79vzX FIMJ9xwcTxQ6rYfwb/AB5HTHf2ajopSZlM9Y66hROmRFgt1sqnVfeHe5F4czbeE9zbfuKxnt6x6 bDTxefYMNikjw+kOc4noij6hR4fIJZMXE1tcEEhOG6yMT1Jh8eUzBKrwRI6EeD5dlesgbNALyNI JDr52FTAPEJxZML/MiB X-Proofpoint-GUID: Iddilkg9rgge7z6L683povkqfllri423 X-Proofpoint-ORIG-GUID: Iddilkg9rgge7z6L683povkqfllri423 Content-Type: text/plain; charset="utf-8" From: Vincent Cloutier MAX17055 ModelCfg.VChg selects the charge-voltage range used by EZ Config. Consume voltage-max-design-microvolt from monitored-battery and set VChg only when the value is strictly greater than 4.275 V. Reject an explicit zero voltage and preserve the register when the property is absent. Update only the VChg bit and verify its read-back so unrelated ModelCfg fields remain intact. Treat a VChg-only override as an accumulator change: read effective DesignCap and dQAcc, derive and verify the matching dPAcc, then request model refresh. This keeps voltage-only firmware descriptions coherent with the selected charge-voltage range. Assisted-by: OpenCode:gpt-5.6-sol Signed-off-by: Vincent Cloutier --- drivers/power/supply/max17042_battery.c | 42 +++++++++++++++++++++++-- 1 file changed, 39 insertions(+), 3 deletions(-) diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply= /max17042_battery.c index b25bf26135e0..0f6bf6d4c031 100644 --- a/drivers/power/supply/max17042_battery.c +++ b/drivers/power/supply/max17042_battery.c @@ -66,6 +66,7 @@ #define MAX17055_DPACC_FACTOR 44138 #define MAX17055_DPACC_VCHG_FACTOR 51200 #define MAX17055_FSTAT_DNR_BIT BIT(0) +#define MAX17055_VCHG_THRESHOLD_UV 4275000 #define MAX17055_DNR_POLL_US 10000 #define MAX17055_DNR_TIMEOUT_US 2000000 #define MAX17055_INIT_RETRY_DELAY_MS 10000 @@ -83,6 +84,7 @@ struct max17042_chip { int task_period; bool enable_current_sense; bool enable_por_init; + bool enable_vchg_override; bool init_complete; bool hib_restore_pending; u16 hib_cfg; @@ -879,7 +881,8 @@ static inline void max17042_override_por_values(struct = max17042_chip *chip) max17042_override_por(map, MAX17047_V_empty, config->vempty); } =20 - if (chip->chip_type =3D=3D MAXIM_DEVICE_TYPE_MAX17055) + if (chip->chip_type =3D=3D MAXIM_DEVICE_TYPE_MAX17055 && + !chip->enable_vchg_override) max17042_override_por(map, MAX17055_ModelCfg, config->model_cfg); } =20 @@ -914,7 +917,19 @@ static int max17055_override_battery_values(struct max= 17042_chip *chip) return ret; } =20 - if (!config->design_cap && !config->dqacc) + if (chip->enable_vchg_override) { + ret =3D regmap_update_bits(map, MAX17055_ModelCfg, + MAX17055_MODELCFG_VCHG_BIT, + config->model_cfg & + MAX17055_MODELCFG_VCHG_BIT); + if (ret) + return ret; + + usleep_range(1000, 2000); + } + + if (!config->design_cap && !config->dqacc && + !chip->enable_vchg_override) return 0; =20 ret =3D regmap_read(map, MAX17042_DesignCap, &design_cap); @@ -928,6 +943,10 @@ static int max17055_override_battery_values(struct max= 17042_chip *chip) ret =3D regmap_read(map, MAX17055_ModelCfg, &model_cfg); if (ret) return ret; + if (chip->enable_vchg_override && + (model_cfg & MAX17055_MODELCFG_VCHG_BIT) !=3D + (config->model_cfg & MAX17055_MODELCFG_VCHG_BIT)) + return -EIO; =20 if (!design_cap || !dqacc) return -ERANGE; @@ -1248,9 +1267,11 @@ static int max17042_apply_battery_properties(struct = max17042_chip *chip, struct device *dev =3D chip->dev; bool have_design_cap; bool have_ichgt_term; + bool have_vchg; u16 design_cap =3D 0; u16 ichgt_term =3D 0; u16 dqacc =3D 0; + u16 model_cfg =3D 0; u64 data64; =20 if (!info || chip->chip_type !=3D MAXIM_DEVICE_TYPE_MAX17055) @@ -1260,7 +1281,8 @@ static int max17042_apply_battery_properties(struct m= ax17042_chip *chip, info->charge_full_design_uah >=3D 0; have_ichgt_term =3D chip->enable_current_sense && info->charge_term_current_ua >=3D 0; - if (!have_design_cap && !have_ichgt_term) + have_vchg =3D info->voltage_max_design_uv >=3D 0; + if (!have_design_cap && !have_ichgt_term && !have_vchg) return 0; =20 if (have_design_cap) { @@ -1301,6 +1323,15 @@ static int max17042_apply_battery_properties(struct = max17042_chip *chip, ichgt_term =3D (u16)data64; } =20 + if (have_vchg) { + if (!info->voltage_max_design_uv) + return dev_err_probe(dev, -EINVAL, + "battery design voltage must be positive\n"); + + if (info->voltage_max_design_uv > MAX17055_VCHG_THRESHOLD_UV) + model_cfg =3D MAX17055_MODELCFG_VCHG_BIT; + } + config =3D chip->config_data; if (!config) { config =3D devm_kzalloc(dev, sizeof(*config), GFP_KERNEL); @@ -1314,6 +1345,11 @@ static int max17042_apply_battery_properties(struct = max17042_chip *chip, } if (have_ichgt_term) config->ichgt_term =3D ichgt_term; + if (have_vchg) { + config->model_cfg &=3D ~MAX17055_MODELCFG_VCHG_BIT; + config->model_cfg |=3D model_cfg; + chip->enable_vchg_override =3D true; + } =20 chip->config_data =3D config; chip->enable_por_init =3D true; --=20 2.55.0 From nobody Sat Jul 25 00:16:12 2026 Received: from outbound.qs.icloud.com (qs-2002l-snip4-7.eps.apple.com [57.103.87.248]) (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 6E80E46EF60 for ; Tue, 21 Jul 2026 18:59:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=57.103.87.248 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660381; cv=none; b=srdrgg4o/78+y+LzG2SKwXMAirUokm37DgUhyTjSLlHt2Lbx5uy2xwywVsQ92eugqsgt/JuUAKkn/bFQo6jM+qSk60IAEwdHapvrv1DC/PFV2Q8BPjjiDzh6oOSdddo3YzcKN2HL1k5IQkiUdkUd+xFi2P1CRAbow9iHPtqlZS0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784660381; c=relaxed/simple; bh=TqO+Qg6fzXhTG3QjMKM/vT1RCeMQLmbqcWW1YTZv9Uk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=C5jWcnnSEjDUou64gvK3PpKewMzXhfRSiSFOhr+4CdVV0N9z5I6IfroxdM8KVhJUNJnouCZ2b7GN+tX+vqW6fJ0rXP+AAebhqbO6Tr45uFLCPyTxwAwVpax/GSKXO6vWfhJu2ol4Hv0S9H9yYWNYGv2x9CkvtV55Yvbpd25Xm3w= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com; spf=pass smtp.mailfrom=icloud.com; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b=YsoktTjH; arc=none smtp.client-ip=57.103.87.248 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=icloud.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b="YsoktTjH" Received: from outbound.qs.icloud.com (unknown [127.0.0.2]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPS id 8FD0C18004DA; Tue, 21 Jul 2026 18:59:36 +0000 (UTC) X-ICL-RepId: 019f860c-3bbb-7f77-a49f-59923bb33e52 X-ICL-Out-Info: HUtFAUMEWwJACUgBTUQeDx5WFlZNRAJCTQhPAEMGXAVeC1YBXwFLVxQEDloDVA5cBBcbXwJCH1sVSzhaDlsERxQXG1wAFw1WTVAbXwJCDxwTVhUTH1RWA0IZEFYBWFZdBU0aXBhZDxwBSlZaDlsERxQXG1wAFxtGAgQjAl8ARQJeCVYBMBcPVk1QG18CQg8cE1YVEwBeDw9MC0gBWwdcAEEIQAZUB18cRAlPBlsaXh8GUBZQCFkfHBNVF0YZXg5AMFAbXwJCDxwTVhUNTUMSQhUEEVABWB5W Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1784660379; x=1787252379; bh=Ws336SOk57oXBc2JW7cgPs4vQiZSL0GtxyqE7/bvjh8=; h=From:To:Subject:Date:Message-ID:MIME-Version:x-icloud-hme; b=YsoktTjHCXGkPgIt2HZCUzYzPD5zpsp6OO33jpfxcNA6Vq/82Twl//L8ogzlJwKurOjCDubUXTcluBlKExsuuAtoXzRzTF8LqMwcnSoAswpF6hydDL9arLK+u1i1tzoAOec4j1x+CaRtKEfDxHVIv+ehK2xLpxiHKIB+JtmNr1AtCcmrL6R0IPUjRKSImf1B/RC02iSzvyjWKrgnH7XjcwfUdRoIU3T7DLFJi3pDGvZdBFcZO1aSQ0Lf2gHw3YyEclgnujrPcRDSqzOBP61xyn6gWko2B7Qpa7woUbyt6alB3lS5esV6TT3TwNbh64WWc/XWrzLrsE3JSR7l9zffbA== Received: from bigre.localdomain (unknown [17.57.155.37]) by p00-icloudmta-asmtp-us-east-2d-60-percent-3 (Postfix) with ESMTPSA id 894A51800140; Tue, 21 Jul 2026 18:59:35 +0000 (UTC) From: Vincent Cloutier To: Sebastian Reichel Cc: Hans de Goede , Krzysztof Kozlowski , Marek Szyprowski , Sebastian Krzyszkowiak , Purism Kernel Team , Rob Herring , Conor Dooley , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Vincent Cloutier Subject: [PATCH v3 4/4] dt-bindings: power: supply: max17042: Allow monitored-battery Date: Tue, 21 Jul 2026 14:57:36 -0400 Message-ID: <20260721185904.40756-5-vincent.cloutier@icloud.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260721185904.40756-1-vincent.cloutier@icloud.com> References: <20260721185904.40756-1-vincent.cloutier@icloud.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 X-Proofpoint-ORIG-GUID: 2gSQn9XY-NOcZmRlnxZWXXRGdQ0GVD8l X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzIxMDE5OSBTYWx0ZWRfX3EV8Ro142D7V xD6FVKQOuM8BmXL67SnloFdVl0nZ5D7Ofqm/PKZh6pBarepI35eB+3oValfWsAmD5n24CucTxMb qQRV2Rb+E5j6DnWxsjNDL+sB3gn2AVGhaUCauhl3/t24mkGC2dLtoeB+r+bZOWAdQQYK4gtrYcu 29ueoW2whA3vZxZa8cakp1XlunlnMPsqhJUqd1yQr82P/ahOaQ0q1nGICUkZRv3kXEGwBavxO0P Kud/0B2zaU/bDP75v+5Nk7WpA6t17Y/husbtiE4ER3YY1goIuH7z17tY/FvR8M+72qaHRoQD12K 3v7WFMCcG/lyiuZmroH X-Proofpoint-GUID: 2gSQn9XY-NOcZmRlnxZWXXRGdQ0GVD8l Content-Type: text/plain; charset="utf-8" From: Vincent Cloutier The MAX17042 family binding references the generic power-supply schema, but additionalProperties only recognizes properties declared directly by this schema. Use unevaluatedProperties so properties inherited from power-supply.yaml, including monitored-battery, are allowed while unknown properties remain rejected. Assisted-by: OpenCode:gpt-5.6-sol Signed-off-by: Vincent Cloutier --- .../devicetree/bindings/power/supply/maxim,max17042.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/power/supply/maxim,max17042.= yaml b/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml index 242b33f2bcba..395253ca7835 100644 --- a/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml +++ b/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml @@ -73,7 +73,7 @@ required: - compatible - reg =20 -additionalProperties: false +unevaluatedProperties: false =20 examples: - | --=20 2.55.0