The simple_write_to_buffer() returns ssize_t type value, which is either positive or negative.
However 'res' is declared as size_t(unsigned int) which contains non-negative type. So 'res < 0' statement is always false, this cannot execute execptional-case handling. To prevent this case, update proper type 'size_t' to 'ssize_t' for execptional handling. Signed-off-by: Austin Kim <austindh....@gmail.com> --- drivers/bus/moxtet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/bus/moxtet.c b/drivers/bus/moxtet.c index 1ee4570..288a9e4 100644 --- a/drivers/bus/moxtet.c +++ b/drivers/bus/moxtet.c @@ -514,7 +514,7 @@ static ssize_t output_write(struct file *file, const char __user *buf, struct moxtet *moxtet = file->private_data; u8 bin[TURRIS_MOX_MAX_MODULES]; u8 hex[sizeof(bin) * 2 + 1]; - size_t res; + ssize_t res; loff_t dummy = 0; int err, i; -- 2.6.2