Commit 9116223b authored by Todd Poynor's avatar Todd Poynor Committed by Greg Kroah-Hartman

staging: gasket: apex_clock_gating simplify logic, reduce indentation

Collapse together two checks and return immediately, avoid conditional
indentation for most of function code.
Reported-by: default avatarGuenter Roeck <groeck@chromium.org>
Signed-off-by: default avatarSimon Que <sque@chromium.org>
Signed-off-by: default avatarTodd Poynor <toddpoynor@google.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e80d8afc
...@@ -666,15 +666,13 @@ static long apex_clock_gating(struct gasket_dev *gasket_dev, ulong arg) ...@@ -666,15 +666,13 @@ static long apex_clock_gating(struct gasket_dev *gasket_dev, ulong arg)
{ {
struct apex_gate_clock_ioctl ibuf; struct apex_gate_clock_ioctl ibuf;
if (bypass_top_level) if (bypass_top_level || !allow_sw_clock_gating)
return 0; return 0;
if (allow_sw_clock_gating) {
if (copy_from_user(&ibuf, (void __user *)arg, sizeof(ibuf))) if (copy_from_user(&ibuf, (void __user *)arg, sizeof(ibuf)))
return -EFAULT; return -EFAULT;
gasket_log_error( gasket_log_error(gasket_dev, "%s %llu", __func__, ibuf.enable);
gasket_dev, "%s %llu", __func__, ibuf.enable);
if (ibuf.enable) { if (ibuf.enable) {
/* Quiesce AXI, gate GCB clock. */ /* Quiesce AXI, gate GCB clock. */
...@@ -693,7 +691,6 @@ static long apex_clock_gating(struct gasket_dev *gasket_dev, ulong arg) ...@@ -693,7 +691,6 @@ static long apex_clock_gating(struct gasket_dev *gasket_dev, ulong arg)
gasket_dev, APEX_BAR_INDEX, gasket_dev, APEX_BAR_INDEX,
APEX_BAR2_REG_AXI_QUIESCE, 0x0, 1, 16); APEX_BAR2_REG_AXI_QUIESCE, 0x0, 1, 16);
} }
}
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment