diff options
| author | Dave Airlie <airlied@redhat.com> | 2017-09-29 15:44:54 +1000 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2017-09-29 13:02:21 -0400 |
| commit | b08c3ca4e90d03b3a120f4e60dba4b2b5e087433 (patch) | |
| tree | a9ccc554e707c30a2e6ccaa6073d0d1b761bca89 /drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c | |
| parent | 4179cd81376112d627216580b09b803eb25e1635 (diff) | |
amdgpu/dc: remove pointless returns in the i2caux constructor paths. (v2)
There was lots of return true, and error checking that was never used
in these paths.
Just remove it all.
v2: I missed one return true.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c')
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c b/drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c index 144f51dc4523..70e20bd47ce4 100644 --- a/drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c +++ b/drivers/gpu/drm/amd/display/dc/i2caux/i2c_engine.c @@ -103,15 +103,12 @@ void dal_i2c_engine_process_channel_reply( } -bool dal_i2c_engine_construct( +void dal_i2c_engine_construct( struct i2c_engine *engine, struct dc_context *ctx) { - if (!dal_i2caux_construct_engine(&engine->base, ctx)) - return false; - + dal_i2caux_construct_engine(&engine->base, ctx); engine->timeout_delay = 0; - return true; } void dal_i2c_engine_destruct( |