mirror of
https://github.com/Telecominfraproject/OpenCellular.git
synced 2025-12-29 10:00:51 +00:00
pd: cleanup PD receive error enum
Cleanup PD receive error enum by including RX in name since we will have a different enum for TX errors. BUG=none BRANCH=none TEST=make -j buildall Change-Id: I355092e0e73a022acb4a92736374cd2289d324bf Signed-off-by: Alec Berg <alecaberg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/267670 Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
This commit is contained in:
committed by
ChromeOS Commit Bot
parent
0086f8a780
commit
5d28e776c9
@@ -147,13 +147,13 @@ static void print_packet(int head, uint32_t *payload)
|
||||
ccputs("\n");
|
||||
}
|
||||
|
||||
static void print_error(enum pd_errors err)
|
||||
static void print_error(enum pd_rx_errors err)
|
||||
{
|
||||
if (err == PD_ERR_INVAL)
|
||||
if (err == PD_RX_ERR_INVAL)
|
||||
ccprintf("%T TMOUT\n");
|
||||
else if (err == PD_ERR_HARD_RESET)
|
||||
else if (err == PD_RX_ERR_HARD_RESET)
|
||||
ccprintf("%T HARD-RST\n");
|
||||
else if (err == -5)
|
||||
else if (err == PD_RX_ERR_UNSUPPORTED_SOP)
|
||||
ccprintf("%T SOP*\n");
|
||||
else
|
||||
ccprintf("ERR %d\n", err);
|
||||
|
||||
@@ -180,9 +180,9 @@ int pd_find_preamble(int port)
|
||||
if (all == 0x36db6db6)
|
||||
return bit - 1; /* should be SYNC-1 */
|
||||
if (all == 0xF33F3F3F)
|
||||
return PD_ERR_HARD_RESET; /* got HARD-RESET */
|
||||
return PD_RX_ERR_HARD_RESET; /* got HARD-RESET */
|
||||
if (all == 0x3c7fe0ff)
|
||||
return PD_ERR_CABLE_RESET; /* got CABLE-RESET */
|
||||
return PD_RX_ERR_CABLE_RESET; /* got CABLE-RESET */
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
|
||||
@@ -1449,7 +1449,7 @@ int pd_analyze_rx(int port, uint32_t *payload)
|
||||
|
||||
/* Detect preamble */
|
||||
bit = pd_find_preamble(port);
|
||||
if (bit == PD_ERR_HARD_RESET || bit == PD_ERR_CABLE_RESET) {
|
||||
if (bit == PD_RX_ERR_HARD_RESET || bit == PD_RX_ERR_CABLE_RESET) {
|
||||
/* Hard reset or cable reset */
|
||||
return bit;
|
||||
} else if (bit < 0) {
|
||||
@@ -1464,10 +1464,10 @@ int pd_analyze_rx(int port, uint32_t *payload)
|
||||
break;
|
||||
} else if (val == PD_SOP_PRIME) {
|
||||
CPRINTF("SOP'\n");
|
||||
return PD_ERR_UNSUPPORTED_SOP;
|
||||
return PD_RX_ERR_UNSUPPORTED_SOP;
|
||||
} else if (val == PD_SOP_PRIME_PRIME) {
|
||||
CPRINTF("SOP''\n");
|
||||
return PD_ERR_UNSUPPORTED_SOP;
|
||||
return PD_RX_ERR_UNSUPPORTED_SOP;
|
||||
}
|
||||
}
|
||||
if (bit < 0) {
|
||||
@@ -1507,7 +1507,7 @@ int pd_analyze_rx(int port, uint32_t *payload)
|
||||
if (bit < 0 || pcrc != ccrc) {
|
||||
msg = "CRC";
|
||||
if (pcrc != ccrc)
|
||||
bit = PD_ERR_CRC;
|
||||
bit = PD_RX_ERR_CRC;
|
||||
if (debug_level >= 1)
|
||||
CPRINTF("CRC%d %08x <> %08x\n", port, pcrc, ccrc);
|
||||
goto packet_err;
|
||||
@@ -1914,7 +1914,7 @@ void pd_task(void)
|
||||
pd_rx_complete(port);
|
||||
if (head > 0)
|
||||
handle_request(port, head, payload);
|
||||
else if (head == PD_ERR_HARD_RESET)
|
||||
else if (head == PD_RX_ERR_HARD_RESET)
|
||||
execute_hard_reset(port);
|
||||
} else {
|
||||
incoming_packet = 0;
|
||||
|
||||
@@ -13,13 +13,13 @@
|
||||
/* PD Host command timeout */
|
||||
#define PD_HOST_COMMAND_TIMEOUT_US SECOND
|
||||
|
||||
enum pd_errors {
|
||||
PD_ERR_INVAL = -1, /* Invalid packet */
|
||||
PD_ERR_HARD_RESET = -2, /* Got a Hard-Reset packet */
|
||||
PD_ERR_CRC = -3, /* CRC mismatch */
|
||||
PD_ERR_ID = -4, /* Invalid ID number */
|
||||
PD_ERR_UNSUPPORTED_SOP = -5, /* Unsupported SOP */
|
||||
PD_ERR_CABLE_RESET = -6 /* Got a Cable-Reset packet */
|
||||
enum pd_rx_errors {
|
||||
PD_RX_ERR_INVAL = -1, /* Invalid packet */
|
||||
PD_RX_ERR_HARD_RESET = -2, /* Got a Hard-Reset packet */
|
||||
PD_RX_ERR_CRC = -3, /* CRC mismatch */
|
||||
PD_RX_ERR_ID = -4, /* Invalid ID number */
|
||||
PD_RX_ERR_UNSUPPORTED_SOP = -5, /* Unsupported SOP */
|
||||
PD_RX_ERR_CABLE_RESET = -6 /* Got a Cable-Reset packet */
|
||||
};
|
||||
|
||||
/* incoming packet event (for the USB PD task) */
|
||||
|
||||
Reference in New Issue
Block a user