oe-kbuild-all.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Can Guo <quic_cang@quicinc.com>,
	bvanassche@acm.org, mani@kernel.org, vkoul@kernel.org,
	abel.vesa@linaro.org, dmitry.baryshkov@linaro.org,
	neil.armstrong@linaro.org
Cc: oe-kbuild-all@lists.linux.dev, Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	Kishon Vijay Abraham I <kishon@kernel.org>,
	linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v9 2/2] phy: qualcomm: phy-qcom-qmp-ufs: Add High Speed Gear 5 support for SM8550
Date: Tue, 26 Dec 2023 06:19:38 +0800	[thread overview]
Message-ID: <202312260623.FFykk4qF-lkp@intel.com> (raw)
In-Reply-To: <1703472701-34197-3-git-send-email-quic_cang@quicinc.com>

Hi Can,

kernel test robot noticed the following build errors:

[auto build test ERROR on linus/master]
[also build test ERROR on v6.7-rc7]
[cannot apply to next-20231222]
[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#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Can-Guo/phy-qualcomm-phy-qcom-qmp-ufs-Add-High-Speed-Gear-5-support-for-SM8550/20231225-154853
base:   linus/master
patch link:    https://lore.kernel.org/r/1703472701-34197-3-git-send-email-quic_cang%40quicinc.com
patch subject: [PATCH v9 2/2] phy: qualcomm: phy-qcom-qmp-ufs: Add High Speed Gear 5 support for SM8550
config: arc-randconfig-001-20231225 (https://download.01.org/0day-ci/archive/20231226/202312260623.FFykk4qF-lkp@intel.com/config)
compiler: arc-elf-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231226/202312260623.FFykk4qF-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202312260623.FFykk4qF-lkp@intel.com/

All error/warnings (new ones prefixed by >>):

>> drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1071:10: error: 'const struct qmp_phy_cfg' has no member named 'tbls_hs_g4'; did you mean 'tbls_hs_b'?
    1071 |         .tbls_hs_g4 = {
         |          ^~~~~~~~~~
         |          tbls_hs_b
>> drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1072:17: error: field name not in record or union initializer
    1072 |                 .tx             = sm8250_ufsphy_hs_g4_tx,
         |                 ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1072:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1073:17: error: field name not in record or union initializer
    1073 |                 .tx_num         = ARRAY_SIZE(sm8250_ufsphy_hs_g4_tx),
         |                 ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1073:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
   In file included from include/linux/kernel.h:16,
                    from include/linux/clk.h:13,
                    from drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:6:
>> include/linux/array_size.h:11:25: warning: initialization of 'const struct qmp_phy_init_tbl *' from 'unsigned int' makes pointer from integer without a cast [-Wint-conversion]
      11 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
         |                         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1073:35: note: in expansion of macro 'ARRAY_SIZE'
    1073 |                 .tx_num         = ARRAY_SIZE(sm8250_ufsphy_hs_g4_tx),
         |                                   ^~~~~~~~~~
   include/linux/array_size.h:11:25: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay[1].serdes')
      11 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
         |                         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1073:35: note: in expansion of macro 'ARRAY_SIZE'
    1073 |                 .tx_num         = ARRAY_SIZE(sm8250_ufsphy_hs_g4_tx),
         |                                   ^~~~~~~~~~
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1074:17: error: field name not in record or union initializer
    1074 |                 .rx             = sc7280_ufsphy_hs_g4_rx,
         |                 ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1074:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
>> drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1075:17: warning: excess elements in array initializer
    1075 |                 .rx_num         = ARRAY_SIZE(sc7280_ufsphy_hs_g4_rx),
         |                 ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1075:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1075:17: error: field name not in record or union initializer
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1075:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
>> include/linux/array_size.h:11:25: warning: initialization of 'const struct qmp_phy_init_tbl *' from 'unsigned int' makes pointer from integer without a cast [-Wint-conversion]
      11 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
         |                         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1075:35: note: in expansion of macro 'ARRAY_SIZE'
    1075 |                 .rx_num         = ARRAY_SIZE(sc7280_ufsphy_hs_g4_rx),
         |                                   ^~~~~~~~~~
   include/linux/array_size.h:11:25: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay[2].serdes')
      11 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
         |                         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1075:35: note: in expansion of macro 'ARRAY_SIZE'
    1075 |                 .rx_num         = ARRAY_SIZE(sc7280_ufsphy_hs_g4_rx),
         |                                   ^~~~~~~~~~
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1076:17: warning: excess elements in array initializer
    1076 |                 .pcs            = sm8150_ufsphy_hs_g4_pcs,
         |                 ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1076:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1076:17: error: field name not in record or union initializer
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1076:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1077:17: warning: excess elements in array initializer
    1077 |                 .pcs_num        = ARRAY_SIZE(sm8150_ufsphy_hs_g4_pcs),
         |                 ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1077:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1077:17: error: field name not in record or union initializer
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1077:17: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
>> include/linux/array_size.h:11:25: warning: initialization of 'const struct qmp_phy_init_tbl *' from 'unsigned int' makes pointer from integer without a cast [-Wint-conversion]
      11 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
         |                         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1077:35: note: in expansion of macro 'ARRAY_SIZE'
    1077 |                 .pcs_num        = ARRAY_SIZE(sm8150_ufsphy_hs_g4_pcs),
         |                                   ^~~~~~~~~~
   include/linux/array_size.h:11:25: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay[2].serdes')
      11 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
         |                         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1077:35: note: in expansion of macro 'ARRAY_SIZE'
    1077 |                 .pcs_num        = ARRAY_SIZE(sm8150_ufsphy_hs_g4_pcs),
         |                                   ^~~~~~~~~~
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1078:9: warning: excess elements in array initializer
    1078 |         },
         |         ^
   drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1078:9: note: (near initialization for 'sc7280_ufsphy_cfg.tbls_hs_overlay')
>> drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1052:53: warning: missing braces around initializer [-Wmissing-braces]
    1052 | static const struct qmp_phy_cfg sc7280_ufsphy_cfg = {
         |                                                     ^
>> drivers/phy/qualcomm/phy-qcom-qmp-ufs.c:1052:53: warning: missing braces around initializer [-Wmissing-braces]


vim +1071 drivers/phy/qualcomm/phy-qcom-qmp-ufs.c

607c101fe9f2f6 Bartosz Golaszewski 2023-04-11  1051  
8abe9792d1ff7e Nitin Rawat         2023-09-19 @1052  static const struct qmp_phy_cfg sc7280_ufsphy_cfg = {
8abe9792d1ff7e Nitin Rawat         2023-09-19  1053  	.lanes                  = 2,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1054  
8abe9792d1ff7e Nitin Rawat         2023-09-19  1055  	.offsets                = &qmp_ufs_offsets,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1056  
8abe9792d1ff7e Nitin Rawat         2023-09-19  1057  	.tbls = {
8abe9792d1ff7e Nitin Rawat         2023-09-19  1058  		.serdes         = sm8150_ufsphy_serdes,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1059  		.serdes_num     = ARRAY_SIZE(sm8150_ufsphy_serdes),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1060  		.tx             = sc7280_ufsphy_tx,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1061  		.tx_num         = ARRAY_SIZE(sc7280_ufsphy_tx),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1062  		.rx             = sc7280_ufsphy_rx,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1063  		.rx_num         = ARRAY_SIZE(sc7280_ufsphy_rx),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1064  		.pcs            = sc7280_ufsphy_pcs,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1065  		.pcs_num        = ARRAY_SIZE(sc7280_ufsphy_pcs),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1066  	},
8abe9792d1ff7e Nitin Rawat         2023-09-19  1067  	.tbls_hs_b = {
8abe9792d1ff7e Nitin Rawat         2023-09-19  1068  		.serdes         = sm8150_ufsphy_hs_b_serdes,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1069  		.serdes_num     = ARRAY_SIZE(sm8150_ufsphy_hs_b_serdes),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1070  	},
8abe9792d1ff7e Nitin Rawat         2023-09-19 @1071  	.tbls_hs_g4 = {
8abe9792d1ff7e Nitin Rawat         2023-09-19 @1072  		.tx             = sm8250_ufsphy_hs_g4_tx,
8abe9792d1ff7e Nitin Rawat         2023-09-19 @1073  		.tx_num         = ARRAY_SIZE(sm8250_ufsphy_hs_g4_tx),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1074  		.rx             = sc7280_ufsphy_hs_g4_rx,
8abe9792d1ff7e Nitin Rawat         2023-09-19 @1075  		.rx_num         = ARRAY_SIZE(sc7280_ufsphy_hs_g4_rx),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1076  		.pcs            = sm8150_ufsphy_hs_g4_pcs,
8abe9792d1ff7e Nitin Rawat         2023-09-19 @1077  		.pcs_num        = ARRAY_SIZE(sm8150_ufsphy_hs_g4_pcs),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1078  	},
8abe9792d1ff7e Nitin Rawat         2023-09-19  1079  	.clk_list               = sm8450_ufs_phy_clk_l,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1080  	.num_clks               = ARRAY_SIZE(sm8450_ufs_phy_clk_l),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1081  	.vreg_list              = qmp_phy_vreg_l,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1082  	.num_vregs              = ARRAY_SIZE(qmp_phy_vreg_l),
8abe9792d1ff7e Nitin Rawat         2023-09-19  1083  	.regs                   = ufsphy_v4_regs_layout,
8abe9792d1ff7e Nitin Rawat         2023-09-19  1084  };
8abe9792d1ff7e Nitin Rawat         2023-09-19  1085  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

           reply	other threads:[~2023-12-25 22:20 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <1703472701-34197-3-git-send-email-quic_cang@quicinc.com>]

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=202312260623.FFykk4qF-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=abel.vesa@linaro.org \
    --cc=agross@kernel.org \
    --cc=andersson@kernel.org \
    --cc=bvanassche@acm.org \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=kishon@kernel.org \
    --cc=konrad.dybcio@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-phy@lists.infradead.org \
    --cc=mani@kernel.org \
    --cc=neil.armstrong@linaro.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=quic_cang@quicinc.com \
    --cc=vkoul@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;
as well as URLs for NNTP newsgroup(s).