Re: [PATCH] Make firmware startup polling timeout configurable, and increase default

2016-05-18 Thread Jes Sorensen
Daniel Lenski writes: > Jes Sorensen writes: >> >> Dan Lenski writes: >> > >> > This issue seems to occur because RTL8XXXU_FIRMWARE_POLL_MAX (1000) is >> > too short, and the MCU fails to start up as quickly as expected. >> > >> >

Re: [PATCH] Make firmware startup polling timeout configurable, and increase default

2016-05-17 Thread Daniel Lenski
Jes Sorensen writes: > > Dan Lenski writes: > > > > This issue seems to occur because RTL8XXXU_FIRMWARE_POLL_MAX (1000) is > > too short, and the MCU fails to start up as quickly as expected. > > > > With a longer value (5000), the driver starts up

Re: [PATCH] Make firmware startup polling timeout configurable, and increase default

2016-05-17 Thread Jes Sorensen
Dan Lenski writes: > This patch: > > - increases the default value for the maximum number of polling loops to > wait for the rtl8xxxu MCU to start after the firmware is loaded (from > 1000 to 5000) > - makes this a configurable module parameter > > With RTL8723AU chipset, I

Re: [PATCH] Make firmware startup polling timeout configurable, and increase default

2016-05-17 Thread Julian Calaby
Hi Dan, Add a "rtl8xxxu:" prefix to the patch title. This makes it easier to determine which patch is for which driver when only the titles are listed. On Wed, May 18, 2016 at 8:48 AM, Dan Lenski wrote: > This patch: > > - increases the default value for the maximum number of

[PATCH] Make firmware startup polling timeout configurable, and increase default

2016-05-17 Thread Dan Lenski
This patch: - increases the default value for the maximum number of polling loops to wait for the rtl8xxxu MCU to start after the firmware is loaded (from 1000 to 5000) - makes this a configurable module parameter With RTL8723AU chipset, I frequently encounter "Firmware failed to start"