Bug 93291
| Summary: | Potentially incorrect interrupt management in bmg160.c | ||
|---|---|---|---|
| Product: | Drivers | Reporter: | Leonid Ryzhyk (ryzhyk) |
| Component: | Other | Assignee: | drivers_other |
| Status: | NEW --- | ||
| Severity: | normal | ||
| Priority: | P1 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Kernel Version: | 3.19 | Subsystem: | |
| Regression: | No | Bisected commit-id: | |
| Attachments: | a copy of the driver for the latest kernel checkout | ||
Created attachment 166961 [details] a copy of the driver for the latest kernel checkout The bmg160_setup_any_motion_interrupt() function seems to contain copy-pasted code that enables/disables the data interrupt, which this function is not supposed to touch. Specifically, after correctly enabling the any motion interrupt, it enables, in line 346, the data interrupt. Likewise, in line 351 it disables the data interrupt instead of the any motion interrupt. Disclaimer: This defect was found via code inspection. I have not tried to reproduce the bug and in fact I don't own the bmg160 gyro hardware that this driver is for.