diff options
| author | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-07-25 19:33:40 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-07-25 19:35:16 -0300 |
| commit | 9c46c7facda384f8fb6078e70b4e84ff84ed949d (patch) | |
| tree | f80bcd7c26cd87dc3d1484b3fbf4222a7011f2d0 /drivers/media/radio | |
| 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>
Diffstat (limited to 'drivers/media/radio')
| -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) bool ret = true; for (i = 0; i < size; i++) { - if (p[i] < 32 || p[i] >= 128) { + if (p[i] < 32) { p[i] = ' '; ret = false; } |