[Skiboot] [PATCH v11 09/11] skiboot: Add opal call to enable/disable Nest IMC

Anju T Sudhakar anju at linux.vnet.ibm.com
Fri May 12 16:26:33 AEST 2017


Hi Stewart,

Thanks for the review comments.

On Friday 12 May 2017 07:50 AM, Stewart Smith wrote:
> Madhavan Srinivasan <maddy at linux.vnet.ibm.com> writes:
>> + * opal_imc_counters_init : This call initialize the IMC engine.
>> + *
>> + * This call is not being used in case of NEST IMC.
>> + * Additional arguments will be added to this call in the following patch.
>> + */
>> +static int64_t opal_imc_counters_init(uint32_t type)
>> +{
>> +	if (type == OPAL_IMC_COUNTERS_NEST)
>> +		prerror("IMC: unknown operation for nest imc\n");
> I don't think you should prerror here, instead, just return
> OPAL_SUCCESS. Initing the nest counters is a no-op, so trying to init
> them will always succeed.
>
ok. will do.


Thanks and regards,
Anju



More information about the Skiboot mailing list