commit 5295cefb5fc27e274af92d4ecad56e7e0d6b4e88
author Adrian Ambrożewicz <adrian.ambrozewicz@intel.com> Wed Sep 22 09:42:42 2021 +0200
committer Willy Tu <wltu@google.com> Tue Mar 15 23:24:13 2022 +0000
tree b1bf2613dbae3ef20f5f838868cd4d9d1d56ca26
parent 897ae72f1790817c67aa2e3a87cc0992eb997959

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).

Ported From:
https://gerrit.openbmc-project.xyz/c/openbmc/intel-ipmi-oem/+/47090

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

dbus-sdr/sensorcommands.cpp

diff --git a/dbus-sdr/sensorcommands.cpp b/dbus-sdr/sensorcommands.cpp
index 81bdf23..ac0b52d 100644
--- a/dbus-sdr/sensorcommands.cpp
+++ b/dbus-sdr/sensorcommands.cpp
@@ -571,7 +571,7 @@
 
         p.unpack(evmRev, sensorType, sensorNum, eventType, eventData1,
                  eventData2, eventData3);
-        generatorID = ctx->rqSA;
+        generatorID = ctx->rqSA << 1;
     }
 
     if (!p.fullyUnpacked())