On Fri, 2 May 2025 at 04:33, Nicholas Piggin <npig...@gmail.com> wrote: > > To prepare to support another USB PCI Host Controller, make some PCI > configuration dynamic. > > Signed-off-by: Nicholas Piggin <npig...@gmail.com>
I think this patch is trying to do too many things at once. It's OK to bundle the small "turn a constant into a state struct field" changes together, but the ones that are more extensive code logic changes should be split out. In particular "add support for pm capability" should be its own patch as that's a new feature, and it looks like there's also a change in here that's fixing a TODO comment? thanks -- PMM
