• Vladimir Kondratiev's avatar
    wil6210: fix PTR_ERR() usage after initialization to constant · 867fa0d4
    Vladimir Kondratiev authored
    Reported by coccinelle:
    
    tree:   git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-next.git master
    head:   6a5d088a
    commit: b7cde470 [18/80] wil6210: convert debugfs to the table mode
    
    coccinelle warnings: (new ones prefixed by >>)
    
    >> drivers/net/wireless/ath/wil6210/debugfs.c:327:17-24: ERROR: PTR_ERR applied after initialization to constant on line 304
    
    vim +327 drivers/net/wireless/ath/wil6210/debugfs.c
    
       298                                          struct dentry *dbg, void *base,
       299                                          const struct dbg_off * const tbl)
       300  {
       301          int i;
       302
       303          for (i = 0; tbl[i].name; i++) {
    
     > 304                  struct dentry *f = NULL;
       305
       306                  switch (tbl[i].type) {
       307                  case doff_u32:
       308                          f = debugfs_create_u32(tbl[i].name, tbl[i].mode, dbg,
       309                                                 base + tbl[i].off);
       310                          break;
       311                  case doff_x32:
       312                          f = debugfs_create_x32(tbl[i].name, tbl[i].mode, dbg,
       313                                                 base + tbl[i].off);
       314                          break;
       315                  case doff_ulong:
       316                          f = wil_debugfs_create_ulong(tbl[i].name, tbl[i].mode,
       317                                                       dbg, base + tbl[i].off);
       318                          break;
       319                  case doff_io32:
       320                          f = wil_debugfs_create_iomem_x32(tbl[i].name,
       321                                                           tbl[i].mode, dbg,
       322                                                           base + tbl[i].off);
       323                          break;
       324                  }
       325                  if (IS_ERR_OR_NULL(f))
       326                          wil_err(wil, "Create file \"%s\": err %ld\n",
    
     > 327                                  tbl[i].name, PTR_ERR(f));
       328          }
       329  }
       330
    Signed-off-by: default avatarVladimir Kondratiev <qca_vkondrat@qca.qualcomm.com>
    Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
    867fa0d4
debugfs.c 30.7 KB