linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* linux-next: Signed-off-by missing for commit in the scsi-fixes tree
@ 2018-11-06 19:48 Stephen Rothwell
       [not found] ` <(Stephen>
  0 siblings, 1 reply; 9+ messages in thread
From: Stephen Rothwell @ 2018-11-06 19:48 UTC (permalink / raw)
  To: Martin K. Petersen; +Cc: Linux-Next Mailing List, Linux Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 211 bytes --]

Hi Martin,

Commit

  85ee0a7b2d53 ("Revert "scsi: ufs: Disable blk-mq for now"")

is missing a Signed-off-by from its author and committer.

Reverts are commits, too.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: linux-next: Signed-off-by missing for commit in the scsi-fixes tree
       [not found]                   ` <+1100")>
@ 2018-11-07  1:52                     ` Martin K. Petersen
  0 siblings, 0 replies; 9+ messages in thread
From: Martin K. Petersen @ 2018-11-07  1:52 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Martin K. Petersen, Linux-Next Mailing List, Linux Kernel Mailing List


Stephen,

> Commit
>
>   85ee0a7b2d53 ("Revert "scsi: ufs: Disable blk-mq for now"")
>
> is missing a Signed-off-by from its author and committer.

Fixed up, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 0/2] two UFS changes
@ 2020-12-07 19:01 ` Bean Huo
       [not found]   ` <(Bean>
                     ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Bean Huo @ 2020-12-07 19:01 UTC (permalink / raw)
  To: alim.akhtar, avri.altman, asutoshd, jejb, martin.petersen,
	stanley.chu, beanhuo, bvanassche, tomas.winkler, cang
  Cc: linux-scsi, linux-kernel

From: Bean Huo <beanhuo@micron.com>



Bean Huo (2):
  scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE
  scsi: ufs: Fix wrong print message in dev_err()

 drivers/scsi/ufs/ufs.h    | 1 -
 drivers/scsi/ufs/ufshcd.c | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

-- 
2.17.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 1/2] scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE
  2020-12-07 19:01 ` [PATCH 0/2] two UFS changes Bean Huo
       [not found]   ` <(Bean>
@ 2020-12-07 19:01   ` Bean Huo
  2020-12-08  7:52     ` Avri Altman
  2020-12-07 19:01   ` [PATCH 2/2] scsi: ufs: Fix wrong print message in dev_err() Bean Huo
  2020-12-08  2:57   ` [PATCH 0/2] two UFS changes Alim Akhtar
  3 siblings, 1 reply; 9+ messages in thread
From: Bean Huo @ 2020-12-07 19:01 UTC (permalink / raw)
  To: alim.akhtar, avri.altman, asutoshd, jejb, martin.petersen,
	stanley.chu, beanhuo, bvanassche, tomas.winkler, cang
  Cc: linux-scsi, linux-kernel

From: Bean Huo <beanhuo@micron.com>

No user uses POWER_DESC_MAX_SIZE, remove it.

Signed-off-by: Bean Huo <beanhuo@micron.com>
---
 drivers/scsi/ufs/ufs.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/scsi/ufs/ufs.h b/drivers/scsi/ufs/ufs.h
index 311d5f7a024d..527ba5c00097 100644
--- a/drivers/scsi/ufs/ufs.h
+++ b/drivers/scsi/ufs/ufs.h
@@ -330,7 +330,6 @@ enum {
 	UFS_DEV_WRITE_BOOSTER_SUP	= BIT(8),
 };
 
-#define POWER_DESC_MAX_SIZE			0x62
 #define POWER_DESC_MAX_ACTV_ICC_LVLS		16
 
 /* Attribute  bActiveICCLevel parameter bit masks definitions */
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 2/2] scsi: ufs: Fix wrong print message in dev_err()
  2020-12-07 19:01 ` [PATCH 0/2] two UFS changes Bean Huo
       [not found]   ` <(Bean>
  2020-12-07 19:01   ` [PATCH 1/2] scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE Bean Huo
@ 2020-12-07 19:01   ` Bean Huo
  2020-12-08  7:53     ` Avri Altman
  2020-12-08  2:57   ` [PATCH 0/2] two UFS changes Alim Akhtar
  3 siblings, 1 reply; 9+ messages in thread
From: Bean Huo @ 2020-12-07 19:01 UTC (permalink / raw)
  To: alim.akhtar, avri.altman, asutoshd, jejb, martin.petersen,
	stanley.chu, beanhuo, bvanassche, tomas.winkler, cang
  Cc: linux-scsi, linux-kernel

From: Bean Huo <beanhuo@micron.com>

Change dev_err() print message from "dme-reset" to "dme_enable" in function
ufshcd_dme_enable().

Signed-off-by: Bean Huo <beanhuo@micron.com>
---
 drivers/scsi/ufs/ufshcd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index 5b2219e44743..f8f5eddad506 100644
--- a/drivers/scsi/ufs/ufshcd.c
+++ b/drivers/scsi/ufs/ufshcd.c
@@ -3645,7 +3645,7 @@ static int ufshcd_dme_enable(struct ufs_hba *hba)
 	ret = ufshcd_send_uic_cmd(hba, &uic_cmd);
 	if (ret)
 		dev_err(hba->dev,
-			"dme-reset: error code %d\n", ret);
+			"dme-enable: error code %d\n", ret);
 
 	return ret;
 }
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* RE: [PATCH 0/2] two UFS changes
  2020-12-07 19:01 ` [PATCH 0/2] two UFS changes Bean Huo
                     ` (2 preceding siblings ...)
  2020-12-07 19:01   ` [PATCH 2/2] scsi: ufs: Fix wrong print message in dev_err() Bean Huo
