Received-SPF: pass (google.com: domain of linux-kernel+bounces-107767-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Frank Li <Frank.Li@nxp.com>
Subject: [PATCH v5 0/5] PCI: dwc: Add common pme_turn_off message by using
 outbound iATU
Date: Tue, 19 Mar 2024 12:07:10 -0400
Message-Id: <20240319-pme_msg-v5-0-af9ffe57f432@nxp.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
To: Bjorn Helgaas <bhelgaas@google.com>, Jingoo Han <jingoohan1@gmail.com>, 
 Gustavo Pimentel <gustavo.pimentel@synopsys.com>, 
 Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>, 
 Lorenzo Pieralisi <lpieralisi@kernel.org>, 
 =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= <kw@linux.com>, 
 Rob Herring <robh@kernel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>, imx@lists.linux.dev
Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, 
 devicetree@vger.kernel.org, Frank Li <Frank.Li@nxp.com>, 
 Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>, 
 Serge Semin <fancer.lancer@gmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?alZ6bFkvUWMrTk02N3c0b0cvM0Z2NmQ3SGFDVldZZkpWbktaUjNtQUdoNlM1?=
 =?utf-8?B?d1kwRVpwTmVQSE9ZSFVGTTdORTNBMFIwRG1HeWVzbGxLSmQ5MFdzYXpkMW1O?=
 =?utf-8?B?bDJzMnpJeUZ6UWpvYTdsWEd6MTN5dmdhSm4xdWRqZnZVVDdJWThOUlJBdGp1?=
 =?utf-8?B?d3p2S2VzSS9Vd01wd1A4ZmdOZmkxMHdPcCt0c3ByTmVudGdIRDFGQkJrKytQ?=
 =?utf-8?B?cXJFcGNoVnlzVXRqT3kxeVpCYWhLaTZxQXZGRGRqT3UzcWFNNmVONzRkN1RT?=
 =?utf-8?B?UzBvVlRlYVZqaVJCU1lqclFDaUZYWlJ5MW0rMXZ2S290d08yWSs5N1l1WXlM?=
 =?utf-8?B?blg1U3g4YTJQN0dnY2V0Yk40SWZpcGNtVUh1MUxqRzlUZS8xOWN0VTlsYlp4?=
 =?utf-8?B?aThKWnFpNngyUXA4STFJd1pGNUNWbk1rREk5WXkxWSt3RzZpYVl4Umh4WHlR?=
 =?utf-8?B?eE1mWnVwanVjeWJVMXVIRFhXd20ySm1KTlJrdi9TZmRTT3p1cnh4RzEvWnV4?=
 =?utf-8?B?bDdkTC9BYmlCMGFjVW1uQzVEOXM4L1JwNUowTzUwMFFTRXFSb3hlWk1qcnBJ?=
 =?utf-8?B?aVp4V2RRWU1vQzMzejhJNzQwd01ERjdTeS94MElkYndETWpqUDVNNmRxOVZs?=
 =?utf-8?B?RDIxeTB0SDhoaUlwc0svT3YwbklmekJzdGkzWWdPRVVMYXpLQW1jQnpuMFhQ?=
 =?utf-8?B?VzNvRUdTbnA0d1ZsemRlZzhNVTBBQkhUd1hHS2NVQm00Y2NOZjUxV0ZteG9I?=
 =?utf-8?B?a3RNaUhad3FGb2Y3OFNpbVYxUU1MV2gzUnFlOS9JR1VpMDQrMXVmeDNYSi9h?=
 =?utf-8?B?cmpkeHZObFBuL1MrT21zUFFCSXhFM3dUaVUwVkM3aGlIbUd5amJNSk9XUUdC?=
 =?utf-8?B?VHdlbHRVN3VxSHRlR2NRNTdDQ1NiMEdUUGNST05ldXZ4MzNnaGp6L3dVNlBU?=
 =?utf-8?B?cXZCdHFrVEpobkxVWDhpOW5BZ204OXlTMnp2WDIxNkM4eTFuMnhEd0lHSnht?=
 =?utf-8?B?d1ExaWpHTnNtbVN6S01WNFdJc1JNS3dtOWFNMGdKMFZuVkVFblRpOUg1Ukdj?=
 =?utf-8?B?RWZCZnNQRnNJL1hqQWlqdnFJTS9Cd3k5TUYybFhmeHdEU1RrczJRMCtrazFh?=
 =?utf-8?B?NVliTGRlODBVWjFRczVRZW9GUTBsZ0hpdjhWNmg2YVZQQndHN3BTQ0huckxL?=
 =?utf-8?B?ZmlZYS9ZbHY2N1FBOFFLTFh5SHBGWVptWktsZ1diVGMwSWxaSGJkZmpEWEJ0?=
 =?utf-8?B?RmxZOFRXeVo2Tm90dDlvZGJXc1AxR3YwbVQ2UnVQOHU4ZnRzRnFDZGJaSUlE?=
 =?utf-8?B?YVZoZGk1Z0VBdGZIdDJIRUd4cFViVjZHQWt4UTkvc0VOaWZFemxEclNLdjdF?=
 =?utf-8?B?dklkc0xCcVFIc01HVUJmaXpBeVhKd0JSLzlJdytRV0VEUjBxbmZzeVpzRzVL?=
 =?utf-8?B?alRYSGV2amw5ODErSUVlbnAvTnlJUXE1aFZIQXgxbUlMSHFYdFVyeGVIOEIr?=
 =?utf-8?B?TEtlWU1IeXJEbFhFYkthUklSdUttem16UytORk1yMldYMmxpbWp4N3ZBMEZq?=
 =?utf-8?B?bnB6bGo4NWt6Z2s0dXRBcEpXMWkzckg1NjBuelRtMUsrbjZaWFA1S0FzYjFG?=
 =?utf-8?B?aStTUlJnT3lXamc0Ty8wdGJ1WjdVUzZJdW9OUmJEYWtCSXVoa0RFckNTL1dW?=
 =?utf-8?B?TkRqS1VLaWpCYmtiQW9uZ3AyK2Eyd2FvWHdlV0docWRxYWZ0RTA0TFBYTnpE?=
 =?utf-8?B?RFpSeEN5ekp2Y2NGUFNid0xtdm5XUzBMWEJVdUxwSTgybkVtS0RoK29Gc2lX?=
 =?utf-8?B?ZWlGTjlhUGJEdk1TOWxKYlBiYmg3cGxVK3k1TitjNTcwOU9MWDNZd1l2YnlQ?=
 =?utf-8?B?clZTb0o5YVBUeEZ4R2laaE1oOENkdTkyY3AxanlvQzZ0MW85MU13T2x2c1pp?=
 =?utf-8?B?QUV2ZmE1SVc4K043RnpDZmVLMkk4MlVQVHZPeHNteCtRSlFQVWlYUlZjdkdy?=
 =?utf-8?B?ZjF6RTN2bm5YclRvbW9NWXdlREZaZWY4KzNaZGVwNmNaK1FMcTBPa05sRlQy?=
 =?utf-8?B?U21NSzZzUnVRVmV1d2pKenlTd3RmZEZ4Rit6ZEh4RXlLUkc0Skh5ZHh4Z2lG?=
 =?utf-8?Q?cSwUvdSKOFMSnugCuDN9Ujtya?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 44714148-45c1-480f-4b78-08dc482ea7a9
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Mar 2024 16:07:19.9138
 (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: naI46od6/sUEIGaFpu7fr5J740eAq0wmxA8yuflUW2aoLu2+yT4MWiU8vvogV7uxT17Fr7rtt5OIAsA/Yifofg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7772

Involve an new and common mathod to send pme_turn_off() message. Previously
pme_turn_off() implement by platform related special register to trigge    
it.                                                                        
                                                                           
But Yoshihiro give good idea by using iATU to send out message. Previously 
Yoshihiro provide patches to raise INTx message by dummy write to outbound 
iATU.                                                                      
                                                                           
Use similar mathod to send out pme_turn_off message.                       
                                                                           
Previous two patches is picked from Yoshihiro' big patch serialise.        
 PCI: dwc: Change arguments of dw_pcie_prog_outbound_atu()                 
 PCI: Add INTx Mechanism Messages macros                                   
                                                                           
PCI: Add PME_TURN_OFF message macro                                        
dt-bindings: PCI: dwc: Add 'msg" register region, Add "msg" region to use  
to map PCI msg.                                                            
                                                                           
PCI: dwc: Add common pme_turn_off message method                           
Using common pme_turn_off() message if platform have not define their.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v5:
- Default disable allocate TLP message memory windows. If driver need use
this feature, need set use_atu_msg = true before call dw_host_init().

- Link to v4: https://lore.kernel.org/r/20240213-pme_msg-v4-0-e2acd4d7a292@nxp.com

Changes in v4:
- Remove dt-binding patch. Needn't change any dts file and binding doc.
  Reserve a region at end of first IORESOURCE_MEM window by call
  request_resource(). So PCIe stack will not use this reserve region to any
PCIe devices.
  I tested it by reserve at begin of IORESOURCE_MEM window. PCIe stack
will skip it as expection.

  Fixed a issue, forget set iATU index when sent PME_turn_off.

- Link to v3: https://lore.kernel.org/r/20240202-pme_msg-v3-0-ff2af57a02ad@nxp.com

Changes in v3:
- fix 'MSG"
- Add pcie spec ref in head file
- using function name dw_pci_pme_turn_off()
- Using PCIE_ prefix macro
- Link to v2: https://lore.kernel.org/r/20240201-pme_msg-v2-0-6767052fe6a4@nxp.com

Changes in v2:
  - Add my sign off at PCI: dwc: Add outbound MSG TLPs support
  - Add Bjorn review tag at  Add INTx Mechanism Messages macros
  - using PME_Turn_Off match PCIe spec
  - ref to pcie spec v6.1
  - using section number.

- Link to v1: https://lore.kernel.org/r/20240130-pme_msg-v1-0-d52b0add5c7c@nxp.com

---
Frank Li (2):
      PCI: Add PCIE_MSG_CODE_PME_TURN_OFF message macro
      PCI: dwc: Add common send PME_Turn_Off message method

Yoshihiro Shimoda (3):
      PCI: Add INTx Mechanism Messages macros
      PCI: dwc: Consolidate args of dw_pcie_prog_outbound_atu() into a structure
      PCI: dwc: Add outbound MSG TLPs support

 drivers/pci/controller/dwc/pcie-designware-ep.c   |  21 ++--
 drivers/pci/controller/dwc/pcie-designware-host.c | 146 +++++++++++++++++++---
 drivers/pci/controller/dwc/pcie-designware.c      |  54 ++++----
 drivers/pci/controller/dwc/pcie-designware.h      |  22 +++-
 drivers/pci/pci.h                                 |  20 +++
 5 files changed, 199 insertions(+), 64 deletions(-)
---
base-commit: e08fc59eee9991afa467d406d684d46d543299a9
change-id: 20240130-pme_msg-dd2d81ee9886

Best regards,
---
Frank Li <Frank.Li@nxp.com>