Finished decoding of DMI type 38 (IPMI).

Author: jdelvare

CHANGELOG

@@ -1,16 +1,21 @@
+2003-03-08  Jean Delvare  <[email protected]>
+
+	* dmidecode.c: Decoded more fields according to the IPMI specification
+	(DMI case 38).
+
 2003-03-07  Jean Delvare  <[email protected]>
 
 	Fixed IPMI device information (DMI case 38). Thanks to Richard Sharpe
 	for pointing the bugs out.
 
 	* dmidecode.c: Fixed IPMI interface type being shifted by one.
 	* dmidecode.c: Fixed NV storage device being improperly displayed.
-	* dmidecode.c: Reword IPMI specification revision into specification
+	* dmidecode.c: Reworded IPMI specification revision into specification
 	version, as suggested in the IPMI specification itself.
 	* dmidecode.c: Added a reference to the IPMI specification.
 	* dmidecode.c: Show I2C address as hexadecimal.
 	* dmidecode.c: Base address is a QWORD, not DWORD.
-	* dmidecode.c: Decode some extra fields according to the IPMI
+	* dmidecode.c: Decoded some extra fields according to the IPMI
 	specification.
 
 2003-03-06  Jean Delvare  <[email protected]>

dmidecode.c

@@ -3668,14 +3668,14 @@ static void dmi_decode(u8 *data, u16 ver)
 			dmi_memory_channel_devices(data[0x06], data+0x07, "\t\t\t");
 			break;
 
-		/*
-		 * We use the word "Version" instead of "Revision", conforming to
-		 * IPMI 1.5 specification. This specification isn't very clear
-		 * regarding the I2C slave address. I couldn't understand wether
-		 * or not we are supposed to shift it by one bit to the right, so
-		 * I leave it untouched. Beware it might be wrong.
-		 */
 		case 38: /* 3.3.39 IPMI Device Information */
+			/*
+			 * We use the word "Version" instead of "Revision", conforming to
+			 * the IPMI 1.5 specification. This specification isn't very clear
+			 * regarding the I2C slave address. I couldn't understand wether
+			 * or not we are supposed to shift it by one bit to the right, so
+			 * I leave it untouched. Beware it might be wrong.
+			 */
 			printf("\tIPMI Device Information\n");
 			if(h->length<0x10) break;
 			printf("\t\tInterface Type: %s\n",
@@ -3697,10 +3697,23 @@ static void dmi_decode(u8 *data, u16 ver)
 				break;
 			}
 			printf("\t\tBase Address: 0x%08X%08X (%s)\n",
-				QWORD(data+0x08).h, (QWORD(data+0x08).l&~1)|((data[0x10]>>5)&1),
+				QWORD(data+0x08).h,
+				(QWORD(data+0x08).l&~1)|((data[0x10]>>5)&1),
 				QWORD(data+0x08).l&1?"I/O":"Memory-mapped");
 			printf("\t\tRegister Spacing: %s\n",
 				dmi_ipmi_register_spacing(data[0x10]>>6));
+			if(data[0x10]&(1<<3))
+			{
+				printf("\t\tInterrupt Polarity: %s\n",
+					data[0x10]&(1<<1)?"Active High":"Active Low");
+				printf("\t\tInterrupt Trigger Mode: %s\n",
+					data[0x10]&(1<<0)?"Level":"Edge");
+			}
+			if(data[0x11]!=0x00)
+			{
+				printf("\t\tInterrupt Number: %x\n",
+					data[0x11]);
+			}
 			break;
 
 		case 39: /* 3.3.40 System Power Supply */