LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] staging: r8188eu: include: remove duplicate declaration.
@ 2021-09-05 7:14 Saurav Girepunje
2021-09-06 9:48 ` Greg KH
0 siblings, 1 reply; 5+ messages in thread
From: Saurav Girepunje @ 2021-09-05 7:14 UTC (permalink / raw)
To: Larry.Finger, phil, gregkh, saurav.girepunje, linux-kernel
Cc: saurav.girepunje
Remove below duplicate declaration from rtl8188e_led.h file
void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
---
drivers/staging/r8188eu/include/rtl8188e_led.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/r8188eu/include/rtl8188e_led.h b/drivers/staging/r8188eu/include/rtl8188e_led.h
index b00954198764..02cdc970bb17 100644
--- a/drivers/staging/r8188eu/include/rtl8188e_led.h
+++ b/drivers/staging/r8188eu/include/rtl8188e_led.h
@@ -12,7 +12,5 @@
/* */
void rtl8188eu_InitSwLeds(struct adapter *padapter);
void rtl8188eu_DeInitSwLeds(struct adapter *padapter);
-void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
-void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
#endif
--
2.32.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: r8188eu: include: remove duplicate declaration.
2021-09-05 7:14 [PATCH] staging: r8188eu: include: remove duplicate declaration Saurav Girepunje
@ 2021-09-06 9:48 ` Greg KH
2021-09-06 16:34 ` Saurav Girepunje
0 siblings, 1 reply; 5+ messages in thread
From: Greg KH @ 2021-09-06 9:48 UTC (permalink / raw)
To: Saurav Girepunje; +Cc: Larry.Finger, phil, linux-kernel, saurav.girepunje
On Sun, Sep 05, 2021 at 12:44:37PM +0530, Saurav Girepunje wrote:
> Remove below duplicate declaration from rtl8188e_led.h file
>
> void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
> void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
>
> Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
> ---
> drivers/staging/r8188eu/include/rtl8188e_led.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/staging/r8188eu/include/rtl8188e_led.h b/drivers/staging/r8188eu/include/rtl8188e_led.h
> index b00954198764..02cdc970bb17 100644
> --- a/drivers/staging/r8188eu/include/rtl8188e_led.h
> +++ b/drivers/staging/r8188eu/include/rtl8188e_led.h
> @@ -12,7 +12,5 @@
> /* */
> void rtl8188eu_InitSwLeds(struct adapter *padapter);
> void rtl8188eu_DeInitSwLeds(struct adapter *padapter);
> -void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
> -void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
>
> #endif
> --
> 2.32.0
>
This email did not make it to lore.kernel.org again:
https://lore.kernel.org/r/YTRuXTu/kePBDwAF@user
Any ideas what is happening?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: r8188eu: include: remove duplicate declaration.
2021-09-06 9:48 ` Greg KH
@ 2021-09-06 16:34 ` Saurav Girepunje
2021-09-07 6:28 ` Greg KH
2021-09-07 6:40 ` Greg KH
0 siblings, 2 replies; 5+ messages in thread
From: Saurav Girepunje @ 2021-09-06 16:34 UTC (permalink / raw)
To: Greg KH; +Cc: saurav.girepunje, Larry.Finger, phil, linux-kernel
On 06/09/21 3:18 pm, Greg KH wrote:
> On Sun, Sep 05, 2021 at 12:44:37PM +0530, Saurav Girepunje wrote:
>> Remove below duplicate declaration from rtl8188e_led.h file
>>
>> void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
>> void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
>>
>> Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
>> ---
>> drivers/staging/r8188eu/include/rtl8188e_led.h | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/drivers/staging/r8188eu/include/rtl8188e_led.h b/drivers/staging/r8188eu/include/rtl8188e_led.h
>> index b00954198764..02cdc970bb17 100644
>> --- a/drivers/staging/r8188eu/include/rtl8188e_led.h
>> +++ b/drivers/staging/r8188eu/include/rtl8188e_led.h
>> @@ -12,7 +12,5 @@
>> /* */
>> void rtl8188eu_InitSwLeds(struct adapter *padapter);
>> void rtl8188eu_DeInitSwLeds(struct adapter *padapter);
>> -void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
>> -void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
>>
>> #endif
>> --
>> 2.32.0
>>
>
> This email did not make it to lore.kernel.org again:
> https://lore.kernel.org/r/YTRuXTu/kePBDwAF@user
>
> Any ideas what is happening?
> linux-staging@lists.linux.dev
> thanks,
>
> greg k-h
>
I missed to add the linux-staging@lists.linux.dev. I will resend it v2
version of this patch.
However I am able to see it on https://lore.kernel.org .
https://lore.kernel.org/all/YTRuXTu%2FkePBDwAF@user/
Regards,
Saurav Girepunje
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: r8188eu: include: remove duplicate declaration.
2021-09-06 16:34 ` Saurav Girepunje
@ 2021-09-07 6:28 ` Greg KH
2021-09-07 6:40 ` Greg KH
1 sibling, 0 replies; 5+ messages in thread
From: Greg KH @ 2021-09-07 6:28 UTC (permalink / raw)
To: Saurav Girepunje; +Cc: saurav.girepunje, Larry.Finger, phil, linux-kernel
On Mon, Sep 06, 2021 at 10:04:07PM +0530, Saurav Girepunje wrote:
>
>
> On 06/09/21 3:18 pm, Greg KH wrote:
> > On Sun, Sep 05, 2021 at 12:44:37PM +0530, Saurav Girepunje wrote:
> > > Remove below duplicate declaration from rtl8188e_led.h file
> > >
> > > void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
> > > void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
> > >
> > > Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
> > > ---
> > > drivers/staging/r8188eu/include/rtl8188e_led.h | 2 --
> > > 1 file changed, 2 deletions(-)
> > >
> > > diff --git a/drivers/staging/r8188eu/include/rtl8188e_led.h b/drivers/staging/r8188eu/include/rtl8188e_led.h
> > > index b00954198764..02cdc970bb17 100644
> > > --- a/drivers/staging/r8188eu/include/rtl8188e_led.h
> > > +++ b/drivers/staging/r8188eu/include/rtl8188e_led.h
> > > @@ -12,7 +12,5 @@
> > > /* */
> > > void rtl8188eu_InitSwLeds(struct adapter *padapter);
> > > void rtl8188eu_DeInitSwLeds(struct adapter *padapter);
> > > -void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
> > > -void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
> > >
> > > #endif
> > > --
> > > 2.32.0
> > >
> >
> > This email did not make it to lore.kernel.org again:
> > https://lore.kernel.org/r/YTRuXTu/kePBDwAF@user
> >
> > Any ideas what is happening?
> > linux-staging@lists.linux.dev
> > thanks,
> >
> > greg k-h
> >
>
> I missed to add the linux-staging@lists.linux.dev. I will resend it v2
> version of this patch.
> However I am able to see it on https://lore.kernel.org .
>
> https://lore.kernel.org/all/YTRuXTu%2FkePBDwAF@user/
Ah, your message id has a '/' character in it, that's what is breaking
the scripts...
Let me see what needs to be fixed up.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: r8188eu: include: remove duplicate declaration.
2021-09-06 16:34 ` Saurav Girepunje
2021-09-07 6:28 ` Greg KH
@ 2021-09-07 6:40 ` Greg KH
1 sibling, 0 replies; 5+ messages in thread
From: Greg KH @ 2021-09-07 6:40 UTC (permalink / raw)
To: Saurav Girepunje; +Cc: saurav.girepunje, Larry.Finger, phil, linux-kernel
On Mon, Sep 06, 2021 at 10:04:07PM +0530, Saurav Girepunje wrote:
>
>
> On 06/09/21 3:18 pm, Greg KH wrote:
> > On Sun, Sep 05, 2021 at 12:44:37PM +0530, Saurav Girepunje wrote:
> > > Remove below duplicate declaration from rtl8188e_led.h file
> > >
> > > void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
> > > void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
> > >
> > > Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
> > > ---
> > > drivers/staging/r8188eu/include/rtl8188e_led.h | 2 --
> > > 1 file changed, 2 deletions(-)
> > >
> > > diff --git a/drivers/staging/r8188eu/include/rtl8188e_led.h b/drivers/staging/r8188eu/include/rtl8188e_led.h
> > > index b00954198764..02cdc970bb17 100644
> > > --- a/drivers/staging/r8188eu/include/rtl8188e_led.h
> > > +++ b/drivers/staging/r8188eu/include/rtl8188e_led.h
> > > @@ -12,7 +12,5 @@
> > > /* */
> > > void rtl8188eu_InitSwLeds(struct adapter *padapter);
> > > void rtl8188eu_DeInitSwLeds(struct adapter *padapter);
> > > -void SwLedOn(struct adapter *padapter, struct LED_871x *pLed);
> > > -void SwLedOff(struct adapter *padapter, struct LED_871x *pLed);
> > >
> > > #endif
> > > --
> > > 2.32.0
> > >
> >
> > This email did not make it to lore.kernel.org again:
> > https://lore.kernel.org/r/YTRuXTu/kePBDwAF@user
> >
> > Any ideas what is happening?
> > linux-staging@lists.linux.dev
> > thanks,
> >
> > greg k-h
> >
>
> I missed to add the linux-staging@lists.linux.dev. I will resend it v2
> version of this patch.
> However I am able to see it on https://lore.kernel.org .
>
> https://lore.kernel.org/all/YTRuXTu%2FkePBDwAF@user/
You are right, that is odd. I've emailed the b4 developer to see what
is up with this, I'll take this patch in a bit...
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-09-07 6:40 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-05 7:14 [PATCH] staging: r8188eu: include: remove duplicate declaration Saurav Girepunje
2021-09-06 9:48 ` Greg KH
2021-09-06 16:34 ` Saurav Girepunje
2021-09-07 6:28 ` Greg KH
2021-09-07 6:40 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).