[Skiboot] [PATCH 01/11] core: add a platform op to read sensors
Cedric Le Goater
clg at fr.ibm.com
Tue Feb 17 05:12:49 AEDT 2015
On 02/12/2015 08:42 AM, Stewart Smith wrote:
> Cédric Le Goater <clg at fr.ibm.com> writes:
>> +static int64_t opal_sensor_read(uint32_t sensor_hndl, int token,
>> + uint32_t *sensor_data)
>> + if (platform.sensor_read)
>> + return platform.sensor_read(sensor_hndl, token, sensor_data);
>> + return OPAL_UNSUPPORTED;
> It seems that previous behavior is to return OPAL_PARAMETER when
> OPAL_SENSOR_READ is called with an unsupported sensor handle (at least
> from my reading of parse_sensor_id in hw/fsp/fsp-sensor.c)
yes but OPAL_UNSUPPORTED seemed to be an improvement in that case. The return
value of opal_sensor_read() is only tested against OPAL_ASYNC_COMPLETION,
so I don't think we are breaking anything.
> It would also be great if you document the OPAL_SENSOR_READ API while
> you're at it too.
OK. will do.
> It seems that on non-fsp systems you would have gotten OPAL_PARAMETER
> back as kernel should have been calling OPAL_CHECK_TOKEN first.
This is not the case. Something to fix in the kernel. I will look into it.
More information about the Skiboot