[PATCH 02/10] libweston: Add initial output API for windowed outputs configuration

[Quentin Glidic]

On 11/08/2016 17:33, Armin Krezović wrote:
> This adds new plugin-specific API for configuring outputs
> on "windowed" backends, such as X11, wayland/non-fullscreen
> and even headless (although, it doesn't have any windows,
> its configuration is very similar). It can be used from
> compositors to configure pending outputs and should be used
> with previously added weston_output_set_{scale,transform}
> to properly configure an output before enabling it.
>
> It also supports creating additional outputs on the mentioned
> backends.
>
> Signed-off-by: Armin Krezović <krezovic.armin at gmail.com>
> ---
>  Makefile.am            |  1 +
>  libweston/output-api.h | 88 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 89 insertions(+)
>  create mode 100644 libweston/output-api.h
>
> diff --git a/Makefile.am b/Makefile.am
> index 32627f5..038e4da 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -245,6 +245,7 @@ libwestoninclude_HEADERS =			\
>  	libweston/compositor-rdp.h		\
>  	libweston/compositor-wayland.h		\
>  	libweston/compositor-x11.h		\
> +	libweston/output-api.h			\
>  	libweston/plugin-registry.h		\
>  	libweston/timeline-object.h		\
>  	shared/matrix.h				\
> diff --git a/libweston/output-api.h b/libweston/output-api.h
> new file mode 100644
> index 0000000..ef33587
> --- /dev/null
> +++ b/libweston/output-api.h
> @@ -0,0 +1,88 @@
> +/*
> + * Copyright © 2016 Collabora, Ltd.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining
> + * a copy of this software and associated documentation files (the
> + * "Software"), to deal in the Software without restriction, including
> + * without limitation the rights to use, copy, modify, merge, publish,
> + * distribute, sublicense, and/or sell copies of the Software, and to
> + * permit persons to whom the Software is furnished to do so, subject to
> + * the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the
> + * next paragraph) shall be included in all copies or substantial
> + * portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + * NONINFRINGEMENT.  IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
> + * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
> + * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> + * SOFTWARE.
> + */
> +
> +#ifndef WESTON_OUTPUT_API_H
> +#define WESTON_OUTPUT_API_H
> +
> +#ifdef  __cplusplus
> +extern "C" {
> +#endif
> +
> +#include "plugin-registry.h"
> +
> +struct weston_compositor;
> +struct weston_output;
> +
> +#define WESTON_WINDOWED_OUTPUT_API_NAME "weston_windowed_output_api_v1"
> +
> +struct weston_windowed_output_api {
> +	/** Configure an output with given width and height.
> +	 *
> +	 * \param output An output to be configured.
> +	 * \param width  Desired width of the output.
> +	 * \param height Desired height of the output.
> +	 *
> +	 * This configures a windowed output with desired width and
> +	 * height. The backend decides what should be done and applies
> +	 * the desired configuration. After using this function and
> +	 * generic weston_output_set_{scale,transform}, a windowed

Might be "weston_output_set_*" to avoid future needs to change it.

Anyway:
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Cheers,


> +	 * output should be in a state where weston_output_enable()
> +	 * can be run.
> +	 */
> +	int (*output_configure)(struct weston_output *output,
> +				int width, int height);
> +
> +	/** Create a new windowed output.
> +	 *
> +	 * \param compositor The compositor instance.
> +	 * \param name       Desired name for a new output.
> +	 *
> +	 * This creates a new output in the backend using this API.
> +	 * After this function is ran, the created output should be
> +	 * ready for configuration using the output_configure() and
> +	 * weston_output_set_{scale,transform}().
> +	 *
> +	 * An optional name can be assigned to it, so it can be used
> +	 * by compositor to configure it. It can be NULL.
> +	 */
> +	int (*output_create)(struct weston_compositor *compositor,
> +			     const char *name);
> +};
> +
> +static inline const struct weston_windowed_output_api *
> +weston_windowed_output_get_api(struct weston_compositor *compositor)
> +{
> +	const void *api;
> +	api = weston_plugin_api_get(compositor, WESTON_WINDOWED_OUTPUT_API_NAME,
> +				    sizeof(struct weston_windowed_output_api));
> +
> +	return (const struct weston_windowed_output_api *)api;
> +}
> +
> +#ifdef  __cplusplus
> +}
> +#endif
> +
> +#endif /* WESTON_OUTPUT_API_H */
>


-- 

Quentin “Sardem FF7” Glidic