Re: [PATCH v9 2/2] phy: qualcomm: phy-qcom-qmp-ufs: Add High Speed Gear 5 support for SM8550

From: kernel test robot
Date: Mon Dec 25 2023 - 17:20:19 EST


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@xxxxxxxxx/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@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202312260623.FFykk4qF-lkp@xxxxxxxxx/

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