]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commitdiff
media: s5p-mfc: Adjust a null pointer check in four functions
authorMarkus Elfring <elfring@users.sourceforge.net>
Fri, 8 Sep 2017 20:37:00 +0000 (13:37 -0700)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 16 Oct 2017 23:51:11 +0000 (16:51 -0700)
The script “checkpatch.pl” pointed information out like the following.

Comparison to NULL could be written...

Thus fix the affected source code places.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/platform/s5p-mfc/s5p_mfc.c

index abfb70b07032452159d72d8232eefac8f4750730..cf68aed59e0d40458732f5a4bdbea5f50b7e7ba7 100644 (file)
@@ -470,7 +470,7 @@ static void s5p_mfc_handle_error(struct s5p_mfc_dev *dev,
 {
        mfc_err("Interrupt Error: %08x\n", err);
 
-       if (ctx != NULL) {
+       if (ctx) {
                /* Error recovery is dependent on the state of context */
                switch (ctx->state) {
                case MFCINST_RES_CHANGE_INIT:
@@ -508,7 +508,7 @@ static void s5p_mfc_handle_seq_done(struct s5p_mfc_ctx *ctx,
 {
        struct s5p_mfc_dev *dev;
 
-       if (ctx == NULL)
+       if (!ctx)
                return;
        dev = ctx->dev;
        if (ctx->c_ops->post_seq_start) {
@@ -562,7 +562,7 @@ static void s5p_mfc_handle_init_buffers(struct s5p_mfc_ctx *ctx,
        struct s5p_mfc_buf *src_buf;
        struct s5p_mfc_dev *dev;
 
-       if (ctx == NULL)
+       if (!ctx)
                return;
        dev = ctx->dev;
        s5p_mfc_hw_call(dev->mfc_ops, clear_int_flags, dev);
@@ -1289,7 +1289,7 @@ static int s5p_mfc_probe(struct platform_device *pdev)
                return PTR_ERR(dev->regs_base);
 
        res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
-       if (res == NULL) {
+       if (!res) {
                dev_err(&pdev->dev, "failed to get irq resource\n");
                return -ENOENT;
        }