[PATCH v3] cmd: Add command to dump drivers and compatible strings

[Sean Anderson]

On 2/19/20 10:45 AM, Bin Meng wrote:
> Hi Simon,
> 
> On Fri, Feb 7, 2020 at 1:46 AM Simon Glass <sjg at chromium.org> wrote:
>>
>> On Thu, 6 Feb 2020 at 10:03, Sean Anderson <seanga2 at gmail.com> wrote:
>>>
>>> This adds a subcommand to dm to dump out what drivers are installed, and their
>>> compatible strings. I have found this useful in ensuring that I have the correct
>>> drivers compiled, and that I have put in the correct compatible strings.
>>>
>>> Signed-off-by: Sean Anderson <seanga2 at gmail.com>
>>> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
>>> Tested-by: Bin Meng <bmeng.cn at gmail.com>
>>>
>>> ---
>>>
>>> Changes in v3:
>>> - Add python test
>>>
>>> Changes in v2:
>>> - Check if entry->of_match is NULL before accessing it
>>> - Remove trailing newline in help message
>>>
>>>  cmd/dm.c                 | 12 +++++++++++-
>>>  drivers/core/dump.c      | 20 ++++++++++++++++++++
>>>  include/dm/util.h        |  3 +++
>>>  test/py/tests/test_dm.py | 17 +++++++++++++++++
> 
> Is this what you're looking for? I thought we need something in
> test/dm C test cases instead?

Tests for commands seem to be python tests. I considered writing a dm
test, but afiak that seems to be for testing function calls. There does
not seem to be an easy way to grab the console output. Additionally,
where would the authoritative list of installed drivers come from? The
easiest way would be to just use the same api that the command is using,
but this doesn't really test anything. As it is, the current test does
an ok job at correlating different methods of accessing the DM system
from the associated commands. It will also catch null-dereference bugs
like the one I found during testing.

--Sean