Hi, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on tmlind-omap/for-next] [also build test WARNING on next-20220504] [cannot apply to dtor-input/next robh/for-next v5.18-rc5] [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/Gireesh-Hiremath-in-bosch-com/ARM-dts-am335x-Guardian-switch-to-AM33XX_PADCONF-pinmux-macro/20220504-185623 base: https://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap.git for-next config: x86_64-allmodconfig (https://download.01.org/0day-ci/archive/20220505/202205050513.PAcNKUP3-lkp@xxxxxxxxx/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 363b3a645a1e30011cc8da624f13dac5fd915628) 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/8dae1bf10aa8e74feb1bca5eb708e98e9492a587 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Gireesh-Hiremath-in-bosch-com/ARM-dts-am335x-Guardian-switch-to-AM33XX_PADCONF-pinmux-macro/20220504-185623 git checkout 8dae1bf10aa8e74feb1bca5eb708e98e9492a587 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/input/keyboard/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@xxxxxxxxx> All warnings (new ones prefixed by >>): >> drivers/input/keyboard/mt_matrix_keypad.c:51:6: warning: no previous prototype for function 'init_phase' [-Wmissing-prototypes] void init_phase(struct mt_matrix_keypad_platform_data *pdata) ^ drivers/input/keyboard/mt_matrix_keypad.c:51:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void init_phase(struct mt_matrix_keypad_platform_data *pdata) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:58:6: warning: no previous prototype for function 'button_init' [-Wmissing-prototypes] void button_init(struct button *btn, bool btn_hw_state, int key) ^ drivers/input/keyboard/mt_matrix_keypad.c:58:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void button_init(struct button *btn, bool btn_hw_state, int key) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:70:1: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers] const struct button_states get_button_state(struct button *btn) ^~~~~~ >> drivers/input/keyboard/mt_matrix_keypad.c:70:28: warning: no previous prototype for function 'get_button_state' [-Wmissing-prototypes] const struct button_states get_button_state(struct button *btn) ^ drivers/input/keyboard/mt_matrix_keypad.c:70:7: note: declare 'static' if the function is not intended to be used outside of this translation unit const struct button_states get_button_state(struct button *btn) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:75:17: warning: no previous prototype for function 'get_and_clear_events' [-Wmissing-prototypes] union typeEvent get_and_clear_events(struct button *btn) ^ drivers/input/keyboard/mt_matrix_keypad.c:75:1: note: declare 'static' if the function is not intended to be used outside of this translation unit union typeEvent get_and_clear_events(struct button *btn) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:84:9: warning: no previous prototype for function 'get_btn_index' [-Wmissing-prototypes] uint8_t get_btn_index(struct mt_matrix_keypad_platform_data *pdata, int btn_key) ^ drivers/input/keyboard/mt_matrix_keypad.c:84:1: note: declare 'static' if the function is not intended to be used outside of this translation unit uint8_t get_btn_index(struct mt_matrix_keypad_platform_data *pdata, int btn_key) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:95:6: warning: no previous prototype for function 'set_btn_state_by_hw' [-Wmissing-prototypes] void set_btn_state_by_hw(struct button *btn, bool boButtonState) ^ drivers/input/keyboard/mt_matrix_keypad.c:95:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void set_btn_state_by_hw(struct button *btn, bool boButtonState) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:100:6: warning: no previous prototype for function 'check_button_changes' [-Wmissing-prototypes] bool check_button_changes(struct button *btn) ^ drivers/input/keyboard/mt_matrix_keypad.c:100:1: note: declare 'static' if the function is not intended to be used outside of this translation unit bool check_button_changes(struct button *btn) ^ static drivers/input/keyboard/mt_matrix_keypad.c:124:1: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers] const struct button_states ^~~~~~ >> drivers/input/keyboard/mt_matrix_keypad.c:125:1: warning: no previous prototype for function 'get_btn_id_state' [-Wmissing-prototypes] get_btn_id_state(const struct mt_matrix_keypad_platform_data *pdata, ^ drivers/input/keyboard/mt_matrix_keypad.c:124:7: note: declare 'static' if the function is not intended to be used outside of this translation unit const struct button_states ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:135:1: warning: no previous prototype for function 'get_and_clear_btn_events' [-Wmissing-prototypes] get_and_clear_btn_events(const struct mt_matrix_keypad_platform_data *pdata, ^ drivers/input/keyboard/mt_matrix_keypad.c:134:1: note: declare 'static' if the function is not intended to be used outside of this translation unit union typeEvent ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:144:6: warning: no previous prototype for function 'button_hdl_init' [-Wmissing-prototypes] void button_hdl_init(struct mt_matrix_keypad_platform_data *pdata) ^ drivers/input/keyboard/mt_matrix_keypad.c:144:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void button_hdl_init(struct mt_matrix_keypad_platform_data *pdata) ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:171:6: warning: no previous prototype for function 'on_button_event' [-Wmissing-prototypes] bool on_button_event(const struct mt_matrix_keypad_platform_data *pdata, ^ drivers/input/keyboard/mt_matrix_keypad.c:171:1: note: declare 'static' if the function is not intended to be used outside of this translation unit bool on_button_event(const struct mt_matrix_keypad_platform_data *pdata, ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:200:7: warning: variable 'any_btn_served' set but not used [-Wunused-but-set-variable] bool any_btn_served = false; ^ >> drivers/input/keyboard/mt_matrix_keypad.c:196:6: warning: no previous prototype for function 'process_button_events' [-Wmissing-prototypes] void process_button_events(const struct mt_matrix_keypad_platform_data *pdata, ^ drivers/input/keyboard/mt_matrix_keypad.c:196:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void process_button_events(const struct mt_matrix_keypad_platform_data *pdata, ^ static >> drivers/input/keyboard/mt_matrix_keypad.c:219:6: warning: no previous prototype for function 'update_buttons' [-Wmissing-prototypes] void update_buttons(struct mt_matrix_keypad_platform_data *pdata, ^ drivers/input/keyboard/mt_matrix_keypad.c:219:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void update_buttons(struct mt_matrix_keypad_platform_data *pdata, ^ static 16 warnings generated. vim +/init_phase +51 drivers/input/keyboard/mt_matrix_keypad.c 43 44 static bool 45 get_gpio_line_value(const struct mt_matrix_keypad_platform_data *pdata, 46 int row); 47 static void 48 activate_line_driving(const struct mt_matrix_keypad_platform_data *pdata, 49 int line, bool on); 50 > 51 void init_phase(struct mt_matrix_keypad_platform_data *pdata) 52 { 53 pdata->phase_state.phase_prepare = -1; 54 pdata->phase_state.phase_start = 0; 55 pdata->phase_state.phase_update_button = pdata->num_line_gpios; 56 } 57 > 58 void button_init(struct button *btn, bool btn_hw_state, int key) 59 { 60 btn->state.boPrevious = btn_hw_state; 61 btn->state.boCurrentStateOfHw = btn_hw_state; 62 btn->state.boCurrentStateOfSw = false; 63 btn->state.boCurrent = btn_hw_state; 64 btn->state.boEnabled = true; 65 btn->state.boStateAtInit = btn_hw_state; 66 btn->event.ui8Register = 0; 67 btn->key = key; 68 } 69 > 70 const struct button_states get_button_state(struct button *btn) 71 { 72 return btn->state; 73 } 74 > 75 union typeEvent get_and_clear_events(struct button *btn) 76 { 77 union typeEvent beTemp = btn->event; 78 79 btn->event.ui8Register = 0; 80 81 return beTemp; 82 } 83 > 84 uint8_t get_btn_index(struct mt_matrix_keypad_platform_data *pdata, int btn_key) 85 { 86 uint8_t i; 87 88 for (i = 0; i < pdata->num_of_button; i++) { 89 if (pdata->button_array[i].key == btn_key) 90 break; 91 } 92 return i; 93 } 94 > 95 void set_btn_state_by_hw(struct button *btn, bool boButtonState) 96 { 97 btn->state.boCurrentStateOfHw = boButtonState; 98 } 99 > 100 bool check_button_changes(struct button *btn) 101 { 102 btn->state.boPrevious = btn->state.boCurrent; 103 btn->state.boCurrent = 104 btn->state.boCurrentStateOfHw || btn->state.boCurrentStateOfSw; 105 106 /* Check if Button is pressed */ 107 if ((btn->state.boPrevious == false) && 108 (btn->state.boCurrent == true)) { 109 btn->event.status.boPressed = true; 110 } 111 112 /* Check if Button is released */ 113 else if ((btn->state.boPrevious == true) && 114 (btn->state.boCurrent == false)) { 115 btn->event.status.boReleased = true; 116 } 117 118 if (btn->event.ui8Register != 0) 119 btn->event.status.boGlobalChanged = true; 120 121 return btn->event.status.boGlobalChanged; 122 } 123 124 const struct button_states > 125 get_btn_id_state(const struct mt_matrix_keypad_platform_data *pdata, 126 int btn_index) 127 { 128 if (btn_index < pdata->num_of_button) 129 return get_button_state(&pdata->button_array[btn_index]); 130 else 131 return get_button_state(&pdata->button_array[0]); 132 } 133 134 union typeEvent > 135 get_and_clear_btn_events(const struct mt_matrix_keypad_platform_data *pdata, 136 int btn_index) 137 { 138 if (btn_index < pdata->num_of_button) 139 return get_and_clear_events(&pdata->button_array[btn_index]); 140 else 141 return get_and_clear_events(&pdata->button_array[0]); 142 } 143 > 144 void button_hdl_init(struct mt_matrix_keypad_platform_data *pdata) 145 { 146 int row, col, index; 147 int i; 148 149 pdata->scan_phase = pdata->phase_state.phase_prepare; 150 pdata->intialize_buttons = true; 151 152 /* Init Button Objects, will be reinited once states are captured */ 153 i = 0; 154 for (row = 1; row < pdata->num_line_gpios; row++) 155 for (col = 0; col < row; col++) { 156 index = (row * pdata->num_line_gpios) + col; 157 if (pdata->button_matrix[index] != 158 pdata->button_matrix[0]) { 159 if (i < pdata->num_of_button) { 160 button_init( 161 &pdata->button_array[i], false, 162 pdata->button_matrix[index]); 163 i++; 164 } 165 } 166 } 167 168 pr_debug("[%s]: %s Done\n", MODULE_NAME, __func__); 169 } 170 > 171 bool on_button_event(const struct mt_matrix_keypad_platform_data *pdata, 172 int btn_index, union typeEvent btn_event, 173 struct input_dev *input_dev) 174 { 175 bool any_btn_served = true; 176 unsigned int key_code = 0; 177 int key_value = 0; 178 179 key_code = pdata->button_array[btn_index].key; 180 181 if (btn_event.status.boPressed) { 182 key_value = 1; 183 pr_debug("[%s]:%d Pressed\n", MODULE_NAME, key_code); 184 } 185 186 if (btn_event.status.boReleased) { 187 key_value = 0; 188 pr_debug("[%s]:%d Released\n", MODULE_NAME, key_code); 189 } 190 191 input_report_key(input_dev, key_code, key_value); 192 input_sync(input_dev); 193 return any_btn_served; 194 } 195 > 196 void process_button_events(const struct mt_matrix_keypad_platform_data *pdata, 197 struct input_dev *input_dev) 198 { 199 int btn_index; > 200 bool any_btn_served = false; 201 202 for (btn_index = 0; btn_index < pdata->num_of_button; btn_index++) { 203 const union typeEvent beEvent = 204 get_and_clear_btn_events(pdata, (int)btn_index); 205 206 if (beEvent.status.boGlobalChanged) { 207 const struct button_states bsState = 208 get_btn_id_state(pdata, (int)btn_index); 209 210 if (bsState.boEnabled) { 211 any_btn_served |= 212 on_button_event(pdata, (int)btn_index, 213 beEvent, input_dev); 214 } 215 } 216 } 217 } 218 > 219 void update_buttons(struct mt_matrix_keypad_platform_data *pdata, 220 struct input_dev *input_dev) 221 { 222 if (pdata->scan_phase == pdata->phase_state.phase_prepare) { 223 pdata->scan_phase = pdata->phase_state.phase_start; 224 activate_line_driving(pdata, (int)pdata->scan_phase, true); 225 } else if (pdata->scan_phase == 226 pdata->phase_state.phase_update_button) { 227 bool btn_changes_occured = false; 228 int btn_index; 229 230 if (pdata->intialize_buttons) { 231 int i; 232 233 pdata->intialize_buttons = false; 234 235 for (i = 0; i < pdata->num_of_button; i++) { 236 const bool btn_curr_hw_state = 237 get_button_state( 238 &pdata->button_array[i]) 239 .boCurrentStateOfHw; 240 button_init(&pdata->button_array[i], 241 btn_curr_hw_state, 242 pdata->button_array[i].key); 243 } 244 } 245 246 for (btn_index = 0; btn_index < pdata->num_of_button; 247 btn_index++) { 248 btn_changes_occured |= check_button_changes( 249 &pdata->button_array[btn_index]); 250 } 251 252 if (btn_changes_occured) 253 process_button_events(pdata, input_dev); 254 255 pdata->scan_phase = pdata->phase_state.phase_start; 256 } else { 257 uint8_t *btn_keylines; 258 uint8_t number_of_buttons_pressed = 0; 259 uint8_t btn_index; 260 uint8_t btn_key; 261 uint16_t index; 262 int i; 263 264 btn_keylines = kcalloc(pdata->num_line_gpios, sizeof(uint8_t), 265 GFP_KERNEL); 266 for (i = 0; i < pdata->num_line_gpios; i++) { 267 index = (pdata->scan_phase * pdata->num_line_gpios) + i; 268 btn_key = pdata->button_matrix[index]; 269 btn_keylines[i] = false; 270 271 if ((btn_key != pdata->button_matrix[0]) && 272 (get_gpio_line_value(pdata, (int)i) != false)) { 273 btn_keylines[i] = true; 274 number_of_buttons_pressed++; 275 } 276 } 277 if (number_of_buttons_pressed < 2) { 278 for (i = 0; i < pdata->num_line_gpios; i++) { 279 index = (pdata->scan_phase * 280 pdata->num_line_gpios) + 281 i; 282 btn_key = pdata->button_matrix[index]; 283 if (btn_key != pdata->button_matrix[0]) { 284 btn_index = 285 get_btn_index(pdata, btn_key); 286 set_btn_state_by_hw( 287 &pdata->button_array[btn_index], 288 btn_keylines[i]); 289 } 290 } 291 } 292 293 kfree(btn_keylines); 294 activate_line_driving(pdata, (int)pdata->scan_phase, false); 295 pdata->scan_phase++; 296 activate_line_driving( 297 pdata, (int)(pdata->scan_phase % pdata->num_line_gpios), 298 true); 299 } 300 } 301 -- 0-DAY CI Kernel Test Service https://01.org/lkp
