[PATCH v2 03/14] dtoc: add support to scan drivers

[Walter Lozano]

Hi Simon,

On 23/6/20 09:28, Simon Glass wrote:
> Hi Walter,
>
> On Fri, 19 Jun 2020 at 08:56, Walter Lozano <walter.lozano at 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. This allows to rise a warning in the case that an U_BOOT_DEVICE
>> entry will try to use a name not valid.
>>
>> Additionally, in order to add more flexibility to the solution, adds the
>> U_BOOT_DRIVER_ALIAS macro, which generates no code at all, but allows an
>> easy way to declare driver name aliases. Thanks to this, dtoc can look
>> for the driver name based on its alias when it populates the U_BOOT_DEVICE
>> entry.
>>
>> Signed-off-by: Walter Lozano <walter.lozano at collabora.com>
>> ---
>>   drivers/clk/at91/pmc.c                |  2 +
>>   drivers/gpio/mxs_gpio.c               |  2 +
>>   drivers/gpio/sandbox.c                |  2 +
>>   drivers/i2c/rk_i2c.c                  |  2 +
>>   drivers/mmc/mxsmmc.c                  |  1 +
>>   drivers/mmc/rockchip_dw_mmc.c         |  3 +
>>   drivers/mtd/spi/sf_probe.c            |  2 +
>>   drivers/pinctrl/nxp/pinctrl-mxs.c     |  2 +
>>   drivers/pinctrl/pinctrl-at91.c        |  2 +
>>   drivers/power/pmic/rk8xx.c            |  2 +
>>   drivers/serial/ns16550.c              |  4 ++
>>   drivers/spi/mxs_spi.c                 |  2 +
>>   drivers/spi/rk_spi.c                  |  2 +
>>   include/dm/device.h                   |  7 +++
>>   tools/dtoc/dtb_platdata.py            | 89 +++++++++++++++++++++++++--
>>   tools/dtoc/dtoc_test_driver_alias.dts | 20 ++++++
>>   tools/dtoc/test_dtoc.py               | 33 ++++++++++
>>   17 files changed, 173 insertions(+), 4 deletions(-)
>>   create mode 100644 tools/dtoc/dtoc_test_driver_alias.dts
>>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> But please can you split out the changes to drivers and device.h into
> a separate, earlier patch? I'd like to keep tools changes separate
> where possible.


Sure, thanks for your suggestion.

Regards,

Walter