On Thu, Jun 27, 2019 at 08:56:03PM +0200, Stefan Wahren wrote: > v4l2_dbg(1, bcm2835_v4l2_debug, &dev->v4l2_dev, "connecting %p to %p\n", > src, dst); > ret = vchiq_mmal_port_connect_tunnel(dev->instance, src, dst); > if (!ret) > ret = vchiq_mmal_port_enable(dev->instance, src, NULL); > -error: > + > return ret; In future patches, you probably want to flip this one around as well. Try to do error handling instead of success handling. In other words, keep the success patch indented one tab and the failure path indented two tabs. Don't make the last failure check in the function special. ret = vchiq_mmal_port_connect_tunnel(dev->instance, src, dst); if (ret) return ret; ret = vchiq_mmal_port_enable(dev->instance, src, NULL); if (ret) return ret; return 0; Or you can make the last check a little special if you want... return vchiq_mmal_port_enable(dev->instance, src, NULL); Either format is good. regards, dan carpenter
