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][next] leds: TI LMU: remove redundant u8 comparisons with less than zero
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] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Security] [Bugtraq] [Linux OMAP] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]

Powered by Linux