Hi Neil, On Tue, 17 Mar 2020 at 08:09, Niel Fourie <lu...@denx.de> wrote: > > This series adds commands for listing the supported partition tables, > listing supported filesystems and expands Driver Model listing commands. > > The existing "dm drivers" command, which lists the DM drivers and their > compatibility strings, segmentation faulted on drivers for which of_match > was unpopulated (which appears to not be uncommon). This was fixed, and > the command was renamed "dm compat", and a new more extensive "dm drivers" > command was added, which list all DM drivers and for each, their uclass > id, uclass driver and the device names for active driver instances. The > purpose is show available drivers, but also to highlight unused drivers > or drivers with uclass ids without uclass drivers, etc. > > The following commands were added: > -"part types", lists partition tables supported > -"fstypes", lists filesystem types supported > -"dm compat", lists drivers and their compatibility strings (equivalent > to existing "dm drivers" command) > -"dm drivers", lists all DM drivers, and for each their uclass id, > uclass driver and the device names for active driver instances. > -"dm static", lists all DM drivers which use static platform data > (instead of the device tree). > > These patches were tested in the Sandbox and on the Wandboard > i.MX6Quad Board rev B1.
This all looks very useful thank you. Can you please add sandbox tests for the commands? For an example, see https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/blob/coral-working/test/dm/acpi.c The test is dm_test_acpi_cmd_items() Regards, Simon > > Niel Fourie (3): > cmd: part: Add subcommand to list supported partition tables > cmd: fs: Add command to list supported fs types > cmd: dm: Fixed/Added DM driver listing subcommands > > cmd/dm.c | 24 ++++++++++++++++-- > cmd/fs.c | 11 +++++++++ > cmd/part.c | 27 ++++++++++++++++++-- > drivers/core/dump.c | 60 ++++++++++++++++++++++++++++++++++++++++++++- > fs/fs.c | 20 +++++++++++++++ > include/dm/util.h | 6 +++++ > include/fs.h | 5 ++++ > 7 files changed, 148 insertions(+), 5 deletions(-) > > -- > 2.24.1 >
