scsi: ufs: Remove check_upiu_size() from ufshcd.h

Message ID 20221103055349epcms2p338f2550c2dd78d00231a83b24719a3d4@epcms2p3
State New
Headers
Series scsi: ufs: Remove check_upiu_size() from ufshcd.h |

Commit Message

Keoseong Park Nov. 3, 2022, 5:53 a.m. UTC
  Commit 68078d5cc1a5 ("[SCSI] ufs: Set fDeviceInit flag to initiate
device initialization") added check_upiu_size(), but no caller.

Cc: Dolev Raviv <draviv@codeaurora.org>
Signed-off-by: Keoseong Park <keosung.park@samsung.com>
---
 include/ufs/ufshcd.h | 6 ------
 1 file changed, 6 deletions(-)
  

Comments

Avri Altman Nov. 3, 2022, 7:04 a.m. UTC | #1
> Commit 68078d5cc1a5 ("[SCSI] ufs: Set fDeviceInit flag to initiate device
> initialization") added check_upiu_size(), but no caller.
Nor there ever was.

> 
> Cc: Dolev Raviv <draviv@codeaurora.org>
> Signed-off-by: Keoseong Park <keosung.park@samsung.com>
Reviewed-by: Avri Altman <avri.altman@wdc.com>

> ---
>  include/ufs/ufshcd.h | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/include/ufs/ufshcd.h b/include/ufs/ufshcd.h index
> 96538eb3a6c0..5cf81dff60aa 100644
> --- a/include/ufs/ufshcd.h
> +++ b/include/ufs/ufshcd.h
> @@ -1072,12 +1072,6 @@ void ufshcd_update_evt_hist(struct ufs_hba *hba,
> u32 id, u32 val);  void ufshcd_hba_stop(struct ufs_hba *hba);  void
> ufshcd_schedule_eh_work(struct ufs_hba *hba);
> 
> -static inline void check_upiu_size(void) -{
> -       BUILD_BUG_ON(ALIGNED_UPIU_SIZE <
> -               GENERAL_UPIU_REQUEST_SIZE + QUERY_DESC_MAX_SIZE);
> -}
> -
>  /**
>   * ufshcd_set_variant - set variant specific data to the hba
>   * @hba: per adapter instance
> --
> 2.17.1
  
Martin K. Petersen Nov. 8, 2022, 3:06 a.m. UTC | #2
Keoseong,

> Commit 68078d5cc1a5 ("[SCSI] ufs: Set fDeviceInit flag to initiate
> device initialization") added check_upiu_size(), but no caller.

Applied to 6.2/scsi-staging, thanks!
  
Martin K. Petersen Nov. 17, 2022, 6:29 p.m. UTC | #3
On Thu, 03 Nov 2022 14:53:49 +0900, Keoseong Park wrote:

> Commit 68078d5cc1a5 ("[SCSI] ufs: Set fDeviceInit flag to initiate
> device initialization") added check_upiu_size(), but no caller.
> 
> 

Applied to 6.2/scsi-queue, thanks!

[1/1] scsi: ufs: Remove check_upiu_size() from ufshcd.h
      https://git.kernel.org/mkp/scsi/c/3d6d7930928a
  

Patch

diff --git a/include/ufs/ufshcd.h b/include/ufs/ufshcd.h
index 96538eb3a6c0..5cf81dff60aa 100644
--- a/include/ufs/ufshcd.h
+++ b/include/ufs/ufshcd.h
@@ -1072,12 +1072,6 @@  void ufshcd_update_evt_hist(struct ufs_hba *hba, u32 id, u32 val);
 void ufshcd_hba_stop(struct ufs_hba *hba);
 void ufshcd_schedule_eh_work(struct ufs_hba *hba);
 
-static inline void check_upiu_size(void)
-{
-	BUILD_BUG_ON(ALIGNED_UPIU_SIZE <
-		GENERAL_UPIU_REQUEST_SIZE + QUERY_DESC_MAX_SIZE);
-}
-
 /**
  * ufshcd_set_variant - set variant specific data to the hba
  * @hba: per adapter instance