On Thu, Jan 28, 2021 at 3:21 AM bingjingc <[email protected]> wrote: > > From: BingJing Chang <[email protected]> > > Will be used by fs parsing options > > Reviewed-by: Robbie Ko<[email protected]> > Reviewed-by: Chung-Chiang Cheng <[email protected]> > Signed-off-by: BingJing Chang <[email protected]> > --- > fs/isofs/inode.c | 16 ++-------------- > fs/udf/super.c | 16 ++-------------- > include/linux/parser.h | 1 + > lib/parser.c | 22 ++++++++++++++++++++++ > 4 files changed, 27 insertions(+), 28 deletions(-) > > diff --git a/fs/isofs/inode.c b/fs/isofs/inode.c > index 342ac19..21edc42 100644 > --- a/fs/isofs/inode.c > +++ b/fs/isofs/inode.c > @@ -335,18 +335,6 @@ static const match_table_t tokens = { > {Opt_err, NULL} > }; > > -static int isofs_match_uint(substring_t *s, unsigned int *res) > -{ > - int err = -ENOMEM; > - char *buf = match_strdup(s); > - > - if (buf) { > - err = kstrtouint(buf, 10, res); > - kfree(buf); > - } > - return err; > -}
I don't see how adding this function and removing it in the same series makes any sense. Why not make this the first patch in the series, simplifying everything? And while at it the referenced fuse implementation can also be converted (as a separate patch). Thanks, Miklos
