summaryrefslogtreecommitdiffstats
path: root/broadcastradio
Commit message (Collapse)AuthorAgeFilesLines
* Remove libhwbinder/libhidltransport depsSteven Moreland2019-09-063-3/+0
| | | | | | | | Since these were combined into libhidlbase. Bug: 135686713 Test: build only (libhwbinder/libhidltransport are empty) Change-Id: I075670b64eebbbbd6a6ae0e84ad51bf1c6f5ba36
* Merge "Add interface info to .rc files"Felix2019-07-082-0/+3
|\ | | | | | | | | | | am: 98d0f4d52c Change-Id: Ie6cfbacd523c4b09f77bee3369ec8e88b3fa0419
| * Add interface info to .rc filesFelix2019-06-252-0/+3
| | | | | | | | | | Signed-off-by: Felix <google@ix5.org> Change-Id: I6d70bbdb66c3dce280bf6908c3750316a6f6cf70
* | Merge "Add dep to ...broadcastradio@common-utils-2x-tests" into qt-devSteven Moreland2019-04-301-0/+1
|\ \ | | | | | | | | | | | | | | | am: d64f48a504 Change-Id: Id77ccc02e5c9f6c2696765a9b13cbc4bb0366b23
| * | Add dep to ...broadcastradio@common-utils-2x-testsSteven Moreland2019-04-261-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Required now because iterators use the base hidl_vec constructor. It's not expected this type of change would be needed in any places. This test originally came with the introduction of the constructor that created the hidl_vec. Bug: 131356202 Test: build only, links (note b/130918604) Change-Id: Ide3cb13b6afd79489c208af2396e7c02b7da436e Merged-In: Ide3cb13b6afd79489c208af2396e7c02b7da436e
* | | Merge "Update hidl makefiles for bpfmt" am: ff0bd741caSteven Moreland2019-04-183-3/+0
|\ \ \ | |/ / |/| / | |/ | | | | am: 96f40f7b02 Change-Id: Idbf030e4993067bdb8181321bca2de00c9b6f7ef
| * Update hidl makefiles for bpfmtSteven Moreland2019-04-173-3/+0
| | | | | | | | | | | | | | | | hidl-generated makefiles are now generated such that bpfmt(file) == file. Bug: 67417008 Test: enable bpfmt hook Change-Id: I1f69d292bc23a7cc293a66110cb02d597e1019ad
* | Initial change to add mock DAB stations (hardcoded for now).Allie2019-03-186-2/+38
| | | | | | | | | | | | Test: VtsHalBroadcastradioV2_0TargetTest Change-Id: Ice8923490728402ed945e8cc0891fd8b2c5bf9dd
* | Merge "Update makefies: no 'types'" am: 4ee5ec1469 am: bab622f6a6Steven Moreland2019-03-043-52/+0
|\| | | | | | | | | | | am: 7224bc9bcf Change-Id: I434939e0770afa436c532a945542fce30a71ef7d
| * Update makefies: no 'types'Steven Moreland2019-03-043-52/+0
| | | | | | | | | | | | Bug: 123976090 Test: N/A Change-Id: I30fb04c81889b62775e1b764b965fdb0f893de17
| * pack VTS cc_test binaries as general-testsKeun Soo YIM2018-12-074-0/+6
| | | | | | | | | | | | | | Test: make general-tests Bug: 120093339 Merged-In: I363450d205868f900e4925ccff1430e2a569f2a4 Change-Id: I363450d205868f900e4925ccff1430e2a569f2a4
* | Fix program info call over-saturation.Tomasz Wasilczyk2018-12-031-3/+3
| | | | | | | | | | | | | | | | | | | | It was an omission to not allow onCurrentProgramInfoChanged to be called more than once. Bug: 120186193 Test: atest VtsHalBroadcastradioV2_0TargetTest Change-Id: I667c552fb88e8fa3be3931901f7ba86ee7743baa (cherry picked from commit 3f71e6c33ae34065cbf40ce1a2e4f9914383f969)
* | pack VTS cc_test binaries as general-testsKeun Soo YIM2018-11-274-0/+6
| | | | | | | | | | | | Test: make general-tests Bug: 120093339 Change-Id: I363450d205868f900e4925ccff1430e2a569f2a4
* | Migrate broadcast radio default HAL to the new logging API.Tomasz Wasilczyk2018-11-1411-80/+42
|/ | | | | | Bug: 112540729 Test: flash and boot Change-Id: I18d8b508971cd9a9b7b2c9c221674d862ff351d6
* Merge "Fix implicit-fallthrough warnings."Chih-hung Hsieh2018-10-221-1/+1
|\
| * Fix implicit-fallthrough warnings.Chih-Hung Hsieh2018-10-191-1/+1
| | | | | | | | | | | | Test: make checkbuild Bug: 112564944 Change-Id: Ifc8b7444b95d667b0d185c7d45b3f15278d4af2f
* | Allow implicit-fallthrough warnings locally.Chih-Hung Hsieh2018-10-221-0/+1
|/ | | | | | | | | It will be a global error by default. Test: make checkbuild Bug: 112564944 Change-Id: I3e914835fcb4ee97662ab51136f143e665d0ba01 Exempt-From-Owner-Approval: do not block on new warnings
* Fix comments and internal names about SEEK operation.Tomasz Wasilczyk2018-06-076-28/+39
| | | | | | | | | Legacy radio HAL named it SCAN and the name slipped through HAL 1.x into HAL 2.0. I finally noticed it after HAL 2.0 was locked down. Bug: 109740376 Test: build ow Change-Id: I5ca4147227a7dd0c32248a279f03fb632d97258e
* Improve current program info validation.Tomasz Wasilczyk2018-05-234-42/+64
| | | | | | Bug: 74022658 Test: atest VtsHalBroadcastradioV2_0TargetTest Change-Id: Ifee19249a9033363df863480e8a5da24334f5e59
* Update comments for doc comments.Steven Moreland2018-05-181-2/+4
| | | | | | | | | Doc comments look like "/** ... */" and they can only be in certain places. Bug: 79865343 Test: m Change-Id: Ic15c08ff7dc6e4f9827c1dbe7f7236c11a572ec1
* Add check for empty RDS_PS.Tomasz Wasilczyk2018-04-301-0/+9
| | | | | | Bug: 78885613 Test: atest VtsHalBroadcastradioV2_0TargetTest Change-Id: I9ac0e8975972636696504a6eeb0ecce58396e3fd
* Extend tune workaround time.Tomasz Wasilczyk2018-04-301-3/+5
| | | | | | Bug: 73963096 Test: atest VtsHalBroadcastradioV2_0TargetTest Change-Id: I6b722fb4f82005fe468a17f0f29af9dc8bd0e8f7
* Clear out ambiguity in broadcastradio@2.0 comments.Tomasz Wasilczyk2018-03-302-5/+13
| | | | | | Bug: 76798863 Test: it builds Change-Id: Icd13a187d3a002e6d1d0af8d360de31a75949b69
* Update program list when switching bands.Tomasz Wasilczyk2018-03-281-0/+1
| | | | | | Test: manual Bug: 74353024 Change-Id: I893cd4f9c7d14951924759792206af798b0ceefc
* Fix boadcastradio HAL 1.1 tuneByProgramSelector implementation.Tomasz Wasilczyk2018-03-084-40/+71
| | | | | | | | | It was not compliant with the HAL definition - it didn't auto-change band if necessary. Bug: 74353024 Test: manual Change-Id: I015faffc42778fa27fca3030306f31b0abe409c7
* Reword ProgramInfo comments.Tomasz Wasilczyk2018-02-211-5/+11
| | | | | | | | It makes clear which fields are required for the program list entries, and which for currently tuned program. Test: none Change-Id: If7ec3ad8f99e22b225063ca71a0c249b5d6d2bc1
* Merge "Convert broadcastradio hal test to use VtsHalHidlTargetTestEnvBase"Zhuoyao Zhang2018-02-146-15/+88
|\
| * Convert broadcastradio hal test to use VtsHalHidlTargetTestEnvBaseZhuoyao Zhang2018-02-136-15/+88
| | | | | | | | | | | | | | | | | | Bug: 64203181 Test: make vts vts-tradefed run vts -m VtsHalBroadcastradioV1_0Target vts-tradefed run vts -m VtsHalBroadcastradioV1_1Target Change-Id: I574b54dc95522486f4be9b50da436e90f3f88b9d
* | Merge "Drop broadcastradio HAL 1.2."Tomasz Wasilczyk2018-02-136-211/+0
|\ \
| * | Drop broadcastradio HAL 1.2.Tomasz Wasilczyk2018-02-106-211/+0
| | | | | | | | | | | | | | | | | | Test: it boots Bug: 62945293 Change-Id: Ie05d50f3084f51aa6e4de53096656b1387503459
* | | Verify physically/logicallyTunedTo field in ProgramInfo.Tomasz Wasilczyk2018-02-121-15/+41
|/ / | | | | | | | | | | Test: it builds Bug: 71614466 Change-Id: Iae6fe660cbadd3a4309c4700fc939289c0528d8f
* / Remove broadcastradio HAL 1.2 support from the framework.Tomasz Wasilczyk2018-02-101-1/+0
|/ | | | | | Bug: 62945293 Test: VTS Change-Id: Ifd54fe476717ae71e5c9ff72498bf702b0c29454
* Move broadcast radio 1.2 default implementation back to 1.1.Tomasz Wasilczyk2018-02-0922-436/+69
| | | | | | | Bug: 62945293 Test: VTS Change-Id: I95d68ea61f5b50c5b510212b0941bddad1158f97
* Merge "Rename getConfigFlag to isConfigFlagSet."TreeHugger Robot2018-01-205-10/+10
|\
| * Rename getConfigFlag to isConfigFlagSet.Tomasz Wasilczyk2018-01-125-10/+10
| | | | | | | | | | | | | | | | The latter matches code guidelines better. Bug: 69958423 Test: it builds Change-Id: I520733a061f03bab57544a1451dd763dc53ce5c6
* | Merge "Allow spaces in DAB frequency labels and make it up to 7 characters ↵Tomasz Wasilczyk2018-01-192-3/+7
|\ \ | | | | | | | | | long."
| * | Allow spaces in DAB frequency labels and make it up to 7 characters long.Tomasz Wasilczyk2018-01-152-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no standard for DAB label format, the regexp is based on our partners knowledge. Bug: 69958423 Test: VTS Change-Id: I917508408de22f8a75d39d4e9badddd385869b03
* | | Fix remaining instrumentation tests.Tomasz Wasilczyk2018-01-155-11/+91
|/ / | | | | | | | | | | Bug: 69958777 Test: VTS, Instrumentation Change-Id: I98956ea4b1d2953b9159fb7d53ce61e57f80a90f
* / Rename AnnouncementObserver to AnnouncementListener.Tomasz Wasilczyk2018-01-116-21/+21
|/ | | | | | | | I feel this wording match java naming more closely. Bug: 68045105 Test: VTS Change-Id: If30fa9b356b6b45fe8853bc5194c7098d0fb2aa5
* Broadcast radio 1.x default implementation: let's support background scan ↵Tomasz Wasilczyk2018-01-091-1/+1
| | | | | | | | for real. Bug: 69860743 Test: VTS, instrumentation Change-Id: Ia636795494c88a82edf681adaa403e3b70f77814
* Merge "Broadcast radio 1.x default implementation: let's support background ↵Tomasz Wasilczyk2018-01-091-1/+5
|\ | | | | | | scan."
| * Broadcast radio 1.x default implementation: let's support background scan.Tomasz Wasilczyk2018-01-081-1/+5
| | | | | | | | | | | | Bug: 69860743 Test: VTS Change-Id: I7f0dc841e2b3fc218e5e7efa9b102697655cc9d5
* | Merge "Fix few typos in broadcast radio HAL 2.0 comments."Tomasz Wasilczyk2018-01-081-3/+4
|\ \
| * | Fix few typos in broadcast radio HAL 2.0 comments.Tomasz Wasilczyk2018-01-051-3/+4
| |/ | | | | | | | | | | Bug: 69958423 Test: none Change-Id: Ic9e5ddd4fb6d033f6f98eb0b537856abe4e4f025
* | Merge "Prepare a best-effort workaround for HD Radio station id abuse."Tomasz Wasilczyk2018-01-048-12/+172
|\ \ | |/ |/|
| * Prepare a best-effort workaround for HD Radio station id abuse.Tomasz Wasilczyk2018-01-048-12/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In theory, 32bit HD Radio station ID + subchannel index (parts of HD_STATION_ID_EXT) is a globally unique identifier. It allows broadcast radio framework to determine which programs are the same and allow the application to match entries from favourite list and the program list provided by the tuner. However, some broadcasters don't perform equipment setup correctly and don't set station ID. As a result, there are some stations with conflicting IDs. As a workaround to treat these stations separately in a given location, FM frequency was added as a part of HD_STATION_ID_EXT. This still doesn't solve the global uniqueness problem: user might save KCQW 105.5 (sid=0) in California, travel to Nevada and find KNAB 105.5 (sid=0). It turns out there is no reliable identifier that might identify the station globally. As a workaround, shortened station name is added for double-checking. This is a best-effort fix, so it's not required for such misbehaving stations to get correctly identified in every corner case. Bug: 69958777 Test: VTS Change-Id: Id11243096f1cde7fdda5cb70a7248d1831985cdd
* | Implement broadcast radio announcements.Tomasz Wasilczyk2018-01-039-2/+196
|/ | | | | | Bug: 68045105 Test: VTS Change-Id: I14d394862ec34d63218c938779e217c28710e230
* Implement regional configuration fetching.Tomasz Wasilczyk2018-01-0211-18/+396
| | | | | | Bug: 69958423 Test: VTS Change-Id: I7c184191b4f4999bd03b06bd3b2283e028694918
* Implement currently tuned ID / channel indication.Tomasz Wasilczyk2017-12-285-1/+112
| | | | | | Bug: 70939328 Test: it builds Change-Id: I62011b70b3786b0d5e5ac24eb480a3c52a35f104
* Merge "Implement program list fetching."Tomasz Wasilczyk2017-12-2212-27/+615
|\