Bug#582095: dahdi-linux: Adding support for Aligera cards

[François Delawarde]

On Wed, 2010-05-19 at 16:07 +0300, Tzafrir Cohen wrote:
> What sort of contact?

E-mail contact with one of the Brazilian devs that helped for our
implantation to work okay.

> Have those drivers been posted for inclusion in DAHDI?

No, I was hoping those could be included in the "extra" patch of debian
like for HFC or OpenVox drivers.

> Lacking that, we need to have someone who can answer questions for them.
> If I need to fix something, I need to have someone who can say that the
> fix won't break anything. Not to mention someone who can actually test
> the new version. Can you do that?

Unfortunately I have no testing server with those cards installed, only
production where testing trunk versions is a bit risky. I can try to
find out if someone from Aligera is willing to do that.

> I tried building those drivers vs. current dahdi trunk.
> 
> sparse (building with 'make C=1'. You need the package 'sparse' from
> non-free) gives the following:
> 
> drivers/dahdi/ap400/ap400_drv.c:2309:2: warning: obsolete struct initializer, use C99 syntax
> drivers/dahdi/ap400/ap400_drv.c:2310:2: warning: obsolete struct initializer, use C99 syntax
> drivers/dahdi/ap400/ap400_drv.c:2314:2: warning: obsolete struct initializer, use C99 syntax
> drivers/dahdi/ap400/ap400_drv.c:2316:2: warning: obsolete struct initializer, use C99 syntax
> drivers/dahdi/ap400/ap400_drv.c:2317:2: warning: obsolete struct initializer, use C99 syntax
> drivers/dahdi/ap400/ap400_drv.c:2318:2: warning: obsolete struct initializer, use C99 syntax
> drivers/dahdi/ap400/ap400_drv.c:691:31: warning: incorrect type in argument 2 (different address spaces)
> drivers/dahdi/ap400/ap400_drv.c:691:31:   expected void const [noderef] <asn:1>*from
> drivers/dahdi/ap400/ap400_drv.c:691:31:   got int *<noident>
> drivers/dahdi/ap400/ap400_drv.c:730:22: warning: incorrect type in argument 1 (different address spaces)
> drivers/dahdi/ap400/ap400_drv.c:730:22:   expected void [noderef] <asn:1>*to
> drivers/dahdi/ap400/ap400_drv.c:730:22:   got int *<noident>
> drivers/dahdi/ap400/ap400_drv.c:735:31: warning: incorrect type in argument 2 (different address spaces)
> drivers/dahdi/ap400/ap400_drv.c:735:31:   expected void const [noderef] <asn:1>*from
> drivers/dahdi/ap400/ap400_drv.c:735:31:   got int *<noident>
> drivers/dahdi/ap400/ap400_drv.c:742:22: warning: incorrect type in argument 1 (different address spaces)
> drivers/dahdi/ap400/ap400_drv.c:742:22:   expected void [noderef] <asn:1>*to
> drivers/dahdi/ap400/ap400_drv.c:742:22:   got int *<noident>
> drivers/dahdi/ap400/ap400_drv.c:1804:23: error: no member 'bpvcount' in struct dahdi_span
> drivers/dahdi/ap400/ap400_drv.c:1807:23: error: no member 'crc4count' in struct dahdi_span
> 
> And the last two lines are also errors by the compiler:
> drivers/dahdi/ap400/ap400_drv.c: In function ‘init_spans’:
> drivers/dahdi/ap400/ap400_drv.c:1025: warning: unused variable ‘c’
> drivers/dahdi/ap400/ap400_drv.c: In function ‘ap4_interrupt’:
> drivers/dahdi/ap400/ap400_drv.c:1804: error: ‘struct dahdi_span’ has no member named ‘bpvcount’
> drivers/dahdi/ap400/ap400_drv.c:1807: error: ‘struct dahdi_span’ has no member named ‘crc4count’
> 
> And indeed:
> svn_2.2:drivers/dahdi/dahdi-base.c: stack.spaninfo.bpvcount = spans[i]->bpvcount;
> svn_2.3:drivers/dahdi/dahdi-base.c: stack.spaninfo.bpvcount = spans[i]->count.bpv;
> http://svnview.digium.com/svn/dahdi?view=revision&revision=8061
> 
> I'm hoping to upload 2.3 soon.

I will try to make it compile against 2.3, but will not be able to test
if it works soon.

Thanks,
François.