[REVIEW REQUEST] staging: unisys: review request for visorbus
gregkh at linuxfoundation.org
gregkh at linuxfoundation.org
Mon Oct 2 15:49:52 UTC 2017
On Mon, Oct 02, 2017 at 03:41:42PM +0000, Kershner, David A wrote:
> Hey Greg, we think the code for visorbus is ready to be moved out
> of staging, can you review it to see if we have missed anything?
I think your html email got rejected by the list :(
> The files include:
> /drivers/staging/unisys/visorbus/
> /drivers/staging/unisys/include/visorchannel.h
> /drivers/staging/unisys/include/visorbus.h
>
> The directories staging/drivers/unisys/visornic, visorhba, and visorinput
> are not part of the review as well as the file
> drivers/staging/unisys/include/iochannel.h.
>
> We currently have 5 checkpatch.pl warnings that I know about:
> - 3 CHECKs in visorchannel that are due to a MACRO that gets passed a FIELD
> instead of just a variable
> - 2 WARNINGS dealing with char * becoming static const
>
>
>
> Dan Carpenter found some extra parenthesis errors that I will address as
> well as look through the code to fix, but I would like to ask for the review
> while we are working on that.
Sure, I'll look at doing it in a week or so when I catch up with other
patches in my queue.
Everyone else is also welcome to do review :)
thanks,
greg k-h
More information about the devel
mailing list