[Sound-open-firmware] [PATCH] Incorrect trace variable name

Curtis Malainey cujomalainey at google.com
Wed Aug 9 18:17:48 CEST 2017


Yep, you are more than welcome to.

Curtis

On Wed, Aug 9, 2017 at 8:22 AM, Liam Girdwood <liam.r.girdwood at intel.com>
wrote:

> On Thu, 2017-08-03 at 13:08 -0700, Curtis Malainey wrote:
> > Minor debugging issue, when verbose trace is turned on it will not
> compile
> > because these variables are incorrect.
> > ---
> >  src/audio/eq_fir.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/audio/eq_fir.c b/src/audio/eq_fir.c
> > index ae87f92..41c6d38 100644
> > --- a/src/audio/eq_fir.c
> > +++ b/src/audio/eq_fir.c
> > @@ -323,9 +323,9 @@ static int eq_fir_cmd(struct comp_dev *dev, int cmd,
> void *data)
> >                       fir_update, PLATFORM_MAX_CHANNELS);
> >
> >               /* Print trace information */
> > -             tracev_value(iir_update->stream_max_channels);
> > +             tracev_value(fir_update->stream_max_channels);
> >               for (i = 0; i < fir_update->stream_max_channels; i++)
> > -                     tracev_value(iir_update->assign_response[i]);
> > +                     tracev_value(fir_update->assign_response[i]);
> >
> >               break;
> >       case COMP_CMD_EQ_FIR_CONFIG:
>
> Can I add your Signed-off-by to both these patches ?
>
> Thanks
>
> Liam
>
> ---------------------------------------------------------------------
> Intel Corporation (UK) Limited
> Registered No. 1134945 (England)
> Registered Office: Pipers Way, Swindon SN3 1RJ
> VAT No: 860 2173 47
>
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.
>


More information about the Sound-open-firmware mailing list