Johannes Berg wrote: > On Thu, 2009-04-02 at 09:44 -0500, Larry Finger wrote: > >>> Can you check the contents of >>> /sys/class/leds/b43legacy-...::radio/trigger >>> please? >> The first thing I noticed is that "radio" gets truncated to "rad", thus the file >> is /sys/class/leds/b43legacy-phy0\:\:rad/trigger, > > Cute. Must be that the limit is 20 bytes (19+NUL) in the buffer it uses > in led core code. > >> with contents "none ide-disk >> ADP1-online BAT0-charging-or-full BAT0-charging BAT0-full phy0rx phy0tx >> phy0assoc phy0radio [(]". I unloaded and reloaded the driver and found that the >> "[(]" characters changed to "[�W[�]". It appears that something is putting >> garbage in that file. > > Ouch. That's bad. I'll try to figure it out. I don't see how my patch > changes this -- does this happen without my patch too? Without your "rfkill rewrite" patch, but with "rfkill-remove-unused-code", "rfkill-remove-deprecated", and "rfkill-remove-user-claim" patches, the file /sys/class/..../trigger contains "none ide-disk BAT0-charging-or-full BAT0-charging BAT0-full ADP1-online phy0rx phy0tx phy0assoc phy0radio [rfkill0]" Now I know what should be inside the []. Larry -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
