[PATCH v5 6/8] PCI: Add FMT, TYPE and CPL status definition for TLP header

Jacky Chou jacky_chou at aspeedtech.com
Wed Nov 19 13:27:20 AEDT 2025


Hi Bjorn,

Thank you for your reply.

> > According to PCIe specification, add FMT, TYPE and CPL status
> > definition for TLP header.
> >
> > Signed-off-by: Jacky Chou <jacky_chou at aspeedtech.com>
> 
> Acked-by: Bjorn Helgaas <bhelgaas at google.com>
> 
> OK by me, but it'd be nice to move up a few lines so this is with the other
> TLP-related items and the unrelated PCI_BUS_BRIDGE_*_WINDOW values
> aren't in the middle.
> 
> Might even consider moving these to be just above the Message Routing
> constants so things are generally in the order they appear in the spec.
> 

I'll move these above the Message Routing constants in the next version.
Thanks for the suggestion.

Thanks,
Jacky


More information about the Linux-aspeed mailing list