On 27 January 2017 at 10:28, Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > On Thu, Jan 26, 2017 at 4:25 PM, João Paulo Rechi Vita > <jprvita@xxxxxxxxx> wrote: >> Signed-off-by: João Paulo Rechi Vita <jprvita@xxxxxxxxxxxx> >> --- >> drivers/platform/x86/asus-wireless.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/platform/x86/asus-wireless.c b/drivers/platform/x86/asus-wireless.c >> index 9f31bc1a47d0..c9b5ac152cf1 100644 >> --- a/drivers/platform/x86/asus-wireless.c >> +++ b/drivers/platform/x86/asus-wireless.c >> @@ -77,7 +77,7 @@ static void led_state_update(struct work_struct *work) >> } >> >> static void led_state_set(struct led_classdev *led, >> - enum led_brightness value) >> + enum led_brightness value) > > I would see benefit if it would be one line. > Yes, I fixed this in a hurry and missed it actually fits in one line. Thanks for catching it! I'll reply with an updated patch. -- João Paulo Rechi Vita http://about.me/jprvita