Page MenuHomeFreeBSD
Differential D1054

Start to allow platforms other than U-Boot to use the FDT code in loader
ClosedPublic
Actions

Authored by andrew on Oct 31 2014, 4:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 1:46 PM
Unknown Object (File)
Thu, Dec 12, 10:58 AM
Unknown Object (File)
Wed, Nov 27, 12:16 PM
Unknown Object (File)
Nov 22 2024, 10:29 AM
Unknown Object (File)
Nov 20 2024, 11:41 PM
Unknown Object (File)
Nov 17 2024, 7:13 AM
Unknown Object (File)
Nov 17 2024, 1:00 AM
Unknown Object (File)
Nov 11 2024, 7:51 PM
View All Files
Subscribers
• ian
rpaulo

Details

Reviewers
rpaulo
• ian
Group Reviewers
ARM
Commits
rS275763: MFC 273927:
rS273934: Start to allow platforms other than U-Boot to use the FDT code in loader by
Test Plan

Boot on a U-Boot plaform

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

andrew updated this revision to Diff 2187.Oct 31 2014, 4:43 PM
andrew retitled this revision from to Start to allow platforms other than U-Boot to use the FDT code in loader.
andrew updated this object.
andrew edited the test plan for this revision. (Show Details)
andrew added a reviewer: ARM.
andrew updated this revision to Diff 2188.Oct 31 2014, 4:47 PM

Add a missing function from the fdt platform header

andrew added a comment.Oct 31 2014, 6:13 PM

This will need a little more work. There is still U-Boot functions called from fdt code.

rpaulo accepted this revision.Nov 1 2014, 1:44 AM
rpaulo added a reviewer: rpaulo.
rpaulo added a subscriber: rpaulo.
rpaulo added inline comments.
sys/boot/uboot/fdt/Makefile
12 ↗(On Diff #2188)

Lots of empty spaces in this file.

This revision is now accepted and ready to land.Nov 1 2014, 1:44 AM
andrew updated this revision to Diff 2213.Nov 1 2014, 4:20 PM
andrew edited edge metadata.

Update to remove all uboot code from the fdt code

• ian accepted this revision.Nov 1 2014, 4:45 PM
• ian added a reviewer: • ian.
• ian added a subscriber: • ian.

Looks good. I'm not sure I would have created a whole new library with just one .o in it, but maybe more stuff will land in there in the future?

andrew added a comment.Nov 1 2014, 5:01 PM

The problem is we need to have the libraries in the correct order for linking and it's easier to create a library and have it ordered correctly.

andrew closed this revision.Nov 1 2014, 5:12 PM
andrew updated this revision to Diff 2214.

Closed by commit rS273934 (authored by @andrew).

Revision Contents
Changeset List

PathSize
head/
sys/
boot/
arm/
uboot/
5 lines
fdt/
3 lines
160 lines
23 lines
powerpc/
uboot/
5 lines
uboot/
6 lines
fdt/
33 lines
181 lines

Diff 2214

View Options

head/sys/boot/arm/uboot/Makefile

Loading...
View Options

head/sys/boot/fdt/Makefile

Loading...
View Options

head/sys/boot/fdt/fdt_loader_cmd.c

Loading...
View Options

head/sys/boot/fdt/fdt_platform.h

Loading...
View Options

head/sys/boot/powerpc/uboot/Makefile

Loading...
View Options

head/sys/boot/uboot/Makefile

Loading...
View Options

head/sys/boot/uboot/fdt/Makefile

Loading...
View Options

head/sys/boot/uboot/fdt/uboot_fdt.c

Loading...