Re: [PATCH v2 6/6] cxlflash: Enable device id for future IBM CXL adapter
From: Andrew Donnellan <hidden>
Date: 2015-12-15 03:12:07
Also in:
linux-scsi
From: Andrew Donnellan <hidden>
Date: 2015-12-15 03:12:07
Also in:
linux-scsi
On 15/12/15 08:07, Uma Krishnan wrote:
From: Manoj Kumar <redacted> This drop enables a future card with a device id of 0x0600 to be recognized by the cxlflash driver.
If you do a V3 - this could be better worded in the imperative mood as: "Add a new device ID (0x0600) for a future IBM CXL Flash Adapter card."
As per the design, the Accelerator Function Unit (AFU) for this new IBM CXL Flash Adapter retains the same host interface as the previous generation. For the early prototypes of the new card, the driver with this change behaves exactly as the driver prior to this behaved with the earlier generation card. Therefore, no card specific programming has been added. These card specific changes can be staged in later if needed. Signed-off-by: Manoj N. Kumar <redacted> Acked-by: Matthew R. Ochs <redacted>
Assuming that the final card won't break in a particularly nasty way in the absence of later code changes, all looks good. Reviewed-by: Andrew Donnellan <redacted> -- Andrew Donnellan Software Engineer, OzLabs andrew.donnellan@au1.ibm.com Australia Development Lab, Canberra +61 2 6201 8874 (work) IBM Australia Limited