commit 3c7ec2d02be653761d963a3710650f999ca6648f
author Adrian Ambrożewicz <adrian.ambrozewicz@intel.com> Wed Sep 22 09:42:42 2021 +0200
committer Adrian Ambrożewicz <adrian.ambrozewicz@linux.intel.com> Wed Sep 29 13:09:44 2021 +0000
tree e9ed62412f39b48fcd395e1e9286b013b1f6fe62
parent e83c70aab0479a8103638166b330a06e499f4449

Fix assigning Generator ID for IPMB path

Per IPMI spec - Generator ID is defined as follows for I2C sender:
[7-1] = I2C Slave Address
[0] = 0 (indicates IPMB Slave Address)

This change is introducing proper shift for IPMB path, so Generator ID
is correctly assigned.

Tested:
- tested on actual system with IPMB events originating from ME,
- after this change SEL entries from ME have correct generator ID (0x2C).

Change-Id: I5b94e2242e5ac2b4d4ecc5c7e7cd9e8a93bd98ab
Signed-off-by: Adrian Ambrożewicz <adrian.ambrozewicz@intel.com>

src/sensorcommands.cpp

diff --git a/src/sensorcommands.cpp b/src/sensorcommands.cpp
index 2512d1b..b27a346 100644
--- a/src/sensorcommands.cpp
+++ b/src/sensorcommands.cpp
@@ -385,7 +385,7 @@
 
         p.unpack(evmRev, sensorType, sensorNum, eventType, eventData1,
                  eventData2, eventData3);
-        generatorID = ctx->rqSA;
+        generatorID = ctx->rqSA << 1;
     }
 
     if (!p.fullyUnpacked())