dtc: change default phandles to ePAPR style instead of both
PATCH 1/1 dtc: change default phandles to ePAPR style instead of both
Currently, both legacy (linux,phandle) and ePAPR (phandle) properties are inserted into dtbs by default. The newer ePAPR style has been supported in dtc and Linux kernel for 7 years. That should be a long enough transition period. We can save a little space by not putting both into the dtb.
--- dtc.c | 2 +- tests/run_tests.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
dtc.c +1 -1
@@ -31,7 +31,7 @@ int reservenum; /* Number of memory reservation slots */int minsize;/* Minimum blob size */ int padsize;/* Additional padding to blob */ int alignsize;/* Additional padding to blob accroding to the alignsize */ -int phandle_format =PHANDLE_BOTH ;/* Use linux,phandle or phandle properties */ +int phandle_format =PHANDLE_EPAPR ;/* Use linux,phandle or phandle properties */ int generate_symbols;/* enable symbols & fixup support */ int generate_fixups;/* suppress generation of fixups on symbol support */ int auto_label_aliases;/* auto generate labels -> aliases */
tests/run_tests.sh +1 -1
@@ -420,7 +420,7 @@ dtc_tests () { run_dtc_test -I dts -O dtb -o dtc_path-references.test.dtb path-references.dts run_test path-references dtc_path-references.test.dtb - run_test phandle_format dtc_references.test.dtb both + run_test phandle_format dtc_references.test.dtb epapr for f in legacy epapr both; do run_dtc_test -I dts -O dtb -H $f -o dtc_references.test.$f.dtb references.dts run_test phandle_format dtc_references.test.$f.dtb $f
Signature · 1 line
2.11.0
A message that is not in the archive (sent off-list or not yet received): 20170712222030.22258-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
Quoted 7 lines · Rob Herring
Currently, both legacy (linux,phandle) and ePAPR (phandle) properties are inserted into dtbs by default. The newer ePAPR style has been supported in dtc and Linux kernel for 7 years. That should be a long enough transition period. We can save a little space by not putting both into the dtb.
Signed-off-by Rob Herring <[email protected]>
Seems reasonable, applied.
Quoted 31 lines
--- dtc.c | 2 +- tests/run_tests.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)dtc.c +1
-1@@ -31,7 +31,7 @@ int reservenum; /* Number of memory reservation slots */int minsize;/* Minimum blob size */ int padsize;/* Additional padding to blob */ int alignsize;/* Additional padding to blob accroding to the alignsize */ -int phandle_format =PHANDLE_BOTH ;/* Use linux,phandle or phandle properties */ +int phandle_format =PHANDLE_EPAPR ;/* Use linux,phandle or phandle properties */ int generate_symbols;/* enable symbols & fixup support */ int generate_fixups;/* suppress generation of fixups on symbol support */ int auto_label_aliases;/* auto generate labels -> aliases */ tests/run_tests.sh +1
-1@@ -420,7 +420,7 @@ dtc_tests () { run_dtc_test -I dts -O dtb -o dtc_path-references.test.dtb path-references.dts run_test path-references dtc_path-references.test.dtb - run_test phandle_format dtc_references.test.dtb both + run_test phandle_format dtc_references.test.dtb epapr for f in legacy epapr both; do run_dtc_test -I dts -O dtb -H $f -o dtc_references.test.$f.dtb references.dts run_test phandle_format dtc_references.test.$f.dtb $f
Signature · 4 lines
David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson