This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#11137 - This is initial patch that adds MPIO HD200 as a new target.
Attached to Project:
Rockbox
Opened by Marcin Bukat (MarcinBukat) - Monday, 22 March 2010, 15:59 GMT+2
Last edited by Marcin Bukat (MarcinBukat) - Tuesday, 27 April 2010, 11:43 GMT+2
Opened by Marcin Bukat (MarcinBukat) - Monday, 22 March 2010, 15:59 GMT+2
Last edited by Marcin Bukat (MarcinBukat) - Tuesday, 27 April 2010, 11:43 GMT+2
|
DetailsIncluded are:
- basic bootloader (but install instruction is not described yet) - port specyfic files - UI simulator currently it is possible to load bootloader int iram with BDM and than boot rockbox. Many things does not work yet. |
This task depends upon
Closed by Marcin Bukat (MarcinBukat)
Tuesday, 27 April 2010, 11:43 GMT+2
Reason for closing: Accepted
Additional comments about closing: Commited as r25724, r25725 and r25726
Tuesday, 27 April 2010, 11:43 GMT+2
Reason for closing: Accepted
Additional comments about closing: Commited as r25724, r25725 and r25726
About the "4 char rule" it's not a problem because we only use it to determine the model number, it can be arbitrarily long as long as it is given as argument to scramble (just saw that, I would have used longer names if i knew..)
Here some remarks:
- I see there are a lot of places where DRAM start is defined to 0x31000000 : could you put that in an header? like firmware/target/coldfire/mpio/hd200/system-target.h
- did Dave Chapman really write wm8750.h 5 years ago ?
- config.h has a FIXME for tuner, can the FIXME be removed, or tuner be commented out ?
- same for pcm-coldfire.c, wmcodec-coldfire.c
- there is an added chunk in coldfire/crt0.S : can you add a comment for this one ?
If you are not sure about how to implement lcd_blit_*() you could ask amiconn on irc
- DRAM start define - I took existing CF ports (iriver,iaudio) as a template and there were no such define.
- wm8750.h is for now copy of wm8751.h (wm8751 is little brother of wm8750). Basic register addresses and fields are the same for both DACs. When I make playback working I am going to extend it to support additional capabilities.
- I think lcd_blit_*() are not critical for now. There are much more important features to develop.
changes in patch:
- commented out unused (for now) options in config/mpiohd200.h. This forces some ifdefs in pcm-coldfire.c
- added more comments in code and remove FIXMEs
in wm8751.c:
#ifdef HAVE_WM8750
/* specific code */
#endif
Unless they differ too much (and you run into a #ifdef nightmare), code should be shared as much as possible i think
Graphic for UI simulator was changed a bit also.
FS#11153.2. Extended delay between setting adc channel and reading. This make battery reading more stable. I don't know why buttons are not affected...
3. ata_enable() is empty now. It makes USB bridge mode work in rockbox. Idealy it should contain handling of GL811E reset line but I am not sure which one it is.
4. Hacked ata-as-coldfire.S to include byteswaps. Now it is moved to ata-as-mpio.S but maby it would be better to integrate the two with #ifdef BYTESWAP magic. Only optimized reading routine is used. I did not measured what is the impact of the change.
5. Changed behaviour of bootloader to reset instead of poweroff in case AC wall adapter is connected.
6. Included lcd_shutdown() as a part of power_off()
I found out that playing mp3 needs ~108MHz and playing ogg needs ~121MHz. This is much much more than raported in http://www.rockbox.org/wiki/CodecPerformanceComparison for other Coldfire based DAPs. The delta between mp3 and ogg is comparable though.
2) moved ADC reading to ISR
3) enabled plugins (keymaps and graphics may need fine tunning)
4) begin work on radio support (not finished yet)