i2c: bcm-kona: Use complete() instead of complete_all()

Daniel Wagner · Wolfram Sang · commit 752c3899f97c · 2016-08-15T00:55:32.000+02:00
There is only one waiter for the completion, therefore there
is no need to use complete_all(). Let's make that clear by
using complete() instead of complete_all().

The usage pattern of the completion is:

bcm_kona_send_i2c_cmd()
  reinit_completion()
  ...
  bcm_kona_i2c_send_cmd_to_ctrl()
  ...
  wait_for_completion_timeout()

Signed-off-by: Daniel Wagner &lt;daniel.wagner@bmw-carit.de&gt;
Acked-by: Ray Jui &lt;ray.jui@broadcom.com&gt;
Reviewed-by: Tim Kryger &lt;tim.kryger@gmail.com&gt;
Signed-off-by: Wolfram Sang &lt;wsa@the-dreams.de&gt;
diff --git a/drivers/i2c/busses/i2c-bcm-kona.c b/drivers/i2c/busses/i2c-bcm-kona.c
@@ -229,7 +229,7 @@ static irqreturn_t bcm_kona_i2c_isr(int irq, void *devid)
 		       dev->base + TXFCR_OFFSET);
 
 	writel(status & ~ISR_RESERVED_MASK, dev->base + ISR_OFFSET);
-	complete_all(&dev->done);
+	complete(&dev->done);
 
 	return IRQ_HANDLED;
 }

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,7 @@ static irqreturn_t bcm_kona_i2c_isr(int irq, void *devid)`
`229`	`229`	`dev->base + TXFCR_OFFSET);`
`230`	`230`
`231`	`231`	`writel(status & ~ISR_RESERVED_MASK, dev->base + ISR_OFFSET);`
`232`		`- complete_all(&dev->done);`
	`232`	`+ complete(&dev->done);`
`233`	`233`
`234`	`234`	`return IRQ_HANDLED;`
`235`	`235`	`}`