On Tue, 26 May 2026 10:52:44 +0200 David Marchand <[email protected]> wrote:
> Introduce a new rte_bus_probe_device_t operation with signature > (struct rte_driver *drv, struct rte_device *dev). > > Replace the existing .plug field in the struct rte_bus with .probe_device. > > Update all in-tree buses to use .probe_device instead of .plug. > Each bus probe() function now calls rte_bus_find_driver() (which uses the > match operation added in previous commit) and passes the found driver > to bus.probe_device(driver, device). > > Signed-off-by: David Marchand <[email protected]> > Acked-by: Bruce Richardson <[email protected]> Acked-by: Stephen Hemminger <[email protected]> FYI - deep dive (more than normal) AI review had these minor findings. I would ignore it. That leaves the following items from the full series review: Patch 13 style nit: dsa_match in drivers/dma/idxd/idxd_bus.c has return type and brace on the same line, inconsistent with surrounding functions in the file. Info-level. Patch 16 (NXP scan init): fslmc process_once = 1 is moved from immediately after the early-return check to the end of the function. If any of the new in-scan init steps fails the function returns 0 early and process_once stays 0, allowing re-scan on the next call. In current EAL flows scan() is called exactly once so this is unreachable. Worth fixing for defensive consistency; not blocking.
