[linux-dvb] [PATCH] wscan: improved frontend autodetection

[Hans Werner]

> Hi,
> 
> Hans Werner wrote:
> > Currently wscan will not autodetect frontends which which have frontend
> != 0,
> > i.e. it only detects /dev/dvb/adapterN/frontend0 where N=0-3.
> >
> > Since multiple frontends per adapter are supported in 2.6.28, this means
> the correct
> > frontend may not be found. For example with the HVR4000, DVB-T is always
> at frontend1.
> >
> > The attached patch fixes this, searching for frontend 0-3 for each
> adapter 0-3.
> >
> > Signed-off-by: Hans Werner <hwerner4 at gmx.de>
> Good idea. :)
> But while testing your patch it seems that it doesn't work as expected. 
> Here some example:
> 
> w_scan version 20081106
> Info: using DVB adapter auto detection.
>    Found DVB-C frontend. Using adapter /dev/dvb/adapter0/frontend0
>    Found DVB-C frontend. Using adapter /dev/dvb/adapter1/frontend0
>    Found DVB-C frontend. Using adapter /dev/dvb/adapter2/frontend0
> Info: unable to open frontend /dev/dvb/adapter3/frontend1'
> Info: unable to open frontend /dev/dvb/adapter3/frontend2'
> Info: unable to open frontend /dev/dvb/adapter3/frontend3'
> -_-_-_-_ Getting frontend capabilities-_-_-_-_
> 
> I'm using three dvb-c frontends.
> The detection doesnt stop anymore with your patch if a matching frontend 
> was found, because it doesnt leave the outer loop.
> Normally this search has to stop at /dev/dvb/adapter0/frontend0.

OK I see, you want to use the first usable card found, not the last as in my patch.

> That means we have to change your patch a little. I also increased the 
> number of adapters to 8, since i use more than 4.
> 
> Can you please test the attached patch and give some feedback? If it 
> works fine for you, i would apply to w_scan.
> 
> -Winfried
> 
> PS: What is actually the maximum number of adapters and frontends per 
> adapter? Can anybody give some hint?

Your patch works fine for me. I would add one thing though: close the frontend if it
is the wrong type. I have attached a new patch.

Thanks,
Hans
-- 
Release early, release often.

"Feel free" - 5 GB Mailbox, 50 FreeSMS/Monat ...
Jetzt GMX ProMail testen: http://www.gmx.net/de/go/promail
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch_wscan_fixautodetection3.diff
Type: text/x-patch
Size: 2167 bytes
Desc: not available
Url : http://www.linuxtv.org/pipermail/linux-dvb/attachments/20081106/aa3cfdfe/attachment.bin