mirror of https://github.com/thesofproject/sof.git
selector: fix compiler warnings on unused 'dev'
comp_dbg() statements may be compiled out, so having a 'dev' variable just for debugs is not useful and just ends up with compiler warnings on unused variables. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
This commit is contained in:
parent
42aec0a186
commit
44d555b8e5
|
@ -597,11 +597,10 @@ static int selector_init(struct processing_module *mod)
|
||||||
{
|
{
|
||||||
struct module_data *md = &mod->priv;
|
struct module_data *md = &mod->priv;
|
||||||
struct module_config *cfg = &md->cfg;
|
struct module_config *cfg = &md->cfg;
|
||||||
struct comp_dev *dev = mod->dev;
|
|
||||||
struct comp_data *cd;
|
struct comp_data *cd;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
comp_dbg(dev, "selector_init()");
|
comp_dbg(mod->dev, "selector_init()");
|
||||||
|
|
||||||
cd = rzalloc(SOF_MEM_ZONE_RUNTIME, 0, SOF_MEM_CAPS_RAM, sizeof(*cd));
|
cd = rzalloc(SOF_MEM_ZONE_RUNTIME, 0, SOF_MEM_CAPS_RAM, sizeof(*cd));
|
||||||
if (!cd)
|
if (!cd)
|
||||||
|
@ -700,9 +699,8 @@ static void set_selector_params(struct comp_dev *dev,
|
||||||
static int selector_free(struct processing_module *mod)
|
static int selector_free(struct processing_module *mod)
|
||||||
{
|
{
|
||||||
struct comp_data *cd = module_get_private_data(mod);
|
struct comp_data *cd = module_get_private_data(mod);
|
||||||
struct comp_dev *dev = mod->dev;
|
|
||||||
|
|
||||||
comp_dbg(dev, "selector_free()");
|
comp_dbg(mod->dev, "selector_free()");
|
||||||
|
|
||||||
rfree(cd);
|
rfree(cd);
|
||||||
|
|
||||||
|
@ -764,9 +762,8 @@ static int selector_process(struct processing_module *mod,
|
||||||
{
|
{
|
||||||
struct comp_data *cd = module_get_private_data(mod);
|
struct comp_data *cd = module_get_private_data(mod);
|
||||||
uint32_t avail_frames = input_buffers[0].size;
|
uint32_t avail_frames = input_buffers[0].size;
|
||||||
struct comp_dev *dev = mod->dev;
|
|
||||||
|
|
||||||
comp_dbg(dev, "selector_process()");
|
comp_dbg(mod->dev, "selector_process()");
|
||||||
|
|
||||||
if (!avail_frames)
|
if (!avail_frames)
|
||||||
return PPL_STATUS_PATH_STOP;
|
return PPL_STATUS_PATH_STOP;
|
||||||
|
@ -871,9 +868,8 @@ static int selector_prepare(struct processing_module *mod)
|
||||||
static int selector_reset(struct processing_module *mod)
|
static int selector_reset(struct processing_module *mod)
|
||||||
{
|
{
|
||||||
struct comp_data *cd = module_get_private_data(mod);
|
struct comp_data *cd = module_get_private_data(mod);
|
||||||
struct comp_dev *dev = mod->dev;
|
|
||||||
|
|
||||||
comp_info(dev, "selector_reset()");
|
comp_info(mod->dev, "selector_reset()");
|
||||||
|
|
||||||
cd->source_period_bytes = 0;
|
cd->source_period_bytes = 0;
|
||||||
cd->sink_period_bytes = 0;
|
cd->sink_period_bytes = 0;
|
||||||
|
|
Loading…
Reference in New Issue