Re: [PATCH v2 2/2] PCI: Add PCI_BRIDGE_NO_ALIASES quirk for ASPEED AST1150

[Jason Gunthorpe]

On Wed, Dec 17, 2025 at 07:45:29AM -0800, Nirmoy Das wrote:
> ASPEED BMC controllers have VGA and USB functions behind a PCIe-to-PCI
> bridge that causes them to share the same stream ID:
> 
>   [e0]---00.0-[e1-e2]----00.0-[e2]--+-00.0  ASPEED Graphics Family
>                                     \-02.0  ASPEED USB Controller
> 
> Both devices get stream ID 0x5e200 due to bridge aliasing, causing the
> USB controller to be rejected with 'Aliasing StreamID unsupported'.
> 
> Per ASPEED, the AST1150 doesn't use a real PCI bus and always forwards
> the original requester ID from downstream devices rather than replacing
> it with any alias.
> 
> Add a new PCI_DEV_FLAGS_PCI_BRIDGE_NO_ALIASES flag and apply it to the
> AST1150.
> 
> Suggested-by: Jason Gunthorpe <jgg@xxxxxxxxxx>
> Signed-off-by: Nirmoy Das <nirmoyd@xxxxxxxxxx>
> ---
> v2:
>   - Use new PCI_DEV_FLAGS_PCI_BRIDGE_NO_ALIASES flag instead of
>     PCI_DEV_FLAGS_BRIDGE_XLATE_ROOT to only skip aliasing at this
>     bridge, not stop the entire upstream alias walk (Jason Gunthorpe)
> 
>  drivers/pci/quirks.c | 10 ++++++++++
>  drivers/pci/search.c |  2 ++
>  include/linux/pci.h  |  5 +++++
>  3 files changed, 17 insertions(+)

I bet there will be more aspeed devices to come, but this is a
reasonable start

Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx>

Thanks,
Jason