* Re: [PATCH v7] media: Add stk1160 new driver [not found] <1343485133-11090-1-git-send-email-elezegarcia@gmail.com> @ 2012-08-06 12:28 ` Ezequiel Garcia 2012-08-06 12:42 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 5+ messages in thread From: Ezequiel Garcia @ 2012-08-06 12:28 UTC (permalink / raw) To: Mauro Carvalho Chehab Cc: Ezequiel Garcia, alsa-devel, linux-media, Takashi Iwai, Hans Verkuil, Sylwester Nawrocki Hi Mauro, On Sat, Jul 28, 2012 at 11:18 AM, Ezequiel Garcia <elezegarcia@gmail.com> wrote: > This driver adds support for stk1160 usb bridge as used in some > video/audio usb capture devices. > It is a complete rewrite of staging/media/easycap driver and > it's expected as a future replacement. > > Cc: Mauro Carvalho Chehab <mchehab@redhat.com> > Cc: Takashi Iwai <tiwai@suse.de> > Cc: Hans Verkuil <hverkuil@xs4all.nl> > Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com> > Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com> > --- > Did you take a look at this? Perhaps we can discuss now you're previous comments: 1. Place for ac97 code: media or alsa? (see Takashis' comments) 2. current_norm usage (see Hans' comments) 3. vb2_dqbuf and O_NONBLOCK flag (also see Hans' comments) I know it's a big patch*, so there is no need to rush. I just wanted to discuss a bit about this before my brain-cache flushes completely :-) Thanks, Ezequiel. * Actually it's huge (12k lines) since I'm removing staging/easycap. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v7] media: Add stk1160 new driver 2012-08-06 12:28 ` [PATCH v7] media: Add stk1160 new driver Ezequiel Garcia @ 2012-08-06 12:42 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 5+ messages in thread From: Mauro Carvalho Chehab @ 2012-08-06 12:42 UTC (permalink / raw) To: Ezequiel Garcia Cc: Takashi Iwai, Hans Verkuil, alsa-devel, Sylwester Nawrocki, linux-media Em 06-08-2012 09:28, Ezequiel Garcia escreveu: > Hi Mauro, > > On Sat, Jul 28, 2012 at 11:18 AM, Ezequiel Garcia <elezegarcia@gmail.com> wrote: >> This driver adds support for stk1160 usb bridge as used in some >> video/audio usb capture devices. >> It is a complete rewrite of staging/media/easycap driver and >> it's expected as a future replacement. >> >> Cc: Mauro Carvalho Chehab <mchehab@redhat.com> >> Cc: Takashi Iwai <tiwai@suse.de> >> Cc: Hans Verkuil <hverkuil@xs4all.nl> >> Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com> >> Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com> >> --- >> > > Did you take a look at this? Patchwork didn't get it[1]. Maybe the patch got mangled? If so, could you please re-post? [1] http://patchwork.linuxtv.org/project/linux-media/list/?state=*&q=stk1160 Thanks! Mauro > > Perhaps we can discuss now you're previous comments: > > 1. Place for ac97 code: media or alsa? (see Takashis' comments) > 2. current_norm usage (see Hans' comments) > 3. vb2_dqbuf and O_NONBLOCK flag (also see Hans' comments) > > I know it's a big patch*, so there is no need to rush. > I just wanted to discuss a bit about this before my brain-cache > flushes completely :-) > > Thanks, > Ezequiel. > > * Actually it's huge (12k lines) since I'm removing staging/easycap. > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v7] media: Add stk1160 new driver @ 2012-08-06 12:42 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 5+ messages in thread From: Mauro Carvalho Chehab @ 2012-08-06 12:42 UTC (permalink / raw) To: Ezequiel Garcia Cc: alsa-devel, linux-media, Takashi Iwai, Hans Verkuil, Sylwester Nawrocki Em 06-08-2012 09:28, Ezequiel Garcia escreveu: > Hi Mauro, > > On Sat, Jul 28, 2012 at 11:18 AM, Ezequiel Garcia <elezegarcia@gmail.com> wrote: >> This driver adds support for stk1160 usb bridge as used in some >> video/audio usb capture devices. >> It is a complete rewrite of staging/media/easycap driver and >> it's expected as a future replacement. >> >> Cc: Mauro Carvalho Chehab <mchehab@redhat.com> >> Cc: Takashi Iwai <tiwai@suse.de> >> Cc: Hans Verkuil <hverkuil@xs4all.nl> >> Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com> >> Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com> >> --- >> > > Did you take a look at this? Patchwork didn't get it[1]. Maybe the patch got mangled? If so, could you please re-post? [1] http://patchwork.linuxtv.org/project/linux-media/list/?state=*&q=stk1160 Thanks! Mauro > > Perhaps we can discuss now you're previous comments: > > 1. Place for ac97 code: media or alsa? (see Takashis' comments) > 2. current_norm usage (see Hans' comments) > 3. vb2_dqbuf and O_NONBLOCK flag (also see Hans' comments) > > I know it's a big patch*, so there is no need to rush. > I just wanted to discuss a bit about this before my brain-cache > flushes completely :-) > > Thanks, > Ezequiel. > > * Actually it's huge (12k lines) since I'm removing staging/easycap. > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v7] media: Add stk1160 new driver 2012-08-06 12:42 ` Mauro Carvalho Chehab (?) @ 2012-08-06 12:46 ` Ezequiel Garcia 2012-08-06 12:51 ` Ezequiel Garcia -1 siblings, 1 reply; 5+ messages in thread From: Ezequiel Garcia @ 2012-08-06 12:46 UTC (permalink / raw) To: Mauro Carvalho Chehab Cc: alsa-devel, linux-media, Takashi Iwai, Hans Verkuil, Sylwester Nawrocki On Mon, Aug 6, 2012 at 9:42 AM, Mauro Carvalho Chehab <mchehab@redhat.com> wrote: > Em 06-08-2012 09:28, Ezequiel Garcia escreveu: >> Hi Mauro, >> >> On Sat, Jul 28, 2012 at 11:18 AM, Ezequiel Garcia <elezegarcia@gmail.com> wrote: >>> This driver adds support for stk1160 usb bridge as used in some >>> video/audio usb capture devices. >>> It is a complete rewrite of staging/media/easycap driver and >>> it's expected as a future replacement. >>> >>> Cc: Mauro Carvalho Chehab <mchehab@redhat.com> >>> Cc: Takashi Iwai <tiwai@suse.de> >>> Cc: Hans Verkuil <hverkuil@xs4all.nl> >>> Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com> >>> Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com> >>> --- >>> >> >> Did you take a look at this? > > Patchwork didn't get it[1]. Maybe the patch got mangled? > If so, could you please re-post? > > [1] http://patchwork.linuxtv.org/project/linux-media/list/?state=*&q=stk1160 > Yes, I noticed. I sent a v5, v6 and v7; and none of them where noticed by patchwork. I can re-send, but I think perhaps it was due to patch size? (alsa-devel list bounced it, for instance). Ezequiel. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v7] media: Add stk1160 new driver 2012-08-06 12:46 ` Ezequiel Garcia @ 2012-08-06 12:51 ` Ezequiel Garcia 0 siblings, 0 replies; 5+ messages in thread From: Ezequiel Garcia @ 2012-08-06 12:51 UTC (permalink / raw) To: Mauro Carvalho Chehab Cc: alsa-devel, linux-media, Takashi Iwai, Hans Verkuil, Sylwester Nawrocki On Mon, Aug 6, 2012 at 9:46 AM, Ezequiel Garcia <elezegarcia@gmail.com> wrote: > On Mon, Aug 6, 2012 at 9:42 AM, Mauro Carvalho Chehab > <mchehab@redhat.com> wrote: >> Em 06-08-2012 09:28, Ezequiel Garcia escreveu: >>> Hi Mauro, >>> >>> On Sat, Jul 28, 2012 at 11:18 AM, Ezequiel Garcia <elezegarcia@gmail.com> wrote: >>>> This driver adds support for stk1160 usb bridge as used in some >>>> video/audio usb capture devices. >>>> It is a complete rewrite of staging/media/easycap driver and >>>> it's expected as a future replacement. >>>> >>>> Cc: Mauro Carvalho Chehab <mchehab@redhat.com> >>>> Cc: Takashi Iwai <tiwai@suse.de> >>>> Cc: Hans Verkuil <hverkuil@xs4all.nl> >>>> Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com> >>>> Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com> >>>> --- >>>> >>> >>> Did you take a look at this? >> >> Patchwork didn't get it[1]. Maybe the patch got mangled? >> If so, could you please re-post? >> >> [1] http://patchwork.linuxtv.org/project/linux-media/list/?state=*&q=stk1160 >> I'm preparing v8 based on v3.7... Ezequiel. ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-08-06 12:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1343485133-11090-1-git-send-email-elezegarcia@gmail.com>
2012-08-06 12:28 ` [PATCH v7] media: Add stk1160 new driver Ezequiel Garcia
2012-08-06 12:42 ` Mauro Carvalho Chehab
2012-08-06 12:42 ` Mauro Carvalho Chehab
2012-08-06 12:46 ` Ezequiel Garcia
2012-08-06 12:51 ` Ezequiel Garcia
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.