diff options
author | axel lin <axel.lin@gmail.com> | 2011-01-14 09:39:11 +0000 |
---|---|---|
committer | Paul Mundt <lethal@linux-sh.org> | 2011-01-18 13:40:34 +0900 |
commit | 0b7f1cc79d61427961e311c6a21f528bdb226e40 (patch) | |
tree | 773b512ff69f55966af6ba0cc69bd82d85723b64 /drivers/video/pxa3xx-gcu.c | |
parent | 6c9571f4b759717e1c938cbc6b53ec8ce5813245 (diff) |
video: pxa3xx-gcu: Return -EFAULT when copy_from_user() fails
Return -EFAULT instead of number of bytes that could not be copied if
copy_from_user() fails.
Also fix a typo in the comments.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Diffstat (limited to 'drivers/video/pxa3xx-gcu.c')
-rw-r--r-- | drivers/video/pxa3xx-gcu.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/pxa3xx-gcu.c b/drivers/video/pxa3xx-gcu.c index b81168df253d..cf4beb9dc9bb 100644 --- a/drivers/video/pxa3xx-gcu.c +++ b/drivers/video/pxa3xx-gcu.c @@ -1,5 +1,5 @@ /* - * pxa3xx-gc.c - Linux kernel module for PXA3xx graphics controllers + * pxa3xx-gcu.c - Linux kernel module for PXA3xx graphics controllers * * This driver needs a DirectFB counterpart in user space, communication * is handled via mmap()ed memory areas and an ioctl. @@ -421,7 +421,7 @@ pxa3xx_gcu_misc_write(struct file *filp, const char *buff, buffer->next = priv->free; priv->free = buffer; spin_unlock_irqrestore(&priv->spinlock, flags); - return ret; + return -EFAULT; } buffer->length = words; |