linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/10] Fix compilation warnings
@ 2020-07-06 12:33 Damien Le Moal
  2020-07-08  6:11 ` [PATCH] " Martin K. Petersen
  0 siblings, 1 reply; 5+ messages in thread
From: Damien Le Moal @ 2020-07-06 12:33 UTC (permalink / raw)
  To: linux-scsi, Martin K . Petersen

These patches fix various compilation warnings showing with W=1 for
the megaraid and mpt3sas drivers as well as sd and sd_zbc drivers.
The warnings are mostly due to incorrectly formatted kdoc comments.
Some "set but unused variable" warnings are also fixed.

Overall, there is no functional change introduced by these patches.

Damien Le Moal (10):
  scsi: megaraid: Fix kdoc comments format
  scsi: megaraid: Fix compilation warnings
  scsi: megaraid: Remove set but unused variable
  scsi: megaraid: Remove set but unused variable
  scsi: megaraid: Fix set but unused variable
  scsi: megaraid: Remove set but unused variable
  scsi: sd: Fix kdoc comment format
  scsi: sd_zbc: Fix kdoc comment format
  scsi: mpt3sas: Fix set but unused variable
  scsi: mpt3sas: Fix kdoc comments format

 drivers/scsi/megaraid.c                     | 222 ++++++++++----------
 drivers/scsi/megaraid/megaraid_mbox.c       |   4 +-
 drivers/scsi/megaraid/megaraid_mm.c         |   1 -
 drivers/scsi/megaraid/megaraid_sas.h        |  20 ++
 drivers/scsi/megaraid/megaraid_sas_base.c   | 175 ++++++++-------
 drivers/scsi/megaraid/megaraid_sas_fp.c     |  11 +-
 drivers/scsi/megaraid/megaraid_sas_fusion.c |  93 ++++----
 drivers/scsi/mpt3sas/mpt3sas_base.c         |  14 +-
 drivers/scsi/mpt3sas/mpt3sas_config.c       |   7 +-
 drivers/scsi/mpt3sas/mpt3sas_ctl.c          |  16 +-
 drivers/scsi/sd.c                           |   2 +-
 drivers/scsi/sd_zbc.c                       |   2 +-
 12 files changed, 304 insertions(+), 263 deletions(-)

-- 
2.26.2


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

* Re: [PATCH] Fix compilation warnings
  2020-07-06 12:33 [PATCH 00/10] Fix compilation warnings Damien Le Moal
@ 2020-07-08  6:11 ` Martin K. Petersen
  2020-07-08  6:24   ` Damien Le Moal
  0 siblings, 1 reply; 5+ messages in thread
From: Martin K. Petersen @ 2020-07-08  6:11 UTC (permalink / raw)
  To: Damien Le Moal, linux-scsi; +Cc: Martin K . Petersen, Johannes Thumshirn

On Mon, 6 Jul 2020 09:04:50 +0900, Damien Le Moal wrote:

> Commit c7e4dd5d84fc ("scsi: mpt3sas: Fix error returns in
> BRM_status_show") introduced a compilation warning:
> [...]

Not sure what was up with this series. Neither b4, nor lore could
figure it out. Had to do each patch individually. In any case: Applied
to 5.9/scsi-queue, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

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

* Re: [PATCH] Fix compilation warnings
  2020-07-08  6:11 ` [PATCH] " Martin K. Petersen
@ 2020-07-08  6:24   ` Damien Le Moal
  2020-07-08  6:30     ` Martin K. Petersen
  0 siblings, 1 reply; 5+ messages in thread
From: Damien Le Moal @ 2020-07-08  6:24 UTC (permalink / raw)
  To: Martin K. Petersen, linux-scsi; +Cc: Johannes Thumshirn

On 2020/07/08 15:11, Martin K. Petersen wrote:
> On Mon, 6 Jul 2020 09:04:50 +0900, Damien Le Moal wrote:
> 
>> Commit c7e4dd5d84fc ("scsi: mpt3sas: Fix error returns in
>> BRM_status_show") introduced a compilation warning:
>> [...]
> 
> Not sure what was up with this series. Neither b4, nor lore could
> figure it out. Had to do each patch individually. In any case: Applied
> to 5.9/scsi-queue, thanks!

Weird... It was all based on 5.9/scsi-queue and generated with normal git
format-patch + git send-email.

By the way, the patch mentioned above is not part of that series and should go
in 5.8/scsi-fixes, no ?


-- 
Damien Le Moal
Western Digital Research

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

* Re: [PATCH] Fix compilation warnings
  2020-07-08  6:24   ` Damien Le Moal
@ 2020-07-08  6:30     ` Martin K. Petersen
  2020-07-08  6:36       ` Damien Le Moal
  0 siblings, 1 reply; 5+ messages in thread
From: Martin K. Petersen @ 2020-07-08  6:30 UTC (permalink / raw)
  To: Damien Le Moal; +Cc: Martin K. Petersen, linux-scsi, Johannes Thumshirn


Damien,

> By the way, the patch mentioned above is not part of that series and
> should go in 5.8/scsi-fixes, no ?

Oh, sorry. I replied to the wrong ty email.

I amended the offending mpt3sas commit in 5.8/scsi-fixes.

-- 
Martin K. Petersen	Oracle Linux Engineering

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

* Re: [PATCH] Fix compilation warnings
  2020-07-08  6:30     ` Martin K. Petersen
@ 2020-07-08  6:36       ` Damien Le Moal
  0 siblings, 0 replies; 5+ messages in thread
From: Damien Le Moal @ 2020-07-08  6:36 UTC (permalink / raw)
  To: Martin K. Petersen; +Cc: linux-scsi, Johannes Thumshirn

On 2020/07/08 15:33, Martin K. Petersen wrote:
> 
> Damien,
> 
>> By the way, the patch mentioned above is not part of that series and
>> should go in 5.8/scsi-fixes, no ?
> 
> Oh, sorry. I replied to the wrong ty email.

I thought so :)

> 
> I amended the offending mpt3sas commit in 5.8/scsi-fixes.

Thanks !

> 


-- 
Damien Le Moal
Western Digital Research

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

end of thread, other threads:[~2020-07-08  6:36 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-06 12:33 [PATCH 00/10] Fix compilation warnings Damien Le Moal
2020-07-08  6:11 ` [PATCH] " Martin K. Petersen
2020-07-08  6:24   ` Damien Le Moal
2020-07-08  6:30     ` Martin K. Petersen
2020-07-08  6:36       ` Damien Le Moal

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).