Re: [PATCH v6 2/2] phy: mediatek: Add PCIe PHY driver

From: kernel test robot
Date: Fri Apr 22 2022 - 03:59:56 EST


Hi Jianjun,

I love your patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on linus/master v5.18-rc3 next-20220421]
[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/intel-lab-lkp/linux/commits/Jianjun-Wang/phy-mediatek-Add-PCIe-PHY-driver/20220422-121407
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arc-allyesconfig (https://download.01.org/0day-ci/archive/20220422/202204221513.SZZHW8dq-lkp@xxxxxxxxx/config)
compiler: arceb-elf-gcc (GCC) 11.2.0
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
# https://github.com/intel-lab-lkp/linux/commit/c2ea770ff29995df64bc82ad12ece5b671415a62
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jianjun-Wang/phy-mediatek-Add-PCIe-PHY-driver/20220422-121407
git checkout c2ea770ff29995df64bc82ad12ece5b671415a62
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash

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

All errors (new ones prefixed by >>):

drivers/phy/mediatek/phy-mtk-pcie.c: In function 'mtk_pcie_efuse_set_lane':
>> drivers/phy/mediatek/phy-mtk-pcie.c:92:29: error: implicit declaration of function 'FIELD_PREP' [-Werror=implicit-function-declaration]
92 | FIELD_PREP(EFUSE_LN_TX_PMOS_SEL, data->tx_pmos));
| ^~~~~~~~~~
cc1: some warnings being treated as errors


vim +/FIELD_PREP +92 drivers/phy/mediatek/phy-mtk-pcie.c

78
79 static void mtk_pcie_efuse_set_lane(struct mtk_pcie_phy *pcie_phy,
80 unsigned int lane)
81 {
82 struct mtk_pcie_lane_efuse *data = &pcie_phy->efuse[lane];
83 void __iomem *addr;
84
85 if (!data->lane_efuse_supported)
86 return;
87
88 addr = pcie_phy->sif_base + PEXTP_ANA_LN0_TRX_REG +
89 lane * PEXTP_ANA_LANE_OFFSET;
90
91 mtk_phy_update_bits(addr + PEXTP_ANA_TX_REG, EFUSE_LN_TX_PMOS_SEL,
> 92 FIELD_PREP(EFUSE_LN_TX_PMOS_SEL, data->tx_pmos));
93
94 mtk_phy_update_bits(addr + PEXTP_ANA_TX_REG, EFUSE_LN_TX_NMOS_SEL,
95 FIELD_PREP(EFUSE_LN_TX_NMOS_SEL, data->tx_nmos));
96
97 mtk_phy_update_bits(addr + PEXTP_ANA_RX_REG, EFUSE_LN_RX_SEL,
98 FIELD_PREP(EFUSE_LN_RX_SEL, data->rx_data));
99 }
100

--
0-DAY CI Kernel Test Service
https://01.org/lkp