From:   Frank Li <frank.li@nxp.com>
To:     ALOK TIWARI <alok.a.tiwari@oracle.com>,
        "helgaas@kernel.org" <helgaas@kernel.org>
CC:     "bhelgaas@google.com" <bhelgaas@google.com>,
        "imx@lists.linux.dev" <imx@lists.linux.dev>,
        "kw@linux.com" <kw@linux.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        "lpieralisi@kernel.org" <lpieralisi@kernel.org>,
        "M.H. Lian" <minghuan.lian@nxp.com>,
        Mingkai Hu <mingkai.hu@nxp.com>,
        "robh@kernel.org" <robh@kernel.org>, Roy Zang <roy.zang@nxp.com>
Subject: RE: [EXT] Re: [External] : [PATCH v3 1/1] PCI: layerscape: Add EP
 mode support for ls1028a
Thread-Topic: [EXT] Re: [External] : [PATCH v3 1/1] PCI: layerscape: Add EP
 mode support for ls1028a
Thread-Index: AQHZPJi/wYJCGtANx0aZ13DX5flOpa7InnwAgDJG+kA=
Date:   Tue, 14 Mar 2023 19:59:31 +0000
Message-ID: <AM6PR04MB48384886D192DE5339D7B91688BE9@AM6PR04MB4838.eurprd04.prod.outlook.com>
References: <20230209151050.233973-1-Frank.Li@nxp.com>
 <ebda03df-ad02-c89e-882b-03f7514c3d92@oracle.com>
In-Reply-To: <ebda03df-ad02-c89e-882b-03f7514c3d92@oracle.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Zy30y2bN4LUOmF90yqgB4yU6j3WckVa5thTzxzixJJzbk7KpGvZ3agUluoVD?=
 =?us-ascii?Q?byfCGGsWxzdl9AA7zHePPy5SsRf1/219QGvwVqtJSui69NR1cAc3/cABv1Kj?=
 =?us-ascii?Q?t/wWwzTLpN6XKphpSkXbjUYpcyWabWfCJpNpXDiCCGaJhygcY17vRZWTPuCX?=
 =?us-ascii?Q?/8GLUinWilwxM8/7dYwGyiKJhcr3/r9nXCGQ/mqVg5ZyadsNF9oje8r4jYpJ?=
 =?us-ascii?Q?osLgOVLMXbJROZr6P+/8n+kH3ile595qqOQOpH1uc+c/hKLnvfC41FuIw1Yk?=
 =?us-ascii?Q?+IdqIgfEarIyMnsErPHSoK3mU3UCNz0aKagGFfz3mhlrbK6Cip8XR3s6VRPE?=
 =?us-ascii?Q?ew3Rjj3f9nN3BQl+xK7y84YtXsN62mQ5NJfTg1SGqsG+hAUF24iXeDBmrL1n?=
 =?us-ascii?Q?sArJ0FrsfVXGrAkzauS4oAyZQpQVeH42GggVIw9hEl2a4EgZnHsPXIo9fUIL?=
 =?us-ascii?Q?6TDoGoVK+zfBoNlevOZzxhhh/NucreP+24Ytaup0LJKQiQ4fk+xf5C3rAPsb?=
 =?us-ascii?Q?wHX3QbKU57T/g6E51gjLLiIvYyPIukXtjUe7Yk1ywhmhcGm4yKj6LzrJ1yFm?=
 =?us-ascii?Q?vJtWcfXEvPXYc/zI2imyJV4yV5CSDvgnaA3a0Xhg1ERMddF40LxUF9rbyGgP?=
 =?us-ascii?Q?AjqJDVckXhesM6LcYKRDzszqO09hak1Cp7ftNqq+S6LByat+fztut7EXfKnj?=
 =?us-ascii?Q?ilV+/HT7dzQIeFlf/q8VYAqtmXT+nPQLxmjJudE5r0qPo3BT4Ycsw9tUyrDW?=
 =?us-ascii?Q?I4pMJpfxRKZ7RJpk9T6O2/00zYYWL3e9xE6PjULbl0vn+p4sXy2U5NXN37hF?=
 =?us-ascii?Q?VR+XJKAiWtCBQ+3I28CsMdlQ13wwr1SA9NDgzXdR14cOORlFuk0BUvFIjpOW?=
 =?us-ascii?Q?5m0mUn2qI9KVt4VmsuBzn9AmiKg2OcWJXKY3f2/T57wKsrAEw1joP4cWSIYA?=
 =?us-ascii?Q?RALPsHefRJOjjL8wAYp2sDCdWUt4nOE0BjAgvS4F8563Q5dHCE0SZmGpJpyq?=
 =?us-ascii?Q?yXKdlH31e4s5TGESd7+aaYVIcZ8hnpkTq8pJcp3uWqVvpmyqUr+AjaC3h+b+?=
 =?us-ascii?Q?joyJohUy03VS/W9ON0JGCFDEc2PDpFzbqHedoEm268CXgg4HpcwOm7FkUDGc?=
 =?us-ascii?Q?iK0HlOWJ/bUfI5QGqmHdYsB8tP5SFuxHzKxWz+R6V5xveZ+rl4I/fDVbrVdX?=
 =?us-ascii?Q?/efIF2ycxKZj4/uxCvilquMlwjUXa1FkuPJWQBKIY6OV456aj2t6MzfrqDvm?=
 =?us-ascii?Q?pbnSZHjfT5lIVYu3MdHjjjFqD3wwJ9QzLtf/v+n+tkoM9+BqtX3jz5qULwS1?=
 =?us-ascii?Q?IxwYXd1OChFIZDyMopY9qZR6cK6M14VhnpxfNt/UMGJo4qJxECpeqBTGlE57?=
 =?us-ascii?Q?xlZg5FCuLKswsr6ERXpb7QqRp5KBkME9iXEgoreGRMdkW/NxZAswToEz41Ct?=
 =?us-ascii?Q?4WsmIXagX3ZNS6JPLKUaNaaMWxuat343uvJE6fBaOk3j5nuufipVTfacRbnc?=
 =?us-ascii?Q?N7+H2hwTwgb5SEKfZSMpdAfiRiT7LvBCjmx1Wfg06AvlwDgaoFp+IOFIZQeH?=
 =?us-ascii?Q?ull83/548MigU6xdUPo=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 79148b94-2122-42ca-026a-08db24c6a077
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Mar 2023 19:59:31.6710
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: awFwztBYBe+pTgEh8l1PE6XUwdyf3evtyEqXV7TTes5slR4T/Q75s8SyDBskOytyH9M2594F1xG3Hr5hydM4EQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB9124
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

