All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Johan Jonker <jbx6244@gmail.com>, Simon Glass <sjg@chromium.org>,
	Walter Lozano <walter.lozano@collabora.com>
Subject: [PATCH 0/8] dtoc: Improvements to warnings
Date: Sun,  4 Jul 2021 12:19:42 -0600	[thread overview]
Message-ID: <20210704181950.1917106-1-sjg@chromium.org> (raw)

The warning about a missing driver can be a bit confusing and is not
specifically mentioned in the documentation. Also some parse errors result
in the driver simply being ignored, which is hard to diagnose.

This series updates dtoc to try harder to find common errors and report a
useful warning. It also adds a section about possible problems to the
of-platdata documentation.

Unfortunately the use of match-object subscripts is not supported in
Python 3.5 and earlier, so a patch is included to avoid this.

Finally, a short usage string it added to the help output, with dtoc being
converted to use ArgumentParser in the process.


Simon Glass (8):
  dtoc: Avoid using subscripts on match objects
  dtoc: Convert to use ArgumentParser
  dtoc: Allow multiple warnings for a driver
  dtoc: Correct the re_compat regular expression
  dtoc: Add a stdout check in test_normalized_name()
  dtoc: Detect unexpected suffix on .of_match
  dtoc: Detect drivers which do not parse correctly
  dtoc: Update documentation to cover warnings in more detail

 doc/develop/driver-model/of-plat.rst |  53 +++++++++++
 tools/dtoc/main.py                   |  51 +++++-----
 tools/dtoc/src_scan.py               |  45 +++++++--
 tools/dtoc/test_src_scan.py          | 133 ++++++++++++++++++++++++++-
 4 files changed, 247 insertions(+), 35 deletions(-)

-- 
2.32.0.93.g670b81a890-goog


             reply	other threads:[~2021-07-04 18:20 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-04 18:19 Simon Glass [this message]
2021-07-04 18:19 ` [PATCH 1/8] dtoc: Avoid using subscripts on match objects Simon Glass
2021-07-05 18:04   ` Walter Lozano
2021-07-11 23:00   ` Simon Glass
2021-07-04 18:19 ` [PATCH 2/8] dtoc: Convert to use ArgumentParser Simon Glass
2021-07-05 18:04   ` Walter Lozano
2021-07-11 23:00   ` Simon Glass
2021-07-04 18:19 ` [PATCH 3/8] dtoc: Allow multiple warnings for a driver Simon Glass
2021-07-05 18:04   ` Walter Lozano
2021-07-11 23:00   ` Simon Glass
2021-07-12  2:23     ` Simon Glass
2021-07-04 18:19 ` [PATCH 4/8] dtoc: Correct the re_compat regular expression Simon Glass
2021-07-05 18:04   ` Walter Lozano
2021-07-11 23:00   ` Simon Glass
2021-07-04 18:19 ` [PATCH 5/8] dtoc: Add a stdout check in test_normalized_name() Simon Glass
2021-07-04 18:19 ` [PATCH 6/8] dtoc: Detect unexpected suffix on .of_match Simon Glass
2021-07-04 18:19 ` [PATCH 7/8] dtoc: Detect drivers which do not parse correctly Simon Glass
2021-07-05 18:04   ` Walter Lozano
2021-07-11 23:00   ` Simon Glass
2021-07-04 18:19 ` [PATCH 8/8] dtoc: Update documentation to cover warnings in more detail Simon Glass
2021-07-05 18:04   ` Walter Lozano
2021-07-20 18:32     ` Simon Glass
2021-07-11 23:00   ` Simon Glass
2021-07-11 23:00 ` [PATCH 6/8] dtoc: Detect unexpected suffix on .of_match Simon Glass
2021-07-11 23:00 ` [PATCH 5/8] dtoc: Add a stdout check in test_normalized_name() Simon Glass

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210704181950.1917106-1-sjg@chromium.org \
    --to=sjg@chromium.org \
    --cc=jbx6244@gmail.com \
    --cc=u-boot@lists.denx.de \
    --cc=walter.lozano@collabora.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.