lkml.org 
[lkml]   [2018]   [Jul]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 24/32] staging: gasket: apex_clock_gating simplify logic, reduce indentation
    Date
    From: Todd Poynor <toddpoynor@google.com>

    Collapse together two checks and return immediately, avoid conditional
    indentation for most of function code.

    Reported-by: Guenter Roeck <groeck@chromium.org>
    Signed-off-by: Simon Que <sque@chromium.org>
    Signed-off-by: Todd Poynor <toddpoynor@google.com>
    ---
    drivers/staging/gasket/apex_driver.c | 43 +++++++++++++---------------
    1 file changed, 20 insertions(+), 23 deletions(-)

    diff --git a/drivers/staging/gasket/apex_driver.c b/drivers/staging/gasket/apex_driver.c
    index c91c5aff5ab9c..6953b3b35ab50 100644
    --- a/drivers/staging/gasket/apex_driver.c
    +++ b/drivers/staging/gasket/apex_driver.c
    @@ -660,33 +660,30 @@ static long apex_clock_gating(struct gasket_dev *gasket_dev, void __user *arg)
    {
    struct apex_gate_clock_ioctl ibuf;

    - if (bypass_top_level)
    + if (bypass_top_level || !allow_sw_clock_gating)
    return 0;

    - if (allow_sw_clock_gating) {
    - if (copy_from_user(&ibuf, arg, sizeof(ibuf)))
    - return -EFAULT;
    + if (copy_from_user(&ibuf, arg, sizeof(ibuf)))
    + return -EFAULT;

    - gasket_log_error(
    - gasket_dev, "%s %llu", __func__, ibuf.enable);
    + gasket_log_error(gasket_dev, "%s %llu", __func__, ibuf.enable);

    - if (ibuf.enable) {
    - /* Quiesce AXI, gate GCB clock. */
    - gasket_read_modify_write_32(
    - gasket_dev, APEX_BAR_INDEX,
    - APEX_BAR2_REG_AXI_QUIESCE, 0x1, 1, 16);
    - gasket_read_modify_write_32(
    - gasket_dev, APEX_BAR_INDEX,
    - APEX_BAR2_REG_GCB_CLOCK_GATE, 0x1, 2, 18);
    - } else {
    - /* Un-gate GCB clock, un-quiesce AXI. */
    - gasket_read_modify_write_32(
    - gasket_dev, APEX_BAR_INDEX,
    - APEX_BAR2_REG_GCB_CLOCK_GATE, 0x0, 2, 18);
    - gasket_read_modify_write_32(
    - gasket_dev, APEX_BAR_INDEX,
    - APEX_BAR2_REG_AXI_QUIESCE, 0x0, 1, 16);
    - }
    + if (ibuf.enable) {
    + /* Quiesce AXI, gate GCB clock. */
    + gasket_read_modify_write_32(
    + gasket_dev, APEX_BAR_INDEX,
    + APEX_BAR2_REG_AXI_QUIESCE, 0x1, 1, 16);
    + gasket_read_modify_write_32(
    + gasket_dev, APEX_BAR_INDEX,
    + APEX_BAR2_REG_GCB_CLOCK_GATE, 0x1, 2, 18);
    + } else {
    + /* Un-gate GCB clock, un-quiesce AXI. */
    + gasket_read_modify_write_32(
    + gasket_dev, APEX_BAR_INDEX,
    + APEX_BAR2_REG_GCB_CLOCK_GATE, 0x0, 2, 18);
    + gasket_read_modify_write_32(
    + gasket_dev, APEX_BAR_INDEX,
    + APEX_BAR2_REG_AXI_QUIESCE, 0x0, 1, 16);
    }
    return 0;
    }
    --
    2.18.0.203.gfac676dfb9-goog
    \
     
     \ /
      Last update: 2018-07-17 23:05    [W:4.019 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site