Page MenuHomeFreeBSD
Differential D32723

ipq4018: add SoC reset, SMP, reset syscon and qcom_rnd driver
ClosedPublic
Actions

Authored by adrian on Oct 29 2021, 3:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 25, 7:54 PM
Unknown Object (File)
Sat, Jan 25, 7:40 PM
Unknown Object (File)
Sat, Jan 25, 7:15 PM
Unknown Object (File)
Fri, Jan 24, 7:18 PM
Unknown Object (File)
Thu, Jan 23, 1:19 AM
Unknown Object (File)
Sun, Jan 19, 10:35 PM
Unknown Object (File)
Sat, Jan 18, 9:20 PM
Unknown Object (File)
Sat, Jan 18, 5:32 PM
View All Files
Subscribers
andrew
emaste
imp
manu

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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
adrian updated this revision to Diff 97738.Oct 30 2021, 4:22 PM
  • 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?

adrian updated this revision to Diff 97752.Oct 31 2021, 12:06 AM
  • 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
adrian updated this revision to 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 B42469: Diff 97762.Oct 31 2021, 2:53 AM
adrian updated this revision to Diff 97763.Oct 31 2021, 2:57 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
adrian updated this revision to 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 B42471: Diff 97764.Oct 31 2021, 3:46 AM
adrian updated this revision to Diff 97765.Oct 31 2021, 5:50 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 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.

adrian updated this revision to Diff 97903.Nov 3 2021, 5: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
  • 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 B42538: Diff 97903.Nov 3 2021, 5:53 AM
adrian updated this revision to Diff 98012.Nov 4 2021, 3:56 PM
  • ipq4018: add qcom-gcc-ipq4018 and dependencies into the build
Harbormaster completed remote builds in B42581: Diff 98012.Nov 4 2021, 3:56 PM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 4 2021, 4:03 PM
Closed by commit rG6325f105aad2: ipq4018: toggle ps-hold to allow SoC reset (authored by adrian). · Explain Why
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG6325f105aad2: ipq4018: toggle ps-hold to allow SoC reset.
adrian added a commit: rG1492c8c0d825: qcom_rnd: add initial qualcomm prng driver..
adrian added a commit: rGa516ccc4ae04: ipq4018: add SoC reset and qcom_rnd driver.
adrian added a commit: rG960e65d23aaa: qcom: add initial SCM legacy API.
adrian added a commit: rGd3514c294207: ipq401x: add MP core start-up path for the CPU regulator/clock gate used.
adrian added a commit: rGbc06496744a1: ipq401x: flip on SMP for the ASUS AC1300.
adrian added a commit: rGb12a863a1e14: ipq4018: add initial reset driver support for the clock/reset controller..
adrian added a commit: rGcfd06987029a: ipq4018: add qcom-gcc-ipq4018 and dependencies into the build.

Revision Contents
Changeset List

PathSize
sys/
arm/
qualcomm/
42 lines
3 lines

Diff 98013

View Options

sys/arm/qualcomm/ipq4018_machdep.c

Loading...
View Options

sys/arm/qualcomm/ipq4018_reg.h

Loading...