On Fri Feb 6, 2026 at 6:20 PM CET, Gary Guo wrote: > I asked for this to be changed to `pub(crate)` because I think this isn't > something that should be used by drivers. > > As you said, tt might be tricky to enforce that with new build system when > subsystems are inside different crates. But until then I think it's better to > limit visibility.
It should *usually* not be used by drivers, but there are exceptions. For instance, it is perfectly valid to be used by Rust drivers that interact with C drivers. Besides that, my take on this is that we know that the new build system will come and that pub(crate) won't work in the long term, so why bother. I'd only use it in cases where you want to keep things local to the subsystem.
