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

Authored by mw on Aug 5 2017, 10:10 AM.

Details

Reviewers

manu
cognet
loos
nwhitehorn

Commits

rS322285: Enable parsing simple-bus 'ranges' with multiple entries

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

mw created this revision.Aug 5 2017, 10:10 AM

Herald added a subscriber: imp. · View Herald TranscriptAug 5 2017, 10:10 AM

mw added a child revision: D11877: Restore original /soc ranges on Marvell Armada 38x boards.Aug 5 2017, 10:12 AM

Add Nathan as he is the maintainer of ofw.

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

manu accepted this revision.Aug 8 2017, 9:45 AM

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.

Closed by commit rS322285: Enable parsing simple-bus 'ranges' with multiple entries (authored by mw). · Explain WhyAug 9 2017, 12:45 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

fdt/

fdt_common.c

3 lines

Diff 31783

View Options

head/sys/dev/fdt/fdt_common.c

Enable parsing simple-bus 'ranges' with multiple entriesClosedPublicActions