On Wed, 2011-03-16 at 16:24 -0400, Jarod Wilson wrote:
> Signed-off-by: Jarod Wilson <ja...@redhat.com>
> ---
>  drivers/staging/lirc/lirc_zilog.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/staging/lirc/lirc_zilog.c 
> b/drivers/staging/lirc/lirc_zilog.c
> index 407d4b4..5ada643 100644
> --- a/drivers/staging/lirc/lirc_zilog.c
> +++ b/drivers/staging/lirc/lirc_zilog.c
> @@ -950,6 +950,10 @@ static ssize_t read(struct file *filep, char *outbuf, 
> size_t n, loff_t *ppos)
>                               ret = copy_to_user((void *)outbuf+written, buf,
>                                                  rbuf->chunk_size);
>                               written += rbuf->chunk_size;
> +                     } else {
> +                             zilog_error("Buffer read failed!\n");
> +                             ret = -EIO;
> +                             break;

No need to break, just let the non-0 ret value drop you out of the while
loop.

Regards,
Andy

>                       }
>               }
>       }


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to