From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753609AbdCOMYR (ORCPT ); Wed, 15 Mar 2017 08:24:17 -0400 Received: from mail-wm0-f52.google.com ([74.125.82.52]:35721 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752017AbdCOMXF (ORCPT ); Wed, 15 Mar 2017 08:23:05 -0400 Date: Wed, 15 Mar 2017 12:23:00 +0000 From: Lee Jones To: Joe Perches Cc: Linus Walleij , Tony Lindgren , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org Subject: Re: [PATCH 23/35] drivers/mfd: Convert remaining uses of pr_warning to pr_warn Message-ID: <20170315122300.jwuogngm6obgvz5z@dell> References: <20170314140455.umlhdbuj3grqtu6i@dell> <1489511868.2582.2.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1489511868.2582.2.camel@perches.com> User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 14 Mar 2017, Joe Perches wrote: > On Tue, 2017-03-14 at 14:04 +0000, Lee Jones wrote: > > On Thu, 16 Feb 2017, Joe Perches wrote: > > > To enable eventual removal of pr_warning > [] > > > diff --git a/drivers/mfd/twl4030-power.c b/drivers/mfd/twl4030-power.c > [] > > > @@ -502,9 +502,7 @@ static int load_twl4030_script(const struct twl4030_power_data *pdata, > > > } > > > if (tscript->flags & TWL4030_SLEEP_SCRIPT) { > > > if (!order) > > > - pr_warning("TWL4030: Bad order of scripts (sleep "\ > > > - "script before wakeup) Leads to boot"\ > > > - "failure on some boards\n"); > > > + pr_warn("TWL4030: Bad order of scripts (sleep script before wakeup) Leads to boot failure on some boards\n"); > > > > Is 80 chars considered the lessor sin? > > Splitting logging messages into multiple lines is non preferred. > > Documentation/process/coding-style.rst: > > Breaking long lines and strings: > > never break user-visible strings such as > printk messages, because that breaks the ability to grep for them. Right, I'm aware of this. I'm also aware of the other rule we have where lines are not to exceed 80 chars. Hence my question; "is the 80 chars rule the lessor sin of the two". Rock <=> hard place Actually I would say that the error message sub-optimal and could, in fact be edited to be more succinct, thus not breaking any rules. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog