]> git.dujemihanovic.xyz Git - u-boot.git/commit
net: phy: micrel: Fix error handling
authorMarek Vasut <marex@denx.de>
Mon, 14 Nov 2016 14:08:42 +0000 (15:08 +0100)
committerJoe Hershberger <joe.hershberger@ni.com>
Thu, 8 Dec 2016 16:36:21 +0000 (10:36 -0600)
commit75c056d70e404e784099204f2488d7583de9fdd9
tree0e9b63bbe8b97e60ce5af77166be0e8b9d9a5026
parent388019f1e2166638453bc4e0cc5d138c2a19e0c9
net: phy: micrel: Fix error handling

Fix the following error, the $ret variable handling must
be part of the loop, while due to the missing parenthesis
it was not.

drivers/net/phy/micrel.c: In function ‘ksz9021_of_config’:
drivers/net/phy/micrel.c:303:2: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
  for (i = 0; i < ARRAY_SIZE(ofcfg); i++)
  ^~~
drivers/net/phy/micrel.c:305:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’
   if (ret)
   ^~
drivers/net/phy/micrel.c: In function ‘ksz9031_of_config’:
drivers/net/phy/micrel.c:411:2: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
  for (i = 0; i < ARRAY_SIZE(ofcfg); i++)
  ^~~
drivers/net/phy/micrel.c:413:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’
   if (ret)
   ^~

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Joe Hershberger <joe.hershberger@ni.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
drivers/net/phy/micrel.c