mbox series

[v4,0/3] Changes for ufshcd.c

Message ID 20221022213650.626766-1-beanhuo@iokpp.de
Headers show
Series Changes for ufshcd.c | expand

Message

Bean Huo Oct. 22, 2022, 9:36 p.m. UTC
From: Bean Huo <beanhuo@micron.com>

v3--v4:
    1. Change patch 1/3 title
    2. Fix one typo in patch 3/3
    3. Fix coding type issue in  patch 2/3 based on Bart's comment

v2--v3:
    1. Add patch 3/3
    2. Add remove unused parameter param_offset in patch 1/3 based on Daejun's comment
    3. Further optimized patch 2/3 based on Bart's comment

v1--v2:
    1. change in ufshcd_lu_init() in  patch 2/2:
        1) Remove duplicate parameter initialization
        2) Move lun parameter initialization before kmalloc()


Bean Huo (3):
  scsi: ufs: core: Revert "WB is only available on LUN #0 to #7"
  scsi: ufs: core: Cleanup ufshcd_slave_alloc()
  scsi: ufs: core: Use is_visible to control UFS unit descriptor sysfs
    nodes

 drivers/ufs/core/ufs-sysfs.c   |  21 ++++-
 drivers/ufs/core/ufshcd-priv.h |   6 +-
 drivers/ufs/core/ufshcd.c      | 152 ++++++++++++---------------------
 3 files changed, 74 insertions(+), 105 deletions(-)

Comments

Arthur Simchaev Oct. 23, 2022, 9:47 a.m. UTC | #1
> -----Original Message-----
> From: Bean Huo <beanhuo@iokpp.de>
> Sent: Sunday, October 23, 2022 12:37 AM
> To: alim.akhtar@samsung.com; Avri Altman <Avri.Altman@wdc.com>;
> asutoshd@codeaurora.org; jejb@linux.ibm.com;
> martin.petersen@oracle.com; stanley.chu@mediatek.com;
> beanhuo@micron.com; bvanassche@acm.org; tomas.winkler@intel.com;
> cang@codeaurora.org; daejun7.park@samsung.com
> Cc: linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: [PATCH v4 2/3] scsi: ufs: core: Cleanup ufshcd_slave_alloc()
> 
> CAUTION: This email originated from outside of Western Digital. Do not click
> on links or open attachments unless you recognize the sender and know that
> the content is safe.
> 
> 
> From: Bean Huo <beanhuo@micron.com>
> 
> Combine ufshcd_get_lu_power_on_wp_status() and
> ufshcd_set_queue_depth()
> into one single ufshcd_lu_init(), so that we only need to read the LUN
> descriptor once.
> 
> Signed-off-by: Bean Huo <beanhuo@micron.com>

Reviewed-by: Arthur Simchaev <arthur.simchaev@wdc.com>