Re: [zd1211-devs] [PATCH] zd1211rw: fix unbalanced mutex_lock in zd_chip_led_status
Ulrich Kunitz wrote: > Nobody has started on the descape stack, but it is certainly > something which needs to be done in the future. Luis Rodriguez at least started on this, but has not published any code to my knowledge. Daniel - Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 ___ Zd1211-devs mailing list - http://zd1211.ath.cx/ Unsubscribe: https://lists.sourceforge.net/lists/listinfo/zd1211-devs
Re: [zd1211-devs] [PATCH] zd1211rw: fix unbalanced mutex_lock in zd_chip_led_status
On 06-08-23 13:34 Maxime Austruy wrote: > Cool. Btw, are there any plans to move this code to the devicescape > 802.11 stack? I saw some patches flying around on [EMAIL PROTECTED] that add > support for LED to it. I'm guessing that this implements the blinking > that you added but in a generic way: every wireless nic driver just has > to provide a turn led on/off interface. Nobody has started on the descape stack, but it is certainly something which needs to be done in the future. However we cannot use this on/off interface, because every on/off switch would require it's own USB transaction. The ZD1211 chip has a firmware-controlled mode, which doesn't come with this price. Regards, Uli -- Uli Kunitz - Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 ___ Zd1211-devs mailing list - http://zd1211.ath.cx/ Unsubscribe: https://lists.sourceforge.net/lists/listinfo/zd1211-devs
Re: [zd1211-devs] [PATCH] zd1211rw: fix unbalanced mutex_lock in zd_chip_led_status
On 06-08-21 22:33 Maxime Austruy wrote: > zd_chip_led_status is only used by zd_chip_led_flip which itself is not > used, yet. While it's not vital to fix bugs in those functions, it is > nice to have a clean code base, so please consider this patch for > inclusion. It addresses the following problems: > . if read_led_reg fails or if the callsite of zd_chip_led_status passes >a bogus status, we return with chip->mutex locked, > . zd_chip_led_status returns the wrong return value when successful, > . zd_chip_led_flip doesn't properly check the return value of >zd_chip_led_status. > > Signed-off-by: Maxime Austruy <[EMAIL PROTECTED]> Thank you Maxime. However I've fixed it on my own and I support blinking now. This patch contains comparable code: http://www.deine-taler.de/git/?p=zd1211.git;a=commitdiff;h=d5cb64797be22f083fdbb34eb6aadd886b26ab03;hp=4def8e849c5e67a69a606e74ebae85c725c4e0ff Regards, Uli -- Uli Kunitz - Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 ___ Zd1211-devs mailing list - http://zd1211.ath.cx/ Unsubscribe: https://lists.sourceforge.net/lists/listinfo/zd1211-devs