Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756173Ab0A3CTO (ORCPT ); Fri, 29 Jan 2010 21:19:14 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755216Ab0A3CTN (ORCPT ); Fri, 29 Jan 2010 21:19:13 -0500 Received: from usul.overt.org ([204.11.33.43]:34454 "EHLO usul.overt.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754435Ab0A3CTM (ORCPT ); Fri, 29 Jan 2010 21:19:12 -0500 Date: Fri, 29 Jan 2010 18:18:57 -0800 From: Philip Langdale To: Maxim Levitsky Cc: Wolfram Sang , Andrew Morton , Pierre Ossman , linux-kernel , linux-mmc@vger.kernel.org Subject: Re: [PATCH v3] port ricoh_mmc to pci quirk Message-ID: <20100129181857.1f23f013@fido2.homeip.net> In-Reply-To: <1264803033.7095.4.camel@maxim-laptop> References: <20091125085324.1ef9ae1f@fido2.homeip.net> <20091125113501.1f4d42b9.akpm@linux-foundation.org> <1259192401.15916.48.camel@maxim-laptop> <20091125173019.74d0ddb9@fido2.homeip.net> <1259279520.3991.5.camel@maxim-laptop> <1259279584.3991.6.camel@maxim-laptop> <20091126235551.2db699e3@fido2.homeip.net> <1262964284.12577.27.camel@maxim-laptop> <20100108074001.5df6c997@fido2.homeip.net> <20100112154350.8fa96829.akpm@linux-foundation.org> <20100113064657.GB10955@pengutronix.de> <1264772255.4620.3.camel@maxim-laptop> <20100129080639.3b484a59@fido2.homeip.net> <1264803033.7095.4.camel@maxim-laptop> X-Mailer: Claws Mail 3.7.2 (GTK+ 2.18.3; i486-slackware-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-SA-Do-Not-RunX1: Yes Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 989 Lines: 26 On Sat, 30 Jan 2010 00:10:33 +0200 Maxim Levitsky wrote: > Well, I have nothing against this, but, even Linus himself stated that > he dislikes broken printk lines. > > I also noticed that some DECLARE_FIXUP in this file are longer that > 80, and since breaking this doesn't add any value, I thought that is > ok. Remember that main purpose of 80 char limit is to make code > simpler. > > However, I don't really care if you insist. I don't want to be obnoxious either - if no one else is concerned about the lines, I won't kick up a fuss. And thanks for sticking with this patch - I want to see it in soon too; I'd like to sort out the new PCI-e Ricoh parts which are going to need a new magic sequence. --phil -- 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/