Commit graph

64 commits

Author SHA1 Message Date
Tomasz Wasilczyk
88960f1d3d Fix few typos in broadcast radio HAL 2.0 comments.
Bug: 69958423
Test: none
Change-Id: Ic9e5ddd4fb6d033f6f98eb0b537856abe4e4f025
2018-01-05 13:37:44 -08:00
Tomasz Wasilczyk
38b51669e4 Merge "Prepare a best-effort workaround for HD Radio station id abuse." 2018-01-04 20:17:28 +00:00
Tomasz Wasilczyk
c71624f8ed Prepare a best-effort workaround for HD Radio station id abuse.
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
2018-01-04 08:40:31 -08:00
Tomasz Wasilczyk
6a9f856e24 Implement broadcast radio announcements.
Bug: 68045105
Test: VTS
Change-Id: I14d394862ec34d63218c938779e217c28710e230
2018-01-03 10:01:10 -08:00
Tomasz Wasilczyk
8b70ee43b0 Implement regional configuration fetching.
Bug: 69958423
Test: VTS
Change-Id: I7c184191b4f4999bd03b06bd3b2283e028694918
2018-01-02 11:51:59 -08:00
Tomasz Wasilczyk
4ce6382a45 Implement currently tuned ID / channel indication.
Bug: 70939328
Test: it builds
Change-Id: I62011b70b3786b0d5e5ac24eb480a3c52a35f104
2017-12-28 20:31:20 -08:00
Tomasz Wasilczyk
eb2b87ba3e Merge "Implement program list fetching." 2017-12-22 20:44:15 +00:00
Tomasz Wasilczyk
bceb88585d Implement program list fetching.
Bug: 69860743
Test: VTS
Change-Id: I04eb43c1e0e1bb7bad86e123594a473454eed983
2017-12-22 10:19:50 -08:00
Tomasz Wasilczyk
30240f61ed Redefine meaning of DAB/FM linking.
Bug: 69958423
Test: VTS
Change-Id: I0ab4ddd6fc34f12720085cca9d0b821ec660a5c8
2017-12-20 14:49:45 -08:00
Tomasz Wasilczyk
43fe894562 Implement config flags.
Bug: 69958423
Test: VTS
Change-Id: I6221d2bd2c6f1e31c93b105fce4cfc6d673e3b77
2017-12-14 15:04:06 -08:00
Tomasz Wasilczyk
28ca7e9740 Try to fix some builds.
Broken builds: aosp_x86, docs, full, sdk_phone_armv7.

I cannot reproduce the break locally, so making my best guess on the fix.

Test: it still builds
Change-Id: Icf86c3502fede66dba815458b66f2ebdf33afe91
2017-12-13 10:09:22 -08:00
Tomasz Wasilczyk
31e8632d10 Implement broadcast radio HAL 2.0 VTS tests.
Test: VTS
Bug: 69958777
Change-Id: I671c033519a6a41421b9ad73b0b897f832a1c3c0
2017-12-12 10:46:56 -08:00
Tomasz Wasilczyk
06100b39db Implement broadcast radio HAL 2.0 default implementation.
Also, split internal common utils library to 1.x and 2.x.

Test: VTS
Bug: 69958777
Change-Id: I21244c74270a07cc350e8a2b974dafcdae02a0e8
2017-12-12 10:22:11 -08:00
Tomasz Wasilczyk
6e0e1ae2a9 Define broadcast radio HAL 2.0.
The new HAL is a cleanup of 1.x branch of the legacy burden:
 * structure flattened (multi-level factory removed);
 * only one hardware tuner per HAL instance, only one session;
 * front-end app doesn't control region settings anymore;
 * metadata limited to int and string values;
 * removed deprecated methods;
 * result codes redefined.

It also fixes minor mistakes made with HAL 1.1:
 * ProgramSelector simplified;
 * there is no need to control background scan.

There are three features missing compared to the HAL 1.1, as they
are in development with the new design (see design docs attached):
 * Announcements - b/68045105
 * Program list - b/69860743
 * Region handling - b/69958423

Test: VTS
Bug: b/69958777
Change-Id: I0ad83f25630c1250d73dc3941144d345339fbde0
2017-12-05 14:10:04 -08:00