[Discuss-gnuradio] Openbts patches

[Martin DvH]

Hi,

I have been working with OpenBTS in trying to get it to compile and
found two bugs and solved them.

The first bug prevents the OpenBTS tranceiver from starting with the
error "make failed on Rx"

The check on line 173 in USRPDevice should be for !m_uRx in stead of !
m_uTx

The second is that HAVE_BYTESWAP_H is not defined in
Transceiver/USRPDevice.h

This forces usrp_bytesex.h in using its own byteswap code which does NOT
work on 64 bit systems.

The disadvantage of putting #define HAVE_BYTESWAP_H in USRPDevice.h is
that it should be checked first that you actually have byteswap.h.
This should be done by the configure script and put into config.h.
USRPDevice.h then should include config.h

I haven't gotten to implementing the configure stuff yet.

Attached are two mini-patches which implement the changes.

I think patch1 can be submitted to svn right away (Jonathan, I could do
that, should I ?)
patch2 should be changed to use configure stuff to check for byteswap.h.
But for people wanting a quick-and-dirty hack it can be used.

I am not sure if there are more bugs in openbts in gnuradio.
I still get a lot of messages when I start it all up. 
1232403472.992511 1085643088: RadioResource.cpp:347 Pager blocking for
signal
1232403472.992521 1085643088: RadioResource.cpp:347 Pager blocking for
signal
1232403472.992530 1085643088: RadioResource.cpp:347 Pager blocking for
signal
Repeats until infinity

I am not sure if this is an error or normal behaviour.

As it is in gnuradio svn, it could not have worked.
Did anyone try/test and got the gnuradio version to work.

Greetings,
Martin

mdvh_openbts_tranceiver_patch1.diff
Description: Text Data

mdvh_openbts_tranceiver_patch2.diff
Description: Text Data