Clean up peak calculating code.

Mixer needn't keep peak data around that will never be used. Just
pass pcm_peaks structure to it instead of allocating for every
channel. Plugin API becomes incompatible.

vu_meter digital mode was still using global peak calculation;
switch it to playback channel like the rest.

Remove some accumulated soil peaks inside pcm.c and make it more
generic.

Change-Id: Ib4d268d80b6a9d09915eea1c91eab483c1a2c009

1 files changed, 3 insertions, 3 deletions

diff --git a/apps/plugin.h b/apps/plugin.h
index 588a012..4091cec 100644
--- a/apps/plugin.h
+++ b/apps/plugin.h
@@ -154,12 +154,12 @@ void* plugin_get_buffer(size_t *buffer_size);
 #define PLUGIN_MAGIC 0x526F634B /* RocK */
 
 /* increase this every time the api struct changes */
-#define PLUGIN_API_VERSION 219
+#define PLUGIN_API_VERSION 220
 
 /* update this to latest version if a change to the api struct breaks
    backwards compatibility (and please take the opportunity to sort in any
    new function which are "waiting" at the end of the function table) */
-#define PLUGIN_MIN_API_VERSION 219
+#define PLUGIN_MIN_API_VERSION 220
 
 /* plugin return codes */
 /* internal returns start at 0x100 to make exit(1..255) work */
@@ -697,7 +697,7 @@ struct plugin_api {
     const void * (*mixer_channel_get_buffer)(enum pcm_mixer_channel channel,
                                              int *count);
     void (*mixer_channel_calculate_peaks)(enum pcm_mixer_channel channel,
-                                          int *left, int *right);
+                                          struct pcm_peaks *peaks);
     void (*mixer_channel_play_data)(enum pcm_mixer_channel channel,
                                     pcm_play_callback_type get_more,
                                     const void *start, size_t size);