Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(69)

Side by Side Diff: ui/ozone/platform/drm/gpu/gbm_buffer_base.cc

Issue 1422563002: [Ozone] Enables overlay render format setting path and by default use UYVY (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/ozone/platform/drm/gpu/gbm_buffer_base.h" 5 #include "ui/ozone/platform/drm/gpu/gbm_buffer_base.h"
6 6
7 #include <gbm.h> 7 #include <gbm.h>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "ui/ozone/platform/drm/gpu/drm_device.h" 10 #include "ui/ozone/platform/drm/gpu/drm_device.h"
11 11
12 namespace ui { 12 namespace ui {
13 13
14 namespace { 14 namespace {
15 15
16 // Pixel configuration for the current buffer format. 16 // Pixel configuration for the current buffer format.
17 // TODO(dnicoara) These will need to change once we query the hardware for 17 // TODO(dnicoara) These will need to change once we query the hardware for
18 // supported configurations. 18 // supported configurations.
19 const uint8_t kColorDepth = 24; 19 const uint8_t kColorDepth = 24;
20 const uint8_t kPixelDepth = 32; 20 const uint8_t kPixelDepth = 32;
21 21
22 } // namespace 22 } // namespace
23 23
24 GbmBufferBase::GbmBufferBase(const scoped_refptr<DrmDevice>& drm, 24 GbmBufferBase::GbmBufferBase(const scoped_refptr<DrmDevice>& drm,
25 gbm_bo* bo, 25 gbm_bo* bo,
26 bool scanout) 26 bool scanout)
27 : drm_(drm), bo_(bo) { 27 : drm_(drm), bo_(bo) {
28 if (scanout) { 28 if (scanout) {
29 if (!drm_->AddFramebuffer(gbm_bo_get_width(bo), gbm_bo_get_height(bo),
30 kColorDepth, kPixelDepth, gbm_bo_get_stride(bo),
31 gbm_bo_get_handle(bo).u32, &framebuffer_)) {
32 PLOG(ERROR) << "Failed to register buffer";
33 return;
34 }
35
36 fb_pixel_format_ = gbm_bo_get_format(bo); 29 fb_pixel_format_ = gbm_bo_get_format(bo);
37 if (fb_pixel_format_ == GBM_FORMAT_ARGB8888) 30 if (fb_pixel_format_ == GBM_FORMAT_ARGB8888)
alexst (slow to review) 2015/10/22 13:00:19 Please refactor this into a switch statement, ther
william.xie1 2015/10/22 14:18:10 Done.
38 fb_pixel_format_ = GBM_FORMAT_XRGB8888; 31 fb_pixel_format_ = GBM_FORMAT_XRGB8888;
39 32
40 // For now, we always create a frame buffer of 24 bit color depth and 33 if (fb_pixel_format_ == GBM_FORMAT_XRGB8888) {
41 // support only XRGB format. 34 if (!drm_->AddFramebuffer(gbm_bo_get_width(bo), gbm_bo_get_height(bo),
42 DCHECK(fb_pixel_format_ == GBM_FORMAT_XRGB8888); 35 kColorDepth, kPixelDepth, gbm_bo_get_stride(bo),
36 gbm_bo_get_handle(bo).u32, &framebuffer_)) {
37 PLOG(ERROR) << "Failed to register buffer";
38 return;
39 }
40 } else if (fb_pixel_format_ == GBM_FORMAT_UYVY) {
41 uint32_t handles[4] = {0};
42 handles[0] = gbm_bo_get_handle(bo).u32;
43 uint32_t strides[4] = {0};
44 strides[0] = gbm_bo_get_stride(bo);
45 uint32_t offsets[4] = {0};
46
47 if (!drm_->AddFramebuffer2(gbm_bo_get_width(bo), gbm_bo_get_height(bo),
48 fb_pixel_format_, handles, strides, offsets,
49 &framebuffer_, 0)) {
50 PLOG(ERROR) << "Failed to register buffer";
51 return;
52 }
53 } else {
54 NOTIMPLEMENTED();
55 }
43 } 56 }
44 } 57 }
45 58
46 GbmBufferBase::~GbmBufferBase() { 59 GbmBufferBase::~GbmBufferBase() {
47 if (framebuffer_) 60 if (framebuffer_)
48 drm_->RemoveFramebuffer(framebuffer_); 61 drm_->RemoveFramebuffer(framebuffer_);
49 } 62 }
50 63
51 uint32_t GbmBufferBase::GetFramebufferId() const { 64 uint32_t GbmBufferBase::GetFramebufferId() const {
52 return framebuffer_; 65 return framebuffer_;
53 } 66 }
54 67
55 uint32_t GbmBufferBase::GetHandle() const { 68 uint32_t GbmBufferBase::GetHandle() const {
56 return gbm_bo_get_handle(bo_).u32; 69 return gbm_bo_get_handle(bo_).u32;
57 } 70 }
58 71
59 gfx::Size GbmBufferBase::GetSize() const { 72 gfx::Size GbmBufferBase::GetSize() const {
60 return gfx::Size(gbm_bo_get_width(bo_), gbm_bo_get_height(bo_)); 73 return gfx::Size(gbm_bo_get_width(bo_), gbm_bo_get_height(bo_));
61 } 74 }
62 75
63 uint32_t GbmBufferBase::GetFramebufferPixelFormat() const { 76 uint32_t GbmBufferBase::GetFramebufferPixelFormat() const {
64 return fb_pixel_format_; 77 return fb_pixel_format_;
65 } 78 }
66 79
67 bool GbmBufferBase::RequiresGlFinish() const { 80 bool GbmBufferBase::RequiresGlFinish() const {
68 return !drm_->is_primary_device(); 81 return !drm_->is_primary_device();
69 } 82 }
70 83
71 } // namespace ui 84 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698