amdgpu/dc: make opp construct void.
This doesn't return anything except 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>
This commit is contained in:
parent
c60ae11201
commit
9cf29399f6
|
@ -540,7 +540,7 @@ static const struct opp_funcs funcs = {
|
||||||
.opp_program_bit_depth_reduction = dce110_opp_program_bit_depth_reduction
|
.opp_program_bit_depth_reduction = dce110_opp_program_bit_depth_reduction
|
||||||
};
|
};
|
||||||
|
|
||||||
bool dce110_opp_construct(struct dce110_opp *opp110,
|
void dce110_opp_construct(struct dce110_opp *opp110,
|
||||||
struct dc_context *ctx,
|
struct dc_context *ctx,
|
||||||
uint32_t inst,
|
uint32_t inst,
|
||||||
const struct dce_opp_registers *regs,
|
const struct dce_opp_registers *regs,
|
||||||
|
@ -556,8 +556,6 @@ bool dce110_opp_construct(struct dce110_opp *opp110,
|
||||||
opp110->regs = regs;
|
opp110->regs = regs;
|
||||||
opp110->opp_shift = opp_shift;
|
opp110->opp_shift = opp_shift;
|
||||||
opp110->opp_mask = opp_mask;
|
opp110->opp_mask = opp_mask;
|
||||||
|
|
||||||
return true;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
void dce110_opp_destroy(struct output_pixel_processor **opp)
|
void dce110_opp_destroy(struct output_pixel_processor **opp)
|
||||||
|
|
|
@ -272,7 +272,7 @@ struct dce110_opp {
|
||||||
const struct dce_opp_mask *opp_mask;
|
const struct dce_opp_mask *opp_mask;
|
||||||
};
|
};
|
||||||
|
|
||||||
bool dce110_opp_construct(struct dce110_opp *opp110,
|
void dce110_opp_construct(struct dce110_opp *opp110,
|
||||||
struct dc_context *ctx,
|
struct dc_context *ctx,
|
||||||
uint32_t inst,
|
uint32_t inst,
|
||||||
const struct dce_opp_registers *regs,
|
const struct dce_opp_registers *regs,
|
||||||
|
|
|
@ -547,13 +547,9 @@ struct output_pixel_processor *dce100_opp_create(
|
||||||
if (!opp)
|
if (!opp)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
if (dce110_opp_construct(opp,
|
dce110_opp_construct(opp,
|
||||||
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask))
|
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask);
|
||||||
return &opp->base;
|
return &opp->base;
|
||||||
|
|
||||||
BREAK_TO_DEBUGGER();
|
|
||||||
kfree(opp);
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
struct clock_source *dce100_clock_source_create(
|
struct clock_source *dce100_clock_source_create(
|
||||||
|
|
|
@ -586,13 +586,9 @@ static struct output_pixel_processor *dce110_opp_create(
|
||||||
if (!opp)
|
if (!opp)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
if (dce110_opp_construct(opp,
|
dce110_opp_construct(opp,
|
||||||
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask))
|
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask);
|
||||||
return &opp->base;
|
return &opp->base;
|
||||||
|
|
||||||
BREAK_TO_DEBUGGER();
|
|
||||||
kfree(opp);
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
struct clock_source *dce110_clock_source_create(
|
struct clock_source *dce110_clock_source_create(
|
||||||
|
|
|
@ -586,13 +586,9 @@ struct output_pixel_processor *dce112_opp_create(
|
||||||
if (!opp)
|
if (!opp)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
if (dce110_opp_construct(opp,
|
dce110_opp_construct(opp,
|
||||||
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask))
|
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask);
|
||||||
return &opp->base;
|
return &opp->base;
|
||||||
|
|
||||||
BREAK_TO_DEBUGGER();
|
|
||||||
kfree(opp);
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
struct clock_source *dce112_clock_source_create(
|
struct clock_source *dce112_clock_source_create(
|
||||||
|
|
|
@ -356,13 +356,9 @@ struct output_pixel_processor *dce120_opp_create(
|
||||||
if (!opp)
|
if (!opp)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
if (dce110_opp_construct(opp,
|
dce110_opp_construct(opp,
|
||||||
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask))
|
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask);
|
||||||
return &opp->base;
|
return &opp->base;
|
||||||
|
|
||||||
BREAK_TO_DEBUGGER();
|
|
||||||
kfree(opp);
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static const struct bios_registers bios_regs = {
|
static const struct bios_registers bios_regs = {
|
||||||
|
|
|
@ -422,13 +422,9 @@ static struct output_pixel_processor *dce80_opp_create(
|
||||||
if (!opp)
|
if (!opp)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
if (dce110_opp_construct(opp,
|
dce110_opp_construct(opp,
|
||||||
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask))
|
ctx, inst, &opp_regs[inst], &opp_shift, &opp_mask);
|
||||||
return &opp->base;
|
return &opp->base;
|
||||||
|
|
||||||
BREAK_TO_DEBUGGER();
|
|
||||||
kfree(opp);
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct stream_encoder *dce80_stream_encoder_create(
|
static struct stream_encoder *dce80_stream_encoder_create(
|
||||||
|
|
Loading…
Reference in New Issue