Commit 6b71f60c authored by Lee Jones's avatar Lee Jones Committed by Martin K. Petersen

scsi: atp870u: Fix naming and demote incorrect and non-conformant kernel-doc header

Fixes the following W=1 kernel build warning(s):

 drivers/scsi/atp870u.c:623: warning: expecting prototype for atp870u_queuecommand(). Prototype was for atp870u_queuecommand_lck() instead
 drivers/scsi/atp870u.c:724: warning: Function parameter or member 'dev' not described in 'send_s870'
 drivers/scsi/atp870u.c:724: warning: Function parameter or member 'c' not described in 'send_s870'
 drivers/scsi/atp870u.c:724: warning: Excess function parameter 'host' description in 'send_s870'

Link: https://lore.kernel.org/r/20210312094738.2207817-27-lee.jones@linaro.org
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com>
Cc: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: Marcelo Tosatti <marcelo@conectiva.com.br>
Cc: linux-scsi@vger.kernel.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 167b7e6b
...@@ -612,7 +612,7 @@ static irqreturn_t atp870u_intr_handle(int irq, void *dev_id) ...@@ -612,7 +612,7 @@ static irqreturn_t atp870u_intr_handle(int irq, void *dev_id)
return IRQ_HANDLED; return IRQ_HANDLED;
} }
/** /**
* atp870u_queuecommand - Queue SCSI command * atp870u_queuecommand_lck - Queue SCSI command
* @req_p: request block * @req_p: request block
* @done: completion function * @done: completion function
* *
...@@ -711,16 +711,15 @@ static int atp870u_queuecommand_lck(struct scsi_cmnd *req_p, ...@@ -711,16 +711,15 @@ static int atp870u_queuecommand_lck(struct scsi_cmnd *req_p,
static DEF_SCSI_QCMD(atp870u_queuecommand) static DEF_SCSI_QCMD(atp870u_queuecommand)
/** /*
* send_s870 - send a command to the controller * send_s870 - send a command to the controller
* @host: host
* *
* On entry there is work queued to be done. We move some of that work to the * On entry there is work queued to be done. We move some of that work to the
* controller itself. * controller itself.
* *
* Caller holds the host lock. * Caller holds the host lock.
*/ */
static void send_s870(struct atp_unit *dev,unsigned char c) static void send_s870(struct atp_unit *dev, unsigned char c)
{ {
struct scsi_cmnd *workreq = NULL; struct scsi_cmnd *workreq = NULL;
unsigned int i;//,k; unsigned int i;//,k;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment