Page MenuHomeFreeBSD
Differential D44026

JH7110 MMC driver
ClosedPublic
Actions

Authored by mhorne on Feb 22 2024, 8:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 8, 5:39 AM
Unknown Object (File)
Wed, May 8, 5:39 AM
Unknown Object (File)
Wed, May 8, 5:39 AM
Unknown Object (File)
Wed, May 8, 5:39 AM
Unknown Object (File)
Wed, May 8, 5:39 AM
Unknown Object (File)
Wed, May 8, 5:39 AM
Unknown Object (File)
Wed, May 8, 2:06 AM
Unknown Object (File)
Tue, May 7, 9:38 PM
View All 11 Files
Subscribers
imp
jrtc27

Details

Reviewers
jsihv_gmx.com
Group Reviewers
riscv
Commits
rGc8b472aa4b9d: jh7110: enable MMC driver

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jsihv_gmx.com created this revision.Feb 22 2024, 8:41 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 22 2024, 8:41 PM
jsihv_gmx.com requested review of this revision.Feb 22 2024, 8:41 PM
jsihv_gmx.com added a reviewer: riscv.Feb 22 2024, 9:12 PM
jsihv_gmx.com updated this revision to Diff 135222.Feb 29 2024, 2:40 PM

Fixing obvious style issues

mhorne added inline comments.Mar 7 2024, 5:22 PM
sys/dev/mmc/host/dwmmc_starfive.c
46

Where does this number come from?

jsihv_gmx.com added inline comments.Mar 7 2024, 6:35 PM
sys/dev/mmc/host/dwmmc_starfive.c
46

It's the actual frequency this clock has in vendor's kernel

jrtc27 added a subscriber: jrtc27.Mar 7 2024, 7:26 PM
jrtc27 added inline comments.
sys/dev/mmc/host/dwmmc_starfive.c
46

Why isn't it using the frequency in the mmc_ios?

jsihv_gmx.com added inline comments.Mar 9 2024, 10:53 PM
sys/dev/mmc/host/dwmmc_starfive.c
46

I had made tests with more complicated algorithms here. At some point merely reading ios->clock caused a crash. Somehow I had eventually dispensed a now obvious looking solution to get the frequency from there.

jsihv_gmx.com updated this revision to Diff 135570.Mar 9 2024, 10:54 PM

doing it like it should be done

mhorne commandeered this revision.Wed, Apr 17, 5:32 PM
mhorne edited reviewers, added: jsihv_gmx.com; removed: mhorne.

Taking this over so I can publish some additions/changes...

mhorne updated this revision to Diff 137266.Wed, Apr 17, 5:33 PM

Add check for generic compatible string exported by current u-boot.

Harbormaster completed remote builds in B57147: Diff 137266.Wed, Apr 17, 5:33 PM
This revision was not accepted when it landed; it landed in state Needs Review.Tue, May 7, 4:05 PM
Closed by commit rGc8b472aa4b9d: jh7110: enable MMC driver (authored by mhorne). · Explain Why
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rGc8b472aa4b9d: jh7110: enable MMC driver.

Revision Contents
Changeset List

PathSize
sys/
dev/
mmc/
host/
110 lines

Diff 135222

View Options

sys/dev/mmc/host/dwmmc_starfive.c

Loading...