Differential D37882

QCOM: various bits
Needs ReviewPublic
Actions

Authored by bz on Dec 27 2022, 10:41 AM.

Tags

None

Referenced Files

	F156401907: D37882.id114570.diff
	Wed, May 13, 10:40 AM

	Unknown Object (File)
	Wed, May 13, 5:38 AM

	Unknown Object (File)
	Mon, May 4, 2:52 PM

	Unknown Object (File)
	Sun, May 3, 4:09 PM

	Unknown Object (File)
	Thu, Apr 30, 8:03 AM

	Unknown Object (File)
	Thu, Apr 30, 5:52 AM

	Unknown Object (File)
	Wed, Apr 29, 3:23 PM

	Unknown Object (File)
	Wed, Apr 29, 3:15 PM

View All 70 Files

Subscribers

Details

Reviewers

adrian
manu

Summary

NOT INTENDED TO BE COMMITTED LIKE THIS.

add files to the build so they are (a) usable and (b) do not rot by not getting compiled
fix an insta-panic in qcom_mdio_ipq4018 due to a missing bus function.
cleanup the qcom_rnd some in order to move forward for more support, less racyness and less panicing.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 48891
Build 45780: arc lint + arc unit

Event Timeline

bz created this revision.Dec 27 2022, 10:41 AM

Herald added a reviewer: manu. · View Herald TranscriptDec 27 2022, 10:41 AM

Herald added subscribers: emaste, andrew, imp. · View Herald Transcript

bz requested review of this revision.Dec 27 2022, 10:41 AM

Harbormaster completed remote builds in B48891: Diff 114570.Dec 27 2022, 10:41 AM

Just put this up so I won't lose it. The qcom_mdio_ipq4018.c 1-line change could probably go in along with adding that file to the build.

Thanks for the cleanup!

bz mentioned this in rGbfed2417f472: qcom: fix panic in qcom_mdio_ipq4018 implementation.Aug 11 2023, 2:13 AM

Revision Contents
Changeset List

Path

Size

sys/

arm64/

conf/

2 lines

conf/

2 lines

dev/

qcom_mdio/

qcom_mdio_ipq4018.c

3 lines

qcom_rnd/

211 lines

Diff 114570

sys/arm64/conf/std.qcom

Loading...

sys/conf/files.arm64

Loading...

sys/dev/qcom_mdio/qcom_mdio_ipq4018.c

Loading...

sys/dev/qcom_rnd/qcom_rnd.c

Loading...