You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the battery monitor on K5EM hardware, it's necessary to tell the battery monitor what the capacity of the battery is. This is currently set in the max17260 driver when the config structure is passed to Max17620::default_setup
If a user wants to use this hardware with a different battery size, it's advised to set the appropriate battery capacity. This came up as I'm preparing to send Brian a new hardware that I believe he's planning to use with a KBT 300mAh battery. For this combination to measure and support the smaller battery size, it will be necessary to let the max17260 know that the capacity is different so that it can properly build the model.
My suggestion would be to have this as a default setting for the 500mAh battery, but to let the user modify it in the settings page, possibly under an "advanced settings" section that's normally hidden. Note that when this is changed, it will be necessary to force a re-set of the battery monitor over I2C either by physically removing power, or by forcing a power on reset of the 17620 by setting POR to 1. The max17260 driver will need an added method to force this re-load, as the default init method will see that the battery monitor is already initialized and skip re-initializing it.
The text was updated successfully, but these errors were encountered:
With the battery monitor on K5EM hardware, it's necessary to tell the battery monitor what the capacity of the battery is. This is currently set in the
max17260
driver when the config structure is passed toMax17620::default_setup
SOTAcat/src/max17260.cpp
Line 77 in 3d0b9d8
If a user wants to use this hardware with a different battery size, it's advised to set the appropriate battery capacity. This came up as I'm preparing to send Brian a new hardware that I believe he's planning to use with a KBT 300mAh battery. For this combination to measure and support the smaller battery size, it will be necessary to let the max17260 know that the capacity is different so that it can properly build the model.
My suggestion would be to have this as a default setting for the 500mAh battery, but to let the user modify it in the settings page, possibly under an "advanced settings" section that's normally hidden. Note that when this is changed, it will be necessary to force a re-set of the battery monitor over I2C either by physically removing power, or by forcing a power on reset of the 17620 by setting POR to 1. The max17260 driver will need an added method to force this re-load, as the default init method will see that the battery monitor is already initialized and skip re-initializing it.
The text was updated successfully, but these errors were encountered: