diff options
author | Scott Murray <scott.murray@konsulko.com> | 2020-09-08 18:38:10 -0400 |
---|---|---|
committer | Scott Murray <scott.murray@konsulko.com> | 2020-09-08 18:43:20 -0400 |
commit | 8afc720a98e4fdf4f5c1f10492e465927b8ffd86 (patch) | |
tree | 4f88905908c9cd67b2177652f1ce0a00486f610e /binding/radio_impl_tef665x.c | |
parent | 4cdb281367813ffc12face0bbcba300fe6e7e2d3 (diff) |
Add null backendjellyfish_9.99.4jellyfish/9.99.49.99.4
Changes:
- Add null radio implementation that acts as the fallthrough default
when the hardware implementations are not detected.
- Tweaked the probing in the tef665x implementation to not return an
error if the I2C node is not even present. This cleans up the logs
on start up on most platforms. These changes include some minor
whitespace consistency clean ups.
- Added the run-by-default widget permission now that the binding
should always work. This will ease testing, and potentially decrease
app start up time.
Bug-AGL: SPEC-3570
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I3c2e60ce7272d248ab87e3b79d6a334e17fe5531
Diffstat (limited to 'binding/radio_impl_tef665x.c')
-rw-r--r-- | binding/radio_impl_tef665x.c | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/binding/radio_impl_tef665x.c b/binding/radio_impl_tef665x.c index eeac170..17454b8 100644 --- a/binding/radio_impl_tef665x.c +++ b/binding/radio_impl_tef665x.c @@ -2144,15 +2144,19 @@ static int tef665x_init() { char gst_pipeline_str[GST_PIPELINE_LEN]; int rc; - int ret = i2c_init(I2C_DEV, _open, &file_desc); current_am_frequency = known_am_band_plans[am_bandplan].min; current_fm_frequency = known_fm_band_plans[fm_bandplan].min; + rc = i2c_init(I2C_DEV, _open, &file_desc); + if(rc < 0) { + AFB_NOTICE("tef665x not present"); + return -1; + } _debug("file_desc= ", file_desc); - ret = appl_get_identification(file_desc); - if(ret != 1){ + rc = appl_get_identification(file_desc); + if(rc != 1){ AFB_ERROR("no tef665x!"); return -1; } @@ -2187,11 +2191,11 @@ static int tef665x_init() } // Start pipeline in paused state - ret = gst_element_set_state(pipeline, GST_STATE_PAUSED); - _debug("gst_element_set_state to pause (at the begining)", ret); + rc = gst_element_set_state(pipeline, GST_STATE_PAUSED); + _debug("gst_element_set_state to pause (at the begining)", rc); - ret = gst_bus_add_watch(gst_element_get_bus(pipeline), (GstBusFunc) handle_message, NULL); - _debug("gst_bus_add_watch ret", ret); + rc = gst_bus_add_watch(gst_element_get_bus(pipeline), (GstBusFunc) handle_message, NULL); + _debug("gst_bus_add_watch rc", rc); present = true; @@ -2200,7 +2204,7 @@ static int tef665x_init() pthread_mutex_init (&RDS_Mutex, NULL); tef665x_start(); - return 0; + return 0; } static void tef665x_set_frequency_callback(radio_freq_callback_t callback, |