Page MenuHomeFreeBSD

Switch Rockchip RK3188 SoC to use the platform code
ClosedPublic

Authored by ganbold on Dec 13 2016, 10:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 29, 11:05 AM
Unknown Object (File)
Mon, Jan 27, 5:46 PM
Unknown Object (File)
Fri, Jan 24, 6:02 PM
Unknown Object (File)
Tue, Jan 21, 5:53 PM
Unknown Object (File)
Dec 15 2024, 5:21 AM
Unknown Object (File)
Dec 8 2024, 12:31 PM
Unknown Object (File)
Nov 24 2024, 4:31 PM
Unknown Object (File)
Nov 24 2024, 4:02 PM
Subscribers

Diff Detail

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

Event Timeline

ganbold retitled this revision from to Switch Rockchip RK3188 SoC to use the platform code.
ganbold updated this object.
ganbold edited the test plan for this revision. (Show Details)
ganbold added a reviewer: andrew.
ganbold set the repository for this revision to rS FreeBSD src repository - subversion.
ganbold added a subscriber: ARM.

Include changes to RK3188 config

manu added inline comments.
sys/arm/rockchip/rk30xx_machdep.c
93 ↗(On Diff #22862)

It should be in a #ifdef SOC_RK3188

98 ↗(On Diff #22862)

#ifdef SMP

Added SOC_ROCKCHIP_RK3188 option.

Added SOC_ROCKCHIP_RK3188 option and SMP check.

andrew edited edge metadata.

Consider my comments as suggestions for future improvements.

/usr/src/sys/conf/options.arm
67 ↗(On Diff #22865)

I'm wondering if we should start moving to opt_soc.h on arm.

sys/arm/conf/RK3188
26 ↗(On Diff #22865)

There seems to be inconsistent indenting here.

sys/arm/rockchip/rk30xx_machdep.c
55–60 ↗(On Diff #22865)

This should probably be the default implementation

This revision is now accepted and ready to land.Dec 13 2016, 11:16 AM
This revision was automatically updated to reflect the committed changes.