[linux-dvb] [PATCH] Twinhan + clones : Correct polarization for the DST frontend

Manu Abraham manu at kromtek.com
Wed Jun 22 20:54:21 CEST 2005


Allan Stirling wrote:
> Olaf Titz wrote:
> 
>>> +            state->tx_tuna[8] =~ 0x40;  //1
>>> +            state->tx_tuna[8] |= 0x40;  // 0
>>>   
>>
>> I don't know anything about that device, but this code looks wrong to
>> me. The "1" line sets the whole register and the "0" line a single
>> bit. Shouldn't that rather be something like
>>
>>  
>>
>>> +            state->tx_tuna[8] &= ~0x40;  //1
>>>   
>>
>> If the wanted effect is really setting the whole register to ~0x40 vs.
>> all ones and the other bits are not affected independently, that
>> should be reformulated just for clarity:
>>
>>  
>>
>>> +            state->tx_tuna[8] = ~0x40;  //1
>>> +            state->tx_tuna[8] = ~0x00;  // 0
>>>   
>>
>> (note where the blanks are...)
>>
>> Olaf
>>
>>
>> _______________________________________________
>> linux-dvb mailing list
>> linux-dvb at linuxtv.org
>> http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
>>  
>>
> (One of these days I'll figure out why Thunderbird is defaulting to 
> reply-to-author rather than the list)
> 
> Thanks to Olaf and Edgar for pointing this out. Assuming it's correct
> (again, it works for me), here's a revised patch:
> 
> Not sure if this still applies (I'm new here :) but just in case:
> 
> Signed-off-by: Allan Stirling <Dibblahdvb0042 at pendor.org>
> 

Applied to CVS ..

Manu




More information about the linux-dvb mailing list