Hello Marek, On 05/24/2016 09:31 AM, Marek Szyprowski wrote: > This patch removes custom code for initialization and handling of > reserved memory regions in s5p-mfc driver and replaces it with generic > reserved memory regions api. > > s5p-mfc driver now handles two reserved memory regions defined by > generic reserved memory bindings. Support for non-dt platform has been > removed, because all supported platforms have been already converted to > device tree. > > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > --- Similar question than for patch #2, shouldn't we include the in-kernel DTS changes with this patch to keep bisectability? Otherwise the mfc-{l,r} mem allocation will fail after this patch. The patch looks good to me though: Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> I couldn't test this exact patch because my Odroid XU4 died but I've tested the previous version and the only difference is that the memory reserve is made by index now instead of name, so I think you can add: Tested-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- 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