OSDN Git Service

resolve merge conflicts of 42ca6159e52c5a2217825da02a26cf504998301b to stage-aosp...
[android-x86/external-minigbm.git] / mediatek.c
index 7282437..fd79521 100644 (file)
@@ -49,18 +49,12 @@ static int mediatek_init(struct driver *drv)
        struct format_metadata metadata;
 
        drv_add_combinations(drv, render_target_formats, ARRAY_SIZE(render_target_formats),
-                            &LINEAR_METADATA, BO_USE_RENDER_MASK);
+                            &LINEAR_METADATA, BO_USE_RENDER_MASK | BO_USE_SCANOUT);
 
        drv_add_combinations(drv, texture_source_formats, ARRAY_SIZE(texture_source_formats),
                             &LINEAR_METADATA, BO_USE_TEXTURE_MASK);
 
-       drv_add_combination(drv, DRM_FORMAT_R8, &metadata, BO_USE_SW_MASK | BO_USE_LINEAR);
-       /*
-        * Chrome uses DMA-buf mmap to write to YV12 buffers, which are then accessed by the
-        * Video Encoder Accelerator (VEA). It could also support NV12 potentially in the future.
-        */
-       drv_modify_combination(drv, DRM_FORMAT_YVU420, &LINEAR_METADATA, BO_USE_HW_VIDEO_ENCODER);
-       drv_modify_combination(drv, DRM_FORMAT_NV12, &LINEAR_METADATA, BO_USE_HW_VIDEO_ENCODER);
+       drv_add_combination(drv, DRM_FORMAT_R8, &LINEAR_METADATA, BO_USE_SW_MASK | BO_USE_LINEAR);
 
        /* Android CTS tests require this. */
        drv_add_combination(drv, DRM_FORMAT_BGR888, &LINEAR_METADATA, BO_USE_SW_MASK);
@@ -71,20 +65,35 @@ static int mediatek_init(struct driver *drv)
        metadata.modifier = DRM_FORMAT_MOD_LINEAR;
        drv_modify_combination(drv, DRM_FORMAT_YVU420, &metadata, BO_USE_HW_VIDEO_DECODER);
        drv_modify_combination(drv, DRM_FORMAT_YVU420_ANDROID, &metadata, BO_USE_HW_VIDEO_DECODER);
+#ifdef MTK_MT8183
+       // TODO(hiroh): Switch to use NV12 for video decoder on MT8173 as well.
        drv_modify_combination(drv, DRM_FORMAT_NV12, &metadata, BO_USE_HW_VIDEO_DECODER);
+#endif
+
+       /*
+        * R8 format is used for Android's HAL_PIXEL_FORMAT_BLOB for input/output from
+        * hardware decoder/encoder.
+        */
+       drv_modify_combination(drv, DRM_FORMAT_R8, &metadata,
+                              BO_USE_HW_VIDEO_DECODER | BO_USE_HW_VIDEO_ENCODER |
+                                  BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE);
+
+       /* NV12 format for encoding and display. */
+       drv_modify_combination(drv, DRM_FORMAT_NV12, &metadata,
+                              BO_USE_SCANOUT | BO_USE_HW_VIDEO_ENCODER | BO_USE_CAMERA_READ |
+                                  BO_USE_CAMERA_WRITE);
 
 #ifdef MTK_MT8183
        /* Only for MT8183 Camera subsystem */
-       drv_modify_combination(drv, DRM_FORMAT_NV12, &metadata,
-                              BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE);
        drv_modify_combination(drv, DRM_FORMAT_NV21, &metadata,
                               BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE);
        drv_modify_combination(drv, DRM_FORMAT_YUYV, &metadata,
                               BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE);
        drv_modify_combination(drv, DRM_FORMAT_YVU420, &metadata,
                               BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE);
-       drv_modify_combination(drv, DRM_FORMAT_R8, &metadata,
-                              BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE);
+       /* Private formats for private reprocessing in camera */
+       drv_add_combination(drv, DRM_FORMAT_MTISP_SXYZW10, &metadata,
+                           BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE | BO_USE_SW_MASK);
 #endif
 
        return drv_modify_linear_combinations(drv);
