Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
From: Konrad Dybcio <konradybcio@kernel.org>
To: Bartosz Golaszewski <brgl@bgdev.pl>,
	Dmitry Baryshkov <dbaryshkov@gmail.com>,
	Johan Hovold <johan@kernel.org>,
	Konrad Dybcio <konradybcio@kernel.org>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Bartosz Golaszewski <bartosz.golaszewski@linaro.org>,
	Kalle Valo <kvalo@kernel.org>
Subject: Re: [PATCH v3 1/4] arm64: dts: qcom: sc8280xp-crd: model the PMU of the on-board wcn6855
Date: Thu, 5 Sep 2024 15:13:45 +0200	[thread overview]
Message-ID: <d6d5a943-ab29-4034-b465-b62d9d1efa61@kernel.org> (raw)
In-Reply-To: <CAMRc=Mci-8R1Oe3Fe+1E+K-7khzwBPgn_8SQSUPXthpE4032Pw@mail.gmail.com>

On 5.09.2024 3:00 PM, Bartosz Golaszewski wrote:
> On Thu, Sep 5, 2024 at 2:56 PM Dmitry Baryshkov <dbaryshkov@gmail.com> wrote:
>>
>>>>
>>>> As you are going to post another revision, please also add
>>>>
>>>> qcom,ath11k-calibration-variant
>>>>
>>>
>>> I had it in earlier revisions. The only one we could add here would be
>>> the one from X13s as QCom has not yet released the data for the CRD.
>>> Johan and Konrad were against adding this here if it doesn't refer to
>>> the correct one so I dropped it.
>>
>> As Kalle usually merges data with some delay it's not infrequent to
>> have DTS which names calibration variant, but board-2.bin doesn't have
>> corresponding data. The driver safely falls back to the data without
>> variant if it can find it. Als  usually it's us who supply the
>> calibration name.
>>
> 
> Johan, Konrad,
> 
> What do you think? Do we know the exact name and should I add it or
> should we wait until it's in board-2.bin?

If we can agree on the string identifier with Kalle in advance, we can
add it even before the boardfile drops

Konrad

  reply	other threads:[~2024-09-05 13:13 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-05 12:20 [PATCH v3 0/4] arm64: dts: qcom: sc8280xp: enable WLAN and Bluetooth Bartosz Golaszewski
2024-09-05 12:20 ` [PATCH v3 1/4] arm64: dts: qcom: sc8280xp-crd: model the PMU of the on-board wcn6855 Bartosz Golaszewski
2024-09-05 12:50   ` Dmitry Baryshkov
2024-09-05 12:53     ` Bartosz Golaszewski
2024-09-05 12:56       ` Dmitry Baryshkov
2024-09-05 13:00         ` Bartosz Golaszewski
2024-09-05 13:13           ` Konrad Dybcio [this message]
2024-09-05 18:41             ` Kalle Valo
2024-09-05 19:26               ` Dmitry Baryshkov
2024-09-06  7:45                 ` Bartosz Golaszewski
2024-09-06  9:37                   ` Dmitry Baryshkov
2024-09-06  9:45                     ` Bartosz Golaszewski
2024-09-09 11:18                       ` Kalle Valo
2024-09-06 14:53                 ` Johan Hovold
2024-09-06 21:37                   ` Dmitry Baryshkov
2024-09-05 12:20 ` [PATCH v3 2/4] arm64: dts: qcom: sc8280xp-crd: enable bluetooth Bartosz Golaszewski
2024-09-05 12:20 ` [PATCH v3 3/4] arm64: dts: qcom: sc8280xp-x13s: model the PMU of the on-board wcn6855 Bartosz Golaszewski
2024-09-05 12:27   ` Dmitry Baryshkov
2024-09-05 12:29     ` Bartosz Golaszewski
2024-09-05 12:20 ` [PATCH v3 4/4] arm64: dts: qcom: sc8280xp-x13s: keep less regulators always-on Bartosz Golaszewski
2024-09-05 14:04   ` Johan Hovold
2024-09-05 18:23     ` Bartosz Golaszewski
2024-09-05 19:27       ` Dmitry Baryshkov
2024-09-06 14:49         ` Johan Hovold
2024-09-06 21:37           ` Dmitry Baryshkov

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=d6d5a943-ab29-4034-b465-b62d9d1efa61@kernel.org \
    --to=konradybcio@kernel.org \
    --cc=andersson@kernel.org \
    --cc=bartosz.golaszewski@linaro.org \
    --cc=brgl@bgdev.pl \
    --cc=conor+dt@kernel.org \
    --cc=dbaryshkov@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=johan@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=kvalo@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --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