Re: [Xen-devel] [PATCH] xen/cameraif: add ABI for para-virtual camera

[Oleksandr Andrushchenko <andr2000@xxxxxxxxx>]

On 4/9/19 12:28 PM, Juergen Gross wrote:
> On 09/04/2019 11:15, Oleksandr Andrushchenko wrote:
> > From: Oleksandr Andrushchenko <oleksandr_andrushchenko@xxxxxxxx>
> >
> > This is the ABI for the two halves of a para-virtualized
> > camera driver which extends Xen's reach multimedia capabilities even
> > further enabling it for video conferencing, In-Vehicle Infotainment,
> > high definition maps etc.
> >
> > The initial goal is to support most needed functionality with the
> > final idea to make it possible to extend the protocol if need be:
> >
> > 1. Provide means for base virtual device configuration:
> >   - pixel formats
> >   - resolutions
> >   - frame rates
> > 2. Support basic camera controls:
> >   - contrast
> >   - brightness
> >   - hue
> >   - saturation
> > 3. Support streaming control
> >
> > Signed-off-by: Oleksandr Andrushchenko <oleksandr_andrushchenko@xxxxxxxx>
> > Cc: Juergen Gross <jgross@xxxxxxxx>
> > Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> > Cc: Hans Verkuil <hverkuil@xxxxxxxxx>
> I'm in principle fine with this patch, but it should be sent as part of
> a series using that header. In case the related driver isn't accepted
> we'd end up with a stale header file.
Hm, I am doing this as I did for the rest of protocols before:
1. Get the protocol in Xen
2. Get the copy in Linux
3. Upstream driver (use header)

Even if the driver is not accepted (hope this won't happen)
then this patch makes Xen and Linux properly aligned anyway.
Which is IMO good
> Juergen
Thank you,
Oleksandr

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel