On Fri, 2007-07-13 at 11:48 +0800, WANG Cong wrote: > if(map_banks[idx] == NULL) { > ret = -ENOMEM; > - /* FIXME: What if some MTD devices were probed already? */ > + /* FIXME: What if some MTD devices were probed > + already? */ > goto error_mem; > } That's just horrid. Leave it as it was. -- dwmw2