On 22.05.2025 02:25, Stefano Stabellini wrote: > On Mon, 19 May 2025, Oleksii Moisieiev wrote: >> + ret = dt_find_node_by_gpath(domctl->u.assign_device.u.dt.path, >> + domctl->u.assign_device.u.dt.size, >> &dev); >> + if ( ret ) >> + return ret; >> + >> + ret = sci_assign_dt_device(d, dev); >> + if ( ret ) >> + break; >> + >> + break; >> + default: >> + /* do not fail here as call is chained with iommu handling */ > > It looks like this should be an error
The way the hooking is done at the call site, I think it can't be an error here. But I previously questioned the way that hooking is done. Jan