tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 9ff9b0d392ea08090cd1780fb196f36dbb586529 commit: 6f413b559f86a2894188e082e389ff95ee428345 usb: typec: tcpci_maxim: Chip level TCPC driver config: nios2-randconfig-m031-20201016 (attached as .config) compiler: nios2-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>
Reported-by: Dan Carpenter <dan.carpen...@oracle.com>
New smatch warnings:
drivers/usb/typec/tcpm/tcpci_maxim.c:324 max_tcpci_irq() error: uninitialized
symbol 'irq_return'.
drivers/usb/typec/tcpm/tcpci_maxim.c:407 max_tcpci_probe() warn: passing zero
to 'PTR_ERR'
Old smatch warnings:
drivers/usb/typec/tcpm/tcpci_maxim.c:408 max_tcpci_probe() warn: passing zero
to 'PTR_ERR'
vim +/irq_return +324 drivers/usb/typec/tcpm/tcpci_maxim.c
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 301 static irqreturn_t
max_tcpci_irq(int irq, void *dev_id)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 302 {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 303 struct
max_tcpci_chip *chip = dev_id;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 304 u16 status;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 305 irqreturn_t
irq_return;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 306 int ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 307
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 308 if (!chip->port)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 309 return
IRQ_HANDLED;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 310
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 311 ret =
max_tcpci_read16(chip, TCPC_ALERT, &status);
Can status be zero? If it is then then "irq_return" is uninitialized.
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 312 if (ret < 0) {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 313
dev_err(chip->dev, "ALERT read failed\n");
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 314 return
ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 315 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 316 while (status) {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 317
irq_return = _max_tcpci_irq(chip, status);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 318 /* Do
not return if the ALERT is already set. */
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 319 ret =
max_tcpci_read16(chip, TCPC_ALERT, &status);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 320 if (ret
< 0)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 321
break;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 322 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 323
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 @324 return
irq_return;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 325 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 326
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 327 static irqreturn_t
max_tcpci_isr(int irq, void *dev_id)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 328 {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 329 struct
max_tcpci_chip *chip = dev_id;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 330
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 331
pm_wakeup_event(chip->dev, PD_ACTIVITY_TIMEOUT_MS);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 332
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 333 if (!chip->port)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 334 return
IRQ_HANDLED;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 335
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 336 return
IRQ_WAKE_THREAD;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 337 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 338
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 339 static int
max_tcpci_init_alert(struct max_tcpci_chip *chip, struct i2c_client *client)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 340 {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 341 int ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 342
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 343 ret =
devm_request_threaded_irq(chip->dev, client->irq, max_tcpci_isr, max_tcpci_irq,
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 344
(IRQF_TRIGGER_LOW | IRQF_ONESHOT), dev_name(chip->dev),
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 345
chip);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 346
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 347 if (ret < 0)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 348 return
ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 349
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 350
enable_irq_wake(client->irq);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 351 return 0;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 352 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 353
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 354 static int
max_tcpci_start_toggling(struct tcpci *tcpci, struct tcpci_data *tdata,
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 355
enum typec_cc_status cc)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 356 {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 357 struct
max_tcpci_chip *chip = tdata_to_max_tcpci(tdata);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 358
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 359
max_tcpci_init_regs(chip);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 360
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 361 return 0;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 362 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 363
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 364 static int
tcpci_init(struct tcpci *tcpci, struct tcpci_data *data)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 365 {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 366 /*
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 367 * Generic
TCPCI overwrites the regs once this driver initializes
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 368 * them.
Prevent this by returning -1.
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 369 */
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 370 return -1;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 371 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 372
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 373 static int
max_tcpci_probe(struct i2c_client *client, const struct i2c_device_id *i2c_id)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 374 {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 375 int ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 376 struct
max_tcpci_chip *chip;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 377 u8 power_status;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 378
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 379 chip =
devm_kzalloc(&client->dev, sizeof(*chip), GFP_KERNEL);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 380 if (!chip)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 381 return
-ENOMEM;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 382
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 383 chip->client =
client;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 384
chip->data.regmap = devm_regmap_init_i2c(client, &max_tcpci_regmap_config);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 385 if
(IS_ERR(chip->data.regmap)) {
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 386
dev_err(&client->dev, "Regmap init failed\n");
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 387 return
PTR_ERR(chip->data.regmap);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 388 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 389
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 390 chip->dev =
&client->dev;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 391
i2c_set_clientdata(client, chip);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 392
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 393 ret =
max_tcpci_read8(chip, TCPC_POWER_STATUS, &power_status);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 394 if (ret < 0)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 395 return
ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 396
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 397 /* Chip level
tcpci callbacks */
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 398
chip->data.set_vbus = max_tcpci_set_vbus;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 399
chip->data.start_drp_toggling = max_tcpci_start_toggling;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 400
chip->data.TX_BUF_BYTE_x_hidden = true;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 401 chip->data.init
= tcpci_init;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 402
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 403
max_tcpci_init_regs(chip);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 404 chip->tcpci =
tcpci_register_port(chip->dev, &chip->data);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 405 if
(IS_ERR_OR_NULL(chip->tcpci)) {
When a function returns both NULL and error pointers that means that
it is an optional feature and it has been deliberately disabled. In
that case we cannot return a pointer to the feature (it is turned off)
but it's also not an error so we don't want to return an error pointer
so we return NULL.
So probe should handle the NULL return and continue.
But actually in this case tcpci_register_port() is not optional and
never returns NULL. Just replace IS_ERR_OR_NULL() with IS_ERR().
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 406
dev_err(&client->dev, "TCPCI port registration failed");
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 @407 ret =
PTR_ERR(chip->tcpci);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 408 return
PTR_ERR(chip->tcpci);
returning NULL would mean returning success here.
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 409 }
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 410 chip->port =
tcpci_get_tcpm_port(chip->tcpci);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 411 ret =
max_tcpci_init_alert(chip, client);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 412 if (ret < 0)
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 413 goto
unreg_port;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 414
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 415
device_init_wakeup(chip->dev, true);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 416 return 0;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 417
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 418 unreg_port:
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 419
tcpci_unregister_port(chip->tcpci);
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 420
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 421 return ret;
6f413b559f86a28 Badhri Jagan Sridharan 2020-10-07 422 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
