* [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
@ 2021-04-10 13:00 Mitali Borkar
2021-04-10 13:14 ` Greg KH
2021-04-10 13:23 ` Julia Lawall
0 siblings, 2 replies; 7+ messages in thread
From: Mitali Borkar @ 2021-04-10 13:00 UTC (permalink / raw)
To: gregkh; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
Fixed Comparison to NULL can be written as '!...' by replacing it with
simpler form i.e. boolean expression. This makes code more readable
alternative.
Reported by checkpatch.
Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
---
Changes from v1:- added pointer to the function, which was missed during
fixing v1.
drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index 4457c1acfbf6..78b5b4eaec5f 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -327,7 +327,7 @@ bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS,
}
*ppTS = SearchAdmitTRStream(ieee, Addr, UP, TxRxSelect);
- if (ppTS)
+ if (*ppTS)
return true;
if (!bAddNewTs) {
--
2.30.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
2021-04-10 13:00 [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*' Mitali Borkar
@ 2021-04-10 13:14 ` Greg KH
2021-04-10 13:37 ` Mitali Borkar
2021-04-10 13:23 ` Julia Lawall
1 sibling, 1 reply; 7+ messages in thread
From: Greg KH @ 2021-04-10 13:14 UTC (permalink / raw)
To: Mitali Borkar; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Sat, Apr 10, 2021 at 06:30:38PM +0530, Mitali Borkar wrote:
> Fixed Comparison to NULL can be written as '!...' by replacing it with
> simpler form i.e. boolean expression. This makes code more readable
> alternative.
> Reported by checkpatch.
Checkpatch did not report this specific problem, Julia did :)
And this changelog text does not reflect the commit you made here.
>
> Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
We need a "Reported-by:" line here to reflect that someone reported the
problem as well.
Ideally it will have a "Fixes:" tag also, but I can add that if you
don't know how to do that just yet.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
2021-04-10 13:00 [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*' Mitali Borkar
@ 2021-04-10 13:23 ` Julia Lawall
2021-04-10 13:23 ` Julia Lawall
1 sibling, 0 replies; 7+ messages in thread
From: Julia Lawall @ 2021-04-10 13:23 UTC (permalink / raw)
To: Mitali Borkar
Cc: gregkh, linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Sat, 10 Apr 2021, Mitali Borkar wrote:
> Fixed Comparison to NULL can be written as '!...' by replacing it with
> simpler form i.e. boolean expression. This makes code more readable
> alternative.
> Reported by checkpatch.
>
> Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> ---
> Changes from v1:- added pointer to the function, which was missed during
> fixing v1.
>
> drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
> index 4457c1acfbf6..78b5b4eaec5f 100644
> --- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
> +++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
> @@ -327,7 +327,7 @@ bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS,
> }
>
> *ppTS = SearchAdmitTRStream(ieee, Addr, UP, TxRxSelect);
> - if (ppTS)
> + if (*ppTS)
This looks like a patch against your previous patch, and not against the
original code.
julia
> return true;
>
> if (!bAddNewTs) {
> --
> 2.30.2
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/YHGhdtldqAlRsPHT%40kali.
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
@ 2021-04-10 13:23 ` Julia Lawall
0 siblings, 0 replies; 7+ messages in thread
From: Julia Lawall @ 2021-04-10 13:23 UTC (permalink / raw)
To: Mitali Borkar
Cc: gregkh, linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Sat, 10 Apr 2021, Mitali Borkar wrote:
> Fixed Comparison to NULL can be written as '!...' by replacing it with
> simpler form i.e. boolean expression. This makes code more readable
> alternative.
> Reported by checkpatch.
>
> Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> ---
> Changes from v1:- added pointer to the function, which was missed during
> fixing v1.
>
> drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
> index 4457c1acfbf6..78b5b4eaec5f 100644
> --- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
> +++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
> @@ -327,7 +327,7 @@ bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS,
> }
>
> *ppTS = SearchAdmitTRStream(ieee, Addr, UP, TxRxSelect);
> - if (ppTS)
> + if (*ppTS)
This looks like a patch against your previous patch, and not against the
original code.
julia
> return true;
>
> if (!bAddNewTs) {
> --
> 2.30.2
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/YHGhdtldqAlRsPHT%40kali.
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
2021-04-10 13:23 ` Julia Lawall
(?)
@ 2021-04-10 13:26 ` Greg KH
-1 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2021-04-10 13:26 UTC (permalink / raw)
To: Julia Lawall
Cc: Mitali Borkar, linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Sat, Apr 10, 2021 at 03:23:35PM +0200, Julia Lawall wrote:
>
>
> On Sat, 10 Apr 2021, Mitali Borkar wrote:
>
> > Fixed Comparison to NULL can be written as '!...' by replacing it with
> > simpler form i.e. boolean expression. This makes code more readable
> > alternative.
> > Reported by checkpatch.
> >
> > Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> > ---
> > Changes from v1:- added pointer to the function, which was missed during
> > fixing v1.
> >
> > drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
> > index 4457c1acfbf6..78b5b4eaec5f 100644
> > --- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
> > +++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
> > @@ -327,7 +327,7 @@ bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS,
> > }
> >
> > *ppTS = SearchAdmitTRStream(ieee, Addr, UP, TxRxSelect);
> > - if (ppTS)
> > + if (*ppTS)
>
> This looks like a patch against your previous patch, and not against the
> original code.
That's fine as I already accepted the previous patch.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
2021-04-10 13:14 ` Greg KH
@ 2021-04-10 13:37 ` Mitali Borkar
2021-04-10 13:48 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Mitali Borkar @ 2021-04-10 13:37 UTC (permalink / raw)
To: Greg KH; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Sat, Apr 10, 2021 at 03:14:24PM +0200, Greg KH wrote:
> On Sat, Apr 10, 2021 at 06:30:38PM +0530, Mitali Borkar wrote:
> > Fixed Comparison to NULL can be written as '!...' by replacing it with
> > simpler form i.e. boolean expression. This makes code more readable
> > alternative.
> > Reported by checkpatch.
>
> Checkpatch did not report this specific problem, Julia did :)
>
> And this changelog text does not reflect the commit you made here.
>
Making the changes now.
> >
> > Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
>
> We need a "Reported-by:" line here to reflect that someone reported the
> problem as well.
>
I am resending this as patch v3. I have to add reprted by Julia, right?
> Ideally it will have a "Fixes:" tag also, but I can add that if you
> don't know how to do that just yet.
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*'
2021-04-10 13:37 ` Mitali Borkar
@ 2021-04-10 13:48 ` Greg KH
0 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2021-04-10 13:48 UTC (permalink / raw)
To: Mitali Borkar; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Sat, Apr 10, 2021 at 07:07:08PM +0530, Mitali Borkar wrote:
> On Sat, Apr 10, 2021 at 03:14:24PM +0200, Greg KH wrote:
> > On Sat, Apr 10, 2021 at 06:30:38PM +0530, Mitali Borkar wrote:
> > > Fixed Comparison to NULL can be written as '!...' by replacing it with
> > > simpler form i.e. boolean expression. This makes code more readable
> > > alternative.
> > > Reported by checkpatch.
> >
> > Checkpatch did not report this specific problem, Julia did :)
> >
> > And this changelog text does not reflect the commit you made here.
> >
> Making the changes now.
>
> > >
> > > Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> >
> > We need a "Reported-by:" line here to reflect that someone reported the
> > problem as well.
> >
> I am resending this as patch v3. I have to add reprted by Julia, right?
Yes please.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-04-10 13:48 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-10 13:00 [PATCH v2] staging: rtl8192e: fixed pointer error by adding '*' Mitali Borkar
2021-04-10 13:14 ` Greg KH
2021-04-10 13:37 ` Mitali Borkar
2021-04-10 13:48 ` Greg KH
2021-04-10 13:23 ` [Outreachy kernel] " Julia Lawall
2021-04-10 13:23 ` Julia Lawall
2021-04-10 13:26 ` Greg KH
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.