On Saturday 23 June 2007, Atsushi Nemoto wrote: > Thank you for excellent review! I'll look at each comments surely > will update the driver but it may take a few days.
That's fine. > For now, I'm quite sure your patch is OK for me except for one thing: > > > + * spi_tx99.c - TXx9 SPI controller driver. > > Name it spi_txx9.c, please ;) Sorry, typo! ... please fix when you resubmit. > And for mmiowb() issue, I put it just only I was not sure whether > gpio_set_value() guarantee I/O barrier. Now I see i2c-gpio.c, etc. do > not have such barriers. I will remove these barriers and fix platform > gpio codes. I don't think this is a case where there'd be a benefit to allowing non-barriered implementations, and thus requiring all portable code to include platform-neutral I/O barriers. I don't know that such neutral primitives actually exist... I'll update the GPIO docs to make that clear, unless you have some strong argument to the contrary. - Dave ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ spi-devel-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/spi-devel-general
