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
F87218538: D1054.diff
Sun, Jun 30, 10:09 AM
Unknown Object (File)
Fri, Jun 28, 3:26 PM
Unknown Object (File)
Sat, Jun 22, 9:06 PM
Unknown Object (File)
May 4 2024, 11:06 AM
Unknown Object (File)
Apr 30 2024, 2:23 AM
Unknown Object (File)
Apr 30 2024, 1:56 AM
Unknown Object (File)
Apr 30 2024, 1:55 AM
Unknown Object (File)
Apr 30 2024, 1:52 AM
View All 75 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
No Lint Coverage
Unit
No Test Coverage

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
13

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
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 2213

View Options

sys/boot/arm/uboot/Makefile

Loading...
View Options

sys/boot/fdt/Makefile

Loading...
View Options

sys/boot/fdt/fdt_loader_cmd.c

Loading...
View Options

sys/boot/fdt/fdt_platform.h

Loading...
View Options

sys/boot/powerpc/uboot/Makefile

Loading...
View Options

sys/boot/uboot/Makefile

Loading...
View Options

sys/boot/uboot/fdt/Makefile

Loading...
View Options

sys/boot/uboot/fdt/uboot_fdt.c

Loading...