diff mbox

[1/1] asus-wireless: Fix identation

Message ID 20170126142554.8812-1-jprvita@endlessm.com (mailing list archive)
State Superseded, archived
Delegated to: Andy Shevchenko
Headers show

Commit Message

João Paulo Rechi Vita Jan. 26, 2017, 2:25 p.m. UTC
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
 drivers/platform/x86/asus-wireless.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andy Shevchenko Jan. 27, 2017, 3:28 p.m. UTC | #1
On Thu, Jan 26, 2017 at 4:25 PM, João Paulo Rechi Vita
<jprvita@gmail.com> wrote:
> Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
> ---
>  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.

>  {
>         struct asus_wireless_data *data;
João Paulo Rechi Vita Feb. 1, 2017, 12:24 p.m. UTC | #2
On 27 January 2017 at 10:28, Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
> On Thu, Jan 26, 2017 at 4:25 PM, João Paulo Rechi Vita
> <jprvita@gmail.com> wrote:
>> Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
>> ---
>>  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
diff mbox

Patch

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)
 {
 	struct asus_wireless_data *data;