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

From: Dan Carpenter
Date: Wed May 22 2019 - 05:56:17 EST

Next message: Bharath Vedartham: "[PATCH] message/fusion/mptbase.c: Use kmemdup instead of memcpy and kmalloc"
Previous message: Philippe Mazenauer: "[PATCH] arm: arch_timer: add missing include <asm/arch_timer.h>"
In reply to: Colin King: "[PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero"
Next in thread: Colin Ian King: "Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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

regards,
dan carpenter

Next message: Bharath Vedartham: "[PATCH] message/fusion/mptbase.c: Use kmemdup instead of memcpy and kmalloc"
Previous message: Philippe Mazenauer: "[PATCH] arm: arch_timer: add missing include <asm/arch_timer.h>"
In reply to: Colin King: "[PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero"
Next in thread: Colin Ian King: "Re: [PATCH][next] leds: TI LMU: remove redundant u8 comparisons with less than zero"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]