From e56ba2e689d2814b8428a14aee2ea769a80744d8 Mon Sep 17 00:00:00 2001 From: David Hendricks Date: Fri, 20 Apr 2012 17:06:24 -0700 Subject: 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 Change-Id: Id4634076ad63f45783354179dfebea4fd450fc1e --- include/message.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'include') 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. * -- cgit v1.2.1