> 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?
ACK from me :-)
Regards,
Hans
>
> Cheers,
> Matti
>
>> Regards,
>>
>> Hans
>>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Hans Verkuil - video4linux developer - sponsored by TANDBERG, part of Cisco
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
