Hi Simon,

On 3/7/20 13:08, Simon Glass wrote:
Hi Walter,

On Fri, 3 Jul 2020 at 05:07, Walter Lozano <walter.loz...@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.loz...@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:

https://pastebin.com/K7GBPiqC

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

Reply via email to