From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 48E8047ECC7 for ; Thu, 2 Jul 2026 09:32:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782984752; cv=none; b=tEFWFEZ3ySzca4Q7qz43IBvrrnpmo/L8mkU+uakXpHeOvV4h7vedDZtHgUlBwpm8lPIDLo8xncavnofYMITTfn7PFmFePYXEz3ZDPL6eQPuxTBgtuQDT2cd3Lz62O4j0+2FbcnLBVdh26bgCm1DEEgMUmcLUsrz6ZixRKV3j9x0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782984752; c=relaxed/simple; bh=x171MZA9tmoWMFsEZIltUkzuz6HwatXsPVOsCwvOgnM=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=rRGGAjUzkyVmM7eJIogXdJ/Kgqpz+9XqeWfuDkkPfsP8UIugmCZzltAOPARioLvN3T19lt/LRlr0gYSKS5WcDl1BBYMRctZPkBId0UditNXJPHzzVrKGpATn7CcNcRo4fKfR2Lv5mIO4Cr9lwax6frt7jo4Xs2CGD3lGG9I/4A8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=XmxP3Mrd; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=PojEDEs3; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="XmxP3Mrd"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="PojEDEs3" Received: from pps.filterd (m0279862.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 6629Knlm4116761 for ; Thu, 2 Jul 2026 09:32:30 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= ezVVq2U50VUNDTO74pfMslMnYi0egB4QQ7ic+2Doqcw=; b=XmxP3Mrd0lfibHS/ 6AA3jEyMX1ipSvtPD2Gh6XvzG5cPAwFp4qnqZZmh91WlNhs2dt4UwDz8bOn3WQjb W8cYZIwrYsPjqKlreA9VCKRHCHET7r9wv7Q2bTpwTpTGSuKIO0gctPPPNlk0MHmm tmWUxFDDVVfvdT6Pm8mC/kgYtVE3HHi5nTogSE7MIJ+Lpzhp2FWuKIfYCuj3LUzQ 8fDh7aFnsN9ZHRlFF5kHSpdq4a+gtJS77gPMuUCun3l1IErSDA3mgLG3v1ubFeM4 uEiWlXaevV7WDefPM7QfRMghA0T8KwEHjDrO2E8dMqBdgRJY0jyHJ2KXNFSxkOFX O5ylLQ== Received: from mail-pf1-f198.google.com (mail-pf1-f198.google.com [209.85.210.198]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4f5n9bg1mj-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Thu, 02 Jul 2026 09:32:30 +0000 (GMT) Received: by mail-pf1-f198.google.com with SMTP id d2e1a72fcca58-846f50381a6so1210517b3a.0 for ; Thu, 02 Jul 2026 02:32:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1782984750; x=1783589550; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:content-language :from:references:cc:to:subject:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to:content-type; bh=ezVVq2U50VUNDTO74pfMslMnYi0egB4QQ7ic+2Doqcw=; b=PojEDEs3WC1cR2mYKxALMgKEqeYcKhTayG2tzZ4eZ5ItRu62ANbZlxWGmgGYnZPkNt dZM1kMKXz3sF+TThe4Mq7yJ3LjuVlni+w0G/4jXglevWJ3WgM12YvHqfiJrg3vaShOes aTIC+R3125Xi0HbkdD787vU0qYwtcpY/MT94tN2wkZ+6mwK4hOLUtY1huwMOkTUsxkez pjecZsIwo9TAi54/3zsC7oZvhGJquRYMdPbHRE/9jbFezWSmdIAGtNLUCPDvgW/HzpEN FytxbzMwDR387uy2tD2C52JaFMo4R+6PVNwxssj8a8FReYb0aLwbvl9I2n0oxG8pBHQM kJ2A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782984750; x=1783589550; h=content-transfer-encoding:content-type:in-reply-to:content-language :from:references:cc:to:subject:user-agent:mime-version:date :message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=ezVVq2U50VUNDTO74pfMslMnYi0egB4QQ7ic+2Doqcw=; b=ps1kZ4Epg7EcQxUsCieNHNoURHO93znHyFETvm6qrTMm89klQQJoPzGV7U2YPQEZ8P 09Zb4T5YOQV/DbsmNMn1O9TCVvIUqAV/a+aZ0kljdnigbvc8m/bSg5oikJB5l1+eQzWM PaC3QHSNlfH/damj6U2CCiEHyKS+MYIFhUU8lti/1n3w84vhQFFVrOowoHGiPx4paW7J vy0XigKvNG4m5mr3pxTEbFnrhvZh6E3xLG2p38YemKCCeVAQghg0ToPtdQTC39L8hIiA ENNWAZR1eue7vOw7QifhSI2f9Z4LFXHXxRU6lQViyz2V72CkxPiet6ZWwPPRyoTC9JIb QmrA== X-Forwarded-Encrypted: i=1; AFNElJ8CF7CPGAui+K7MPKqivJw1xdpt/TV6xQ2fMQz/Ych/L+74LUZ2l5ruWKdhEN/b2LHTFd39xdSSg3xDRk4=@vger.kernel.org X-Gm-Message-State: AOJu0Yz3n/RmEm/fPusKsFIbZbjw0uLfpW4J8HuL3PWwgeUlqnT7rBMc BvWy0xKaq5Hemp86XICnvHgsebqZ0aPTGoxUSTCy6lMW+u1Hkdi025URbxoJJ0rLxBQhbTtbOkb zWf1oSFyfeeGcpDAKGpH72bdguhZisYwfhp6mM/P156Nym4BBmxdgVg55kCZj4mbSNRQ= X-Gm-Gg: AfdE7clIWZkdOXV6Ui1cUFWb2PKPZycoadAeEcAo990a6ZDdRe4k8MnCgLtQt66z/aP sAdl79bWnZyY4BPIwsv/V4oxyhRbdv+k6li1I47ewUBkNDOXAiERvn/lZVy2wzsQghcu2AqL8WD b2l8xzw48d7B3tUIgrsJBwXJa1a5oGh9M9wQv8Tms0htY26crLjyxrR2SOBhzDuNjr4vSX1G5n7 22a2+b5lHaR67WsXc4IQXfUXZ5fq27OHzJ7z4X3tYQfhqW0Btg2M0LiLsbrlLuj1dCoeJaZhOlS ajWO4GeR8NFnHHYor3HbROFaGKXUgIXZp67/nNQu+aqOZ7PpydkOXKFCP+DgOLadxVgSO7yA5H+ k7bUoFtHgqxgsB8FmdpYKENLE70LTxCb60zolNVWHnLIh9U05QzHPDsAXl40jDh8QVoBKKsOEq1 0+Nyhd6rDa X-Received: by 2002:a05:6a20:d808:b0:3bf:b228:10ae with SMTP id adf61e73a8af0-3bff402bcd9mr5542813637.5.1782984749519; Thu, 02 Jul 2026 02:32:29 -0700 (PDT) X-Received: by 2002:a05:6a20:d808:b0:3bf:b228:10ae with SMTP id adf61e73a8af0-3bff402bcd9mr5542772637.5.1782984748824; Thu, 02 Jul 2026 02:32:28 -0700 (PDT) Received: from [10.133.33.211] (tpe-colo-wan-fw-bordernet.qualcomm.com. [103.229.16.4]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c9e92b81be9sm994089a12.30.2026.07.02.02.32.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 02 Jul 2026 02:32:28 -0700 (PDT) Message-ID: <8024a15f-bbab-47ba-bd18-f85eb1fe3dcd@oss.qualcomm.com> Date: Thu, 2 Jul 2026 17:32:23 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/2] wifi: ath12k: support calibration-variant from device tree To: Andrew LaMarche , Johannes Berg , Jeff Johnson , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-wireless@vger.kernel.org, devicetree@vger.kernel.org, ath12k@lists.infradead.org, linux-kernel@vger.kernel.org, Ernest Van Hoecke References: <20260630133001.1426824-1-andrewjlamarche@gmail.com> From: Baochen Qiang Content-Language: en-US In-Reply-To: <20260630133001.1426824-1-andrewjlamarche@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-ORIG-GUID: pWSx2aANpvQZ-ewQiCL3sW_prn-2sEUV X-Proofpoint-Spam-Info: AW1haW4tMjYwNzAyMDA5NyBTYWx0ZWRfX4H2himyRCegp 6bx107rVwVkgnz8Oc8qFK+HCxO31rKQWTb5KAFWPPu+SQ7HCWUaS4VI17L9pgjBzwQJP0KaSKn0 y7yAz5OlE+7iPOrt8I3bz11F1mTodww= X-Proofpoint-GUID: pWSx2aANpvQZ-ewQiCL3sW_prn-2sEUV X-Authority-Analysis: v=2.4 cv=bOom5v+Z c=1 sm=1 tr=0 ts=6a46302e cx=c_pps a=m5Vt/hrsBiPMCU0y4gIsQw==:117 a=nuhDOHQX5FNHPW3J6Bj6AA==:17 a=IkcTkHD0fZMA:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=_K5XuSEh1TEqbUxoQ0s3:22 a=VwQbUJbxAAAA:8 a=pGLkceISAAAA:8 a=DTxyy6x_7cgoCtCqgnsA:9 a=QEXdDO2ut3YA:10 a=IoOABgeZipijB_acs4fv:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzAyMDA5NyBTYWx0ZWRfX2WlEGwwsiHG+ CHkwRvKYcoWoiudI3b+BYr3DzxcJ/zCS4x5R1UyOZ7XPYl/ICiBI75AV9sXkiC5VzSz6ULo8cvb i9E59wXHSi0NTpMYTSOGCiV7BPMZuflOPDHD0x+YUdgttV1kwtKM75H8tzfiyAtwLZam0VsGmzI jA3CGApu2l31pDdugJk/sbEng6aLOHe78bqV6KwrGlTz+JY+Z2CbTC5ne7q+YD9UXKo3qRlF4Ff L0wkQ03tVxxh7i7Jum8JYMY/WyqfU78HXwbzo7vV/tV0rZH1mNlI7hkkifsVzTJ5WOx7P4i/GYp 9+bshoXWLb+T6YTR6iR8vEewSDpLYeHxtpbYM9l8f0e2HoizuMQ3YnKMHy946Gu7BXBTCuaYknQ 4SilhXD4jnotPU0wZ96VOLC1mQSxdSR3XMoyeIcauEVbTWATGRIiny75iT+CCx7leRIxhS2Lvf1 eV055vZo2W0SqJQ+3Mg== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.125,FMLib:17.12.100.49 definitions=2026-07-02_01,2026-06-26_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 lowpriorityscore=0 bulkscore=0 adultscore=0 clxscore=1015 suspectscore=0 phishscore=0 priorityscore=1501 impostorscore=0 malwarescore=0 spamscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607020097 On 6/30/2026 9:30 PM, Andrew LaMarche wrote: > ath10k and ath11k support reading calibration variants from the device > tree to locate the correct Board Description File (BDF). The ath12k-wsi > binding already describes using qcom,calibration-variant but it is not > implemented in the driver. > > Many ath12k designs expose all the radios under a single phy, each of > which typically require a separate BDF. Without this, the radios may not > come up or will not be calibrated correctly. > > Fix this by parsing the device tree for the generation-agnostic > qcom,calibration-variant. This allows the driver to properly select, > read and apply the correct BDF. > > Signed-off-by: Andrew LaMarche > --- > Changes in v2: > - Drop ath12k-specific binding qcom,ath12k-calibration-variant in favor > of generation agnostic qcom,calibration-variant. > - Link to v1: https://lore.kernel.org/all/20260131003222.2011259-1-andrewjlamarche@gmail.com > --- > drivers/net/wireless/ath/ath12k/core.c | 22 ++++++++++++++++++++++ > drivers/net/wireless/ath/ath12k/qmi.c | 4 ++++ > 2 files changed, 26 insertions(+) > > diff --git a/drivers/net/wireless/ath/ath12k/core.c b/drivers/net/wireless/ath/ath12k/core.c > index 742d4fd1b598..52c28864c63c 100644 > --- a/drivers/net/wireless/ath/ath12k/core.c > +++ b/drivers/net/wireless/ath/ath12k/core.c > @@ -812,6 +812,28 @@ int ath12k_core_check_smbios(struct ath12k_base *ab) > return 0; > } > > +int ath12k_core_check_dt(struct ath12k_base *ab) > +{ > + size_t max_len = sizeof(ab->qmi.target.bdf_ext); > + const char *variant = NULL; > + struct device_node *node; > + > + node = ab->dev->of_node; > + if (!node) > + return -ENOENT; > + > + of_property_read_string(node, "qcom,calibration-variant", > + &variant); > + if (!variant) > + return -ENODATA; > + > + if (strscpy(ab->qmi.target.bdf_ext, variant, max_len) < 0) > + ath12k_dbg(ab, ATH12K_DBG_BOOT, > + "bdf variant string is longer than the buffer can accommodate (variant: %s)\n", variant); > + > + return 0; > +} > + > static int ath12k_core_soc_create(struct ath12k_base *ab) > { > int ret; > diff --git a/drivers/net/wireless/ath/ath12k/qmi.c b/drivers/net/wireless/ath/ath12k/qmi.c > index fd762b5d7bb5..3cf39ae06fd3 100644 > --- a/drivers/net/wireless/ath/ath12k/qmi.c > +++ b/drivers/net/wireless/ath/ath12k/qmi.c > @@ -2945,6 +2945,10 @@ int ath12k_qmi_request_target_cap(struct ath12k_base *ab) > if (r) > ath12k_dbg(ab, ATH12K_DBG_QMI, "SMBIOS bdf variant name not set.\n"); > > + r = ath12k_core_check_dt(ab); I understand there are already some variant exacting helper (ACPI, SMBIOS) being called here, but they really should not. After all, how could a qmi message relates to ACPI or SMBIOS? I'd like firstly move the existing helpers out and then add the new DT helper. > + if (r)> + ath12k_dbg(ab, ATH12K_DBG_QMI, "DT bdf variant name not set.\n"); > + > r = ath12k_acpi_start(ab); > if (r) > /* ACPI is optional so continue in case of an error */