1.Could you please review this patch and suggest me any changes?

2. How do I know whether this patch is approved or not?

Thanks
Sanjana.


On Wed, 13 Mar 2019, 08:39 Sanjana Sanikommu, <sanjana99reddy99@gmail.com>
wrote:

> Subject: [PATCH] Staging: rtl8188eu: rtw_recv.c: fix a tab indent issue
>
> Fix a tab issue in 2 lines which removed no extra space before starting
> a line warning. This warning appeared when I compiled checkpatch.pl in
> rtw_recv.c,
>
> Signed-off-by: Sanjana Sanikommu<sanjana99reddy99@gmail.com>
>
> ---
>  drivers/staging/rtl8188eu/core/rtw_recv.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/rtl8188eu/core/rtw_recv.c
> b/drivers/staging/rtl8188eu/core/rtw_recv.c
> index 1d83affc08ce..cc6f1a23da01 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_recv.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_recv.c
> @@ -24,11 +24,11 @@ static u8 SNAP_ETH_TYPE_APPLETALK_AARP[2] = {0x80,
> 0xf3};
>
>  /* Bridge-Tunnel header (for EtherTypes ETH_P_AARP and ETH_P_IPX) */
>  static u8 rtw_bridge_tunnel_header[] = {
> -       0xaa, 0xaa, 0x03, 0x00, 0x00, 0xf8
> +        0xaa, 0xaa, 0x03, 0x00, 0x00, 0xf8
>  };
>
>  static u8 rtw_rfc1042_header[] = {
> -       0xaa, 0xaa, 0x03, 0x00, 0x00, 0x00
> +       0xaa, 0xaa, 0x03, 0x00, 0x00, 0x00
>  };
>
>  static void rtw_signal_stat_timer_hdl(struct timer_list *t);
> --
> 2.17.1
>
>
> On Wed, Mar 13, 2019 at 8:36 AM Sanjana Sanikommu <
> sanjana99reddy99@gmail.com> wrote:
>
>> I reedited the file by taking note of the above mentioned mistakes. I
>> would be resending the patch as this has lot of mistakes :)
>>
>> On Wed, Mar 13, 2019 at 1:18 AM Greg KH <gregkh@linuxfoundation.org>
>> wrote:
>>
>>> On Tue, Mar 12, 2019 at 12:41:37PM -0700, Greg KH wrote:
>>> > On Wed, Mar 13, 2019 at 12:56:34AM +0530, Sanjana Reddy wrote:
>>> > > Fixed a space in 2 lines which removed warnings.
>>> > > Signed-off-by: Sanjana <sanjana99reddy99@gmail.com>
>>> >
>>> > Please put a blank line between the changelog text and your
>>> > signed-off-by: line, otherwise the tools will choke :(
>>>
>>> Also we need a "real" name here, not just a first name.  And your email
>>> address should be correct, it's bouncing right now :(
>>>
>>> thanks,
>>>
>>> greg k-h
>>>
>>