Differential D32154

MMCCAM - Convert driver to use the mmc_sim interface
ClosedPublic
Actions

Authored by hlh_restart.be on Sep 27 2021, 9:53 AM.

Tags

None

Referenced Files

	F168064356: D32154.id101838.diff
	Wed, Aug 26, 5:14 AM

	F168052179: D32154.id102353.diff
	Wed, Aug 26, 3:50 AM

	F168020584: D32154.diff
	Tue, Aug 25, 11:15 PM

	F168014645: D32154.id.diff
	Tue, Aug 25, 10:24 PM

	F167957178: D32154.id95774.diff
	Tue, Aug 25, 3:15 PM

	F167941776: D32154.id102353.diff
	Tue, Aug 25, 1:16 PM

	Unknown Object (File)
	Sun, Aug 23, 12:51 PM

	Unknown Object (File)
	Sat, Aug 22, 4:34 AM

Subscribers

Details

Reviewers

Commits

rG785a5b8d91ab: rtsx: Update driver version number to 2.1c
rG23566767c37e: rtsx: Do not display pci_read_config() errors during rtsx_init()
rGa1b4c74dadeb: rtsx: Add CTLFLAG_STATS flag for read and write counters
rG4efab785f6c4: rtsx: Prefer __FreeBSD_version over __FreeBSD__
rG545def055b4a: rtsx: Convert driver to use the mmc_sim interface
rGad494d3b2d4d: rtsx: Update driver version number to 2.1c
rG1e800a593445: rtsx: Do not display pci_read_config() errors during rtsx_init()
rGec1f122b56f9: rtsx: Add CTLFLAG_STATS flag for read and write counters
rG7e5933b33301: rtsx: Prefer __FreeBSD_version over __FreeBSD__
rG8e9740b62e95: rtsx: Convert driver to use the mmc_sim interface

Summary

A lot more generic cam related things are done in mmc_sim so this simplify
the driver a lot. (see https://reviews.freebsd.org/D27486)

Test Plan

add 'options MMCCAM' to kernel config

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 43898
Build 40786: arc lint + arc unit

Event Timeline

hlh_restart.be created this revision.Sep 27 2021, 9:53 AM

Herald added a subscriber: imp. · View Herald TranscriptSep 27 2021, 9:53 AM

hlh_restart.be requested review of this revision.Sep 27 2021, 9:53 AM

Harbormaster completed remote builds in B41781: Diff 95774.Sep 27 2021, 9:54 AM

Generally I think this looks good, modulo the comment about comments on #endif.

sys/dev/rtsx/rtsx.c
657	Here and elsewhere: style(9) says that this is incorrect. The comment at the endif line should match the #if line. #ifdef COMPAT_43 /* A large region here, or other conditional code. / #else / !COMPAT_43 / / Or here. / #endif / COMPAT_43 */

Correct style(9): comments on #endif

Harbormaster completed remote builds in B41864: Diff 96004.Sep 30 2021, 6:40 AM

hlh_restart.be marked an inline comment as done.Sep 30 2021, 6:41 AM

Don't display pci_read_config() errors during rtsx_init()
Add CTLFLAG_STATS for read and write counters

Harbormaster completed remote builds in B43898: Diff 101409.Jan 13 2022, 10:27 AM

LGTM

This revision is now accepted and ready to land.Jan 13 2022, 2:32 PM

Replace FreeBSD by __FreeBSD_version

This revision now requires review to proceed.Jan 24 2022, 9:35 AM

Harbormaster completed remote builds in B44062: Diff 101838.Jan 24 2022, 9:35 AM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 3 2022, 11:44 PM

Closed by commit rG8e9740b62e95: rtsx: Convert driver to use the mmc_sim interface (authored by hlh_restart.be, committed by jkim). · Explain Why

This revision was automatically updated to reflect the committed changes.

jkim added a commit: rG8e9740b62e95: rtsx: Convert driver to use the mmc_sim interface.

jkim added a commit: rG7e5933b33301: rtsx: Prefer __FreeBSD_version over __FreeBSD__.

jkim added a commit: rGec1f122b56f9: rtsx: Add CTLFLAG_STATS flag for read and write counters.

jkim added a commit: rG1e800a593445: rtsx: Do not display pci_read_config() errors during rtsx_init().

jkim added a commit: rGad494d3b2d4d: rtsx: Update driver version number to 2.1c.

mav added a commit: rG545def055b4a: rtsx: Convert driver to use the mmc_sim interface.Feb 23 2022, 1:06 AM

mav added a commit: rG4efab785f6c4: rtsx: Prefer __FreeBSD_version over __FreeBSD__.

mav added a commit: rGa1b4c74dadeb: rtsx: Add CTLFLAG_STATS flag for read and write counters.

mav added a commit: rG23566767c37e: rtsx: Do not display pci_read_config() errors during rtsx_init().

mav added a commit: rG785a5b8d91ab: rtsx: Update driver version number to 2.1c.

Revision Contents
Changeset List

Path

Size

sys/

dev/

rtsx/

302 lines

modules/

rtsx/

2 lines

Diff 101409

sys/dev/rtsx/rtsx.c

Loading...

sys/modules/rtsx/Makefile

Loading...