Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Colin King <colin.king@xxxxxxxxxxxxx>
Subject: Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Wed, 22 May 2019 12:53:17 +0300
Cc: Dan Murphy <dmurphy@xxxxxx>, Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx>, Pavel Machek <pavel@xxxxxx>, linux-leds@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20190522082627.18354-1-colin.king@canonical.com>
References: <20190522082627.18354-1-colin.king@canonical.com>
User-agent: Mutt/1.9.4 (2018-02-28)

No.  This isn't the right fix.  We should declare them as int instead.

regards,
dan carpenter

Follow-Ups:
- Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
  - From: Colin Ian King

References:
- [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
  - From: Colin King

Prev by Date: [PATCH] mISDN: make sure device name is NUL terminated
Next by Date: Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
Previous by thread: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
Next by thread: Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]

Powered by Linux