add more comments, adjust size from 255 to 254.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@6372 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/MdePkg/Include/IndustryStandard/Atapi.h b/MdePkg/Include/IndustryStandard/Atapi.h
index 9d2e5a9c3d17758173cc4a925a61feba990dc283..0f34d8dfc4760e02b65d55e076c4ab2c0501443c 100644 (file)
--- a/MdePkg/Include/IndustryStandard/Atapi.h
+++ b/MdePkg/Include/IndustryStandard/Atapi.h
@@ -84,14 +84,14 @@ typedef struct {
 } ATAPI_IDENTIFY_DATA;\r
 \r
 ///\r
-/// Quiry Standard Data fomrat, defined in SFF-8070i(ATAPI Removable Rewritable Specification)\r
+/// Standard Quiry Data format, defined in SFF-8070i(ATAPI Removable Rewritable Specification)\r
 ///\r
 typedef struct {\r
   UINT8 peripheral_type;\r
   UINT8 RMB;\r
   UINT8 version;\r
   UINT8 response_data_format;\r
-  UINT8 addnl_length;\r
+  UINT8 addnl_length;     ///< n - 4, Numbers of bytes following this one\r
   UINT8 reserved_5;\r
   UINT8 reserved_6;\r
   UINT8 reserved_7;\r
@@ -101,10 +101,10 @@ typedef struct {
   UINT8 vendor_specific_36_55[55 - 36 + 1];\r
   UINT8 reserved_56_95[95 - 56 + 1];\r
   ///\r
-  /// Vendor specific parameters fields, the sizeof (ATAPI_INQUIRY_DATA) is 255\r
+  /// Vendor specific parameters fields, the sizeof (ATAPI_INQUIRY_DATA) is 254\r
   /// since allocation_length is one byte in ATAPI_INQUIRY_CMD.\r
   ///\r
-  UINT8 vendor_specific_96_255[254 - 96 + 1];\r
+  UINT8 vendor_specific_96_253[253 - 96 + 1];\r
 } ATAPI_INQUIRY_DATA;\r
 \r
 ///\r
@@ -135,10 +135,10 @@ typedef struct {
   UINT8 sense_key_specific_17;\r
   ///\r
   /// Followed by additional sense bytes.\r
-  /// the sizeof (ATAPI_REQUEST_SENSE_DATA) is 255, \r
+  /// the sizeof (ATAPI_REQUEST_SENSE_DATA) is 254, \r
   /// since allocation_length is one byte in ATAPI_INQUIRY_CMD.\r
   ///\r
-  UINT8 additional_sense_bytes[254 - 18 + 1];\r
+  UINT8 additional_sense_bytes_18_253[253 - 18 + 1];\r
 } ATAPI_REQUEST_SENSE_DATA;\r
 \r
 ///\r
@@ -417,6 +417,8 @@ typedef union {
 \r
 #define ATAPI_MAX_BYTE_COUNT  (0xfffe)\r
 \r
+#define ATA_REQUEST_SENSE_ERROR (0x70) ///< defined in SFF-8070i\r
+\r
 ///\r
 /// Sense Key, Additional Sense Codes and Additional Sense Code Qualifier\r
 /// defined in MultiMedia Commands (MMC, MMC-2) \r
@@ -440,9 +442,9 @@ typedef union {
 #define ATA_SK_MISCOMPARE       (0xE)\r
 #define ATA_SK_RESERVED_F       (0xF)\r
 \r
-//\r
-// Additional Sense Codes\r
-//\r
+///\r
+/// Additional Sense Codes\r
+///\r
 #define ATA_ASC_NOT_READY                   (0x04)\r
 #define ATA_ASC_MEDIA_ERR1                  (0x10)\r
 #define ATA_ASC_MEDIA_ERR2                  (0x11)\r