[PATCH net 0/2] r8169: fix rx vlan · Hayes Wang <hidden> · 2014-09-12
[PATCH net 1/2] r8169: fix the default setting of rx vlan · Hayes Wang <hidden> · 2014-09-12
Re: [PATCH net 1/2] r8169: fix the default setting of rx vlan · Francois Romieu <romieu@fr.zoreil.com> · 2014-09-12
RE: [PATCH net 1/2] r8169: fix the default setting of rx vlan · Hayes Wang <hidden> · 2014-09-15
Re: [PATCH net 1/2] r8169: fix the default setting of rx vlan · Francois Romieu <romieu@fr.zoreil.com> · 2014-09-16
RE: [PATCH net 1/2] r8169: fix the default setting of rx vlan · Hayes Wang <hidden> · 2014-09-16
[PATCH net] r8169: adjust __rtl8169_set_features · Hayes Wang <hidden> · 2014-09-16
Re: [PATCH net] r8169: adjust __rtl8169_set_features · David Miller <davem@davemloft.net> · 2014-09-16
[PATCH net 2/2] r8169: fix setting rx vlan · Hayes Wang <hidden> · 2014-09-12
Re: [PATCH net 0/2] r8169: fix rx vlan · David Miller <davem@davemloft.net> · 2014-09-13

RE: [PATCH net 1/2] r8169: fix the default setting of rx vlan

From: Hayes Wang <hidden>
Date: 2014-09-16 03:23:44
Also in: lkml

 From: Francois Romieu [mailto:romieu@fr.zoreil.com]

Sent: Tuesday, September 16, 2014 8:03 AM

[...]

However it's getting messy and the code stomps CPlusCmd in 
rtl_open right
before it's given a chance to be read in hw_start. Something like the
untested patch below should avoid it and keep related things together.

You are right. Your patch is more reasonable. I would sent the patch.
Thanks.
 
Best Regards,
Hayes

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help