The iwm(4) iwm_poll_bit() function returns 1 on success, and 0 on failure,
whereas the iwl_poll_bit() in Linux iwlwifi returns < 0 on failure.
So the (ret < 0) check ended up ignoring any error returned by
iwm_poll_bit().
Differential D7932 Authored by ivadasz on Sep 18 2016, 9:29 AM.
Details Summary The iwm(4) iwm_poll_bit() function returns 1 on success, and 0 on failure, So the (ret < 0) check ended up ignoring any error returned by
Diff Detail
Event Timelineivadasz updated this object. Herald added a subscriber: imp. · View Herald TranscriptSep 18 2016, 9:29 AM2016-09-18 09:29:06 (UTC+0) adrian edited edge metadata. This revision is now accepted and ready to land.Sep 19 2016, 9:36 PM2016-09-19 21:36:28 (UTC+0) Closed by commit rS306002: [iwm] Fix iwm_poll_bit() error value check in iwm_attach(). (authored by ivadasz). · Explain WhySep 19 2016, 10:13 PM2016-09-19 22:13:20 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 20502 head/sys/dev/iwm/if_iwm.c
|
Should we use the Oxford comma like in the original?