Hi Laurent, On Wed, Oct 29, 2014 at 01:20:06AM +0200, Laurent Pinchart wrote: > > ret = -EPIPE; > > + dev_dbg(entity->parent->dev, > > + "\"%s\":%u must be connected by an enabled > > link, error %d\n", > > + entity->name, > > + find_first_zero_bit(active, entity->num_pads), > > + ret); > > Given that ret is always set to -EPIPE, I wouldn't print ", error %d". > > Apart from that, > > Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Thanks for the ack. Good point as well, I'll fix that for v3. -- Sakari Ailus e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html