Page MenuHomeFreeBSD
Differential D48286

arm64: Add a new SOC_ROCKCHIP option
AcceptedPublic
Actions

Authored by manu on Thu, Jan 2, 12:52 PM.
Tags
None
Referenced Files
F106831156: D48286.diff
Mon, Jan 6, 3:04 AM
F106804104: D48286.diff
Sun, Jan 5, 3:58 PM
Unknown Object (File)
Fri, Jan 3, 10:50 PM
Subscribers
emaste
imp

Details

Reviewers
andrew
Group Reviewers
arm64
Summary

A lot of drivers are shared between all rockchip SoCs, each time we add
suppot for a new SoC we need to add the options in the files.arm64 lines.
Add a new option SOC_ROCKCHIP that will help simplify this file.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 61456
Build 58340: arc lint + arc unit

Event Timeline

manu created this revision.Thu, Jan 2, 12:52 PM
Herald added a reviewer: andrew. · View Herald TranscriptThu, Jan 2, 12:52 PM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a reviewer: andrew. · View Herald Transcript
Herald added subscribers: emaste, imp. · View Herald Transcript
manu requested review of this revision.Thu, Jan 2, 12:52 PM
Harbormaster completed remote builds in B61456: Diff 148670.Thu, Jan 2, 12:52 PM
manu added a reviewer: arm64.Thu, Jan 2, 12:54 PM
andrew added a comment.Thu, Jan 2, 4:07 PM

Can you add a comment that SOC_ROCKCHIP_* depend on SOC_ROCKCHIP

andrew accepted this revision.Thu, Jan 2, 4:07 PM
This revision is now accepted and ready to land.Thu, Jan 2, 4:07 PM
manu added a comment.Thu, Jan 2, 7:33 PM
In D48286#1101250, @andrew wrote:

Can you add a comment that SOC_ROCKCHIP_* depend on SOC_ROCKCHIP

Sure, where do you think it would be best, in options.arm64 ?

andrew added a comment.Fri, Jan 3, 1:45 PM

Yes, that's probably the best place for it

Revision Contents
Changeset List

PathSize
sys/
arm64/
conf/
1 line
conf/
30 lines
1 line

Diff 148670

View Options

sys/arm64/conf/std.rockchip

Loading...
View Options

sys/conf/files.arm64

Loading...
View Options

sys/conf/options.arm64

Loading...