Messages in this thread |  | | From | Rafał Miłecki <> | Date | Tue, 1 Nov 2016 08:31:58 +0100 | Subject | Re: [PATCH v2] rtl8xxxu: Fix for agressive power saving by rtl8723bu wireless IC |
| |
On 1 November 2016 at 08:24, John Heenan <john@zgus.com> wrote: > @@ -5779,6 +5779,12 @@ static int rtl8xxxu_start(struct ieee80211_hw *hw) > > ret = 0; > > + if(priv->fops == &rtl8723bu_fops) {
OK, let me be the first. Documentation/CodingStyle also says to use space between "if" and "(" ;)
> @@ -6080,9 +6086,11 @@ static int rtl8xxxu_probe(struct usb_interface *interface, > goto exit; > } > > - ret = rtl8xxxu_init_device(hw); > - if (ret) > - goto exit; > + if(priv->fops != &rtl8723bu_fops) {
Same here.
I reviewed style only.
-- Rafał
|  |