On Wed, Sep 09, 2015 at 04:08:12PM +0900, Tony Cho wrote: > From: Glen Lee <glen.lee@xxxxxxxxx> > > wilc1000 driver runs on Linux. No need to have oswrapper or platform > dependent files. > Before delete two header files, move two structures in wilc_platform.h to > wilc_msgqueue.h where it is used. Include linux header files where is needs > since wilc_platform is deleted. > Finally, remove wilc_oswrapper.h and wilc_platform.h. That's a lot of different things to be doing all in one patch. Please break this up into individual patches. And you can't just delete the oswrapper.h file, you still have things in it, don't move it for no reason, get rid of those wrapper functions properly first, making the file empty, and then you can delete it. Moving them to a different .h file just hides the fact that this work still needs to be done. 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
