u-boot.git/tools/dtoc/test_src_scan.py, branch v2022.01

dtoc: Detect drivers which do not parse correctly

2021-07-21T16:27:34Z

At present if a driver is missing a uclass or compatible stirng, this is silently ignored. This makes sense in most cases, particularly for the compatible string, since it is not required except when the driver is used with of-platdata. But it is also not very helpful. When there is some sort of problem with a driver, the missing compatible string (for example) may be the cause. Add a warning in this case, showing it only for drivers which are used by the build. Signed-off-by: Simon Glass Reviewed-by: Walter Lozano

dtoc: Detect unexpected suffix on .of_match

2021-07-21T16:27:34Z

Some rockchip drivers use a suffix on the of_match line which is not strictly valid. At present this causes the parsing to fail. Fix this and offer a warning. Signed-off-by: Simon Glass

dtoc: Add a stdout check in test_normalized_name()

2021-07-21T16:27:34Z

This test captures output but does not always check it. Add the missing code and drop the old comment. Signed-off-by: Simon Glass

dtoc: Show driver warnings once at the end

2021-04-06T04:33:19Z

At present warnings are shown as soon as they are discovered in the source scannner. But the function that detects them may be called multiple times. Collect all the warnings and show them at the end. Signed-off-by: Simon Glass

dtoc: Support processing the root node

2021-03-22T06:23:27Z

The device for the root node is normally bound by driver model on init. With devices being instantiated at build time, we must handle the root device also. Add support for processing the root node, which may not have a compatible string. Signed-off-by: Simon Glass

dtoc: Set up the uclasses that are used

2021-03-22T06:23:27Z

We only care about uclasses that are actually used. This is determined by the drivers that use them. Check all the used drivers and build a list of 'valid' uclasses. Also add references to the uclasses so we can generate C code that uses them. Attach a uclass to each valid driver. For the tests, now that we have uclasses we must create an explicit test for the case where a node does not have one. This should only happen if the source code does not build, or the source-code scanning fails to find it. Signed-off-by: Simon Glass

dtoc: Warn of duplicate drivers

2021-03-22T06:23:27Z

If drivers have the same name then we cannot distinguish them. This only matters if the driver is actually used by dtoc, but in that case, issue a warning. Signed-off-by: Simon Glass

dtoc: Support headers needed for drivers

2021-03-22T06:23:27Z

Typically dtoc can detect the header file needed for a driver by looking for the structs that it uses. For example, if a driver as a .priv_auto that uses 'struct serial_priv', then dtoc can search header files for the definition of that struct and use the file. In some cases, enums are used in drivers, typically with the .data field of struct udevice_id. Since dtoc does not support searching for these, add a way to tell dtoc which header to use. This works as a macro included in the driver definition. Signed-off-by: Simon Glass

dtoc: Support tracking the phase of U-Boot

2021-03-22T06:23:27Z

U-Boot operates in several phases, typically TPL, SPL and U-Boot proper. The latter does not use dtoc. In some rare cases different drivers are used for two phases. For example, in TPL it may not be necessary to use the full PCI subsystem, so a simple driver can be used instead. This works in the build system simply by compiling in one driver or the other (e.g. PCI driver + uclass for SPL; simple_bus for TPL). But dtoc has no way of knowing which code is compiled in for which phase, since it does not inspect Makefiles or dependency graphs. So to make this work for dtoc, we need to be able to explicitly mark drivers with their phase. This is done by adding an empty macro to the driver. Add support for this in dtoc. Signed-off-by: Simon Glass

dtoc: Track nodes which are actually used

2021-03-22T06:23:27Z

Mark all nodes that are actually used, so we can perform extra checks on them. Signed-off-by: Simon Glass