Differential D8793

jz4780: add driver for SMB controller
ClosedPublic
Actions

Authored by jmcneill on Dec 14 2016, 11:18 AM.

Tags

Referenced Files

	Unknown Object (File)
	Sat, Nov 30, 5:08 PM

	Unknown Object (File)
	Oct 18 2024, 12:44 AM

	Unknown Object (File)
	Oct 17 2024, 6:27 PM

	Unknown Object (File)
	Oct 12 2024, 6:22 PM

	Unknown Object (File)
	Oct 12 2024, 5:31 PM

	Unknown Object (File)
	Oct 3 2024, 3:36 PM

	Unknown Object (File)
	Oct 1 2024, 3:56 PM

	Unknown Object (File)
	Oct 1 2024, 3:28 PM

View All 84 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rS310172: Add support for Ingenic JZ4780 SMBus controller.

Summary

This patch adds support for the JZ4780 SMB controller.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

jmcneill updated this revision to Diff 22909.Dec 14 2016, 11:18 AM

jmcneill retitled this revision from to jz4780: add driver for SMB controller.

jmcneill updated this object.

jmcneill edited the test plan for this revision. (Show Details)

jmcneill added reviewers: kan, MIPS.

jmcneill set the repository for this revision to rS FreeBSD src repository - subversion.

jmcneill added a project: MIPS.

Herald added a subscriber: imp. · View Herald TranscriptDec 14 2016, 11:18 AM

kan added inline comments.Dec 14 2016, 3:06 PM

sys/mips/ingenic/jz4780_smb.c
142	oldaddr is unused?
248	timevalsub ? Or use timespec/getnanouptime and use timespecsub macro instead of rolling own?
360	Do thins only on error?

jmcneill added inline comments.Dec 14 2016, 4:11 PM

sys/mips/ingenic/jz4780_smb.c
142	Mirrored signature from jzsmb_reset, I will remove it.
248	Is there a tstohz somewhere? I couldn't find one.
360	Clearing the stop-hold bit causes the controller to send an I2C STOP condition. We need this in the success path too. It's a (harmless) noop only when we are performing 0-byte transfers.

Remove unused parameters in jzsmb_reset_locked
Replace timeval usage with timespec

jmcneill marked 6 inline comments as done.Dec 14 2016, 10:47 PM

kan accepted this revision.Dec 15 2016, 4:37 PM

kan edited edge metadata.

This revision is now accepted and ready to land.Dec 15 2016, 4:37 PM

Closed by commit rS310172: Add support for Ingenic JZ4780 SMBus controller. (authored by jmcneill). · Explain WhyDec 16 2016, 8:05 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

mips/

ingenic/

98 lines

484 lines

Diff 22909

sys/mips/ingenic/jz4780_smb.h

Loading...

sys/mips/ingenic/jz4780_smb.c

Loading...