OSDN Git Service

minigbm/msm: Disable UBWC for video gtest
[android-x86/external-minigbm.git] / mediatek.c
index 6d77970..fd79521 100644 (file)
@@ -49,19 +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, &LINEAR_METADATA,
-                           BO_USE_SW_MASK | BO_USE_LINEAR | BO_USE_PROTECTED);
-       /*
-        * 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);
@@ -72,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);
@@ -98,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;
@@ -112,9 +120,31 @@ 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));
+       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);
@@ -140,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);
@@ -152,8 +181,8 @@ 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;
        }
@@ -234,19 +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
-               /* MT8183 camera and decoder subsystems require NV12. */
-               if (use_flags &
-                   (BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE | BO_USE_HW_VIDEO_DECODER))
+               // 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;