diff mbox series

[v1] arm64: dts: qcom: sc7280: Add bluetooth node on SC7280 crd board

Message ID 1644207878-19839-1-git-send-email-quic_saluvala@quicinc.com (mailing list archive)
State Handled Elsewhere
Delegated to: Luiz Von Dentz
Headers show
Series [v1] arm64: dts: qcom: sc7280: Add bluetooth node on SC7280 crd board | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
tedd_an/checkpatch success Checkpatch PASS
tedd_an/gitlint success Gitlint PASS
tedd_an/subjectprefix fail "Bluetooth: " is not specified in the subject
tedd_an/buildkernel success Build Kernel PASS
tedd_an/buildkernel32 success Build Kernel32 PASS
tedd_an/incremental_build success Pass
tedd_an/testrunnersetup success Test Runner Setup PASS
tedd_an/testrunnerl2cap-tester success Total: 40, Passed: 40 (100.0%), Failed: 0, Not Run: 0
tedd_an/testrunnerbnep-tester success Total: 1, Passed: 1 (100.0%), Failed: 0, Not Run: 0
tedd_an/testrunnermgmt-tester success Total: 493, Passed: 493 (100.0%), Failed: 0, Not Run: 0
tedd_an/testrunnerrfcomm-tester success Total: 9, Passed: 9 (100.0%), Failed: 0, Not Run: 0
tedd_an/testrunnersco-tester success Total: 12, Passed: 12 (100.0%), Failed: 0, Not Run: 0
tedd_an/testrunnersmp-tester success Total: 8, Passed: 8 (100.0%), Failed: 0, Not Run: 0
tedd_an/testrunneruserchan-tester success Total: 4, Passed: 4 (100.0%), Failed: 0, Not Run: 0

Commit Message

Sai Teja Aluvala Feb. 7, 2022, 4:24 a.m. UTC
Add Bluetooth SoC WCN6750 node for SC7280 crd board

Signed-off-by: Sai Teja Aluvala <quic_saluvala@quicinc.com>

---
 arch/arm64/boot/dts/qcom/sc7280-crd.dts | 4 ++++
 1 file changed, 4 insertions(+)

Comments

bluez.test.bot@gmail.com Feb. 7, 2022, 7:06 a.m. UTC | #1
This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=611723

---Test result---

Test Summary:
CheckPatch                    PASS      1.68 seconds
GitLint                       PASS      1.02 seconds
SubjectPrefix                 FAIL      0.90 seconds
BuildKernel                   PASS      33.70 seconds
BuildKernel32                 PASS      29.33 seconds
Incremental Build with patchesPASS      40.42 seconds
TestRunner: Setup             PASS      539.08 seconds
TestRunner: l2cap-tester      PASS      14.95 seconds
TestRunner: bnep-tester       PASS      7.10 seconds
TestRunner: mgmt-tester       PASS      122.04 seconds
TestRunner: rfcomm-tester     PASS      8.38 seconds
TestRunner: sco-tester        PASS      8.60 seconds
TestRunner: smp-tester        PASS      8.68 seconds
TestRunner: userchan-tester   PASS      7.49 seconds

Details
##############################
Test: SubjectPrefix - FAIL - 0.90 seconds
Check subject contains "Bluetooth" prefix
"Bluetooth: " is not specified in the subject



---
Regards,
Linux Bluetooth
Sai Teja Aluvala Feb. 7, 2022, 8:42 a.m. UTC | #2
+ bjorn

-----Original Message-----
From: Sai Teja Aluvala <quic_saluvala@quicinc.com> 
Sent: Monday, February 7, 2022 9:55 AM
To: marcel@holtmann.org; johan.hedberg@gmail.com
Cc: mka@chromium.org; linux-kernel@vger.kernel.org; linux-bluetooth@vger.kernel.org; Hemant Gupta (QUIC) <quic_hemantg@quicinc.com>; linux-arm-msm@vger.kernel.org; quic_bgodavar <quic_bgodavar@quicinc.com>; quic_rjliao <quic_rjliao@quicinc.com>; quic_hbandi <quic_hbandi@quicinc.com>; abhishekpandit@chromium.org; mcchou@chromium.org; Sai Teja Aluvala (Temp) (QUIC) <quic_saluvala@quicinc.com>
Subject: [PATCH v1] arm64: dts: qcom: sc7280: Add bluetooth node on SC7280 crd board

