Re: [PATCH] net: ethernet: support "fixed-link" DT node on nb8800 driver

From: Måns Rullgård
Date: Fri Feb 05 2016 - 10:58:54 EST


Andy Shevchenko <andy.shevchenko@xxxxxxxxx> writes:

> On Fri, Feb 5, 2016 at 3:39 PM, Måns Rullgård <mans@xxxxxxxxx> wrote:
>>> + if (ret < 0) {
>>> + dev_err(&pdev->dev, "broken fixed-link specification\n");
>>
>> Line is longer than 80 chars.
>
> This is actually okay, though I would recommend to move long string
> literal to the next line.

I only mentioned it because fixing it was trivial.

--
Måns Rullgård