On Tue, Jul 28, 2015 at 05:47:40PM +0900, Tony Cho wrote: > The TODO list is updated for the WILC1000 kernel driver. The TODO list > contains many items listed in time sequence. I don't understand what you mean by "in time sequence" > > Signed-off-by: Tony Cho <tony.cho@xxxxxxxxx> > Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > drivers/staging/wilc1000/TODO | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/drivers/staging/wilc1000/TODO b/drivers/staging/wilc1000/TODO > index 95199d8..6b745cd 100644 > --- a/drivers/staging/wilc1000/TODO > +++ b/drivers/staging/wilc1000/TODO > @@ -1,14 +1,15 @@ > TODO: > -- remove the defined feature as kernel versions > -- remove OS wrapper functions > -- remove custom debug and tracing functions > +- remove most of warnings reported by checkpatch.pl "most"? Why not all? > +- remove OS wrapper functions to use kernel API as is Why reword this, the original line is just fine. > - rework comments and function headers(also coding style) > - replace all semaphores with mutexes or completions > -- make spi and sdio components coexist in one build Why remove this item? It's not done and needs to be done. > +- remove compile warnings related to 64 bit machine That's a given, I'm not enabling the build of the driver until that happens. Please work on that next, coding style fixes can happen after that. As of right now, this driver is marked BROKEN. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
