diff options
author | Eino-Ville Talvala <etalvala@google.com> | 2018-12-04 13:35:46 -0800 |
---|---|---|
committer | Eino-Ville Talvala <etalvala@google.com> | 2019-03-28 16:51:39 -0700 |
commit | 7c51e8492e7dd1350683c5847d171e9e84228156 (patch) | |
tree | 2be091ef335ebca2f1591281e6e9dd756a95a78f /graphics | |
parent | 7cec015f227833f8d5bff005a8ed44da94cb534f (diff) | |
download | android_hardware_interfaces-7c51e8492e7dd1350683c5847d171e9e84228156.tar.gz android_hardware_interfaces-7c51e8492e7dd1350683c5847d171e9e84228156.tar.bz2 android_hardware_interfaces-7c51e8492e7dd1350683c5847d171e9e84228156.zip |
graphics.mapper@2.0: Document previously-implicit accessRegion behavior.
Existing Android framework code (and transitively, CTS test) require
that an accessRegion of (0,0,0,0) is treated the same as an
accessRegion covering the entire buffer, when calling lock() or
lockYCbCr().
Document this so that there is no confusion about this going forward,
since this requirement pre-dates the HIDL HALs.
Bug: 119440345
Test: Builds
Change-Id: Id16831d3da4ec3dc74dbdca18447581a50ee1193
Diffstat (limited to 'graphics')
-rw-r--r-- | graphics/mapper/2.0/IMapper.hal | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/graphics/mapper/2.0/IMapper.hal b/graphics/mapper/2.0/IMapper.hal index 45661357a..0e18f4267 100644 --- a/graphics/mapper/2.0/IMapper.hal +++ b/graphics/mapper/2.0/IMapper.hal @@ -147,6 +147,9 @@ interface IMapper { * outside of accessRegion is undefined, except that it must not cause * process termination. * + * An accessRegion of all-zeros means the entire buffer. That is, it is + * equivalent to '(0,0)-(buffer width, buffer height)'. + * * data will be filled with a pointer to the locked buffer memory. This * address will represent the top-left corner of the entire buffer, even * if accessRegion does not begin at the top-left corner. |