diff mbox

[media] media: videobuf2: fix compare_const_fl.cocci warnings

Message ID alpine.DEB.2.10.1511131143290.2561@hadrien (mailing list archive)
State New, archived
Headers show

Commit Message

Julia Lawall Nov. 13, 2015, 10:46 a.m. UTC 
Move constants to the right of binary operators.

Generated by: scripts/coccinelle/misc/compare_const_fl.cocci

CC: Junghak Sung <jh1009.sung@samsung.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
---

!b->m.planes could also be possible.  Up to you.

 videobuf2-v4l2.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Sakari Ailus Nov. 15, 2015, 11:05 p.m. UTC | #1 
On Fri, Nov 13, 2015 at 11:46:00AM +0100, Julia Lawall wrote:
>  Move constants to the right of binary operators.
> 
> Generated by: scripts/coccinelle/misc/compare_const_fl.cocci
> 
> CC: Junghak Sung <jh1009.sung@samsung.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>

Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
diff mbox

Patch

--- a/drivers/media/v4l2-core/videobuf2-v4l2.c
+++ b/drivers/media/v4l2-core/videobuf2-v4l2.c
@@ -52,7 +52,7 @@  static int __verify_planes_array(struct
 		return 0;

 	/* Is memory for copying plane information present? */
-	if (NULL == b->m.planes) {
+	if (b->m.planes == NULL) {
 		dprintk(1, "multi-planar buffer passed but "
 			   "planes array not provided\n");
 		return -EINVAL;