Skip to content

Commit 6ca469e

Browse files
committed
Revert "drm/exynos/decon5433: implement frame counter"
This reverts commit 0586feb This patch makes it to need get_vblank_counter callback in crtc to get frame counter from decon driver. However, drm_dev->max_vblank_count is a member unique to vendor's DRM driver but in case of ARM DRM, some CRTC devices don't provide the frame counter value. As a result, this patch made extension and clone mode not working. Instead of this patch, we may need separated max_vblank_count which belongs to each CRTC device, or need to implement frame counter emulation for them who don't support HW frame counter. Signed-off-by: Inki Dae <inki.dae@samsung.com>
1 parent 8727b23 commit 6ca469e

File tree

3 files changed

+0
-21
lines changed

3 files changed

+0
-21
lines changed

drivers/gpu/drm/exynos/exynos5433_drm_decon.c

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -164,13 +164,6 @@ static u32 decon_get_frame_count(struct decon_context *ctx, bool end)
164164
return frm;
165165
}
166166

167-
static u32 decon_get_vblank_counter(struct exynos_drm_crtc *crtc)
168-
{
169-
struct decon_context *ctx = crtc->ctx;
170-
171-
return decon_get_frame_count(ctx, false);
172-
}
173-
174167
static void decon_setup_trigger(struct decon_context *ctx)
175168
{
176169
if (!ctx->crtc->i80_mode && !(ctx->out_type & I80_HW_TRG))
@@ -536,7 +529,6 @@ static const struct exynos_drm_crtc_ops decon_crtc_ops = {
536529
.disable = decon_disable,
537530
.enable_vblank = decon_enable_vblank,
538531
.disable_vblank = decon_disable_vblank,
539-
.get_vblank_counter = decon_get_vblank_counter,
540532
.atomic_begin = decon_atomic_begin,
541533
.update_plane = decon_update_plane,
542534
.disable_plane = decon_disable_plane,
@@ -554,7 +546,6 @@ static int decon_bind(struct device *dev, struct device *master, void *data)
554546
int ret;
555547

556548
ctx->drm_dev = drm_dev;
557-
drm_dev->max_vblank_count = 0xffffffff;
558549

559550
for (win = ctx->first_win; win < WINDOWS_NR; win++) {
560551
ctx->configs[win].pixel_formats = decon_formats;

drivers/gpu/drm/exynos/exynos_drm_crtc.c

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -162,16 +162,6 @@ static void exynos_drm_crtc_disable_vblank(struct drm_crtc *crtc)
162162
exynos_crtc->ops->disable_vblank(exynos_crtc);
163163
}
164164

165-
static u32 exynos_drm_crtc_get_vblank_counter(struct drm_crtc *crtc)
166-
{
167-
struct exynos_drm_crtc *exynos_crtc = to_exynos_crtc(crtc);
168-
169-
if (exynos_crtc->ops->get_vblank_counter)
170-
return exynos_crtc->ops->get_vblank_counter(exynos_crtc);
171-
172-
return 0;
173-
}
174-
175165
static const struct drm_crtc_funcs exynos_crtc_funcs = {
176166
.set_config = drm_atomic_helper_set_config,
177167
.page_flip = drm_atomic_helper_page_flip,
@@ -181,7 +171,6 @@ static const struct drm_crtc_funcs exynos_crtc_funcs = {
181171
.atomic_destroy_state = drm_atomic_helper_crtc_destroy_state,
182172
.enable_vblank = exynos_drm_crtc_enable_vblank,
183173
.disable_vblank = exynos_drm_crtc_disable_vblank,
184-
.get_vblank_counter = exynos_drm_crtc_get_vblank_counter,
185174
};
186175

187176
struct exynos_drm_crtc *exynos_drm_crtc_create(struct drm_device *drm_dev,

drivers/gpu/drm/exynos/exynos_drm_drv.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,6 @@ struct exynos_drm_crtc_ops {
135135
void (*disable)(struct exynos_drm_crtc *crtc);
136136
int (*enable_vblank)(struct exynos_drm_crtc *crtc);
137137
void (*disable_vblank)(struct exynos_drm_crtc *crtc);
138-
u32 (*get_vblank_counter)(struct exynos_drm_crtc *crtc);
139138
enum drm_mode_status (*mode_valid)(struct exynos_drm_crtc *crtc,
140139
const struct drm_display_mode *mode);
141140
bool (*mode_fixup)(struct exynos_drm_crtc *crtc,

0 commit comments

Comments
 (0)