[PATCH] staging: qlge: qlge_main.c: fixed spaces coding style issues

2020-05-29 Thread Jil Rouceau
Fixed the missing spaces before and after binary operators.

Signed-off-by: Jil Rouceau 
---
 drivers/staging/qlge/qlge_main.c | 18 +-
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/qlge/qlge_main.c b/drivers/staging/qlge/qlge_main.c
index c92820f07968..36f8d2890f9a 100644
--- a/drivers/staging/qlge/qlge_main.c
+++ b/drivers/staging/qlge/qlge_main.c
@@ -702,7 +702,7 @@ static int ql_get_8000_flash_params(struct ql_adapter *qdev)
 
size = sizeof(struct flash_params_8000) / sizeof(u32);
for (i = 0; i < size; i++, p++) {
-   status = ql_read_flash_word(qdev, i+offset, p);
+   status = ql_read_flash_word(qdev, i + offset, p);
if (status) {
netif_err(qdev, ifup, qdev->ndev,
  "Error reading flash.\n");
@@ -765,7 +765,7 @@ static int ql_get_8012_flash_params(struct ql_adapter *qdev)
return -ETIMEDOUT;
 
for (i = 0; i < size; i++, p++) {
-   status = ql_read_flash_word(qdev, i+offset, p);
+   status = ql_read_flash_word(qdev, i + offset, p);
if (status) {
netif_err(qdev, ifup, qdev->ndev,
  "Error reading flash.\n");
@@ -1543,7 +1543,7 @@ static void ql_process_mac_rx_page(struct ql_adapter 
*qdev,
struct iphdr *iph =
(struct iphdr *)((u8 *)addr + hlen);
if (!(iph->frag_off &
-   htons(IP_MF|IP_OFFSET))) {
+   htons(IP_MF | IP_OFFSET))) {
skb->ip_summed = CHECKSUM_UNNECESSARY;
netif_printk(qdev, rx_status, KERN_DEBUG,
 qdev->ndev,
@@ -1650,7 +1650,7 @@ static void ql_process_mac_rx_skb(struct ql_adapter *qdev,
struct iphdr *iph = (struct iphdr *) skb->data;
 
if (!(iph->frag_off &
-   htons(IP_MF|IP_OFFSET))) {
+   htons(IP_MF | IP_OFFSET))) {
skb->ip_summed = CHECKSUM_UNNECESSARY;
netif_printk(qdev, rx_status, KERN_DEBUG,
 qdev->ndev,
@@ -1939,7 +1939,7 @@ static void ql_process_mac_split_rx_intr(struct 
ql_adapter *qdev,
struct iphdr *iph = (struct iphdr *) skb->data;
 
if (!(iph->frag_off &
-   htons(IP_MF|IP_OFFSET))) {
+   htons(IP_MF | IP_OFFSET))) {
skb->ip_summed = CHECKSUM_UNNECESSARY;
netif_printk(qdev, rx_status, KERN_DEBUG, 
qdev->ndev,
 "TCP checksum done!\n");
@@ -4563,7 +4563,7 @@ static void ql_timer(struct timer_list *t)
return;
}
 
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
 }
 
 static int qlge_probe(struct pci_dev *pdev,
@@ -4635,7 +4635,7 @@ static int qlge_probe(struct pci_dev *pdev,
 * the bus goes dead
 */
timer_setup(&qdev->timer, ql_timer, TIMER_DEFERRABLE);
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
ql_link_off(qdev);
ql_display_dev_info(ndev);
atomic_set(&qdev->lb_count, 0);
@@ -4769,7 +4769,7 @@ static void qlge_io_resume(struct pci_dev *pdev)
netif_err(qdev, ifup, qdev->ndev,
  "Device was not running prior to EEH.\n");
}
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
netif_device_attach(ndev);
 }
 
@@ -4831,7 +4831,7 @@ static int qlge_resume(struct pci_dev *pdev)
return err;
}
 
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
netif_device_attach(ndev);
 
return 0;
-- 
2.25.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] staging: qlge: qlge_main.c: fixed spaces coding style issues

2020-06-01 Thread Jil Rouceau
On Mon, 1 Jun 2020 14:19:47 +0900
Benjamin Poirier  wrote:

> On 2020-05-29 17:17 +0200, Jil Rouceau wrote:
> > Fixed the missing spaces before and after binary operators.
> > 
> > Signed-off-by: Jil Rouceau   
> 
> This patch does not apply cleanly. I think your base tree is missing
> commit ec269f1250c6 ("staging: qlge: Remove unnecessary spaces in
> qlge_main.c").

Indeed, I was using Linus' tree instead of linux-next, thank you.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH v2] staging: qlge: qlge_main.c: fixed spaces coding style issues

2020-06-01 Thread Jil Rouceau
Fixed the missing spaces before and after binary operators.

Signed-off-by: Jil Rouceau 
---
Changes in v2:
- Based tree changed from Linus' to linux-next.

 drivers/staging/qlge/qlge_main.c | 18 +-
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/qlge/qlge_main.c b/drivers/staging/qlge/qlge_main.c
index 402edaeffe12..1650de13842f 100644
--- a/drivers/staging/qlge/qlge_main.c
+++ b/drivers/staging/qlge/qlge_main.c
@@ -687,7 +687,7 @@ static int ql_get_8000_flash_params(struct ql_adapter *qdev)
 
size = sizeof(struct flash_params_8000) / sizeof(u32);
for (i = 0; i < size; i++, p++) {
-   status = ql_read_flash_word(qdev, i+offset, p);
+   status = ql_read_flash_word(qdev, i + offset, p);
if (status) {
netif_err(qdev, ifup, qdev->ndev,
  "Error reading flash.\n");
@@ -750,7 +750,7 @@ static int ql_get_8012_flash_params(struct ql_adapter *qdev)
return -ETIMEDOUT;
 
for (i = 0; i < size; i++, p++) {
-   status = ql_read_flash_word(qdev, i+offset, p);
+   status = ql_read_flash_word(qdev, i + offset, p);
if (status) {
netif_err(qdev, ifup, qdev->ndev,
  "Error reading flash.\n");
@@ -1528,7 +1528,7 @@ static void ql_process_mac_rx_page(struct ql_adapter 
*qdev,
struct iphdr *iph =
(struct iphdr *)((u8 *)addr + hlen);
if (!(iph->frag_off &
-   htons(IP_MF|IP_OFFSET))) {
+   htons(IP_MF | IP_OFFSET))) {
skb->ip_summed = CHECKSUM_UNNECESSARY;
netif_printk(qdev, rx_status, KERN_DEBUG,
 qdev->ndev,
@@ -1635,7 +1635,7 @@ static void ql_process_mac_rx_skb(struct ql_adapter *qdev,
struct iphdr *iph = (struct iphdr *)skb->data;
 
if (!(iph->frag_off &
-   htons(IP_MF|IP_OFFSET))) {
+   htons(IP_MF | IP_OFFSET))) {
skb->ip_summed = CHECKSUM_UNNECESSARY;
netif_printk(qdev, rx_status, KERN_DEBUG,
 qdev->ndev,
@@ -1924,7 +1924,7 @@ static void ql_process_mac_split_rx_intr(struct 
ql_adapter *qdev,
struct iphdr *iph = (struct iphdr *)skb->data;
 
if (!(iph->frag_off &
-   htons(IP_MF|IP_OFFSET))) {
+   htons(IP_MF | IP_OFFSET))) {
skb->ip_summed = CHECKSUM_UNNECESSARY;
netif_printk(qdev, rx_status, KERN_DEBUG, 
qdev->ndev,
 "TCP checksum done!\n");
@@ -4547,7 +4547,7 @@ static void ql_timer(struct timer_list *t)
return;
}
 
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
 }
 
 static int qlge_probe(struct pci_dev *pdev,
@@ -4619,7 +4619,7 @@ static int qlge_probe(struct pci_dev *pdev,
 * the bus goes dead
 */
timer_setup(&qdev->timer, ql_timer, TIMER_DEFERRABLE);
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
ql_link_off(qdev);
ql_display_dev_info(ndev);
atomic_set(&qdev->lb_count, 0);
@@ -4753,7 +4753,7 @@ static void qlge_io_resume(struct pci_dev *pdev)
netif_err(qdev, ifup, qdev->ndev,
  "Device was not running prior to EEH.\n");
}
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
netif_device_attach(ndev);
 }
 
@@ -4815,7 +4815,7 @@ static int qlge_resume(struct pci_dev *pdev)
return err;
}
 
-   mod_timer(&qdev->timer, jiffies + (5*HZ));
+   mod_timer(&qdev->timer, jiffies + (5 * HZ));
netif_device_attach(ndev);
 
return 0;
-- 
2.25.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel