Hi Zhu, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on v4.17-rc5] [also build test WARNING on next-20180516] [cannot apply to input/next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Mark-Jonas/Input-add-bu21029-touch-driver/20180517-133332 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> drivers/input/touchscreen/bu21029_ts.c:289:13: sparse: cast to restricted __be16 >> drivers/input/touchscreen/bu21029_ts.c:289:13: sparse: cast to restricted __be16 >> drivers/input/touchscreen/bu21029_ts.c:289:13: sparse: cast to restricted __be16 >> drivers/input/touchscreen/bu21029_ts.c:289:13: sparse: cast to restricted __be16 vim +289 drivers/input/touchscreen/bu21029_ts.c 258 259 static int bu21029_start_chip(struct input_dev *dev) 260 { 261 struct bu21029_ts_data *bu21029 = input_get_drvdata(dev); 262 struct i2c_client *i2c = bu21029->client; 263 struct { 264 u8 reg; 265 u8 value; 266 } init_table[] = { 267 {BU21029_CFR0_REG, CFR0_VALUE}, 268 {BU21029_CFR1_REG, CFR1_VALUE}, 269 {BU21029_CFR2_REG, CFR2_VALUE}, 270 {BU21029_CFR3_REG, CFR3_VALUE}, 271 {BU21029_LDO_REG, LDO_VALUE} 272 }; 273 int error, i; 274 u16 hwid; 275 276 /* take chip out of reset */ 277 gpiod_set_value_cansleep(bu21029->reset_gpios, 0); 278 mdelay(START_DELAY_MS); 279 280 error = i2c_smbus_read_i2c_block_data(i2c, 281 BU21029_HWID_REG, 282 2, 283 (u8 *)&hwid); 284 if (error < 0) { 285 dev_err(&i2c->dev, "failed to read HW ID\n"); 286 goto out; 287 } 288 > 289 if (be16_to_cpu(hwid) != SUPPORTED_HWID) { 290 dev_err(&i2c->dev, "unsupported HW ID 0x%x\n", hwid); 291 error = -ENODEV; 292 goto out; 293 } 294 295 for (i = 0; i < ARRAY_SIZE(init_table); ++i) { 296 error = i2c_smbus_write_byte_data(i2c, 297 init_table[i].reg, 298 init_table[i].value); 299 if (error < 0) { 300 dev_err(&i2c->dev, 301 "failed to write 0x%x to register 0x%x\n", 302 init_table[i].value, 303 init_table[i].reg); 304 goto out; 305 } 306 } 307 308 error = i2c_smbus_write_byte(i2c, BU21029_AUTOSCAN); 309 if (error < 0) { 310 dev_err(&i2c->dev, "failed to start autoscan\n"); 311 goto out; 312 } 313 314 enable_irq(bu21029->client->irq); 315 return 0; 316 317 out: 318 bu21029_stop_chip(dev); 319 return error; 320 } 321 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
