Skip to content

Commit

Permalink
[media] s5p-g2d: Fix NULL pointer warnings in g2d.c file
Browse files Browse the repository at this point in the history
Fixes the following warnings detected by sparse:
warning: Using plain integer as NULL pointer

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Acked-by: Kamil Debski <k.debski@samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Sachin Kamat authored and Mauro Carvalho Chehab committed May 20, 2012
1 parent a13bba4 commit 2356877
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions drivers/media/video/s5p-g2d/g2d.c
Original file line number Diff line number Diff line change
Expand Up @@ -546,11 +546,11 @@ static void job_abort(void *prv)
struct g2d_dev *dev = ctx->dev;
int ret;

if (dev->curr == 0) /* No job currently running */
if (dev->curr == NULL) /* No job currently running */
return;

ret = wait_event_timeout(dev->irq_queue,
dev->curr == 0,
dev->curr == NULL,
msecs_to_jiffies(G2D_TIMEOUT));
}

Expand Down Expand Up @@ -599,19 +599,19 @@ static irqreturn_t g2d_isr(int irq, void *prv)
g2d_clear_int(dev);
clk_disable(dev->gate);

BUG_ON(ctx == 0);
BUG_ON(ctx == NULL);

src = v4l2_m2m_src_buf_remove(ctx->m2m_ctx);
dst = v4l2_m2m_dst_buf_remove(ctx->m2m_ctx);

BUG_ON(src == 0);
BUG_ON(dst == 0);
BUG_ON(src == NULL);
BUG_ON(dst == NULL);

v4l2_m2m_buf_done(src, VB2_BUF_STATE_DONE);
v4l2_m2m_buf_done(dst, VB2_BUF_STATE_DONE);
v4l2_m2m_job_finish(dev->m2m_dev, ctx->m2m_ctx);

dev->curr = 0;
dev->curr = NULL;
wake_up(&dev->irq_queue);
return IRQ_HANDLED;
}
Expand Down

0 comments on commit 2356877

Please sign in to comment.