Differential D44026

JH7110 MMC driver
ClosedPublic
Actions

Authored by mhorne on Feb 22 2024, 8:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 16, 5:20 AM

	Unknown Object (File)
	Fri, Nov 15, 8:53 PM

	Unknown Object (File)
	Fri, Nov 15, 1:37 PM

	Unknown Object (File)
	Fri, Nov 15, 3:10 AM

	Unknown Object (File)
	Fri, Nov 15, 12:33 AM

	Unknown Object (File)
	Thu, Nov 14, 11:38 PM

	Unknown Object (File)
	Fri, Nov 8, 3:59 PM

	Unknown Object (File)
	Fri, Nov 8, 7:27 AM

View All 49 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rGc8b472aa4b9d: jh7110: enable MMC driver

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 57147
Build 54035: arc lint + arc unit

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

Fixing obvious style issues

mhorne added inline comments.Mar 7 2024, 5:22 PM

sys/dev/mmc/host/dwmmc_starfive.c
47	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
47	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
47	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
47	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.

doing it like it should be done

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

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

Harbormaster completed remote builds in B57147: Diff 137266.Apr 17 2024, 5:33 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 7 2024, 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

Path

Size

sys/

dev/

mmc/

host/

dwmmc_starfive.c

115 lines

riscv/

conf/

4 lines

starfive/

1 line

Diff 137266

sys/dev/mmc/host/dwmmc_starfive.c

Loading...

sys/riscv/conf/std.starfive

Loading...

sys/riscv/starfive/files.starfive

Loading...