Re: [PATCH v7] media: Add stk1160 new driver

[Ezequiel Garcia <elezegarcia@xxxxxxxxx>]

Hi Mauro,

On Sat, Jul 28, 2012 at 11:18 AM, Ezequiel Garcia <elezegarcia@xxxxxxxxx> 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@xxxxxxxxxx>
> Cc: Takashi Iwai <tiwai@xxxxxxx>
> Cc: Hans Verkuil <hverkuil@xxxxxxxxx>
> Cc: Sylwester Nawrocki <sylvester.nawrocki@xxxxxxxxx>
> Signed-off-by: Ezequiel Garcia <elezegarcia@xxxxxxxxx>
> ---
>

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.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html