On Wed, 2018-08-01 at 11:04 +-0300, Avri Altman wrote: +AD4- +-/+ACoAKg- +AD4- +- +ACo- ufshcd+AF8-exec+AF8-raw+AF8-upiu+AF8-cmd - API function for sending raw upiu commands +AD4- +- +ACo- +AEA-hba: per-adapter instance +AD4- +- +ACo- +AEA-req+AF8-upiu: upiu request - 8 dwards +AD4- +- +ACo- +AEA-rsp+AF8-upiu: upiu reply - 8 dwards +AD4- +- +ACo- +AEA-msgcode: message code, one of UPIU Transaction Codes Initiator to Target +AD4- +- +ACo- +AEA-desc+AF8-buff: pointer to descriptor buffer, NULL if NA +AD4- +- +ACo- +AEA-buff+AF8-len: descriptor size, 0 if NA +AD4- +- +ACo- +AEA-rw: either READ or WRITE +AD4- +- +ACo- +AD4- +- +ACo- Supports UTP Transfer requests (nop and query), and UTP Task +AD4- +- +ACo- Management requests. +AD4- +- +ACo- It is up to the caller to fill the upiu conent properly, as it will +AD4- +- +ACo- be copied without any further input validations. +AD4- +- +ACo-/ +AD4- +-int ufshcd+AF8-exec+AF8-raw+AF8-upiu+AF8-cmd(struct ufs+AF8-hba +ACo-hba, +AD4- +- +AF8AXw-be32 +ACo-req+AF8-upiu, +AF8AXw-be32 +ACo-rsp+AF8-upiu, +AD4- +- int msgcode, +AD4- +- u8 +ACo-desc+AF8-buff, int +ACo-buff+AF8-len, int rw) +AD4- +-+AHs- Again, what are +ACI-dwards+ACI-? Documenting the size of a data structure in a function header is very weird. Please change the data type of req+AF8-upiu and rsp+AF8-upio into a pointer to a structure of the proper size. Thanks, Bart.
