Hi Huijin, Huijin Park <[email protected]> wrote on Thu, 15 Nov 2018 00:07:10 -0500:
> From: "huijin.park" <[email protected]> > > This patch casts the "len" parameter to an unsigned int. > The callers of erase_write() pass the "len" parameter as unsigned int. Indeed. Perhaps it is worth backporting this patch to a stable releases? > > Signed-off-by: huijin.park <[email protected]> > --- > drivers/mtd/mtdblock.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/mtdblock.c b/drivers/mtd/mtdblock.c > index a5b1933..b2d5ed1 100644 > --- a/drivers/mtd/mtdblock.c > +++ b/drivers/mtd/mtdblock.c > @@ -56,7 +56,7 @@ struct mtdblk_dev { > */ > > static int erase_write (struct mtd_info *mtd, unsigned long pos, > - int len, const char *buf) > + unsigned int len, const char *buf) > { > struct erase_info erase; > size_t retlen; Reviewed-by: Miquel Raynal <[email protected]> Thanks, Miquèl
