+ 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 >