On 30 April 2015 at 20:43, Marek Vasut <marex@xxxxxxx> wrote: > On Thursday, April 30, 2015 at 03:33:47 PM, Michal Suchanek wrote: >> The size of written data was added to user supplied value rather than >> written at the provided address. > > You might want to work on the commit message a little, something like > the following, but feel free to reword as seen fit. > > The 'retlen' points to a variable representing the number of data bytes > written/read (see include/linux/mtd/mtd.h) by the current invocation of > the function. This variable must be set, not incremented. > > Otherwise, the patch is correct I believe: > > Acked-by: Marek Vasut <marex@xxxxxxx> > ok, I will send an updated version. Thanks Michal -- 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