Details

Reviewers: None
Commits: rGcfd06987029a: ipq4018: add qcom-gcc-ipq4018 and dependencies into the build
rGb12a863a1e14: ipq4018: add initial reset driver support for the clock/reset controller.
rGbc06496744a1: ipq401x: flip on SMP for the ASUS AC1300
rGd3514c294207: ipq401x: add MP core start-up path for the CPU regulator/clock gate used
rG960e65d23aaa: qcom: add initial SCM legacy API
rGa516ccc4ae04: ipq4018: add SoC reset and qcom_rnd driver
rG1492c8c0d825: qcom_rnd: add initial qualcomm prng driver.
rG6325f105aad2: ipq4018: toggle ps-hold to allow SoC reset

Summary

MVP for doing an SoC reset
A simple qcom_rnd random entropy driver
A simple SCM implementation for doing SMC calls into the QCOM TZ firmware, enough for this chipset to set the cold boot address for MP
Bring the MPs up and out of reset to start SMP
The reset controller side of the gcc (clock + reset) hardware block

The registers and reset index -> register/bit came from Linux; I reimplemented all the code and comments myself.

Test Plan

ASUS RT-AC58U router, IPQ4019

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 42431
Build 39319: arc lint + arc unit

Event Timeline

adrian created this revision.Oct 29 2021, 3:17 AM

Herald added subscribers: andrew, imp. · View Herald TranscriptOct 29 2021, 3:17 AM

adrian requested review of this revision.Oct 29 2021, 3:17 AM

Harbormaster completed remote builds in B42431: Diff 97645.Oct 29 2021, 3:17 AM

qcom: add initial SCM legacy API
ipq401x: add MP core start-up path for the CPU regulator/clock gate used
ipq401x: flip on SMP for the ASUS AC1300

Harbormaster completed remote builds in B42458: Diff 97738.Oct 30 2021, 4:22 PM

emaste added a subscriber: emaste.Oct 30 2021, 8:49 PM

emaste added inline comments.

sys/arm/conf/ASUS_AC1300
41 ↗	(On Diff #97738)	still true?

ipq401x: flip on SMP for the ASUS AC1300

Harbormaster completed remote builds in B42467: Diff 97752.Oct 31 2021, 12:06 AM

adrian marked an inline comment as done.Oct 31 2021, 12:07 AM

qcom: add initial SCM legacy API
ipq401x: add MP core start-up path for the CPU regulator/clock gate used
ipq401x: flip on SMP for the ASUS AC1300

Harbormaster completed remote builds in B42469: Diff 97762.Oct 31 2021, 2:53 AM

qcom: add initial SCM legacy API
ipq401x: add MP core start-up path for the CPU regulator/clock gate used
ipq401x: flip on SMP for the ASUS AC1300

Harbormaster completed remote builds in B42470: Diff 97763.Oct 31 2021, 2:57 AM

ipq4018: add initial reset driver support for the clock/reset controller.
ipq4018: add qcom-gcc-ipq4018 and dependencies into the build

Harbormaster completed remote builds in B42471: Diff 97764.Oct 31 2021, 3:46 AM

ipq4018: add initial reset driver support for the clock/reset controller.
ipq4018: add qcom-gcc-ipq4018 and dependencies into the build

Harbormaster completed remote builds in B42472: Diff 97765.Oct 31 2021, 5:50 AM

adrian retitled this revision from ipq4018: add SoC reset and qcom_rnd driver to ipq4018: add SoC reset, SMP, reset syscon and qcom_rnd driver.Oct 31 2021, 6:00 AM

adrian edited the summary of this revision. (Show Details)

adrian added inline comments.Nov 2 2021, 2:50 AM

sys/arm/qualcomm/qcom_gcc_ipq4018.c
150 ↗	(On Diff #97765)	is this the right api for fdt reset resources? or something else?
sys/arm/qualcomm/qcom_scm_legacy_defs.h
87 ↗	(On Diff #97765)	hey i left myself a comment to address before landing!

manu added a subscriber: manu.Nov 2 2021, 7:16 AM

manu added inline comments.

sys/arm/qualcomm/qcom_cpu_kpssv2.c
103 ↗	(On Diff #97765)	No need for those checks as you check below that the properties exists and are correct.
sys/arm/qualcomm/qcom_gcc_ipq4018.c
90 ↗	(On Diff #97765)	style(9) no {} for singe if clause.
94 ↗	(On Diff #97765)	Same
150 ↗	(On Diff #97765)	yep it's the correct one.

andrew added inline comments.Nov 2 2021, 10:41 AM

sys/arm/qualcomm/qcom_cpu_kpssv2.c
31 ↗	(On Diff #97765)	If you're not planning on an MFC to `stable/12` you can drop this in new files.
59–60 ↗	(On Diff #97765)	`DELAY` should work from just before the `platform_attach` function is called.
sys/arm/qualcomm/qcom_gcc_ipq4018.c
90 ↗	(On Diff #97765)	It was changed to allow them in R10:abb14e5deda396bc3fb070ec80cb8a6ced66dd87
163–165 ↗	(On Diff #97765)	Do you need to pass `qcom_gcc_ipq4018_modevent` here? Other drivers use `0`.

adrian marked 5 inline comments as done.Nov 3 2021, 5:53 AM

adrian added inline comments.

sys/arm/qualcomm/qcom_cpu_kpssv2.c
59–60 ↗	(On Diff #97765)	it doesn't, I've tried. We should totally try to fix it in a different commit stack. :)
sys/arm/qualcomm/qcom_gcc_ipq4018.c
163–165 ↗	(On Diff #97765)	I'm .. not sure if I want to support unloading the driver later on. I'll delete the mod deletion stuff later on from these drivers if I decide that it's not worth it.

qcom: add initial SCM legacy API
ipq401x: add MP core start-up path for the CPU regulator/clock gate used
ipq401x: flip on SMP for the ASUS AC1300
ipq4018: add initial reset driver support for the clock/reset controller.
ipq4018: add qcom-gcc-ipq4018 and dependencies into the build