[PATCH v5 0/2] dm: core: drivers: add function uclass_probe_all()

Vabhav Sharma (OSS) vabhav.sharma at oss.nxp.com
Wed Dec 16 11:31:44 CET 2020


Dear Sean Anderson,
I have incorporated all your review comments and request your review/ack to merge the changes

Regards,
Vabhav

> -----Original Message-----
> From: Vabhav Sharma <vabhav.sharma at oss.nxp.com>
> Sent: Wednesday, December 9, 2020 10:42 AM
> To: sjg at chromium.org; sr at denx.de; seanga2 at gmail.com
> Cc: u-boot at lists.denx.de; Varun Sethi <V.Sethi at nxp.com>;
> andre.przywara at arm.com; Vabhav Sharma <vabhav.sharma at nxp.com>
> Subject: [PATCH v5 0/2] dm: core: drivers: add function uclass_probe_all()
> 
> From: Vabhav Sharma <vabhav.sharma at nxp.com>
> 
> - Add common method to probe devices belonging to same uclass
> - Add config in serial uclass to support optional inclusion of uclass_probe_all
> - Enable support for available serial devices probe
> 
> Changes for v5:
> Incorporated review comments from Sean Anderson
>   - Added error check
>   - Replaced for() loop with while() loop
>   - Updated macro description
>   - Modified function description
> 
> Changes for v4:
> Incorporated review comments from Simon
>   - Removed if (dev)..  conditional check in function uclass_probe_all()
> 
> Changes for v3:
>   Incorporated Simon and Stephan review comment
>   - Define generic function uclass_probe_all(enum uclass_id)
>     in drivers/core/uclass.c
>   - Added the function in caller of serial_find_console_or_panic()
>   - Removed repeated sequence with generic function call uclass_probe_all()
> 
> Changes for v2:
>   Incorporated Stefan review comment,Update #ifdef with macro if
> (IS_ENABLED).
> 
> Vabhav Sharma (2):
>   dm: core: add function uclass_probe_all() to probe all devices
>   drivers: serial: probe all uart devices
> 
>  drivers/core/uclass.c          | 19 +++++++++++++++++++
>  drivers/serial/Kconfig         | 16 ++++++++++++++++
>  drivers/serial/serial-uclass.c |  9 +++++++++
>  include/dm/uclass.h            | 11 +++++++++++
>  4 files changed, 55 insertions(+)
> 
> --
> 2.7.4



More information about the U-Boot mailing list