| From | Vladimir Kondratiev <> | Subject | Re: [PATCH] wil6210: Fix switch operator "missing break?" warning | Date | Mon, 10 Feb 2014 12:54:13 +0200 |
| |
On Friday, February 07, 2014 03:46:40 PM Alexander Gordeev wrote: > This update fixes a warning introduced with commit bc977ba1 > ("wil6210: Use pci_enable_msi_range() instead of pci_enable_msi_block()") > > drivers/net/wireless/ath/wil6210/pcie_bus.c:65 wil_if_pcie_enable() > warn: missing break? reassigning 'use_msi' >
I can't reproduce this warning. What tools used to get it? Neither gcc (I have 4.8.1) nor sparse report it.
Anyway, I am fine with both 'switch' and 'if'.
Thanks, Vladimir.
|