From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Ball Subject: Re: [PATCH] dw_mmc: modify quirks bit-shift control Date: Thu, 17 Feb 2011 04:13:12 +0000 Message-ID: <20110217041309.GA23241@void.printf.net> References: <4D5B6C70.2030809@samsung.com> <20110216194232.GA17634@void.printf.net> <4D5C9F60.9020306@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from void.printf.net ([89.145.121.20]:41064 "EHLO void.printf.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750981Ab1BQENQ (ORCPT ); Wed, 16 Feb 2011 23:13:16 -0500 Content-Disposition: inline In-Reply-To: <4D5C9F60.9020306@samsung.com> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Jaehoon Chung Cc: "linux-mmc@vger.kernel.org" , Will Newton , Kyungmin Park Hi, On Thu, Feb 17, 2011 at 01:09:04PM +0900, Jaehoon Chung wrote: > Hi.. > > > Why skip (1 << 1)? (There's also BIT(0) -> BIT(2) if you prefer.) > > My mistake..sorry..and thanks for your comment.. > I resend the patch applied BIT(0)-BIT(2) No problem. Will, may I re-use your ACK for this patch? > > Signed-off-by: Jaehoon Chung > --- > include/linux/mmc/dw_mmc.h | 8 +++----- > 1 files changed, 3 insertions(+), 5 deletions(-) > > diff --git a/include/linux/mmc/dw_mmc.h b/include/linux/mmc/dw_mmc.h > index 16b0261..3f22c20 100644 > --- a/include/linux/mmc/dw_mmc.h > +++ b/include/linux/mmc/dw_mmc.h > @@ -165,14 +165,12 @@ struct dw_mci_dma_ops { > }; > > /* IP Quirks/flags. */ > -/* No special quirks or flags to cater for */ > -#define DW_MCI_QUIRK_NONE 0 > /* DTO fix for command transmission with IDMAC configured */ > -#define DW_MCI_QUIRK_IDMAC_DTO 1 > +#define DW_MCI_QUIRK_IDMAC_DTO BIT(0) > /* delay needed between retries on some 2.11a implementations */ > -#define DW_MCI_QUIRK_RETRY_DELAY 2 > +#define DW_MCI_QUIRK_RETRY_DELAY BIT(1) > /* High Speed Capable - Supports HS cards (upto 50MHz) */ > -#define DW_MCI_QUIRK_HIGHSPEED 4 > +#define DW_MCI_QUIRK_HIGHSPEED BIT(2) > > > struct dma_pdata; -- Chris Ball One Laptop Per Child