Messages in this thread | | | From | Avri Altman <> | Subject | RE: [PATCH] scsi: ufs: revamp string descriptor reading | Date | Tue, 11 Dec 2018 16:18:27 +0000 |
| |
Hello Tomas,
> > Define new a type: uc_string_id for easier string > handling and less casting. Reduce number or string > copies in price of a dynamic allocation. > > Signed-off-by: Tomas Winkler <tomas.winkler@intel.com> Tested-by: Avri Altman <avri.altman@wdc.com>
Just one nit - doesn't really matters.
Cheers, Avri
> --- > drivers/scsi/ufs/ufs-sysfs.c | 20 ++--- > drivers/scsi/ufs/ufs.h | 2 +- > drivers/scsi/ufs/ufshcd.c | 164 +++++++++++++++++++++-------------- > drivers/scsi/ufs/ufshcd.h | 9 +- > 4 files changed, 115 insertions(+), 80 deletions(-)
> > ufs_fixup_device_setup(hba, &card); > + ufs_put_device_desc(&card); ufs_get_device_desc() and ufs_put_device_desc() actually serves the quirks setup. Make sense to call it from within so the logic is clear and in one place. Might also save ufs_put_device_desc().
| |