[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<SEYPR06MB51340B04D9423EDEABFE9BA79DD7A@SEYPR06MB5134.apcprd06.prod.outlook.com>
Date: Wed, 19 Nov 2025 02:27:20 +0000
From: Jacky Chou <jacky_chou@...eedtech.com>
To: Bjorn Helgaas <helgaas@...nel.org>
CC: Vinod Koul <vkoul@...nel.org>, Kishon Vijay Abraham I <kishon@...nel.org>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Joel Stanley <joel@....id.au>, Andrew
Jeffery <andrew@...econstruct.com.au>, Bjorn Helgaas <bhelgaas@...gle.com>,
Lorenzo Pieralisi <lpieralisi@...nel.org>,
Krzysztof WilczyĆski <kwilczynski@...nel.org>, Manivannan
Sadhasivam <mani@...nel.org>, Linus Walleij <linus.walleij@...aro.org>,
Philipp Zabel <p.zabel@...gutronix.de>, "linux-aspeed@...ts.ozlabs.org"
<linux-aspeed@...ts.ozlabs.org>, "linux-pci@...r.kernel.org"
<linux-pci@...r.kernel.org>, "linux-phy@...ts.infradead.org"
<linux-phy@...ts.infradead.org>, "devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, Andrew Jeffery <andrew@...id.au>,
"openbmc@...ts.ozlabs.org" <openbmc@...ts.ozlabs.org>,
"linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>
Subject: [PATCH v5 6/8] PCI: Add FMT, TYPE and CPL status definition for TLP
header
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@...eedtech.com>
>
> Acked-by: Bjorn Helgaas <bhelgaas@...gle.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
Powered by blists - more mailing lists