On 11/2/20 7:12 PM, Tom Rix wrote: > > 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. Yes make sense. Thanks for confirming. Thanks, Kajol Jain > > 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, >>> >
