8230 e1000api: misleading-indentation

Review Request #514 - Created May 15, 2017 and submitted

Information
Toomas Soome
illumos-gate
8230
065fd82...
Reviewers
general
../../common/io/e1000api/e1000_ich8lan.c: In function 'e1000_disable_ulp_lpt_lp':
../../common/io/e1000api/e1000_ich8lan.c:1499:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
  if (ret_val)
  ^~
../../common/io/e1000api/e1000_ich8lan.c:1501:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
   phy_reg &= ~(I218_ULP_CONFIG1_IND |
   ^~~~~~~
../../common/io/e1000api/e1000_ich8lan.c: In function 'e1000_check_for_copper_link_ich8lan':
../../common/io/e1000api/e1000_ich8lan.c:1558:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
  if (!mac->get_link_status)
  ^~
../../common/io/e1000api/e1000_ich8lan.c:1565:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
   ret_val = e1000_phy_has_link_generic(hw, 1, 0, &link);
   ^~~~~~~
cc1: all warnings being treated as errors

  
Yuri Pankov
Yuri Pankov
Robert Mustacchi
Toomas Soome
Review request changed

Status: Closed (submitted)

Loading...