[ACCTON][A9817-32] Refine the error messages and add PSU revision for psu driver

Signed-off-by: Willy Liu <willy@accton.com>

Author: Willy Liu

packages/platforms/accton/x86-64/as9817-32/src/modules/accton_ipmi_intf.c

@@ -141,6 +141,13 @@ static int _ipmi_send_message(struct ipmi_data *ipmi, unsigned char cmd,
 			      unsigned char *rx_data, unsigned short rx_len)
 {
 	int err;
+	int i, pos;
+	char raw_cmd[RAW_CMD_BUF_SIZE] = { 0 };
+
+	pos = snprintf(raw_cmd, sizeof(raw_cmd), "0x%02x", cmd);
+	for (i = 0; i < tx_len && pos < sizeof(raw_cmd); i++) {
+		pos += snprintf(raw_cmd + pos, sizeof(raw_cmd) - pos," 0x%02x", tx_data[i]);
+	}
 
 	// Validate the input parameters
 	if ((tx_len && !tx_data) || (rx_len && !rx_data)) {
@@ -157,7 +164,7 @@ static int _ipmi_send_message(struct ipmi_data *ipmi, unsigned char cmd,
 	// Validate the IPMI address
 	err = ipmi_validate_addr(&ipmi->address, sizeof(ipmi->address));
 	if (err) {
-		dev_err(ipmi->dev, "Invalid IPMI address: %x\n", err);
+		dev_err(ipmi->dev, "Invalid IPMI address: %x raw_cmd=[%s]\n", err, raw_cmd);
 		return err;
 	}
 
@@ -166,14 +173,14 @@ static int _ipmi_send_message(struct ipmi_data *ipmi, unsigned char cmd,
 	err = ipmi_request_settime(ipmi->user, &ipmi->address, ipmi->tx_msgid,
 				   &ipmi->tx_message, ipmi, 0, 0, 0);
 	if (err) {
-		dev_err(ipmi->dev, "IPMI request_settime failed: %x\n", err);
+		dev_err(ipmi->dev, "IPMI request_settime failed: %x raw_cmd=[%s]\n", err, raw_cmd);
 		return err;
 	}
 
 	// Wait for the message to complete
 	err = wait_for_completion_timeout(&ipmi->read_complete, IPMI_TIMEOUT);
 	if (!err) {
-		dev_err(ipmi->dev, "IPMI command timeout\n");
+		dev_err(ipmi->dev, "IPMI command timeout raw_cmd=[%s]\n", raw_cmd);
 		return -ETIMEDOUT;
 	}