[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-parisc
Subject:    Re: [PATCH v2] ipmi/parisc: Add IPMI chassis poweroff for certain HP PA-RISC and IA-64 servers
From:       Corey Minyard <minyard () acm ! org>
Date:       2018-03-27 16:05:17
Message-ID: 45a3f748-0485-442b-6b6f-6dac49b89990 () acm ! org
[Download RAW message or body]

On 03/27/2018 10:58 AM, 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.

Perfect, thanks for the update.   Included for next release.

-corey

> Signed-off-by: Helge Deller <deller@gmx.de>
>
> diff --git a/drivers/char/ipmi/ipmi_poweroff.c b/drivers/char/ipmi/ipmi_poweroff.c
> index 38e6af1c8e38..943c8bc1c364 100644
> --- a/drivers/char/ipmi/ipmi_poweroff.c
> +++ b/drivers/char/ipmi/ipmi_poweroff.c
> @@ -457,6 +457,24 @@ static int ipmi_dell_chassis_detect(ipmi_user_t user)
>   }
>   
>   /*
> + * ipmi_hp_chassis_detect()
> + * HP PA-RISC servers rp3410/rp3440, the C8000 workstation and the rx2600 and
> + * zx6000 machines support IPMI vers 1 and don't set the chassis capability bit
> + * but they can handle a chassis poweroff or powercycle command.
> + */
> +
> +#define HP_IANA_MFR_ID 0x0b
> +#define HP_BMC_PROD_ID 0x8201
> +static int ipmi_hp_chassis_detect(ipmi_user_t user)
> +{
> +	if (mfg_id == HP_IANA_MFR_ID
> +		&& prod_id == HP_BMC_PROD_ID
> +		&& ipmi_version == 1)
> +		return 1;
> +	return 0;
> +}
> +
> +/*
>    * Standard chassis support
>    */
>   
> @@ -533,6 +551,9 @@ static struct poweroff_function poweroff_functions[] = {
>   	{ .platform_type	= "chassis",
>   	  .detect		= ipmi_dell_chassis_detect,
>   	  .poweroff_func	= ipmi_poweroff_chassis },
> +	{ .platform_type	= "chassis",
> +	  .detect		= ipmi_hp_chassis_detect,
> +	  .poweroff_func	= ipmi_poweroff_chassis },
>   	/* Chassis should generally be last, other things should override
>   	   it. */
>   	{ .platform_type	= "chassis",


--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]