diff options
author | Florian Fainelli <f.fainelli@gmail.com> | 2015-07-22 17:28:23 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-07-26 17:21:56 -0700 |
commit | 7d5d3075b5910cff7b33a68d3b065566fece4ba1 (patch) | |
tree | 953ed5ae4110cf53d229e139eb86dc591571533b /drivers/net/ethernet/broadcom/genet/bcmmii.c | |
parent | b3e6b82a0099dfef038e40c630a554ed1e402504 (diff) |
net: bcmgenet: Remove checks on clock handles
Instead of multiplying the number of checks for IS_ERR(priv->clk),
simply NULLify the 'struct clk' pointer which is something the Linux
common clock framework perfectly deals with and does early return for
each and every single clk_* API functions.
Having every single function check for !IS_ERR(priv->clk) is both
redundant and error prone, as it turns out, we were doing it for the
main GENET clock: priv->clk, but not for the Wake-on-LAN or EEE clock,
so let's just be consistent here.
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Acked-by: Petri Gynther <pgynther@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom/genet/bcmmii.c')
0 files changed, 0 insertions, 0 deletions