Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751390AbeAEKoJ (ORCPT <rfc822;ralf@linux-mips.org> + 1 other);
        Fri, 5 Jan 2018 05:44:09 -0500
Received: from mail-oi0-f65.google.com ([209.85.218.65]:35562 "EHLO
        mail-oi0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751008AbeAEKoH (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 5 Jan 2018 05:44:07 -0500
X-Google-Smtp-Source: ACJfBotAScub53VkqK9kKcr8SyMLLc2Ty83Ki6VuBAELoa5XTbE3C6REC9koPC+yp5khsj6ygwjffcOvnV7eSga9nro=
MIME-Version: 1.0
In-Reply-To: <1515148327-10384-1-git-send-email-pundirsumit11@gmail.com>
References: <1515148327-10384-1-git-send-email-pundirsumit11@gmail.com>
From: Shyam Saini <mayhs11saini@gmail.com>
Date: Fri, 5 Jan 2018 16:14:05 +0530
Message-ID: <CAOfkYf7wexn7tMPX-dmqKAqOvug+CKA3z0m+JG+8nZHTj-fhKA@mail.gmail.com>
Subject: Re: [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned'
To: Sumit Pundir <pundirsumit11@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>

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
>