Re: [PATCH] mmc_cmds: add HS400 data rates

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+ Avri

On Thu, 2 Dec 2021 at 17:19, Quentin Schulz
<quentin.schulz@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> JEDEC 5.1 JESD84-B50.1 DEVICE_TYPE [196] specifies that bit 6 is for
> "HS400 Dual Data Rate e•MMC at 200 MHz – 1.8 V I/O" and bit 7 for
> "HS400 Dual Data Rate e•MMC at 200 MHz – 1.2 V I/O" so let's add those.
>
> Cc: Quentin Schulz <foss+kernel@xxxxxxxxx>

Future wise, please don't use this to cc yourself another email. No
need to resend this time.

> Signed-off-by: Quentin Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxxxxx>

Even if the patch is trivial and looks good to me, I have added Avri
who helps to maintain mmc-utils, to make sure there are no objections.

Kind regards
Uffe

> ---
>  mmc_cmds.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/mmc_cmds.c b/mmc_cmds.c
> index 73bd32a..769e317 100644
> --- a/mmc_cmds.c
> +++ b/mmc_cmds.c
> @@ -1613,6 +1613,8 @@ int do_read_extcsd(int nargs, char **argv)
>         /* DEVICE_TYPE in A45, CARD_TYPE in A441 */
>         reg = ext_csd[196];
>         printf("Card Type [CARD_TYPE: 0x%02x]\n", reg);
> +       if (reg & 0x80) printf(" HS400 Dual Data Rate eMMC @200MHz 1.2VI/O\n");
> +       if (reg & 0x40) printf(" HS400 Dual Data Rate eMMC @200MHz 1.8VI/O\n");
>         if (reg & 0x20) printf(" HS200 Single Data Rate eMMC @200MHz 1.2VI/O\n");
>         if (reg & 0x10) printf(" HS200 Single Data Rate eMMC @200MHz 1.8VI/O\n");
>         if (reg & 0x08) printf(" HS Dual Data Rate eMMC @52MHz 1.2VI/O\n");
> --
> 2.33.1
>




[Index of Archives]     [Linux Memonry Technology]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux