[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