From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaehoon Chung Date: Wed, 27 May 2020 15:47:18 +0900 Subject: [PATCH 2/7] mmc: zynq_sdhci: Define timing macro's In-Reply-To: <22b8d8014e6539a876c9761f70d9cd80fb8c1964.1590144247.git.michal.simek@xilinx.com> References: <22b8d8014e6539a876c9761f70d9cd80fb8c1964.1590144247.git.michal.simek@xilinx.com> Message-ID: <691b2b31-ad2f-ca0f-e199-f176804fbf5b@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 5/22/20 7:44 PM, Michal Simek wrote: > From: Ashok Reddy Soma > > Define timing macro's for all the available speeds of mmc. This is > done similar to linux. Replace other macro's used in zynq_sdhci.c > with these new macro's. Even though it's similar to linux, does it need to add new macro? > > Signed-off-by: Ashok Reddy Soma > Signed-off-by: Michal Simek > --- > > drivers/mmc/zynq_sdhci.c | 24 +++++++++++------------- > include/mmc.h | 13 +++++++++++++ > 2 files changed, 24 insertions(+), 13 deletions(-) > > diff --git a/drivers/mmc/zynq_sdhci.c b/drivers/mmc/zynq_sdhci.c > index 94c69cf1c1bd..02583f76f936 100644 > --- a/drivers/mmc/zynq_sdhci.c > +++ b/drivers/mmc/zynq_sdhci.c > @@ -32,20 +32,18 @@ struct arasan_sdhci_priv { > }; > > #if defined(CONFIG_ARCH_ZYNQMP) > -#define MMC_HS200_BUS_SPEED 5 > - > static const u8 mode2timing[] = { > - [MMC_LEGACY] = UHS_SDR12_BUS_SPEED, > - [MMC_HS] = HIGH_SPEED_BUS_SPEED, > - [SD_HS] = HIGH_SPEED_BUS_SPEED, > - [MMC_HS_52] = HIGH_SPEED_BUS_SPEED, > - [MMC_DDR_52] = HIGH_SPEED_BUS_SPEED, > - [UHS_SDR12] = UHS_SDR12_BUS_SPEED, > - [UHS_SDR25] = UHS_SDR25_BUS_SPEED, > - [UHS_SDR50] = UHS_SDR50_BUS_SPEED, > - [UHS_DDR50] = UHS_DDR50_BUS_SPEED, > - [UHS_SDR104] = UHS_SDR104_BUS_SPEED, > - [MMC_HS_200] = MMC_HS200_BUS_SPEED, > + [MMC_LEGACY] = MMC_TIMING_LEGACY, > + [MMC_HS] = MMC_TIMING_MMC_HS, > + [SD_HS] = MMC_TIMING_SD_HS, > + [MMC_HS_52] = MMC_TIMING_UHS_SDR50, > + [MMC_DDR_52] = MMC_TIMING_UHS_DDR50, > + [UHS_SDR12] = MMC_TIMING_UHS_SDR12, > + [UHS_SDR25] = MMC_TIMING_UHS_SDR25, > + [UHS_SDR50] = MMC_TIMING_UHS_SDR50, > + [UHS_DDR50] = MMC_TIMING_UHS_DDR50, > + [UHS_SDR104] = MMC_TIMING_UHS_SDR104, > + [MMC_HS_200] = MMC_TIMING_MMC_HS200, > }; > > #define SDHCI_TUNING_LOOP_COUNT 40 > diff --git a/include/mmc.h b/include/mmc.h > index 82562193cc48..05d8ab8eeac6 100644 > --- a/include/mmc.h > +++ b/include/mmc.h > @@ -360,6 +360,19 @@ enum mmc_voltage { > #define MMC_NUM_BOOT_PARTITION 2 > #define MMC_PART_RPMB 3 /* RPMB partition number */ > > +/* timing specification used */ > +#define MMC_TIMING_LEGACY 0 > +#define MMC_TIMING_MMC_HS 1 > +#define MMC_TIMING_SD_HS 2 > +#define MMC_TIMING_UHS_SDR12 3 > +#define MMC_TIMING_UHS_SDR25 4 > +#define MMC_TIMING_UHS_SDR50 5 > +#define MMC_TIMING_UHS_SDR104 6 > +#define MMC_TIMING_UHS_DDR50 7 > +#define MMC_TIMING_MMC_DDR52 8 > +#define MMC_TIMING_MMC_HS200 9 > +#define MMC_TIMING_MMC_HS400 10 > + > /* Driver model support */ > > /** >