On 09/22/2014 03:51 PM, Ezequiel Garcia wrote: > > > On 09/22/2014 09:22 AM, Roger Quadros wrote: >> Hi Ezequiel, >> >> On 09/20/2014 07:53 PM, Ezequiel Garcia wrote: >>> I've collected all the changes recently discussed in a whole patchset. >>> If you are just reading this for the first time please take a look at: >>> >>> [1] http://www.spinics.net/lists/linux-omap/msg110965.html >>> [2] http://www.spinics.net/lists/linux-omap/msg110958.html >>> >>> As requested by Roger, the omap_elm is a 'bool' driver now. The plan >>> is to merge it inside omap2_nand, but my queue is really filled, so >>> if anyone feels like doing, please be my guest. >>> >>> Although I'm hoping this to be the last round, feedback and testing >>> is welcome. >> >> Patches 3 and 4 still have checkpatch issues. >> > > I think you didn't look at them in detail. Unless I'm missing something, > it's just warning about MAINTAINERS update: > > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? > #16: > drivers/mtd/{devices/elm.c => nand/omap_elm.c} | 0 > > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? > #13: > drivers/mtd/nand/{omap2.c => omap2_nand.c} | 0 > > This driver is not in MAINTAINERS. If you want to start maintaining it, > it'd be good to add you to MAINTAINERS so people Cc you properly on patches. > Sorry for the noise. I was re-generating the patch without the rename detection flag causing errors which are present in the original file itself. cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html