Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 12 Jun 2023 12:16:24 -0400
From:   Frank Li <Frank.li@nxp.com>
To:     "helgaas@kernel.org" <helgaas@kernel.org>
Cc:     "imx@lists.linux.dev" <imx@lists.linux.dev>,
        "bhelgaas@google.com" <bhelgaas@google.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "gustavo.pimentel@synopsys.com" <gustavo.pimentel@synopsys.com>,
        "kw@linux.com" <kw@linux.com>, Leo Li <leoyang.li@nxp.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "lorenzo.pieralisi@arm.com" <lorenzo.pieralisi@arm.com>,
        "M.H. Lian" <minghuan.lian@nxp.com>,
        Mingkai Hu <mingkai.hu@nxp.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        Roy Zang <roy.zang@nxp.com>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "Z.Q. Hou" <zhiqiang.hou@nxp.com>
Subject: Re: [PATCH v3 1/2] PCI: dwc: Implement general suspend/resume
 functionality for L2/L3 transitions
Message-ID: <ZIdE2KjOuyHmLqz0@lizhi-Precision-Tower-5810>
References: <20230419164118.596300-1-Frank.Li@nxp.com>
 <AM6PR04MB48383AB3FD2062334ECB66EB88759@AM6PR04MB4838.eurprd04.prod.outlook.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AM6PR04MB48383AB3FD2062334ECB66EB88759@AM6PR04MB4838.eurprd04.prod.outlook.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?lIK0hnnFFowGHyWW9VkXRWeQdRsCNfciJhpuXURCjnB23AgX2AvcQCO7tMUb?=
 =?us-ascii?Q?9EBE3yw3YuIz/Bz595S4YLMbz3Xd6PTCbT6OOHnkWy0u2nupyl/JKnlxW9Ui?=
 =?us-ascii?Q?yBh/DtK/mekG3eYhwBJTA6RS7o/F8ftNUIcGhi06zbVcsWAnwNkCjFr2Uxdw?=
 =?us-ascii?Q?iGoF5UMg1wTkaJy0WlcyyevOLx2ODWN7fs8SewVOLiMQWtlrmwHbir53r1QU?=
 =?us-ascii?Q?gDkK8YcbA8vUXHvwyWPW7+5C78MeGL2FvaXSGLi8k3k5xo74dcyZl8sLL3sF?=
 =?us-ascii?Q?yPq4KbFDrUfuUFZ9bIOg3Rm5elAPcFUcAhlVuk7ytMbsLSxQI6tkk9FEjPXh?=
 =?us-ascii?Q?fr5R4QW0fourjBdqnIp4WAt2XW15/mxxYna2NjzlQv0zc1ATH4yqht0my2Px?=
 =?us-ascii?Q?Q8w0/AdXJEi6Xw4WGiCoR8S/EPvV7OakSvWdsf9Ket58SXDlAIf7mlnUIcvO?=
 =?us-ascii?Q?lIMs5PmfySEtKotbsAqOTbU9zglNLKsOzqrL0cASYW/QrzuZY7K8euZLb+jf?=
 =?us-ascii?Q?5XKwrOj9duKDZmDLWHGeNwbeBLWgX9Q+k0VyimpeROKU9Zqs7Z0XGX+IYJyi?=
 =?us-ascii?Q?8qpodGVrZCYMEGDKLSkBQZsALmZsQAaAAcGTeZkwX0eKtVHzmmjuWlBFPQrZ?=
 =?us-ascii?Q?Ysvb2KmAtnIsuJGsC0xiXYwingRKNQMMMH/CTl2V9vwg2YbWJaa/3u5ipnF2?=
 =?us-ascii?Q?1J9CAsBq0FXCqm2osp9V+YfDmq+pvkIzWdMTPbkGMO4NEzcxYu4oDZK3v3Eq?=
 =?us-ascii?Q?nfkk9HsmelY7e+hvewTDqPs7QQKXRj2waZ7virwBpv5FRFE/w68ibrEVpdEc?=
 =?us-ascii?Q?2mxVPMSGc2f0/1QdERoUhauWSlhnbseS0CcBsad7tfzVK7+pl9UvbwU7SFHR?=
 =?us-ascii?Q?W0CIse4Y96L0ljL7q+0KeCU6SPXFbOpxFC+f5VS/vTk3Y1bkqPXU6OFEL5GD?=
 =?us-ascii?Q?MUWBohF7W0jEgS3pTOVg8cPX9alSCPwbVfKKQ7wmYuX0UcBaTRieWLSP1/MJ?=
 =?us-ascii?Q?AK0WZNqv6MEiLJ2lviDDOlYjrFYMFHfUW2krtTen6wqWQBO3QJgLNHN6yYz2?=
 =?us-ascii?Q?EG4TFcaXKPoFenXFOrd/6OA85Bdik55DMiO5iiHDmDLk0ChSNBU0/HBTwt8q?=
 =?us-ascii?Q?g2Ba715a0p8E5kgdiNssw8Jk9bBpzfOikaQVk2c5nsw3M41h3WiuyU77qVYn?=
 =?us-ascii?Q?4JKZ5JF3iYJmVIvUWsm2ZC18h9GI2z37kP9QtFhzFx+DFf1jZV/2gacOidka?=
 =?us-ascii?Q?/H7alXvyK65KjmceF6Q1eTG4m+ksivBMhJlyaONuAOtZidNpCJr77QjEP5gO?=
 =?us-ascii?Q?Sjm6d5kr+VaiwHMRWct+B3bbR2yN2HeFayEbeM2lHHn5mLMTKquxe17GAfXw?=
 =?us-ascii?Q?RN8AZWEuLIDtWduWR8F4REsN+e4TpgMoHDdJRZf22xh3w3NstjKKkRJpFcJK?=
 =?us-ascii?Q?BOtduvTgL7/kqkpCwjaBjJBFC3Gp23awsOhOfZVH7GEU0nrdMKArjmIhSBOY?=
 =?us-ascii?Q?RVkawDrMYanUr5SQGG6JA31cpNbaJO1WmomJHr2xw8ZWT/70i5nevcfHDWCS?=
 =?us-ascii?Q?3I7GCz2rEgMB6/aHPkk=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6354262f-2654-4516-941c-08db6b606797
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2023 16:16:40.5572
 (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: pFfd9dR6du46hmBISu3h3qi0L1jE22nlzNEIUQxryVuUbPACDIbI0342R8Zuqits9z2VQ9WJCE+xdnps9H6/Pw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8852
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_NONE,
        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

On Fri, May 12, 2023 at 02:47:46PM +0000, Frank Li wrote:
> > 
> > Introduced helper function dw_pcie_get_ltssm to retrieve
> > SMLH_LTSS_STATE.
> > Added API pme_turn_off and exit_from_l2 for managing L2/L3 state
> > transitions.
> > 
> > Typical L2 entry workflow:
> > 
> > 1. Transmit PME turn off signal to PCI devices.
> > 2. Await link entering L2_IDLE state.
> > 3. Transition Root complex to D3 state.
> > 
> > Typical L2 exit workflow:
> > 
> > 1. Transition Root complex to D0 state.
> > 2. Issue exit from L2 command.
> > 3. Reinitialize PCI host.
> > 4. Wait for link to become active.
> > 
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> 
> Ping

Ping

> 
> > Change from v2 to v3:
> > - Basic rewrite whole patch according rob herry suggestion.
> >   put common function into dwc, so more soc can share the same logic.
> > 
>