Rolf Eike Beer wrote:

>IIRC the call to pci_enable_device() must be the first thing you do. This will 
>do the things like assigning memory regions to the device and so on.
>
>  
>
I fixed this one

>Returning 0 in error cases is just wrong. And you free the assignments even in 
>case of success AFAICS. Try the return I introduced above and see what 
>happens.
>
>  
>
I fixed this one too ..


I have fixed most of the stuff, it is partly working, not ready yet as 
there are some more things to be added to  ..
I have attached what i was working on.


Thanks,
Manu