On Fri, Nov 29, 2013 at 12:19:04PM +0000, Lee Jones wrote: > --- a/drivers/mtd/devices/st_spi_fsm.c > +++ b/drivers/mtd/devices/st_spi_fsm.c > @@ -346,9 +348,48 @@ static void stfsm_fetch_platform_configs(struct > platform_device *pdev) ... > + /* Use device size to determine address width */
Since you're splitting this patch series up so much, shouldn't this comment go in when you add the two lines below? > if (info->sector_size * info->n_sectors > 0xFFFFFF) > info->flags |= FLASH_FLAG_32BIT_ADDR; ... BTW, even though it doesn't exactly make for the best (runtime) bisectability (since your driver is not functional yet), this approach to splitting your driver introduction into logical chunks does make it a little easier to review. So thanks. Brian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
