lkml.org 
[lkml]   [2022]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] scsi: ufs: Remove redundant function definitions from ufshcd.h
From
Date
Hi, Arthur,


On Mon, 2022-09-19 at 11:21 +0300, Arthur Simchaev wrote:
> This patch removes Query-Request API functions declaration from
> from include/ufs/ufshcd.h. Those declarations belongs solely to
^ unnecessary "from" ^belong

> ufs core privet header.
>
> Signed-off-by: Arthur Simchaev <Arthur.Simchaev@wdc.com>
> ---
>  drivers/ufs/core/ufshcd-priv.h |  5 +++++
>  include/ufs/ufshcd.h           | 19 +------------------
>  2 files changed, 6 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/ufs/core/ufshcd-priv.h
> b/drivers/ufs/core/ufshcd-priv.h
> index d00dba1..f68ca33 100644
> --- a/drivers/ufs/core/ufshcd-priv.h
> +++ b/drivers/ufs/core/ufshcd-priv.h
> @@ -42,6 +42,11 @@ static inline void ufs_hwmon_remove(struct ufs_hba
> *hba) {}
>  static inline void ufs_hwmon_notify_event(struct ufs_hba *hba, u8
> ee_mask) {}
>  #endif
>  
> +int ufshcd_query_descriptor_retry(struct ufs_hba *hba,
> +                                 enum query_opcode opcode,
> +                                 enum desc_idn idn, u8 index,
> +                                 u8 selector,
> +                                 u8 *desc_buf, int *buf_len);
>  int ufshcd_read_desc_param(struct ufs_hba *hba,
>                            enum desc_idn desc_id,
>                            int desc_index,
> diff --git a/include/ufs/ufshcd.h b/include/ufs/ufshcd.h
> index 24c97e0..3647a71 100644
> --- a/include/ufs/ufshcd.h
> +++ b/include/ufs/ufshcd.h
> @@ -1176,24 +1176,7 @@ static inline int
> ufshcd_disable_host_tx_lcc(struct ufs_hba *hba)
>  }
>  
>  /* Expose Query-Request API */
This comment should be removed, too?



Reviewed-by: Bean Huo <beanhuo@micron.com>


Kind regards,
Bean
\
 
 \ /
  Last update: 2022-09-20 11:09    [W:0.217 / U:0.272 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site