Em 05-07-2012 11:41, Antti Palosaari escreveu:
> On 07/05/2012 05:32 PM, Mauro Carvalho Chehab wrote:
>> Em 18-05-2012 15:47, Thomas Mair escreveu:
> 
>>> +static int rtl2832_read_signal_strength(struct dvb_frontend *fe, u16 
>>> *strength)
>>> +{
>>> +    *strength = 0;
>>> +    return 0;
>>> +}
>>
>> Why to implement the above, if they're doing nothing?
> 
> Other your findings were correct but for that I would like to comment.
> 
> Have you ever tested what happens you lest those without stub implementation?
> IIRC ugly errors are seen for example zap and femon outputs. Some kind of 
> DVB-core
> changes are needed. And IIRC there was some error code defined too for API - 
> but such 
> code does not exists.

So, let's fix the dvb_core, and not fill drivers with stubs. With regards to 
userspace
tools, they should be patched to accept the error code that signalizes that an 
ioctl
was not implemented, and not try to fix an userpace issue in Kernel.

Regards,
Mauro
--
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

Reply via email to