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
F173838939: D6269.id16061.diff
Mon, Sep 28, 7:44 PM
Unknown Object (File)
Mon, Sep 28, 8:04 AM
Unknown Object (File)
Mon, Sep 28, 1:23 AM
Unknown Object (File)
Sun, Sep 27, 7:09 PM
Unknown Object (File)
Wed, Sep 23, 5:18 AM
Unknown Object (File)
Tue, Sep 22, 6:01 PM
Unknown Object (File)
Tue, Sep 22, 2:11 AM
Unknown Object (File)
Mon, Sep 21, 1:48 PM
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...