From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrian Hunter Subject: Re: [v2, 1/2] mmc: sdhci-esdhc: clean up register definitions Date: Thu, 12 Jan 2017 08:55:44 +0200 Message-ID: <9658d989-d935-ce32-a875-9146fa1870b5@intel.com> References: <1482745590-29718-1-git-send-email-yangbo.lu@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from mga05.intel.com ([192.55.52.43]:25011 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750719AbdALHEC (ORCPT ); Thu, 12 Jan 2017 02:04:02 -0500 In-Reply-To: <1482745590-29718-1-git-send-email-yangbo.lu@nxp.com> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Yangbo Lu , linux-mmc@vger.kernel.org, ulf.hansson@linaro.org On 26/12/16 11:46, Yangbo Lu wrote: > The eSDHC register definitions in header file were messy and confusing. > This patch is to clean up these definitions. > > Signed-off-by: Yangbo Lu Acked-by: Adrian Hunter > --- > Changes for v2: > - added Adrian into to list > --- > drivers/mmc/host/sdhci-esdhc.h | 39 ++++++++++++++++++++------------------- > 1 file changed, 20 insertions(+), 19 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-esdhc.h b/drivers/mmc/host/sdhci-esdhc.h > index de132e2..8cd8449 100644 > --- a/drivers/mmc/host/sdhci-esdhc.h > +++ b/drivers/mmc/host/sdhci-esdhc.h > @@ -24,30 +24,31 @@ > SDHCI_QUIRK_PIO_NEEDS_DELAY | \ > SDHCI_QUIRK_NO_HISPD_BIT) > > -#define ESDHC_PROCTL 0x28 > - > -#define ESDHC_SYSTEM_CONTROL 0x2c > -#define ESDHC_CLOCK_MASK 0x0000fff0 > -#define ESDHC_PREDIV_SHIFT 8 > -#define ESDHC_DIVIDER_SHIFT 4 > -#define ESDHC_CLOCK_PEREN 0x00000004 > -#define ESDHC_CLOCK_HCKEN 0x00000002 > -#define ESDHC_CLOCK_IPGEN 0x00000001 > - > /* pltfm-specific */ > #define ESDHC_HOST_CONTROL_LE 0x20 > > /* > - * P2020 interpretation of the SDHCI_HOST_CONTROL register > + * eSDHC register definition > */ > -#define ESDHC_CTRL_4BITBUS (0x1 << 1) > -#define ESDHC_CTRL_8BITBUS (0x2 << 1) > -#define ESDHC_CTRL_BUSWIDTH_MASK (0x3 << 1) > - > -/* OF-specific */ > -#define ESDHC_DMA_SYSCTL 0x40c > -#define ESDHC_DMA_SNOOP 0x00000040 > > -#define ESDHC_HOST_CONTROL_RES 0x01 > +/* Protocol Control Register */ > +#define ESDHC_PROCTL 0x28 > +#define ESDHC_CTRL_4BITBUS (0x1 << 1) > +#define ESDHC_CTRL_8BITBUS (0x2 << 1) > +#define ESDHC_CTRL_BUSWIDTH_MASK (0x3 << 1) > +#define ESDHC_HOST_CONTROL_RES 0x01 > + > +/* System Control Register */ > +#define ESDHC_SYSTEM_CONTROL 0x2c > +#define ESDHC_CLOCK_MASK 0x0000fff0 > +#define ESDHC_PREDIV_SHIFT 8 > +#define ESDHC_DIVIDER_SHIFT 4 > +#define ESDHC_CLOCK_PEREN 0x00000004 > +#define ESDHC_CLOCK_HCKEN 0x00000002 > +#define ESDHC_CLOCK_IPGEN 0x00000001 > + > +/* Control Register for DMA transfer */ > +#define ESDHC_DMA_SYSCTL 0x40c > +#define ESDHC_DMA_SNOOP 0x00000040 > > #endif /* _DRIVERS_MMC_SDHCI_ESDHC_H */ >