From: kernel test robot <lkp@intel.com>
To: Shawn Guo <shawn.guo@linaro.org>, Stephen Boyd <sboyd@kernel.org>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
Rob Herring <robh+dt@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Loic Poulain <loic.poulain@linaro.org>,
devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org,
linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org,
Shawn Guo <shawn.guo@linaro.org>
Subject: Re: [PATCH 2/2] clk: qcom: Add Global Clock Controller driver for QCM2290
Date: Tue, 14 Sep 2021 23:24:03 +0800 [thread overview]
Message-ID: <202109142311.GcFcnCD1-lkp@intel.com> (raw)
In-Reply-To: <20210914032041.6547-3-shawn.guo@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 5621 bytes --]
Hi Shawn,
I love your patch! Perhaps something to improve:
[auto build test WARNING on clk/clk-next]
[also build test WARNING on robh/for-next v5.15-rc1 next-20210914]
[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/Shawn-Guo/Add-QCM2290-Global-Clock-Controller-driver/20210914-112253
base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
config: arm64-buildonly-randconfig-r004-20210914 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/b510487ae2d062b96312e0947f3ed37b5faa79fc
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Shawn-Guo/Add-QCM2290-Global-Clock-Controller-driver/20210914-112253
git checkout b510487ae2d062b96312e0947f3ed37b5faa79fc
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/clk/qcom/gcc-qcm2290.c:623:32: warning: unused variable 'gcc_parent_map_11' [-Wunused-const-variable]
static const struct parent_map gcc_parent_map_11[] = {
^
>> drivers/clk/qcom/gcc-qcm2290.c:633:37: warning: unused variable 'gcc_parents_11' [-Wunused-const-variable]
static const struct clk_parent_data gcc_parents_11[] = {
^
>> drivers/clk/qcom/gcc-qcm2290.c:683:32: warning: unused variable 'gcc_parent_map_15' [-Wunused-const-variable]
static const struct parent_map gcc_parent_map_15[] = {
^
>> drivers/clk/qcom/gcc-qcm2290.c:688:37: warning: unused variable 'gcc_parents_15' [-Wunused-const-variable]
static const struct clk_parent_data gcc_parents_15[] = {
^
>> drivers/clk/qcom/gcc-qcm2290.c:693:32: warning: unused variable 'gcc_parent_map_16' [-Wunused-const-variable]
static const struct parent_map gcc_parent_map_16[] = {
^
>> drivers/clk/qcom/gcc-qcm2290.c:699:37: warning: unused variable 'gcc_parents_16' [-Wunused-const-variable]
static const struct clk_parent_data gcc_parents_16[] = {
^
6 warnings generated.
vim +/gcc_parent_map_11 +623 drivers/clk/qcom/gcc-qcm2290.c
622
> 623 static const struct parent_map gcc_parent_map_11[] = {
624 { P_BI_TCXO, 0 },
625 { P_GPLL0_OUT_EARLY, 1 },
626 { P_GPLL0_OUT_AUX2, 2 },
627 { P_GPLL5_OUT_MAIN, 3 },
628 { P_GPLL6_OUT_MAIN, 4 },
629 { P_GPLL6_OUT_EARLY, 5 },
630 { P_GPLL3_OUT_EARLY, 6 },
631 };
632
> 633 static const struct clk_parent_data gcc_parents_11[] = {
634 { .fw_name = "bi_tcxo" },
635 { .hw = &gpll0.clkr.hw },
636 { .hw = &gpll0_out_aux2.clkr.hw },
637 { .hw = &gpll5.clkr.hw },
638 { .hw = &gpll6_out_main.clkr.hw },
639 { .hw = &gpll6.clkr.hw },
640 { .hw = &gpll3.clkr.hw },
641 };
642
643 static const struct parent_map gcc_parent_map_12[] = {
644 { P_BI_TCXO, 0 },
645 { P_GPLL0_OUT_EARLY, 1 },
646 { P_GPLL0_OUT_AUX2, 2 },
647 { P_GPLL7_OUT_MAIN, 3 },
648 { P_GPLL4_OUT_MAIN, 5 },
649 };
650
651 static const struct clk_parent_data gcc_parents_12[] = {
652 { .fw_name = "bi_tcxo" },
653 { .hw = &gpll0.clkr.hw },
654 { .hw = &gpll0_out_aux2.clkr.hw },
655 { .hw = &gpll7.clkr.hw },
656 { .hw = &gpll4.clkr.hw },
657 };
658
659 static const struct parent_map gcc_parent_map_13[] = {
660 { P_BI_TCXO, 0 },
661 { P_SLEEP_CLK, 5 },
662 };
663
664 static const struct clk_parent_data gcc_parents_13[] = {
665 { .fw_name = "bi_tcxo" },
666 { .fw_name = "sleep_clk" },
667 };
668
669 static const struct parent_map gcc_parent_map_14[] = {
670 { P_BI_TCXO, 0 },
671 { P_GPLL11_OUT_MAIN, 1 },
672 { P_GPLL11_OUT_AUX, 2 },
673 { P_GPLL11_OUT_AUX2, 3 },
674 };
675
676 static const struct clk_parent_data gcc_parents_14[] = {
677 { .fw_name = "bi_tcxo" },
678 { .hw = &gpll11.clkr.hw },
679 { .hw = &gpll11.clkr.hw },
680 { .hw = &gpll11.clkr.hw },
681 };
682
> 683 static const struct parent_map gcc_parent_map_15[] = {
684 { P_BI_TCXO, 0 },
685 { P_GPLL0_OUT_EARLY, 1 },
686 };
687
> 688 static const struct clk_parent_data gcc_parents_15[] = {
689 { .fw_name = "bi_tcxo" },
690 { .hw = &gpll0.clkr.hw },
691 };
692
> 693 static const struct parent_map gcc_parent_map_16[] = {
694 { P_BI_TCXO, 0 },
695 { P_GPLL0_OUT_EARLY, 1 },
696 { P_GPLL6_OUT_MAIN, 4 },
697 };
698
> 699 static const struct clk_parent_data gcc_parents_16[] = {
700 { .fw_name = "bi_tcxo" },
701 { .hw = &gpll0.clkr.hw },
702 { .hw = &gpll6_out_main.clkr.hw },
703 };
704
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 39902 bytes --]
prev parent reply other threads:[~2021-09-14 15:29 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-14 3:20 [PATCH 0/2] Add QCM2290 Global Clock Controller driver Shawn Guo
2021-09-14 3:20 ` [PATCH 1/2] dt-bindings: clk: qcom: Add QCM2290 Global Clock Controller bindings Shawn Guo
2021-09-14 3:20 ` [PATCH 2/2] clk: qcom: Add Global Clock Controller driver for QCM2290 Shawn Guo
2021-09-14 13:20 ` kernel test robot
2021-09-14 15:24 ` kernel test robot [this message]
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=202109142311.GcFcnCD1-lkp@intel.com \
--to=lkp@intel.com \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=kbuild-all@lists.01.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=loic.poulain@linaro.org \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=shawn.guo@linaro.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;
as well as URLs for NNTP newsgroup(s).