Re: [PATCH v2 4/8] media: vidc: encoder: add video encoder files

Hi Hans, On 09/19/2016 01:15 PM, Hans Verkuil wrote: > Many of my review comments for the decoder apply to the encoder as well, > so I won't repeat those. Sure, will address them too. > > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> This adds encoder part of the driver plus encoder

Re: [PATCH v2 4/8] media: vidc: encoder: add video encoder files

Hi Hans, On 09/19/2016 01:15 PM, Hans Verkuil wrote: > Many of my review comments for the decoder apply to the encoder as well, > so I won't repeat those. Sure, will address them too. > > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> This adds encoder part of the driver plus encoder

Re: [PATCH v2 4/8] media: vidc: encoder: add video encoder files

Many of my review comments for the decoder apply to the encoder as well, so I won't repeat those. On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: > This adds encoder part of the driver plus encoder controls. > > Signed-off-by: Stanimir Varbanov > --- >

Re: [PATCH v2 4/8] media: vidc: encoder: add video encoder files

Many of my review comments for the decoder apply to the encoder as well, so I won't repeat those. On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: > This adds encoder part of the driver plus encoder controls. > > Signed-off-by: Stanimir Varbanov > --- > drivers/media/platform/qcom/vidc/venc.c

[PATCH v2 4/8] media: vidc: encoder: add video encoder files

This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/vidc/venc.c | 1252 + drivers/media/platform/qcom/vidc/venc.h | 29 +

[PATCH v2 4/8] media: vidc: encoder: add video encoder files

This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/vidc/venc.c | 1252 + drivers/media/platform/qcom/vidc/venc.h | 29 + drivers/media/platform/qcom/vidc/venc_ctrls.c | 396