On Sat, Jun 13, 2015 at 09:57:05AM -0700, Joe Perches wrote: > > It seems ready to me. > > As far as I can tell, there's just a few niggles > that could be done: > > Something like (too lazy to separate them into multiple patches) Thanks. I will break into patches. Call me lazy for not having it done till now. > > o reduce indentation a couple of places > o add newlines to logging messages > o add const to static array > o use consistent function declaration style > > It's unfortunate there are so many #ifdef __BIG_ENDIAN uses. instead of #ifdef __BIG_ENDIAN can i then use a bool flag to check by if-else? regards sudip _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
