The variable 'i' does not need to be passed, as we set it to 0 (zero) anyways when starting the iteration here. Suggested-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Signed-off-by: Matthias Beyer <mail@xxxxxxxxxxxxxxxx> --- drivers/staging/bcm/nvm.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/nvm.c b/drivers/staging/bcm/nvm.c index ccf8745..ce09473 100644 --- a/drivers/staging/bcm/nvm.c +++ b/drivers/staging/bcm/nvm.c @@ -1037,9 +1037,9 @@ static int bulk_read_complete_sector(struct bcm_mini_adapter *ad, UCHAR read_bk[], PCHAR tmpbuff, unsigned int offset, - unsigned int partoff, - unsigned int i) + unsigned int partoff) { + unsigned int i; int j; int bulk_read_stat; FP_FLASH_WRITE_STATUS writef = @@ -1213,8 +1213,7 @@ static int BeceemFlashBulkWrite(struct bcm_mini_adapter *Adapter, ucReadBk, pTempBuff, uiOffsetFromSectStart, - uiPartOffset, - uiIndex)) { + uiPartOffset)) { Status = STATUS_FAILURE; goto BeceemFlashBulkWrite_EXIT; } -- 2.0.2 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel