mbox series

[v2,0/3] Adjust how autoprobe is implemented

Message ID 20241101115028.214678-1-sjg@chromium.org
Headers show
Series Adjust how autoprobe is implemented | expand

Message

Simon Glass Nov. 1, 2024, 11:50 a.m. UTC 
This little series makes a minor change to how autoprobe is
implemeneted, as discussed on the list:

   https://patchwork.ozlabs.org/project/uboot/patch/
      20240626235717.272219-1-marex@denx.de/


Changes in v2:
- Add autoprobe to SPL also
- Leave the function name the same
- Fix 'Prove' typo
- Update cover letter since SPL is now covered also
- Update the documentation too, with a discussion link

Simon Glass (3):
  common: Drop check for DM in initf_dm()
  dm: core: Simplify dm_probe_devices()
  common: Move autoprobe out to board init

 common/board_f.c                    |  9 +++++++--
 common/board_r.c                    |  4 ++++
 common/spl/spl.c                    |  4 ++++
 doc/develop/driver-model/design.rst | 15 +++++++++++++++
 drivers/core/root.c                 | 27 ++++++++++++++++-----------
 include/dm/root.h                   | 10 ++++++++++
 6 files changed, 56 insertions(+), 13 deletions(-)

Comments

Tom Rini Nov. 1, 2024, 2:13 p.m. UTC | #1 
On Fri, Nov 01, 2024 at 12:50:23PM +0100, Simon Glass wrote:

> This little series makes a minor change to how autoprobe is
> implemeneted, as discussed on the list:
> 
>    https://patchwork.ozlabs.org/project/uboot/patch/
>       20240626235717.272219-1-marex@denx.de/

Patchwork is not great for re-reading discussions, please use lore:
https://lore.kernel.org/u-boot/20240626235717.272219-1-marex@denx.de/
(and since patchwork uses message-id for URLs now, it's easy to find the
lore URL if you're looking at a patchwork link.

And now that I know this is what I thought it was, this needs Marek's
Tested-by that it doesn't break the use case he was solving.