Re: DM1105: could not attach frontend 195d:1105

[Nameer Kazzaz <nameer.kazzaz@gmail.com>]

Igor M. Liplianin wrote:
> On 3 февраля 2010 00:07:36 Nameer Kazzaz wrote:
>   
>> Nameer Kazzaz wrote:
>>     
>>> Igor M. Liplianin wrote:
>>>       
>>>> On 2 февраля 2010 17:21:46 Nameer Kazzaz wrote:
>>>>         
>>>>> Hi Igor,
>>>>>     What do you think ? if I can help you solve this, let me know
>>>>> what I
>>>>> can do.
>>>>>
>>>>> Thanks
>>>>> Nameer
>>>>>
>>>>> Nameer Kazzaz wrote:
>>>>>           
>>>>>> Hi there,
>>>>>> dmesg output with patched dm1105.c against current v4l-dvb 'modprob
>>>>>> dm1105 card=4'
>>>>>>
>>>>>> dm1105 0000:05:0f.0: PCI INT A -> GSI 16 (level, low) -> IRQ 16
>>>>>> DVB: registering new adapter (dm1105)
>>>>>> dm1105 0000:05:0f.0: MAC dd49b0dc
>>>>>> dm1105 0000:05:0f.0: could not attach frontend
>>>>>> dm1105 0000:05:0f.0: PCI INT A disabled
>>>>>>
>>>>>> Thanks
>>>>>> Nameer Kazzaz
>>>>>>
>>>>>> Igor M. Liplianin wrote:
>>>>>>             
>>>>>>> On 20 ?????? 2010 23:20:20 paul10@planar.id.au wrote:
>>>>>>>               
>>>>>>>> Igor wrote:
>>>>>>>>                 
>>>>>>>>> Oh, that is wrong. It is registers addresses, Never touch this.
>>>>>>>>>
>>>>>>>>> Let's look on that part of code:
>>>>>>>>>
>>>>>>>>> /* GPIO's for LNB power control */
>>>>>>>>> #define DM1105_LNB_MASK                         0x00000000 //
>>>>>>>>> later in
>>>>>>>>>                   
>>>>>>>> code write it to
>>>>>>>>
>>>>>>>>                 
>>>>>>>>> DM1105_GPIOCTR, all GPIO's as OUT
>>>>>>>>> #define DM1105_LNB_OFF                          0x00020000 //
>>>>>>>>> later in
>>>>>>>>>                   
>>>>>>>> code write it to
>>>>>>>>
>>>>>>>>                 
>>>>>>>>> DM1105_GPIOVAL, set GPIO17 to HIGH
>>>>>>>>>
>>>>>>>>> But you have not to change this.
>>>>>>>>> Right way is to write another entry in cards structure and so on.
>>>>>>>>> Better leave it to me.
>>>>>>>>>
>>>>>>>>> Regards
>>>>>>>>> Igor
>>>>>>>>>                   
>>>>>>>> Thanks for all your help, I understand better now.  I have moved to
>>>>>>>> code
>>>>>>>> like that at the bottom.  It still doesn't work, but feels a lot
>>>>>>>> closer.
>>>>>>>>
>>>>>>>> Before I keep playing with values, I want to check I'm on the right
>>>>>>>> track.
>>>>>>>> Does it look right?  Specific questions:
>>>>>>>> 1. I see there is a hw_init function.  Should I be using that?  I
>>>>>>>> put the
>>>>>>>> logic into fe_attach because there was already card-specific
>>>>>>>> logic in
>>>>>>>> there.  But this feels like hw initialisation.
>>>>>>>>
>>>>>>>> 2. Should I set the control to input or output?  I'm assuming input
>>>>>>>> = 1.
>>>>>>>>
>>>>>>>> 3. Would pin 15 be numbered from the left or right - is it 0x4, or
>>>>>>>> 0x2000?
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>>
>>>>>>>> Paul
>>>>>>>>
>>>>>>>> *** dm1105.c.old        2010-01-13 16:15:00.000000000 +1100
>>>>>>>> --- dm1105.c    2010-01-21 08:13:14.000000000 +1100
>>>>>>>> ***************
>>>>>>>> *** 51,56 ****
>>>>>>>> --- 51,57 ----
>>>>>>>>   #define DM1105_BOARD_DVBWORLD_2002    1
>>>>>>>>   #define DM1105_BOARD_DVBWORLD_2004    2
>>>>>>>>   #define DM1105_BOARD_AXESS_DM05               3
>>>>>>>> + #define DM1105_BOARD_UNBRANDED                4
>>>>>>>>
>>>>>>>>   /* ----------------------------------------------- */
>>>>>>>>   /*
>>>>>>>> ***************
>>>>>>>> *** 171,176 ****
>>>>>>>> --- 172,181 ----
>>>>>>>>   #define DM05_LNB_13V                          0x00020000
>>>>>>>>   #define DM05_LNB_18V                          0x00030000
>>>>>>>>
>>>>>>>> + /* GPIO's for demod reset for unbranded 195d:1105 */
>>>>>>>> + #define UNBRANDED_DEMOD_MASK                  0x00008000
>>>>>>>> + #define UNBRANDED_DEMOD_RESET                 0x00008000
>>>>>>>> +
>>>>>>>>   static unsigned int card[]  = {[0 ... 3] = UNSET };
>>>>>>>>   module_param_array(card,  int, NULL, 0444);
>>>>>>>>   MODULE_PARM_DESC(card, "card type");
>>>>>>>> ***************
>>>>>>>> *** 206,211 ****
>>>>>>>> --- 211,219 ----
>>>>>>>>         [DM1105_BOARD_AXESS_DM05] = {
>>>>>>>>                 .name           = "Axess/EasyTv DM05",
>>>>>>>>         },
>>>>>>>> +       [DM1105_BOARD_UNBRANDED] = {
>>>>>>>> +               .name           = "Unbranded 195d:1105",
>>>>>>>> +         },
>>>>>>>>   };
>>>>>>>>
>>>>>>>>   static const struct dm1105_subid dm1105_subids[] = {
>>>>>>>> ***************
>>>>>>>> *** 229,234 ****
>>>>>>>> --- 237,246 ----
>>>>>>>>                 .subvendor = 0x195d,
>>>>>>>>                 .subdevice = 0x1105,
>>>>>>>>                 .card      = DM1105_BOARD_AXESS_DM05,
>>>>>>>> +       }, {
>>>>>>>> +               .subvendor = 0x195d,
>>>>>>>> +               .subdevice = 0x1105,
>>>>>>>> +               .card      = DM1105_BOARD_UNBRANDED,
>>>>>>>>         },
>>>>>>>>   };
>>>>>>>>
>>>>>>>> ***************
>>>>>>>> *** 698,703 ****
>>>>>>>> --- 710,727 ----
>>>>>>>>                         dm1105dvb->fe->ops.set_voltage =
>>>>>>>> dm1105dvb_set_voltage;
>>>>>>>>
>>>>>>>>                 break;
>>>>>>>> +       case DM1105_BOARD_UNBRANDED:
>>>>>>>> +                 printk(KERN_ERR "Attaching as board_unbranded\n");
>>>>>>>> +               outl(UNBRANDED_DEMOD_MASK,
>>>>>>>> dm_io_mem(DM1105_GPIOCTR));
>>>>>>>> +               outl(UNBRANDED_DEMOD_RESET ,
>>>>>>>> dm_io_mem(DM1105_GPIOVAL));
>>>>>>>> +               dm1105dvb->fe = dvb_attach(
>>>>>>>> +                       si21xx_attach, &serit_config,
>>>>>>>> +                       &dm1105dvb->i2c_adap);
>>>>>>>> +                       if (dm1105dvb->fe)
>>>>>>>> +                               dm1105dvb->fe->ops.set_voltage =
>>>>>>>> +                                       dm1105dvb_set_voltage;
>>>>>>>> +
>>>>>>>> +               break;
>>>>>>>>         case DM1105_BOARD_DVBWORLD_2002:
>>>>>>>>         case DM1105_BOARD_AXESS_DM05:
>>>>>>>>         default:
>>>>>>>>                 
>>>>>>> Some things are missed, like keep GPIO15 high in set_voltage
>>>>>>> function.
>>>>>>> Try attached patch against current v4l-dvb tree with modprobe option
>>>>>>> card=4
>>>>>>>     modprobe dm1105 card=4
>>>>>>>               
>>>> Hi Nameer,
>>>> You can modify sended by me patch to guess GPIO pin. Simply try all
>>>> of them, it is only 17. Just replace all appearances DM1105_GPIO(15)
>>>> with number you want.
>>>>         
>>> Ok I will do that.
>>>
>>> Thanks
>>> Nameer
>>>       
>> Hi Igor,
>>     I tried all DM1105_GPIO(0) to DM1105_GPIO(17), same error. Any Idea ?
>>
>> Nameer
>>     
> Hi
>
> I have some ideas, for example, to add to driver i2c_scan function.
>   
Hi Igor,
    Ok so if you need any more info from the pci board and testing any 
patches please let me know. And thanks for your help on this.

Nameer