On 12/12/2016 04:40 PM, Cédric Le Goater wrote: > This driver adds mtd support for the Aspeed AST2500 SoC static memory > controllers : [...] > +#define DEVICE_NAME "aspeed-smc" > + > +/* > + * The driver only support SPI flash > + */ > +enum aspeed_smc_flash_type { > + smc_type_nor = 0, > + smc_type_nand = 1, > + smc_type_spi = 2, > +}; So why is this here ? :) > +struct aspeed_smc_chip; > + > +struct aspeed_smc_info { > + u32 maxsize; /* maximum size of chip window */ > + u8 nce; /* number of chip enables */ > + bool hastype; /* flash type field exists in config reg */ > + u8 we0; /* shift for write enable bit for CE0 */ > + u8 ctl0; /* offset in regs of ctl for CE0 */ > + > + void (*set_4b)(struct aspeed_smc_chip *chip); > +}; Otherwise looks good: Reviewed-by: Marek Vasut <marek.vasut@xxxxxxxxx> -- Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html