On Thu, Jan 09, 2014 at 01:36:15AM -0800, Insop Song wrote: > --- /dev/null > +++ b/drivers/staging/gs_fpgaboot/TODO > @@ -0,0 +1,14 @@ > +TODO: > + - get bus width input instead of hardcoded bus width > + > + - make it easier to config different programming method for > + other embedded targets, such as, different combination of gpio > + or serial programming
Odd formatting. And why is this last one a requirement to get this out of the staging directory? > +DONE: > + - run checkpatch > + - build tested No need to have this. > + > +Please send any patches for this driver to Insop Song<insop.s...@gmail.com> That's not the email address you sent this patch from. > +and Greg Kroah-Hartman <gre...@linuxfoundation.org>. And please CC linux-usb > +<linux-...@vger.kernel.org> too. What does the linux-usb mailing list have to do with this driver? > /* G100 specific bit swap and remmap (to gpio pins) for byte 0 */ > @@ -225,7 +217,7 @@ static inline void byte1_out(unsigned char data) > /* > * TODO: > * - configurable per device type for different I/O config > - * so that this can be easily extended to G200 and more > + * so that this can be easily extended other target devices Don't add incorrect lines if at all possible, others will just have to go back and fix them up later. Oh, and please merge with the first patch, as asked for before. thanks, greg k-h _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
