Commit 971b4eef authored by Abdulrasaq Lawani's avatar Abdulrasaq Lawani Committed by Hans Verkuil
Browse files

media: i2c: ov5647: replacing of_node_put with __free(device_node) 



Replace instance of of_node_put with __free(device_node)
to protect against any memory leaks due to future changes
in control flow.

Signed-off-by: default avatarAbdulrasaq Lawani <abdulrasaqolawani@gmail.com>
Acked-by: default avatarDave Stevenson <dave.stevenson@raspberrypi.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent 15765ff7

drivers/media/i2c/ov5647.c

+4 −7
Original line number Diff line number Diff line
@@ -1360,24 +1360,21 @@ static int ov5647_parse_dt(struct ov5647 *sensor, struct device_node *np) 
	struct v4l2_fwnode_endpoint bus_cfg = {

		.bus_type = V4L2_MBUS_CSI2_DPHY,

	};

	struct device_node *ep;

	struct device_node *ep __free(device_node) =

		of_graph_get_endpoint_by_regs(np, 0, -1);

	int ret;



	ep = of_graph_get_endpoint_by_regs(np, 0, -1);

	if (!ep)

		return -EINVAL;



	ret = v4l2_fwnode_endpoint_parse(of_fwnode_handle(ep), &bus_cfg);

	if (ret)

		goto out;

		return ret;



	sensor->clock_ncont = bus_cfg.bus.mipi_csi2.flags &

			      V4L2_MBUS_CSI2_NONCONTINUOUS_CLOCK;



out:

	of_node_put(ep);



	return ret;

	return 0;

}



static int ov5647_probe(struct i2c_client *client)