Dear Neil, On 11/6/20 6:27 PM, Neil Armstrong wrote: > Amlogic SM1 SoCs doesn't handle very well high clocks from the DIV2 input > Thus we limit the max freq to 26MHz on SM1 SoCs until we handle higher > frequencies via the first input from a composite clock. > > Here 26MHz corresponds to MMC_HS clock speed.
When i have checked, it's working with 52MHz. mmc1(part 0) is current device Odroid N2> mmcinfo Device: mmc@ffe07000 Manufacturer ID: 15 OEM: 100 Name: BJTD4 Bus Speed: 52000000 Mode: MMC High Speed (52MHz) Rd Block Len: 512 MMC version 5.1 High Capacity: Yes Capacity: 29.1 GiB Bus Width: 8-bit Erase Group Size: 512 KiB HC WP Group Size: 8 MiB User Capacity: 29.1 GiB WRREL Boot Capacity: 4 MiB ENH RPMB Capacity: 4 MiB ENH Device: sd@ffe05000 Manufacturer ID: 3 OEM: 5344 Name: SB16G Bus Speed: 50000000 Mode: SD High Speed (50MHz) Rd Block Len: 512 SD version 3.0 High Capacity: Yes Capacity: 14.8 GiB Bus Width: 4-bit Erase Group Size: 512 Bytes Odroid N2> ums 0 mmc 0 If you can wait for more time, i can fix it. I think that it's better than applying this patch. But i don't know which boards are SM1 SoC..(Odroid-c4 and VIM3L?) Best Regards, Jaehoon Chung > > We also add a u-boot only sm1 compatible to distinguish the controller in a > new > meson-sm1-u-boot.dtsi and reworks the other -u-boot.dtsi to use this for SM1 > based boards. > > Finally a TOFIX is added to precise the clock management should use > the clock controller instead of local management with fixed clock rates. > > Neil Armstrong (3): > mmc: meson-gx: move arch header to local header > mmc: meson-gx: limit max frequency on SM1 SoCs > ARM: dts: meson-sm1: add u-boot specific MMC controller compatible > > .../meson-g12b-a311d-khadas-vim3-u-boot.dtsi | 1 + > arch/arm/dts/meson-khadas-vim3-u-boot.dtsi | 2 -- > .../dts/meson-sm1-khadas-vim3l-u-boot.dtsi | 1 + > arch/arm/dts/meson-sm1-odroid-c4-u-boot.dtsi | 2 +- > arch/arm/dts/meson-sm1-sei610-u-boot.dtsi | 2 +- > arch/arm/dts/meson-sm1-u-boot.dtsi | 20 +++++++++++++ > drivers/mmc/meson_gx_mmc.c | 28 ++++++++++++++++--- > .../sd_emmc.h => drivers/mmc/meson_gx_mmc.h | 10 ++++--- > 8 files changed, 54 insertions(+), 12 deletions(-) > create mode 100644 arch/arm/dts/meson-sm1-u-boot.dtsi > rename arch/arm/include/asm/arch-meson/sd_emmc.h => > drivers/mmc/meson_gx_mmc.h (95%) >