Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759795AbZAGSgs (ORCPT ); Wed, 7 Jan 2009 13:36:48 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753321AbZAGSgj (ORCPT ); Wed, 7 Jan 2009 13:36:39 -0500 Received: from ftp.linux-mips.org ([213.58.128.207]:39471 "EHLO ftp.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752506AbZAGSgj (ORCPT ); Wed, 7 Jan 2009 13:36:39 -0500 Date: Wed, 7 Jan 2009 18:36:33 +0000 (GMT) From: "Maciej W. Rozycki" To: Jaswinder Singh Rajput cc: Ingo Molnar , x86 maintainers , LKML Subject: Re: [PATCH -tip] x86: i8259.c fix style problems In-Reply-To: <1231067032.27852.34.camel@jaswinder.satnam> Message-ID: References: <1231067032.27852.34.camel@jaswinder.satnam> User-Agent: Alpine 1.10 (LFD 962 2008-03-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1473 Lines: 46 On Sun, 4 Jan 2009, Jaswinder Singh Rajput wrote: > @@ -11,15 +11,15 @@ > #include > #include > #include > +#include > +#include > +#include > > -#include > #include > #include > -#include > #include > #include > #include > -#include > #include > #include > #include My feeling is this hunk and the following one should be submitted as separate patches. > @@ -323,7 +323,7 @@ void init_8259A(int auto_eoi) > outb_pic(0x11, PIC_MASTER_CMD); /* ICW1: select 8259A-1 init */ > > /* ICW2: 8259A-1 IR0-7 mapped to 0x30-0x37 on x86-64, > - to 0x20-0x27 on i386 */ > + to 0x20-0x27 on i386 */ > outb_pic(IRQ0_VECTOR, PIC_MASTER_IMR); > > /* 8259A-1 (the master) has a slave on IR2 */ I think the original formatting is more readable; I would align the two 'to's vertically even. You can fix the comment to adhere to the Linux coding standard though and the whitespace "problem" will vanish automagically. Maciej -- 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/