Hi Heiko, On 2024-11-21 15:27, Heiko Stuebner wrote: > Add support for RK3576 to the rockchip sdhci driver. > > It's pretty similar to its cousins found in the RK3568 and RK3588 and the > specific hs400-tx-tap number was taken from the vendor-u-boot. > > Signed-off-by: Heiko Stuebner <he...@sntech.de> > --- > drivers/mmc/rockchip_sdhci.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/mmc/rockchip_sdhci.c b/drivers/mmc/rockchip_sdhci.c > index da630b9d97a..9571e7d66c9 100644 > --- a/drivers/mmc/rockchip_sdhci.c > +++ b/drivers/mmc/rockchip_sdhci.c > @@ -656,6 +656,14 @@ static const struct sdhci_data rk3568_data = { > .hs400_txclk_tapnum = 0x8, > }; > > +static const struct sdhci_data rk3576_data = { > + .set_ios_post = rk3568_sdhci_set_ios_post, > + .set_clock = rk3568_sdhci_set_clock, > + .config_dll = rk3568_sdhci_config_dll, > + .hs200_txclk_tapnum = DLL_TXCLK_TAPNUM_DEFAULT, > + .hs400_txclk_tapnum = 0x7,
Please add following to fix HS200+ mode together with [1] and [2]. .hs400_cmdout_tapnum = 0x7, .hs400_strbin_tapnum = 0x5, .ddr50_strbin_delay_num = 0xa, [1] https://patchwork.ozlabs.org/patch/2038152/ [2] https://patchwork.ozlabs.org/patch/2038155/ Regards, Jonas > +}; > + > static const struct sdhci_data rk3588_data = { > .set_ios_post = rk3568_sdhci_set_ios_post, > .set_clock = rk3568_sdhci_set_clock, > @@ -673,6 +681,10 @@ static const struct udevice_id sdhci_ids[] = { > .compatible = "rockchip,rk3568-dwcmshc", > .data = (ulong)&rk3568_data, > }, > + { > + .compatible = "rockchip,rk3576-dwcmshc", > + .data = (ulong)&rk3576_data, > + }, > { > .compatible = "rockchip,rk3588-dwcmshc", > .data = (ulong)&rk3588_data,