[PATCH 2/3] Add a 'serial' property on weston_output
Rob Bradford
robert.bradford at intel.com
Tue Apr 23 08:37:18 PDT 2013
On Mon, Apr 22, 2013 at 12:57:03PM +0100, Richard Hughes wrote:
> ---
> src/compositor-drm.c | 1 +
> src/compositor.h | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/compositor-drm.c b/src/compositor-drm.c
> index 61ef97e..a454676 100644
> --- a/src/compositor-drm.c
> +++ b/src/compositor-drm.c
> @@ -1522,6 +1522,7 @@ create_output_for_connector(struct drm_compositor *ec,
> output->base.subpixel = drm_subpixel_to_wayland(connector->subpixel);
> output->base.make = "unknown";
> output->base.model = "unknown";
> + output->base.serial = "unknown";
> wl_list_init(&output->base.mode_list);
>
> if (connector->connector_type < ARRAY_LENGTH(connector_type_names))
> diff --git a/src/compositor.h b/src/compositor.h
> index 1e999a6..fa6162c 100644
> --- a/src/compositor.h
> +++ b/src/compositor.h
> @@ -178,7 +178,7 @@ struct weston_output {
> uint32_t frame_time;
> int disable_planes;
>
> - char *make, *model;
> + char *make, *model, *serial;
> uint32_t subpixel;
> uint32_t transform;
Hi Richard,
When I read the commit message I wondered "why does he need a serial"
for the wl_output. I was of course thinking about the wayland use of
"serial" as a sequence number.
Are you intending to expose this through the wayland protocol? If so
have you considered how this ambiguity might be avoided.
Cheers,
Rob
More information about the wayland-devel
mailing list