Skip to content

rtl8187: Reduce delays to reduce startup time

Larry Finger linux-wireless 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6rtl8187: Remove large delaysdrivers/net/wireless/rtl8187_dev.c drivers/net/wireless/rtl8187_rtl8225.c+1 -30 replies
2/6rtl8187: Remove OFDM delaysdrivers/net/wireless/rtl8187_rtl8225.c+82 -860 replies
3/6rtl8187: Remove CCK delaysdrivers/net/wireless/rtl8187_rtl8225.c+46 -460 replies
4/6rtl8187: Remove msleep calls after calls to rtl8225_writedrivers/net/wireless/rtl8187_rtl8225.c+58 -570 replies
5/6rtl8187: Remove primitive write delaysdrivers/net/wireless/rtl8187_dev.c drivers/net/wireless/rtl8187_rtl8225.c+0 -40 replies
6/6rtl8187: Reduce channel switch delaydrivers/net/wireless/rtl8187_dev.c+0 -10 replies

Reviews trailers per patch

None of the 6 patches has a Reviewed-by or Acked-by.

PatchHerton Ronaldo KrzesinskiHin-Tak Leung
SeriesT-b
1/6T-b T-b
2/6T-b T-b
3/6T-b T-b
4/6T-b T-b
5/6T-b T-b
6/6T-b T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/6] rtl8187: Reduce delays to reduce startup time

The current version of the rtl8187 driver contains a large number of msleep() commands that cause initialization to take a long time. This series of patches include revisions that were found by looking at the latest version of the vendor's driver (rtl8187B_linux_26.1036.0708.2008), and others that were found to be OK with testing.

To keep each of the patches relatively small, these changes are split into 6 parts. The changes in them are as follows:

1. Remove all delays of greater than 1 sec.
2. Remove the msleep(1) that immediately follow calls to rtl8225_write_phy_ofdm().
3. Remove the msleep(1) that immediately follow calls to rtl8225_write_phy_cck().
4. Remove the msleep(1) that immediately follow calls to rtl8225_write().
5. Remove the nsleep() calls from rtl8187_write_phy(), rtl8225_write_bitbang(), and rtl8225_write_8051().
6. Remove the msleep(10) that preceeds a channel change.

Before these changes, routine rtl8187b_init_hw() took 15.82 sec to execute. It now takes 3.76 sec.

---