public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 5/5] mptscsih: Fix first line of kernel-doc for a few functions
       [not found] <1266157055.10567.13.camel@localhost>
@ 2010-02-14 14:21 ` Ben Hutchings
  2010-03-24  4:27 ` [PATCH RESEND] " Ben Hutchings
  1 sibling, 0 replies; 3+ messages in thread
From: Ben Hutchings @ 2010-02-14 14:21 UTC (permalink / raw)
  To: Eric Moore; +Cc: linux-scsi, LKML

The function name must be followed by a space, hypen, space, and a
short description.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
---
 drivers/message/fusion/mptscsih.c |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/message/fusion/mptscsih.c b/drivers/message/fusion/mptscsih.c
index 4a7d1af..53516ed 100644
--- a/drivers/message/fusion/mptscsih.c
+++ b/drivers/message/fusion/mptscsih.c
@@ -2560,9 +2560,7 @@ mptscsih_getclear_scsi_lookup(MPT_ADAPTER *ioc, int i)
 }
 
 /**
- * mptscsih_set_scsi_lookup
- *
- * writes a scmd entry into the ScsiLookup[] array list
+ * mptscsih_set_scsi_lookup - write a scmd entry into the ScsiLookup[] array list
  *
  * @ioc: Pointer to MPT_ADAPTER structure
  * @i: index into the array
@@ -2725,7 +2723,7 @@ mptscsih_scandv_complete(MPT_ADAPTER *ioc, MPT_FRAME_HDR *req,
 

 /**
- *	mptscsih_get_completion_code -
+ *	mptscsih_get_completion_code - get completion code from MPT request
  *	@ioc: Pointer to MPT_ADAPTER structure
  *	@req: Pointer to original MPT request frame
  *	@reply: Pointer to MPT reply frame (NULL if TurboReply)
-- 
1.6.6.2

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

* [PATCH RESEND] mptscsih: Fix first line of kernel-doc for a few functions
       [not found] <1266157055.10567.13.camel@localhost>
  2010-02-14 14:21 ` [PATCH 5/5] mptscsih: Fix first line of kernel-doc for a few functions Ben Hutchings
@ 2010-03-24  4:27 ` Ben Hutchings
  2010-03-31 16:13   ` Randy Dunlap
  1 sibling, 1 reply; 3+ messages in thread
From: Ben Hutchings @ 2010-03-24  4:27 UTC (permalink / raw)
  To: James Bottomley; +Cc: Eric Moore, linux-scsi

The function name must be followed by a space, hypen, space, and a
short description.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
---
 drivers/message/fusion/mptscsih.c |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/message/fusion/mptscsih.c b/drivers/message/fusion/mptscsih.c
index 4a7d1af..53516ed 100644
--- a/drivers/message/fusion/mptscsih.c
+++ b/drivers/message/fusion/mptscsih.c
@@ -2560,9 +2560,7 @@ mptscsih_getclear_scsi_lookup(MPT_ADAPTER *ioc, int i)
 }
 
 /**
- * mptscsih_set_scsi_lookup
- *
- * writes a scmd entry into the ScsiLookup[] array list
+ * mptscsih_set_scsi_lookup - write a scmd entry into the ScsiLookup[] array list
  *
  * @ioc: Pointer to MPT_ADAPTER structure
  * @i: index into the array
@@ -2725,7 +2723,7 @@ mptscsih_scandv_complete(MPT_ADAPTER *ioc, MPT_FRAME_HDR *req,
 

 /**
- *	mptscsih_get_completion_code -
+ *	mptscsih_get_completion_code - get completion code from MPT request
  *	@ioc: Pointer to MPT_ADAPTER structure
  *	@req: Pointer to original MPT request frame
  *	@reply: Pointer to MPT reply frame (NULL if TurboReply)
-- 
1.6.6.2



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

* Re: [PATCH RESEND] mptscsih: Fix first line of kernel-doc for a few functions
  2010-03-24  4:27 ` [PATCH RESEND] " Ben Hutchings
@ 2010-03-31 16:13   ` Randy Dunlap
  0 siblings, 0 replies; 3+ messages in thread
From: Randy Dunlap @ 2010-03-31 16:13 UTC (permalink / raw)
  To: Ben Hutchings; +Cc: James Bottomley, Eric Moore, linux-scsi

On Wed, 24 Mar 2010 04:27:59 +0000 Ben Hutchings wrote:

> The function name must be followed by a space, hypen, space, and a
> short description.
> 
> Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Acked-by: Randy Dunlap <randy.dunlap@oracle.com>

James, will you be handling this patch or should I?


> ---
>  drivers/message/fusion/mptscsih.c |    6 ++----
>  1 files changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/message/fusion/mptscsih.c b/drivers/message/fusion/mptscsih.c
> index 4a7d1af..53516ed 100644
> --- a/drivers/message/fusion/mptscsih.c
> +++ b/drivers/message/fusion/mptscsih.c
> @@ -2560,9 +2560,7 @@ mptscsih_getclear_scsi_lookup(MPT_ADAPTER *ioc, int i)
>  }
>  
>  /**
> - * mptscsih_set_scsi_lookup
> - *
> - * writes a scmd entry into the ScsiLookup[] array list
> + * mptscsih_set_scsi_lookup - write a scmd entry into the ScsiLookup[] array list
>   *
>   * @ioc: Pointer to MPT_ADAPTER structure
>   * @i: index into the array
> @@ -2725,7 +2723,7 @@ mptscsih_scandv_complete(MPT_ADAPTER *ioc, MPT_FRAME_HDR *req,
>  
> 
>  /**
> - *	mptscsih_get_completion_code -
> + *	mptscsih_get_completion_code - get completion code from MPT request
>   *	@ioc: Pointer to MPT_ADAPTER structure
>   *	@req: Pointer to original MPT request frame
>   *	@reply: Pointer to MPT reply frame (NULL if TurboReply)
> -- 

Thanks,
---
~Randy

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

end of thread, other threads:[~2010-03-31 16:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1266157055.10567.13.camel@localhost>
2010-02-14 14:21 ` [PATCH 5/5] mptscsih: Fix first line of kernel-doc for a few functions Ben Hutchings
2010-03-24  4:27 ` [PATCH RESEND] " Ben Hutchings
2010-03-31 16:13   ` Randy Dunlap

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox