On Mon, 20 Jul 2015 10:47:46 +0200,
Dan Carpenter wrote:
>
> On Tue, Jul 14, 2015 at 06:34:06PM -0600, Shuah Khan wrote:
> > + ret = media_entity_setup_link(link, flags);
> > + if (ret) {
> > + dev_err(mctl->media_dev->dev,
> > + "Couldn't change tuner link",
> > + "%s->%s to %s. Error %d\n",
>
> Add a space after "link".
>
> "Couldn't change tuner link ",
> "%s->%s to %s. Error %d\n",
Such a message string would be better to be in a single line even if
it's over 80 chars. Otherwise it becomes hard to grep.
thanks,
Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
