[PATCH] mmc: rockchip_sdhci: Revert 4 blocks PIO mode read limit for RK35xx

Dragan Simic dsimic at manjaro.org
Wed Mar 27 11:00:26 CET 2024


On 2024-03-27 09:56, Quentin Schulz wrote:
> On 3/27/24 00:35, Jonas Karlman wrote:
>> The commit 2cc6cde647e2 ("mmc: rockchip_sdhci: Limit number of blocks
>> read in a single command") introduced a limit of number of blocks to
>> read to fix a Data End Bit Error on RK3568 and RK3588. This had a side
>> affect of significant slowing down reading FIT from eMMC.
>> 
>> After the commit 6de9d7b2f13c ("rockchip: rk35xx: Enable eMMC HS200 
>> mode
>> by default") the limit of number of blocks to read workaround is no
>> longer necessary and a Data End Bit Error is no longer happening using
>> PIO mode.
>> 
>> Revert this limitation to allow reading more than 4 blocks with a 
>> single
>> CMD18 command in PIO mode and speed up reading FIT from eMMC.
> 
> Should we instead keep this code but surround it with
> #if !IS_ENABLED(MMC_HS200_SUPPORT)
> ?
> 
> After all, HS200 is only implied for RK35xx boards, so one could 
> disable it.
> 
> Also wondering whether we should do this dynamically based on the MMC
> mode that could be negotiated with the chip? E.g. I think it's not
> guaranteed that with hs200 support built and enabled in DT that it'll
> be negotiated as HS200?

Totally agreed with Quentin's remarks.


More information about the U-Boot mailing list