fwk: ucsi: do not process the ucsi if no active pd chip

If there is no active pd chip exist, the *cci and *message_in
will not point to the correct address and cause the EC panic.

This change adds the condition to avoid the data point to the wrong
address.

BRANCH=fwk-main
BUG=EC panic when process the ucsi data
TEST=EC will not process the ucsi data if no active pd chip.

Signed-off-by: Josh Tsai <Josh_Tsai@compal.com>
(cherry picked from commit de061e56e2ac5224c1b63a77197d49e7be34ae79)

Author: Josh-Tsai

zephyr/program/framework/src/ucsi.c

@@ -243,6 +243,10 @@ static void resend_ucsi_connector_change_event(void)
 	static int process_port = 1;
 	static int resume_flag;
 
+	/* If no active pd chip, don't process the UCSI data */
+	if (active_pd_chip_count == 0)
+		return;
+
 	if (s0ix_connector_change_indicator == 0) {
 		process_port = 1;
 		resume_flag = 0;
@@ -491,6 +495,10 @@ void check_ucsi_event_from_host(void)
 	int i;
 	int rv;
 
+	/* If no active pd chip, don't process the UCSI data */
+	if (active_pd_chip_count == 0)
+		return;
+
 	if (read_complete == 0 && !timestamp_expired(ucsi_wait_time, NULL)) {
 		if (ucsi_debug_enable)
 			CPRINTS("UCSI waiting for time expired");
@@ -667,13 +675,25 @@ void ucsi_pd_port_mapping(void)
 	}
 }
 
+/**
+ * CHIPSET_RESET for warmboot/coldboot
+ * CHIPSET_INIT for first power on
+ */
 void setup_ucsi_pd_mapping(void)
 {
-	active_pd_chip_count = cypd_get_active_pd_chip_count();
-	ucsi_pd_port_mapping();
-	if (ucsi_debug_enable) {
-		CPRINTS("PD mapping setup: active_chips=%d, valid_ports=%d",
-			active_pd_chip_count, valid_ucsi_port_count);
+	int pd_chip_count = cypd_get_active_pd_chip_count();
+
+	if (pd_chip_count == 0)
+		CPRINTS("WARNNING: No active PD chip");
+
+	if (active_pd_chip_count != pd_chip_count) {
+		active_pd_chip_count = pd_chip_count;
+		ucsi_pd_port_mapping();
+		if (ucsi_debug_enable) {
+			CPRINTS("PD mapping setup: active_chips=%d, valid_ports=%d",
+				active_pd_chip_count, valid_ucsi_port_count);
+		}
 	}
 }
 DECLARE_HOOK(HOOK_CHIPSET_RESET, setup_ucsi_pd_mapping, HOOK_PRIO_DEFAULT);
+DECLARE_HOOK(HOOK_INIT, setup_ucsi_pd_mapping, HOOK_PRIO_DEFAULT);