Hi Sven,
Thanks for the patch.
On Wednesday 27 July 2011 11:47:44 Sven Eckelmann wrote:
> atomic_dec_not_zero is defined for each architecture through
> <linux/atomic.h> to provide the functionality of
> atomic_add_unless(x, -1, 0).
>
> Signed-off-by: Sven Eckelmann <sven@xxxxxxxxxxxxx>
> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
I'll queue this to my tree for v3.2. Please let me know if you would rather
push the patch through another tree.
> Cc: linux-media@xxxxxxxxxxxxxxx
> ---
> drivers/media/video/omap3isp/ispstat.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/media/video/omap3isp/ispstat.c
> b/drivers/media/video/omap3isp/ispstat.c index b44cb68..81b1ec9 100644
> --- a/drivers/media/video/omap3isp/ispstat.c
> +++ b/drivers/media/video/omap3isp/ispstat.c
> @@ -652,7 +652,7 @@ static int isp_stat_buf_process(struct ispstat *stat,
> int buf_state) {
> int ret = STAT_NO_BUF;
>
> - if (!atomic_add_unless(&stat->buf_err, -1, 0) &&
> + if (!atomic_dec_not_zero(&stat->buf_err) &&
> buf_state == STAT_BUF_DONE && stat->state == ISPSTAT_ENABLED) {
> ret = isp_stat_buf_queue(stat);
> isp_stat_buf_next(stat);
--
Regards,
Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
