Skip to contentSkip to conversation

fusion: fix excess parameter kernel-doc warning

PATCH In review Randy Dunlap 3 participants 5 messages Aug 16, 2014 – Aug 28, 2014

Subsystems (2)
SCSI LSI Logic MPT fusion drivers (FC/SAS/SPI)
lore
Share & tools
lore

Randy Dunlap

author
#0 lore raw

PATCH 1/1 fusion: fix excess parameter kernel-doc warning

From: Randy Dunlap <[email protected]>

Fix kernel-doc excess parameter warning:

Warning(..//drivers/message/fusion/mptbase.c:1411): Excess function parameter 'prod_name' description in 'mpt_get_product_name'

Signed-off-by Randy Dunlap <[email protected]>
Cc [email protected]
Cc [email protected]
---
 drivers/message/fusion/mptbase.c |    1 -
 1 file changed, 1 deletion(-)
drivers/message/fusion/mptbase.c +0 -1
Index: lnx-317-rc1/drivers/message/fusion/mptbase.c
===================================================================
@@ -1400,7 +1400,6 @@ mpt_verify_adapter(int iocid, MPT_ADAPTE
  *	@vendor: pci vendor id
  *	@device: pci device id
  *	@revision: pci revision id
- *	@prod_name: string returned
  *
  *	Returns product string displayed when driver loads,
  *	in /proc/mpt/summary and /sysfs/class/scsi_host/host<X>/version_product
Reviewed-by on v1 1/1
#1 lore raw
Quoted 18 lines · Randy Dunlap

From: Randy Dunlap <[email protected]>

Fix kernel-doc excess parameter warning:

Warning(..//drivers/message/fusion/mptbase.c:1411): Excess function parameter 'prod_name' description in 'mpt_get_product_name'

Signed-off-by Randy Dunlap <[email protected]>
Cc [email protected]
Cc [email protected]
---
 drivers/message/fusion/mptbase.c |    1 -
 1 file changed, 1 deletion(-)

Index: lnx-317-rc1/drivers/message/fusion/mptbase.c
===================================================================

drivers/message/fusion/mptbase.c mpt_verify_adapter(int iocid, MPT_ADAPTE

Quoted 8 lines
drivers/message/fusion/mptbase.c +0 -2
@@ -1400,7 +1400,6 @@ mpt_verify_adapter(int iocid, MPT_ADAPTE
  *	@vendor: pci vendor id
  *	@device: pci device id
  *	@revision: pci revision id
- *	@prod_name: string returned
  *
  *	Returns product string displayed when driver loads,
  *	in /proc/mpt/summary and /sysfs/class/scsi_host/host<X>/version_product
--

Goes with commit c9834c70efbaaa1461ec04289d97a842244fb294.

Reviewed-by Ewan D. Milne <[email protected]>
author
#2 lore raw
Quoted 30 lines · Ewan Milne
Quoted 18 lines · Randy Dunlap

From: Randy Dunlap <[email protected]>

Fix kernel-doc excess parameter warning:

Warning(..//drivers/message/fusion/mptbase.c:1411): Excess function parameter 'prod_name' description in 'mpt_get_product_name'

Signed-off-by Randy Dunlap <[email protected]>
Cc [email protected]
Cc [email protected]
---
 drivers/message/fusion/mptbase.c |    1 -
 1 file changed, 1 deletion(-)

Index: lnx-317-rc1/drivers/message/fusion/mptbase.c
===================================================================

drivers/message/fusion/mptbase.c mpt_verify_adapter(int iocid, MPT_ADAPTE

Quoted 8 lines
drivers/message/fusion/mptbase.c +0 -2
@@ -1400,7 +1400,6 @@ mpt_verify_adapter(int iocid, MPT_ADAPTE
  *	@vendor: pci vendor id
  *	@device: pci device id
  *	@revision: pci revision id
- *	@prod_name: string returned
  *
  *	Returns product string displayed when driver loads,
  *	in /proc/mpt/summary and /sysfs/class/scsi_host/host<X>/version_product
--

Goes with commit c9834c70efbaaa1461ec04289d97a842244fb294.

Quoted 1 line
Reviewed-by Ewan D. Milne <[email protected]>

Christoph, did you pick up this patch or should I merge it with my documentation patches?

Thanks.

Signature · 1 line
~Randy
#3 lore raw
Quoted 6 lines · Randy Dunlap
Quoted 3 lines

Goes with commit c9834c70efbaaa1461ec04289d97a842244fb294.

Reviewed-by Ewan D. Milne <[email protected]>

Christoph, did you pick up this patch or should I merge it with my documentation patches?

I did as mentioned in my reply to your other patch.

author
#4 lore raw
Quoted 9 lines · Christoph Hellwig
Quoted 6 lines · Randy Dunlap
Quoted 3 lines

Goes with commit c9834c70efbaaa1461ec04289d97a842244fb294.

Reviewed-by Ewan D. Milne <[email protected]>

Christoph, did you pick up this patch or should I merge it with my documentation patches?

I did as mentioned in my reply to your other patch.

Thanks. Sorry I missed that...

Signature · 1 line
~Randy