summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2017-08-22 17:01:35 +0200
committerwm4 <wm4@nowhere>2017-08-22 17:01:35 +0200
commit03cf150ff3516789d581214177f291d46310aaf4 (patch)
tree2f352bf9c6d36e10001accdb72cef2d7683ab04a
parentd67aa6da6bbd3cba9eabd12cda4703821fecc7a1 (diff)
downloadmpv-03cf150ff3516789d581214177f291d46310aaf4.tar.bz2
mpv-03cf150ff3516789d581214177f291d46310aaf4.tar.xz
video: redo video equalizer option handling
I really wouldn't care much about this, but some parts of the core code are under HAVE_GPL, so there's some need to get rid of it. Simply turn the video equalizer from its current fine-grained handling with vf/vo fallbacks into global options. This makes updating them much simpler. This removes any possibility of applying video equalizers in filters, which affects vf_scale, and the previously removed vf_eq. Not a big loss, since the preferred VOs have this builtin. Remove video equalizer handling from vo_direct3d, vo_sdl, vo_vaapi, and vo_xv. I'm not going to waste my time on these legacy VOs. vo.eq_opts_cache exists _only_ to send a VOCTRL_SET_EQUALIZER, which exists _only_ to trigger a redraw. This seems silly, but for now I feel like this is less of a pain. The rest of the equalizer using code is self-updating. See commit 96b906a51d5 for how some video equalizer code was GPL only. Some command line option names and ranges can probably be traced back to a GPL only committer, but we don't consider these copyrightable.
-rw-r--r--options/options.c15
-rw-r--r--options/options.h8
-rw-r--r--player/command.c38
-rw-r--r--player/video.c75
-rw-r--r--video/csputils.c67
-rw-r--r--video/csputils.h36
-rw-r--r--video/filter/vf.h7
-rw-r--r--video/filter/vf_scale.c19
-rw-r--r--video/out/opengl/video.c23
-rw-r--r--video/out/opengl/video.h3
-rw-r--r--video/out/vo.c11
-rw-r--r--video/out/vo.h1
-rw-r--r--video/out/vo_direct3d.c19
-rw-r--r--video/out/vo_opengl.c19
-rw-r--r--video/out/vo_opengl_cb.c37
-rw-r--r--video/out/vo_sdl.c85
-rw-r--r--video/out/vo_vaapi.c92
-rw-r--r--video/out/vo_vdpau.c40
-rw-r--r--video/out/vo_xv.c9
-rw-r--r--video/sws_utils.c31
-rw-r--r--video/sws_utils.h4
-rw-r--r--video/vdpau_mixer.c9
-rw-r--r--video/vdpau_mixer.h4
23 files changed, 83 insertions, 569 deletions
diff --git a/options/options.c b/options/options.c
index b35dd9b265..258cba0d31 100644
--- a/options/options.c
+++ b/options/options.c
@@ -572,15 +572,7 @@ const m_option_t mp_opts[] = {
.deprecation_message = "use Lua scripting instead"),
OPT_FLOAT("heartbeat-interval", heartbeat_interval, CONF_MIN, 0),
-#if HAVE_GPL
- OPT_INTRANGE("brightness", gamma_brightness, 0, -100, 100),
- OPT_INTRANGE("saturation", gamma_saturation, 0, -100, 100),
- OPT_INTRANGE("contrast", gamma_contrast, 0, -100, 100),
- OPT_INTRANGE("hue", gamma_hue, 0, -100, 100),
- OPT_INTRANGE("gamma", gamma_gamma, 0, -100, 100),
- OPT_CHOICE_C("video-output-levels", video_output_levels, 0,
- mp_csp_levels_names),
-#endif
+ OPT_SUBSTRUCT("", video_equalizer, mp_csp_equalizer_conf, 0),
OPT_FLAG("use-filedir-conf", use_filedir_conf, 0),
OPT_CHOICE("osd-level", osd_level, 0,
@@ -856,11 +848,6 @@ const struct MPOpts mp_default_opts = {
.heartbeat_interval = 30.0,
.stop_screensaver = 1,
.cursor_autohide_delay = 1000,
- .gamma_gamma = 0,
- .gamma_brightness = 0,
- .gamma_contrast = 0,
- .gamma_saturation = 0,
- .gamma_hue = 0,
.video_osd = 1,
.osd_level = 1,
.osd_duration = 1000,
diff --git a/options/options.h b/options/options.h
index 9ea09a83b3..ecb38e3995 100644
--- a/options/options.h
+++ b/options/options.h
@@ -119,13 +119,7 @@ typedef struct MPOpts {
char *media_title;
int force_rgba_osd;
- // ranges -100 - 100, 1000 if the vo default should be used
- int gamma_gamma;
- int gamma_brightness;
- int gamma_contrast;
- int gamma_saturation;
- int gamma_hue;
- int video_output_levels;
+ struct mp_csp_equalizer_opts *video_equalizer;
int stop_screensaver;
int cursor_autohide_delay;
diff --git a/player/command.c b/player/command.c
index 80c0c970fa..d94f819992 100644
--- a/player/command.c
+++ b/player/command.c
@@ -2613,35 +2613,6 @@ static int mp_property_frame_count(void *ctx, struct m_property *prop,
return m_property_int_ro(action, arg, frames);
}
-#if HAVE_GPL
-static int mp_property_video_color(void *ctx, struct m_property *prop,
- int action, void *arg)
-{
- const char *name = prop->priv ? prop->priv : prop->name;
- MPContext *mpctx = ctx;
- if (!mpctx->vo_chain)
- return mp_property_generic_option(mpctx, prop, action, arg);
-
- switch (action) {
- case M_PROPERTY_SET: {
- if (video_set_colors(mpctx->vo_chain, name, *(int *) arg) <= 0)
- return M_PROPERTY_UNAVAILABLE;
- break;
- }
- case M_PROPERTY_GET:
- if (video_get_colors(mpctx->vo_chain, name, (int *)arg) <= 0)
- return M_PROPERTY_UNAVAILABLE;
- // Write new value to option variable
- mp_property_generic_option(mpctx, prop, M_PROPERTY_SET, arg);
- return M_PROPERTY_OK;
- case M_PROPERTY_GET_NEUTRAL:
- *(int *)arg = 0;
- return M_PROPERTY_OK;
- }
- return mp_property_generic_option(mpctx, prop, action, arg);
-}
-#endif
-
/// Video codec tag (RO)
static int mp_property_video_format(void *ctx, struct m_property *prop,
int action, void *arg)
@@ -4041,15 +4012,6 @@ static const struct m_property mp_properties_base[] = {
{"ontop", mp_property_ontop},
{"border", mp_property_border},
{"on-all-workspaces", mp_property_all_workspaces},
-#if HAVE_GPL
- {"gamma", mp_property_video_color},
- {"brightness", mp_property_video_color},
- {"contrast", mp_property_video_color},
- {"saturation", mp_property_video_color},
- {"hue", mp_property_video_color},
- {"video-output-levels", mp_property_video_color,
- .priv = (void *)"output-levels"},
-#endif
{"video-out-params", mp_property_vo_imgparams},
{"video-dec-params", mp_property_dec_imgparams},
{"video-params", mp_property_vd_imgparams},
diff --git a/player/video.c b/player/video.c
index 3a9941d8e0..1fdf69042e 100644
--- a/player/video.c
+++ b/player/video.c
@@ -70,37 +70,6 @@ static const char av_desync_help_text[] =
"position will not match to the video (see A-V status field).\n"
"\n";
-#if HAVE_GPL
-int video_set_colors(struct vo_chain *vo_c, const char *item, int value)
-{
- vf_equalizer_t data;
-
- data.item = item;
- data.value = value;
-
- MP_VERBOSE(vo_c, "set video colors %s=%d \n", item, value);
- if (video_vf_vo_control(vo_c, VFCTRL_SET_EQUALIZER, &data) == CONTROL_TRUE)
- return 1;
- MP_VERBOSE(vo_c, "Video attribute '%s' is not supported by selected vo.\n",
- item);
- return 0;
-}
-
-int video_get_colors(struct vo_chain *vo_c, const char *item, int *value)
-{
- vf_equalizer_t data;
-
- data.item = item;
-
- MP_VERBOSE(vo_c, "get video colors %s \n", item);
- if (video_vf_vo_control(vo_c, VFCTRL_GET_EQUALIZER, &data) == CONTROL_TRUE) {
- *value = data.value;
- return 1;
- }
- return 0;
-}
-#endif
-
// Send a VCTRL, or if it doesn't work, translate it to a VOCTRL and try the VO.
int video_vf_vo_control(struct vo_chain *vo_c, int vf_cmd, void *data)
{
@@ -110,26 +79,6 @@ int video_vf_vo_control(struct vo_chain *vo_c, int vf_cmd, void *data)
return r;
}
- switch (vf_cmd) {
- case VFCTRL_SET_EQUALIZER: {
- vf_equalizer_t *eq = data;
- if (!vo_c->vo->config_ok)
- return CONTROL_FALSE; // vo not configured?
- struct voctrl_set_equalizer_args param = {
- eq->item, eq->value
- };
- return vo_control(vo_c->vo, VOCTRL_SET_EQUALIZER, &param) == VO_TRUE;
- }
- case VFCTRL_GET_EQUALIZER: {
- vf_equalizer_t *eq = data;
- if (!vo_c->vo->config_ok)
- return CONTROL_FALSE; // vo not configured?
- struct voctrl_get_equalizer_args param = {
- eq->item, &eq->value
- };
- return vo_control(vo_c->vo, VOCTRL_GET_EQUALIZER, &param) == VO_TRUE;
- }
- }
return CONTROL_UNKNOWN;
}
@@ -980,28 +929,6 @@ static void update_av_diff(struct MPContext *mpctx, double offset)
}
}
-static void init_vo(struct MPContext *mpctx)
-{
- struct MPOpts *opts = mpctx->opts;
- struct vo_chain *vo_c = mpctx->vo_chain;
-
-#if HAVE_GPL
- if (opts->gamma_gamma != 0)
- video_set_colors(vo_c, "gamma", opts->gamma_gamma);
- if (opts->gamma_brightness != 0)
- video_set_colors(vo_c, "brightness", opts->gamma_brightness);
- if (opts->gamma_contrast != 0)
- video_set_colors(vo_c, "contrast", opts->gamma_contrast);
- if (opts->gamma_saturation != 0)
- video_set_colors(vo_c, "saturation", opts->gamma_saturation);
- if (opts->gamma_hue != 0)
- video_set_colors(vo_c, "hue", opts->gamma_hue);
- video_set_colors(vo_c, "output-levels", opts->video_output_levels);
-#endif
-
- mp_notify(mpctx, MPV_EVENT_VIDEO_RECONFIG, NULL);
-}
-
double calc_average_frame_duration(struct MPContext *mpctx)
{
double total = 0;
@@ -1445,7 +1372,7 @@ void write_video(struct MPContext *mpctx)
mpctx->error_playing = MPV_ERROR_VO_INIT_FAILED;
goto error;
}
- init_vo(mpctx);
+ mp_notify(mpctx, MPV_EVENT_VIDEO_RECONFIG, NULL);
}
mpctx->time_frame -= get_relative_time(mpctx);
diff --git a/video/csputils.c b/video/csputils.c
index e9dc8e654d..d9a5c29491 100644
--- a/video/csputils.c
+++ b/video/csputils.c
@@ -31,6 +31,7 @@
#include "mp_image.h"
#include "csputils.h"
+#include "options/m_config.h"
#include "options/m_option.h"
const struct m_opt_choice_alternatives mp_csp_names[] = {
@@ -96,15 +97,6 @@ const struct m_opt_choice_alternatives mp_csp_light_names[] = {
{0}
};
-const char *const mp_csp_equalizer_names[MP_CSP_EQ_COUNT] = {
- "brightness",
- "contrast",
- "hue",
- "saturation",
- "gamma",
- "output-levels",
-};
-
const struct m_opt_choice_alternatives mp_chroma_names[] = {
{"unknown", MP_CHROMA_AUTO},
{"mpeg2/4/h264",MP_CHROMA_LEFT},
@@ -817,9 +809,25 @@ bool mp_colorspace_equal(struct mp_colorspace c1, struct mp_colorspace c2)
c1.sig_peak == c2.sig_peak;
}
+#define OPT_BASE_STRUCT struct mp_csp_equalizer_opts
+
+const struct m_sub_options mp_csp_equalizer_conf = {
+ .opts = (const m_option_t[]) {
+ OPT_INTRANGE("brightness", values[MP_CSP_EQ_BRIGHTNESS], 0, -100, 100),
+ OPT_INTRANGE("saturation", values[MP_CSP_EQ_SATURATION], 0, -100, 100),
+ OPT_INTRANGE("contrast", values[MP_CSP_EQ_CONTRAST], 0, -100, 100),
+ OPT_INTRANGE("hue", values[MP_CSP_EQ_HUE], 0, -100, 100),
+ OPT_INTRANGE("gamma", values[MP_CSP_EQ_GAMMA], 0, -100, 100),
+ OPT_CHOICE_C("video-output-levels", values[MP_CSP_EQ_OUTPUT_LEVELS], 0,
+ mp_csp_levels_names),
+ {0}
+ },
+ .size = sizeof(struct mp_csp_equalizer_opts),
+};
+
// Copy settings from eq into params.
void mp_csp_copy_equalizer_values(struct mp_csp_params *params,
- const struct mp_csp_equalizer *eq)
+ const struct mp_csp_equalizer_opts *eq)
{
params->brightness = eq->values[MP_CSP_EQ_BRIGHTNESS] / 100.0;
params->contrast = (eq->values[MP_CSP_EQ_CONTRAST] + 100) / 100.0;
@@ -829,37 +837,28 @@ void mp_csp_copy_equalizer_values(struct mp_csp_params *params,
params->levels_out = eq->values[MP_CSP_EQ_OUTPUT_LEVELS];
}
-static int find_eq(int capabilities, const char *name)
+struct mp_csp_equalizer_state *mp_csp_equalizer_create(void *ta_parent,
+ struct mpv_global *global)
{
- for (int i = 0; i < MP_CSP_EQ_COUNT; i++) {
- if (strcmp(name, mp_csp_equalizer_names[i]) == 0)
- return ((1 << i) & capabilities) ? i : -1;
- }
- return -1;
+ struct m_config_cache *c = m_config_cache_alloc(ta_parent, global,
+ &mp_csp_equalizer_conf);
+ // The terrible, terrible truth.
+ return (struct mp_csp_equalizer_state *)c;
}
-int mp_csp_equalizer_get(struct mp_csp_equalizer *eq, const char *property,
- int *out_value)
+bool mp_csp_equalizer_state_changed(struct mp_csp_equalizer_state *state)
{
- int index = find_eq(eq->capabilities, property);
- if (index < 0)
- return -1;
-
- *out_value = eq->values[index];
-
- return 0;
+ struct m_config_cache *c = (struct m_config_cache *)state;
+ return m_config_cache_update(c);
}
-int mp_csp_equalizer_set(struct mp_csp_equalizer *eq, const char *property,
- int value)
+void mp_csp_equalizer_state_get(struct mp_csp_equalizer_state *state,
+ struct mp_csp_params *params)
{
- int index = find_eq(eq->capabilities, property);
- if (index < 0)
- return 0;
-
- eq->values[index] = value;
-
- return 1;
+ struct m_config_cache *c = (struct m_config_cache *)state;
+ m_config_cache_update(c);
+ struct mp_csp_equalizer_opts *opts = c->opts;
+ mp_csp_copy_equalizer_values(params, opts);
}
void mp_invert_cmat(struct mp_cmat *out, struct mp_cmat *in)
diff --git a/video/csputils.h b/video/csputils.h
index 2119cca45a..246dfdcfcf 100644
--- a/video/csputils.h
+++ b/video/csputils.h
@@ -185,6 +185,8 @@ enum mp_chroma_location {
extern const struct m_opt_choice_alternatives mp_chroma_names[];
+extern const struct m_sub_options mp_csp_equalizer_conf;
+
enum mp_csp_equalizer_param {
MP_CSP_EQ_BRIGHTNESS,
MP_CSP_EQ_CONTRAST,
@@ -195,28 +197,23 @@ enum mp_csp_equalizer_param {
MP_CSP_EQ_COUNT,
};
-#define MP_CSP_EQ_CAPS_COLORMATRIX \
- ( (1 << MP_CSP_EQ_BRIGHTNESS) \
- | (1 << MP_CSP_EQ_CONTRAST) \
- | (1 << MP_CSP_EQ_HUE) \
- | (1 << MP_CSP_EQ_SATURATION) \
- | (1 << MP_CSP_EQ_OUTPUT_LEVELS) )
-
-#define MP_CSP_EQ_CAPS_GAMMA (1 << MP_CSP_EQ_GAMMA)
-#define MP_CSP_EQ_CAPS_BRIGHTNESS (1 << MP_CSP_EQ_BRIGHTNESS)
-
-extern const char *const mp_csp_equalizer_names[MP_CSP_EQ_COUNT];
-
// Default initialization with 0 is enough, except for the capabilities field
-struct mp_csp_equalizer {
- // Bit field of capabilities. For example (1 << MP_CSP_EQ_HUE) means hue
- // support is available.
- int capabilities;
+struct mp_csp_equalizer_opts {
// Value for each property is in the range [-100, 100].
// 0 is default, meaning neutral or no change.
int values[MP_CSP_EQ_COUNT];
};
+void mp_csp_copy_equalizer_values(struct mp_csp_params *params,
+ const struct mp_csp_equalizer_opts *eq);
+
+struct mpv_global;
+struct mp_csp_equalizer_state *mp_csp_equalizer_create(void *ta_parent,
+ struct mpv_global *global);
+bool mp_csp_equalizer_state_changed(struct mp_csp_equalizer_state *state);
+void mp_csp_equalizer_state_get(struct mp_csp_equalizer_state *state,
+ struct mp_csp_params *params);
+
struct mp_csp_col_xy {
float x, y;
};
@@ -233,13 +230,6 @@ struct mp_csp_primaries {
struct mp_csp_col_xy red, green, blue, white;
};
-void mp_csp_copy_equalizer_values(struct mp_csp_params *params,
- const struct mp_csp_equalizer *eq);
-int mp_csp_equalizer_set(struct mp_csp_equalizer *eq, const char *property,
- int value);
-int mp_csp_equalizer_get(struct mp_csp_equalizer *eq, const char *property,
- int *out_value);
-
enum mp_csp avcol_spc_to_mp_csp(int avcolorspace);
enum mp_csp_levels avcol_range_to_mp_csp_levels(int avrange);
enum mp_csp_prim avcol_pri_to_mp_csp_prim(int avpri);
diff --git a/video/filter/vf.h b/video/filter/vf.h
index 546fd823ad..8a9815078d 100644
--- a/video/filter/vf.h
+++ b/video/filter/vf.h
@@ -141,15 +141,8 @@ struct vf_chain {
void *wakeup_callback_ctx;
};
-typedef struct vf_seteq {
- const char *item;
- int value;
-} vf_equalizer_t;
-
enum vf_ctrl {
VFCTRL_SEEK_RESET = 1, // reset on picture and PTS discontinuities
- VFCTRL_SET_EQUALIZER, // set color options (brightness,contrast etc)
- VFCTRL_GET_EQUALIZER, // get color options (brightness,contrast etc)
#if HAVE_GPL
VFCTRL_SET_DEINTERLACE, // Set deinterlacing status
VFCTRL_GET_DEINTERLACE, // Get deinterlacing status
diff --git a/video/filter/vf_scale.c b/video/filter/vf_scale.c
index 0759d28723..9c74db363d 100644
--- a/video/filter/vf_scale.c
+++ b/video/filter/vf_scale.c
@@ -202,24 +202,6 @@ static struct mp_image *filter(struct vf_instance *vf, struct mp_image *mpi)
return dmpi;
}
-static int control(struct vf_instance *vf, int request, void *data)
-{
- struct mp_sws_context *sws = vf->priv->sws;
-
- switch (request) {
- case VFCTRL_GET_EQUALIZER:
- if (mp_sws_get_vf_equalizer(sws, data) < 1)
- break;
- return CONTROL_TRUE;
- case VFCTRL_SET_EQUALIZER:
- if (mp_sws_set_vf_equalizer(sws, data) < 1)
- break;
- return CONTROL_TRUE;
- }
-
- return CONTROL_UNKNOWN;
-}
-
static int query_format(struct vf_instance *vf, unsigned int fmt)
{
if (IMGFMT_IS_HWACCEL(fmt) || sws_isSupportedInput(imgfmt2pixfmt(fmt)) < 1)
@@ -236,7 +218,6 @@ static int vf_open(vf_instance_t *vf)
vf->reconfig = reconfig;
vf->filter = filter;
vf->query_format = query_format;
- vf->control = control;
vf->uninit = uninit;
vf->priv->sws = mp_sws_alloc(vf);
vf->priv->sws->log = vf->log;
diff --git a/video/out/opengl/video.c b/video/out/opengl/video.c
index bbf88e2747..f95222081c 100644
--- a/video/out/opengl/video.c
+++ b/video/out/opengl/video.c
@@ -242,7 +242,7 @@ struct gl_video {
// state for configured scalers
struct scaler scaler[SCALER_COUNT];
- struct mp_csp_equalizer video_eq;
+ struct mp_csp_equalizer_state *video_eq;
struct mp_rect src_rect; // displayed part of the source video
struct mp_rect dst_rect; // video rectangle on output window
@@ -848,13 +848,6 @@ static void init_video(struct gl_video *p)
mp_image_params_guess_csp(&p->image_params);
- int eq_caps = MP_CSP_EQ_CAPS_GAMMA;
- if (p->image_params.color.space != MP_CSP_BT_2020_C)
- eq_caps |= MP_CSP_EQ_CAPS_COLORMATRIX;
- if (p->image_params.color.space == MP_CSP_XYZ)
- eq_caps |= MP_CSP_EQ_CAPS_BRIGHTNESS;
- p->video_eq.capabilities = eq_caps;
-
av_lfg_init(&p->lfg, 1);
debug_check_gl(p, "before video texture creation");
@@ -2173,7 +2166,7 @@ static void pass_convert_yuv(struct gl_video *p)
struct mp_csp_params cparams = MP_CSP_PARAMS_DEFAULTS;
cparams.gray = p->is_gray;
mp_csp_set_image_params(&cparams, &p->image_params);
- mp_csp_copy_equalizer_values(&cparams, &p->video_eq);
+ mp_csp_equalizer_state_get(p->video_eq, &cparams);
p->user_gamma = 1.0 / (cparams.gamma * p->opts.gamma);
pass_describe(p, "color conversion");
@@ -3582,6 +3575,7 @@ struct gl_video *gl_video_init(struct ra *ra, struct mp_log *log,
.global = g,
.log = log,
.sc = gl_sc_create(ra, g, log),
+ .video_eq = mp_csp_equalizer_create(p, g),
.opts_cache = m_config_cache_alloc(p, g, &gl_video_conf),
};
// make sure this variable is initialized to *something*
@@ -3674,16 +3668,6 @@ void gl_video_configure_queue(struct gl_video *p, struct vo *vo)
vo_set_queue_params(vo, 0, queue_size);
}
-struct mp_csp_equalizer *gl_video_eq_ptr(struct gl_video *p)
-{
- return &p->video_eq;
-}
-
-// Call when the mp_csp_equalizer returned by gl_video_eq_ptr() was changed.
-void gl_video_eq_update(struct gl_video *p)
-{
-}
-
static int validate_scaler_opt(struct mp_log *log, const m_option_t *opt,
struct bstr name, struct bstr param)
{
@@ -3758,7 +3742,6 @@ void gl_video_set_ambient_lux(struct gl_video *p, int lux)
float gamma = gl_video_scale_ambient_lux(16.0, 64.0, 2.40, 1.961, lux);
MP_VERBOSE(p, "ambient light changed: %dlux (gamma: %f)\n", lux, gamma);
p->opts.gamma = MPMIN(1.0, 1.961 / gamma);
- gl_video_eq_update(p);
}
}
diff --git a/video/out/opengl/video.h b/video/out/opengl/video.h
index 0db9773712..e5e7ae70ec 100644
--- a/video/out/opengl/video.h
+++ b/video/out/opengl/video.h
@@ -165,9 +165,6 @@ void gl_video_resize(struct gl_video *p,
void gl_video_set_fb_depth(struct gl_video *p, int fb_depth);
struct voctrl_performance_data;
void gl_video_perfdata(struct gl_video *p, struct voctrl_performance_data *out);
-struct mp_csp_equalizer;
-struct mp_csp_equalizer *gl_video_eq_ptr(struct gl_video *p);
-void gl_video_eq_update(struct gl_video *p);
void gl_video_set_clear_color(struct gl_video *p, struct m_color color);
void gl_video_set_osd_pts(struct gl_video *p, double pts);
bool gl_video_check_osd_change(struct gl_video *p, struct mp_osd_res *osd,
diff --git a/video/out/vo.c b/video/out/vo.c
index 217baac632..f9c5d04e24 100644
--- a/video/out/vo.c
+++ b/video/out/vo.c
@@ -228,6 +228,12 @@ static void update_opts(void *p)
if (vo->driver->control)
vo->driver->control(vo, VOCTRL_UPDATE_RENDER_OPTS, NULL);
}
+
+ if (m_config_cache_update(vo->eq_opts_cache)) {
+ // "Legacy" update of video equalizer related options.
+ if (vo->driver->control)
+ vo->driver->control(vo, VOCTRL_SET_EQUALIZER, NULL);
+ }
}
// Does not include thread- and VO uninit.
@@ -238,6 +244,7 @@ static void dealloc_vo(struct vo *vo)
// These must be free'd before vo->in->dispatch.
talloc_free(vo->opts_cache);
talloc_free(vo->gl_opts_cache);
+ talloc_free(vo->eq_opts_cache);
pthread_mutex_destroy(&vo->in->lock);
pthread_cond_destroy(&vo->in->wakeup);
@@ -291,6 +298,10 @@ static struct vo *vo_create(bool probing, struct mpv_global *global,
update_opts, vo);
#endif
+ vo->eq_opts_cache = m_config_cache_alloc(NULL, global, &mp_csp_equalizer_conf);
+ m_config_cache_set_dispatch_change_cb(vo->eq_opts_cache, vo->in->dispatch,
+ update_opts, vo);
+
mp_input_set_mouse_transform(vo->input_ctx, NULL, NULL);
if (vo->driver->encode != !!vo->encode_lavc_ctx)
goto error;
diff --git a/video/out/vo.h b/video/out/vo.h
index c3145d5a5d..c6751c7524 100644
--- a/video/out/vo.h
+++ b/video/out/vo.h
@@ -401,6 +401,7 @@ struct vo {
struct m_config_cache *opts_cache; // cache for ->opts
struct mp_vo_opts *opts;
struct m_config_cache *gl_opts_cache;
+ struct m_config_cache *eq_opts_cache;
bool want_redraw; // redraw as soon as possible
diff --git a/video/out/vo_direct3d.c b/video/out/vo_direct3d.c
index e18cf8e4e0..687bb972a9 100644
--- a/video/out/vo_direct3d.c
+++ b/video/out/vo_direct3d.c
@@ -185,7 +185,6 @@ typedef struct d3d_priv {
D3DFORMAT osd_fmt_table[SUBBITMAP_COUNT];
D3DMATRIX d3d_colormatrix;
- struct mp_csp_equalizer video_eq;
struct osdpart *osd[MAX_OSD_PARTS];
} d3d_priv;
@@ -1171,7 +1170,6 @@ static void update_colorspace(d3d_priv *priv)
{
struct mp_csp_params csp = MP_CSP_PARAMS_DEFAULTS;
mp_csp_set_image_params(&csp, &priv->params);
- mp_csp_copy_equalizer_values(&csp, &priv->video_eq);
if (priv->use_shaders) {
csp.input_bits = priv->planes[0].bits_per_pixel;
@@ -1245,23 +1243,6 @@ static int control(struct vo *vo, uint32_t request, void *data)
case VOCTRL_REDRAW_FRAME:
d3d_draw_frame(priv);
return VO_TRUE;
- case VOCTRL_SET_EQUALIZER: {
- if (!priv->use_shaders)
- break;
- struct voctrl_set_equalizer_args *args = data;
- if (mp_csp_equalizer_set(&priv->video_eq, args->name, args->value) < 0)
- return VO_NOTIMPL;
- update_colorspace(priv);
- vo->want_redraw = true;
- return VO_TRUE;
- }
- case VOCTRL_GET_EQUALIZER: {
- if (!priv->use_shaders)
- break;
- struct voctrl_get_equalizer_args *args = data;
- return mp_csp_equalizer_get(&priv->video_eq, args->name, args->valueptr)
- >= 0 ? VO_TRUE : VO_NOTIMPL;
- }
case VOCTRL_SET_PANSCAN:
calc_fs_rect(priv);
priv->vo->want_redraw = true;
diff --git a/video/out/vo_opengl.c b/video/out/vo_opengl.c
index 02a7318970..72691e56c2 100644
--- a/video/out/vo_opengl.c
+++ b/video/out/vo_opengl.c
@@ -262,22 +262,9 @@ static int control(struct vo *vo, uint32_t request, void *data)
case VOCTRL_SET_PANSCAN:
resize(p);
return VO_TRUE;
- case VOCTRL_GET_EQUALIZER: {
- struct voctrl_get_equalizer_args *args = data;
- struct mp_csp_equalizer *eq = gl_video_eq_ptr(p->renderer);
- bool r = mp_csp_equalizer_get(eq, args->name, args->valueptr) >= 0;
- return r ? VO_TRUE : VO_NOTIMPL;
- }
- case VOCTRL_SET_EQUALIZER: {
- struct voctrl_set_equalizer_args *args = data;
- struct mp_csp_equalizer *eq = gl_video_eq_ptr(p->renderer);
- if (mp_csp_equalizer_set(eq, args->name, args->value) >= 0) {
- gl_video_eq_update(p->renderer);
- vo->want_redraw = true;
- return VO_TRUE;
- }
- return VO_NOTIMPL;
- }
+ case VOCTRL_SET_EQUALIZER:
+ vo->want_redraw = true;
+ return VO_TRUE;
case VOCTRL_SCREENSHOT_WIN: {
struct mp_image *screen = gl_read_fbo_contents(p->gl, p->glctx->main_fb,
vo->dwidth, vo->dheight);
diff --git a/video/out/vo_opengl_cb.c b/video/out/vo_opengl_cb.c
index 4a9b60d4ff..ea6aaa9048 100644
--- a/video/out/vo_opengl_cb.c
+++ b/video/out/vo_opengl_cb.c
@@ -76,8 +76,6 @@ struct mpv_opengl_cb_context {
bool force_update;
bool imgfmt_supported[IMGFMT_END - IMGFMT_START];
bool update_new_opts;
- bool eq_changed;
- struct mp_csp_equalizer eq;
struct vo *active;
// --- This is only mutable while initialized=false, during which nothing
@@ -187,10 +185,6 @@ int mpv_opengl_cb_init_gl(struct mpv_opengl_cb_context *ctx, const char *exts,
gl_video_set_hwdec(ctx->renderer, ctx->hwdec);
pthread_mutex_lock(&ctx->lock);
- // We don't know the exact caps yet - use a known superset
- ctx->eq.capabilities = MP_CSP_EQ_CAPS_GAMMA | MP_CSP_EQ_CAPS_BRIGHTNESS |
- MP_CSP_EQ_CAPS_COLORMATRIX;
- ctx->eq_changed = true;
for (int n = IMGFMT_START; n < IMGFMT_END; n++) {
ctx->imgfmt_supported[n - IMGFMT_START] =
gl_video_check_format(ctx->renderer, n);
@@ -277,7 +271,6 @@ int mpv_opengl_cb_draw(mpv_opengl_cb_context *ctx, int fbo, int vp_w, int vp_h)
if (ctx->reconfigured) {
gl_video_set_osd_source(ctx->renderer, vo ? vo->osd : NULL);
gl_video_config(ctx->renderer, &ctx->img_params);
- ctx->eq_changed = true;
}
if (ctx->update_new_opts) {
gl_video_update_options(ctx->renderer);
@@ -301,13 +294,6 @@ int mpv_opengl_cb_draw(mpv_opengl_cb_context *ctx, int fbo, int vp_w, int vp_h)
ctx->cur_frame->still = true;
}
- struct mp_csp_equalizer *eq = gl_video_eq_ptr(ctx->renderer);
- if (ctx->eq_changed) {
- memcpy(eq->values, ctx->eq.values, sizeof(eq->values));
- gl_video_eq_update(ctx->renderer);
- }
- ctx->eq_changed = false;
-
struct vo_frame *frame = ctx->next_frame;
int64_t wait_present_count = ctx->present_count;
if (frame) {
@@ -469,24 +455,9 @@ static int control(struct vo *vo, uint32_t request, void *data)
case VOCTRL_PAUSE:
vo->want_redraw = true;
return VO_TRUE;
- case VOCTRL_GET_EQUALIZER: {
- struct voctrl_get_equalizer_args *args = data;
- pthread_mutex_lock(&p->ctx->lock);
- bool r = mp_csp_equalizer_get(&p->ctx->eq, args->name, args->valueptr) >= 0;
- pthread_mutex_unlock(&p->ctx->lock);
- return r ? VO_TRUE : VO_NOTIMPL;
- }
- case VOCTRL_SET_EQUALIZER: {
- struct voctrl_set_equalizer_args *args = data;
- pthread_mutex_lock(&p->ctx->lock);
- bool r = mp_csp_equalizer_set(&p->ctx->eq, args->name, args->value) >= 0;
- if (r) {
- p->ctx->eq_changed = true;
- update(p);
- }
- pthread_mutex_unlock(&p->ctx->lock);
- return r ? VO_TRUE : VO_NOTIMPL;
- }
+ case VOCTRL_SET_EQUALIZER:
+ vo->want_redraw = true;
+ return VO_TRUE;
case VOCTRL_SET_PANSCAN:
pthread_mutex_lock(&p->ctx->lock);
p->ctx->force_update = true;
@@ -535,8 +506,6 @@ static int preinit(struct vo *vo)
p->ctx->active = vo;
p->ctx->reconfigured = true;
p->ctx->update_new_opts = true;
- memset(p->ctx->eq.values, 0, sizeof(p->ctx->eq.values));
- p->ctx->eq_changed = true;
pthread_mutex_unlock(&p->ctx->lock);
vo->hwdec_devs = p->ctx->hwdec_devs;
diff --git a/video/out/vo_sdl.c b/video/out/vo_sdl.c
index a1d0b248eb..d902c09cad 100644
--- a/video/out/vo_sdl.c
+++ b/video/out/vo_sdl.c
@@ -844,44 +844,11 @@ static void draw_image(struct vo *vo, mp_image_t *mpi)
{
struct priv *vc = vo->priv;
- // decode brightness/contrast
- int color_add = 0;
- int color_mod = 255;
- int brightness = vc->brightness;
- int contrast = vc->contrast;
-
- // only in this range it is possible to do brightness/contrast control
- // properly, using just additive render operations and color modding
- // (SDL2 provides no subtractive rendering, sorry)
- if (2 * brightness < contrast) {
- //brightness = (brightness + 2 * contrast) / 5; // closest point
- brightness = (brightness + contrast) / 3; // equal adjustment
- contrast = 2 * brightness;
- }
-
- // convert to values SDL2 likes
- color_mod = ((contrast + 100) * 255 + 50) / 100;
- color_add = ((2 * brightness - contrast) * 255 + 100) / 200;
-
- // clamp
- if (color_mod < 0)
- color_mod = 0;
- if (color_mod > 255)
- color_mod = 255;
- // color_add can't be < 0
- if (color_add > 255)
- color_add = 255;
-