On Fri, Oct 25, 2019 at 12:32:25PM +0000, Tudor.Ambarus@xxxxxxxxxxxxx wrote: > From: Tudor Ambarus <tudor.ambarus@xxxxxxxxxxxxx> > > When the controller is not under the SPI-MEM interface it may implement > the optional controller_ops->erase() method. > > nor->spimem and nor->controller_ops are disjunctive. Move the > nor->controller_ops->erase != NULL check as an 'else if' case to > nor->spimem, in order to avoid the nor->controller_ops != NULL > check. > > Signed-off-by: Tudor Ambarus <tudor.ambarus@xxxxxxxxxxxxx> > --- > Intended for better readability. Thanks! regards, dan carpenter ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/