From patchwork Tue May 28 19:03:08 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bartosz Golaszewski X-Patchwork-Id: 800917 Received: from mail-wr1-f43.google.com (mail-wr1-f43.google.com [209.85.221.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7A7DA172BC8 for ; Tue, 28 May 2024 19:03:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716923032; cv=none; b=RRQBTMpJgMp9sge17BCH1bSIQNQC3Hnv/n855saUbvs0/5HEgHoGMzuakeWNSthS/SejThZWS01FZ2Jf3CYeVOmvzKRt2aYRrKXd/6+/CjeKg47GvjpTymYT07KEscO71kMP/kkzbeEj10SaBdQ7iNsru5IBuRw3EXcK+pn+L2s= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716923032; c=relaxed/simple; bh=AkwqYaVuKCDhZN+CcHqahreb1dxHDLVHWIlkeAeBCw4=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=BS0jalb9IB2dKtKg8G6HfNTQfLabkycPqnkXCNu47zovDhzjQvHDkTSjkNSr+6D0wBqOTle5MEetngxEDrONdyKCiDCa6oNErQdO5nWQW+Uj5kmE1/AReSrZcN37n3q8j7U0fDYOWNUHqCDZt9mFfr3LotqN4yNbVzWJa2KbEVw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=AAy/qrFO; arc=none smtp.client-ip=209.85.221.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="AAy/qrFO" Received: by mail-wr1-f43.google.com with SMTP id ffacd0b85a97d-35b6467754cso605741f8f.3 for ; Tue, 28 May 2024 12:03:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1716923028; x=1717527828; darn=vger.kernel.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=k7xJgANA6A1kgFK/1xiN79ujRUyXZwyictvo50DvnRg=; b=AAy/qrFOIbtHLSckmkxqaX7OHU9DyjfAGVHEvJdwme//pa6B6oacXH3f1fLSxajEVP eKbTqt2nYhgpK5HXuhSmm6omc5ljIQtVdHCD/OymwJ+NcSklKRSNU9Y65fz8LZtCNFge /44mFrBx0trwNzvXPQYh/7COoISDVN8T87RWM+y7xjx5MWbvqP6URVMQaHuqceJ4vPdM uW86RaEO6ELR5su/h6T72iMVBbgf86c4CIGKcGL+U6xqfiC/Y961CS5BJ3Za0skHr03D 9uOnCAyemX1/C98oe7PXzy3oaIESV/+bbhzWvnfN1jdQGpuA+NfWak5FmWYXgPK2QkPu I9sw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1716923028; x=1717527828; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=k7xJgANA6A1kgFK/1xiN79ujRUyXZwyictvo50DvnRg=; b=XpoFapQz2XYyY2ePpXINz/NxdIl3+W7uH/egnQIOhHlU1sw4xTWhlB7e6vdOSpxeam gpuu86iofULQUqdD2qfJo/2XNdAz+5vpgPaJplSzzEAvwD8d7hOeY17W6ncAJHOaFqrC Z4dCzKPwirxPQav/SlpISSVoCDkhKPypUnNxa/0MbF6iH/CfED95JYSsF+DeVDrow/YW qxzaLmEoZqyAqgciFBz+cFAibbwKG8jUe1yFOXycio899wQ1TmuiuoAAU4JvQtt61gIR DfE48cNmb8usWI2hEMxaScWWCOYuAH26t0BTWQ7btRWetvxhZHR9WJ5P05/CGRyvGYwS N8hA== X-Forwarded-Encrypted: i=1; AJvYcCXC4p5yh7mCqxLsh9UNKYcZjfddGW9uMp+1M5oCXHz9frx7ik6wEZkagunGJdybD5Lf/KViljYLYTH2xKzfoZegO+Tm6PBIEKJedGKODew= X-Gm-Message-State: AOJu0YzOhapGiSNyHuAVprE1bN1bwjbVVl0iyhzbDS4aRv+V0CjfeTg5 0UjjY7rAWJQVmTdimdtkf+LPzG6V+9NEc+hz4JBEL5j9ZkJuSkMHeWrVeBx8Vgg= X-Google-Smtp-Source: AGHT+IEj04ZZ6ktd4DPL3ZbhqEUWdm6BK6mOrxC80mRmuyVXOSaAGFEaeoA5VCOU9CO7LN8jJKZWbQ== X-Received: by 2002:a05:6000:402c:b0:357:e0ac:fabf with SMTP id ffacd0b85a97d-357e0acfd45mr7520764f8f.43.1716923027495; Tue, 28 May 2024 12:03:47 -0700 (PDT) Received: from [127.0.1.1] ([2a01:cb1d:75a:e000:93eb:927a:e851:8a2f]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-42100ee954bsm183895415e9.4.2024.05.28.12.03.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 May 2024 12:03:47 -0700 (PDT) From: Bartosz Golaszewski Subject: [PATCH v8 00/17] power: sequencing: implement the subsystem and add first users Date: Tue, 28 May 2024 21:03:08 +0200 Message-Id: <20240528-pwrseq-v8-0-d354d52b763c@linaro.org> Precedence: bulk X-Mailing-List: linux-wireless@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAGwqVmYC/6tWKk4tykwtVrJSqFYqSi3LLM7MzwNyLHQUlJIzE vPSU3UzU4B8JSMDIxMDUyNz3YLyouLUQl1zs7RkAyNTIxOLRCMloOKCotS0zAqwQdGxtbUAWmE JMVgAAAA= To: Liam Girdwood , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Marcel Holtmann , Luiz Augusto von Dentz , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Balakrishna Godavarthi , Rocky Liao , Kalle Valo , Jeff Johnson , Bjorn Andersson , Konrad Dybcio , Bartosz Golaszewski , Bjorn Helgaas , Srini Kandagatla , Elliot Berman , Caleb Connolly , Neil Armstrong , Dmitry Baryshkov , Alex Elder Cc: linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-bluetooth@vger.kernel.org, netdev@vger.kernel.org, linux-wireless@vger.kernel.org, ath11k@lists.infradead.org, Jeff Johnson , ath12k@lists.infradead.org, linux-pm@vger.kernel.org, linux-pci@vger.kernel.org, Bartosz Golaszewski , kernel@quicinc.com, Krzysztof Kozlowski , Amit Pundir X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=15276; i=bartosz.golaszewski@linaro.org; h=from:subject:message-id; bh=AkwqYaVuKCDhZN+CcHqahreb1dxHDLVHWIlkeAeBCw4=; b=owEBbQKS/ZANAwAKARGnLqAUcddyAcsmYgBmViqHCxPKxNMfGLqv9Cs7n3CrsKpW253UyQdfc cHaOZrayX2JAjMEAAEKAB0WIQQWnetsC8PEYBPSx58Rpy6gFHHXcgUCZlYqhwAKCRARpy6gFHHX ctCLD/9sMGL+LGoXNu97fLpVuPT81kam7bLPgiZeYXej4Y85GYvNIk7soVksVNZIk5YqGJ8YVbw 0iD7PyL79nJr1+UFYbz7RlITpRmhZKgGXM/cODNthqBs1ly5zUqzkD9y0l7MnEajTPpbbzfzeph n1O8/uvkn3ALC7fCWn3B998WIT+DDRT64GI34zMYuAgBcGm0Uyy94aL5bhtuHWH4TSnHScc9aRB 4IZxHS+qjx+lqCM/WhsAgGX1+VaLlPDXGeRonYrmh/Qz1b8t6TlHkkG7n99ffgcq3v2RX3caZWM kXoydL7oe0RQnMWnE5L/jIKT2BZ+JRYykC+U/LpbML0vXVjhXCQH5iZLCQoJlFVLdqUZ0fjWhDq MxKslSSkOlgY1/qWg34geg7nAubhGOvKVQTjqQI+EvZIkCRvE0wWNHNekEluDECMv4De9LEd8wa Y48fR2o+X5ngH+nfLrs5nkTxJrPttPhkwvTht6aT5w9wQ/K3ONoYBUjijvXOgmdY/3hhOdiFnlX NFmTRPFk0wCCTgDqBV3ROgVO1/RrwS9XTrIWKYO6wJ6aBvgSralIxMSvC1/JVvQtaArqfIW29YO Sr+hKfkPGC7TewXWcxS6APvQloErnMV+3MaX1xcl5Q/BMZeiJyLyzWoNX4ShWW+3cpOqHPJ7H3Y lt1CfboLv4qsGbA== X-Developer-Key: i=bartosz.golaszewski@linaro.org; a=openpgp; fpr=169DEB6C0BC3C46013D2C79F11A72EA01471D772 Note: I am resending this series in its entirety once more for discussions and reviews. If there won't be any major objections, I'll then start sending individual bits and pieces to appropriate trees. Merging strategy: The DT binding and DTS changes are a no-brainer, they can go through the wireless, regulator and arm-msm trees separately. The bluetooth and PCI changes have a build-time dependency on the power sequencing code. The bluetooth changes also have a run-time dependency on the PCI pwrctl part. In order to get it into next I plan to pick up the power sequencing code into my own tree and maintain it. I can then provide an immutable tag for the BT and PCI trees to pull. I wouldn't stress about the BT runtime dependency as it will be fixed once all changes are in next. The actual cover letter follows: --- Problem statement #1: Dynamic bus chicken-and-egg problem. Certain on-board PCI devices need to be powered up before they are can be detected but their PCI drivers won't get bound until the device is powered-up so enabling the relevant resources in the PCI device driver itself is impossible. Problem statement #2: Sharing inter-dependent resources between devices. Certain devices that use separate drivers (often on different busses) share resources (regulators, clocks, etc.). Typically these resources are reference-counted but in some cases there are additional interactions between them to consider, for example specific power-up sequence timings. === The reason for tackling both of these problems in a single series is the fact the the platform I'm working on - Qualcomm RB5 - deals with both and both need to be addressed in order to enable WLAN and Bluetooth support upstream. The on-board WLAN/BT package - QCA6391 - has a Power Management Unit that takes inputs from the host and exposes LDO outputs consumed by the BT and WLAN modules which can be powered-up and down independently. However a delay of 100ms must be respected between enabling the BT- and WLAN-enable GPIOs. A similar design with a discreet PMU is also employed in other models of the WCN family of chips although we can often do without the delays. With this series we add support for the WCN7850 as well. === We introduce a new subsystem here - the power sequencing framework. The qcom-wcn driver that we add is its first user. It implements the power-up sequences for QCA6390 and WCN7850 chips. However - we only use it to power-up the bluetooth module in the former. We use it to driver the WLAN modules in both. The reason for this is that for WCN7850 we have comprehensive bindings already upstream together with existing DT users. Porting them to using the pwrseq subsystem can be done separately and in an incremental manner once the subsystem itself is upstream. We will also have to ensure backward DT compatibility. To avoid overcomplicating this series, let's leave it out for now. === This series is logically split into several sections. I'll go patch-by-patch and explain each step. Patches 1/16-5/16: These contain all relevant DT bindings changes. We add new documents for the QCA6390 & WCN7850 PMUs and ATH12K devices as well as extend the bindings for the Qualcomm Bluetooth and ATH11K modules with regulators used by them in QCA6390. Patches 6/16-8/16: These contain changes to device-tree sources for the three platforms we work with in this series. We model the PMUs of the WLAN/BT chips as top-level platform devices on the device tree. In order to limit the scope of this series and not introduce an excessive amount of confusion with deprecating DT bindings, we leave the Bluetooth nodes on sm8650 and sm8550 as is (meaning: they continue to consumer the GPIOs and power inputs from the host). As the WCN7850 module doesn't require any specific timings, we can incrementally change that later. In both cases we add WLAN nodes that consume the power outputs of the PMU. For QCA6390 we also make the Bluetooth node of the RB5 consume the outputs of the PMU - we can do it as the bindings for this chip did not define any supply handles prior to this series meaning we are able to get this correct right away. Patches 9/16-12/16: These contain the bulk of the PCI changes for this series. We introduce a simple framework for powering up PCI devices before detecting them on the bus. The general approach is as follows: PCI devices that need special treatment before they can be powered up, scanned and bound to their PCI drivers must be described on the device-tree as child nodes of the PCI port node. These devices will be instantiated on the platform bus. They will in fact be generic platform devices with the compatible of the form used for PCI devices already upstream ("pci,). We add a new directory under drivers/pci/pwrctl/ that contains PCI pwrctl drivers. These drivers are platform drivers that will now be matched against the devices instantiated from port children just like any other platform pairs. Both the power control platform device *AND* the associated PCI device reuse the same OF node and have access to the same properties. The goal of the platform driver is to request and bring up any required resources and let the pwrctl framework know that it's now OK to rescan the bus and detect the devices. When the device is bound, we are notified about it by the PCI bus notifier event and can establish a device link between the power control device and the PCI device so that any future extension for power-management will already be able to work with the correct hierachy. The reusing of the OF node is the reason for the small changes to the PCI OF core: as the bootloader can possibly leave the relevant regulators on before booting linux, the PCI device can be detected before its platform abstraction is probed. In this case, we find that device first and mark its OF node as reused. The pwrctl framework handles the opposite case (when the PCI device is detected only after the platform driver successfully enabled it). Patch 13/16 - 14/16: These add a relatively simple power sequencing subsystem and the first driver using it: the pwrseq module for the PMUs on the WCN family of chips. I'm proposing to add a subsystem that allows different devices to use a shared power sequence split into consumer-specific as well as common "units". A power sequence provider driver registers a set of units with pwrseq core. Each unit can be enabled and disabled and contains an optional list of other units which must be enabled before it itself can be. A unit represents a discreet chunk of the power sequence. It also registers a list of targets: a target is an abstraction wrapping a unit which allows consumers to tell pwrseq which unit they want to reach. Real-life example is the driver we're adding here: there's a set of common regulators, two PCIe-specific ones and two enable GPIOs: one for Bluetooth and one for WLAN. The Bluetooth driver requests a descriptor to the power sequencer and names the target it wants to reach: pwrseq = devm_pwrseq_get(dev, "bluetooth"); The pwrseq core then knows that when the driver calls: pwrseq_power_on(pwrseq); It must enable the "bluetooth-enable" unit but it depends on the "regulators-common" unit so this one is enabled first. The provider driver is also in charge of assuring an appropriate delay between enabling the BT and WLAN enable GPIOs. The WLAN-specific resources are handled by the "wlan-enable" unit and so are not enabled until the WLAN driver requests the "wlan" target to be powered on. Another thing worth discussing is the way we associate the consumer with the relevant power sequencer. DT maintainers have expressed a discontent with the existing mmc pwrseq bindings and have NAKed an earlier initiative to introduce global pwrseq bindings to the kernel[1]. In this approach, we model the existing regulators and GPIOs in DT but the pwrseq subsystem requires each provider to provide a .match() callback. Whenever a consumer requests a power sequencer handle, we iterate over the list of pwrseq drivers and call .match() for each. It's up to the driver to verify in a platform-specific way whether it deals with its consumer and let the core pwrseq code know. The advantage of this over reusing the regulator or reset subsystem is that it's more generalized and can handle resources of all kinds as well as deal with any kind of power-on sequences: for instance, Qualcomm has a PCI switch they want a driver for but this switch requires enabling some resources first (PCI pwrctl) and then configuring the device over I2C (which can be handled by the pwrseq provider). Patch 15: This patch makes the Qualcomm Bluetooth driver get and use the power sequencer for QCA6390. Patch 16: While tiny, this patch is possibly the highlight of the entire series. It uses the two abstraction layers we introduced before to create an elegant power sequencing PCI power control driver and supports the ath11k module on QCA6390 and ath12k on WCN7850. With this series we can now enable BT and WLAN on several new Qualcomm boards upstream. Tested on RB5, sm8650-qrd, sm8650-hdk and sm8550-qrd. Changelog: Since v7: - added DTS changes for sm8650-hdk - added circular dependency detection for pwrseq units - fixed a KASAN reported use-after-free error in remove path - improve Kconfig descriptions - fix typos in bindings and Kconfig - fixed issues reported by smatch - fix the unbind path in PCI pwrctl - lots of minor improvements to the pwrseq core Since v6: - kernel doc fixes - drop myself from the DT bindings maintainers list for ath12k - wait until the PCI bridge device is fully added before creating the PCI pwrctl platform devices for its sub-nodes, otherwise we may see sysfs and procfs attribute failures (due to duplication, we're basically trying to probe the same device twice at the same time) - I kept the regulators for QCA6390's ath11k as required as they only apply to this specific Qualcomm package Since v5: - unify the approach to modelling the WCN WLAN/BT chips by always exposing the PMU node on the device tree and making the WLAN and BT nodes become consumers of its power outputs; this includes a major rework of the DT sources, bindings and driver code; there's no more a separate PCI pwrctl driver for WCN7850, instead its power-up sequence was moved into the pwrseq driver common for all WCN chips - don't set load_uA from new regulator consumers - fix reported kerneldoc issues - drop voltage ranges for PMU outputs from DT - many minor tweaks and reworks v1: Original RFC: https://lore.kernel.org/lkml/20240104130123.37115-1-brgl@bgdev.pl/T/ v2: First real patch series (should have been PATCH v2) adding what I referred to back then as PCI power sequencing: https://lore.kernel.org/linux-arm-kernel/2024021413-grumbling-unlivable-c145@gregkh/T/ v3: RFC for the DT representation of the PMU supplying the WLAN and BT modules inside the QCA6391 package (was largely separate from the series but probably should have been called PATCH or RFC v3): https://lore.kernel.org/all/CAMRc=Mc+GNoi57eTQg71DXkQKjdaoAmCpB=h2ndEpGnmdhVV-Q@mail.gmail.com/T/ v4: Second attempt at the full series with changed scope (introduction of the pwrseq subsystem, should have been RFC v4) https://lore.kernel.org/lkml/20240201155532.49707-1-brgl@bgdev.pl/T/ v5: Two different ways of handling QCA6390 and WCN7850: https://lore.kernel.org/lkml/20240216203215.40870-1-brgl@bgdev.pl/ Signed-off-by: Bartosz Golaszewski --- Bartosz Golaszewski (16): regulator: dt-bindings: describe the PMU module of the QCA6390 package regulator: dt-bindings: describe the PMU module of the WCN7850 package dt-bindings: net: bluetooth: qualcomm: describe regulators for QCA6390 dt-bindings: net: wireless: qcom,ath11k: describe the ath11k on QCA6390 dt-bindings: net: wireless: describe the ath12k PCI module arm64: dts: qcom: sm8550-qrd: add the Wifi node arm64: dts: qcom: sm8650-qrd: add the Wifi node arm64: dts: qcom: qrb5165-rb5: add the Wifi node power: sequencing: implement the pwrseq core power: pwrseq: add a driver for the PMU module on the QCom WCN chipsets PCI: hold the rescan mutex when scanning for the first time PCI/pwrctl: reuse the OF node for power controlled devices PCI/pwrctl: create platform devices for child OF nodes of the port node PCI/pwrctl: add PCI power control core code PCI/pwrctl: add a PCI power control driver for power sequenced devices Bluetooth: qca: use the power sequencer for QCA6390 Neil Armstrong (1): arm64: dts: qcom: sm8650-hdk: add the Wifi node .../bindings/net/bluetooth/qualcomm-bluetooth.yaml | 17 + .../bindings/net/wireless/qcom,ath11k-pci.yaml | 46 + .../bindings/net/wireless/qcom,ath12k.yaml | 99 ++ .../bindings/regulator/qcom,qca6390-pmu.yaml | 185 ++++ MAINTAINERS | 8 + arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 103 +- arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 +- arch/arm64/boot/dts/qcom/sm8550-qrd.dts | 97 ++ arch/arm64/boot/dts/qcom/sm8550.dtsi | 2 +- arch/arm64/boot/dts/qcom/sm8650-hdk.dts | 89 ++ arch/arm64/boot/dts/qcom/sm8650-qrd.dts | 89 ++ arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 +- drivers/bluetooth/hci_qca.c | 74 +- drivers/pci/Kconfig | 1 + drivers/pci/Makefile | 1 + drivers/pci/bus.c | 9 + drivers/pci/of.c | 14 +- drivers/pci/probe.c | 2 + drivers/pci/pwrctl/Kconfig | 17 + drivers/pci/pwrctl/Makefile | 6 + drivers/pci/pwrctl/core.c | 137 +++ drivers/pci/pwrctl/pci-pwrctl-pwrseq.c | 89 ++ drivers/pci/remove.c | 3 +- drivers/power/Kconfig | 1 + drivers/power/Makefile | 1 + drivers/power/sequencing/Kconfig | 28 + drivers/power/sequencing/Makefile | 6 + drivers/power/sequencing/core.c | 1105 ++++++++++++++++++++ drivers/power/sequencing/pwrseq-qcom-wcn.c | 336 ++++++ include/linux/pci-pwrctl.h | 51 + include/linux/pwrseq/consumer.h | 56 + include/linux/pwrseq/provider.h | 75 ++ 32 files changed, 2717 insertions(+), 34 deletions(-) --- base-commit: 6dc544b66971c7f9909ff038b62149105272d26a change-id: 20240527-pwrseq-76fc025248a2 Best regards,