diff mbox series

[05/30] net: eepro100: Fix braces

Message ID 20200523163837.407592-5-marek.vasut+renesas@gmail.com
State New
Headers show
Series [01/30] net: eepro100: Remove EEPRO100_SROM_WRITE | expand

Commit Message

Marek Vasut May 23, 2020, 4:38 p.m. UTC
This is automated cleanup via checkpatch, no functional change.
./scripts/checkpatch.pl --show-types -f drivers/net/eepro100.c
./scripts/checkpatch.pl --types BRACES -f --fix --fix-inplace drivers/net/eepro100.c

Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
---
 drivers/net/eepro100.c | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

Comments

Ramon Fried May 23, 2020, 5:12 p.m. UTC | #1
On Sat, May 23, 2020 at 7:40 PM Marek Vasut <marek.vasut at gmail.com> wrote:
>
> This is automated cleanup via checkpatch, no functional change.
> ./scripts/checkpatch.pl --show-types -f drivers/net/eepro100.c
> ./scripts/checkpatch.pl --types BRACES -f --fix --fix-inplace drivers/net/eepro100.c
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> ---
>  drivers/net/eepro100.c | 11 ++++-------
>  1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/net/eepro100.c b/drivers/net/eepro100.c
> index 1b6d5375f8..6dccd59bda 100644
> --- a/drivers/net/eepro100.c
> +++ b/drivers/net/eepro100.c
> @@ -358,9 +358,8 @@ static int wait_for_eepro100(struct eth_device *dev)
>         int i;
>
>         for (i = 0; INW(dev, SCBCmd) & (CU_CMD_MASK | RU_CMD_MASK); i++) {
> -               if (i >= TOUT_LOOP) {
> +               if (i >= TOUT_LOOP)
>                         return 0;
> -               }
>         }
>
>         return 1;
> @@ -383,9 +382,9 @@ int eepro100_initialize(bd_t * bis)
>
>         while (1) {
>                 /* Find PCI device */
> -               if ((devno = pci_find_devices(supported, idx++)) < 0) {
> +               devno = pci_find_devices(supported, idx++);
> +               if (devno < 0)
>                         break;
> -               }
>
>                 pci_read_config_dword(devno, PCI_BASE_ADDRESS_0, &iobase);
>                 iobase &= ~0xf;
> @@ -639,9 +638,8 @@ static int eepro100_recv(struct eth_device *dev)
>         for (;;) {
>                 status = le16_to_cpu(rx_ring[rx_next].status);
>
> -               if (!(status & RFD_STATUS_C)) {
> +               if (!(status & RFD_STATUS_C))
>                         break;
> -               }
>
>                 /* Valid frame status. */
>                 if ((status & RFD_STATUS_OK)) {
> @@ -668,7 +666,6 @@ static int eepro100_recv(struct eth_device *dev)
>         }
>
>         if (stat & SCB_STATUS_RNR) {
> -
>                 printf("%s: Receiver is not ready, restart it !\n", dev->name);
>
>                 /* Reinitialize Rx ring. */
> --
> 2.25.1
>
Reviewed-By: Ramon Fried <rfried.dev at gmail.com
diff mbox series

Patch

diff --git a/drivers/net/eepro100.c b/drivers/net/eepro100.c
index 1b6d5375f8..6dccd59bda 100644
--- a/drivers/net/eepro100.c
+++ b/drivers/net/eepro100.c
@@ -358,9 +358,8 @@  static int wait_for_eepro100(struct eth_device *dev)
 	int i;
 
 	for (i = 0; INW(dev, SCBCmd) & (CU_CMD_MASK | RU_CMD_MASK); i++) {
-		if (i >= TOUT_LOOP) {
+		if (i >= TOUT_LOOP)
 			return 0;
-		}
 	}
 
 	return 1;
@@ -383,9 +382,9 @@  int eepro100_initialize(bd_t * bis)
 
 	while (1) {
 		/* Find PCI device */
-		if ((devno = pci_find_devices(supported, idx++)) < 0) {
+		devno = pci_find_devices(supported, idx++);
+		if (devno < 0)
 			break;
-		}
 
 		pci_read_config_dword(devno, PCI_BASE_ADDRESS_0, &iobase);
 		iobase &= ~0xf;
@@ -639,9 +638,8 @@  static int eepro100_recv(struct eth_device *dev)
 	for (;;) {
 		status = le16_to_cpu(rx_ring[rx_next].status);
 
-		if (!(status & RFD_STATUS_C)) {
+		if (!(status & RFD_STATUS_C))
 			break;
-		}
 
 		/* Valid frame status. */
 		if ((status & RFD_STATUS_OK)) {
@@ -668,7 +666,6 @@  static int eepro100_recv(struct eth_device *dev)
 	}
 
 	if (stat & SCB_STATUS_RNR) {
-
 		printf("%s: Receiver is not ready, restart it !\n", dev->name);
 
 		/* Reinitialize Rx ring. */