Page MenuHomeFreeBSD
Differential D16472

release: arm: Enable multicons for arm64
ClosedPublic
Actions

Authored by manu on Jul 27 2018, 5:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 10:18 PM
Unknown Object (File)
Wed, Apr 10, 12:34 AM
Unknown Object (File)
Tue, Apr 9, 8:51 PM
Unknown Object (File)
Fri, Apr 5, 1:26 AM
Unknown Object (File)
Mar 20 2024, 12:27 PM
Unknown Object (File)
Mar 20 2024, 12:27 PM
Unknown Object (File)
Jan 23 2024, 5:38 AM
Unknown Object (File)
Jan 6 2024, 11:06 AM
View All 42 Files
Subscribers
andrew
imp

Details

Reviewers
lwhsu
gjb
Group Reviewers
releng
Commits
rS336999: release: arm: Enable multicons for arm64
Summary

Since we have now EFI framebuffer enabled for ARM64 if we boot on a board
with an screen, u-boot will set up a EFI GOP framebuffer and we won't boot
using the serial console.
Also on RPI3 the firmware always setup the framebuffer area resulting in u-boot
always setup the EFI GOP and FreeBSD never using the serial console.




Reported by: lwshu
Test Plan
Build release image with the change and tested on RPI3 without an hdmi screen attached.
Diff Detail
Repository 
rS FreeBSD src repository - subversion 
Lint 
Lint Not Applicable
 
Unit 
Tests Not Applicable
 
Event Timeline
manu created this revision.Jul 27 2018, 5:39 PM
Herald added subscribers: andrew, imp.  ·  View Herald TranscriptJul 27 2018, 5:39 PM
Harbormaster completed remote builds in B18373: Diff 45917.Jul 27 2018, 5:39 PM
lwhsu accepted this revision.Jul 27 2018, 5:43 PM
LGTM, although I don't have a src bit right now. :-)
This revision is now accepted and ready to land.Jul 27 2018, 5:43 PM
manu added inline comments.Jul 27 2018, 11:25 PM
release/tools/arm.subr


131 ↗(On Diff #45917)
It needs EMBEDDED_TARGET_ARCH, running build to see if everything is ok.
manu updated this revision to Diff 45935.Jul 28 2018, 2:43 AM
manu edited the test plan for this revision. (Show Details)
This revision now requires review to proceed.Jul 28 2018, 2:43 AM
Harbormaster completed remote builds in B18388: Diff 45935.Jul 28 2018, 2:43 AM
manu updated this revision to Diff 45951.Jul 28 2018, 3:03 PM
Add boot_serial too so all kernel messages appears on the serial console.
Harbormaster completed remote builds in B18393: Diff 45951.Jul 28 2018, 3:03 PM
imp added inline comments.Jul 28 2018, 7:33 PM
release/tools/arm.subr


131 ↗(On Diff #45951)
We could also do this for arm, no? Those systems with video will see it, and it won't hurt those w/o.


140 ↗(On Diff #45951)
This is  good.
manu added inline comments.Jul 28 2018, 7:37 PM
release/tools/arm.subr


131 ↗(On Diff #45951)
It's unneeded as I only implemented efifb for arm64 due to pmap difference between arm and arm64.
imp added inline comments.Jul 28 2018, 7:56 PM
release/tools/arm.subr


131 ↗(On Diff #45951)
I thought we had other frame buffer support for RPi, Allwinner and BBB, which would benefit from this as well...
manu added inline comments.Jul 28 2018, 8:01 PM
release/tools/arm.subr


131 ↗(On Diff #45951)
Allwinner is dead because nobody care to push the (wrong) dts binding we hacked at the time.

For RPI and BBB those are VT driver that come up later in the boot process. This is not what this review is attempting to fix (I don't even know if it makes a difference on those plateform).
gjb accepted this revision.Jul 31 2018, 7:04 PM
This revision is now accepted and ready to land.Jul 31 2018, 7:04 PM
Closed by commit rS336999: release: arm: Enable multicons for arm64 (authored by manu).  ·  Explain WhyJul 31 2018, 7:14 PM
This revision was automatically updated to reflect the committed changes.
Revision Contents
Changeset List
PathSize
head/
release/
tools/
14 lines
Diff 46095
View Options
head/release/tools/arm.subr
Loading...