media: staging: media: zoran: multiple assignments should be avoided
authorCorentin Labbe <clabbe@baylibre.com>
Tue, 18 May 2021 12:41:10 +0000 (14:41 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Wed, 2 Jun 2021 12:02:45 +0000 (14:02 +0200)
Remove all multiple assignments.

Signed-off-by: Corentin Labbe <clabbe@baylibre.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/zoran/zoran_driver.c
drivers/staging/media/zoran/zr36016.c
drivers/staging/media/zoran/zr36050.c
drivers/staging/media/zoran/zr36060.c

index e8902f824d6c44e53efd2a81ee5333c1d1ae1877..46382e43f1bf72c4c4f307b48425d22e20bb38d6 100644 (file)
@@ -678,12 +678,14 @@ static int zoran_g_selection(struct file *file, void *__fh, struct v4l2_selectio
                sel->r.height = zr->jpg_settings.img_height;
                break;
        case V4L2_SEL_TGT_CROP_DEFAULT:
-               sel->r.top = sel->r.left = 0;
+               sel->r.top = 0;
+               sel->r.left = 0;
                sel->r.width = BUZ_MIN_WIDTH;
                sel->r.height = BUZ_MIN_HEIGHT;
                break;
        case V4L2_SEL_TGT_CROP_BOUNDS:
-               sel->r.top = sel->r.left = 0;
+               sel->r.top = 0;
+               sel->r.left = 0;
                sel->r.width = BUZ_MAX_WIDTH;
                sel->r.height = BUZ_MAX_HEIGHT;
                break;
index 2d7dc7abde793a2b781dfe2693f243c36269bc7b..82702a13b05fdcbf8fe4de9c4e80ddca14e149fe 100644 (file)
@@ -361,7 +361,8 @@ static int zr36016_setup(struct videocodec *codec)
                return -ENOSPC;
        }
        //mem structure init
-       codec->data = ptr = kzalloc(sizeof(struct zr36016), GFP_KERNEL);
+       ptr = kzalloc(sizeof(struct zr36016), GFP_KERNEL);
+       codec->data = ptr;
        if (!ptr)
                return -ENOMEM;
 
index 2826f4e5d37babb2741ce7bfa3fa4e83b53aae58..a78862852a477952583f145dccc063076a0b0609 100644 (file)
@@ -754,7 +754,8 @@ static int zr36050_setup(struct videocodec *codec)
                return -ENOSPC;
        }
        //mem structure init
-       codec->data = ptr = kzalloc(sizeof(struct zr36050), GFP_KERNEL);
+       ptr = kzalloc(sizeof(struct zr36050), GFP_KERNEL);
+       codec->data = ptr;
        if (!ptr)
                return -ENOMEM;
 
index 4f9eb9ff2c42977d33b057833a6d331b38cc1bdf..1c3af11b5f24d7bc012ff385bf4538c26784b21f 100644 (file)
@@ -790,7 +790,8 @@ static int zr36060_setup(struct videocodec *codec)
                return -ENOSPC;
        }
        //mem structure init
-       codec->data = ptr = kzalloc(sizeof(*ptr), GFP_KERNEL);
+       ptr = kzalloc(sizeof(*ptr), GFP_KERNEL);
+       codec->data = ptr;
        if (!ptr)
                return -ENOMEM;