diff options
author | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-07-25 18:33:40 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-07-25 18:35:16 -0400 |
commit | 9c46c7facda384f8fb6078e70b4e84ff84ed949d (patch) | |
tree | f80bcd7c26cd87dc3d1484b3fbf4222a7011f2d0 | |
parent | 69cc4bfab9e65168f50a1f1d163c0d70a394bae3 (diff) |
[media] radio-miropcm20: fix a compilation warning
drivers/media/radio/radio-miropcm20.c: In function 'sanitize':
drivers/media/radio/radio-miropcm20.c:216:3: warning: comparison is always false due to limited range of data type [-Wtype-limits]
if (p[i] < 32 || p[i] >= 128) {
^
As p is declared as a char array, it is signed. So, it can never
be bigger than 127.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
-rw-r--r-- | drivers/media/radio/radio-miropcm20.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/radio/radio-miropcm20.c b/drivers/media/radio/radio-miropcm20.c index 72df00eee81f..ac9915d15f66 100644 --- a/drivers/media/radio/radio-miropcm20.c +++ b/drivers/media/radio/radio-miropcm20.c | |||
@@ -213,7 +213,7 @@ static bool sanitize(char *p, int size) | |||
213 | bool ret = true; | 213 | bool ret = true; |
214 | 214 | ||
215 | for (i = 0; i < size; i++) { | 215 | for (i = 0; i < size; i++) { |
216 | if (p[i] < 32 || p[i] >= 128) { | 216 | if (p[i] < 32) { |
217 | p[i] = ' '; | 217 | p[i] = ' '; |
218 | ret = false; | 218 | ret = false; |
219 | } | 219 | } |