From: Dave Chapman <dave_at_dchapman.com>
Date: Tue, 12 Jun 2007 09:33:06 +0100

mailer_at_svn.rockbox.org wrote:
> Modified: trunk/firmware/drivers/audio/tlv320.c
> ===================================================================
> --- trunk/firmware/drivers/audio/tlv320.c 2007-06-11 13:32:29 UTC (rev 13619)
> +++ trunk/firmware/drivers/audio/tlv320.c 2007-06-11 23:39:07 UTC (rev 13620)
> _at__at_ -219,10 +219,10 _at__at_
> }
>
> /**
> - * Mute (mute=true) or enable sound (mute=false)
> + * Mute (mute=1) or enable sound (mute=0)
> *
> */
> -void audiohw_mute(bool mute)
> +void audiohw_mute(int mute)

Was there a reason for making this parameter an int in all drivers,
rather than bool?

I realise that only the tlv320 driver used bool before your commit, but
that would seem clearer IMO.

Dave.
Received on 2007-06-12