mbox series

[v7,0/6] scsi: ufs: wb: Add sysfs attribute and cleanup

Message ID 20220804074224epcms2p18f0ac2f92073aeed981718a9cab23866@epcms2p1
Headers show
Series scsi: ufs: wb: Add sysfs attribute and cleanup | expand

Message

Jinyoung CHOI Aug. 4, 2022, 7:42 a.m. UTC 
This patch series is to clean up UFS's Write Booster code and
adds sysfs attribute which can control the specific feature of it.

V2:
	- modify commit message
	- move & modify err messages
	- remove unnesscessary debug messages
V3:
	- split patch (functional, non-functional)
V4:
	- split patch (The number of patches from 2 to 7)
	- modify dev messages
	- modify commit message
V5:
	- drop (scsi: ufs: wb: Move ufshcd_is_wb_allowed() to callee)
	- fix condition check
	- add document for sysfs attribute
	- move ufshcd_is_wb_buf_flush_allowed() to ufs-priv.h
V6:
	- Change the function name from ufshcd_wb_config() to
	ufshcd_configure_wb()
	- modify document description for wb_buf_flush_en attribute
	- fix some dev_err messages
V7:
	- renaming sysfs attribute name 
	  (wb_buf_flush_en -> enable_wb_buf_flush)
	- update document description
	- change debug message level (info -> dbg)
	  (5.19/scsi-fixes, https://git.kernel.org/mkp/scsi/c/2ae57c995003)

Jinyoung Choi (6):
  scsi: ufs: wb: Change wb_enabled condition test
  scsi: ufs: wb: Change functions name and modify parameter name
  scsi: ufs: wb: Add explicit flush sysfs attribute
  scsi: ufs: wb: Introduce ufshcd_is_wb_buf_flush_allowed() to improve
    readability
  scsi: ufs: wb: Modify messages
  scsi: ufs: wb: Move the comment to the right position

 Documentation/ABI/testing/sysfs-driver-ufs |  9 +++
 drivers/ufs/core/ufs-sysfs.c               | 47 ++++++++++++++-
 drivers/ufs/core/ufshcd-priv.h             |  6 ++
 drivers/ufs/core/ufshcd.c                  | 70 ++++++++++++----------
 include/ufs/ufshcd.h                       |  1 +
 5 files changed, 99 insertions(+), 34 deletions(-)

Comments

Martin K. Petersen Aug. 23, 2022, 3:19 a.m. UTC | #1 
Jinyoung,

> This patch series is to clean up UFS's Write Booster code and
> adds sysfs attribute which can control the specific feature of it.

Applied to 6.1/scsi-staging, thanks!