Page MenuHomeFreeBSD
Differential D7932

[iwm] Fix iwm_poll_bit() error value check in iwm_attach().
ClosedPublic
Actions

Authored by ivadasz on Sep 18 2016, 9:29 AM.
Tags
None
Referenced Files
F103553676: D7932.diff
Tue, Nov 26, 11:12 AM
Unknown Object (File)
Sun, Nov 24, 3:53 PM
Unknown Object (File)
Sun, Nov 24, 3:53 PM
Unknown Object (File)
Sun, Nov 24, 3:53 PM
Unknown Object (File)
Sun, Nov 24, 3:52 PM
Unknown Object (File)
Tue, Nov 19, 11:57 PM
Unknown Object (File)
Tue, Nov 19, 10:53 AM
Unknown Object (File)
Fri, Nov 15, 4:48 PM
View All 56 Files
Subscribers
imp

Details

Reviewers
cognet
adrian
Commits
rS306002: [iwm] Fix iwm_poll_bit() error value check in iwm_attach().
Summary

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().

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ivadasz updated this revision to Diff 20435.Sep 18 2016, 9:29 AM
ivadasz retitled this revision from to [iwm] Fix iwm_poll_bit() error value check in iwm_attach()..
ivadasz updated this object.
ivadasz added reviewers: adrian, cognet.
Herald added a subscriber: imp. · View Herald TranscriptSep 18 2016, 9:29 AM
adrian accepted this revision.Sep 19 2016, 9:36 PM
adrian edited edge metadata.
This revision is now accepted and ready to land.Sep 19 2016, 9:36 PM
Closed by commit rS306002: [iwm] Fix iwm_poll_bit() error value check in iwm_attach(). (authored by ivadasz). · Explain WhySep 19 2016, 10:13 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
iwm/
2 lines

Diff 20502

View Options

head/sys/dev/iwm/if_iwm.c

Loading...