7c44c2f9cb
The recent change to the lexer to only recognize property and node names in the appropriate context removed a number of lexical warts in our language that would have gotten ugly as we add expression support and so forth. But there's one nasty one remaining: references can contain a full path, including the various problematic node name characters (',', '+' and '-', for example). This would cause trouble with expressions, and it also causes trouble with the patch I'm working on to allow expanding references to paths rather than phandles. This patch therefore reworks the lexer to mitigate these problems. - References to labels cause no problems. These are now recognized separately from references to full paths. No syntax change here. - References to full paths, including problematic characters are allowed by "quoting" the path with braces e.g. &{/pci@10000/somedevice@3,8000}. The braces protect any internal problematic characters from being confused with operators or whatever. - For compatibility with existing dts files, in v0 dts files we allow bare references to paths as before &/foo/bar/whatever - but *only* if the path contains no troublesome characters. Specifically only [a-zA-Z0-9_@/] are allowed. This is an incompatible change to the dts-v1 format, but since AFAIK no-one has yet switched to dts-v1 files, I think we can get away with it. Better to make the transition when people to convert to v1, and get rid of the problematic old syntax. Strictly speaking, it's also an incompatible change to the v0 format, since some path references that were allowed before are no longer allowed. I suspect no-one has been using the no-longer-supported forms (certainly none of the kernel dts files will cause trouble). We might need to think about this harder, though. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> |
||
---|---|---|
.. | ||
base01.asm | ||
base01.cmd | ||
base01.dts | ||
base01.stderr | ||
del_node.c | ||
del_property.c | ||
dtbs_equal_ordered.c | ||
dtc-checkfails.sh | ||
dtc.sh | ||
dumptrees.c | ||
dup-nodename.dts | ||
dup-phandle.dts | ||
dup-propname.dts | ||
empty.dts | ||
escapes.dts | ||
find_property.c | ||
get_mem_rsv.c | ||
get_name.c | ||
get_path.c | ||
get_phandle.c | ||
getprop.c | ||
label01.dts | ||
Makefile.tests | ||
mangle-layout.c | ||
mangle-layout.supp | ||
minusone-phandle.dts | ||
move_and_save.c | ||
node_check_compatible.c | ||
node_offset_by_compatible.c | ||
node_offset_by_phandle.c | ||
node_offset_by_prop_value.c | ||
nonexist-label-ref.dts | ||
nonexist-node-ref.dts | ||
nop_node.c | ||
nop_property.c | ||
notfound.c | ||
open_pack.c | ||
open_pack.supp | ||
parent_offset.c | ||
path_offset.c | ||
references.c | ||
references.dts | ||
references_dts0.dts | ||
root_node.c | ||
run_tests.sh | ||
rw_tree1.c | ||
setprop.c | ||
setprop_inplace.c | ||
string_escapes.c | ||
subnode_offset.c | ||
supernode_atdepth_offset.c | ||
sw_tree1.c | ||
test01.asm | ||
test01.dts | ||
test01.stderr | ||
test_kernel_dts | ||
test_tree1.dts | ||
test_tree1_dts0.dts | ||
testdata.h | ||
tests.h | ||
tests.sh | ||
testutils.c | ||
trees.S | ||
truncated_property.c | ||
zero-phandle.dts |