mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org
Subject: Re: [PATCH RFC] arm64: dts: mediatek: mt8183: Switch to undeprecated qcom,calibration-variant
Date: Tue, 25 Feb 2025 11:23:23 +0100	[thread overview]
Message-ID: <c50e114a-bb5b-40a4-ab01-69ee33b984b8@collabora.com> (raw)
In-Reply-To: <20250225101828.107509-1-krzysztof.kozlowski@linaro.org>

Il 25/02/25 11:18, Krzysztof Kozlowski ha scritto:
> The property qcom,ath10k-calibration-variant was deprecated in favor of
> recently introduced generic qcom,calibration-variant, common to all
> Qualcomm Atheros WiFi bindings.
> 
> Change will affect out of tree users, like other projects, of this DTS.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 

Ah, finally. That's nice :-)

Regardless of the RFC status....:

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

> ---
> 
> Dependency/RFC!
> 
> RFC, because this should be merged release after driver support is
> merged:
> https://lore.kernel.org/linux-devicetree/20250225-b-wifi-qcom-calibration-variant-v1-0-3b2aa3f89c53@linaro.org/T/#t
> ---
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dts     | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts      | 2 +-
>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dts      | 2 +-
>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dts      | 2 +-
>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku7.dts      | 2 +-
>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14-sku2.dts    | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dts  | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper.dtsi  | 2 +-
>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku0.dts      | 2 +-
>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku1.dts      | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow.dtsi   | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtsi           | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku32.dts       | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku38.dts       | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama.dtsi           | 2 +-
>   arch/arm64/boot/dts/mediatek/mt8183-kukui-krane.dtsi            | 2 +-
>   16 files changed, 16 insertions(+), 16 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dts
> index 83bbcfe62083..f2303d9f125f 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dts
> @@ -35,5 +35,5 @@ trackpad@2c {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_COZMO";
> +	qcom,calibration-variant = "GO_COZMO";
>   };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts
> index 3935d83a047e..13ec15fa284f 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts
> @@ -31,7 +31,7 @@ &mt6358codec {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_DAMU";
> +	qcom,calibration-variant = "GO_DAMU";
>   };
>   
>   &i2c2 {
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dts
> index 72852b760038..b576b974cf23 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dts
> @@ -40,6 +40,6 @@ &touchscreen {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL";
> +	qcom,calibration-variant = "GO_FENNEL";
>   };
>   
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dts
> index 757d0afd14fb..d64581f35d2c 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dts
> @@ -28,6 +28,6 @@ &touchscreen {
>   
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL";
> +	qcom,calibration-variant = "GO_FENNEL";
>   };
>   
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku7.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku7.dts
> index 6641b087e7c5..d0c98d0aba95 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku7.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku7.dts
> @@ -28,6 +28,6 @@ &touchscreen {
>   
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL";
> +	qcom,calibration-variant = "GO_FENNEL";
>   };
>   
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14-sku2.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14-sku2.dts
> index 877256eab262..3a81786084da 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14-sku2.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14-sku2.dts
> @@ -14,5 +14,5 @@ / {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL14";
> +	qcom,calibration-variant = "GO_FENNEL14";
>   };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dts
> index b981dd31a430..959c4d3d9707 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dts
> @@ -14,5 +14,5 @@ / {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL14";
> +	qcom,calibration-variant = "GO_FENNEL14";
>   };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper.dtsi
> index 078bc765646f..4788edaeab87 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper.dtsi
> @@ -22,6 +22,6 @@ trackpad@2c {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_JUNIPER";
> +	qcom,calibration-variant = "GO_JUNIPER";
>   };
>   
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku0.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku0.dts
> index ddb993521bbf..01cd59993a7c 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku0.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku0.dts
> @@ -14,7 +14,7 @@ / {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL14";
> +	qcom,calibration-variant = "GO_FENNEL14";
>   };
>   
>   &mmc1_pins_uhs {
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku1.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku1.dts
> index 10c4f920a7d8..c939ef3965ec 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku1.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-makomo-sku1.dts
> @@ -14,7 +14,7 @@ / {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_FENNEL14";
> +	qcom,calibration-variant = "GO_FENNEL14";
>   };
>   
>   &mmc1_pins_uhs {
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow.dtsi
> index c942e461a177..14d03419b92c 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow.dtsi
> @@ -37,5 +37,5 @@ trackpad@2c {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_JUNIPER";
> +	qcom,calibration-variant = "GO_JUNIPER";
>   };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtsi
> index ff02f63bac29..c4c08c0f715e 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtsi
> @@ -370,7 +370,7 @@ keyboard-controller {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_KAKADU";
> +	qcom,calibration-variant = "GO_KAKADU";
>   };
>   
>   &panel {
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku32.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku32.dts
> index 2b5a8d1f900e..7428efec95a0 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku32.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku32.dts
> @@ -32,5 +32,5 @@ &panel {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_KATSU";
> +	qcom,calibration-variant = "GO_KATSU";
>   };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku38.dts b/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku38.dts
> index 75fadf2c7059..a36b6ddb71f6 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku38.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-katsu-sku38.dts
> @@ -32,7 +32,7 @@ &panel {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_KATSU";
> +	qcom,calibration-variant = "GO_KATSU";
>   };
>   
>   &sound {
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama.dtsi
> index da6e767b4cee..2b283a55f653 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama.dtsi
> @@ -349,7 +349,7 @@ keyboard-controller {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "GO_KODAMA";
> +	qcom,calibration-variant = "GO_KODAMA";
>   };
>   
>   &i2c_tunnel {
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane.dtsi
> index 8b56b8564ed7..00da50a41574 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane.dtsi
> @@ -353,7 +353,7 @@ keyboard-controller {
>   };
>   
>   &qca_wifi {
> -	qcom,ath10k-calibration-variant = "LE_Krane";
> +	qcom,calibration-variant = "LE_Krane";
>   };
>   
>   &sound {



  reply	other threads:[~2025-02-25 10:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-25 10:18 Krzysztof Kozlowski
2025-02-25 10:23 ` AngeloGioacchino Del Regno [this message]
2026-10-02  8:51   ` Chen-Yu Tsai

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=c50e114a-bb5b-40a4-ab01-69ee33b984b8@collabora.com \
    --to=angelogioacchino.delregno@collabora.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.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®