On 11/1/20 10:51 PM, kajoljain wrote: > > On 10/31/20 7:18 PM, trix@xxxxxxxxxx wrote: >> From: Tom Rix <trix@xxxxxxxxxx> >> >> A semicolon is not needed after a switch statement. > Hi Tom, > I was checking this patch. Not sure if it will come under as fix patch. Since this is not fixing > any logical issue do we still need to add fix tag? My rule of thumb is a fix means the kernel needs to be recompiled. This isn't a fix. Tom > > Thanks, > Kajol Jain >> Signed-off-by: Tom Rix <trix@xxxxxxxxxx> >> --- >> drivers/memstick/core/mspro_block.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/memstick/core/mspro_block.c b/drivers/memstick/core/mspro_block.c >> index cd6b8d4f2335..afb892e7ffc6 100644 >> --- a/drivers/memstick/core/mspro_block.c >> +++ b/drivers/memstick/core/mspro_block.c >> @@ -276,7 +276,7 @@ static const char *mspro_block_attr_name(unsigned char tag) >> return "attr_devinfo"; >> default: >> return NULL; >> - }; >> + } >> } >> >> typedef ssize_t (*sysfs_show_t)(struct device *dev, >>