Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Hi Sachin, On Tue, Dec 10, 2013 at 10:38 AM, Sachin Kamat wrote: > Hi Arun, > > On 10 December 2013 10:15, Arun Kumar K wrote: >> Hi Pawel, >> >> On Tue, Dec 10, 2013 at 6:21 AM, Pawel Osciak wrote: >>> Hi Arun, >>> > >> >> Ok will make the change in next version. >> > > While at it also update

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Sachin Kamat
Hi Arun, On 10 December 2013 10:15, Arun Kumar K wrote: > Hi Pawel, > > On Tue, Dec 10, 2013 at 6:21 AM, Pawel Osciak wrote: >> Hi Arun, >> > > Ok will make the change in next version. > While at it also update the patch subject appropriately. -- With warm regards, Sachin -- To unsubscribe f

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Hi Pawel, On Tue, Dec 10, 2013 at 6:21 AM, Pawel Osciak wrote: > Hi Arun, > > On Mon, Dec 9, 2013 at 10:16 PM, Arun Kumar K wrote: >> Add v4l2 controls to set desired profile for VP8 encoder. >> Acceptable levels for VP8 encoder are >> 0: Version 0 >> 1: Version 1 >> 2: Version 2 >> 3: Version 3

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Hi Hans, Thanks for the review. On Mon, Dec 9, 2013 at 6:52 PM, Hans Verkuil wrote: > Hi Arun, > > Some comments below... > > On 12/09/2013 02:16 PM, Arun Kumar K wrote: >> Add v4l2 controls to set desired profile for VP8 encoder. >> Acceptable levels for VP8 encoder are >> 0: Version 0 >> 1: Ve

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Pawel Osciak
Hi Arun, On Mon, Dec 9, 2013 at 10:16 PM, Arun Kumar K wrote: > Add v4l2 controls to set desired profile for VP8 encoder. > Acceptable levels for VP8 encoder are > 0: Version 0 > 1: Version 1 > 2: Version 2 > 3: Version 3 > > Signed-off-by: Pawel Osciak Sorry, but I'm not the author of this pat

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Hans Verkuil
Hi Arun, Some comments below... On 12/09/2013 02:16 PM, Arun Kumar K wrote: > Add v4l2 controls to set desired profile for VP8 encoder. > Acceptable levels for VP8 encoder are > 0: Version 0 > 1: Version 1 > 2: Version 2 > 3: Version 3 > > Signed-off-by: Pawel Osciak > Signed-off-by: Kiran AVND

[PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Add v4l2 controls to set desired profile for VP8 encoder. Acceptable levels for VP8 encoder are 0: Version 0 1: Version 1 2: Version 2 3: Version 3 Signed-off-by: Pawel Osciak Signed-off-by: Kiran AVND Signed-off-by: Arun Kumar K --- This patch is rebased over another VP8 control patch from me: