Hi Simon,
On 3/7/20 13:08, Simon Glass wrote:
Hi Walter,
On Fri, 3 Jul 2020 at 05:07, Walter Lozano walter.lozano@collabora.com wrote:
Currently dtoc scans dtbs to convert them to struct platdata and to generate U_BOOT_DEVICE entries. These entries need to be filled with the driver name, but at this moment the information used is the compatible name present in the dtb. This causes that only nodes with a compatible name that matches a driver name generate a working entry.
In order to improve this behaviour, this patch adds to dtoc the capability of scan drivers source code to generate a list of valid driver names and aliases. This allows to generate U_BOOT_DEVICE entries using valid driver names and rise a warning in the case a name is not valid.
Signed-off-by: Walter Lozano walter.lozano@collabora.com
(no changes since v1)
tools/dtoc/dtb_platdata.py | 91 +++++++++++++++++++++++++-- tools/dtoc/dtoc_test_driver_alias.dts | 20 ++++++ tools/dtoc/test_dtoc.py | 33 ++++++++++ 3 files changed, 140 insertions(+), 4 deletions(-) create mode 100644 tools/dtoc/dtoc_test_driver_alias.dts
This seems to do the trick, thanks!
But there is one problem remaining - lots of warning output from the various tests:
The warnings are removed by the next patch in the series, "[PATCH v4 05/14] dtoc: add option to disable warnings", by adding additional arguments.
I thought having these two things in different patches made sense, but If you prefer I can re send the series with those two patches squashed.
Please confirm.
Regards,
Walter