Received-SPF: pass (google.com: domain of linux-kernel+bounces-49992-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 v3 0/6] PCI: dwc: Add common pme_turn_off message by using
 outbound iATU
Date: Fri, 02 Feb 2024 10:11:22 -0500
Message-Id: <20240202-pme_msg-v3-0-ff2af57a02ad@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?bG81U2Naa2F6UC9Xd1hvZ01GMHpWUDEvcG5SZnl0MDlDVG85Tld1OTNUUHBx?=
 =?utf-8?B?WFpHVjMrYkdqRlhGb3pQRzUzTzcvRjFhbzhKZVZ5dy9NaDVkaE0xbkFaN3lR?=
 =?utf-8?B?TlVNNzdQRkRNWCtqaGZHaXlhNXVTcGU2dXAyS2tacWtWRTRDcHBJY3Q3Z0sw?=
 =?utf-8?B?Rk9JaGtTdUtXd1dtL0IwR1FPYTVyanNCQm5YNkZVbGlWOTNUWU1NV3VmcmF2?=
 =?utf-8?B?cndOUmx6MGZUMjZOR1c1RWx3aFA2SnVnOE1kZ0hHN2xBeVplclcyelNEakNG?=
 =?utf-8?B?TXFLYVhFNFpSZmNyRTM4MisxRm5tblZpbkF2MmI1MzF6cys0UDlMOWNtN1NF?=
 =?utf-8?B?c24ydDBBV25WR3dmdlE5ZHpvdWRCT1dqZXlKM0V6ZFNwRHVDMWxnT1ZtVTNZ?=
 =?utf-8?B?ZkJWRy9pb0loNDY0cnh4b0hqaWVNOWhiSmwwWC9lZ3JPTC8wOWdBajRMNEVs?=
 =?utf-8?B?VHYvMFFnL0lCUzZpUGgrQkM1QlJQRmZKaEhYWUVBYW1vZnNZNjJhbFhybDJk?=
 =?utf-8?B?bzk5cFVpRVdFYTc5UHNrZHRNaWtJMjlQUmY0Rkt4UHRKV1IyQXRmUmZ0RlNM?=
 =?utf-8?B?bnVSZDhFalNkZmFkcldwYTZpR3IxRmFoYmptNUI3Skd6cHA2NjVmZFFHWmZF?=
 =?utf-8?B?WnEzUUVEYUpCcWxRZ1d5NXVjcWsrTXo5eFR5dUdlOFVvNG9kMUpuYzJTdDlx?=
 =?utf-8?B?S085VVFjT0lZVVdtSjVXcFNnWFRKRlpwZDdqSGFJdDkrWCtvYnl2TUpmVUI0?=
 =?utf-8?B?djlab2YxU2ZlWTh6MkkrL2dkZ3paSnIybmN2UXVVZXgxemw2Z1FMRjFnNVRo?=
 =?utf-8?B?U2tENmxsNGIrK000bWd0b21mSzdPNDlXa3Vha0JxSDhEbDYxNldBeWtsSGRD?=
 =?utf-8?B?ajBnZDNyMEZDenRCZ3drb0R6WlU2SEd0L3NaWU9GVU9mWmgvb2xpK1dyYWha?=
 =?utf-8?B?Z3Y3SDRGaU5UN3BwVXZmbUVkTGdNeDU4NkVZbmhIWk5vVjRkb2JuNFY1TUdM?=
 =?utf-8?B?S2lKUnFKUG4yRnY3R3pZNHkvZUJPd1h2aG04Wjc3aitOM0VjUXhUWGJOZ2RS?=
 =?utf-8?B?UVZlQ0lDUU5NSCthSEE2MU90aU01MkUzYWsvUnVhY2ROcHJRVlVqZ3NuRW15?=
 =?utf-8?B?Yis2c25BVm5zTUZpYjZQMDFWZlkvN0RxUTh3cHFpR3ArSzF3cTM1dTNLUlRP?=
 =?utf-8?B?TG9UdHEwVWN5ZWdsblRVTUpZWWhJWXFmT2lMWnk5a0VwT0VQeGt2K2tvNk1l?=
 =?utf-8?B?bmZ1a2YyOWJpTzF6TVhYN29ESHorYVRhd0Z6NHpDNmNRYXNvVG0ycHdqL2Jw?=
 =?utf-8?B?aENyOTFmSzNaTHJ1Q01seldaL3JsTlZFNXU0SGhQdnJHQ2ZML2hNdXFXVUJ0?=
 =?utf-8?B?WmJVZitrMVMzN3I0MTQ1cGQwbFVDYUFQbnpGL0VpcjdCYkZPZ3d3MnI1OHRn?=
 =?utf-8?B?RjBxWXdmU1pmWW9wTDA1RHV5OG5uaUVVaFJ0ZG9sZTZwM2RYTEZ6K0dCSUY2?=
 =?utf-8?B?T1U1UHJuOWFhUW02WmZHcmdjaitrT1pUaEJLbUtZSHBHNjdhNHo3aHpOYXVT?=
 =?utf-8?B?cGxkVWEvZmVVQ0FaKzJTb2pnbzhSMFZyTXFKR0JUbDZQQzA0MWF1SFJPVFNy?=
 =?utf-8?B?MG1zeTQyWWNQYVVldWp6WjQ1Um9HOGNmS1I0c3FqQ2dtUENDelBRbzBER0lC?=
 =?utf-8?B?bUVwYTZSVmpSczVETmNhQkprNTc2d3Z6Z25JLzBPMjg4U05uK2dVcWRLSWI4?=
 =?utf-8?B?bEJiQjJybEY2dzJldEJkckFJR0l6YjgxdGZYL3A0MUlrMjB0cHRVZ1piSGpn?=
 =?utf-8?B?RkYwbmF0cFZkTGJoT3RsMGZKcm9acWRneTg2S1NQZk1BVlluSzF3M01ZRUlx?=
 =?utf-8?B?UVBaSHJqMERkNUQ3MFRFaDdTUjRwUGdXZXU2ZFc1N0pPVWRiMWY1VDhNYVgz?=
 =?utf-8?B?QVFqSFN6aHNaclFsdlVkYWZRSDdKWUtZMXZXUEVOcFU2L3lOSkhwS0ZYUU5X?=
 =?utf-8?B?Snd2Wi9OK2IzRGJuQWsrY1BIbmpHVS9DMEN3NFJRZktwQWJhd1lsNldiTnh0?=
 =?utf-8?B?MWI0Y3hCbFVjRzdLSGFEdU9VL0N3Q1dscU0yU290ckw4N2dPNWlWaGlqNXZG?=
 =?utf-8?Q?5PkDgEg4IZPvCxkmVIohO+gn2?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b551f54e-bc9c-4bb4-db65-08dc24014239
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Feb 2024 15:11:40.5722
 (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: HC8SPo/bFlUFui73CpGFiqkpVBYdB5HCge/ONf2/f3S6KEWgRCDtMbiM0kTVIomNlYWHYkRzd1ImSZ0n8UYREQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8425

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 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 (3):
      PCI: Add PCIE_MSG_CODE_PME_TURN_OFF message macro
      dt-bindings: PCI: dwc: Add 'msg' register region
      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

 .../devicetree/bindings/pci/snps,dw-pcie.yaml      |   4 +
 drivers/pci/controller/dwc/pcie-designware-ep.c    |  21 +++--
 drivers/pci/controller/dwc/pcie-designware-host.c  | 104 +++++++++++++++++----
 drivers/pci/controller/dwc/pcie-designware.c       |  62 ++++++------
 drivers/pci/controller/dwc/pcie-designware.h       |  22 ++++-
 drivers/pci/pci.h                                  |  20 ++++
 6 files changed, 169 insertions(+), 64 deletions(-)
---
base-commit: e08fc59eee9991afa467d406d684d46d543299a9
change-id: 20240130-pme_msg-dd2d81ee9886

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