[PATCH] Fix erase timeout in M25P80 driver

Mike Frysinger vapier.adi at gmail.com
Sat Apr 4 14:47:30 EDT 2009


On Sat, Apr 4, 2009 at 10:38, Peter Horton wrote:
> +       for (deadline = jiffies + MAX_READY_WAIT_JIFFIES; !time_after_eq(jiffies, deadline);) {

this would make more sense as a while loop:
  deadline = jiffies + MAX_READY_WAIT_JIFFIES;
  while (!time_after_eq(jiffies, deadline)) {
maybe even more sense as a do { ... } while (...) ...

a for loop just looks weird ... but it should do what you want, so whatever
-mike



More information about the linux-mtd mailing list