diff mbox series

scsi: ufs: core: Remove unused field in struct ufs_hba

Message ID 413601558.101648105683746.JavaMail.epsvc@epcpadp4 (mailing list archive)
State Accepted
Headers show
Series scsi: ufs: core: Remove unused field in struct ufs_hba | expand

Commit Message

Keoseong Park March 24, 2022, 7:01 a.m. UTC
Remove unused field "rpm_lvl_attr" and "spm_lvl_attr" in struct ufs_hba.
Commit cbb6813ee771 ("scsi: ufs: sysfs: attribute group for existing
sysfs entries.") removed all code using that field.

Signed-off-by: Keoseong Park <keosung.park@samsung.com>
---
 drivers/scsi/ufs/ufshcd.h | 2 --
 1 file changed, 2 deletions(-)

Comments

Avri Altman March 24, 2022, 9:08 a.m. UTC | #1
> Remove unused field "rpm_lvl_attr" and "spm_lvl_attr" in struct ufs_hba.
> Commit cbb6813ee771 ("scsi: ufs: sysfs: attribute group for existing sysfs
> entries.") removed all code using that field.
> 
> Signed-off-by: Keoseong Park <keosung.park@samsung.com>
Acked-by: Avri Altman <avri.altman@wdc.com>

> ---
>  drivers/scsi/ufs/ufshcd.h | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h index
> 88c20f3608c2..94f545be183a 100644
> --- a/drivers/scsi/ufs/ufshcd.h
> +++ b/drivers/scsi/ufs/ufshcd.h
> @@ -820,8 +820,6 @@ struct ufs_hba {
>         enum ufs_pm_level rpm_lvl;
>         /* Desired UFS power management level during system PM */
>         enum ufs_pm_level spm_lvl;
> -       struct device_attribute rpm_lvl_attr;
> -       struct device_attribute spm_lvl_attr;
>         int pm_op_in_progress;
> 
>         /* Auto-Hibernate Idle Timer register value */
> --
> 2.17.1
Bart Van Assche March 24, 2022, 12:44 p.m. UTC | #2
On 3/24/22 00:01, Keoseong Park wrote:
> Remove unused field "rpm_lvl_attr" and "spm_lvl_attr" in struct ufs_hba.
> Commit cbb6813ee771 ("scsi: ufs: sysfs: attribute group for existing
> sysfs entries.") removed all code using that field.
> 
> Signed-off-by: Keoseong Park <keosung.park@samsung.com>
> ---
>   drivers/scsi/ufs/ufshcd.h | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h
> index 88c20f3608c2..94f545be183a 100644
> --- a/drivers/scsi/ufs/ufshcd.h
> +++ b/drivers/scsi/ufs/ufshcd.h
> @@ -820,8 +820,6 @@ struct ufs_hba {
>   	enum ufs_pm_level rpm_lvl;
>   	/* Desired UFS power management level during system PM */
>   	enum ufs_pm_level spm_lvl;
> -	struct device_attribute rpm_lvl_attr;
> -	struct device_attribute spm_lvl_attr;
>   	int pm_op_in_progress;
>   
>   	/* Auto-Hibernate Idle Timer register value */

Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Martin K. Petersen March 30, 2022, 3:28 a.m. UTC | #3
Keoseong,

> Remove unused field "rpm_lvl_attr" and "spm_lvl_attr" in struct
> ufs_hba.  Commit cbb6813ee771 ("scsi: ufs: sysfs: attribute group for
> existing sysfs entries.") removed all code using that field.

Applied to 5.18/scsi-staging, thanks!
Martin K. Petersen April 7, 2022, 1:35 p.m. UTC | #4
On Thu, 24 Mar 2022 16:01:46 +0900, Keoseong Park wrote:

> Remove unused field "rpm_lvl_attr" and "spm_lvl_attr" in struct ufs_hba.
> Commit cbb6813ee771 ("scsi: ufs: sysfs: attribute group for existing
> sysfs entries.") removed all code using that field.
> 
> 

Applied to 5.18/scsi-fixes, thanks!

[1/1] scsi: ufs: core: Remove unused field in struct ufs_hba
      https://git.kernel.org/mkp/scsi/c/8ee15ea779c3
diff mbox series

Patch

diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h
index 88c20f3608c2..94f545be183a 100644
--- a/drivers/scsi/ufs/ufshcd.h
+++ b/drivers/scsi/ufs/ufshcd.h
@@ -820,8 +820,6 @@  struct ufs_hba {
 	enum ufs_pm_level rpm_lvl;
 	/* Desired UFS power management level during system PM */
 	enum ufs_pm_level spm_lvl;
-	struct device_attribute rpm_lvl_attr;
-	struct device_attribute spm_lvl_attr;
 	int pm_op_in_progress;
 
 	/* Auto-Hibernate Idle Timer register value */