On Wed, 2007-07-18 at 23:31 -0500, Steven J. Hill wrote: > + /* check if lock/unlock functions are supported * Unterminated comment? > + if (!chip->lock) > + mtd->lock = NULL; > + else > + mtd->lock = chip->lock; Entirely needless conditional. -- dwmw2