Re: PATCH: Query DVB frontend capabilities

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em 11-11-2011 20:07, Manu Abraham escreveu:
> On Fri, Nov 11, 2011 at 3:42 PM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>> Em 11-11-2011 04:26, Manu Abraham escreveu:
>>> On Fri, Nov 11, 2011 at 2:50 AM, Mauro Carvalho Chehab
>>> <mchehab@redhat.com> wrote:
>>>> Em 10-11-2011 13:30, Manu Abraham escreveu:
>>> The purpose of the patch is to
>>> query DVB delivery system capabilities alone, rather than DVB frontend
>>> info/capability.
>>>
>>> Attached is a revised version 2 of the patch, which addresses the
>>> issues that were raised.
>>
>> It looks good for me. I would just rename it to DTV_SUPPORTED_DELIVERY.
>> Please, when submitting upstream, don't forget to increment DVB version and
>> add touch at DocBook, in order to not increase the gap between API specs and the
>> implementation.
> 
> Ok, thanks for the feedback, will do that.
> 
> The naming issue is trivial. I would like to have a shorter name
> rather that SUPPORTED. CAPS would have been ideal, since it refers to
> device capability.

CAPS is not a good name, as there are those two CAPABILITIES calls there
(well, currently not implemented). So, it can lead in to some confusion.

DTV_ENUM_DELIVERY could be an alternative for a short name to be used there.

> 
> Regards,
> Manu
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html