On 03/24/2018 02:45 PM, Helge Deller wrote:
This patch allows HP PA-RISC servers like rp3410/rp3440 and the HP C8000
workstation with an IPMI controller that predate IPMI 1.5 to use the standard
poweroff or powercycle commands.
These systems firmware don't set the chassis capability bit in the Get
Device ID, but they do implement the standard poweroff and powercycle
commands.
A few comments on this:
That function is named ipmi_dell_chassis_detect(), just adding a HP
chassis to it doesn't seem like a good idea. There is already an array
of poweroff functions, please use that and create your own function.
You would have to restructure the dell function to make it clean,
and I don't want to do that.
The CONFIG_PARISC clause probably doesn't matter.
Please create defines for the magic numbers.
Thanks,
-corey
Signed-off-by: Helge Deller <del...@gmx.de>
diff --git a/drivers/char/ipmi/ipmi_poweroff.c
b/drivers/char/ipmi/ipmi_poweroff.c
index 38e6af1c8e38..09f34ea370da 100644
--- a/drivers/char/ipmi/ipmi_poweroff.c
+++ b/drivers/char/ipmi/ipmi_poweroff.c
@@ -453,6 +453,13 @@ static int ipmi_dell_chassis_detect(ipmi_user_t user)
ipmi_version_major <= 1 &&
ipmi_version_minor < 5)
return 1;
+
+ /* BMC in HP C8000 can handle chassis functions too. */
+ if (IS_ENABLED(CONFIG_PARISC) &&
+ mfg_id == 0x0b && prod_id == 0x8201 &&
+ ipmi_version == 1)
+ return 1;
+
return 0;
}
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openipmi-developer mailing list
Openipmi-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openipmi-developer
