lkml.org 
[lkml]   [2019]   [Apr]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [Xen-devel][PATCH] xen/cameraif: add ABI for para-virtual camera
From
Date
On 4/9/19 12:46 PM, Juergen Gross wrote:
> On 09/04/2019 11:33, Oleksandr Andrushchenko wrote:
>> On 4/9/19 12:28 PM, Juergen Gross wrote:
>>> On 09/04/2019 11:15, Oleksandr Andrushchenko wrote:
>>>> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
>>>>
>>>> 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@epam.com>
>>>> Cc: Juergen Gross <jgross@suse.com>
>>>> Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
>>>> Cc: Hans Verkuil <hverkuil@xs4all.nl>
>>> 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)
> Sorry for not having done the review properly in the past. ;-)
>
>> 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
> Having an unused header file is not good.
>
> Be assured I'm going to give you the needed Reviewed-by: tag for
> the patch adding the header in a series using it.
Ok, sounds good
>
>
> Juergen

\
 
 \ /
  Last update: 2019-04-09 15:27    [W:0.050 / U:2.852 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site