Fwk: clean up some debug logs

Signed-off-by: Josh-Tsai <josh_tsai@compal.com>

Author: Josh-Tsai

board/hx30/cypress5525.c

@@ -240,11 +240,10 @@ int cypd_write_reg8_wait_ack(int controller, int reg, int data, int delay_ms)
 	rv = cypd_get_int(controller, &intr_status);
 	if (rv != EC_SUCCESS)
 		return rv;
+
 	if (intr_status & CYP5525_DEV_INTR) {
-		CPRINTS("data = 0x%04x", data);
 		if (data == CYP5525_AC_AT_PORT) {
 			rv = cypd_read_reg8(controller, CYP5525_RESPONSE_REG, &event);
-			CPRINTS("event = 0x%04x", event);
 			if (rv != EC_SUCCESS)
 				CPRINTS("fail to read response");
 			switch (event) {
@@ -257,9 +256,10 @@ int cypd_write_reg8_wait_ack(int controller, int reg, int data, int delay_ms)
 			case CYPD_RESPONSE_NO_AC:
 			case CYPD_RESPONSE_EC_MODE:
 				break;
+			default:
+				CPRINTS("Check AC get unknown event 0x%04x", event);
 			}
-		} else
-			CPRINTS("unknown event");
+		}
 		cypd_clear_int(controller, CYP5525_DEV_INTR);
 	}
 	usleep(50);
@@ -1813,16 +1813,13 @@ int check_power_on_port(void)
 static int prev_charge_port = -1;
 int board_set_active_charge_port(int charge_port)
 {
-	CPRINTS("start change port = %d, prev_charge_port = %d", charge_port, prev_charge_port);
-
 	/* if no battery, EC should not control C_CTRL */
 	if (board_batt_is_present() != BP_YES) {
 		/* check if CYPD ready */
 		if (charge_port == -1)
 			return EC_ERROR_TRY_AGAIN;
 
 		/* store current port and update power limit */
-		CPRINTS("No batt, no change");
 		prev_charge_port = charge_port;
 		hook_call_deferred(&update_power_limit_deferred_data, 100 * MSEC);
 		CPRINTS("Updating %s port %d", __func__, charge_port);
@@ -1832,7 +1829,7 @@ int board_set_active_charge_port(int charge_port)
 	/* port need change, stop all power and ready to switch. */
 	if (prev_charge_port != -1 && prev_charge_port != charge_port) {
 		update_soc_power_limit(false, true);
-		CPRINTS("all off");
+		CPRINTS("Disable all type-c port to change the charger port");
 		cypd_write_reg8(0, CYP5525_CUST_C_CTRL_CONTROL_REG, CYP5525_P0P1_TURN_OFF_C_CTRL);
 		cypd_write_reg8(1, CYP5525_CUST_C_CTRL_CONTROL_REG, CYP5525_P0P1_TURN_OFF_C_CTRL);
 		usleep(250*MSEC);
@@ -1844,15 +1841,11 @@ int board_set_active_charge_port(int charge_port)
 	if (charge_port >=0 ) {
 		int pd_controller = (charge_port & 0x02) >> 1;
 		int pd_port = charge_port & 0x01;
-
-		CPRINTS("force off ctrler = %d", (~pd_controller) & 0x01);
 		cypd_write_reg8((~pd_controller) & 0x01, CYP5525_CUST_C_CTRL_CONTROL_REG,
 			CYP5525_P0P1_TURN_OFF_C_CTRL);
-		CPRINTS("choose p %d, ctrler = %d, p = %d", charge_port, pd_controller, pd_port);
 		cypd_write_reg8(pd_controller, CYP5525_CUST_C_CTRL_CONTROL_REG,
 			pd_port ? CYP5525_P0_OFF_P1_CY : CYP5525_P0_CY_P1_OFF);
 	} else {
-		CPRINTS("else = %d, set all ports auto", charge_port);
 		cypd_write_reg8(0, CYP5525_CUST_C_CTRL_CONTROL_REG, CYP5525_P0P1_TURN_OFF_C_CTRL);
 		cypd_write_reg8(1, CYP5525_CUST_C_CTRL_CONTROL_REG, CYP5525_P0P1_TURN_OFF_C_CTRL);
 	}

common/charge_manager.c

@@ -579,7 +579,7 @@ static void charge_manager_get_best_charge_port(int *new_port,
 	/* If system initially power on w/o dc, CYPD will control C_CTRL */
 	if ((charge_port == CHARGE_SUPPLIER_NONE) && (board_batt_is_present() != BP_YES)) {
 		*new_port = check_power_on_port();
-		CPRINTS("NO DC, choose AC by CYPD: %d", *new_port);
+		CPRINTS("AC only power on port %d", *new_port);
 		*new_supplier = CHARGE_SUPPLIER_PD;
 		if (*new_port == -1)
 			*new_supplier = CHARGE_SUPPLIER_NONE;