Page MenuHomeFreeBSD

Enable parsing simple-bus 'ranges' with multiple entries
ClosedPublic

Authored by mw on Aug 5 2017, 10:10 AM.
Tags
Referenced Files
Unknown Object (File)
Mon, Nov 10, 6:07 AM
Unknown Object (File)
Fri, Oct 31, 12:47 PM
Unknown Object (File)
Mon, Oct 27, 8:47 PM
Unknown Object (File)
Mon, Oct 27, 4:07 PM
Unknown Object (File)
Tue, Oct 21, 5:11 PM
Unknown Object (File)
Tue, Oct 21, 5:10 PM
Unknown Object (File)
Tue, Oct 21, 4:56 PM
Unknown Object (File)
Tue, Oct 21, 5:06 AM

Details

Summary

This patch makes possible to boot with up to 8 ranges in soc.
Dynamic allocation cannot be used, because ftd_get_ranges
function is called early, when malloc is not available.

Change is required for the alignment of Marvell Armada 38x
device trees present in sys/gnu/dts/arm - originally
the platform has 6 entries in simple-bus 'ranges'.

Diff Detail

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

Event Timeline

manu edited edge metadata.

Add Nathan as he is the maintainer of ofw.

Hi @nwhitehorn - do you have any objections to the patch?

This revision is now accepted and ready to land.Aug 8 2017, 9:45 AM

Looks good. We may need to think some more about the implementation of this function if we need to bump it again, but I cannot think of a better approach offhand.

As a general comment, we should eventually remove all of these fdt_* functions, since they duplicate the equivalent OF_* and ofw_* functions in dev/ofw. The existing code is there for compat with old code only.

This revision was automatically updated to reflect the committed changes.