[Skiboot] [PATCH] hw/bt: fix IPMI_GET_BT_CAPS definition
Cédric Le Goater
clg at fr.ibm.com
Wed Jan 6 19:08:03 AEDT 2016
On 01/06/2016 07:36 AM, Cyril Bur wrote:
> On Wed, 06 Jan 2016 16:31:51 +1100
> Stewart Smith <stewart at linux.vnet.ibm.com> wrote:
>
>> Cédric Le Goater <clg at fr.ibm.com> writes:
>>> Signed-off-by: Cédric Le Goater <clg at fr.ibm.com>
>>> ---
>>> include/ipmi.h | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> Index: skiboot.git/include/ipmi.h
>>> ===================================================================
>>> --- skiboot.git.orig/include/ipmi.h
>>> +++ skiboot.git/include/ipmi.h
>>> @@ -117,7 +117,7 @@
>>> #define IPMI_GET_MESSAGE_FLAGS IPMI_CODE(IPMI_NETFN_APP, 0x31)
>>> #define IPMI_GET_MESSAGE IPMI_CODE(IPMI_NETFN_APP, 0x33)
>>> #define IPMI_READ_EVENT IPMI_CODE(IPMI_NETFN_APP, 0x35)
>>> -#define IPMI_GET_BT_CAPS IPMI_CODE(IPMI_NETFN_APP, 0x56)
>>> +#define IPMI_GET_BT_CAPS IPMI_CODE(IPMI_NETFN_APP, 0x36)
>>> #define IPMI_SET_SENSOR_READING IPMI_CODE(IPMI_NETFN_SE, 0x30)
>>>
>>> /* AMI OEM comamnds. AMI uses NETFN 0x3a and 0x32 */
>>
>> Which bit of doc/standard has this in it? From quickly leafing through
>> the IPMI spec I didn't see it, but then again, I was doing naive search :)
>>
>
> Those documents are terrible, don't go to section 22.10 because despite being
> where you might expect to find the information you want, you wont. In
> http://www.intel.com.au/content/dam/www/public/us/en/documents/product-briefs/ipmi-second-gen-interface-spec-v2-rev1-1.pdf
> page 587 under BMC Device and Messaging Commands :)
>
yes. the IPMI specs is a but hard to digest ... Appendix G has all
the command numbers.
C.
More information about the Skiboot
mailing list