[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAKh2mn4xfpO-vMZGnikuS29xrCc_XJMGu0gexyn1EBeG4prgog@mail.gmail.com>
Date: Tue, 27 Nov 2012 17:23:15 +0000
From: Steve Glendinning <steve@...well.net>
To: Joe Perches <joe@...ches.com>
Cc: netdev <netdev@...r.kernel.org>, David Miller <davem@...emloft.net>
Subject: Re: [PATCH 2/2] smsc75xx: support PHY wakeup source
Hi Joe,
On 27 November 2012 15:56, Joe Perches <joe@...ches.com> wrote:
> On Tue, 2012-11-27 at 14:28 +0000, Steve Glendinning wrote:
>> + ret = smsc75xx_read_reg_nopm(dev, PMT_CTL, &val);
>> + check_warn_return(ret, "Error reading PMT_CTL");
>
> Hi Steve, can you please add newlines to these new
> check_warn_<foo> messages and the netdev_<level> ones too?
> It helps avoid message interleaving.
Sorry about that, I did see your earlier patch to fix existing instances.
David: please discard this patch too, I'll resubmit.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists