From: Dmitry Baryshkov <email@example.com> To: Steev Klimaszewski <firstname.lastname@example.org> Cc: Andy Gross <email@example.com>, Bjorn Andersson <firstname.lastname@example.org>, Ulf Hansson <email@example.com>, Marcel Holtmann <firstname.lastname@example.org>, Johan Hedberg <email@example.com>, Luiz Augusto von Dentz <firstname.lastname@example.org>, Kalle Valo <email@example.com>, "David S. Miller" <firstname.lastname@example.org>, Jakub Kicinski <email@example.com>, Stanimir Varbanov <firstname.lastname@example.org>, "open list:DRM DRIVER FOR MSM ADRENO GPU" <email@example.com>, linux-mmc <firstname.lastname@example.org>, open list <email@example.com>, "open list:BLUETOOTH SUBSYSTEM" <firstname.lastname@example.org>, email@example.com, linux-wireless <firstname.lastname@example.org>, netdev <email@example.com> Subject: Re: [RFC v2 00/13] create power sequencing subsystem Date: Wed, 6 Oct 2021 06:49:08 +0300 [thread overview] Message-ID: <CAA8EJprsfzFWP1KH61UEkjJmY8rDFTN5i_53Mc0e9n3oxJsBNA@mail.gmail.com> (raw) In-Reply-To: <firstname.lastname@example.org> Hi Steev, On Tue, 14 Sept 2021 at 02:39, Steev Klimaszewski <email@example.com> wrote: > > > On 8/29/21 8:12 AM, Dmitry Baryshkov wrote: > > This is the second RFC on the proposed power sequencer subsystem. This > > is a generification of the MMC pwrseq code. The subsystem tries to > > abstract the idea of complex power-up/power-down/reset of the devices. > > > > To ease migration to pwrseq and to provide compatibility with older > > device trees, while keeping drivers simple, this iteration of RFC > > introduces pwrseq fallback support: pwrseq driver can register fallback > > providers. If another device driver requests pwrseq instance and none > > was declared, the pwrseq fallback code would go through the list of > > fallback providers and if the match is found, driver would return a > > crafted pwrseq instance. For now this mechanism is limited to the OF > > device matching, but it can be extended further to use any combination > > of device IDs. > > > > The primary set of devices that promted me to create this patchset is > > the Qualcomm BT+WiFi family of chips. They reside on serial+platform or > > serial + SDIO interfaces (older generations) or on serial+PCIe (newer > > generations). They require a set of external voltage regulators to be > > powered on and (some of them) have separate WiFi and Bluetooth enable > > GPIOs. > > > > This patchset being an RFC tries to demonstrate the approach, design and > > usage of the pwrseq subsystem. Following issues are present in the RFC > > at this moment but will be fixed later if the overall approach would be > > viewed as acceptable: > > > > - No documentation > > While the code tries to be self-documenting proper documentation > > would be required. > > > > - Minimal device tree bindings changes > > There are no proper updates for the DT bindings (thus neither Rob > > Herring nor devicetree are included in the To/Cc lists). The dt > > schema changes would be a part of v1. > > > > - Lack of proper PCIe integration > > At this moment support for PCIe is hacked up to be able to test the > > PCIe part of qca6390. Proper PCIe support would require automatically > > powering up the devices before the bus scan depending on the proper > > device structure in the device tree. > > > > Changes since RFC v1: > > - Provider pwrseq fallback support > > - Implement fallback support in pwrseq_qca. > > - Mmove susclk handling to pwrseq_qca. > > - Significantly simplify hci_qca.c changes, by dropping all legacy > > code. Now hci_qca uses only pwrseq calls to power up/down bluetooth > > parts of the chip. > > > I tested this here, on the Lenovo Yoga C630, after creating a patch to > do basically the same thing as the db845c does. One thing I noticed, if > PWRSEQ=y and the rest are =m, there is a build error. I suppose once > the full set is posted and not RFC, I can send the patch for that. Please excuse me for the delay in the response. I was carried away by other duties. Yes, could you please provide a fixup patch. I'm going to send v1 now, containing mostly cosmetical and documentation changes. I'll include your patch in v2. > One question I have, if you don't mind, in patch 11, you add a second > channel to qca power sequencer. I've added that here, but in the c630's > dts, "vreg_l23a_3p3: ldo23" is empty, so I added the same numbers in for > the regulator, and I'm wondering how to test that it's actually working > correctly? That's a good question. I have not looked in the details in the ath10k documentation. I'll try finding it. Maybe Kalle Valo can answer your question. Could you please duplicate your question on the ath10k mailing list? -- With best wishes Dmitry _______________________________________________ ath10k mailing list firstname.lastname@example.org http://lists.infradead.org/mailman/listinfo/ath10k
prev parent reply other threads:[~2021-10-06 3:50 UTC|newest] Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-29 13:12 Dmitry Baryshkov 2021-08-29 13:12 ` [RFC v2 01/13] power: add power sequencer subsystem Dmitry Baryshkov 2021-09-10 10:02 ` Ulf Hansson 2021-09-13 12:32 ` Dmitry Baryshkov 2021-09-13 13:42 ` Ulf Hansson 2021-08-29 13:12 ` [RFC v2 02/13] pwrseq: port MMC's pwrseq drivers to new pwrseq subsystem Dmitry Baryshkov 2021-08-29 13:12 ` [RFC v2 03/13] mmc: core: switch " Dmitry Baryshkov 2021-08-29 13:12 ` [RFC v2 04/13] pwrseq: add support for QCA BT+WiFi power sequencer Dmitry Baryshkov 2021-08-29 13:12 ` [RFC v2 05/13] pwrseq: add fallback support Dmitry Baryshkov 2021-08-29 13:12 ` [RFC v2 06/13] pwrseq: pwrseq_qca: implement " Dmitry Baryshkov 2021-08-29 13:12 ` [RFC v2 07/13] Bluetooth: hci_qca: switch to using pwrseq Dmitry Baryshkov 2021-08-29 13:13 ` [RFC v2 08/13] ath10k: add support for pwrseq sequencing Dmitry Baryshkov 2021-08-29 13:13 ` [RFC v2 09/13] arm64: dts: qcom: sdm845-db845c: switch bt+wifi to qca power sequencer Dmitry Baryshkov 2021-08-29 13:13 ` [RFC v2 10/13] arm64: dts: qcom: qrb5165-rb5: add bluetooth support Dmitry Baryshkov 2021-08-29 13:13 ` [RFC v2 11/13] arm64: dts: qcom: sdm845-db845c: add second channel to qca power sequencer Dmitry Baryshkov 2021-08-29 13:13 ` [RFC v2 12/13] WIP: PCI: qcom: use pwrseq to power up bus devices Dmitry Baryshkov 2021-08-29 13:13 ` [RFC v2 13/13] WIP: arm64: dts: qcom: qrb5165-rb5: add bus-pwrseq property to pcie0 Dmitry Baryshkov 2021-09-13 23:39 ` [RFC v2 00/13] create power sequencing subsystem Steev Klimaszewski 2021-10-06 3:49 ` Dmitry Baryshkov [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAA8EJprsfzFWP1KH61UEkjJmY8rDFTN5i_53Mc0e9n3oxJsBNA@mail.gmail.com \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [RFC v2 00/13] create power sequencing subsystem' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).