Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755174Ab0BACQ1 (ORCPT ); Sun, 31 Jan 2010 21:16:27 -0500 Received: from gate.crashing.org ([63.228.1.57]:57765 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751039Ab0BACQ0 (ORCPT ); Sun, 31 Jan 2010 21:16:26 -0500 Subject: Re: [PATCH 01/10] arch/powerpc: Fix continuation line formats From: Benjamin Herrenschmidt To: Joe Perches Cc: linux-kernel@vger.kernel.org, Paul Mackerras , linuxppc-dev@ozlabs.org In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Date: Mon, 01 Feb 2010 13:16:14 +1100 Message-ID: <1264990574.8287.15.camel@pasglop> Mime-Version: 1.0 X-Mailer: Evolution 2.28.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3079 Lines: 86 On Sun, 2010-01-31 at 12:02 -0800, Joe Perches wrote: > String constants that are continued on subsequent lines with \ > are not good. > > Signed-off-by: Joe Perches You want me to take that in the powerpc tree ? A minor glitch below tho... > --- > arch/powerpc/kernel/nvram_64.c | 6 +++--- > arch/powerpc/platforms/pseries/hotplug-cpu.c | 8 ++++---- > arch/powerpc/platforms/pseries/smp.c | 4 ++-- > 3 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/arch/powerpc/kernel/nvram_64.c b/arch/powerpc/kernel/nvram_64.c > index ad461e7..9cf197f 100644 > --- a/arch/powerpc/kernel/nvram_64.c > +++ b/arch/powerpc/kernel/nvram_64.c > @@ -338,8 +338,8 @@ static int __init nvram_create_os_partition(void) > > rc = nvram_write_header(new_part); > if (rc <= 0) { > - printk(KERN_ERR "nvram_create_os_partition: nvram_write_header \ > - failed (%d)\n", rc); > + printk(KERN_ERR "nvram_create_os_partition: nvram_write_header " > + "failed (%d)\n", rc); > return rc; > } > > @@ -349,7 +349,7 @@ static int __init nvram_create_os_partition(void) > rc = ppc_md.nvram_write((char *)&seq_init, sizeof(seq_init), &tmp_index); > if (rc <= 0) { > printk(KERN_ERR "nvram_create_os_partition: nvram_write " > - "failed (%d)\n", rc); > + "failed (%d)\n", rc); > return rc; > } The above is objectionable :-) > > diff --git a/arch/powerpc/platforms/pseries/hotplug-cpu.c b/arch/powerpc/platforms/pseries/hotplug-cpu.c > index 6ea4698..a70de10 100644 > --- a/arch/powerpc/platforms/pseries/hotplug-cpu.c > +++ b/arch/powerpc/platforms/pseries/hotplug-cpu.c > @@ -397,12 +397,12 @@ static int parse_cede_parameters(void) > CEDE_LATENCY_PARAM_MAX_LENGTH); > > if (call_status != 0) > - printk(KERN_INFO "CEDE_LATENCY: \ > - %s %s Error calling get-system-parameter(0x%x)\n", > + printk(KERN_INFO "CEDE_LATENCY: " > + "%s %s Error calling get-system-parameter(0x%x)\n", > __FILE__, __func__, call_status); > else > - printk(KERN_INFO "CEDE_LATENCY: \ > - get-system-parameter successful.\n"); > + printk(KERN_INFO "CEDE_LATENCY: " > + "get-system-parameter successful.\n"); > > return call_status; > } > diff --git a/arch/powerpc/platforms/pseries/smp.c b/arch/powerpc/platforms/pseries/smp.c > index b488663..4e7f89a 100644 > --- a/arch/powerpc/platforms/pseries/smp.c > +++ b/arch/powerpc/platforms/pseries/smp.c > @@ -144,8 +144,8 @@ static void __devinit smp_pSeries_kick_cpu(int nr) > hcpuid = get_hard_smp_processor_id(nr); > rc = plpar_hcall_norets(H_PROD, hcpuid); > if (rc != H_SUCCESS) > - printk(KERN_ERR "Error: Prod to wake up processor %d\ > - Ret= %ld\n", nr, rc); > + printk(KERN_ERR "Error: Prod to wake up processor %d " > + "Ret= %ld\n", nr, rc); > } > } > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/