X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;ds=sidebyside;f=plugins%2Faudio%2Faudio-api.c;h=f8c51cca926cdbfadd59f10f19e98722a237a2cf;hb=36a165503297903204be32a07e0440ca56dac7f0;hp=637f9d036789f45709a4f054c15710561082074e;hpb=3da6e9fad4c6788098cf425369fb3d7860cb824f;p=src%2Fapp-framework-binder.git
diff --git a/plugins/audio/audio-api.c b/plugins/audio/audio-api.c
index 637f9d03..f8c51cca 100644
--- a/plugins/audio/audio-api.c
+++ b/plugins/audio/audio-api.c
@@ -2,38 +2,50 @@
* Copyright (C) 2015 "IoT.bzh"
* Author "Manuel Bachmann"
*
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
*
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
+ * http://www.apache.org/licenses/LICENSE-2.0
*
- * You should have received a copy of the GNU General Public License
- * along with this program. If not, see .
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
*/
+#define _GNU_SOURCE
+#include
+#include
+
#include "audio-api.h"
+#include "audio-alsa.h"
+#ifdef HAVE_PULSE
+#include "audio-pulse.h"
+#endif
+
+#include "afb-plugin.h"
+#include "afb-req-itf.h"
/* ------ BACKEND FUNCTIONS ------- */
-void _backend_init (const char *name, audioCtxHandleT *ctx) {
+unsigned char _backend_init (const char *name, audioCtxHandleT *ctx) {
char *backend_env = getenv ("AFB_AUDIO_OUTPUT");
- unsigned char res = -1;
+ unsigned char res = 0;
# ifdef HAVE_PULSE
- if (!backend_env || (strcasecmp (backend_env, "Alsa") != 0))
+ if (!backend_env || (strcasecmp (backend_env, "Pulse") == 0))
res = _pulse_init (name, ctx);
- if (res < 0)
+ if (!res)
#endif
res = _alsa_init (name, ctx);
- if (res < 0 && verbose)
+ if (!res)
fprintf (stderr, "Could not initialize Audio backend\n");
+
+ return res;
}
void _backend_free (audioCtxHandleT *ctx) {
@@ -41,7 +53,7 @@ void _backend_free (audioCtxHandleT *ctx) {
# ifdef HAVE_PULSE
if (ctx->audio_dev) _pulse_free (ctx); else
# endif
- _alsa_free (ctx->idx);
+ _alsa_free (ctx->name);
}
void _backend_play (audioCtxHandleT *ctx) {
@@ -60,7 +72,7 @@ void _backend_stop (audioCtxHandleT *ctx) {
_alsa_stop (ctx->idx);
}
-int _backend_get_volume (audioCtxHandleT *ctx, unsigned int channel) {
+unsigned int _backend_get_volume (audioCtxHandleT *ctx, unsigned int channel) {
# ifdef HAVE_PULSE
if (ctx->audio_dev) return _pulse_get_volume (ctx, channel); else
@@ -68,7 +80,7 @@ int _backend_get_volume (audioCtxHandleT *ctx, unsigned int channel) {
return _alsa_get_volume (ctx->idx, channel);
}
-void _backend_set_volume (audioCtxHandleT *ctx, unsigned int channel, int vol) {
+void _backend_set_volume (audioCtxHandleT *ctx, unsigned int channel, unsigned int vol) {
# ifdef HAVE_PULSE
if (ctx->audio_dev) _pulse_set_volume (ctx, channel, vol); else
@@ -76,7 +88,7 @@ void _backend_set_volume (audioCtxHandleT *ctx, unsigned int channel, int vol) {
_alsa_set_volume (ctx->idx, channel, vol);
}
-void _backend_set_volume_all (audioCtxHandleT *ctx, int vol) {
+void _backend_set_volume_all (audioCtxHandleT *ctx, unsigned int vol) {
# ifdef HAVE_PULSE
if (ctx->audio_dev) _pulse_set_volume_all (ctx, vol); else
@@ -110,14 +122,15 @@ void _backend_set_channels (audioCtxHandleT *ctx, unsigned int channels) {
/* ------ LOCAL HELPER FUNCTIONS --------- */
-/* private client context creation ; default values */
-STATIC audioCtxHandleT* initAudioCtx () {
+/* private client context constructor ; default values */
+static audioCtxHandleT* initAudioCtx () {
audioCtxHandleT *ctx;
int i;
ctx = malloc (sizeof(audioCtxHandleT));
ctx->audio_dev = NULL;
+ ctx->name = NULL;
ctx->idx = -1;
for (i = 0; i < 8; i++)
ctx->volume[i] = 25;
@@ -128,50 +141,54 @@ STATIC audioCtxHandleT* initAudioCtx () {
return ctx;
}
-STATIC AFB_error releaseAudio (audioCtxHandleT *ctx) {
+static void releaseAudioCtx (void *context) {
+
+ audioCtxHandleT *ctx = (audioCtxHandleT*) context;
/* power it off */
_backend_free (ctx);
/* clean client context */
+ ctx->audio_dev = NULL;
+ if (ctx->name)
+ free (ctx->name);
ctx->idx = -1;
-
- return AFB_SUCCESS;
-}
-
-/* called when client session dies [e.g. client quits for more than 15mns] */
-STATIC void freeAudio (void *context) {
- free (context);
+ free (ctx);
}
/* ------ PUBLIC PLUGIN FUNCTIONS --------- */
-STATIC json_object* init (AFB_request *request) { /* AFB_SESSION_CHECK */
+static void init (struct afb_req request) { /* AFB_SESSION_CHECK */
+ audioCtxHandleT *ctx = (audioCtxHandleT*) afb_req_context_get(request);
json_object *jresp;
- int idx;
/* create a private client context */
- if (!request->context)
- request->context = initAudioCtx();
+ if (!ctx) {
+ ctx = initAudioCtx();
+ afb_req_context_set (request, ctx, releaseAudioCtx);
+ }
- _backend_init("default", request->context);
+ if (!_backend_init ("default", ctx)) {
+ afb_req_fail (request, "failed", "backend initialization failed");
+ }
jresp = json_object_new_object();
- json_object_object_add (jresp, "info", json_object_new_string ("Audio initialised"));
- return (jresp);
+ json_object_object_add (jresp, "init", json_object_new_string ("success"));
+ afb_req_success (request, jresp, "Audio initialized");
}
-STATIC json_object* volume (AFB_request *request) { /* AFB_SESSION_CHECK */
+static void volume (struct afb_req request) { /* AFB_SESSION_CHECK */
- audioCtxHandleT *ctx = (audioCtxHandleT*)request->context;
- const char *value = getQueryValue (request, "value");
+ audioCtxHandleT *ctx = (audioCtxHandleT*) afb_req_context_get(request);
+ struct afb_arg arg = afb_req_get (request, "value");
+ const char *value = arg.value;
json_object *jresp;
- int volume[8], i;
+ unsigned int volume[8], i;
char *volume_i;
char volume_str[256];
- int len_str = 0;
+ size_t len_str = 0;
/* no "?value=" parameter : return current state */
if (!value) {
@@ -188,12 +205,12 @@ STATIC json_object* volume (AFB_request *request) { /* AFB_SESSION_CHECK */
else {
volume_i = strdup (value);
volume_i = strtok (volume_i, ",");
- volume[0] = atoi (volume_i);
+ volume[0] = (unsigned int) atoi (volume_i);
- if (100 < volume[0] < 0) {
+ if (100 < volume[0]) {
free (volume_i);
- request->errcode = MHD_HTTP_SERVICE_UNAVAILABLE;
- return (jsonNewMessage (AFB_FAIL, "Volume must be between 0 and 100"));
+ afb_req_fail (request, "failed", "volume must be between 0 and 100");
+ return;
}
ctx->volume[0] = volume[0];
_backend_set_volume (ctx, 0, ctx->volume[0]);
@@ -204,10 +221,10 @@ STATIC json_object* volume (AFB_request *request) { /* AFB_SESSION_CHECK */
/* if there is only one value, set all channels to this one */
if (!volume_i && i == 1)
_backend_set_volume_all (ctx, ctx->volume[0]);
- if (!volume_i || 100 < atoi(volume_i) < 0) {
+ if (!volume_i || 100 < atoi(volume_i) || atoi(volume_i) < 0) {
ctx->volume[i] = _backend_get_volume (ctx, i);
} else {
- ctx->volume[i] = atoi(volume_i);
+ ctx->volume[i] = (unsigned int) atoi(volume_i);
_backend_set_volume (ctx, i, ctx->volume[i]);
}
len_str = strlen(volume_str);
@@ -217,13 +234,14 @@ STATIC json_object* volume (AFB_request *request) { /* AFB_SESSION_CHECK */
json_object_object_add (jresp, "volume", json_object_new_string(volume_str));
}
- return jresp;
+ afb_req_success (request, jresp, "Audio - Volume changed");
}
-STATIC json_object* channels (AFB_request *request) { /* AFB_SESSION_CHECK */
+static void channels (struct afb_req request) { /* AFB_SESSION_CHECK */
- audioCtxHandleT *ctx = (audioCtxHandleT*)request->context;
- const char *value = getQueryValue (request, "value");
+ audioCtxHandleT *ctx = (audioCtxHandleT*) afb_req_context_get(request);
+ struct afb_arg arg = afb_req_get (request, "value");
+ const char *value = arg.value;
json_object *jresp = json_object_new_object();
char channels_str[256];
@@ -235,20 +253,21 @@ STATIC json_object* channels (AFB_request *request) { /* AFB_SESSION_CHECK */
/* "?value=" parameter, set channels */
else {
- ctx->channels = atoi (value);
+ ctx->channels = (unsigned int) atoi (value);
_backend_set_channels (ctx, ctx->channels);
snprintf (channels_str, sizeof(channels_str), "%d", ctx->channels);
json_object_object_add (jresp, "channels", json_object_new_string (channels_str));
}
- return jresp;
+ afb_req_success (request, jresp, "Audio - Channels set");
}
-STATIC json_object* mute (AFB_request *request) { /* AFB_SESSION_CHECK */
+static void mute (struct afb_req request) { /* AFB_SESSION_CHECK */
- audioCtxHandleT *ctx = (audioCtxHandleT*)request->context;
- const char *value = getQueryValue (request, "value");
+ audioCtxHandleT *ctx = (audioCtxHandleT*) afb_req_context_get(request);
+ struct afb_arg arg = afb_req_get (request, "value");
+ const char *value = arg.value;
json_object *jresp = json_object_new_object();
/* no "?value=" parameter : return current state */
@@ -275,13 +294,14 @@ STATIC json_object* mute (AFB_request *request) { /* AFB_SESSION_CHECK */
json_object_object_add (jresp, "mute", json_object_new_string ("off"));
}
- return jresp;
+ afb_req_success (request, jresp, "Audio - Mute set");
}
-STATIC json_object* play (AFB_request *request) { /* AFB_SESSION_CHECK */
+static void play (struct afb_req request) { /* AFB_SESSION_CHECK */
- audioCtxHandleT *ctx = (audioCtxHandleT*)request->context;
- const char *value = getQueryValue (request, "value");
+ audioCtxHandleT *ctx = (audioCtxHandleT*) afb_req_context_get(request);
+ struct afb_arg arg = afb_req_get (request, "value");
+ const char *value = arg.value;
json_object *jresp = json_object_new_object();
/* no "?value=" parameter : return current state */
@@ -307,31 +327,31 @@ STATIC json_object* play (AFB_request *request) { /* AFB_SESSION_CHECK */
json_object_object_add (jresp, "play", json_object_new_string ("off"));
}
- return jresp;
+ afb_req_success (request, jresp, "Audio - Play");
}
-STATIC json_object* ping (AFB_request *request) { /* AFB_SESSION_NONE */
- return jsonNewMessage(AFB_SUCCESS, "Ping Binder Daemon - Radio API");
+static void ping (struct afb_req request) { /* AFB_SESSION_NONE */
+ afb_req_success (request, NULL, "Audio - Ping success");
}
-STATIC AFB_restapi pluginApis[]= {
- {"init" , AFB_SESSION_CHECK, (AFB_apiCB)init , "Audio API - init"},
- {"volume" , AFB_SESSION_CHECK, (AFB_apiCB)volume , "Audio API - volume"},
- {"channels", AFB_SESSION_CHECK, (AFB_apiCB)channels , "Audio API - channels"},
- {"mute" , AFB_SESSION_CHECK, (AFB_apiCB)mute , "Audio API - mute"},
- {"play" , AFB_SESSION_CHECK, (AFB_apiCB)play , "Audio API - play"},
- {"ping" , AFB_SESSION_NONE, (AFB_apiCB)ping , "Audio API - ping"},
+static const struct AFB_restapi pluginApis[]= {
+ {"init" , AFB_SESSION_CHECK, init , "Audio API - init"},
+ {"volume" , AFB_SESSION_CHECK, volume , "Audio API - volume"},
+ {"channels", AFB_SESSION_CHECK, channels , "Audio API - channels"},
+ {"mute" , AFB_SESSION_CHECK, mute , "Audio API - mute"},
+ {"play" , AFB_SESSION_CHECK, play , "Audio API - play"},
+ {"ping" , AFB_SESSION_NONE, ping , "Audio API - ping"},
{NULL}
};
-PUBLIC AFB_plugin *pluginRegister () {
- AFB_plugin *plugin = malloc (sizeof(AFB_plugin));
- plugin->type = AFB_PLUGIN_JSON;
- plugin->info = "Application Framework Binder - Audio plugin";
- plugin->prefix = "audio";
- plugin->apis = pluginApis;
-
- plugin->freeCtxCB = (AFB_freeCtxCB)freeAudio;
-
- return (plugin);
+static const struct AFB_plugin pluginDesc = {
+ .type = AFB_PLUGIN_JSON,
+ .info = "Application Framework Binder - Audio plugin",
+ .prefix = "audio",
+ .apis = pluginApis
};
+
+const struct AFB_plugin *pluginRegister (const struct AFB_interface *itf)
+{
+ return &pluginDesc;
+}