Page MenuHomeFreeBSD
Differential D6269

Allwinner MP merge a20_mp_start_ap and a31_mp_start_ap
ClosedPublic
Actions

Authored by manu on May 9 2016, 3:34 PM.
Tags
Referenced Files
F150326754: D6269.diff
Tue, Mar 31, 5:37 AM
Unknown Object (File)
Mon, Mar 30, 6:40 AM
Unknown Object (File)
Mon, Mar 30, 3:38 AM
Unknown Object (File)
Sun, Mar 29, 7:42 PM
Unknown Object (File)
Fri, Mar 27, 2:30 PM
Unknown Object (File)
Thu, Mar 26, 9:00 AM
Unknown Object (File)
Thu, Mar 26, 7:38 AM
Unknown Object (File)
Wed, Mar 25, 5:44 AM
View All Files
Subscribers
andrew
imp
jmcneill

Details

Reviewers
jmcneill
Group Reviewers
ARM
Commits
rS299380: Merge a20_mp_start_ap and a31_mp_start_ap into one function.
Summary

Merge a20_mp_start_ap and a31_mp_start_ap into one function.
This function works with all smp non-multicluster allwinner SoC (A20, A31, A31S and H3).

Test Plan

Tested on A20 and A31S

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

manu updated this revision to Diff 16055.May 9 2016, 3:34 PM
manu retitled this revision from to Allwinner MP merge a20_mp_start_ap and a31_mp_start_ap.
manu updated this object.
manu edited the test plan for this revision. (Show Details)
manu added a reviewer: ARM.
manu set the repository for this revision to rS FreeBSD src repository - subversion.
manu added a project: ARM.
Herald added subscribers: andrew, imp. · View Herald TranscriptMay 9 2016, 3:34 PM
jmcneill added a subscriber: jmcneill.May 9 2016, 4:01 PM
jmcneill added inline comments.
sys/arm/allwinner/aw_mp.c
200 ↗(On Diff #16055)

Unmapping the same handle twice, this should be prcm instead?

manu added inline comments.May 9 2016, 4:03 PM
sys/arm/allwinner/aw_mp.c
200 ↗(On Diff #16055)

Damn, yeah, let me fix this ...

manu updated this revision to Diff 16061.May 9 2016, 4:06 PM

Unmap prcm and not cpucfg a second time.

manu marked 2 inline comments as done.May 9 2016, 4:06 PM
jmcneill added inline comments.May 9 2016, 4:08 PM
sys/arm/allwinner/aw_mp.c
114 ↗(On Diff #16061)

Do you still need to initialize this to 0? You test soc_family later on instead of prcm now.

manu updated this revision to Diff 16067.May 9 2016, 4:25 PM

No need to initialize prcm variable.

manu marked an inline comment as done.May 9 2016, 4:26 PM
jmcneill added inline comments.May 9 2016, 4:28 PM
sys/arm/allwinner/aw_mp.c
124 ↗(On Diff #16067)

Should be } else { on a single line.

manu updated this revision to Diff 16098.May 10 2016, 3:38 AM

Fix style(9)

manu marked an inline comment as done.May 10 2016, 3:38 AM
jmcneill accepted this revision.May 10 2016, 8:31 AM
jmcneill added a reviewer: jmcneill.
This revision is now accepted and ready to land.May 10 2016, 8:31 AM
Closed by commit rS299380: Merge a20_mp_start_ap and a31_mp_start_ap into one function. (authored by manu). · Explain WhyMay 10 2016, 6:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
arm/
allwinner/
6 lines
3 lines
61 lines

Diff 16150

View Options

head/sys/arm/allwinner/allwinner_machdep.c

Loading...
View Options

head/sys/arm/allwinner/aw_mp.h

Loading...
View Options

head/sys/arm/allwinner/aw_mp.c

Loading...