Re: [PATCH RFC 2/3] pinctrl: Add driver support for Amlogic SoCs

[kernel test robot <lkp@intel.com>]

Hi Xianwei,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build errors:

[auto build test ERROR on 171aa289a6fe65faffeb92a1fda283c055435a62]

url:    https://github.com/intel-lab-lkp/linux/commits/Xianwei-Zhao-via-B4-Relay/dt-bindings-pinctrl-Add-support-for-Amlogic-SoCs/20241211-144931
base:   171aa289a6fe65faffeb92a1fda283c055435a62
patch link:    https://lore.kernel.org/r/20241211-amlogic-pinctrl-v1-2-410727335119%40amlogic.com
patch subject: [PATCH RFC 2/3] pinctrl: Add driver support for Amlogic SoCs
config: i386-buildonly-randconfig-004-20241212 (https://download.01.org/0day-ci/archive/20241212/202412120425.1OdD2sa1-lkp@intel.com/config)
compiler: clang version 19.1.3 (https://github.com/llvm/llvm-project ab51eccf88f5321e7c60591c5546b254b6afab99)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241212/202412120425.1OdD2sa1-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/202412120425.1OdD2sa1-lkp@intel.com/

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

>> drivers/pinctrl/pinctrl-amlogic.c:38:2: error: expected identifier
      38 |         REG_OUT,
         |         ^
   arch/x86/include/asm/arch_hweight.h:12:17: note: expanded from macro 'REG_OUT'
      12 | #define REG_OUT "a"
         |                 ^
   drivers/pinctrl/pinctrl-amlogic.c:39:2: error: expected identifier
      39 |         REG_IN,
         |         ^
   arch/x86/include/asm/arch_hweight.h:11:16: note: expanded from macro 'REG_IN'
      11 | #define REG_IN "a"
         |                ^
>> drivers/pinctrl/pinctrl-amlogic.c:45:14: warning: excess elements in array initializer [-Wexcess-initializers]
      45 |         1, 1, 1, 1, 1, 2
         |                     ^
   drivers/pinctrl/pinctrl-amlogic.c:49:14: warning: excess elements in array initializer [-Wexcess-initializers]
      49 |         3, 4, 2, 1, 0, 7
         |                     ^
>> drivers/pinctrl/pinctrl-amlogic.c:311:45: error: incompatible pointer to integer conversion passing 'char[2]' to parameter of type 'unsigned int' [-Wint-conversion]
     311 |         return aml_pinconf_get_gpio_bit(info, pin, REG_OUT);
         |                                                    ^~~~~~~
   arch/x86/include/asm/arch_hweight.h:12:17: note: expanded from macro 'REG_OUT'
      12 | #define REG_OUT "a"
         |                 ^~~
   drivers/pinctrl/pinctrl-amlogic.c:284:22: note: passing argument to parameter 'reg_type' here
     284 |                                     unsigned int reg_type)
         |                                                  ^
   drivers/pinctrl/pinctrl-amlogic.c:446:45: error: incompatible pointer to integer conversion passing 'char[2]' to parameter of type 'unsigned int' [-Wint-conversion]
     446 |         return aml_pinconf_set_gpio_bit(info, pin, REG_OUT, high);
         |                                                    ^~~~~~~
   arch/x86/include/asm/arch_hweight.h:12:17: note: expanded from macro 'REG_OUT'
      12 | #define REG_OUT "a"
         |                 ^~~
   drivers/pinctrl/pinctrl-amlogic.c:425:22: note: passing argument to parameter 'reg_type' here
     425 |                                     unsigned int reg_type,
         |                                                  ^
>> drivers/pinctrl/pinctrl-amlogic.c:936:34: error: incompatible pointer to integer conversion passing 'char[2]' to parameter of type 'enum aml_reg_type' [-Wint-conversion]
     936 |         aml_gpio_calc_reg_and_bit(bank, REG_OUT, gpio, &reg, &bit);
         |                                         ^~~~~~~
   arch/x86/include/asm/arch_hweight.h:12:17: note: expanded from macro 'REG_OUT'
      12 | #define REG_OUT "a"
         |                 ^~~
   drivers/pinctrl/pinctrl-amlogic.c:884:28: note: passing argument to parameter 'reg_type' here
     884 |                                             enum aml_reg_type reg_type,
         |                                                               ^
   drivers/pinctrl/pinctrl-amlogic.c:948:34: error: incompatible pointer to integer conversion passing 'char[2]' to parameter of type 'enum aml_reg_type' [-Wint-conversion]
     948 |         aml_gpio_calc_reg_and_bit(bank, REG_OUT, gpio, &reg, &bit);
         |                                         ^~~~~~~
   arch/x86/include/asm/arch_hweight.h:12:17: note: expanded from macro 'REG_OUT'
      12 | #define REG_OUT "a"
         |                 ^~~
   drivers/pinctrl/pinctrl-amlogic.c:884:28: note: passing argument to parameter 'reg_type' here
     884 |                                             enum aml_reg_type reg_type,
         |                                                               ^
   drivers/pinctrl/pinctrl-amlogic.c:960:34: error: incompatible pointer to integer conversion passing 'char[2]' to parameter of type 'enum aml_reg_type' [-Wint-conversion]
     960 |         aml_gpio_calc_reg_and_bit(bank, REG_IN, gpio, &reg, &bit);
         |                                         ^~~~~~
   arch/x86/include/asm/arch_hweight.h:11:16: note: expanded from macro 'REG_IN'
      11 | #define REG_IN "a"
         |                ^~~
   drivers/pinctrl/pinctrl-amlogic.c:884:28: note: passing argument to parameter 'reg_type' here
     884 |                                             enum aml_reg_type reg_type,
         |                                                               ^
   2 warnings and 7 errors generated.


vim +38 drivers/pinctrl/pinctrl-amlogic.c

    28	
    29	#define gpio_chip_to_bank(chip) \
    30			container_of(chip, struct aml_gpio_bank, gpio_chip)
    31	/**
    32	 * enum aml_reg_type - type of registers encoded in @aml_reg_desc
    33	 */
    34	enum aml_reg_type {
    35		REG_PULLEN,
    36		REG_PULL,
    37		REG_DIR,
  > 38		REG_OUT,
    39		REG_IN,
    40		REG_DS,
    41		NUM_REG,
    42	};
    43	
    44	static const unsigned int aml_bit_strides[NUM_REG] = {
  > 45		1, 1, 1, 1, 1, 2
    46	};
    47	

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