diff options
| author | David Hendricks <dhendrix@chromium.org> | 2012-04-20 17:06:24 -0700 |
|---|---|---|
| committer | David Hendricks <dhendrix@chromium.org> | 2012-04-23 14:05:41 -0700 |
| commit | e56ba2e689d2814b8428a14aee2ea769a80744d8 (patch) | |
| tree | 98da4a65b05cdbbf3ca4d167572ae69bbae0c014 /include | |
| parent | 66fa2685f021ca1fc482f46f60bca49b69527f37 (diff) | |
| download | chrome-ec-e56ba2e689d2814b8428a14aee2ea769a80744d8.tar.gz | |
Update message protocol length byte to include postamble
This simply changes the constant '4' to MSG_PROTO_BYTES, which
includes the postamble. This helps reduce magic constant values
used in processing a packet.
BUG=chrome-os-partner:8975
TEST=tested on daisy (with kernel mkbp driver change)
Signed-off-by: David Hendricks <dhendrix@chromium.org>
Change-Id: Id4634076ad63f45783354179dfebea4fd450fc1e
Diffstat (limited to 'include')
| -rw-r--r-- | include/message.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/message.h b/include/message.h index 867dff8b58..fa41d9597c 100644 --- a/include/message.h +++ b/include/message.h @@ -46,8 +46,8 @@ enum message_cmd_t { * The hdr byte is just a tag to indicate that the real message follows. It * signals the end of any preamble required by the interface. * - * The 16-bit length includes the length of the header, len bytes, message - * and checksum, but not the final preamble byte. + * The 16-bit length is the entire packet size, including the header, length + * bytes, message payload, checksum, and postamble byte. * * The checksum is calculated as the sum of the header, len byte and message. * |