Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751350AbeAELK0 (ORCPT <rfc822;ralf@linux-mips.org> + 1 other);
        Fri, 5 Jan 2018 06:10:26 -0500
Received: from mail-io0-f193.google.com ([209.85.223.193]:41009 "EHLO
        mail-io0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1750837AbeAELKZ (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 5 Jan 2018 06:10:25 -0500
X-Google-Smtp-Source: ACJfBova0cx3S5hanEvZgPbefqmpL/DD81DumQUX8YF6GcCJ1HBW47dx96abQKNEBbdlzSatbzzHUxNHlEs7/9s8CGw=
MIME-Version: 1.0
In-Reply-To: <CAOfkYf7wexn7tMPX-dmqKAqOvug+CKA3z0m+JG+8nZHTj-fhKA@mail.gmail.com>
References: <1515148327-10384-1-git-send-email-pundirsumit11@gmail.com> <CAOfkYf7wexn7tMPX-dmqKAqOvug+CKA3z0m+JG+8nZHTj-fhKA@mail.gmail.com>
From: Sumit Pundir <pundirsumit11@gmail.com>
Date: Fri, 5 Jan 2018 16:40:23 +0530
Message-ID: <CAKunu93NKCLC180+Q2ZyL9NyVvBiMV9GDJ48PZO4O4d43zPC9Q@mail.gmail.com>
Subject: Re: [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned'
To: Shyam Saini <mayhs11saini@gmail.com>
Cc: samuel@sortiz.org, Greg KH <gregkh@linuxfoundation.org>,
        netdev@vger.kernel.org, devel@driverdev.osuosl.org,
        linux-kernel <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Return-Path: <linux-kernel-owner@vger.kernel.org>

On Fri, Jan 5, 2018 at 4:14 PM, Shyam Saini <mayhs11saini@gmail.com> wrote:
> Hi,
>
> Did you read this?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/staging/irda/TODO
>
>
> Thanks,
> Shyam

Hi Shyam,

Thanks for the update. Totally forgot to read it.

Thanks,
Sumit

On Fri, Jan 5, 2018 at 4:14 PM, Shyam Saini <mayhs11saini@gmail.com> wrote:
> Hi,
>
> Did you read this?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/staging/irda/TODO
>
>
> Thanks,
> Shyam
>
>
>> This patch fixes the following checkpatch.pl issue at multiple lines:
>>
>> WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
>>
>> Signed-off-by: Sumit Pundir <pundirsumit11@gmail.com>
>> ---
>>  drivers/staging/irda/drivers/act200l-sir.c | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/staging/irda/drivers/act200l-sir.c b/drivers/staging/irda/drivers/act200l-sir.c
>> index e891751..62a02e5 100644
>> --- a/drivers/staging/irda/drivers/act200l-sir.c
>> +++ b/drivers/staging/irda/drivers/act200l-sir.c
>> @@ -29,7 +29,7 @@
>>  static int act200l_reset(struct sir_dev *dev);
>>  static int act200l_open(struct sir_dev *dev);
>>  static int act200l_close(struct sir_dev *dev);
>> -static int act200l_change_speed(struct sir_dev *dev, unsigned speed);
>> +static int act200l_change_speed(struct sir_dev *dev, unsigned int speed);
>>
>>  /* Regsiter 0: Control register #1 */
>>  #define ACT200L_REG0    0x00
>> @@ -134,7 +134,7 @@ static int act200l_close(struct sir_dev *dev)
>>   *    Set the speed for the ACTiSYS ACT-IR200L type dongle.
>>   *
>>   */
>> -static int act200l_change_speed(struct sir_dev *dev, unsigned speed)
>> +static int act200l_change_speed(struct sir_dev *dev, unsigned int speed)
>>  {
>>         u8 control[3];
>>         int ret = 0;
>> @@ -191,8 +191,8 @@ static int act200l_change_speed(struct sir_dev *dev, unsigned speed)
>>
>>  static int act200l_reset(struct sir_dev *dev)
>>  {
>> -       unsigned state = dev->fsm.substate;
>> -       unsigned delay = 0;
>> +       unsigned int state = dev->fsm.substate;
>> +       unsigned int delay = 0;
>>         static const u8 control[9] = {
>>                 ACT200L_REG15,
>>                 ACT200L_REG13 | ACT200L_SHDW,
>> --
>> 2.7.4
>>