media: v4l2-fwnode: Don't zero parts of struct v4l2_fwnode_endpoint anymore
authorSakari Ailus <sakari.ailus@linux.intel.com>
Wed, 30 Sep 2020 12:28:54 +0000 (14:28 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Mon, 16 Nov 2020 09:31:13 +0000 (10:31 +0100)
Don't zero parts of the vep argument to v4l2_fwnode_endpoint_parse()
anymore as this can no longer be done while still supporting defaults on
multiple bus types.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/v4l2-core/v4l2-fwnode.c

index dfc53d1..44dd04b 100644 (file)
@@ -416,20 +416,8 @@ static int __v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwnode,
        enum v4l2_mbus_type mbus_type;
        int rval;
 
-       if (vep->bus_type == V4L2_MBUS_UNKNOWN) {
-               /* Zero fields from bus union to until the end */
-               memset(&vep->bus, 0,
-                      sizeof(*vep) - offsetof(typeof(*vep), bus));
-       }
-
        pr_debug("===== begin parsing endpoint %pfw\n", fwnode);
 
-       /*
-        * Zero the fwnode graph endpoint memory in case we don't end up parsing
-        * the endpoint.
-        */
-       memset(&vep->base, 0, sizeof(vep->base));
-
        fwnode_property_read_u32(fwnode, "bus-type", &bus_type);
        pr_debug("fwnode video bus type %s (%u), mbus type %s (%u)\n",
                 v4l2_fwnode_bus_type_to_string(bus_type), bus_type,