diff options
author | Gregory Nutt <gnutt@nuttx.org> | 2014-02-10 18:08:49 -0600 |
---|---|---|
committer | Gregory Nutt <gnutt@nuttx.org> | 2014-02-10 18:08:49 -0600 |
commit | 6db1f6f71c8867cbae3a2c5597b01f321064a370 (patch) | |
tree | bd0b760c6467a0b528b7b29480f944c99c186693 /nuttx/drivers/audio/vs1053.c | |
parent | 07d355e8f77afbc04ea94089071b0154e747acec (diff) | |
download | px4-nuttx-6db1f6f71c8867cbae3a2c5597b01f321064a370.tar.gz px4-nuttx-6db1f6f71c8867cbae3a2c5597b01f321064a370.tar.bz2 px4-nuttx-6db1f6f71c8867cbae3a2c5597b01f321064a370.zip |
Many changes to reduce complaints from CppCheck. Several latent bugs fixes, but probably some new typos introduced
Diffstat (limited to 'nuttx/drivers/audio/vs1053.c')
-rw-r--r-- | nuttx/drivers/audio/vs1053.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/nuttx/drivers/audio/vs1053.c b/nuttx/drivers/audio/vs1053.c index e44c7a6e4..e7b7cc059 100644 --- a/nuttx/drivers/audio/vs1053.c +++ b/nuttx/drivers/audio/vs1053.c @@ -1255,9 +1255,12 @@ static void *vs1053_workerthread(pthread_addr_t pvarg) { FAR struct vs1053_struct_s *dev = (struct vs1053_struct_s *) pvarg; struct audio_msg_s msg; - FAR struct ap_buffer_s *pBuf; - int size, prio; + FAR struct ap_buffer_s *pBuf; + int size; + int prio; +#ifndef CONFIG_AUDIO_EXCLUDE_STOP uint16_t reg; +#endif uint8_t timeout; auddbg("Entry\n"); @@ -1454,7 +1457,7 @@ static int vs1053_start(FAR struct audio_lowerhalf_s *lower) /* Pop the first enqueued buffer */ - if ((ret = (sem_wait(&dev->apbq_sem)) == OK)) + if ((ret = sem_wait(&dev->apbq_sem)) == OK) { dev->pBuf = (FAR struct ap_buffer_s *) dq_remfirst(&dev->apbq); apb_reference(dev->pBuf); /* Add our buffer reference */ |