Re: bcmdhd: Strange Power Save messages

2016-10-01 Thread Arend van Spriel
On 29-09-16 13:32, Gucea Doru wrote: > On Tue, Sep 27, 2016 at 12:03 PM, Gucea Doru wrote: >> What is the decision triggering the exit from the PS mode immediately >> after the ping request? I am asking this because 802.11 PS legacy >> specifies that the client should wait for a beacon with TIM

Re: nl80211 fine timing measurement support

2016-10-01 Thread Arend van Spriel
On 20-09-16 22:57, Lior David wrote: > On 9/20/2016 9:39 AM, Luca Coelho wrote: >> On Mon, 2016-09-19 at 20:42 +0200, Arend Van Spriel wrote: >>> >>> On 19-9-2016 19:31, Lior David wrote: Hi Johannes, We are working on adding indoor location support to the wil6210 11ad driv

Re: bcmdhd: Strange Power Save messages

2016-10-01 Thread Krishna Chaitanya
On Sat, Oct 1, 2016 at 2:52 PM, Arend van Spriel wrote: > > > On 29-09-16 13:32, Gucea Doru wrote: >> On Tue, Sep 27, 2016 at 12:03 PM, Gucea Doru wrote: >>> What is the decision triggering the exit from the PS mode immediately >>> after the ping request? I am asking this because 802.11 PS legacy

Re: [PATCH 1/2] rtl8xxxu: Fix rtl8723bu driver reload issue

2016-10-01 Thread Greg KH
On Fri, Sep 30, 2016 at 07:35:17PM -0400, jes.soren...@redhat.com wrote: > From: Jes Sorensen > > The generic disable_rf() function clears bits 22 and 23 in > REG_RX_WAIT_CCA, however we did not re-enable them again in > rtl8723b_enable_rf() > > This resolves the problem for me with 8723bu devic

Re: [PATCH 1/2] rtl8xxxu: Fix rtl8723bu driver reload issue

2016-10-01 Thread Jes Sorensen
Greg KH writes: > On Fri, Sep 30, 2016 at 07:35:17PM -0400, jes.soren...@redhat.com wrote: >> From: Jes Sorensen >> >> The generic disable_rf() function clears bits 22 and 23 in >> REG_RX_WAIT_CCA, however we did not re-enable them again in >> rtl8723b_enable_rf() >> >> This resolves the proble

[PATCH] realtek: rtl8xxxu: Use const init arrays

2016-10-01 Thread Joe Perches
Make the init arrays const to reduce data. $ size drivers/net/wireless/realtek/rtl8xxxu/built-in.o* (allyesconfig: x86-32) textdata bss dec hex filename 80107 13651 58 93816 16e78 drivers/net/wireless/realtek/rtl8xxxu/built-in.o.new 65303 28435 58 93796

Re: [PATCH] realtek: rtl8xxxu: Use const init arrays

2016-10-01 Thread Jes Sorensen
Joe Perches writes: > Make the init arrays const to reduce data. > > $ size drivers/net/wireless/realtek/rtl8xxxu/built-in.o* (allyesconfig: > x86-32) >text data bss dec hex filename > 80107 13651 58 93816 16e78 > drivers/net/wireless/realtek/rtl8xxxu/

Re: [PATCH] realtek: rtl8xxxu: Use const init arrays

2016-10-01 Thread Joe Perches
On Sat, 2016-10-01 at 14:53 -0400, Jes Sorensen wrote: > Joe Perches writes: > > Make the init arrays const to reduce data. > > $ size drivers/net/wireless/realtek/rtl8xxxu/built-in.o* (allyesconfig: > > x86-32) > >   textdata bss dec hex filename > >   80107 13651

Re: [PATCH] realtek: rtl8xxxu: Use const init arrays

2016-10-01 Thread Jes Sorensen
Joe Perches writes: > On Sat, 2016-10-01 at 14:53 -0400, Jes Sorensen wrote: >> Joe Perches writes: >> > Make the init arrays const to reduce data. >> > $ size drivers/net/wireless/realtek/rtl8xxxu/built-in.o* >> > (allyesconfig: x86-32) >> >   text data bss dec hex filename >>

Re: [PATCH] realtek: rtl8xxxu: Use const init arrays

2016-10-01 Thread Joe Perches
On Sat, 2016-10-01 at 16:32 -0400, Jes Sorensen wrote: > Your output shows it moving to the text segment - if it's in a different > segment, eg. rodata, you should use output demonstrating that to justify > the change. For size, rodata _is_ text

Re: [PATCH] realtek: rtl8xxxu: Use const init arrays

2016-10-01 Thread Jes Sorensen
Joe Perches writes: > On Sat, 2016-10-01 at 16:32 -0400, Jes Sorensen wrote: >> Your output shows it moving to the text segment - if it's in a different >> segment, eg. rodata, you should use output demonstrating that to justify >> the change. > > For size, rodata _is_ text Well then maybe use so