Re: [PATCH v47 12/19] hw/sd/sdcard: Simplify EXT_CSD values for spec v4.3

[Philippe Mathieu-Daudé]

On 9/7/24 17:43, Cédric Le Goater wrote:
> On 7/9/24 5:25 PM, Philippe Mathieu-Daudé wrote:
> > - Set some keys to not defined / implemented:
> >    . EXT_CSD_HPI_FEATURES
> >    . EXT_CSD_BKOPS_SUPPORT
> >    . EXT_CSD_SEC_FEATURE_SUPPORT
> >    . EXT_CSD_ERASE_TIMEOUT_MULT
> >    . EXT_CSD_PART_SWITCH_TIME
> >    . EXT_CSD_OUT_OF_INTERRUPT_TIME
> >
> > - Simplify:
> >    . EXT_CSD_ACC_SIZE (6 -> 1)
> >        16KB of super_page_size -> 512B (BDRV_SECTOR_SIZE)
> >    . EXT_CSD_HC_ERASE_GRP_SIZE (4 -> 1)
> >    . EXT_CSD_HC_WP_GRP_SIZE (4 -> 1)
> >    . EXT_CSD_S_C_VCC[Q] (8 -> 1)
> >    . EXT_CSD_S_A_TIMEOUT (17 -> 1)
> >    . EXT_CSD_CARD_TYPE (7 -> 3)
> >        Dual data rate -> High-Speed mode
> >
> > - Update:
> >    . EXT_CSD_CARD_TYPE (7 -> 3)
> >        High-Speed MultiMediaCard @ 26MHz & 52MHz
> >    . Performances (0xa -> 0x46)
> >        Class B at 3MB/s. -> Class J at 21MB/s
> >    . EXT_CSD_REV (5 -> 3)
> >        Rev 1.5 (spec v4.41) -> Rev 1.3 (spec v4.3)
> >
> > - Use load/store API to set EXT_CSD_SEC_CNT
> >
> > - Remove R/W keys, normally zeroed at reset
> >    . EXT_CSD_BOOT_INFO
> >
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>
> This should be merged in patch 10.

OK.