Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
From:   Roy Zang <roy.zang@nxp.com>
To:     Frank Li <frank.li@nxp.com>,
        "manivannan.sadhasivam@linaro.org" <manivannan.sadhasivam@linaro.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>
Subject: RE: [PATCH v5 2/4] PCI: layerscape: Add suspend/resume for ls1021a
Thread-Topic: [PATCH v5 2/4] PCI: layerscape: Add suspend/resume for ls1021a
Thread-Index: AQHaJHHnnq69VzUD20q7VrwvBWLj1LCUpguQ
Date:   Fri, 1 Dec 2023 16:57:54 +0000
Message-ID: <AM0PR04MB59568824DB8F8509363471B38B81A@AM0PR04MB5956.eurprd04.prod.outlook.com>
References: <20231201161712.1645987-1-Frank.Li@nxp.com>
 <20231201161712.1645987-3-Frank.Li@nxp.com>
In-Reply-To: <20231201161712.1645987-3-Frank.Li@nxp.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?ajpnF2icG0/+QPBB0/sT+mBeQcNO/pu8jtQxN0zKuhjsfaG7ZFKA/laWNaXv?=
 =?us-ascii?Q?glL02RW1wPNYAs+GHgxYWFvPgi1JLDABJEhrSsjrIieOcL7DbwWFqJbY3ukH?=
 =?us-ascii?Q?BXS5GqoSQzgpxy8IyhlsRlYAjuIONXJWBUkYasLu0r48BYEEKkBXk5wKyKtE?=
 =?us-ascii?Q?bt4XtUVabw8rSTsqwI0P0M8Hg3FwFWnFwisCry+5LwILg05VWZZvOfwzzWGy?=
 =?us-ascii?Q?IZT23ReMFV2dAzJeV4TczVYMpcBxUYMqIceI2R7R4l5YjEW6KsvdeqXYgvmo?=
 =?us-ascii?Q?fvZiyG7EGJVw0c4npHtKrMAI8KGkwcwEgfF2RzcCMrka6SKDrP9Yt+oVyK6x?=
 =?us-ascii?Q?r6FIeWHOXiLnnXjUNaX95RS25nZ52urodfyerEaWVcp711V2sg+fqFfcNG06?=
 =?us-ascii?Q?TXKO6KEzPj6IPzHKzFdlEyWHyjcxzCWP3rW/wpSkgch81MBKUaHT+b9fzUJv?=
 =?us-ascii?Q?x3RCtJ9cssFIvkCCtckmPO/lxO5gBKrAgGP2Zab1CvE/E9jM0yk9xxPXDWaT?=
 =?us-ascii?Q?ftu1dYNwhHsfrLhqak8WuypJ3ehgN2hb+1ljdr6luASiX9kaVKhuClIUHVzo?=
 =?us-ascii?Q?9/Tbzk90Ry6X6y+yU88j8x/LFs+PTVrCRFS/jHkVWtoCz6od9pBAoCBlvJdJ?=
 =?us-ascii?Q?gDnnGuFUOS77St4b5zMaSlSdAFk8cIbHYjumSb97eCQo0ljT/EpqQNpCGbU3?=
 =?us-ascii?Q?IjLSQBhv+qYq+P6LB8vtrj1Y2CKwyBIS2ri9ix44W7iubf+ATaou0LqLfiTC?=
 =?us-ascii?Q?xV6Qx/2YpF2OvpYt6BeabwWBBKLSMDRxOgfYr/rEyPdO9GAutjlFQFOPkHNd?=
 =?us-ascii?Q?aG/ohfHkU9jql+aP0HPJpAJZkHwAQLdLrxWNMjJMXQ/zJPHd4pLKbGhQGYM+?=
 =?us-ascii?Q?5x376FqP2t5SkW5FmX0alLJPOswbOw9BZoZAgSD5q8byuDm02rV8Sct4538I?=
 =?us-ascii?Q?dnOlpQsZrO9l79cK3qoG6WhId3HzuV2f07TsovYSt3w1a1rv4m569C4uDmG8?=
 =?us-ascii?Q?8+hW0qL4wQaCHuPadtD4f33MKdi2Xj3WEe6ph2nmfcRZD9CO3Gra8l2U7raE?=
 =?us-ascii?Q?s4EM1KFAP0aGH8xswADKzXD/GbeI50RWSKWtHq++rtqYDSBfoRhzzp3LeBmq?=
 =?us-ascii?Q?vSbxKXmD5n2PmzZzD+g/ngvUUNKvz2QpUeu7FRWFuCLYt6Zervb7HiZq3azz?=
 =?us-ascii?Q?qQUEVyfUcJXodUiY0fy/UFIWUqgGyirgzbth/uDJjWk4w1CjljfciW8oDaKP?=
 =?us-ascii?Q?9NYkcRsaS2z2L+cK020Rz7ADETAMdwjrHDLctgDvQNcGhPax6xE7z8PCWGNC?=
 =?us-ascii?Q?ap5zOi0S2YHAgPPKPvGiAbRJjNopEyf7d/eL2zhkr19MONy9H3kTl1w5PELy?=
 =?us-ascii?Q?Gunt8nT495CcCF0OyRpaU5FjHZnuGYQYGxX7DwZGmXtgP3OtwLC4/OEv5yrn?=
 =?us-ascii?Q?JI+9/PSWUlxzfaL2/EHhe1cXZbW9fLpCELhOkp4zD9x1CBcsV7D52pCD+x4Q?=
 =?us-ascii?Q?5zgGw6AxTraSv191YK/ciiiGbC7JLcIw7Ck1cIvs7d6pon44cdV2+0MzaYct?=
 =?us-ascii?Q?PlJvqkG1az3tSDqWODc=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: AM0PR04MB5956.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 20834806-c2d6-4e64-2632-08dbf28ea998
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Dec 2023 16:57:54.7026
 (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: hzHziuYGgakWOheNxGL4OSwV2PDn0EgWZ5KORdU34IAAFtv2fL/a5hBMxiZSWPsbtJBftphE16vdcAK0W1QYxw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS5PR04MB9800
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Fri, 01 Dec 2023 08:58:04 -0800 (PST)

> From: Frank Li <frank.li@nxp.com>
> Subject: [PATCH v5 2/4] PCI: layerscape: Add suspend/resume for ls1021a
>=20
> Add suspend/resume support for Layerscape LS1021a.
>=20
> In the suspend path, PME_Turn_Off message is sent to the endpoint to
> transition the link to L2/L3_Ready state. In this SoC, there is no way to=
 check if
> the controller has received the PME_To_Ack from the endpoint or not. So t=
o be
> on the safer side, the driver just waits for PCIE_PME_TO_L2_TIMEOUT_US
> before asserting the SoC specific PMXMTTURNOFF bit to complete the
> PME_Turn_Off handshake. Then the link would enter L2/L3 state depending o=
n
> the VAUX supply.
>=20
> In the resume path, the link is brought back from L2 to L0 by doing a sof=
tware
> reset.
>=20
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
Acked-by:  Roy Zang <Roy.Zang@nxp.com>
Roy