Hi Andy, > > + int ret; > > + > > + ret = mtd_device_unregister(nand_to_mtd(chip)); > > + WARN_ON(ret); > > Is this a requirement by MTD to have return value being checked? Yes, for now this is the preferred way, with the hope some day to turn the return value into void. Thanks, Miquèl