RE: [EXT] Re: [PATCH 1/1] PCI: layerscape: Add the workaround for A-010305

From: Frank Li <frank.li@nxp.com>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: "M.H. Lian" <minghuan.lian@nxp.com>,
	"Mingkai Hu" <mingkai.hu@nxp.com>, "Roy Zang" <roy.zang@nxp.com>,
	"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"open list:PCI DRIVER FOR FREESCALE LAYERSCAPE"
	<linuxppc-dev@lists.ozlabs.org>,
	"open list:PCI DRIVER FOR FREESCALE LAYERSCAPE"
	<linux-pci@vger.kernel.org>,
	"moderated list:PCI DRIVER FOR FREESCALE LAYERSCAPE"
	<linux-arm-kernel@lists.infradead.org>,
	"open list" <linux-kernel@vger.kernel.org>,
	"imx@lists.linux.dev" <imx@lists.linux.dev>
Subject: RE: [EXT] Re: [PATCH 1/1] PCI: layerscape: Add the workaround for A-010305
Date: Fri, 17 Mar 2023 15:58:13 +0000	[thread overview]
Message-ID: <AM6PR04MB4838BBCA8E49EE13C9545F4388BD9@AM6PR04MB4838.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <20230314203249.GA1673140@bhelgaas>

> >       pci->ep.ops = &ls_pcie_ep_ops;
> >
> > +     pcie->big_endian = of_property_read_bool(dev->of_node, "big-
> endian");
> 
> Somewhat surprising that 6c389328c985 ("dt-bindings: pci:
> layerscape-pci: Add a optional property big-endian") added this
> property a year ago, but it has been unused until now?
> 

No, it also for pci host part. Zhiqiang send patch 
https://lore.kernel.org/lkml/20210407030948.3845-1-Zhiqiang.Hou@nxp.com/

Not sure why bind-doc accepted, but driver code patch have not accepted. 

The same case happen at https://lore.kernel.org/imx/20230209151050.233973-1-Frank.Li@nxp.com/T/#t

I tried repost the missed part. But no any response over months.  The above one is just one line change. 

> > --
> > 2.34.1
> >