From: Brad P. <br...@he...> - 2003-10-02 19:16:28
|
Hi I made some changes to the blob+lineo code for my A400 board and got disgusted and did some refactoring of their code (mostly to look the 4,000,000 #ifdef's they put everywhere). Then, while working on an xscale project I noticed the latest blob cvs seems to have done a lot of this refactoring already and integrated some new cpu's cleanly. So I add "put your changes into the newest blob cvs" to my to-do list and promptly started ignoring it :-) My biggest beef with the lineo code is the use of "LH7A400" when they really ment KEV7A400 and "LH79520" when they really ment KEV79520. i.e. mixing the notion of the cpu with the board instantiation as if there would only ever be one A400 board in the world. So if some/any/all of the lineo code gets put in, please define the Sharp eval boards as KEV7A400 and KEV79520 like this snipped below (from configure) ... cat >> confdefs.h <<\EOF #define LH79520 1 EOF BLOB_PLATFORM_OBJ="kev79520.o" BLOB_FLASH_OBJS="sharp_lh28f320.o" use_cpu="lh79520" ... that way we can have multiple boards which use the same cpu and not all pull our hair out. I sure you know all this, and sorry for the rant - I had some pent up frustration I needed to release :-) [and having said that, I'm happy to build and test blob for the KEV79520 and KEV7A400 since I have one of each right now, as well as my own A400 board] -brad Marc Singer wrote: >I think that 'clean' is part of the problem. I'm working with the >Lineo patches for the Sharp ARM SoC's. Have you seen their work? I'm >making some changes so that their code works properly with another >implementation of the Sharp CPU. > >I'll post something when I've got it working. > >On Thu, Oct 02, 2003 at 10:06:31AM -0700, Christopher Hoover wrote: >> >> > Also, I'm looking at the state of blob wrt there being no >> > maintainer. I have patches, too, that need to be integrated.n >> >> Send the patches (in clean, concise pieces, i.e., standard patch submission >> technique) to blo...@li.... >> >> One of us with write access to cvs will apply them, if they are acceptable. >> >> -ch >-- >To unsubscribe from this list: send the line "unsubscribe lart" in >the body of a message to maj...@la... >Please read the LART FAQ at http://www.lart.tudelft.nl/faq.php3 > |