Re: [PATCH v13 1/1] Documentation: v4l: Add hw_seek spacing and two TUNER_RDS_CAP flags.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi.

On Mon, 2010-10-18 at 15:57 +0200, ext Hans Verkuil wrote:
> > Em 18-10-2010 11:17, Hans Verkuil escreveu:
> >> Just a few very small comments:
> >
> >>> +For future use the
> >>> +flag <constant>V4L2_TUNER_SUB_RDS_CONTROLS</constant> has also been
> >>> +defined. However, a driver for a radio tuner with this capability does
> >>> +not yet exist, so if you are planning to write such a driver the best
> >>> +way to start would probably be by opening a discussion about it on
> >>> +the linux-media mailing list: &v4l-ml;. </para>
> >>
> >> Change to:
> >>
> >> not yet exist, so if you are planning to write such a driver you
> >> should discuss this on the linux-media mailing list: &v4l-ml;.</para>
> >
> > No, please. Don't add any API capabilities at the DocBook without having a
> > driver
> > using it. At the time a driver start needing it, we can just add the API
> > bits
> > and doing the needed discussions as you've proposed. This is already
> > implicit.
> 
> These caps are shared between tuner and modulator. And for the modulator
> both caps *are* used in Matti's driver. But while RDS_CONTROLS is
> available for modulators, it is not yet applicable to tuners and for that
> we need to make this note in the spec.
> 
> So this is an exception to the rule, I'm afraid.

So it's ACK... or NACK?

Cheers,
Matti

> Regards,
> 
>         Hans
> 


--
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux