count is unsigned and cannot be less than 0.

Signed-off-by: Roel Kluin <[email protected]>
---
Can these be removed or do we need an `if (count > MAX)' test, and
what should MAX be then?

diff --git a/drivers/misc/hdpuftrs/hdpu_cpustate.c 
b/drivers/misc/hdpuftrs/hdpu_cpustate.c
index 176fe4e..35000cf 100644
--- a/drivers/misc/hdpuftrs/hdpu_cpustate.c
+++ b/drivers/misc/hdpuftrs/hdpu_cpustate.c
@@ -121,8 +121,6 @@ static ssize_t cpustate_read(struct file *file, char *buf,
 {
        unsigned char data;
 
-       if (count < 0)
-               return -EFAULT;
        if (count == 0)
                return 0;
 
@@ -137,9 +135,6 @@ static ssize_t cpustate_write(struct file *file, const char 
*buf,
 {
        unsigned char data;
 
-       if (count < 0)
-               return -EFAULT;
-
        if (count == 0)
                return 0;
 
_______________________________________________
Linuxppc-dev mailing list
[email protected]
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to