Re: Bug fix in cl_8900a ethernet driver

[Bob Koninckx <bob dot koninckx at mech dot kuleuven dot ac dot be>]

[snip]

>
> Bang, you're dead ..
>
>
> > , was because I believe yours 
> > wouldn't be right for len==1 in the last iteration of the for loop (it 
> > will HAL_WRITE saved_data which would contain the real data in the MSB, 
>
>
>Which is exactly where you want to have it! From (the very difficult to
>interprete correctly, took me some time :) ) AN205 from Cirrus
>


How about the attached patch ? Works ok for me on powerpc and still incorporates
the alignment fix. I also added some braces to silence a compiler warning.

Bob


-- 
----------------------------------------------------------------------
ir. Bob Koninckx
Katholieke Universiteit Leuven
Division Production Engineering,                   tel.  +32 16 322535
Machine Design and Automation                      fax.  +32 16 322987
Celestijnenlaan 300B                  bob dot koninckx at mech dot kuleuven dot ac dot be
B-3001 Leuven Belgium               http://www.mech.kuleuven.ac.be/pma
----------------------------------------------------------------------

Attachment: if_cs8900a.c.patch
Description: Text document