mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Bartosz Golaszewski <brgl@kernel.org>,
	Konrad Dybcio <konradybcio@kernel.org>
Cc: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>,
	linux-arm-msm@vger.kernel.org, linux-bluetooth@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Jeff Johnson <jeff.johnson@oss.qualcomm.com>,
	Bartosz Golaszewski <brgl@bgdev.pl>,
	Marcel Holtmann <marcel@holtmann.org>,
	Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Balakrishna Godavarthi <quic_bgodavar@quicinc.com>,
	Rocky Liao <quic_rjliao@quicinc.com>,
	Bjorn Andersson <andersson@kernel.org>
Subject: Re: [PATCH 00/10] Fix up WCN6855 RFA power supply name
Date: Mon, 8 Dec 2025 13:29:40 +0100	[thread overview]
Message-ID: <4d66229b-5d27-41bc-950f-534d489a8371@oss.qualcomm.com> (raw)
In-Reply-To: <CAMRc=Mcx1E3w7roY55HhKZg6pqV+NRkp87KYa5=BUEVJHOXVLw@mail.gmail.com>

On 12/6/25 11:36 AM, Bartosz Golaszewski wrote:
> On Fri, 5 Dec 2025 13:47:19 +0100, Konrad Dybcio <konradybcio@kernel.org> said:
>> Commit 5f4f954bba12 ("dt-bindings: bluetooth: bring the HW description
>> closer to reality for wcn6855") renamed the supply representing the
>> VDD17_PMU_RFA leg to mention "1p8" instead.
>>
>> While the supply's voltage is normally 1.8 V, the bindings should
>> reflect the actual naming of the pin. Moreover, almost all DTs define
>> the output as 1p7, so that ends up looking a little odd..
>>
>> Ultimately, this is a "fake" regulator that is consumed for the sake
>> of DT sanity and this series is meant to bring uniformity and squash
>> checker errors.
>>
>> The last patch fixes up a less-trivial warning.
>> sc8280xp-microsoft-arcata is left out because I don't know whether the
>> mounting of the chip is on-board or as an M.2-y card (although I would
>> guesstimate the prior).
>>
>> Depends on the long-ready-for-the-merge series by Krzysztof (per-chip
>> bindings split of qualcomm-bluetooth.yaml), mentioned below
>>
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
>> ---
> 
> I want to belive there was a reason for it but I no longer remember. :( Is it
> possible the schematics for the lenovo X13s laptop had it described as 1p7?

I think I checked the schematics of an add-in card used with Makena
and Lemans designs and the WCN6855 datasheet.

At the end of the day, one could argue for both levels but let's just
have one!

Thanks for the rb!

Konrad

  reply	other threads:[~2025-12-08 12:29 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-05 12:47 Konrad Dybcio
2025-12-05 12:47 ` [PATCH 01/10] dt-bindings: net: bluetooth: qualcomm: Fix WCN6855 regulator names Konrad Dybcio
2025-12-08  6:59   ` Krzysztof Kozlowski
2025-12-08 12:30     ` Konrad Dybcio
2025-12-09  6:04       ` Krzysztof Kozlowski
2026-01-19 19:38   ` Bjorn Andersson
2025-12-05 12:47 ` [PATCH 02/10] arm64: dts: qcom: qcs615-ride: Fix BT RFA supply name Konrad Dybcio
2025-12-05 12:47 ` [PATCH 03/10] arm64: dts: qcom: sc8280xp-crd: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 04/10] arm64: dts: qcom: sc8280xp-gaokun3: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 05/10] arm64: dts: qcom: sc8280xp-x13s: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 06/10] arm64: dts: qcom: sc8280xp-blackrock: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 07/10] arm64: dts: qcom: sm8450-hdk: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 08/10] arm64: dts: qcom: x1-omnibook-x14: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 09/10] arm64: dts: qcom: x1-zenbook-a14: " Konrad Dybcio
2025-12-05 12:47 ` [PATCH 10/10] arm64: dts: qcom: lemans-ride-common: Fix up WCN power grid Konrad Dybcio
2025-12-06 10:36 ` [PATCH 00/10] Fix up WCN6855 RFA power supply name Bartosz Golaszewski
2025-12-08 12:29   ` Konrad Dybcio [this message]
2025-12-07 21:49 ` Abel Vesa

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=4d66229b-5d27-41bc-950f-534d489a8371@oss.qualcomm.com \
    --to=konrad.dybcio@oss.qualcomm.com \
    --cc=andersson@kernel.org \
    --cc=brgl@bgdev.pl \
    --cc=brgl@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jeff.johnson@oss.qualcomm.com \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luiz.dentz@gmail.com \
    --cc=luiz.von.dentz@intel.com \
    --cc=marcel@holtmann.org \
    --cc=quic_bgodavar@quicinc.com \
    --cc=quic_rjliao@quicinc.com \
    --cc=robh@kernel.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®