@ 2020-12-08  2:57   ` Alim Akhtar
  3 siblings, 0 replies; 9+ messages in thread
From: Alim Akhtar @ 2020-12-08  2:57 UTC (permalink / raw)
  To: 'Bean Huo',
	avri.altman, asutoshd, jejb, martin.petersen, stanley.chu,
	beanhuo, bvanassche, tomas.winkler, cang
  Cc: linux-scsi, linux-kernel

Hi Bean,

> -----Original Message-----
> From: Bean Huo <huobean@gmail.com>
> Sent: 08 December 2020 00:32
> To: alim.akhtar@samsung.com; avri.altman@wdc.com;
> asutoshd@codeaurora.org; jejb@linux.ibm.com;
> martin.petersen@oracle.com; stanley.chu@mediatek.com;
> beanhuo@micron.com; bvanassche@acm.org; tomas.winkler@intel.com;
> cang@codeaurora.org
> Cc: linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: [PATCH 0/2] two UFS changes
> 
> From: Bean Huo <beanhuo@micron.com>
> 
> 
> 
> Bean Huo (2):
>   scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE
>   scsi: ufs: Fix wrong print message in dev_err()
> 
>  drivers/scsi/ufs/ufs.h    | 1 -
>  drivers/scsi/ufs/ufshcd.c | 2 +-
>  2 files changed, 1 insertion(+), 2 deletions(-)
> 
Thanks!
Acked-by: Alim Akhtar <alim.akhtar@samsung.com>

> --
> 2.17.1



^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH 1/2] scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE
  2020-12-07 19:01   ` [PATCH 1/2] scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE Bean Huo
@ 2020-12-08  7:52     ` Avri Altman
  0 siblings, 0 replies; 9+ messages in thread
From: Avri Altman @ 2020-12-08  7:52 UTC (permalink / raw)
  To: Bean Huo, alim.akhtar, asutoshd, jejb, martin.petersen,
	stanley.chu, beanhuo, bvanassche, tomas.winkler, cang
  Cc: linux-scsi, linux-kernel

 
> From: Bean Huo <beanhuo@micron.com>
> 
> No user uses POWER_DESC_MAX_SIZE, remove it.
> 
> Signed-off-by: Bean Huo <beanhuo@micron.com>
Acked-by: Avri Altman <avri.altman@wdc.com>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH 2/2] scsi: ufs: Fix wrong print message in dev_err()
  2020-12-07 19:01   ` [PATCH 2/2] scsi: ufs: Fix wrong print message in dev_err() Bean Huo
@ 2020-12-08  7:53     ` Avri Altman
  0 siblings, 0 replies; 9+ messages in thread
From: Avri Altman @ 2020-12-08  7:53 UTC (permalink / raw)
  To: Bean Huo, alim.akhtar, asutoshd, jejb, martin.petersen,
	stanley.chu, beanhuo, bvanassche, tomas.winkler, cang
  Cc: linux-scsi, linux-kernel

> From: Bean Huo <beanhuo@micron.com>
> 
> Change dev_err() print message from "dme-reset" to "dme_enable" in
> function
> ufshcd_dme_enable().
> 
> Signed-off-by: Bean Huo <beanhuo@micron.com>
Acked-by: Avri Altman <avri.altman@wdc.com?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 0/2] two UFS changes
       [not found]                   ` <+0100")>
@ 2020-12-09  2:17                     ` Martin K. Petersen
  0 siblings, 0 replies; 9+ messages in thread
From: Martin K. Petersen @ 2020-12-09  2:17 UTC (permalink / raw)
  To: Bean Huo
  Cc: alim.akhtar, avri.altman, asutoshd, jejb, martin.petersen,
	stanley.chu, beanhuo, bvanassche, tomas.winkler, cang,
	linux-scsi, linux-kernel


Bean,

> Bean Huo (2):
>   scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE
>   scsi: ufs: Fix wrong print message in dev_err()

Applied to 5.11/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-12-09  2:20 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20201207190149epcas5p2d877f4e3f6d31548d97f9b486d243a05@epcas5p2.samsung.com>
2020-12-07 19:01 ` [PATCH 0/2] two UFS changes Bean Huo
     [not found]   ` <(Bean>
2020-12-07 19:01   ` [PATCH 1/2] scsi: ufs: Remove an unused macro definition POWER_DESC_MAX_SIZE Bean Huo
2020-12-08  7:52     ` Avri Altman
2020-12-07 19:01   ` [PATCH 2/2] scsi: ufs: Fix wrong print message in dev_err() Bean Huo
2020-12-08  7:53     ` Avri Altman
2020-12-08  2:57   ` [PATCH 0/2] two UFS changes Alim Akhtar
2018-11-06 19:48 linux-next: Signed-off-by missing for commit in the scsi-fixes tree Stephen Rothwell
     [not found] ` <(Stephen>
     [not found]   ` <Rothwell's>
     [not found]     ` <message>
     [not found]       ` <of>
     [not found]         ` <"Wed>
     [not found]           ` <7>
     [not found]             ` <Nov>
     [not found]               ` <2018>
     [not found]                 ` <06:48:34>
     [not found]                   ` <+1100")>
2018-11-07  1:52                     ` Martin K. Petersen
     [not found]             ` <Dec>
     [not found]               ` <2020>
     [not found]                 ` <20:01:35>
     [not found]                   ` <+0100")>
2020-12-09  2:17                     ` [PATCH 0/2] two UFS changes Martin K. Petersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).