lkml.org 
[lkml]   [2014]   [May]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] staging: rtl8723au: rtw_sta_mgt: change rtw_alloc_stainfo23a to use gfp_t instead if int
Date
Dan Carpenter <dan.carpenter@oracle.com> writes:
> On Sun, May 18, 2014 at 07:39:23AM +0100, Karim Raslan wrote:
>> This should fix a few sparse warnings like the following:
>>
>> CHECK drivers/staging/rtl8723au/core/rtw_ap.c
>> drivers/staging/rtl8723au/core/rtw_ap.c:1054:45: warning: incorrect
>> type in argument 3 (different base types)
>> drivers/staging/rtl8723au/core/rtw_ap.c:1054:45: expected int [signed] gfp
>> drivers/staging/rtl8723au/core/rtw_ap.c:1054:45: got restricted gfp_t
>>
>
> Thanks, Karim, looks good.
>
> Jes's patch which added this never went to the list so I didn't review
> it. This annoys me.
>
> I try to be fair when I review code. I try to allow lots of patches in
> and let people send fix up patches for minor stuff. But, obviously, you
> should be running Sparse as part of the review process so if I made you
> redo this patch it's because your process is broken. When I review
> patches I am trying to help you.

Dan,

Calm down, I haven't been bombarding the list, due to the sheer volume
of commits. If you want me to spam the list, tell me where and I'll add
that to the CC list next time.

This particular case isn't exactly a real problem either.

Jes


\
 
 \ /
  Last update: 2014-05-19 16:41    [W:1.743 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site