| commit | fcd665a26cc0f95fd525af09eaf237c52a8b98f9 | [log] [tgz] |
|---|---|---|
| author | Ulf Magnusson <Ulf.Magnusson@nordicsemi.no> | Wed Aug 28 00:22:01 2019 +0200 |
| committer | Kumar Gala <kumar.gala@gmail.com> | Sat Sep 07 10:25:02 2019 -0500 |
| tree | 321888743ee8a46620d95170e66fe71f1820a8b1 | |
| parent | e8524965c537e978453d53871d4fb5377563e3db [diff] |
dts: bindings: Have 'required: true/false' instead of 'category: ...'
The 'category: required/optional' setting for properties is just a
yes/no thing. Using a boolean makes it clearer, so have
'required: true/false' instead.
Print a clear error when 'category:' is used:
edtlib.EDTError: please put 'required: true' instead of 'category:
required' in 'properties: foo: ...' in
test-bindings/sub-node-parent.yaml - 'category' has been removed
The old scripts in scripts/dts/ ignore this setting, and only print a
warning if 'category: required' in an inherited binding is changed to
'category: optional'. Remove that code, since the new scripts already
have the same check.
The replacement was done with
git ls-files 'dts/bindings/*.yaml' | xargs sed -i \
-e 's/category:\s*required/required: true/' \
-e 's/category:\s*optional/required: false/'
dts/binding-template.yaml is updated as well.
Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>