{"id":811827,"url":"http://patchwork.ozlabs.org/api/patches/811827/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-imx/patch/e794361b-8f2a-7457-007f-72ef4fa66d02@users.sourceforge.net/","project":{"id":19,"url":"http://patchwork.ozlabs.org/api/projects/19/?format=json","name":"Linux IMX development","link_name":"linux-imx","list_id":"linux-imx-kernel.lists.patchwork.ozlabs.org","list_email":"linux-imx-kernel@lists.patchwork.ozlabs.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<e794361b-8f2a-7457-007f-72ef4fa66d02@users.sourceforge.net>","list_archive_url":null,"date":"2017-09-08T20:53:34","name":"[3/3,media] s5p-mfc: Adjust a null pointer check in four functions","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"9aa8b5f8e464c2f29fdc14ffe7af2ea554aeb65b","submitter":{"id":65077,"url":"http://patchwork.ozlabs.org/api/people/65077/?format=json","name":"SF Markus Elfring","email":"elfring@users.sourceforge.net"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-imx/patch/e794361b-8f2a-7457-007f-72ef4fa66d02@users.sourceforge.net/mbox/","series":[{"id":2290,"url":"http://patchwork.ozlabs.org/api/series/2290/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-imx/list/?series=2290","date":"2017-09-08T20:50:13","name":"S5P MFC: Adjustments for five function implementations","version":1,"mbox":"http://patchwork.ozlabs.org/series/2290/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/811827/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/811827/checks/","tags":{},"related":[],"headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"Qj4GlVnO\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xpqLZ2rJ8z9sBW\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tSat,  9 Sep 2017 06:54:14 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dqQHz-0001aa-Aw; Fri, 08 Sep 2017 20:54:11 +0000","from mout.web.de ([217.72.192.78])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dqQHr-0001Va-IJ for linux-arm-kernel@lists.infradead.org;\n\tFri, 08 Sep 2017 20:54:10 +0000","from [192.168.1.2] ([92.228.166.205]) by smtp.web.de (mrweb103\n\t[213.165.67.124]) with ESMTPSA (Nemesis) id 0MDgDS-1dg2IA2ptm-00H9Dp;\n\tFri, 08 Sep 2017 22:53:36 +0200"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:References:To:From:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=QCsEM59wbONcpogpUg5qovwhqTZuxqPcr+7WUcZxsBg=;\n\tb=Qj4GlVnOIgLURg\n\tKYF9rsfauf4Brz0kVISHKJWieCnclbAd9qlTrVqnfq08QTaLZ/bK+n3R+mzBMAtsbgKVPbme5KumZ\n\tQis1cwwdLwgUqG5ZowfI5Ob2oJjXkad4pVM7pdA+NRkr52OFqsUjnHne0rnE67H60uYSic6xZcltk\n\t+LWZ3pPqJbHwg0Yh7vHUKMl3eFW0qsIxjztg95Lj/FGhf8bj+kCpaThETGfUpxnlBEG3Cn+o8qSG0\n\tTI9NRc5Ztpi3aJXIa8blhfrK0+BX2LmSpGG4qwg0J/pcIR4YwyLmVskJesDyofBGdkQMFujsj6fXp\n\t7qjTf+cHYzut9gu1p4wQ==;","Subject":"[PATCH 3/3] [media] s5p-mfc: Adjust a null pointer check in four\n\tfunctions","From":"SF Markus Elfring <elfring@users.sourceforge.net>","To":"linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org,\n\tAndrzej Hajda <a.hajda@samsung.com>,\n\tJeongtae Park <jtp.park@samsung.com>, Kamil Debski <kamil@wypas.org>,\n\tKyungmin Park <kyungmin.park@samsung.com>, \n\tMarek Szyprowski <m.szyprowski@samsung.com>,\n\tMauro Carvalho Chehab <mchehab@kernel.org>","References":"<482a6c92-a85e-0bcd-edf7-3c2f63ea74c5@users.sourceforge.net>","Message-ID":"<e794361b-8f2a-7457-007f-72ef4fa66d02@users.sourceforge.net>","Date":"Fri, 8 Sep 2017 22:53:34 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<482a6c92-a85e-0bcd-edf7-3c2f63ea74c5@users.sourceforge.net>","Content-Language":"en-GB","X-Provags-ID":"V03:K0:rvBWpKC9cR+S4kDMNyyx+ovEMLYokT6K7bMVN3+dKIvCDp05j6A\n\tJOWMy7U/XpsMpAvMZN4q1MUMRYGcEbhsPmul1YiI0OzxkbLWesMN7WKQQt5vwYJ0cYc7ZbY\n\tjz6Q+AiWPi7pqziwZ04VxSROCkkT2hZNXbfll8P4Qop938tKcOpHogXXiMECZZIVX22cHJC\n\t1ajuGRCZA638CkPlR5Pqw==","X-UI-Out-Filterresults":"notjunk:1; V01:K0:IItmpIMTf58=:dXWjSMciECVJU1fd/Dr03N\n\t3JwATxaqUhbEOUk0rSCcFkV911ZRdmkV6azs4A0e5JhtEYDnouwpbJntVKM26wsWt7K0u4W7R\n\tFVVg6wl3JlrcOoMhuoMF/oWNTKECU1GjafZaJsf4DXboBfcMRvBU1xHDm/ifjL9xGcLfuWWbf\n\t7ZOFkdkCXQb2p4jNug5J/w+/ETuS+dGyz+KB4k8t8VlLuBgOI2Xvkaq7JWbHJe3Hk9BAuF2qu\n\t4TW/pUDfTPSHYVJFifENwLbynZoHmzOzsn0jeuXdn7qzSyASeGKu27RlRAz1dyIv5+OpsyjOq\n\tJr2gU7ftMDWG/dPlSZsjDW47dnhM739tlOZZsdFKYCrNftjxz9e8W0GROVLT38cTEJvYjqhcN\n\tj3srjhAqjz8uiD7SiDiEQlRfDsMk/08wfJ5T1mR3iVWsUnrMhf34JD/XP39rxb5Fz3LcWea2Y\n\tFAbGJ6SF4o2L6P2vEk/EkbK5ZQg2M9dPwFAq18+IDZXO4fTpdzNRx1eVj/slRv+E1hH0vgNHM\n\tFsM1Em+OxvyqGoHvK/0Y9xhHsBQ3okGLlLBbWfhcekVKQ7CWQEnrW9VIyLdymh8w/+aYwXdOk\n\tTgnrcIb+r7f4XhYgTtJfAmcPVPhvXcavzmsa3iOYVjyX5o964xBvUkV6uybXoMyJHsCtcQ9t0\n\tJmPymlvn5gQp1rqbjnsz7W3jp9MehdrRrJ/6kfwHJmUs5kF0MXk4+0ABwiLfLUDHKftLHG6G4\n\tQdepfUMCrn8rpfbZ81uwEeoDChPBu7NSmtLroR8di4OBQqvtJO/yOHIc+CzZ8qEXQUx/yyAzC\n\tbDZ3fCjCUs340uBK3zqw8+2YM/d5Q85YTVUMz85qrWNqmlGu68=","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170908_135404_047999_9B81F474 ","X-CRM114-Status":"GOOD (  11.21  )","X-Spam-Score":"-5.4 (-----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-5.4 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/,\n\tlow trust [217.72.192.78 listed in list.dnswl.org]\n\t-2.8 RCVD_IN_MSPIKE_H2      RBL: Average reputation (+2)\n\t[217.72.192.78 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"kernel-janitors@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"},"content":"From: Markus Elfring <elfring@users.sourceforge.net>\nDate: Fri, 8 Sep 2017 22:37:00 +0200\nMIME-Version: 1.0\nContent-Type: text/plain; charset=UTF-8\nContent-Transfer-Encoding: 8bit\n\nThe script “checkpatch.pl” pointed information out like the following.\n\nComparison to NULL could be written …\n\nThus fix the affected source code places.\n\nSigned-off-by: Markus Elfring <elfring@users.sourceforge.net>\n---\n drivers/media/platform/s5p-mfc/s5p_mfc.c | 8 ++++----\n 1 file changed, 4 insertions(+), 4 deletions(-)","diff":"diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c b/drivers/media/platform/s5p-mfc/s5p_mfc.c\nindex abfb70b07032..cf68aed59e0d 100644\n--- a/drivers/media/platform/s5p-mfc/s5p_mfc.c\n+++ b/drivers/media/platform/s5p-mfc/s5p_mfc.c\n@@ -470,7 +470,7 @@ static void s5p_mfc_handle_error(struct s5p_mfc_dev *dev,\n {\n \tmfc_err(\"Interrupt Error: %08x\\n\", err);\n \n-\tif (ctx != NULL) {\n+\tif (ctx) {\n \t\t/* Error recovery is dependent on the state of context */\n \t\tswitch (ctx->state) {\n \t\tcase MFCINST_RES_CHANGE_INIT:\n@@ -508,7 +508,7 @@ static void s5p_mfc_handle_seq_done(struct s5p_mfc_ctx *ctx,\n {\n \tstruct s5p_mfc_dev *dev;\n \n-\tif (ctx == NULL)\n+\tif (!ctx)\n \t\treturn;\n \tdev = ctx->dev;\n \tif (ctx->c_ops->post_seq_start) {\n@@ -562,7 +562,7 @@ static void s5p_mfc_handle_init_buffers(struct s5p_mfc_ctx *ctx,\n \tstruct s5p_mfc_buf *src_buf;\n \tstruct s5p_mfc_dev *dev;\n \n-\tif (ctx == NULL)\n+\tif (!ctx)\n \t\treturn;\n \tdev = ctx->dev;\n \ts5p_mfc_hw_call(dev->mfc_ops, clear_int_flags, dev);\n@@ -1289,7 +1289,7 @@ static int s5p_mfc_probe(struct platform_device *pdev)\n \t\treturn PTR_ERR(dev->regs_base);\n \n \tres = platform_get_resource(pdev, IORESOURCE_IRQ, 0);\n-\tif (res == NULL) {\n+\tif (!res) {\n \t\tdev_err(&pdev->dev, \"failed to get irq resource\\n\");\n \t\treturn -ENOENT;\n \t}\n","prefixes":["3/3","media"]}