1 // SPDX-License-Identifier: GPL-2.0-or-later
10 #define TRACE(c, ...) \
12 fprintf(stderr, "=== %s: ", (c)->name); \
13 fprintf(stderr, __VA_ARGS__); \
14 fprintf(stderr, "\n"); \
17 #define TRACE(c, fmt, ...) do { } while (0)
29 typedef void (*check_fn)(struct check *c, struct dt_info *dti, struct node *node);
36 enum checkstatus status;
39 struct check **prereq;
42 #define CHECK_ENTRY(nm_, fn_, d_, w_, e_, ...) \
43 static struct check *nm_##_prereqs[] = { __VA_ARGS__ }; \
44 static struct check nm_ = { \
50 .status = UNCHECKED, \
51 .num_prereqs = ARRAY_SIZE(nm_##_prereqs), \
52 .prereq = nm_##_prereqs, \
54 #define WARNING(nm_, fn_, d_, ...) \
55 CHECK_ENTRY(nm_, fn_, d_, true, false, __VA_ARGS__)
56 #define ERROR(nm_, fn_, d_, ...) \
57 CHECK_ENTRY(nm_, fn_, d_, false, true, __VA_ARGS__)
58 #define CHECK(nm_, fn_, d_, ...) \
59 CHECK_ENTRY(nm_, fn_, d_, false, false, __VA_ARGS__)
61 static inline void PRINTF(5, 6) check_msg(struct check *c, struct dt_info *dti,
63 struct property *prop,
68 struct srcpos *pos = NULL;
71 if (!(c->warn && (quiet < 1)) && !(c->error && (quiet < 2)))
74 if (prop && prop->srcpos)
76 else if (node && node->srcpos)
80 file_str = srcpos_string(pos);
81 xasprintf(&str, "%s", file_str);
83 } else if (streq(dti->outname, "-")) {
84 xasprintf(&str, "<stdout>");
86 xasprintf(&str, "%s", dti->outname);
89 xasprintf_append(&str, ": %s (%s): ",
90 (c->error) ? "ERROR" : "Warning", c->name);
94 xasprintf_append(&str, "%s:%s: ", node->fullpath, prop->name);
96 xasprintf_append(&str, "%s: ", node->fullpath);
100 xavsprintf_append(&str, fmt, ap);
103 xasprintf_append(&str, "\n");
110 file_str = srcpos_string(pos);
111 xasprintf_append(&str, " also defined at %s\n", file_str);
119 #define FAIL(c, dti, node, ...) \
121 TRACE((c), "\t\tFAILED at %s:%d", __FILE__, __LINE__); \
122 (c)->status = FAILED; \
123 check_msg((c), dti, node, NULL, __VA_ARGS__); \
126 #define FAIL_PROP(c, dti, node, prop, ...) \
128 TRACE((c), "\t\tFAILED at %s:%d", __FILE__, __LINE__); \
129 (c)->status = FAILED; \
130 check_msg((c), dti, node, prop, __VA_ARGS__); \
134 static void check_nodes_props(struct check *c, struct dt_info *dti, struct node *node)
138 TRACE(c, "%s", node->fullpath);
142 for_each_child(node, child)
143 check_nodes_props(c, dti, child);
146 static bool is_multiple_of(int multiple, int divisor)
149 return multiple == 0;
151 return (multiple % divisor) == 0;
154 static bool run_check(struct check *c, struct dt_info *dti)
156 struct node *dt = dti->dt;
160 assert(!c->inprogress);
162 if (c->status != UNCHECKED)
165 c->inprogress = true;
167 for (i = 0; i < c->num_prereqs; i++) {
168 struct check *prq = c->prereq[i];
169 error = error || run_check(prq, dti);
170 if (prq->status != PASSED) {
172 check_msg(c, dti, NULL, NULL, "Failed prerequisite '%s'",
177 if (c->status != UNCHECKED)
180 check_nodes_props(c, dti, dt);
182 if (c->status == UNCHECKED)
185 TRACE(c, "\tCompleted, status %d", c->status);
188 c->inprogress = false;
189 if ((c->status != PASSED) && (c->error))
195 * Utility check functions
198 /* A check which always fails, for testing purposes only */
199 static inline void check_always_fail(struct check *c, struct dt_info *dti,
202 FAIL(c, dti, node, "always_fail check");
204 CHECK(always_fail, check_always_fail, NULL);
206 static void check_is_string(struct check *c, struct dt_info *dti,
209 struct property *prop;
210 char *propname = c->data;
212 prop = get_property(node, propname);
214 return; /* Not present, assumed ok */
216 if (!data_is_one_string(prop->val))
217 FAIL_PROP(c, dti, node, prop, "property is not a string");
219 #define WARNING_IF_NOT_STRING(nm, propname) \
220 WARNING(nm, check_is_string, (propname))
221 #define ERROR_IF_NOT_STRING(nm, propname) \
222 ERROR(nm, check_is_string, (propname))
224 static void check_is_string_list(struct check *c, struct dt_info *dti,
228 struct property *prop;
229 char *propname = c->data;
232 prop = get_property(node, propname);
234 return; /* Not present, assumed ok */
239 l = strnlen(str, rem);
241 FAIL_PROP(c, dti, node, prop, "property is not a string list");
248 #define WARNING_IF_NOT_STRING_LIST(nm, propname) \
249 WARNING(nm, check_is_string_list, (propname))
250 #define ERROR_IF_NOT_STRING_LIST(nm, propname) \
251 ERROR(nm, check_is_string_list, (propname))
253 static void check_is_cell(struct check *c, struct dt_info *dti,
256 struct property *prop;
257 char *propname = c->data;
259 prop = get_property(node, propname);
261 return; /* Not present, assumed ok */
263 if (prop->val.len != sizeof(cell_t))
264 FAIL_PROP(c, dti, node, prop, "property is not a single cell");
266 #define WARNING_IF_NOT_CELL(nm, propname) \
267 WARNING(nm, check_is_cell, (propname))
268 #define ERROR_IF_NOT_CELL(nm, propname) \
269 ERROR(nm, check_is_cell, (propname))
272 * Structural check functions
275 static void check_duplicate_node_names(struct check *c, struct dt_info *dti,
278 struct node *child, *child2;
280 for_each_child(node, child)
281 for (child2 = child->next_sibling;
283 child2 = child2->next_sibling)
284 if (streq(child->name, child2->name))
285 FAIL(c, dti, child2, "Duplicate node name");
287 ERROR(duplicate_node_names, check_duplicate_node_names, NULL);
289 static void check_duplicate_property_names(struct check *c, struct dt_info *dti,
292 struct property *prop, *prop2;
294 for_each_property(node, prop) {
295 for (prop2 = prop->next; prop2; prop2 = prop2->next) {
298 if (streq(prop->name, prop2->name))
299 FAIL_PROP(c, dti, node, prop, "Duplicate property name");
303 ERROR(duplicate_property_names, check_duplicate_property_names, NULL);
305 #define LOWERCASE "abcdefghijklmnopqrstuvwxyz"
306 #define UPPERCASE "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
307 #define DIGITS "0123456789"
308 #define NODECHARS LOWERCASE UPPERCASE DIGITS ",._+-@"
309 #define PROPCHARS LOWERCASE UPPERCASE DIGITS ",._+*#?-"
310 #define PROPNODECHARSSTRICT LOWERCASE UPPERCASE DIGITS ",-"
312 static void check_node_name_chars(struct check *c, struct dt_info *dti,
315 size_t n = strspn(node->name, c->data);
317 if (n < strlen(node->name))
318 FAIL(c, dti, node, "Bad character '%c' in node name",
321 ERROR(node_name_chars, check_node_name_chars, NODECHARS);
323 static void check_node_name_chars_strict(struct check *c, struct dt_info *dti,
326 int n = strspn(node->name, c->data);
328 if (n < node->basenamelen)
329 FAIL(c, dti, node, "Character '%c' not recommended in node name",
332 CHECK(node_name_chars_strict, check_node_name_chars_strict, PROPNODECHARSSTRICT);
334 static void check_node_name_format(struct check *c, struct dt_info *dti,
337 if (strchr(get_unitname(node), '@'))
338 FAIL(c, dti, node, "multiple '@' characters in node name");
340 ERROR(node_name_format, check_node_name_format, NULL, &node_name_chars);
342 static void check_node_name_vs_property_name(struct check *c,
349 if (get_property(node->parent, node->name)) {
350 FAIL(c, dti, node, "node name and property name conflict");
353 WARNING(node_name_vs_property_name, check_node_name_vs_property_name,
354 NULL, &node_name_chars);
356 static void check_unit_address_vs_reg(struct check *c, struct dt_info *dti,
359 const char *unitname = get_unitname(node);
360 struct property *prop = get_property(node, "reg");
362 if (get_subnode(node, "__overlay__")) {
363 /* HACK: Overlay fragments are a special case */
368 prop = get_property(node, "ranges");
369 if (prop && !prop->val.len)
375 FAIL(c, dti, node, "node has a reg or ranges property, but no unit name");
378 FAIL(c, dti, node, "node has a unit name, but no reg or ranges property");
381 WARNING(unit_address_vs_reg, check_unit_address_vs_reg, NULL);
383 static void check_property_name_chars(struct check *c, struct dt_info *dti,
386 struct property *prop;
388 for_each_property(node, prop) {
389 size_t n = strspn(prop->name, c->data);
391 if (n < strlen(prop->name))
392 FAIL_PROP(c, dti, node, prop, "Bad character '%c' in property name",
396 ERROR(property_name_chars, check_property_name_chars, PROPCHARS);
398 static void check_property_name_chars_strict(struct check *c,
402 struct property *prop;
404 for_each_property(node, prop) {
405 const char *name = prop->name;
406 size_t n = strspn(name, c->data);
408 if (n == strlen(prop->name))
411 /* Certain names are whitelisted */
412 if (streq(name, "device_type"))
416 * # is only allowed at the beginning of property names not counting
419 if (name[n] == '#' && ((n == 0) || (name[n-1] == ','))) {
421 n = strspn(name, c->data);
423 if (n < strlen(name))
424 FAIL_PROP(c, dti, node, prop, "Character '%c' not recommended in property name",
428 CHECK(property_name_chars_strict, check_property_name_chars_strict, PROPNODECHARSSTRICT);
430 #define DESCLABEL_FMT "%s%s%s%s%s"
431 #define DESCLABEL_ARGS(node,prop,mark) \
432 ((mark) ? "value of " : ""), \
433 ((prop) ? "'" : ""), \
434 ((prop) ? (prop)->name : ""), \
435 ((prop) ? "' in " : ""), (node)->fullpath
437 static void check_duplicate_label(struct check *c, struct dt_info *dti,
438 const char *label, struct node *node,
439 struct property *prop, struct marker *mark)
441 struct node *dt = dti->dt;
442 struct node *othernode = NULL;
443 struct property *otherprop = NULL;
444 struct marker *othermark = NULL;
446 othernode = get_node_by_label(dt, label);
449 otherprop = get_property_by_label(dt, label, &othernode);
451 othermark = get_marker_label(dt, label, &othernode,
457 if ((othernode != node) || (otherprop != prop) || (othermark != mark))
458 FAIL(c, dti, node, "Duplicate label '%s' on " DESCLABEL_FMT
459 " and " DESCLABEL_FMT,
460 label, DESCLABEL_ARGS(node, prop, mark),
461 DESCLABEL_ARGS(othernode, otherprop, othermark));
464 static void check_duplicate_label_node(struct check *c, struct dt_info *dti,
468 struct property *prop;
470 for_each_label(node->labels, l)
471 check_duplicate_label(c, dti, l->label, node, NULL, NULL);
473 for_each_property(node, prop) {
474 struct marker *m = prop->val.markers;
476 for_each_label(prop->labels, l)
477 check_duplicate_label(c, dti, l->label, node, prop, NULL);
479 for_each_marker_of_type(m, LABEL)
480 check_duplicate_label(c, dti, m->ref, node, prop, m);
483 ERROR(duplicate_label, check_duplicate_label_node, NULL);
485 static cell_t check_phandle_prop(struct check *c, struct dt_info *dti,
486 struct node *node, const char *propname)
488 struct node *root = dti->dt;
489 struct property *prop;
493 prop = get_property(node, propname);
497 if (prop->val.len != sizeof(cell_t)) {
498 FAIL_PROP(c, dti, node, prop, "bad length (%d) %s property",
499 prop->val.len, prop->name);
503 m = prop->val.markers;
504 for_each_marker_of_type(m, REF_PHANDLE) {
505 assert(m->offset == 0);
506 if (node != get_node_by_ref(root, m->ref))
507 /* "Set this node's phandle equal to some
508 * other node's phandle". That's nonsensical
509 * by construction. */ {
510 FAIL(c, dti, node, "%s is a reference to another node",
513 /* But setting this node's phandle equal to its own
514 * phandle is allowed - that means allocate a unique
515 * phandle for this node, even if it's not otherwise
516 * referenced. The value will be filled in later, so
517 * we treat it as having no phandle data for now. */
521 phandle = propval_cell(prop);
523 if (!phandle_is_valid(phandle)) {
524 FAIL_PROP(c, dti, node, prop, "bad value (0x%x) in %s property",
525 phandle, prop->name);
532 static void check_explicit_phandles(struct check *c, struct dt_info *dti,
535 struct node *root = dti->dt;
537 cell_t phandle, linux_phandle;
539 /* Nothing should have assigned phandles yet */
540 assert(!node->phandle);
542 phandle = check_phandle_prop(c, dti, node, "phandle");
544 linux_phandle = check_phandle_prop(c, dti, node, "linux,phandle");
546 if (!phandle && !linux_phandle)
547 /* No valid phandles; nothing further to check */
550 if (linux_phandle && phandle && (phandle != linux_phandle))
551 FAIL(c, dti, node, "mismatching 'phandle' and 'linux,phandle'"
554 if (linux_phandle && !phandle)
555 phandle = linux_phandle;
557 other = get_node_by_phandle(root, phandle);
558 if (other && (other != node)) {
559 FAIL(c, dti, node, "duplicated phandle 0x%x (seen before at %s)",
560 phandle, other->fullpath);
564 node->phandle = phandle;
566 ERROR(explicit_phandles, check_explicit_phandles, NULL);
568 static void check_name_properties(struct check *c, struct dt_info *dti,
571 struct property **pp, *prop = NULL;
573 for (pp = &node->proplist; *pp; pp = &((*pp)->next))
574 if (streq((*pp)->name, "name")) {
580 return; /* No name property, that's fine */
582 if ((prop->val.len != node->basenamelen + 1U)
583 || (memcmp(prop->val.val, node->name, node->basenamelen) != 0)) {
584 FAIL(c, dti, node, "\"name\" property is incorrect (\"%s\" instead"
585 " of base node name)", prop->val.val);
587 /* The name property is correct, and therefore redundant.
591 data_free(prop->val);
595 ERROR_IF_NOT_STRING(name_is_string, "name");
596 ERROR(name_properties, check_name_properties, NULL, &name_is_string);
599 * Reference fixup functions
602 static void fixup_phandle_references(struct check *c, struct dt_info *dti,
605 struct node *dt = dti->dt;
606 struct property *prop;
608 for_each_property(node, prop) {
609 struct marker *m = prop->val.markers;
610 struct node *refnode;
613 for_each_marker_of_type(m, REF_PHANDLE) {
614 assert(m->offset + sizeof(cell_t) <= prop->val.len);
616 refnode = get_node_by_ref(dt, m->ref);
618 if (!(dti->dtsflags & DTSF_PLUGIN))
619 FAIL(c, dti, node, "Reference to non-existent node or "
620 "label \"%s\"\n", m->ref);
621 else /* mark the entry as unresolved */
622 *((fdt32_t *)(prop->val.val + m->offset)) =
623 cpu_to_fdt32(0xffffffff);
627 phandle = get_node_phandle(dt, refnode);
628 *((fdt32_t *)(prop->val.val + m->offset)) = cpu_to_fdt32(phandle);
630 reference_node(refnode);
634 ERROR(phandle_references, fixup_phandle_references, NULL,
635 &duplicate_node_names, &explicit_phandles);
637 static void fixup_path_references(struct check *c, struct dt_info *dti,
640 struct node *dt = dti->dt;
641 struct property *prop;
643 for_each_property(node, prop) {
644 struct marker *m = prop->val.markers;
645 struct node *refnode;
648 for_each_marker_of_type(m, REF_PATH) {
649 assert(m->offset <= prop->val.len);
651 refnode = get_node_by_ref(dt, m->ref);
653 FAIL(c, dti, node, "Reference to non-existent node or label \"%s\"\n",
658 path = refnode->fullpath;
659 prop->val = data_insert_at_marker(prop->val, m, path,
662 reference_node(refnode);
666 ERROR(path_references, fixup_path_references, NULL, &duplicate_node_names);
668 static void fixup_omit_unused_nodes(struct check *c, struct dt_info *dti,
671 if (generate_symbols && node->labels)
673 if (node->omit_if_unused && !node->is_referenced)
676 ERROR(omit_unused_nodes, fixup_omit_unused_nodes, NULL, &phandle_references, &path_references);
681 WARNING_IF_NOT_CELL(address_cells_is_cell, "#address-cells");
682 WARNING_IF_NOT_CELL(size_cells_is_cell, "#size-cells");
684 WARNING_IF_NOT_STRING(device_type_is_string, "device_type");
685 WARNING_IF_NOT_STRING(model_is_string, "model");
686 WARNING_IF_NOT_STRING(status_is_string, "status");
687 WARNING_IF_NOT_STRING(label_is_string, "label");
689 WARNING_IF_NOT_STRING_LIST(compatible_is_string_list, "compatible");
691 static void check_names_is_string_list(struct check *c, struct dt_info *dti,
694 struct property *prop;
696 for_each_property(node, prop) {
697 if (!strends(prop->name, "-names"))
700 c->data = prop->name;
701 check_is_string_list(c, dti, node);
704 WARNING(names_is_string_list, check_names_is_string_list, NULL);
706 static void check_alias_paths(struct check *c, struct dt_info *dti,
709 struct property *prop;
711 if (!streq(node->name, "aliases"))
714 for_each_property(node, prop) {
715 if (streq(prop->name, "phandle")
716 || streq(prop->name, "linux,phandle")) {
720 if (!prop->val.val || !get_node_by_path(dti->dt, prop->val.val)) {
721 FAIL_PROP(c, dti, node, prop, "aliases property is not a valid node (%s)",
725 if (strspn(prop->name, LOWERCASE DIGITS "-") != strlen(prop->name))
726 FAIL(c, dti, node, "aliases property name must include only lowercase and '-'");
729 WARNING(alias_paths, check_alias_paths, NULL);
731 static void fixup_addr_size_cells(struct check *c, struct dt_info *dti,
734 struct property *prop;
736 node->addr_cells = -1;
737 node->size_cells = -1;
739 prop = get_property(node, "#address-cells");
741 node->addr_cells = propval_cell(prop);
743 prop = get_property(node, "#size-cells");
745 node->size_cells = propval_cell(prop);
747 WARNING(addr_size_cells, fixup_addr_size_cells, NULL,
748 &address_cells_is_cell, &size_cells_is_cell);
750 #define node_addr_cells(n) \
751 (((n)->addr_cells == -1) ? 2 : (n)->addr_cells)
752 #define node_size_cells(n) \
753 (((n)->size_cells == -1) ? 1 : (n)->size_cells)
755 static void check_reg_format(struct check *c, struct dt_info *dti,
758 struct property *prop;
759 int addr_cells, size_cells, entrylen;
761 prop = get_property(node, "reg");
763 return; /* No "reg", that's fine */
766 FAIL(c, dti, node, "Root node has a \"reg\" property");
770 if (prop->val.len == 0)
771 FAIL_PROP(c, dti, node, prop, "property is empty");
773 addr_cells = node_addr_cells(node->parent);
774 size_cells = node_size_cells(node->parent);
775 entrylen = (addr_cells + size_cells) * sizeof(cell_t);
777 if (!is_multiple_of(prop->val.len, entrylen))
778 FAIL_PROP(c, dti, node, prop, "property has invalid length (%d bytes) "
779 "(#address-cells == %d, #size-cells == %d)",
780 prop->val.len, addr_cells, size_cells);
782 WARNING(reg_format, check_reg_format, NULL, &addr_size_cells);
784 static void check_ranges_format(struct check *c, struct dt_info *dti,
787 struct property *prop;
788 int c_addr_cells, p_addr_cells, c_size_cells, p_size_cells, entrylen;
789 const char *ranges = c->data;
791 prop = get_property(node, ranges);
796 FAIL_PROP(c, dti, node, prop, "Root node has a \"%s\" property",
801 p_addr_cells = node_addr_cells(node->parent);
802 p_size_cells = node_size_cells(node->parent);
803 c_addr_cells = node_addr_cells(node);
804 c_size_cells = node_size_cells(node);
805 entrylen = (p_addr_cells + c_addr_cells + c_size_cells) * sizeof(cell_t);
807 if (prop->val.len == 0) {
808 if (p_addr_cells != c_addr_cells)
809 FAIL_PROP(c, dti, node, prop, "empty \"%s\" property but its "
810 "#address-cells (%d) differs from %s (%d)",
811 ranges, c_addr_cells, node->parent->fullpath,
813 if (p_size_cells != c_size_cells)
814 FAIL_PROP(c, dti, node, prop, "empty \"%s\" property but its "
815 "#size-cells (%d) differs from %s (%d)",
816 ranges, c_size_cells, node->parent->fullpath,
818 } else if (!is_multiple_of(prop->val.len, entrylen)) {
819 FAIL_PROP(c, dti, node, prop, "\"%s\" property has invalid length (%d bytes) "
820 "(parent #address-cells == %d, child #address-cells == %d, "
821 "#size-cells == %d)", ranges, prop->val.len,
822 p_addr_cells, c_addr_cells, c_size_cells);
825 WARNING(ranges_format, check_ranges_format, "ranges", &addr_size_cells);
826 WARNING(dma_ranges_format, check_ranges_format, "dma-ranges", &addr_size_cells);
828 static const struct bus_type pci_bus = {
832 static void check_pci_bridge(struct check *c, struct dt_info *dti, struct node *node)
834 struct property *prop;
837 prop = get_property(node, "device_type");
838 if (!prop || !streq(prop->val.val, "pci"))
841 node->bus = &pci_bus;
843 if (!strprefixeq(node->name, node->basenamelen, "pci") &&
844 !strprefixeq(node->name, node->basenamelen, "pcie"))
845 FAIL(c, dti, node, "node name is not \"pci\" or \"pcie\"");
847 prop = get_property(node, "ranges");
849 FAIL(c, dti, node, "missing ranges for PCI bridge (or not a bridge)");
851 if (node_addr_cells(node) != 3)
852 FAIL(c, dti, node, "incorrect #address-cells for PCI bridge");
853 if (node_size_cells(node) != 2)
854 FAIL(c, dti, node, "incorrect #size-cells for PCI bridge");
856 prop = get_property(node, "bus-range");
860 if (prop->val.len != (sizeof(cell_t) * 2)) {
861 FAIL_PROP(c, dti, node, prop, "value must be 2 cells");
864 cells = (cell_t *)prop->val.val;
865 if (fdt32_to_cpu(cells[0]) > fdt32_to_cpu(cells[1]))
866 FAIL_PROP(c, dti, node, prop, "1st cell must be less than or equal to 2nd cell");
867 if (fdt32_to_cpu(cells[1]) > 0xff)
868 FAIL_PROP(c, dti, node, prop, "maximum bus number must be less than 256");
870 WARNING(pci_bridge, check_pci_bridge, NULL,
871 &device_type_is_string, &addr_size_cells);
873 static void check_pci_device_bus_num(struct check *c, struct dt_info *dti, struct node *node)
875 struct property *prop;
876 unsigned int bus_num, min_bus, max_bus;
879 if (!node->parent || (node->parent->bus != &pci_bus))
882 prop = get_property(node, "reg");
886 cells = (cell_t *)prop->val.val;
887 bus_num = (fdt32_to_cpu(cells[0]) & 0x00ff0000) >> 16;
889 prop = get_property(node->parent, "bus-range");
891 min_bus = max_bus = 0;
893 cells = (cell_t *)prop->val.val;
894 min_bus = fdt32_to_cpu(cells[0]);
895 max_bus = fdt32_to_cpu(cells[1]);
897 if ((bus_num < min_bus) || (bus_num > max_bus))
898 FAIL_PROP(c, dti, node, prop, "PCI bus number %d out of range, expected (%d - %d)",
899 bus_num, min_bus, max_bus);
901 WARNING(pci_device_bus_num, check_pci_device_bus_num, NULL, ®_format, &pci_bridge);
903 static void check_pci_device_reg(struct check *c, struct dt_info *dti, struct node *node)
905 struct property *prop;
906 const char *unitname = get_unitname(node);
908 unsigned int dev, func, reg;
911 if (!node->parent || (node->parent->bus != &pci_bus))
914 prop = get_property(node, "reg");
918 cells = (cell_t *)prop->val.val;
919 if (cells[1] || cells[2])
920 FAIL_PROP(c, dti, node, prop, "PCI reg config space address cells 2 and 3 must be 0");
922 reg = fdt32_to_cpu(cells[0]);
923 dev = (reg & 0xf800) >> 11;
924 func = (reg & 0x700) >> 8;
926 if (reg & 0xff000000)
927 FAIL_PROP(c, dti, node, prop, "PCI reg address is not configuration space");
928 if (reg & 0x000000ff)
929 FAIL_PROP(c, dti, node, prop, "PCI reg config space address register number must be 0");
932 snprintf(unit_addr, sizeof(unit_addr), "%x", dev);
933 if (streq(unitname, unit_addr))
937 snprintf(unit_addr, sizeof(unit_addr), "%x,%x", dev, func);
938 if (streq(unitname, unit_addr))
941 FAIL(c, dti, node, "PCI unit address format error, expected \"%s\"",
944 WARNING(pci_device_reg, check_pci_device_reg, NULL, ®_format, &pci_bridge);
946 static const struct bus_type simple_bus = {
947 .name = "simple-bus",
950 static bool node_is_compatible(struct node *node, const char *compat)
952 struct property *prop;
953 const char *str, *end;
955 prop = get_property(node, "compatible");
959 for (str = prop->val.val, end = str + prop->val.len; str < end;
960 str += strnlen(str, end - str) + 1) {
961 if (streq(str, compat))
967 static void check_simple_bus_bridge(struct check *c, struct dt_info *dti, struct node *node)
969 if (node_is_compatible(node, "simple-bus"))
970 node->bus = &simple_bus;
972 WARNING(simple_bus_bridge, check_simple_bus_bridge, NULL,
973 &addr_size_cells, &compatible_is_string_list);
975 static void check_simple_bus_reg(struct check *c, struct dt_info *dti, struct node *node)
977 struct property *prop;
978 const char *unitname = get_unitname(node);
982 cell_t *cells = NULL;
984 if (!node->parent || (node->parent->bus != &simple_bus))
987 prop = get_property(node, "reg");
989 cells = (cell_t *)prop->val.val;
991 prop = get_property(node, "ranges");
992 if (prop && prop->val.len)
993 /* skip of child address */
994 cells = ((cell_t *)prop->val.val) + node_addr_cells(node);
998 if (node->parent->parent && !(node->bus == &simple_bus))
999 FAIL(c, dti, node, "missing or empty reg/ranges property");
1003 size = node_addr_cells(node->parent);
1005 reg = (reg << 32) | fdt32_to_cpu(*(cells++));
1007 snprintf(unit_addr, sizeof(unit_addr), "%"PRIx64, reg);
1008 if (!streq(unitname, unit_addr))
1009 FAIL(c, dti, node, "simple-bus unit address format error, expected \"%s\"",
1012 WARNING(simple_bus_reg, check_simple_bus_reg, NULL, ®_format, &simple_bus_bridge);
1014 static const struct bus_type i2c_bus = {
1018 static void check_i2c_bus_bridge(struct check *c, struct dt_info *dti, struct node *node)
1020 if (strprefixeq(node->name, node->basenamelen, "i2c-bus") ||
1021 strprefixeq(node->name, node->basenamelen, "i2c-arb")) {
1022 node->bus = &i2c_bus;
1023 } else if (strprefixeq(node->name, node->basenamelen, "i2c")) {
1025 for_each_child(node, child) {
1026 if (strprefixeq(child->name, node->basenamelen, "i2c-bus"))
1029 node->bus = &i2c_bus;
1033 if (!node->children)
1036 if (node_addr_cells(node) != 1)
1037 FAIL(c, dti, node, "incorrect #address-cells for I2C bus");
1038 if (node_size_cells(node) != 0)
1039 FAIL(c, dti, node, "incorrect #size-cells for I2C bus");
1042 WARNING(i2c_bus_bridge, check_i2c_bus_bridge, NULL, &addr_size_cells);
1044 #define I2C_OWN_SLAVE_ADDRESS (1U << 30)
1045 #define I2C_TEN_BIT_ADDRESS (1U << 31)
1047 static void check_i2c_bus_reg(struct check *c, struct dt_info *dti, struct node *node)
1049 struct property *prop;
1050 const char *unitname = get_unitname(node);
1054 cell_t *cells = NULL;
1056 if (!node->parent || (node->parent->bus != &i2c_bus))
1059 prop = get_property(node, "reg");
1061 cells = (cell_t *)prop->val.val;
1064 FAIL(c, dti, node, "missing or empty reg property");
1068 reg = fdt32_to_cpu(*cells);
1069 /* Ignore I2C_OWN_SLAVE_ADDRESS */
1070 reg &= ~I2C_OWN_SLAVE_ADDRESS;
1071 snprintf(unit_addr, sizeof(unit_addr), "%x", reg);
1072 if (!streq(unitname, unit_addr))
1073 FAIL(c, dti, node, "I2C bus unit address format error, expected \"%s\"",
1076 for (len = prop->val.len; len > 0; len -= 4) {
1077 reg = fdt32_to_cpu(*(cells++));
1078 /* Ignore I2C_OWN_SLAVE_ADDRESS */
1079 reg &= ~I2C_OWN_SLAVE_ADDRESS;
1081 if ((reg & I2C_TEN_BIT_ADDRESS) && ((reg & ~I2C_TEN_BIT_ADDRESS) > 0x3ff))
1082 FAIL_PROP(c, dti, node, prop, "I2C address must be less than 10-bits, got \"0x%x\"",
1084 else if (reg > 0x7f)
1085 FAIL_PROP(c, dti, node, prop, "I2C address must be less than 7-bits, got \"0x%x\". Set I2C_TEN_BIT_ADDRESS for 10 bit addresses or fix the property",
1089 WARNING(i2c_bus_reg, check_i2c_bus_reg, NULL, ®_format, &i2c_bus_bridge);
1091 static const struct bus_type spi_bus = {
1095 static void check_spi_bus_bridge(struct check *c, struct dt_info *dti, struct node *node)
1097 int spi_addr_cells = 1;
1099 if (strprefixeq(node->name, node->basenamelen, "spi")) {
1100 node->bus = &spi_bus;
1102 /* Try to detect SPI buses which don't have proper node name */
1105 if (node_addr_cells(node) != 1 || node_size_cells(node) != 0)
1108 for_each_child(node, child) {
1109 struct property *prop;
1110 for_each_property(child, prop) {
1111 if (strprefixeq(prop->name, 4, "spi-")) {
1112 node->bus = &spi_bus;
1116 if (node->bus == &spi_bus)
1120 if (node->bus == &spi_bus && get_property(node, "reg"))
1121 FAIL(c, dti, node, "node name for SPI buses should be 'spi'");
1123 if (node->bus != &spi_bus || !node->children)
1126 if (get_property(node, "spi-slave"))
1128 if (node_addr_cells(node) != spi_addr_cells)
1129 FAIL(c, dti, node, "incorrect #address-cells for SPI bus");
1130 if (node_size_cells(node) != 0)
1131 FAIL(c, dti, node, "incorrect #size-cells for SPI bus");
1134 WARNING(spi_bus_bridge, check_spi_bus_bridge, NULL, &addr_size_cells);
1136 static void check_spi_bus_reg(struct check *c, struct dt_info *dti, struct node *node)
1138 struct property *prop;
1139 const char *unitname = get_unitname(node);
1142 cell_t *cells = NULL;
1144 if (!node->parent || (node->parent->bus != &spi_bus))
1147 if (get_property(node->parent, "spi-slave"))
1150 prop = get_property(node, "reg");
1152 cells = (cell_t *)prop->val.val;
1155 FAIL(c, dti, node, "missing or empty reg property");
1159 reg = fdt32_to_cpu(*cells);
1160 snprintf(unit_addr, sizeof(unit_addr), "%x", reg);
1161 if (!streq(unitname, unit_addr))
1162 FAIL(c, dti, node, "SPI bus unit address format error, expected \"%s\"",
1165 WARNING(spi_bus_reg, check_spi_bus_reg, NULL, ®_format, &spi_bus_bridge);
1167 static void check_unit_address_format(struct check *c, struct dt_info *dti,
1170 const char *unitname = get_unitname(node);
1172 if (node->parent && node->parent->bus)
1178 if (!strncmp(unitname, "0x", 2)) {
1179 FAIL(c, dti, node, "unit name should not have leading \"0x\"");
1180 /* skip over 0x for next test */
1183 if (unitname[0] == '0' && isxdigit(unitname[1]))
1184 FAIL(c, dti, node, "unit name should not have leading 0s");
1186 WARNING(unit_address_format, check_unit_address_format, NULL,
1187 &node_name_format, &pci_bridge, &simple_bus_bridge);
1192 static void check_avoid_default_addr_size(struct check *c, struct dt_info *dti,
1195 struct property *reg, *ranges;
1198 return; /* Ignore root node */
1200 reg = get_property(node, "reg");
1201 ranges = get_property(node, "ranges");
1203 if (!reg && !ranges)
1206 if (node->parent->addr_cells == -1)
1207 FAIL(c, dti, node, "Relying on default #address-cells value");
1209 if (node->parent->size_cells == -1)
1210 FAIL(c, dti, node, "Relying on default #size-cells value");
1212 WARNING(avoid_default_addr_size, check_avoid_default_addr_size, NULL,
1215 static void check_avoid_unnecessary_addr_size(struct check *c, struct dt_info *dti,
1218 struct property *prop;
1220 bool has_reg = false;
1222 if (!node->parent || node->addr_cells < 0 || node->size_cells < 0)
1225 if (get_property(node, "ranges") || !node->children)
1228 for_each_child(node, child) {
1229 prop = get_property(child, "reg");
1235 FAIL(c, dti, node, "unnecessary #address-cells/#size-cells without \"ranges\" or child \"reg\" property");
1237 WARNING(avoid_unnecessary_addr_size, check_avoid_unnecessary_addr_size, NULL, &avoid_default_addr_size);
1239 static bool node_is_disabled(struct node *node)
1241 struct property *prop;
1243 prop = get_property(node, "status");
1245 char *str = prop->val.val;
1246 if (streq("disabled", str))
1253 static void check_unique_unit_address_common(struct check *c,
1254 struct dt_info *dti,
1258 struct node *childa;
1260 if (node->addr_cells < 0 || node->size_cells < 0)
1263 if (!node->children)
1266 for_each_child(node, childa) {
1267 struct node *childb;
1268 const char *addr_a = get_unitname(childa);
1270 if (!strlen(addr_a))
1273 if (disable_check && node_is_disabled(childa))
1276 for_each_child(node, childb) {
1277 const char *addr_b = get_unitname(childb);
1278 if (childa == childb)
1281 if (disable_check && node_is_disabled(childb))
1284 if (streq(addr_a, addr_b))
1285 FAIL(c, dti, childb, "duplicate unit-address (also used in node %s)", childa->fullpath);
1290 static void check_unique_unit_address(struct check *c, struct dt_info *dti,
1293 check_unique_unit_address_common(c, dti, node, false);
1295 WARNING(unique_unit_address, check_unique_unit_address, NULL, &avoid_default_addr_size);
1297 static void check_unique_unit_address_if_enabled(struct check *c, struct dt_info *dti,
1300 check_unique_unit_address_common(c, dti, node, true);
1302 CHECK_ENTRY(unique_unit_address_if_enabled, check_unique_unit_address_if_enabled,
1303 NULL, false, false, &avoid_default_addr_size);
1305 static void check_obsolete_chosen_interrupt_controller(struct check *c,
1306 struct dt_info *dti,
1309 struct node *dt = dti->dt;
1310 struct node *chosen;
1311 struct property *prop;
1317 chosen = get_node_by_path(dt, "/chosen");
1321 prop = get_property(chosen, "interrupt-controller");
1323 FAIL_PROP(c, dti, node, prop,
1324 "/chosen has obsolete \"interrupt-controller\" property");
1326 WARNING(obsolete_chosen_interrupt_controller,
1327 check_obsolete_chosen_interrupt_controller, NULL);
1329 static void check_chosen_node_is_root(struct check *c, struct dt_info *dti,
1332 if (!streq(node->name, "chosen"))
1335 if (node->parent != dti->dt)
1336 FAIL(c, dti, node, "chosen node must be at root node");
1338 WARNING(chosen_node_is_root, check_chosen_node_is_root, NULL);
1340 static void check_chosen_node_bootargs(struct check *c, struct dt_info *dti,
1343 struct property *prop;
1345 if (!streq(node->name, "chosen"))
1348 prop = get_property(node, "bootargs");
1352 c->data = prop->name;
1353 check_is_string(c, dti, node);
1355 WARNING(chosen_node_bootargs, check_chosen_node_bootargs, NULL);
1357 static void check_chosen_node_stdout_path(struct check *c, struct dt_info *dti,
1360 struct property *prop;
1362 if (!streq(node->name, "chosen"))
1365 prop = get_property(node, "stdout-path");
1367 prop = get_property(node, "linux,stdout-path");
1370 FAIL_PROP(c, dti, node, prop, "Use 'stdout-path' instead");
1373 c->data = prop->name;
1374 check_is_string(c, dti, node);
1376 WARNING(chosen_node_stdout_path, check_chosen_node_stdout_path, NULL);
1379 const char *prop_name;
1380 const char *cell_name;
1384 static void check_property_phandle_args(struct check *c,
1385 struct dt_info *dti,
1387 struct property *prop,
1388 const struct provider *provider)
1390 struct node *root = dti->dt;
1391 unsigned int cell, cellsize = 0;
1393 if (!is_multiple_of(prop->val.len, sizeof(cell_t))) {
1394 FAIL_PROP(c, dti, node, prop,
1395 "property size (%d) is invalid, expected multiple of %zu",
1396 prop->val.len, sizeof(cell_t));
1400 for (cell = 0; cell < prop->val.len / sizeof(cell_t); cell += cellsize + 1) {
1401 struct node *provider_node;
1402 struct property *cellprop;
1404 unsigned int expected;
1406 phandle = propval_cell_n(prop, cell);
1408 * Some bindings use a cell value 0 or -1 to skip over optional
1409 * entries when each index position has a specific definition.
1411 if (!phandle_is_valid(phandle)) {
1412 /* Give up if this is an overlay with external references */
1413 if (dti->dtsflags & DTSF_PLUGIN)
1420 /* If we have markers, verify the current cell is a phandle */
1421 if (prop->val.markers) {
1422 struct marker *m = prop->val.markers;
1423 for_each_marker_of_type(m, REF_PHANDLE) {
1424 if (m->offset == (cell * sizeof(cell_t)))
1428 FAIL_PROP(c, dti, node, prop,
1429 "cell %d is not a phandle reference",
1433 provider_node = get_node_by_phandle(root, phandle);
1434 if (!provider_node) {
1435 FAIL_PROP(c, dti, node, prop,
1436 "Could not get phandle node for (cell %d)",
1441 cellprop = get_property(provider_node, provider->cell_name);
1443 cellsize = propval_cell(cellprop);
1444 } else if (provider->optional) {
1447 FAIL(c, dti, node, "Missing property '%s' in node %s or bad phandle (referred from %s[%d])",
1448 provider->cell_name,
1449 provider_node->fullpath,
1454 expected = (cell + cellsize + 1) * sizeof(cell_t);
1455 if ((expected <= cell) || prop->val.len < expected) {
1456 FAIL_PROP(c, dti, node, prop,
1457 "property size (%d) too small for cell size %u",
1458 prop->val.len, cellsize);
1464 static void check_provider_cells_property(struct check *c,
1465 struct dt_info *dti,
1468 struct provider *provider = c->data;
1469 struct property *prop;
1471 prop = get_property(node, provider->prop_name);
1475 check_property_phandle_args(c, dti, node, prop, provider);
1477 #define WARNING_PROPERTY_PHANDLE_CELLS(nm, propname, cells_name, ...) \
1478 static struct provider nm##_provider = { (propname), (cells_name), __VA_ARGS__ }; \
1479 WARNING_IF_NOT_CELL(nm##_is_cell, cells_name); \
1480 WARNING(nm##_property, check_provider_cells_property, &nm##_provider, &nm##_is_cell, &phandle_references);
1482 WARNING_PROPERTY_PHANDLE_CELLS(clocks, "clocks", "#clock-cells");
1483 WARNING_PROPERTY_PHANDLE_CELLS(cooling_device, "cooling-device", "#cooling-cells");
1484 WARNING_PROPERTY_PHANDLE_CELLS(dmas, "dmas", "#dma-cells");
1485 WARNING_PROPERTY_PHANDLE_CELLS(hwlocks, "hwlocks", "#hwlock-cells");
1486 WARNING_PROPERTY_PHANDLE_CELLS(interrupts_extended, "interrupts-extended", "#interrupt-cells");
1487 WARNING_PROPERTY_PHANDLE_CELLS(io_channels, "io-channels", "#io-channel-cells");
1488 WARNING_PROPERTY_PHANDLE_CELLS(iommus, "iommus", "#iommu-cells");
1489 WARNING_PROPERTY_PHANDLE_CELLS(mboxes, "mboxes", "#mbox-cells");
1490 WARNING_PROPERTY_PHANDLE_CELLS(msi_parent, "msi-parent", "#msi-cells", true);
1491 WARNING_PROPERTY_PHANDLE_CELLS(mux_controls, "mux-controls", "#mux-control-cells");
1492 WARNING_PROPERTY_PHANDLE_CELLS(phys, "phys", "#phy-cells");
1493 WARNING_PROPERTY_PHANDLE_CELLS(power_domains, "power-domains", "#power-domain-cells");
1494 WARNING_PROPERTY_PHANDLE_CELLS(pwms, "pwms", "#pwm-cells");
1495 WARNING_PROPERTY_PHANDLE_CELLS(resets, "resets", "#reset-cells");
1496 WARNING_PROPERTY_PHANDLE_CELLS(sound_dai, "sound-dai", "#sound-dai-cells");
1497 WARNING_PROPERTY_PHANDLE_CELLS(thermal_sensors, "thermal-sensors", "#thermal-sensor-cells");
1499 static bool prop_is_gpio(struct property *prop)
1502 * *-gpios and *-gpio can appear in property names,
1503 * so skip over any false matches (only one known ATM)
1505 if (strends(prop->name, ",nr-gpios"))
1508 return strends(prop->name, "-gpios") ||
1509 streq(prop->name, "gpios") ||
1510 strends(prop->name, "-gpio") ||
1511 streq(prop->name, "gpio");
1514 static void check_gpios_property(struct check *c,
1515 struct dt_info *dti,
1518 struct property *prop;
1520 /* Skip GPIO hog nodes which have 'gpios' property */
1521 if (get_property(node, "gpio-hog"))
1524 for_each_property(node, prop) {
1525 struct provider provider;
1527 if (!prop_is_gpio(prop))
1530 provider.prop_name = prop->name;
1531 provider.cell_name = "#gpio-cells";
1532 provider.optional = false;
1533 check_property_phandle_args(c, dti, node, prop, &provider);
1537 WARNING(gpios_property, check_gpios_property, NULL, &phandle_references);
1539 static void check_deprecated_gpio_property(struct check *c,
1540 struct dt_info *dti,
1543 struct property *prop;
1545 for_each_property(node, prop) {
1546 if (!prop_is_gpio(prop))
1549 if (!strends(prop->name, "gpio"))
1552 FAIL_PROP(c, dti, node, prop,
1553 "'[*-]gpio' is deprecated, use '[*-]gpios' instead");
1557 CHECK(deprecated_gpio_property, check_deprecated_gpio_property, NULL);
1559 static bool node_is_interrupt_provider(struct node *node)
1561 struct property *prop;
1563 prop = get_property(node, "interrupt-controller");
1567 prop = get_property(node, "interrupt-map");
1574 static void check_interrupt_provider(struct check *c,
1575 struct dt_info *dti,
1578 struct property *prop;
1579 bool irq_provider = node_is_interrupt_provider(node);
1581 prop = get_property(node, "#interrupt-cells");
1582 if (irq_provider && !prop) {
1584 "Missing '#interrupt-cells' in interrupt provider");
1588 if (!irq_provider && prop) {
1590 "'#interrupt-cells' found, but node is not an interrupt provider");
1594 WARNING(interrupt_provider, check_interrupt_provider, NULL, &interrupts_extended_is_cell);
1596 static void check_interrupt_map(struct check *c,
1597 struct dt_info *dti,
1600 struct node *root = dti->dt;
1601 struct property *prop, *irq_map_prop;
1602 size_t cellsize, cell, map_cells;
1604 irq_map_prop = get_property(node, "interrupt-map");
1608 if (node->addr_cells < 0) {
1610 "Missing '#address-cells' in interrupt-map provider");
1613 cellsize = node_addr_cells(node);
1614 cellsize += propval_cell(get_property(node, "#interrupt-cells"));
1616 prop = get_property(node, "interrupt-map-mask");
1617 if (prop && (prop->val.len != (cellsize * sizeof(cell_t))))
1618 FAIL_PROP(c, dti, node, prop,
1619 "property size (%d) is invalid, expected %zu",
1620 prop->val.len, cellsize * sizeof(cell_t));
1622 if (!is_multiple_of(irq_map_prop->val.len, sizeof(cell_t))) {
1623 FAIL_PROP(c, dti, node, irq_map_prop,
1624 "property size (%d) is invalid, expected multiple of %zu",
1625 irq_map_prop->val.len, sizeof(cell_t));
1629 map_cells = irq_map_prop->val.len / sizeof(cell_t);
1630 for (cell = 0; cell < map_cells; ) {
1631 struct node *provider_node;
1632 struct property *cellprop;
1634 size_t parent_cellsize;
1636 if ((cell + cellsize) >= map_cells) {
1637 FAIL_PROP(c, dti, node, irq_map_prop,
1638 "property size (%d) too small, expected > %zu",
1639 irq_map_prop->val.len, (cell + cellsize) * sizeof(cell_t));
1644 phandle = propval_cell_n(irq_map_prop, cell);
1645 if (!phandle_is_valid(phandle)) {
1646 /* Give up if this is an overlay with external references */
1647 if (!(dti->dtsflags & DTSF_PLUGIN))
1648 FAIL_PROP(c, dti, node, irq_map_prop,
1649 "Cell %zu is not a phandle(%d)",
1654 provider_node = get_node_by_phandle(root, phandle);
1655 if (!provider_node) {
1656 FAIL_PROP(c, dti, node, irq_map_prop,
1657 "Could not get phandle(%d) node for (cell %zu)",
1662 cellprop = get_property(provider_node, "#interrupt-cells");
1664 parent_cellsize = propval_cell(cellprop);
1666 FAIL(c, dti, node, "Missing property '#interrupt-cells' in node %s or bad phandle (referred from interrupt-map[%zu])",
1667 provider_node->fullpath, cell);
1671 cellprop = get_property(provider_node, "#address-cells");
1673 parent_cellsize += propval_cell(cellprop);
1675 cell += 1 + parent_cellsize;
1678 WARNING(interrupt_map, check_interrupt_map, NULL, &phandle_references, &addr_size_cells, &interrupt_provider);
1680 static void check_interrupts_property(struct check *c,
1681 struct dt_info *dti,
1684 struct node *root = dti->dt;
1685 struct node *irq_node = NULL, *parent = node;
1686 struct property *irq_prop, *prop = NULL;
1687 cell_t irq_cells, phandle;
1689 irq_prop = get_property(node, "interrupts");
1693 if (!is_multiple_of(irq_prop->val.len, sizeof(cell_t)))
1694 FAIL_PROP(c, dti, node, irq_prop, "size (%d) is invalid, expected multiple of %zu",
1695 irq_prop->val.len, sizeof(cell_t));
1697 while (parent && !prop) {
1698 if (parent != node && node_is_interrupt_provider(parent)) {
1703 prop = get_property(parent, "interrupt-parent");
1705 phandle = propval_cell(prop);
1706 if (!phandle_is_valid(phandle)) {
1707 /* Give up if this is an overlay with
1708 * external references */
1709 if (dti->dtsflags & DTSF_PLUGIN)
1711 FAIL_PROP(c, dti, parent, prop, "Invalid phandle");
1715 irq_node = get_node_by_phandle(root, phandle);
1717 FAIL_PROP(c, dti, parent, prop, "Bad phandle");
1720 if (!node_is_interrupt_provider(irq_node))
1721 FAIL(c, dti, irq_node,
1722 "Missing interrupt-controller or interrupt-map property");
1727 parent = parent->parent;
1731 FAIL(c, dti, node, "Missing interrupt-parent");
1735 prop = get_property(irq_node, "#interrupt-cells");
1737 /* We warn about that already in another test. */
1741 irq_cells = propval_cell(prop);
1742 if (!is_multiple_of(irq_prop->val.len, irq_cells * sizeof(cell_t))) {
1743 FAIL_PROP(c, dti, node, prop,
1744 "size is (%d), expected multiple of %d",
1745 irq_prop->val.len, (int)(irq_cells * sizeof(cell_t)));
1748 WARNING(interrupts_property, check_interrupts_property, &phandle_references);
1750 static const struct bus_type graph_port_bus = {
1751 .name = "graph-port",
1754 static const struct bus_type graph_ports_bus = {
1755 .name = "graph-ports",
1758 static void check_graph_nodes(struct check *c, struct dt_info *dti,
1763 for_each_child(node, child) {
1764 if (!(strprefixeq(child->name, child->basenamelen, "endpoint") ||
1765 get_property(child, "remote-endpoint")))
1768 node->bus = &graph_port_bus;
1770 /* The parent of 'port' nodes can be either 'ports' or a device */
1771 if (!node->parent->bus &&
1772 (streq(node->parent->name, "ports") || get_property(node, "reg")))
1773 node->parent->bus = &graph_ports_bus;
1779 WARNING(graph_nodes, check_graph_nodes, NULL);
1781 static void check_graph_child_address(struct check *c, struct dt_info *dti,
1787 if (node->bus != &graph_ports_bus && node->bus != &graph_port_bus)
1790 for_each_child(node, child) {
1791 struct property *prop = get_property(child, "reg");
1793 /* No error if we have any non-zero unit address */
1794 if (prop && propval_cell(prop) != 0)
1800 if (cnt == 1 && node->addr_cells != -1)
1801 FAIL(c, dti, node, "graph node has single child node '%s', #address-cells/#size-cells are not necessary",
1802 node->children->name);
1804 WARNING(graph_child_address, check_graph_child_address, NULL, &graph_nodes);
1806 static void check_graph_reg(struct check *c, struct dt_info *dti,
1810 const char *unitname = get_unitname(node);
1811 struct property *prop;
1813 prop = get_property(node, "reg");
1814 if (!prop || !unitname)
1817 if (!(prop->val.val && prop->val.len == sizeof(cell_t))) {
1818 FAIL(c, dti, node, "graph node malformed 'reg' property");
1822 snprintf(unit_addr, sizeof(unit_addr), "%x", propval_cell(prop));
1823 if (!streq(unitname, unit_addr))
1824 FAIL(c, dti, node, "graph node unit address error, expected \"%s\"",
1827 if (node->parent->addr_cells != 1)
1828 FAIL_PROP(c, dti, node, get_property(node, "#address-cells"),
1829 "graph node '#address-cells' is %d, must be 1",
1830 node->parent->addr_cells);
1831 if (node->parent->size_cells != 0)
1832 FAIL_PROP(c, dti, node, get_property(node, "#size-cells"),
1833 "graph node '#size-cells' is %d, must be 0",
1834 node->parent->size_cells);
1837 static void check_graph_port(struct check *c, struct dt_info *dti,
1840 if (node->bus != &graph_port_bus)
1843 if (!strprefixeq(node->name, node->basenamelen, "port"))
1844 FAIL(c, dti, node, "graph port node name should be 'port'");
1846 check_graph_reg(c, dti, node);
1848 WARNING(graph_port, check_graph_port, NULL, &graph_nodes);
1850 static struct node *get_remote_endpoint(struct check *c, struct dt_info *dti,
1851 struct node *endpoint)
1855 struct property *prop;
1857 prop = get_property(endpoint, "remote-endpoint");
1861 phandle = propval_cell(prop);
1862 /* Give up if this is an overlay with external references */
1863 if (!phandle_is_valid(phandle))
1866 node = get_node_by_phandle(dti->dt, phandle);
1868 FAIL_PROP(c, dti, endpoint, prop, "graph phandle is not valid");
1873 static void check_graph_endpoint(struct check *c, struct dt_info *dti,
1876 struct node *remote_node;
1878 if (!node->parent || node->parent->bus != &graph_port_bus)
1881 if (!strprefixeq(node->name, node->basenamelen, "endpoint"))
1882 FAIL(c, dti, node, "graph endpoint node name should be 'endpoint'");
1884 check_graph_reg(c, dti, node);
1886 remote_node = get_remote_endpoint(c, dti, node);
1890 if (get_remote_endpoint(c, dti, remote_node) != node)
1891 FAIL(c, dti, node, "graph connection to node '%s' is not bidirectional",
1892 remote_node->fullpath);
1894 WARNING(graph_endpoint, check_graph_endpoint, NULL, &graph_nodes);
1896 static struct check *check_table[] = {
1897 &duplicate_node_names, &duplicate_property_names,
1898 &node_name_chars, &node_name_format, &property_name_chars,
1899 &name_is_string, &name_properties, &node_name_vs_property_name,
1904 &phandle_references, &path_references,
1907 &address_cells_is_cell, &size_cells_is_cell,
1908 &device_type_is_string, &model_is_string, &status_is_string,
1911 &compatible_is_string_list, &names_is_string_list,
1913 &property_name_chars_strict,
1914 &node_name_chars_strict,
1916 &addr_size_cells, ®_format, &ranges_format, &dma_ranges_format,
1918 &unit_address_vs_reg,
1919 &unit_address_format,
1923 &pci_device_bus_num,
1934 &avoid_default_addr_size,
1935 &avoid_unnecessary_addr_size,
1936 &unique_unit_address,
1937 &unique_unit_address_if_enabled,
1938 &obsolete_chosen_interrupt_controller,
1939 &chosen_node_is_root, &chosen_node_bootargs, &chosen_node_stdout_path,
1943 &cooling_device_property,
1944 &cooling_device_is_cell,
1949 &interrupts_extended_property,
1950 &interrupts_extended_is_cell,
1951 &io_channels_property,
1952 &io_channels_is_cell,
1957 &msi_parent_property,
1958 &msi_parent_is_cell,
1959 &mux_controls_property,
1960 &mux_controls_is_cell,
1963 &power_domains_property,
1964 &power_domains_is_cell,
1969 &sound_dai_property,
1971 &thermal_sensors_property,
1972 &thermal_sensors_is_cell,
1974 &deprecated_gpio_property,
1976 &interrupts_property,
1977 &interrupt_provider,
1982 &graph_nodes, &graph_child_address, &graph_port, &graph_endpoint,
1987 static void enable_warning_error(struct check *c, bool warn, bool error)
1991 /* Raising level, also raise it for prereqs */
1992 if ((warn && !c->warn) || (error && !c->error))
1993 for (i = 0; i < c->num_prereqs; i++)
1994 enable_warning_error(c->prereq[i], warn, error);
1996 c->warn = c->warn || warn;
1997 c->error = c->error || error;
2000 static void disable_warning_error(struct check *c, bool warn, bool error)
2004 /* Lowering level, also lower it for things this is the prereq
2006 if ((warn && c->warn) || (error && c->error)) {
2007 for (i = 0; i < ARRAY_SIZE(check_table); i++) {
2008 struct check *cc = check_table[i];
2011 for (j = 0; j < cc->num_prereqs; j++)
2012 if (cc->prereq[j] == c)
2013 disable_warning_error(cc, warn, error);
2017 c->warn = c->warn && !warn;
2018 c->error = c->error && !error;
2021 void parse_checks_option(bool warn, bool error, const char *arg)
2024 const char *name = arg;
2027 if ((strncmp(arg, "no-", 3) == 0)
2028 || (strncmp(arg, "no_", 3) == 0)) {
2033 for (i = 0; i < ARRAY_SIZE(check_table); i++) {
2034 struct check *c = check_table[i];
2036 if (streq(c->name, name)) {
2038 enable_warning_error(c, warn, error);
2040 disable_warning_error(c, warn, error);
2045 die("Unrecognized check name \"%s\"\n", name);
2048 void process_checks(bool force, struct dt_info *dti)
2053 for (i = 0; i < ARRAY_SIZE(check_table); i++) {
2054 struct check *c = check_table[i];
2056 if (c->warn || c->error)
2057 error = error || run_check(c, dti);
2062 fprintf(stderr, "ERROR: Input tree has errors, aborting "
2063 "(use -f to force output)\n");
2065 } else if (quiet < 3) {
2066 fprintf(stderr, "Warning: Input tree has errors, "