Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754182Ab0FHIwm (ORCPT ); Tue, 8 Jun 2010 04:52:42 -0400 Received: from mail-fx0-f46.google.com ([209.85.161.46]:65090 "EHLO mail-fx0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754045Ab0FHIwk (ORCPT ); Tue, 8 Jun 2010 04:52:40 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=subject:from:to:cc:in-reply-to:references:content-type:date :message-id:mime-version:x-mailer:content-transfer-encoding; b=FMa+9S7K3tHReL0aHkYbqwKh9iS2tkK9mQhqJ9FK5X/JgVg2KpNcUgtSsH7xh5m6Z6 H+imRuK9if600oPWS9TQK0FnX9YRuv2vrk0iJhCKiHzPDR7aMiXTW+BJ4LNAWitlcPLS TAha754KOL/zJFg8hrru+K324oyJ3pkYWyVNY= Subject: Re: [PATCH 2/2] mmc: fix mmc card disappearence on resume on ricoh mmc controllers From: Maxim Levitsky To: Chris Ball Cc: "linux-mmc@vger.kernel.org" , Philip Langdale , linux-kernel In-Reply-To: References: <1275859475.3151.4.camel@maxim-laptop> <1275859731-2898-2-git-send-email-maximlevitsky@gmail.com> <20100606232313.GA5444@void.printf.net> <1275870795.24091.4.camel@maxim-laptop> Content-Type: text/plain Date: Tue, 08 Jun 2010 11:52:36 +0300 Message-Id: <1275987156.30596.6.camel@dell16> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1229 Lines: 37 On Mon, 2010-06-07 at 01:47 -0400, Chris Ball wrote: > Hi Maxim, > > >> It looks like your editor is set to four-space instead of > >> eight-space tab characters, else you wouldn't be using so > >> many tabs here. > > > Nope, I think indention is right here. > > > > the break is inside 'if' condition. > > Please look again, I think you're mistaken. For example, why do you > use seven tab characters for the "& SDHCI_CARD_PRESENT" after the if > line? With eight-space tabs, it looks like this (converted to > spaces): > > + if (sdhci_readl(host, SDHCI_PRESENT_STATE) > + & SDHCI_CARD_PRESENT) { Ah, this. I just break the line to avoid hitting the 80 char limit... You probably meant I need to write: + if (sdhci_readl(host, SDHCI_PRESENT_STATE) + & SDHCI_CARD_PRESENT) { Nothing against it, Best regards, Maxim Levitsky -- 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/