[cholero]

  There is a merge conflict with the latest trunk.
firmware/common/rbpaths.c does not exist anymore.
The related functionality is now located in
firmware/target/hosted/filesystem-app.c

 
Thanks for your information. I know, I'm still changing from revision a731780. Every command I used that has to do with git or gerrit was the first time for me (yes I'm a git-noob). I don't know if I can update the current patchset. If you know how, please tell me...
 

  Noticing too that the boot selector code isn't part of the gerrit.
The original one of Ilya located here
https://bitbucket.org/isergachev/dx50sw/src
which is already adapte by you.
 
I would guess it might go somewhere into
rbutil/ibassopatcher
similar to the rbutil/ipodpatcher

 
Yes, thank you. I had no idea where to put it. I will include it there. I never opened that folder (rbutil) before, to be honest.
 
I get more help from head-fi than I expected. Thank you guys!

 

[Brendanus]

   
Thanks for your information. I know, I'm still changing from revision a731780. Every command I used that has to do with git or gerrit was the first time for me (yes I'm a git-noob). I don't know if I can update the current patchset. If you know how, please tell me...

Hm, I don't know as well. You already created some patch sets, can't you create a new one from the latest trunk revision?
Anyhow, maybe you can ask the rockbox devs by adding a new comment to gerrit?

 

[Brendanus]

   
Thanks for your information. I know, I'm still changing from revision a731780. Every command I used that has to do with git or gerrit was the first time for me (yes I'm a git-noob). I don't know if I can update the current patchset. If you know how, please tell me...

I'm not too familiar with git as well (and I don't know gerrit at all), but I guess what you need to do is a "git pull --rebase" on your local rockbox git repo. That will apply your local changes ontop of the pulled rockbox master.
That will then cause then a merge conflict with that missing rbpaths.c file, where you can choose to delete it.
Afterwards you can adapt then the filesystem-app.c
But I have no idea if that is sufficient in the context of gerrit.

 

[cholero]

I think merge commits are not possible on gerrit. I've exactly tried bevore what you wrote and it would have been a different commit, no more in with the same id, commit --amend is not possible anymore.
For the moment I leave it as it is.
For you, to compile it by yourself, don't expect any additional features, it is the same as my 08-27-classic build.

 

[Brendanus]

I think merge commits are not possible on gerrit. I've exactly tried bevore what you wrote and it would have been a different commit, no more in with the same id, commit --amend is not possible anymore.
For the moment I leave it as it is.
For you, to compile it by yourself, don't expect any additional features, it is the same as my 08-27-classic build.

I see, thanks for your info.

 

[cholero]

But @Brendanus I don't want to hold you back from setting up the compiling environment for the DX50 port, in contrary, please do it! Everyone who has set it up, finds out something more to optimize or modify the port (see all the thing that headwhacker tries out for DX90).
It is very interesting to get a bit into the code of rockbox!

 

[Brendanus]

  But @Brendanus I don't want to hold you back from setting up the compiling environment for the DX50 port, in contrary, please do it! Everyone who has set it up, finds out something more to optimize or modify the port (see all the thing that headwhacker tries out for DX90).
It is very interesting to get a bit into the code of rockbox!

Indeed.
I've managed to install the android sdk and ndk, and now I can compile the rockbox by my own. (I had to fiddle around a while to get the correct android version installed via the eclipse gui). Fortunately I'm working on a native linux PC, compiling under Windows seems not to work according
http://www.rockbox.org/wiki/AndroidPort
I've applied also your latest gerrit commit (thread size increase), that solves not only the gather runtime data, but as well the database initialization itself.
The number of songs on my device is rather big (>5000), and trying to init the database resulted in a panic, now that is solved as well with the bigger stack size.

 

[cholero]

  Indeed.
I've managed to install the android sdk and ndk, and now I can compile the rockbox by my own. (I had to fiddle around a while to get the correct android version installed via the eclipse gui). 

Great, linux all the way 

