* [PATCH v2 0/4] staging: vt6655: Fix checkpatch issues
@ 2016-09-18 9:04 Namrata A Shettar
2016-09-18 9:04 ` [PATCH v2 1/4] staging: vt6655: device_main: Replace NULL comparison with !x Namrata A Shettar
` (3 more replies)
0 siblings, 4 replies; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 9:04 UTC (permalink / raw)
To: Forest Bond, Greg Kroah-Hartman, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
Fix multiple checkpatch issues in vt6655 driver files.
Changes in v2:
- Subject line and commit log messages
Namrata A Shettar (4):
staging: vt6655: device_main: Replace NULL comparison with !x
staging: vt6655: device_main: Replace explicit NULL comparison
staging: vt6655: card: Break multiple assignments
staging: vt6655: device_main: Break up multiple assignments
drivers/staging/vt6655/card.c | 6 ++++--
drivers/staging/vt6655/device_main.c | 7 ++++---
2 files changed, 8 insertions(+), 5 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v2 1/4] staging: vt6655: device_main: Replace NULL comparison with !x
2016-09-18 9:04 [PATCH v2 0/4] staging: vt6655: Fix checkpatch issues Namrata A Shettar
@ 2016-09-18 9:04 ` Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 2/4] staging: vt6655: device_main: Replace explicit NULL comparison Namrata A Shettar
` (2 subsequent siblings)
3 siblings, 0 replies; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 9:04 UTC (permalink / raw)
To: Forest Bond, Greg Kroah-Hartman, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
Replace NULL comparison to resolve checkpatch issue.
- x == NULL => !x
Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
---
drivers/staging/vt6655/device_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
index ed12b5c..62418c2 100644
--- a/drivers/staging/vt6655/device_main.c
+++ b/drivers/staging/vt6655/device_main.c
@@ -477,7 +477,7 @@ static bool device_init_rings(struct vnt_private *priv)
CB_MAX_BUF_SIZE,
&priv->tx_bufs_dma0,
GFP_ATOMIC);
- if (priv->tx0_bufs == NULL) {
+ if (!priv->tx0_bufs) {
dev_err(&priv->pcid->dev, "allocate buf dma memory failed\n");
dma_free_coherent(&priv->pcid->dev,
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v2 2/4] staging: vt6655: device_main: Replace explicit NULL comparison
2016-09-18 9:04 [PATCH v2 0/4] staging: vt6655: Fix checkpatch issues Namrata A Shettar
2016-09-18 9:04 ` [PATCH v2 1/4] staging: vt6655: device_main: Replace NULL comparison with !x Namrata A Shettar
@ 2016-09-18 9:05 ` Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 4/4] staging: vt6655: device_main: Break up " Namrata A Shettar
3 siblings, 0 replies; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 9:05 UTC (permalink / raw)
To: Forest Bond, Greg Kroah-Hartman, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
Replace NULL comparison to resolve checkpatch issue.
- x == NULL => !x
Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
---
drivers/staging/vt6655/device_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
index 62418c2..f109eea 100644
--- a/drivers/staging/vt6655/device_main.c
+++ b/drivers/staging/vt6655/device_main.c
@@ -735,7 +735,7 @@ static bool device_alloc_rx_buf(struct vnt_private *priv,
struct vnt_rd_info *rd_info = rd->rd_info;
rd_info->skb = dev_alloc_skb((int)priv->rx_buf_sz);
- if (rd_info->skb == NULL)
+ if (!rd_info->skb)
return false;
rd_info->skb_dma =
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments
2016-09-18 9:04 [PATCH v2 0/4] staging: vt6655: Fix checkpatch issues Namrata A Shettar
2016-09-18 9:04 ` [PATCH v2 1/4] staging: vt6655: device_main: Replace NULL comparison with !x Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 2/4] staging: vt6655: device_main: Replace explicit NULL comparison Namrata A Shettar
@ 2016-09-18 9:05 ` Namrata A Shettar
2016-09-18 10:40 ` Greg Kroah-Hartman
2016-09-18 9:05 ` [PATCH v2 4/4] staging: vt6655: device_main: Break up " Namrata A Shettar
3 siblings, 1 reply; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 9:05 UTC (permalink / raw)
To: Forest Bond, Greg Kroah-Hartman, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
Break multiple assignments on same line to resolve checkpatch issue.
Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
---
drivers/staging/vt6655/card.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/vt6655/card.c b/drivers/staging/vt6655/card.c
index afb1e8b..00e7858 100644
--- a/drivers/staging/vt6655/card.c
+++ b/drivers/staging/vt6655/card.c
@@ -528,8 +528,10 @@ CARDvSafeResetTx(
struct vnt_tx_desc *pCurrTD;
/* initialize TD index */
- priv->apTailTD[0] = priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
- priv->apTailTD[1] = priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
+ priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
+ priv->apTailTD[0] = priv->apCurrTD[0];
+ priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
+ priv->apTailTD[1] = priv->apCurrTD[1];
for (uu = 0; uu < TYPE_MAXTD; uu++)
priv->iTDUsed[uu] = 0;
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v2 4/4] staging: vt6655: device_main: Break up multiple assignments
2016-09-18 9:04 [PATCH v2 0/4] staging: vt6655: Fix checkpatch issues Namrata A Shettar
` (2 preceding siblings ...)
2016-09-18 9:05 ` [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments Namrata A Shettar
@ 2016-09-18 9:05 ` Namrata A Shettar
2016-09-18 10:41 ` Greg Kroah-Hartman
3 siblings, 1 reply; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 9:05 UTC (permalink / raw)
To: Forest Bond, Greg Kroah-Hartman, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
Break up multiple assignments on the same line to resolve checkpatch
issue.
Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
---
drivers/staging/vt6655/device_main.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
index f109eea..86532ca 100644
--- a/drivers/staging/vt6655/device_main.c
+++ b/drivers/staging/vt6655/device_main.c
@@ -669,7 +669,8 @@ static void device_init_td1_ring(struct vnt_private *priv)
if (i > 0)
priv->apTD1Rings[i-1].next_desc = cpu_to_le32(priv->td1_pool_dma);
- priv->apTailTD[1] = priv->apCurrTD[1] = &priv->apTD1Rings[0];
+ priv->apCurrTD[1] = &priv->apTD1Rings[0];
+ priv->apTailTD[1] = priv->apCurrTD[1];
}
static void device_free_td0_ring(struct vnt_private *priv)
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments
2016-09-18 9:05 ` [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments Namrata A Shettar
@ 2016-09-18 10:40 ` Greg Kroah-Hartman
2016-09-18 10:54 ` Namrata A Shettar
0 siblings, 1 reply; 11+ messages in thread
From: Greg Kroah-Hartman @ 2016-09-18 10:40 UTC (permalink / raw)
To: Namrata A Shettar
Cc: Forest Bond, Malcolm Priestley, Anson Jacob, Haneen Mohammed,
outreachy-kernel
On Sun, Sep 18, 2016 at 02:35:39PM +0530, Namrata A Shettar wrote:
> Break multiple assignments on same line to resolve checkpatch issue.
>
> Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> ---
> drivers/staging/vt6655/card.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/vt6655/card.c b/drivers/staging/vt6655/card.c
> index afb1e8b..00e7858 100644
> --- a/drivers/staging/vt6655/card.c
> +++ b/drivers/staging/vt6655/card.c
> @@ -528,8 +528,10 @@ CARDvSafeResetTx(
> struct vnt_tx_desc *pCurrTD;
>
> /* initialize TD index */
> - priv->apTailTD[0] = priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> - priv->apTailTD[1] = priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> + priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> + priv->apTailTD[0] = priv->apCurrTD[0];
> + priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> + priv->apTailTD[1] = priv->apCurrTD[1];
Does that look correct to you?
(hint, odd indentation)
And really, the original code is fine, I would recommend just leaving
this alone, it makes more sense as-is, don't you think?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 4/4] staging: vt6655: device_main: Break up multiple assignments
2016-09-18 9:05 ` [PATCH v2 4/4] staging: vt6655: device_main: Break up " Namrata A Shettar
@ 2016-09-18 10:41 ` Greg Kroah-Hartman
2016-09-18 10:56 ` Namrata A Shettar
0 siblings, 1 reply; 11+ messages in thread
From: Greg Kroah-Hartman @ 2016-09-18 10:41 UTC (permalink / raw)
To: Namrata A Shettar
Cc: Forest Bond, Malcolm Priestley, Anson Jacob, Haneen Mohammed,
outreachy-kernel
On Sun, Sep 18, 2016 at 02:35:53PM +0530, Namrata A Shettar wrote:
> Break up multiple assignments on the same line to resolve checkpatch
> issue.
>
> Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> ---
> drivers/staging/vt6655/device_main.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
> index f109eea..86532ca 100644
> --- a/drivers/staging/vt6655/device_main.c
> +++ b/drivers/staging/vt6655/device_main.c
> @@ -669,7 +669,8 @@ static void device_init_td1_ring(struct vnt_private *priv)
>
> if (i > 0)
> priv->apTD1Rings[i-1].next_desc = cpu_to_le32(priv->td1_pool_dma);
> - priv->apTailTD[1] = priv->apCurrTD[1] = &priv->apTD1Rings[0];
> + priv->apCurrTD[1] = &priv->apTD1Rings[0];
> + priv->apTailTD[1] = priv->apCurrTD[1];
Same problem here, please be careful with your indentation, always run
your patches through checkpatch.pl before sending them out so that
people don't have to remind you to run your patches through
checkpatch.pl before sending them out.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments
2016-09-18 10:40 ` Greg Kroah-Hartman
@ 2016-09-18 10:54 ` Namrata A Shettar
2016-09-18 11:07 ` [Outreachy kernel] " Greg Kroah-Hartman
0 siblings, 1 reply; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 10:54 UTC (permalink / raw)
To: Greg Kroah-Hartman, Forest Bond, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
On Sun, Sep 18, 2016 at 12:40:59PM +0200, Greg Kroah-Hartman wrote:
> On Sun, Sep 18, 2016 at 02:35:39PM +0530, Namrata A Shettar wrote:
> > Break multiple assignments on same line to resolve checkpatch issue.
> >
> > Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> > ---
> > drivers/staging/vt6655/card.c | 6 ++++--
> > 1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/staging/vt6655/card.c b/drivers/staging/vt6655/card.c
> > index afb1e8b..00e7858 100644
> > --- a/drivers/staging/vt6655/card.c
> > +++ b/drivers/staging/vt6655/card.c
> > @@ -528,8 +528,10 @@ CARDvSafeResetTx(
> > struct vnt_tx_desc *pCurrTD;
> >
> > /* initialize TD index */
> > - priv->apTailTD[0] = priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> > - priv->apTailTD[1] = priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> > + priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> > + priv->apTailTD[0] = priv->apCurrTD[0];
> > + priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> > + priv->apTailTD[1] = priv->apCurrTD[1];
>
> Does that look correct to you?
>
> (hint, odd indentation)
>
> And really, the original code is fine, I would recommend just leaving
> this alone, it makes more sense as-is, don't you think?
>
> thanks,
>
> greg k-h
Does this hold true for any multiple assignments warning by running
checkpatch.pl? or is it only for the change that has been made here?
thanks,
namrata
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 4/4] staging: vt6655: device_main: Break up multiple assignments
2016-09-18 10:41 ` Greg Kroah-Hartman
@ 2016-09-18 10:56 ` Namrata A Shettar
0 siblings, 0 replies; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 10:56 UTC (permalink / raw)
To: Greg Kroah-Hartman, Forest Bond, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
On Sun, Sep 18, 2016 at 12:41:40PM +0200, Greg Kroah-Hartman wrote:
> On Sun, Sep 18, 2016 at 02:35:53PM +0530, Namrata A Shettar wrote:
> > Break up multiple assignments on the same line to resolve checkpatch
> > issue.
> >
> > Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> > ---
> > drivers/staging/vt6655/device_main.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c
> > index f109eea..86532ca 100644
> > --- a/drivers/staging/vt6655/device_main.c
> > +++ b/drivers/staging/vt6655/device_main.c
> > @@ -669,7 +669,8 @@ static void device_init_td1_ring(struct vnt_private *priv)
> >
> > if (i > 0)
> > priv->apTD1Rings[i-1].next_desc = cpu_to_le32(priv->td1_pool_dma);
> > - priv->apTailTD[1] = priv->apCurrTD[1] = &priv->apTD1Rings[0];
> > + priv->apCurrTD[1] = &priv->apTD1Rings[0];
> > + priv->apTailTD[1] = priv->apCurrTD[1];
>
> Same problem here, please be careful with your indentation, always run
> your patches through checkpatch.pl before sending them out so that
> people don't have to remind you to run your patches through
> checkpatch.pl before sending them out.
>
> thanks,
>
> greg k-h
Really sorry for that :/ Will be more careful the next time around.
thanks,
namrata
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments
2016-09-18 10:54 ` Namrata A Shettar
@ 2016-09-18 11:07 ` Greg Kroah-Hartman
2016-09-18 11:13 ` Namrata A Shettar
0 siblings, 1 reply; 11+ messages in thread
From: Greg Kroah-Hartman @ 2016-09-18 11:07 UTC (permalink / raw)
To: Namrata A Shettar
Cc: Forest Bond, Malcolm Priestley, Anson Jacob, Haneen Mohammed,
outreachy-kernel
On Sun, Sep 18, 2016 at 04:24:48PM +0530, Namrata A Shettar wrote:
> On Sun, Sep 18, 2016 at 12:40:59PM +0200, Greg Kroah-Hartman wrote:
> > On Sun, Sep 18, 2016 at 02:35:39PM +0530, Namrata A Shettar wrote:
> > > Break multiple assignments on same line to resolve checkpatch issue.
> > >
> > > Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> > > ---
> > > drivers/staging/vt6655/card.c | 6 ++++--
> > > 1 file changed, 4 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/staging/vt6655/card.c b/drivers/staging/vt6655/card.c
> > > index afb1e8b..00e7858 100644
> > > --- a/drivers/staging/vt6655/card.c
> > > +++ b/drivers/staging/vt6655/card.c
> > > @@ -528,8 +528,10 @@ CARDvSafeResetTx(
> > > struct vnt_tx_desc *pCurrTD;
> > >
> > > /* initialize TD index */
> > > - priv->apTailTD[0] = priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> > > - priv->apTailTD[1] = priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> > > + priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> > > + priv->apTailTD[0] = priv->apCurrTD[0];
> > > + priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> > > + priv->apTailTD[1] = priv->apCurrTD[1];
> >
> > Does that look correct to you?
> >
> > (hint, odd indentation)
> >
> > And really, the original code is fine, I would recommend just leaving
> > this alone, it makes more sense as-is, don't you think?
> >
> > thanks,
> >
> > greg k-h
>
> Does this hold true for any multiple assignments warning by running
> checkpatch.pl? or is it only for the change that has been made here?
Use your best judgement :)
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments
2016-09-18 11:07 ` [Outreachy kernel] " Greg Kroah-Hartman
@ 2016-09-18 11:13 ` Namrata A Shettar
0 siblings, 0 replies; 11+ messages in thread
From: Namrata A Shettar @ 2016-09-18 11:13 UTC (permalink / raw)
To: Greg Kroah-Hartman, Forest Bond, Malcolm Priestley, Anson Jacob,
Haneen Mohammed, outreachy-kernel
On Sun, Sep 18, 2016 at 01:07:18PM +0200, Greg Kroah-Hartman wrote:
> On Sun, Sep 18, 2016 at 04:24:48PM +0530, Namrata A Shettar wrote:
> > On Sun, Sep 18, 2016 at 12:40:59PM +0200, Greg Kroah-Hartman wrote:
> > > On Sun, Sep 18, 2016 at 02:35:39PM +0530, Namrata A Shettar wrote:
> > > > Break multiple assignments on same line to resolve checkpatch issue.
> > > >
> > > > Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> > > > ---
> > > > drivers/staging/vt6655/card.c | 6 ++++--
> > > > 1 file changed, 4 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/drivers/staging/vt6655/card.c b/drivers/staging/vt6655/card.c
> > > > index afb1e8b..00e7858 100644
> > > > --- a/drivers/staging/vt6655/card.c
> > > > +++ b/drivers/staging/vt6655/card.c
> > > > @@ -528,8 +528,10 @@ CARDvSafeResetTx(
> > > > struct vnt_tx_desc *pCurrTD;
> > > >
> > > > /* initialize TD index */
> > > > - priv->apTailTD[0] = priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> > > > - priv->apTailTD[1] = priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> > > > + priv->apCurrTD[0] = &(priv->apTD0Rings[0]);
> > > > + priv->apTailTD[0] = priv->apCurrTD[0];
> > > > + priv->apCurrTD[1] = &(priv->apTD1Rings[0]);
> > > > + priv->apTailTD[1] = priv->apCurrTD[1];
> > >
> > > Does that look correct to you?
> > >
> > > (hint, odd indentation)
> > >
> > > And really, the original code is fine, I would recommend just leaving
> > > this alone, it makes more sense as-is, don't you think?
> > >
> > > thanks,
> > >
> > > greg k-h
> >
> > Does this hold true for any multiple assignments warning by running
> > checkpatch.pl? or is it only for the change that has been made here?
>
> Use your best judgement :)
Alright! Thanks :)
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2016-09-18 11:13 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-18 9:04 [PATCH v2 0/4] staging: vt6655: Fix checkpatch issues Namrata A Shettar
2016-09-18 9:04 ` [PATCH v2 1/4] staging: vt6655: device_main: Replace NULL comparison with !x Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 2/4] staging: vt6655: device_main: Replace explicit NULL comparison Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 3/4] staging: vt6655: card: Break multiple assignments Namrata A Shettar
2016-09-18 10:40 ` Greg Kroah-Hartman
2016-09-18 10:54 ` Namrata A Shettar
2016-09-18 11:07 ` [Outreachy kernel] " Greg Kroah-Hartman
2016-09-18 11:13 ` Namrata A Shettar
2016-09-18 9:05 ` [PATCH v2 4/4] staging: vt6655: device_main: Break up " Namrata A Shettar
2016-09-18 10:41 ` Greg Kroah-Hartman
2016-09-18 10:56 ` Namrata A Shettar
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.