fwk: main: avoid extender overwritten charger limit

if disable extender is and setting the charger limit at bios option same moment,
host will send charger limit cmd first(3e03) and then disable extender(3e24),
but if disable extender it will set charger to normal and let sustainer to default
(battery_sustainer_set(-1, -1)), so charger limit will be overwritten to disable

BRANCH=fwk-main
BUG=https://community.frame.work/t/12th-gen-intel-core-bios-3-09-release-beta/65716/15
TEST=test on marigold, into bios setup, disable battery extender and setting chargerlimit(80%)
press f10 saving and reboot, after boot up check battery percentage will stop at 80%

Signed-off-by: LeoCX_Tsai <[email protected]>
(cherry picked from commit 09386f6ad0c97b7129f4447af8a593f59bd45570)

Author: LeoCX-Tsai

baseboard/fwk/battery_extender.c

@@ -195,8 +195,11 @@ static enum ec_status battery_extender_hc(struct host_cmd_handler_args *args)
 		if (batt_extender_disable != p->disable) {
 			batt_extender_disable = p->disable;
 			if (batt_extender_disable) {
-				battery_sustainer_set(-1, -1);
-				set_chg_ctrl_mode(CHARGE_CONTROL_NORMAL);
+				/* if charger limit has been set don't control sustainer again */
+				if (charger_sustainer_percentage() == 100) {
+					battery_sustainer_set(-1, -1);
+					set_chg_ctrl_mode(CHARGE_CONTROL_NORMAL);
+				}
 				stage = BATT_EXTENDER_STAGE_0;
 			}
 		}