<font size=2 face="sans-serif">The interface looks good to me.</font><br><font size=2 face="sans-serif"><br>--<br>Dan Crowell<br>Senior Software Engineer - Power Systems Enablement Firmware<br>IBM Rochester: t/l 553-2987<br>dcrowell@us.ibm.com</font><br><br><br><br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Jeremy Kerr <jk@ozlabs.org></font><br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">skiboot@lists.ozlabs.org,
Daniel M Crowell/Rochester/IBM@IBMUS</font><br><font size=1 color=#5f5f5f face="sans-serif">Cc:      
 </font><font size=1 face="sans-serif">Benjamin Herrenschmidt
<benh@kernel.crashing.org>, Corey Swenson/Rochester/IBM@IBMUS, Patrick
Barrett/Rochester/IBM@IBMUS, Jeremy Kerr <jk@ozlabs.org></font><br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">09/14/2016 08:57 PM</font><br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">[PATCH v2 0/2]
opal-prd: Fix ignored error reporting on xscom operations, introduce prd
flags</font><br><hr noshade><br><br><br><tt><font size=2>Hi all,<br><br>Currently, we have a bug in opal-prd where we do not properly report<br>errors from xscom read/write ioctls returned from the kernel / firmware.<br><br>Patch 2 in this series fixes this issue.<br><br>However, we'd also like to report to HBRT that the fix is present. Patch<br>1 introduces a get_prd_flags() callback to the hostboot interface, to<br>allow indicications of variations in behaviour like the fix above.<br><br>Dan: this is a proposed change to struct host_interfaces, so we'll need<br>an Acked-by from your side before merging.<br><br>Cheers,<br><br><br>Jeremy<br><br>---<br>v2:<br>  - rename callback to get_interface_capabilities<br>  - implement capability sets<br><br>Benjamin Herrenschmidt (1):<br>  opal-prd: Fix error code from scom_read & scom_write<br><br>Jeremy Kerr (1):<br>  opal-prd: Add get_interface_capabilities to host interfaces<br><br> external/opal-prd/hostboot-interface.h | 24 +++++++++++++++++++++++-<br> external/opal-prd/opal-prd.c           | 22 ++++++++++++++++------<br> external/opal-prd/thunk.S              |
 3 ++-<br> 3 files changed, 41 insertions(+), 8 deletions(-)<br><br>-- <br>2.7.4<br><br></font></tt><br><br><BR>