@@ -97,7 +106,7 @@ static int mediatek_bo_create_with_modifiers(struct bo *bo, uint32_t width, uint
        int ret;
        size_t plane;
        uint32_t stride;
-       struct drm_mtk_gem_create gem_create;
+       struct drm_mtk_gem_create gem_create = { 0 };
 
        if (!drv_has_modifier(modifiers, count, DRM_FORMAT_MOD_LINEAR)) {
                errno = EINVAL;
@@ -111,10 +120,32 @@ static int mediatek_bo_create_with_modifiers(struct bo *bo, uint32_t width, uint
         */
        stride = drv_stride_from_format(format, width, 0);
        stride = ALIGN(stride, 64);
-       drv_bo_from_format(bo, stride, height, format);
 
-       memset(&gem_create, 0, sizeof(gem_create));
-       gem_create.size = bo->total_size;
+       if (bo->meta.use_flags & BO_USE_HW_VIDEO_ENCODER) {
+               uint32_t aligned_height = ALIGN(height, 32);
+               uint32_t padding[DRV_MAX_PLANES] = { 0 };
+
+               for (plane = 0; plane < bo->meta.num_planes; ++plane) {
+                       uint32_t plane_stride = drv_stride_from_format(format, stride, plane);
+                       padding[plane] = plane_stride *
+                                        (32 / drv_vertical_subsampling_from_format(format, plane));
+               }
+
+               drv_bo_from_format_and_padding(bo, stride, aligned_height, format, padding);
+       } else {
+#ifdef MTK_MT8183
+               /*
+                * JPEG Encoder Accelerator requires 16x16 alignment. We want the buffer
+                * from camera can be put in JEA directly so align the height to 16
+                * bytes.
+                */
+               if (format == DRM_FORMAT_NV12)
+                       height = ALIGN(height, 16);
+#endif
+               drv_bo_from_format(bo, stride, height, format);
+       }
+
+       gem_create.size = bo->meta.total_size;
 
        ret = drmIoctl(bo->drv->fd, DRM_IOCTL_MTK_GEM_CREATE, &gem_create);
        if (ret) {
@@ -122,7 +153,7 @@ static int mediatek_bo_create_with_modifiers(struct bo *bo, uint32_t width, uint
                return -errno;
        }
 
-       for (plane = 0; plane < bo->num_planes; plane++)
+       for (plane = 0; plane < bo->meta.num_planes; plane++)
                bo->handles[plane].u32 = gem_create.handle;
 
        return 0;
@@ -139,10 +170,9 @@ static int mediatek_bo_create(struct bo *bo, uint32_t width, uint32_t height, ui
 static void *mediatek_bo_map(struct bo *bo, struct vma *vma, size_t plane, uint32_t map_flags)
 {
        int ret, prime_fd;
-       struct drm_mtk_gem_map_off gem_map;
+       struct drm_mtk_gem_map_off gem_map = { 0 };
        struct mediatek_private_map_data *priv;
 
-       memset(&gem_map, 0, sizeof(gem_map));
        gem_map.handle = bo->handles[0].u32;
 
        ret = drmIoctl(bo->drv->fd, DRM_IOCTL_MTK_GEM_MAP_OFFSET, &gem_map);
@@ -151,23 +181,23 @@ static void *mediatek_bo_map(struct bo *bo, struct vma *vma, size_t plane, uint3
                return MAP_FAILED;
        }
 
-       ret = drmPrimeHandleToFD(bo->drv->fd, gem_map.handle, DRM_CLOEXEC, &prime_fd);
-       if (ret) {
+       prime_fd = drv_bo_get_plane_fd(bo, 0);
+       if (prime_fd < 0) {
                drv_log("Failed to get a prime fd\n");
                return MAP_FAILED;
        }
 
-       void *addr = mmap(0, bo->total_size, drv_get_prot(map_flags), MAP_SHARED, bo->drv->fd,
+       void *addr = mmap(0, bo->meta.total_size, drv_get_prot(map_flags), MAP_SHARED, bo->drv->fd,
                          gem_map.offset);
 
-       vma->length = bo->total_size;
+       vma->length = bo->meta.total_size;
 
        priv = calloc(1, sizeof(*priv));
        priv->prime_fd = prime_fd;
        vma->priv = priv;
 
-       if (bo->use_flags & BO_USE_RENDERSCRIPT) {
-               priv->cached_addr = calloc(1, bo->total_size);
+       if (bo->meta.use_flags & BO_USE_RENDERSCRIPT) {
+               priv->cached_addr = calloc(1, bo->meta.total_size);
                priv->gem_addr = addr;
                addr = priv->cached_addr;
        }
@@ -213,7 +243,7 @@ static int mediatek_bo_invalidate(struct bo *bo, struct mapping *mapping)
                        drv_log("poll prime_fd failed\n");
 
                if (priv->cached_addr)
-                       memcpy(priv->cached_addr, priv->gem_addr, bo->total_size);
+                       memcpy(priv->cached_addr, priv->gem_addr, bo->meta.total_size);
        }
 
        return 0;
@@ -223,7 +253,7 @@ static int mediatek_bo_flush(struct bo *bo, struct mapping *mapping)
 {
        struct mediatek_private_map_data *priv = mapping->vma->priv;
        if (priv && priv->cached_addr && (mapping->vma->map_flags & BO_MAP_WRITE))
-               memcpy(priv->gem_addr, priv->cached_addr, bo->total_size);
+               memcpy(priv->gem_addr, priv->cached_addr, bo->meta.total_size);
 
        return 0;
 }
@@ -233,18 +263,28 @@ static uint32_t mediatek_resolve_format(struct driver *drv, uint32_t format, uin
        switch (format) {
        case DRM_FORMAT_FLEX_IMPLEMENTATION_DEFINED:
 #ifdef MTK_MT8183
-               /* Only for MT8183 Camera subsystem requires NV12. */
-               if (use_flags & (BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE))
-                       return DRM_FORMAT_NV12;
+               /* Only MT8183 Camera subsystem offers private reprocessing
+                * capability. CAMERA_READ indicates the buffer is intended for
+                * reprocessing and hence given the private format for MTK. */
+               if (use_flags & BO_USE_CAMERA_READ)
+                       return DRM_FORMAT_MTISP_SXYZW10;
 #endif
+               if (use_flags & BO_USE_CAMERA_WRITE)
+                       return DRM_FORMAT_NV12;
+
                /*HACK: See b/28671744 */
                return DRM_FORMAT_XBGR8888;
        case DRM_FORMAT_FLEX_YCbCr_420_888:
 #ifdef MTK_MT8183
-               /* Only for MT8183 Camera subsystem requires NV12 */
-               if (use_flags & (BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE))
+               // TODO(hiroh): Switch to use NV12 for video decoder on MT8173 as well.
+               if (use_flags & (BO_USE_HW_VIDEO_DECODER)) {
                        return DRM_FORMAT_NV12;
+               }
 #endif
+               if (use_flags &
+                   (BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE | BO_USE_HW_VIDEO_ENCODER)) {
+                       return DRM_FORMAT_NV12;
+               }
                return DRM_FORMAT_YVU420;
        default:
                return format;