Add Bluetooth SoC WCN6750 node for SC7280 crd board

Signed-off-by: Sai Teja Aluvala <quic_saluvala@quicinc.com>

---
 arch/arm64/boot/dts/qcom/sc7280-crd.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sc7280-crd.dts b/arch/arm64/boot/dts/qcom/sc7280-crd.dts
index cd2755c..53ea3b4 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-crd.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-crd.dts
@@ -23,6 +23,10 @@
 	};
 };
 
+&bluetooth {
+	vddio-supply = <&vreg_l18b_1p8>;
+};
+
 ap_tp_i2c: &i2c0 {
 	status = "okay";
 	clock-frequency = <400000>;
kernel test robot Feb. 7, 2022, 11:12 a.m. UTC | #3
Hi Sai,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on v5.17-rc3]
[cannot apply to next-20220207]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Sai-Teja-Aluvala/arm64-dts-qcom-sc7280-Add-bluetooth-node-on-SC7280-crd-board/20220207-141304
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm64-randconfig-s031-20220207 (https://download.01.org/0day-ci/archive/20220207/202202071900.HBQ0tQbr-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 11.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # apt-get install sparse
        # sparse version: v0.6.4-dirty
        # https://github.com/0day-ci/linux/commit/36d7dc511b4181b7243171a2949e2f097d799d40
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Sai-Teja-Aluvala/arm64-dts-qcom-sc7280-Add-bluetooth-node-on-SC7280-crd-board/20220207-141304
        git checkout 36d7dc511b4181b7243171a2949e2f097d799d40
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=arm64 SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> Error: arch/arm64/boot/dts/qcom/sc7280-crd.dts:26.1-11 Label or path bluetooth not found
   FATAL ERROR: Syntax error parsing input tree

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Matthias Kaehlcke Feb. 7, 2022, 5:13 p.m. UTC | #4
On Mon, Feb 07, 2022 at 09:54:38AM +0530, Sai Teja Aluvala wrote:

> Subject: arm64: dts: qcom: sc7280: Add bluetooth node on SC7280 crd board
>
> Add Bluetooth SoC WCN6750 node for SC7280 crd board

That doesn't describe accurately what this patch does. The CRD already
has a Bluetooth node - from including sc7280-idp.dtsi - what this patch
does is setting the vddio supply for Bluetooth on the CRD.
Sai Teja Aluvala Feb. 8, 2022, 4:13 a.m. UTC | #5
-----Original Message-----
From: Matthias Kaehlcke <mka@chromium.org> 
Sent: Monday, February 7, 2022 10:43 PM
To: Sai Teja Aluvala (Temp) (QUIC) <quic_saluvala@quicinc.com>
Cc: marcel@holtmann.org; johan.hedberg@gmail.com; linux-kernel@vger.kernel.org; linux-bluetooth@vger.kernel.org; Hemant Gupta (QUIC) <quic_hemantg@quicinc.com>; linux-arm-msm@vger.kernel.org; quic_bgodavar <quic_bgodavar@quicinc.com>; quic_rjliao <quic_rjliao@quicinc.com>; quic_hbandi <quic_hbandi@quicinc.com>; abhishekpandit@chromium.org; mcchou@chromium.org
Subject: Re: [PATCH v1] arm64: dts: qcom: sc7280: Add bluetooth node on SC7280 crd board

On Mon, Feb 07, 2022 at 09:54:38AM +0530, Sai Teja Aluvala wrote:

> Subject: arm64: dts: qcom: sc7280: Add bluetooth node on SC7280 crd 
> board
>
> Add Bluetooth SoC WCN6750 node for SC7280 crd board

That doesn't describe accurately what this patch does. The CRD already has a Bluetooth node - from including sc7280-idp.dtsi - what this patch does is setting the vddio supply for Bluetooth on the CRD.
[Sai]: will update in next patch
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sc7280-crd.dts b/arch/arm64/boot/dts/qcom/sc7280-crd.dts
index cd2755c..53ea3b4 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-crd.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-crd.dts
@@ -23,6 +23,10 @@ 
 	};
 };
 
+&bluetooth {
+	vddio-supply = <&vreg_l18b_1p8>;
+};
+
 ap_tp_i2c: &i2c0 {
 	status = "okay";
 	clock-frequency = <400000>;