Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-02 10:05:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

power/stats: Update HIDL to use synchronous FMQ

Browse Source 
Unsynchronous queue do not help solve the problem of
supporting multiple clients of power.stats. Since, there
is no current requirement to support multiple clients so
using synchronous queues instead.

BUG: 117424656
BUG: 111185513
Test: run vts -m VtsHalPowerStatsV1_0Target
Change-Id: Iab73de5a460e2a0ec5e949f8b021a87aefed58c8
(cherry picked from commit 84a48b8325)
Merged-In: Iab73de5a460e2a0ec5e949f8b021a87aefed58c8
This commit is contained in:
Vinay Kalia
2018-10-22 19:21:48 -07:00
committed by Benjamin Schwartz
parent f12c171b3a
commit 3b91fd4ab6
2 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
power/stats/1.0/IPowerStats.hal
View File

@@ -50,10 +50,13 @@ interface IPowerStats {
* asynchronous.
*
* @param timeMs Time(in ms) for which energyData should be streamed
* @return mqDesc Unsynchronous Fast Message Queue descriptor - One
* writer(power.stats HAL) multiple readers are supported. Reader
* should read faster than writer otherwise data might be
* overwritten. Data is present in following format in the queue:
* @param samplingRate Frequency(in Hz) at which samples should be
* captured. If the requested sampling rate is not supported then
* SUCCESS is returned and numSamples are reported back according
* to the supported sampling rate.
* @return mqDesc Blocking Synchronous Fast Message Queue descriptor - One
* writer(power.stats HAL) and one reader are supported. Data is
* present in the following format in the queue:
* +-----------------------+ <--
* | EnergyData for rail 1 | |
* +-----------------------+ |
@@ -87,9 +90,10 @@ interface IPowerStats {
* @return numSamples Number of samples which will be generated in timeMs.
* @return railsPerSample Number of rails measured per sample.
* @return status SUCCESS on success or FILESYSTEM_ERROR on filesystem
* nodes access or NOT_SUPPORTED if feature is not enabled.
* nodes access or NOT_SUPPORTED if feature is not enabled or
* INSUFFICIENT_RESOURCES if there are not enough resources.
*/
streamEnergyData(uint32_t timeMs)
generates(fmq_unsync<EnergyData> mqDesc, uint32_t numSamples,
streamEnergyData(uint32_t timeMs, uint32_t samplingRate)
generates(fmq_sync<EnergyData> mqDesc, uint32_t numSamples,
uint32_t railsPerSample, Status status);
};

1
power/stats/1.0/types.hal
View File

@@ -20,6 +20,7 @@ enum Status : uint32_t {
NOT_SUPPORTED = 1,
INVALID_INPUT = 2,
FILESYSTEM_ERROR = 3,
INSUFFICIENT_RESOURCES = 4,
};
struct RailInfo {