[PATCH2] staging: brcm80211: fix suspend/resume issue in brcmsmac

Arend van Spriel arend at broadcom.com
Sat Jan 22 14:42:24 UTC 2011


Thanks, Greg

On 01/22/2011 03:31 PM, Greg KH wrote:
> On Fri, Jan 21, 2011 at 01:36:44PM +0100, Arend van Spriel wrote:
>> @@ -246,7 +243,14 @@ wl_ops_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
>>   static void
>>   wl_ops_remove_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
>>   {
>> -	return;
>> +	struct wl_info *wl;
>> +
>> +	wl = HW_TO_WL(hw);
>> +	
>
> You added trailing whitespace :(
>
> You should fix your editor to flag this with a big red line or something
> to make sure you don't do it again.  I've fixed it up for now, but
> please be more careful in the future.
>

Will fix it in upcoming patches and run everything through checkpatch.pl.

> thanks,
>
> greg k-h
>

Changed email client to stop my "top postings".

-- 
Gr. AvS




More information about the devel mailing list