Re: [PATCH] usb: core: sysfs: constify attribute_group

From: Greg KH
Date: Sat Jun 24 2017 - 03:56:27 EST


On Fri, Jun 23, 2017 at 11:28:10AM +0530, Arvind Yadav wrote:
> File size before:
> text data bss dec hex filename
> 6798 2848 4 9650 25b2 drivers/usb/core/sysfs.o
>
> File size After adding 'const':
> text data bss dec hex filename
> 7230 2400 4 9634 25a2 drivers/usb/core/sysfs.o
>
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@xxxxxxxxx>

You have sent a lot of these types of patches out, and really, this is a
horrible changelog entry. You are assuming that I know what this is,
and why it is a good thing. Now I might, but maybe not...

Please actually explain why you are making this change. Why it is
needed. And big hint, it's not only about the output of the file
size...

Please fix this up for all of the patches you have sent me in this
format, as I'm deleting them all from my queue right now.

thanks,

greg k-h