On 08/06/16 13:25, Parth Sane wrote: > No mate, that aint just cutting it. > I tried putting it, maybe i’m doing something wrong? > Regards, > Parth >> On 08-Jun-2016, at 5:46 PM, Luis de Bethencourt <lui...@osg.samsung.com> >> wrote: >> >> On 08/06/16 12:58, Parth Sane wrote: >>> I’m trying to fix line 224 but its simply not happening. Can you help me >>> with this? >>> Regards, >>> Parth >>>> On 08-Jun-2016, at 5:19 PM, Luis de Bethencourt <lui...@osg.samsung.com> >>>> wrote: >>>> >>>> On 08/06/16 12:37, Parth Sane wrote: >>>>> >>>>>> On 08-Jun-2016, at 5:01 PM, Luis de Bethencourt <lui...@osg.samsung.com> >>>>>> wrote: >>>>>> >>>>>> On 08/06/16 12:27, Parth Sane wrote: >>>>>>> Multiple parenthesis alignment warnings were thrown by checkpatch in >>>>>>> ieee80211.c >>>>>>> This patch effectively fixes that. >>>>>>> >>>>>>> Signed-off-by: Parth Sane <laerdevstud...@gmail.com> >>>>>>> >>>>>>> --- >>>>>>> drivers/staging/rtl8712/ieee80211.c | 25 ++++++++++++------------- >>>>>>> 1 file changed, 12 insertions(+), 13 deletions(-) >>>>>> >>>>>> Some of the lines you change are over 80 characters. You could fix those >>>>>> as well in >>>>>> a second patch :) >>>>>> >>>>>> Thanks, >>>>>> Luis >>>>>> >>>>> >>>>> You’re right on that regard. Maybe once this is accepted I can come back >>>>> to other warnings. I will submit other patches meanwhile. >>>>> I seriously want to see this driver in the next release! >>>>> Regards, >>>>> Parth >>>>> >>>> >>>> You can resend this patch and the one fixing the 80 char limit with the >>>> subject >>>> prefix [RESEND PATCH v2 x/2]. >>>> >>>> AFAIK this driver is already in the release, check menuconfig under >>>> "Device Drivers > Staging drivers > Realtek RTL8172" >>>> >>>> Thanks, >>>> Luis >>> >> >> Sure. >> >> You want something like this: >> - if (memcmp((pbuf + 2), wpa_oui_type, >> sizeof(wpa_oui_type))) >> + if (memcmp((pbuf + 2), wpa_oui_type, >> + sizeof(wpa_oui_type))) >> >> You could check for references of how other parts of the code have solved it >> by doing: >> git grep memcpy * | grep -v ";" >> >> I hope that helps. >> Luis >> >> >> >> Reviewed-by: Luis de Bethencourt <lui...@osg.samsung.com> >
What is the output you get when it doesn't work. Without that it is hard for me to be of help. Luis _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
