On Thu, 15 Nov 2018 00:07:10 -0500 Huijin Park <huijin.park@xxxxxxxxxxx> wrote: > From: "huijin.park" <huijin.park@xxxxxxxxxxx> > > This patch casts the "len" parameter to an unsigned int. Hm, that's not really what this patch actually. Actually it avoids the cast from an int to an unsigned int. > The callers of erase_write() pass the "len" parameter as unsigned int. That's the important part here. Callers of erase_write() always pass an unsigned int, so what you're trying to do here is avoid a cast to an int. > > Signed-off-by: huijin.park <huijin.park@xxxxxxxxxxx> > --- You should have a changelog here. > 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; ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/