[media.git/next] media: atomisp: gmin: Remove duplicate NULL test

[media-ci]

On Thu Apr 3 14:26:41 2025 +0100, Abraham Samuel Adekunle wrote:
> When a value has been tested for NULL in an expression, a
> second NULL test on the same value in another expression
> is unnecessary when the value has not been assigned NULL.
> 
> Remove unnecessary duplicate NULL tests on the same value that
> has previously been NULL tested.
> 
> Found by Coccinelle.
> 
> Signed-off-by: Abraham Samuel Adekunle <abrahamadekunl...@gmail.com>
> Link: 
> https://lore.kernel.org/r/26990d4a9d4419f9d4155a40595bc213acb671a0.1743685415.git.abrahamadekunl...@gmail.com
> Signed-off-by: Hans de Goede <hdego...@redhat.com>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+hua...@kernel.org>

Patch committed.

Thanks,
Mauro Carvalho Chehab

 drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c 
b/drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c
index e7923dc38f2e..b4f2c785e6a6 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c
@@ -1206,7 +1206,7 @@ static int gmin_get_config_dsm_var(struct device *dev,
         * if it founds something different than string, letting it
         * to fall back to the old code.
         */
-       if (cur && cur->type != ACPI_TYPE_STRING) {
+       if (cur->type != ACPI_TYPE_STRING) {
                dev_info(dev, "found non-string _DSM entry for '%s'\n", var);
                ACPI_FREE(obj);
                return -EINVAL;