[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Thu, 4 Apr 2019 21:08:58 +0200
From: Sam Ravnborg <sam@...nborg.org>
To: Joe Perches <joe@...ches.com>
Cc: Guido Günther <agx@...xcpu.org>,
Thierry Reding <thierry.reding@...il.com>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel@...ll.ch>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Kevin Hilman <khilman@...libre.com>,
Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
Shawn Guo <shawnguo@...nel.org>,
Jagan Teki <jagan@...rulasolutions.com>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
Johan Hovold <johan@...nel.org>,
"David S. Miller" <davem@...emloft.net>,
Mauro Carvalho Chehab <mchehab+samsung@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Nicolas Ferre <nicolas.ferre@...rochip.com>,
dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/panel: Rocktech jh057n00900: Add terminating
newlines to logging
Hi Joe.
> > >
> > > ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
> > > if (ret < 0) {
> > > - DRM_DEV_ERROR(dev, "Failed to exit sleep mode");
> > > + DRM_DEV_ERROR(dev, "Failed to exit sleep mode\n");
> >
> > I was under the impression that newlines was optional these days.
> > Should we always use them with logging?
>
> Yes.
>
> The general problem is KERN_CONT/pr_cont uses where interleaving
> is still possible from multiple threads.
>
> > I did not find any obvious clues in linux/printk.h
>
> I'll see about adding something one day to the Documentation.
> Likely in coding-style.rst.
Thanks for the clarification.
A seperate doc on logging would be awesome.
Sam
Powered by blists - more mailing lists