[Skiboot] [PATCH 2/7] opal: Introduce opal_del_host_sync_notifier_with_data()

Vaibhav Jain vaibhav at linux.ibm.com
Fri Sep 21 16:38:55 AEST 2018


Thanks for reviewing thie patch Vasant,

Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
> Patch itself looks good to me. But may be its better to convert existing
> opal_del_host_sync_notifier() itself? Anyway no one is using that interface 
> today and nothing
> harm is adding additional check for `data` as well.
That seems like a better idea. I will address it in v2 I am working on.

-- 
Vaibhav Jain <vaibhav at linux.ibm.com>
Linux Technology Center, IBM India Pvt. Ltd.



More information about the Skiboot mailing list