>=20
> Reviewed-by: Alok Tiwari <alok.a.tiwari@oracle.com>
>=20
> On 2/9/2023 8:40 PM, Frank Li wrote:
> > From: Xiaowei Bao <xiaowei.bao@nxp.com>
> >
> > Add PCIe EP mode support for ls1028a.
> >
> > Signed-off-by: Xiaowei Bao <xiaowei.bao@nxp.com>
> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > Signed-off-by: Frank Li <Frank.Li@nxp>
> > Acked-by:  Roy Zang <Roy.Zang@nxp.com>
> > ---

Ping!
there are no feedback for over 1 month.=20
Just 1 line change.=20

> >
> > Change from v2 to v3
> > order by .compatible
> >
> > Change from v2 to v2
> > Added
> > Signed-off-by: Frank Li <Frank.Li@nxp>
> > Acked-by:  Roy Zang <Roy.Zang@nxp.com>
> >
> >
> > All other patches were already accepte by maintainer in
> >
> https://eur01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Furlde
> fense.com%2Fv3%2F__https%3A%2F%2Flore.kernel.org%2Flkml%2F2021111
> 2223457.10599-1-
> leoyang.li%40nxp.com%2F__%3B!!ACWV5N9M2RV99hQ!NR9EU4fPDwxdyrb
> 9tdBm9VNIMHSlw6dLgXCAPDSrm7ftWVNrh6JldLGzzrKyiE0xRlP5OdiGBN7PCf
> 9gRaA%24&data=3D05%7C01%7CFrank.Li%40nxp.com%7C1d32974e205b4a8591
> 9f08db0ba30b9b%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6381
> 16567129733840%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiL
> CJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sd
> ata=3DhGKffPqpE%2Ft66x71Y47ocGbIuFH7vpjLadlAXbnyBOw%3D&reserved=3D0
> >
> > But missed this one.
> >
> > Re-post
> >
> >   drivers/pci/controller/dwc/pci-layerscape-ep.c | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/pci/controller/dwc/pci-layerscape-ep.c
> b/drivers/pci/controller/dwc/pci-layerscape-ep.c
> > index ad99707b3b99..c640db60edc6 100644
> > --- a/drivers/pci/controller/dwc/pci-layerscape-ep.c
> > +++ b/drivers/pci/controller/dwc/pci-layerscape-ep.c
> > @@ -110,6 +110,7 @@ static const struct ls_pcie_ep_drvdata
> lx2_ep_drvdata =3D {
> >   };
> >
> >   static const struct of_device_id ls_pcie_ep_of_match[] =3D {
> > +     { .compatible =3D "fsl,ls1028a-pcie-ep", .data =3D &ls1_ep_drvdat=
a },
> >       { .compatible =3D "fsl,ls1046a-pcie-ep", .data =3D &ls1_ep_drvdat=
a },
> >       { .compatible =3D "fsl,ls1088a-pcie-ep", .data =3D &ls2_ep_drvdat=
a },
> >       { .compatible =3D "fsl,ls2088a-pcie-ep", .data =3D &ls2_ep_drvdat=
a },