On 12/24/2024 6:57 PM, Krzysztof Kozlowski wrote:
On 24/12/2024 12:51, Janaki Ramaiah Thota wrote:
On 12/24/2024 2:46 PM, Krzysztof Kozlowski wrote:
On Mon, Dec 23, 2024 at 07:27:00PM +0530, Janaki Ramaiah Thota wrote:
Subject: everything is an update... say something useful, e.g. use PMU
abstraction for WCN6750
Sure will correct it in next patch.
Drop the inputs from the host and instead expect the Bluetooth node to
consume the outputs of the internal PMU.
On which device?
It is for BT module wcn6750 attached on qcs6490-rb3gen2 board.
will update same on next commit message.
But anyway I have doubts this is correct. enable GPIO is a pin going
from the host, not from PMU.
Yes you are correct, enable GPIO is out pin of host, but here updated
the entries required for Bluetooth node from PMU, not from host to PMU.
Hm? No, you removed for example enable-gpios, which is input from the
host. Sorry, I don't understand the reason behind this patch and commit
msg does not help me there.
We are migrating to powerseq for the WCN6750. As per Dmitry’s
suggestion, we are now using the on-chip PMU, which necessitates the
removal of some entries that are not mandatory to avoid binding errors.
For example, the enable GPIO is now handled by the PMU with property
bt-enable-gpios, as shown in the patch linked below
https://lore.kernel.org/linux-arm-msm/20241209103455.9675-3-quic_janathot@xxxxxxxxxxx/
Thanks,
Janakiram
