On 27/02/18 09:21, Uma Krishnan wrote:
The OCXL specification supports distributing acTags amongst different
AFUs and functions on the link. As cxlflash devices are expected to only
support a single AFU and function, the entire range that was assigned to
the function is also assigned to the AFU.

Signed-off-by: Uma Krishnan <ukri...@linux.vnet.ibm.com>
Acked-by: Matthew R. Ochs <mro...@linux.vnet.ibm.com>

Reviewed-by: Andrew Donnellan <andrew.donnel...@au1.ibm.com>

diff --git a/drivers/scsi/cxlflash/ocxl_hw.h b/drivers/scsi/cxlflash/ocxl_hw.h
index f6af247..9c675fa 100644
--- a/drivers/scsi/cxlflash/ocxl_hw.h
+++ b/drivers/scsi/cxlflash/ocxl_hw.h
@@ -22,4 +22,6 @@ struct ocxl_hw_afu {

        int fn_actag_base;              /* Function acTag base */
        int fn_actag_enabled;           /* Function acTag number enabled */
+       int afu_actag_base;             /* AFU acTag base */
+       int afu_actag_enabled;          /* AFU acTag number enabled */
  };


Is it ever foreseen that you will need to support more than one AFU on a function? I guess it's good to keep the fn and afu values separate anyway just in case...

--
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnel...@au1.ibm.com  IBM Australia Limited

Reply via email to