[Bug 1657] New BSP code inclusion in CVS HEAD - Milkymist

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1657

--- Comment #8 from Sebastien Bourdeauducq <sebastien at milkymist.org> 2010-08-19 04:51:10 CDT ---
+ The AC97 isn't complete and thus not ready for merging yet (afaik). Please
name the audio device /dev/dsp (as OSS does) instead of /dev/snd.

+ ac97.c, gpio.c, system_conf.h, README (and maybe others) should have $Id:$
(not just $Id$)

+ ckinit.c, console.c, uart.c, clock.h, uart.h, mm_crc32.c, network.c,
network.h (and maybe others) file headers still say "for Lattice Mico32",
please change that to avoid confusion with the LM32 EVR drivers.

+ timer.c: "It is important that the timer start/stop overhead be determined
when porting or modifying this code." -> did you do that?

+ The framebuffer API extension for multiple buffering should be documented.

+ in milkymist.cfg, this is a bit inconsistent:
# optimize flag: typically -O2
# ATM, doesn't work with optimization levels > 0
CFLAGS_OPTIMIZE_V = -O2 -g

+ in linkcmds: upgrade RAM size to 128M, the bug that prevented access to the
upper 64M has been fixed in the SoC.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.