On Thu, Aug 1, 2013 at 8:55 PM, Aaron Lu <aaron.lwe@xxxxxxxxx> wrote: > On 08/02/2013 07:44 AM, Felipe Contreras wrote: >> Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx> > > Change log please. You mean a commit message? That's what it's called in Git lingo, and it's right there: acpi: video: trivial style cleanups >> --- >> drivers/acpi/video.c | 90 +++++++++++++++++++++++----------------------------- >> 1 file changed, 40 insertions(+), 50 deletions(-) >> >> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c >> index f021bf4..b07573f 100644 >> --- a/drivers/acpi/video.c >> +++ b/drivers/acpi/video.c >> @@ -88,7 +88,7 @@ module_param(allow_duplicates, bool, 0644); >> static bool use_bios_initial_backlight = 1; >> module_param(use_bios_initial_backlight, bool, 0644); >> >> -static int register_count = 0; >> +static int register_count; > > This isn't a style clean up. It is. There's no difference before and after, the only difference is the style. ERROR: do not initialise statics to 0 or NULL #91: FILE: acpi/video.c:91: +static int register_count = 0; -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
