mbox series

[v2,00/12] Multiple cleanup patches for the UFS driver

Message ID 20230727194457.3152309-1-bvanassche@acm.org
Headers show
Series Multiple cleanup patches for the UFS driver | expand

Message

Bart Van Assche July 27, 2023, 7:41 p.m. UTC 
Hi Martin,

This patch includes the following changes, none of which should change the
functionality of the UFS host controller driver:
- Improve the kernel-doc headers further.
- Fix multiple W=2 compiler warnings.
- Simplify ufshcd_abort_all().
- Simplify the code for creating and parsing UFS Transport Protocol (UTP)
  headers.

Please consider this patch series for the next merge window.

Thanks,

Bart.

Changes compared to v1:
 - Introduced a name for the data direction enumeration type.
 - Instead of introducing struct utp_upiu_hdr as an alias for struct
   utp_upiu_header, introduce a union in the latter data structure.

Bart Van Assche (12):
  scsi: ufs: Follow the kernel-doc syntax for documenting return values
  scsi: ufs: Document all return values
  scsi: ufs: Fix kernel-doc headers
  scsi: ufs: Rename a function argument
  scsi: ufs: Minimize #include directives
  scsi: ufs: Simplify zero-initialization
  scsi: ufs: Improve type safety
  scsi: ufs: Remove a local variable from ufshcd_abort_all()
  scsi: ufs: Simplify ufshcd_abort_all()
  scsi: ufs: Remove a member variable
  scsi: ufs: Simplify transfer request header initialization
  scsi: ufs: Simplify response header parsing

 drivers/ufs/core/ufs-hwmon.c       |   3 +-
 drivers/ufs/core/ufs-mcq.c         |  17 +-
 drivers/ufs/core/ufs_bsg.c         |   2 +
 drivers/ufs/core/ufshcd-crypto.h   |  20 +-
 drivers/ufs/core/ufshcd-priv.h     |   4 +-
 drivers/ufs/core/ufshcd.c          | 529 +++++++++++++++--------------
 drivers/ufs/host/cdns-pltfrm.c     |  27 +-
 drivers/ufs/host/tc-dwc-g210-pci.c |   2 +-
 drivers/ufs/host/tc-dwc-g210.c     |  32 +-
 drivers/ufs/host/ufs-mediatek.c    |   6 +-
 drivers/ufs/host/ufs-qcom.c        |   8 +-
 drivers/ufs/host/ufshcd-dwc.c      |  22 +-
 drivers/ufs/host/ufshcd-pci.c      |   2 +-
 drivers/ufs/host/ufshcd-pltfrm.c   |   6 +-
 include/uapi/scsi/scsi_bsg_ufs.h   |  52 ++-
 include/ufs/ufs.h                  |  47 +--
 include/ufs/ufshcd.h               |  26 +-
 include/ufs/ufshci.h               |  53 ++-
 18 files changed, 460 insertions(+), 398 deletions(-)

Comments

Martin K. Petersen July 31, 2023, 7:19 p.m. UTC | #1 
Bart,

> This patch includes the following changes, none of which should change the
> functionality of the UFS host controller driver:
> - Improve the kernel-doc headers further.
> - Fix multiple W=2 compiler warnings.
> - Simplify ufshcd_abort_all().
> - Simplify the code for creating and parsing UFS Transport Protocol (UTP)
>   headers.

Applied to 6.6/scsi-staging, thanks!
Martin K. Petersen Aug. 8, 2023, 2:50 a.m. UTC | #2 
On Thu, 27 Jul 2023 12:41:12 -0700, Bart Van Assche wrote:

> This patch includes the following changes, none of which should change the
> functionality of the UFS host controller driver:
> - Improve the kernel-doc headers further.
> - Fix multiple W=2 compiler warnings.
> - Simplify ufshcd_abort_all().
> - Simplify the code for creating and parsing UFS Transport Protocol (UTP)
>   headers.
> 
> [...]

Applied to 6.6/scsi-queue, thanks!

[01/12] scsi: ufs: Follow the kernel-doc syntax for documenting return values
	https://git.kernel.org/mkp/scsi/c/3a17fefe0f19
[02/12] scsi: ufs: Document all return values
        https://git.kernel.org/mkp/scsi/c/fd4bffb54dc0
[03/12] scsi: ufs: Fix kernel-doc headers
        https://git.kernel.org/mkp/scsi/c/8d8af294ce03
[04/12] scsi: ufs: Rename a function argument
        https://git.kernel.org/mkp/scsi/c/f08191520614
[05/12] scsi: ufs: Minimize #include directives
        https://git.kernel.org/mkp/scsi/c/cce9fd602ca0
[06/12] scsi: ufs: Simplify zero-initialization
        https://git.kernel.org/mkp/scsi/c/f99533bd7e3d
[07/12] scsi: ufs: Improve type safety
        https://git.kernel.org/mkp/scsi/c/08108d31129a
[08/12] scsi: ufs: Remove a local variable from ufshcd_abort_all()
        https://git.kernel.org/mkp/scsi/c/e8b0234f8458
[09/12] scsi: ufs: Simplify ufshcd_abort_all()
        https://git.kernel.org/mkp/scsi/c/f9c028e7415a
[10/12] scsi: ufs: Remove a member variable
        https://git.kernel.org/mkp/scsi/c/e2566e0b7937
[11/12] scsi: ufs: Simplify transfer request header initialization
        https://git.kernel.org/mkp/scsi/c/67a2a8973832
[12/12] scsi: ufs: Simplify response header parsing
        https://git.kernel.org/mkp/scsi/c/617bfaa8dd50