. It is a very good feeling to know that there is someone else who can compile the port and is looking at the code.
 

I've applied also your latest gerrit commit (thread size increase), that solves not only the gather runtime data, but as well the database initialization itself.
The number of songs on my device is rather big (>5000), and trying to init the database resulted in a panic, now that is solved as well with the bigger stack size.

 
Great. I was not aware of the database init problem. The stack size increasation came from headwhacker, thanks to him.
 
If you got used to my so called "smartphone UI" you can take a cloer look at the first patch set on gerrit.
 
 
And if you want homework to do  : it would be great to have an entry in the settings menu to switch between usb mass_storage and charge only (with adb)
This is something i would like to look at next and something else: headwhacker has changed many paths in his repo from /sdcard to /mnt/sdcard if I'm correct. I want to  find out if these problems exist also on DX50 and if they cannot be solved differently.

 

[Brendanus]

Originally Posted by cholero /img/forum/go_quote.gif
 
If you got used to my so called "smartphone UI" you can take a cloer look at the first patch set on gerrit.
 

Did that already, made a mixture that suites my personal taste.

 

[Brendanus]

  And if you want homework to do  : it would be great to have an entry in the settings menu to switch between usb mass_storage and charge only (with adb)

???
Actually I never used the adb, I'm creating always cwm rockbox zips, installed via the CWM menu

 

[cholero]

I like this better, too, but if you want to try out something, whith adb you need maybe 30 seconds after compiling and you know if it works. With CWM i have maybe 5 min.
Adb is also very handy if you want to use the console output for debugging things. If you write printf("My variable: %d\n",myvariable);  for debugging in your .c file, you will have this output in your adb console if you start the program. Otherwise you have to write the debug information to files (which is not bad neither but not "live").

 

[headwhacker]

  And if you want homework to do  : it would be great to have an entry in the settings menu to switch between usb mass_storage and charge only (with adb)
This is something i would like to look at next and something else: headwhacker has changed many paths in his repo from /sdcard to /mnt/sdcard if I'm correct. I want to  find out if these problems exist also on DX50 and if they cannot be solved differently.

 
@Brendanus If you got adb working or if you are able to extract the system.img file from the OFW look at /system/bin/openadb.
 
This will let you control the USB function betwee adb and mass storage. You can create a similar script with opposite effect name it maybe "closeadb" and you can add in the settings the options to call on these two scripts.
 
It will be nice to have someone else look at the code. two heads are better than one as the saying goes and so on ... 

 

[Brendanus]

I fear I'm not a big help with that, my C programming knowledge is very limited.
Anyhow, some good news from the rockbox devs:
 

Michael Giacomelli

2:22 AM

Patch Set 5: Code-Review+1
I'm satisfied with the state of the port. If no one complains, and you are ready, I'll probably commit it this week.

 

[cholero]

I was just joking when I gave you the "homework". My C knowledge is also very limited.

About the gerrit comment: Michael, he is the (only) one who was always watching the progress and pushing things forward.

 

[cholero]

New custom CWM-flashable build for/with stock Firmware 1.5.0 including following new changes to the official repository:

1. bugfix: dualboot selector was not shown when a cable was connected to lineout (this was my  bug, not xzcc's)
2. bugfix: enabling "gather runtime data" does not cause a crash anymore (big "thank you" to @headwhacker). According to @Brendanus this fix also fixes a general database initialization error.

List of all old modifications to xzcc's build 2014-06-30:

1. longpress of play button in menu works correctly
2. new dual-boot mechanism (Hold-Switch enables boot-selector, otherwise last player is booted)
3. Headphone output click -> connected on playback not at startup 
4. Battery level calculated from voltage
5. working plugins: all (only startup and keymap tested)
6. "Smartphone UI Style" (modded backlight and button behaviour)
7. lock switch works as intended
8. timers@playback not very slow anymore
9. "gather runtime data" works

Download: 
https://www.sendspace.com/file/ew18mi