| 00:00:23 | AzureSky | k, just seemed weird |
| 00:00:52 | stripwax | yep, it's normal |
| 00:04:07 | stripwax | ok, database seems sane again after reinitialising. pretty odd bug |
| 00:07:04 | *** | Saving seen data "./dancer.seen" |
| 00:07:54 | AzureSky | ok i have a problem compiling http://i45.tinypic.com/oadxmc.jpg |
| 00:08:18 | funman | make clean |
| 00:08:41 | AzureSky | thx |
| 00:08:48 | AzureSky | sorry thanks |
| 00:09:53 | pixelma | stripwax: great chance to investigate ;) |
| 00:13:03 | | Quit GeekShadow (Quit: The cake is a lie !) |
| 00:14:09 | funman | what do you think of adding a (possibly #ifdef DEBUG) per-thread backtrace buffer? |
| 00:14:51 | funman | would be useful when an exception happens in memset/memcpy & friends |
| 00:21:01 | TheSeven | funman: I already wondered if a perror()-like error buffer would make sense, that would be displayed on the panic screen, to catch problems that cause things like data aborts because of missing error handling |
| 00:21:16 | TheSeven | regarding a backtrace buffer: who should fill it? |
| 00:21:52 | funman | TheSeven: so write this buffer each time we modify errno? |
| 00:22:16 | funman | about the buffer, i don't know but we could look at what glibc does, i suppose it's a gcc feature |
| 00:22:21 | TheSeven | basically keep the most recent error message around in case the return code was just ignored |
| 00:22:32 | TheSeven | we might just remove -fomit-frame-pointer and do stack unwinding |
| 00:23:02 | funman | how does unwinding work? |
| 00:23:12 | funman | i've seen unwind.o files in libgcc |
| 00:23:51 | TheSeven | if you have frame pointers, you can reconstruct the return addresses and call arguments from the stack without needing to know anything about the functions |
| 00:24:21 | TheSeven | hm, arguments only if they aren't passed in registers :-/ |
| 00:24:33 | amiconn | That comes at a cost which isn't adequate, imo |
| 00:24:40 | funman | amiconn: which cost? |
| 00:24:40 | TheSeven | but still, knowing the code path that lead to some crash might be helpful |
| 00:24:56 | TheSeven | amiconn: I don't really care about cost for debug builds :-) |
| 00:24:58 | funman | having to remove -fomit-frame-pointers from my builds is adequate enough for me |
| 00:25:03 | amiconn | hmm |
| 00:25:19 | amiconn | More binsize and more register pressure |
| 00:25:35 | amiconn | For debug builds it should be okay - if you get it to build |
| 00:25:40 | TheSeven | and a tiny bit more stack usage |
| 00:26:18 | funman | libgcc/unwind-c.o just needs abort(), but perhaps we can write a simple unwind |
| 00:26:24 | amiconn | Some asm blocks are already difficult to get to compile without frame pointer, because of the amount of registers they use |
| 00:27:07 | funman | frame pointer reserves a register, right? is that ip ? |
| 00:27:19 | amiconn | No, fp |
| 00:27:33 | funman | r? |
| 00:27:36 | TheSeven | 12 iirc |
| 00:27:49 | * | TheSeven wonders what ip is a synonym for |
| 00:27:54 | funman | ip is r12 on arm |
| 00:28:25 | TheSeven | oh, frame pointer is r11 |
| 00:29:06 | | Quit ender` (Quit: A computer program will always do what you tell it to, and seldom what you want it to.) |
| 00:29:21 | TheSeven | r12 seems to be just a scratch register |
| 00:29:45 | funman | Intra Procedure call scratch register |
| 00:31:54 | | Part domonoky |
| 00:32:33 | | Join stripwax_ [0] (~Miranda@87-194-34-169.bethere.co.uk) |
| 00:34:18 | AzureSky | saratoga's volume patch works quite well, but something strange, i updated svn then did a build and the build on my fuze didnt update(says same revision number) rather then r26998 it says r26979 |
| 00:34:31 | | Quit stripwax (Ping timeout: 260 seconds) |
| 00:35:42 | funman | if you want the revision number to be changed, run 'rm version.*' befoer make |
| 00:36:18 | | Quit DataGhost (Ping timeout: 240 seconds) |
| 00:36:33 | | Quit pamaury (Remote host closed the connection) |
| 00:36:33 | pixelma | I never do and my revision number usually changes correctly |
| 00:36:41 | AzureSky | ah thanks |
| 00:38:06 | funman | pixelma: then there's a bug, version.h shouldn't be rebuilt |
| 00:38:36 | pixelma | what? After I update? |
| 00:38:49 | funman | it should never be rebuilt, it has no dependancy |
| 00:39:28 | pixelma | that doesn't make sense to me at all at the moment |
| 00:39:43 | funman | what doesn't make sense? |
| 00:40:51 | pixelma | that the revision number *shouldn't* change after I update and compile again |
| 00:41:39 | funman | well it's a known limitation |
| 00:42:03 | gevaerts | funman: does that mean that the absence of "M" in the revision number no longer implies an unmodified build? |
| 00:42:12 | funman | sure |
| 00:42:14 | gevaerts | hm |
| 00:42:56 | pixelma | how should I know if I test the build I want to test (if I just need to compile, no removing version.h, make clean or reconfigure)? |
| 00:43:12 | funman | svn info |
| 00:43:59 | pixelma | doesn't help me at all on target if e.g. the installed and the rolo'ed build show me the same revision |
| 00:44:19 | * | gevaerts thinks that we need a mechanism to force version.h to be recreated when needed |
| 00:44:31 | funman | make it depend on every file in svn |
| 00:44:34 | gevaerts | Otherwise I don't really see what problem the file solves |
| 00:44:41 | funman | it doesn't solve anything |
| 00:44:43 | pixelma | (e.g. a flashed build on Archos vs. a rolo'ed) |
| 00:45:09 | funman | it just makes it easy to have the version number be correct: use "rm version.*; make" |
| 00:45:26 | pixelma | even if it works this way currently, I don't believe that is the intended way |
| 00:45:42 | gevaerts | hm |
| 00:46:45 | funman | pixelma: if you know an easy way to have version.h be rebuilt each time a file in svn is modified / svn up is run, please tell |
| 00:47:13 | funman | i proposed an unconditional Makefile rule but then make wouldn't output 'nothing to be rebuilt' |
| 00:47:30 | gevaerts | Is that a problem? |
| 00:47:34 | | Quit stripwax_ (Quit: http://miranda-im.org) |
| 00:47:43 | funman | for the devs who objected, it seems |
| 00:48:19 | gevaerts | Currently we risk incorrect version strings. I prefer make not printing "nothing to be rebuilt" to that... |
| 00:48:49 | pixelma | does it still only builds what needs to be rebuild (things that changed, in this case maybe only the version things) |
| 00:48:56 | pixelma | ? |
| 00:49:03 | funman | if someone does it correctly, yes |
| 00:49:31 | gevaerts | from what I can see genversion.sh already handles this correctly, so the only thing that happens is that genversion.sh runs and does nothing |
| 00:49:55 | pixelma | sounds better than incorrect version numbers at a first glance |
| 00:50:00 | gevaerts | If the version changes, of course there's a bit more that gets rebuilt |
| 00:50:13 | | Join BHSPitMini [0] (~BHSPitMon@adsl-70-243-200-93.dsl.rcsntx.swbell.net) |
| 00:50:22 | funman | gevaerts: hm right |
| 00:50:34 | AzureSky | ok tried to run "rm version.* and it says cannot remove 'version.*' : no such file or directory, so i need to run it from inside a specific folder? |
| 00:50:38 | funman | didn't remember that |
| 00:51:03 | funman | gevaerts: s/a bit more/exactly what needs to be rebuilt/ |
| 00:51:12 | gevaerts | well yes |
| 00:51:26 | funman | AzureSky: in the build folder root |
| 00:51:48 | AzureSky | thats where i ran it from |
| 00:53:04 | funman | TheSeven: you got any doc on unwinding handy? |
| 00:53:14 | TheSeven | nope, sorry |
| 00:53:49 | TheSeven | but basically you just follow the stack frames, which point to each other by their frame pointer |
| 00:54:09 | amiconn | gevaerts: Imo it would be best if version.h is only actually rebuilt if the version or date changes |
| 00:54:16 | funman | i wonder how that tells where lr is in this frame |
| 00:54:17 | amiconn | I think that should be possible |
| 00:54:35 | funman | amiconn: obviously |
| 00:54:57 | funman | problem is just Makefile deps |
| 00:55:31 | funman | : when do you check if it version needs change |
| 00:55:58 | amiconn | Can't you just run genversion.sh unconditionally, and then make it only actually rewrite version.h if the content will change? |
| 00:56:14 | gevaerts | yes, it should already do that |
| 00:56:27 | gevaerts | only that bit seems buggy |
| 00:56:33 | funman | http://www.rockbox.org/irc/log-20100527#09:23:25 |
| 00:56:51 | gevaerts | Apart from that firmware.make just needs a .PHONY line I think |
| 00:57:02 | funman | gevaerts: no, it's not run unconditionally |
| 00:57:15 | gevaerts | funman: that's what you need the .PHONY for |
| 00:57:17 | TheSeven | IIRC LR is always immediately above FP in the stack frame |
| 00:57:24 | funman | only when a file depending on version.h has changed |
| 00:57:33 | | Quit bertrik (Quit: De groeten) |
| 00:57:51 | TheSeven | (because FP is basically the parent's SP) |
| 00:58:04 | funman | A phony target is one that is not really the name of a file. It is just a name for some commands to be executed when you make an explicit request. |
| 00:58:28 | funman | TheSeven: but you can have any number of registers before lr ? |
| 00:58:57 | gevaerts | sure, but the result should be the same |
| 00:59:33 | funman | so each explicit rule + the implicit one should depend on this one? |
| 00:59:35 | TheSeven | IIRC a stack frame on ARM looks like this: func2scratch func2FP func2LR func2args func1scratch func1FP func1LR func1args ... |
| 00:59:57 | funman | gevaerts: IIRC i tried that but dependencies are calculated before running the commands so make doesn't see version.h has changed, but feel free to try |
| 01:00 |
| 01:00:30 | amiconn | A leaf function will not (always) put lr on stack |
| 01:00:47 | TheSeven | amiconn: it will, if it calls subfunctions |
| 01:01:03 | amiconn | That's why I said leaf function |
| 01:01:17 | amiconn | If a function calls another one, it isn't a leaf function |
| 01:01:19 | gevaerts | hm, right |
| 01:01:30 | TheSeven | didn't spot that word :-) |
| 01:01:30 | funman | well it's possible to do it anyway |
| 01:01:38 | amiconn | heh |
| 01:01:58 | TheSeven | can one determine if LR is on the stack or still in the reg in a leaf function? |
| 01:02:21 | TheSeven | will it always be on the stack if there are scratch words on the stack? |
| 01:02:29 | amiconn | Same applies to SH1 and its 'pr' register btw |
| 01:03:02 | amiconn | (actually in a more strict way, as pr is separate from the general purpose registers) |
| 01:03:49 | funman | what's "pr" ? |
| 01:04:35 | amiconn | The SH equivalent of arm's lr |
| 01:04:49 | amiconn | Not identical though, as it's not usable as a general register |
| 01:05:03 | amiconn | "procedure return" |
| 01:05:32 | funman | ah fp points to saved "fp, lr" it seems |
| 01:06:26 | amiconn | Coldfire doesn't have an equivalent - it always stacks return addresses. Don't know about mips in that respect |
| 01:07:03 | funman | it's r11 because you shouldn't push r12 (ip), r13 (sp), r15 (pc); so it's always followed by lr or nothing |
| 01:08:11 | funman | perhaps you could try to pop the next word from the stack and if it's different from lr, assume that the real lr comes from the stack |
| 01:08:52 | funman | but that doesn't give you scratch regs content? |
| 01:10:39 | funman | ah i see argument registers are pushed too |
| 01:13:46 | funman | http://sourceware.org/binutils/docs/as/ARM-Unwinding-Tutorial.html <- |
| 01:14:25 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
| 01:16:01 | gevaerts | funman: I suspect I know a way, but I'm not sure yet |
| 01:16:50 | funman | we'd need to modify asm functions to save fp :/ |
| 01:17:17 | gevaerts | The trick I'm thinking of is to put the genversion.sh logic directly in a makefile, and have it write out a makefile fragment that directly echoes the version number to version.h, and then include that makefile fragment |
| 01:18:11 | | Quit bluebroth3r (Ping timeout: 265 seconds) |
| 01:20:09 | funman | looks messy but let's see how that works |
| 01:21:00 | gevaerts | yes indeed :) |
| 01:22:02 | | Join binaryhermit [0] (~binaryher@adsl-99-141-184-240.dsl.emhril.sbcglobal.net) |
| 01:22:29 | | Join Buckwheat [0] (~chatzilla@64.68.182.251) |
| 01:22:30 | | Quit Buckwheat (Client Quit) |
| 01:22:54 | | Join Buckwheat [0] (~chatzilla@64.68.182.251) |
| 01:23:16 | | Quit Buckwheat (Client Quit) |
| 01:24:36 | | Quit DerPapst (Read error: Connection reset by peer) |
| 01:27:30 | funman | clipv2 builds fine without -fomit-frame-pointer |
| 01:28:56 | | Join DerPapst [0] (~Alexander@p4FE8F654.dip.t-dialin.net) |
| 01:40:24 | funman | adding -O0 doesn't build |
| 01:41:56 | funman | *_irq() macros want mask to be a reg, not an immediate |
| 01:43:00 | funman | mdct doesn't build .. |
| 01:46:13 | funman | but then, text_viewer does work fine Oo |
| 01:50:04 | CIA-8 | New commit by gevaerts (r26999): Fix genversion.sh file replacement logic |
| 01:51:41 | CIA-8 | r26999 build result: All green |
| 01:53:51 | funman | gevaerts: i think it is wrong |
| 01:54:03 | gevaerts | funman: it's right, but not all to fix this yet |
| 01:54:10 | funman | % if diff /dev/null /dev/null; then echo 1; else echo 2; fi |
| 01:54:10 | funman | 1 |
| 01:54:23 | CIA-8 | New commit by gevaerts (r27000): Add some magic to force make to regenerate version.h properly when the revision changes |
| 01:54:26 | funman | hmm oops i'm reading the other side |
| 01:54:41 | gevaerts | /dev/null is not readable, so diff will fail |
| 01:54:51 | funman | it sure is |
| 01:56:09 | CIA-8 | r27000 build result: 266 errors, 0 warnings (gevaerts committed) |
| 01:56:18 | | Join thefirstM [0] (~quassel@Brag-08-228.rh.ncsu.edu) |
| 01:57:21 | gevaerts | How did I do that? |
| 01:57:29 | TheSeven | what the hell |
| 01:57:39 | AzureSky | nice!!! |
| 01:58:17 | funman | if you think rockbox is slow, try building with -O0 ! |
| 01:59:05 | funman | gevaerts: version.h needs an argument |
| 02:00 |
| 02:00:01 | gevaerts | funman: where? |
| 02:00:23 | funman | version.sh* |
| 02:00:40 | gevaerts | oh |
| 02:02:38 | CIA-8 | New commit by gevaerts (r27001): Tell version.sh where to look for the svn repository |
| 02:03:17 | CIA-8 | New commit by funman (r27002): debug menu: avoid using constant variables in array initialization ... |
| 02:04:05 | | Quit Dark_Rak3r (Ping timeout: 272 seconds) |
| 02:04:25 | CIA-8 | r27001 build result: All green |
| 02:06:17 | CIA-8 | r27002 build result: All green |
| 02:07:06 | *** | Saving seen data "./dancer.seen" |
| 02:11:20 | gevaerts | funman: is there actually a reason to autogenerate version.c? In my tests, generating version.h seems to be enough |
| 02:11:53 | funman | the symbol must be somewhere |
| 02:12:12 | gevaerts | sure, but version.c can be in firmware/common |
| 02:12:19 | funman | true |
| 02:12:26 | gevaerts | it doesn't matter much of course |
| 02:13:37 | funman | ranma: ping |
| 02:13:38 | * | TheSeven is fighting the last on-stack sector buffer |
| 02:13:52 | gevaerts | except that if version.h is newer than version.c, make keeps trying to regenerate version.c, which is a bit useless |
| 02:14:15 | funman | first script created them both at the same time |
| 02:15:26 | | Quit S_a_i_n_t (Read error: Connection reset by peer) |
| 02:18:01 | | Join S_a_i_n_t [0] (S_a_i_n_t@203.184.1.161) |
| 02:18:32 | CIA-8 | New commit by gevaerts (r27003): Don't autogenerate version.c. Just version.h is enough |
| 02:18:44 | funman | i still hear glitches when playing mp3 on clip+ and there's some failure reports on ABI |
| 02:19:59 | CIA-8 | r27003 build result: All green |
| 02:20:37 | gevaerts | If that last bit causes problems I didn't see, just revert it... |
| 02:20:47 | funman | looks ok |
| 02:21:12 | funman | i haven't used AMSv1 for some time so please tell if current builds still work fine or cause glitches |
| 02:22:49 | | Quit CGL_party (Read error: Connection reset by peer) |
| 02:23:11 | funman | ranma: can you make text_viewer plugin crash on c200v2 and share your text_viewer.elf+rockbox.elf and memory dump? i couldn't see where the bug comes from except that the plugin works when building rockbox with non-eabi compiler or with -O0 |
| 02:23:20 | | Quit funman (Quit: free(random());) |
| 02:23:55 | Unhelpful | perhaps we could report build results only when there are yellows/reds? |
| 02:39:40 | | Quit Dhraakellian (Ping timeout: 264 seconds) |
| 02:56:14 | | Join Dhraakellian [0] (~ntryon@cpe-67-240-248-41.rochester.res.rr.com) |
| 02:59:08 | TheSeven | hm, that sectorbuffer-on-stack-free build seems to work, but it boots up dog slow |
| 02:59:49 | | Quit kadoban (Read error: Connection reset by peer) |
| 02:59:53 | TheSeven | and the database still seems to have bad buffers somewhere |
| 03:00 |
| 03:05:44 | | Join halmi [0] (~netbook@80-123-47-176.adsl.highway.telekom.at) |
| 03:05:50 | | Quit halmi (Read error: Connection reset by peer) |
| 03:09:09 | | Quit thefirstM (Remote host closed the connection) |
| 03:09:58 | | Join kadoban [0] (~mud@cpe-67-247-80-129.rochester.res.rr.com) |
| 03:25:47 | | Quit efyx (Remote host closed the connection) |
| 03:36:21 | S_a_i_n_t | funman, pixelma: S_a_i_n_t would like to point out that his revision number changes correctly after updates. |
| 03:36:42 | S_a_i_n_t | reading that this is a bug I find quite odd, that it is intentional, plain silly. |
| 03:42:30 | | Quit BHSPitMini (Quit: Ex-Chat) |
| 03:47:33 | | Quit steve|m (Ping timeout: 276 seconds) |
| 03:54:47 | | Quit detaos (Ping timeout: 265 seconds) |
| 03:57:08 | S_a_i_n_t | s/is/was/ s/find/found/ backlog reveals I'm now talking inpast tense. Great, thanks gevaerts. |
| 04:00 |
| 04:01:07 | | Join steve|m [0] (~steve@p4FD47D3D.dip.t-dialin.net) |
| 04:02:49 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
| 04:07:09 | *** | Saving seen data "./dancer.seen" |
| 04:09:50 | | Join thefirstM [0] (~quassel@Brag-08-228.rh.ncsu.edu) |
| 04:13:39 | | Join Barahir_ [0] (~jonathan@frnk-590f5b11.pool.mediaWays.net) |
| 04:15:03 | | Quit fdinel (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
| 04:16:12 | | Join retroj [0] (~retroj@unaffiliated/retroj) |
| 04:16:28 | thefirstM | I am trying to build Rockbox with GCC 4.5 on ipod nano2g, but for some reason playing music at the WPS doesn't work. I get an "Undefined instruction at E5958718" error. How might I figure out what source code corresponds to that memory address? |
| 04:16:54 | retroj | morning. just popping by to report broken links for ipodvideo manuals at http://www.rockbox.org/manual.shtml |
| 04:17:23 | | Quit Barahir (Ping timeout: 276 seconds) |
| 04:24:54 | S_a_i_n_t | thefirstM: "rockbox.map" should tell you what that address is IIUC |
| 04:25:09 | thefirstM | Thanks! |
| 04:26:00 | S_a_i_n_t | have a look in the .dir you built in, it'll be in there unless you cleaned up afterward. |
| 04:26:15 | thefirstM | Its still there. I am checking now. |
| 04:28:55 | thefirstM | That's weird. I can't find it in there. There is a bunch of stuff starting with 0x08 and few things with 0x00, but nothing in the range I am looking for. |
| 04:29:21 | | Join saratoga [0] (9803c6dd@gateway/web/freenode/ip.152.3.198.221) |
| 04:29:32 | thefirstM | What base is the number from the error message in? |
| 04:29:35 | S_a_i_n_t | I was just looking at mine, it doesn't resemble any value in the .map file. |
| 04:29:49 | saratoga | thats pretty common actually, since one of the normal ways to get an undefined instruction is to load from something thats not memory :) |
| 04:30:04 | thefirstM | Oh. Duh! |
| 04:30:04 | S_a_i_n_t | *facepalm* |
| 04:30:06 | S_a_i_n_t | indeed. |
| 04:30:32 | saratoga | probably means the binary contained nonsense |
| 04:30:48 | saratoga | unfortunately, thats going to make tracking down your issue fairly hard |
| 04:31:40 | thefirstM | Most of it (everything except for that problem) seems to work just fine, including the plugins. |
| 04:32:41 | | Join DerPapst1 [0] (~Alexander@p4FE8F1F8.dip.t-dialin.net) |
| 04:33:20 | saratoga | but audio playback doesn't work at all? |
| 04:33:43 | thefirstM | No, outside of the WPS, audio works just fine. (I tried mpegplayer and Doom) |
| 04:34:22 | saratoga | well that narrows the problem down to about 2/3 of all rockbox :) |
| 04:34:39 | | Quit amiconn (Disconnected by services) |
| 04:34:41 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
| 04:34:45 | | Quit DerPapst (Ping timeout: 272 seconds) |
| 04:34:59 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
| 04:35:06 | JdGordon | thefirstM: which theme are you using? |
| 04:35:20 | thefirstM | Cabbiev2 (the default) |
| 04:35:22 | JdGordon | very possibly it failed to load the theme and couldnt fallback on the default |
| 04:35:26 | JdGordon | bah |
| 04:35:30 | | Quit pixelma (Disconnected by services) |
| 04:35:33 | | Join pixelma_ [0] (quassel@rockbox/staff/pixelma) |
| 04:35:38 | thefirstM | Should I try a different theme? |
| 04:35:48 | | Nick pixelma_ is now known as pixelma (quassel@rockbox/staff/pixelma) |
| 04:36:03 | JdGordon | maybe |
| 04:36:24 | saratoga | try wav decoding |
| 04:37:04 | thefirstM | Trying a different theme does not work, I still get the same error at the same address. .wav files also produce the same error. |
| 04:40:49 | thefirstM | I do see just a flash of the WPS before I get the error. However, no audio is played before the crash. |
| 04:41:24 | S_a_i_n_t | sounds like it is failing to load the default WPS and the user selected one, then quitting. |
| 04:41:57 | thefirstM | Is there any way I can try playing the music without the WPS? |
| 04:42:06 | | Quit Strife89 (Quit: Leaving) |
| 04:42:17 | S_a_i_n_t | ...though, if that were the case you shouldn't even see a "flash" of it. |
| 04:42:17 | thefirstM | I tried loading it (an MP3 file) into mpegplayer, but that didn't work. |
| 04:42:31 | S_a_i_n_t | and, no, there isn't IIUC |
| 04:43:42 | | Join Strife89 [0] (~Strife89@adsl-67-57-76.mcn.bellsouth.net) |
| 04:45:34 | thefirstM | Are you guys planning to switch to GCC 4.5 at any point? |
| 04:46:26 | Unhelpful | i think some folks are starting to experiment with it for link-time optimization. we only *just* switched ARM to 4.4 so we could use EABI ;) |
| 04:47:02 | Strife89 | Unhelpful: Oh? What revision? |
| 04:47:20 | thefirstM | Yeah, I saw a conversation on here about that the other day. I guess they aren't having undefined instruction problems. |
| 04:47:21 | Strife89 | Sounds like I've been away from my compiler for too long. |
| 04:49:34 | JdGordon | thefirstM: oh, you''re not using the recommended compiler? that could very well explain it |
| 04:49:46 | * | Unhelpful points to r26769: "Make EABI gcc the default for ARM platforms" |
| 04:50:25 | thefirstM | JdGordon: Sorry, I thought you had read that part. |
| 04:50:27 | Strife89 | Unhelpful: Yeah, it's definitely been a while. |
| 04:51:23 | thefirstM | Is it possible to create a memory dump from rockbox? |
| 04:51:33 | Strife89 | Unhelpful: Call me a noob, but I suppose I need to install a new cross-compiler to use it? |
| 04:51:52 | thefirstM | Strife89: Yes |
| 04:51:56 | | Quit panni_ (Read error: Connection reset by peer) |
| 04:52:23 | thefirstM | Just press "e" after executing rockboxdev.sh |
| 04:53:06 | S_a_i_n_t | and clear out existing cross-compilers first. |
| 04:53:12 | S_a_i_n_t | Or expect weird errors. |
| 04:53:29 | thefirstM | Oh, yes. I shot myself in the foot with that. Several times. |
| 04:53:42 | Unhelpful | S_a_i_n_t: um, why should he need to do that? the eabi toolchain has a different executable prefix... |
| 04:54:04 | S_a_i_n_t | Unhelpful: I had to, several others had to also. |
| 04:54:10 | S_a_i_n_t | Why? I can't say. |
| 04:54:17 | S_a_i_n_t | but it cuases problems. |
| 04:54:34 | | Join BHSPitMonkey [0] (~stephen@unaffiliated/bhspitmonkey) |
| 04:55:11 | Unhelpful | worst case you should only have to clear them from your path :P |
| 04:55:50 | S_a_i_n_t | *should*. |
| 04:56:04 | S_a_i_n_t | things don't always seem to make sense though ;) |
| 04:56:07 | | Quit TheSeven (Ping timeout: 240 seconds) |
| 04:56:24 | S_a_i_n_t | My eabi install just wouldn;t work with the existing toolchain in place. |
| 04:56:45 | S_a_i_n_t | cleared it, bingo. |
| 04:56:46 | * | Unhelpful will simply assume this to mean that you screwed something up ;) |
| 04:57:03 | S_a_i_n_t | I've seen similar comments here also. |
| 04:57:18 | S_a_i_n_t | something regarding "as" I didn't really parse too well. |
| 04:57:28 | | Quit MethoS- (Read error: Connection reset by peer) |
| 04:59:09 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
| 04:59:33 | saratoga | for what its worth i did not have to clear out the cross compilers before building EABI |
| 04:59:45 | saratoga | rockboxdev.sh just worked for me on ubuntu |
| 05:00 |
| 05:01:44 | | Quit kadoban (Remote host closed the connection) |
| 05:01:47 | Unhelpful | see! ;) |
| 05:02:10 | | Join TheSeven [0] (~TheSeven@rockbox/developer/TheSeven) |
| 05:03:36 | | Join JdGordon| [0] (~jonno@rockbox/developer/JdGordon) |
| 05:04:56 | | Join kadoban [0] (~mud@cpe-67-247-80-129.rochester.res.rr.com) |
| 05:05:34 | | Quit Rob2222 (Quit: Rob2222) |
| 05:08:14 | saratoga | we should really do a new rbutil release so that theme installing works again |
| 05:08:39 | * | S_a_i_n_t suggests that missing currupted thems get fixed up too. |
| 05:08:53 | S_a_i_n_t | but, tha doesn't seem to be much a priority |
| 05:09:13 | S_a_i_n_t | *missing/currupted |
| 05:16:28 | | Quit JdGordon| (Quit: leaving) |
| 05:17:06 | | Join JdGordon| [0] (~jonno@rockbox/developer/JdGordon) |
| 05:17:29 | | Quit JdGordon| (Client Quit) |
| 05:25:23 | saratoga | don't you spend a lot of time looking at wps code? maybe figuring out what the converter couldn't handle would help |
| 05:26:57 | S_a_i_n_t | It seems to have skipped a few purely at random, and I've noticed it doesn't like coordinates after the backdrop %Xd line...but other than that I've noticed no real "pattern" to what themes were missed. |
| 05:27:37 | S_a_i_n_t | and a few seem to have has a % added to the front of every character in the WPS. :/ |
| 05:28:37 | S_a_i_n_t | I managed to contactthe author of one particularly affected theme as contacting themesite admins was beginning to get painful. |
| 05:29:59 | S_a_i_n_t | Oh, sublines barf on some also, the thing is I know of at least one error that escapes checkwps, so running the conversion globally from the old themesite would perhaps be a good thing to try. |
| 05:30:18 | S_a_i_n_t | The skinupdater recieved at least one fix since it was run initially. |
| 05:31:20 | S_a_i_n_t | "%Xd(backdropimage) 0| 0|" seems to pass checkwps :/ but fails on device. |
| 05:32:35 | saratoga | do the ones it skips update if you manually run the update script on them? |
| 05:34:19 | S_a_i_n_t | Mine did at least. It skipped two of my themes, then when I ran the skinupdater it outputted perfect syntax. |
| 05:34:32 | S_a_i_n_t | I need to make a list of all that failed, but it's a daunting task. |
| 05:34:48 | S_a_i_n_t | there's also no way of knowing which ones "passed", but are actually broken. |
| 05:35:00 | S_a_i_n_t | (other than trial and error) |
| 05:37:11 | | Quit Horscht (Quit: Verlassend) |
| 05:54:41 | | Quit kramer3d_ (Quit: Leaving) |
| 05:55:44 | AzureSky | m |
| 05:55:57 | S_a_i_n_t | n, o, p? |
| 05:56:17 | AzureSky | sorry was trying to type make clean into cygwin |
| 05:56:29 | | Join jennifur [0] (~jennifur@cpe-72-224-19-1.nycap.res.rr.com) |
| 05:59:51 | | Quit binaryhermit (Ping timeout: 260 seconds) |
| 06:00 |
| 06:07:11 | *** | Saving seen data "./dancer.seen" |
| 06:13:16 | | Quit whydoubt (Ping timeout: 265 seconds) |
| 06:14:03 | | Quit BHSPitMonkey (Quit: Ex-Chat) |
| 06:20:28 | | Join will [0] (~chatzilla@99.139.149.143) |
| 06:20:51 | will | does the mpeg player typically cause a lot more battery drain than your average audio file? |
| 06:20:59 | | Join whydoubt [0] (~whydoubt@ip68-12-76-9.ok.ok.cox.net) |
| 06:24:22 | JdGordon | will: presumably yes |
| 06:24:35 | JdGordon | bieber: OK, I'm making a new theme in the editor and loving it so far :) |
| 06:24:46 | JdGordon | even just for the viewport positioning helper |
| 06:24:52 | | Quit fdinel (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
| 06:25:10 | bieber | Oh, awesome :) |
| 06:25:22 | bieber | It should be getting a lot more helpful this week |
| 06:26:46 | JdGordon | It would be nice if the UI area was drawn if you are in a sbs |
| 06:27:00 | JdGordon | using a flat colour like 255,0,255 would be good even |
| 06:27:33 | | Quit will (Read error: Connection reset by peer) |
| 06:30:00 | JdGordon | arg, not having the screen magically resize is annoying :p |
| 06:31:29 | JdGordon | what are the magic settings? #screenwidth? |
| 06:33:33 | JdGordon | A, it doesnt like a space between the # and setting name |
| 06:37:56 | JdGordon | bieber: is it possible to add tag parameter hints? i.e if you start typeing %T( it will tell you the expected params? |
| 06:38:17 | bieber | Ooh, that would be nice |
| 06:38:27 | bieber | I'll have to give it a go once I've gotten some more of the rendering done |
| 06:42:51 | JdGordon | also drawing touch regions maybe |
| 06:43:01 | * | JdGordon loves adding to other peoples todo lists :) |
| 06:45:33 | JdGordon | buttons in the sbs work :) |
| 06:45:40 | JdGordon | DAMN that was simple :) |
| 06:46:35 | JdGordon | http://imagebin.ca/view/8spS9wXX.html my uber awesome sbs |
| 06:59:11 | | Quit anewuser (Quit: for SELL 2 by the price of 1 now!) |
| 07:00 |
| 07:08:09 | | Quit shai (Quit: Leaving) |
| 07:11:58 | | Quit whydoubt (Ping timeout: 240 seconds) |
| 07:13:34 | * | JdGordon finds a really odd bug in the settings list screen |
| 07:13:41 | | Join shai [0] (~Shai@l192-117-110-233.cable.actcom.net.il) |
| 07:14:13 | JdGordon | no, all lists when using touchscreen regions in the sbs |
| 07:15:10 | S_a_i_n_t | bug == ? |
| 07:16:08 | S_a_i_n_t | ARG!?! |
| 07:16:18 | S_a_i_n_t | those aren't the arrows I made, surely. |
| 07:17:16 | JdGordon | no, these are easier to test with :) |
| 07:17:21 | JdGordon | I dont need good looking to test |
| 07:17:23 | JdGordon | mspaint ftw |
| 07:17:24 | S_a_i_n_t | no, no they're not. S_a_i_n_t breathes a sigh of relief. |
| 07:17:43 | S_a_i_n_t | I made mine with mspaint I'll have you know! |
| 07:17:49 | S_a_i_n_t | and mine are pretty! :P |
| 07:18:41 | | Quit storm` (Ping timeout: 265 seconds) |
| 07:18:51 | JdGordon | it appears the current list handling touchscreen code is braindead! |
| 07:19:16 | JdGordon | and by that I mean I'm using it wrong :p |
| 07:19:24 | | Join hebz0rl [0] (~hebz0rl@dslb-088-067-204-104.pools.arcor-ip.net) |
| 07:19:27 | JdGordon | but I cant say that because IIRC it is mostly my code anyway :D |
| 07:19:41 | | Join whydoubt [0] (~whydoubt@ip68-12-76-9.ok.ok.cox.net) |
| 07:19:53 | S_a_i_n_t | ots possible to use your own implementation wrongly, I do it all the time :P |
| 07:19:59 | S_a_i_n_t | *its even |
| 07:23:30 | | Join storm` [0] (~lol@ip98-179-182-180.ri.ri.cox.net) |
| 07:28:09 | Mode | "#rockbox +o pixelma" by ChanServ (ChanServ@services.) |
| 07:29:15 | Topic | "Daily manuals currently available at http://www.alice-dsl.net/dominik.riebeling/rockbox/ | Rockbox 3.6 now released! | Please read before speaking: http://www.rockbox.org/wiki/IrcGuidelines | Please direct offtopic/social chat to #rockbox-community | This channel is logged at http://www.rockbox.org/irc" by pixelma (quassel@rockbox/staff/pixelma) |
| 07:29:39 | Mode | "#rockbox -o pixelma" by ChanServ (ChanServ@services.) |
| 07:30:16 | AzureSky | got a compile error, any advice to fix it would be helpfull http://i48.tinypic.com/25jguf4.jpg |
| 07:38:35 | * | S_a_i_n_t wonders if JdGordon or bieber or anyone really would be kind enough to compile an SVN themeeditor for Win. |
| 07:38:48 | S_a_i_n_t | Qmake and I are having some issues still :/ |
| 07:39:09 | JdGordon | you probably need to ping bluebrother.... I havnt got the cross compiling stuff giong |
| 07:39:38 | S_a_i_n_t | aha, I do, but QT has paked a massive sad one me :/ |
| 07:39:46 | S_a_i_n_t | *packed even |
| 07:40:29 | AzureSky | a nice theme editor would be great, and any clue how to fix that compile error i have? |
| 07:40:41 | S_a_i_n_t | make clean? |
| 07:41:00 | S_a_i_n_t | if it spits again, then I'll have a serious think about it. |
| 07:41:00 | AzureSky | tried that, i can try it again(3rd time) |
| 07:41:10 | S_a_i_n_t | "make veryclean" |
| 07:41:25 | AzureSky | ok |
| 07:41:48 | AzureSky | strange, even redownloaded the svn clean >.< |
| 07:42:09 | S_a_i_n_t | that won't matter if the build directory is still "dirty" |
| 07:42:31 | JdGordon | S_a_i_n_t: if you find cross compiler instructions I might be able to get you going |
| 07:42:46 | S_a_i_n_t | and, fwiw, if you want to return the checkout to an SVN state just use "svn revert" |
| 07:42:53 | S_a_i_n_t | save yourself some bandwidth. |
| 07:43:09 | S_a_i_n_t | AzureSky: ^ |
| 07:43:25 | AzureSky | thx :D |
| 07:43:31 | AzureSky | thanks* |
| 07:43:38 | S_a_i_n_t | All these commands should be available with "svn −−help" which I'm fairly sure I've mentioned. |
| 07:44:34 | S_a_i_n_t | I also have a line that will delete all non-svn entries...but it's pretty brutal. |
| 07:45:07 | S_a_i_n_t | It really should move them to a dir, but instead it just uses "svn stat", and deletes anything not in that list :P |
| 07:45:12 | S_a_i_n_t | ie. brutal ;) |
| 07:45:46 | S_a_i_n_t | (but good for cleaning up after lots of patches etc.) |
| 07:45:53 | AzureSky | the only patch i want to use is saratoga's volume patch |
| 07:46:14 | S_a_i_n_t | you'll find others when you get more confident, trust me. |
| 07:46:28 | S_a_i_n_t | I have about 20 or so in my builds. |
| 07:46:41 | AzureSky | didnt see any for the fuze that looked worth my time honestly |
| 07:46:58 | S_a_i_n_t | Most aren't specific to a target. |
| 07:47:19 | saratoga | most useful patches end up in SVN pretty quickly |
| 07:47:36 | * | S_a_i_n_t awaits the pong AI inclusion then :P |
| 07:47:41 | AzureSky | your volume patch helps with some of my phones :D |
| 07:48:05 | S_a_i_n_t | FS #5855 == pure awesome. |
| 07:48:37 | S_a_i_n_t | FS #6697 should also replace the current "dict" IMO. |
| 07:48:48 | S_a_i_n_t | that dictionary is braindead. |
| 07:51:39 | S_a_i_n_t | FS #6697 actually has scrolling, and fuzzy searching, and is just better...it still needs a little work (which I'm getting through slowly) but it is vastly superior to dict.rock |
| 07:52:02 | pixelma | last things I heard about stardict is that it has problems with scrolling etc. and is a bit overkill with typesetting. I admit I haven't tried myself but couldn't find motivation to do it, reading about the former and having rather "small screen targets" |
| 07:53:13 | S_a_i_n_t | pixelma: If you care to try it, I'd appreciate it. I have "small screen" playrs also. (well, 176x132 is fairly small to me) and it wraps the text fine. |
| 07:53:18 | S_a_i_n_t | scrolling is good too. |
| 07:53:51 | pixelma | ahem... the target with the biggest screen I have is 160x128... |
| 07:54:24 | S_a_i_n_t | still, I suspect that shouldn't be a problem. |
| 07:54:28 | pixelma | or better: its screen size is 160x128 greyscale ;) |
| 07:54:58 | pixelma | and how does the thing work on greyscale and monochromme, have you per chance tested in a sim? |
| 07:55:11 | S_a_i_n_t | The desc's in the dict I use are formatted quite sanely, and text wrapping works very well in this plugin. |
| 07:55:34 | S_a_i_n_t | I haven't tested for greyscale I'll admit, though I can do so tonight. |
| 07:55:38 | S_a_i_n_t | (a few hours) |
| 07:56:38 | S_a_i_n_t | The thing I find awesome about this is that the search makes it very easy to use premade dictionaries for language translation. |
| 07:57:04 | S_a_i_n_t | dict.rock not scrolling is a real PITA |
| 07:57:19 | S_a_i_n_t | especially for large definitions |
| 07:58:03 | AzureSky | same error, giving it one more go after another make veryclean |
| 07:58:28 | S_a_i_n_t | If it keeps doing it, I suspect all the make clean/verycleans in the world won't fix it. |
| 07:58:35 | S_a_i_n_t | the *exact* same error? |
| 08:00 |
| 08:01:05 | | Join stoffel [0] (~quassel@p57B4D74E.dip.t-dialin.net) |
| 08:02:47 | AzureSky | yeah it was |
| 08:03:10 | S_a_i_n_t | and, it has compiled properly before, with the same exact setup? |
| 08:03:17 | S_a_i_n_t | or, was this a first time compile? |
| 08:04:09 | | Join kaylinsigswort [0] (~Zigtown@CPE00259ce0fdb2-CM0014f8cc807a.cpe.net.cable.rogers.com) |
| 08:04:14 | AzureSky | yeah same one, now im seeing others for some reasion....bah, guess i try your hardcore command |
| 08:04:22 | CIA-8 | New commit by jdgordon (r27004): Touchregion support for the Base Skin and FM Skins. display obviously needs to be in stylus mode for this to work. Just about all screens should be ... |
| 08:04:37 | JdGordon | ^ that should make the D2 far more usable :) |
| 08:04:53 | S_a_i_n_t | AzureSky: the thing I was talking about will only delete files not normally in SVN. |
| 08:04:58 | S_a_i_n_t | not much use here. |
| 08:05:11 | S_a_i_n_t | you said it is a "clean" checkout though right? |
| 08:05:28 | S_a_i_n_t | what does "svn stat" report? |
| 08:05:49 | AzureSky | it removed the volume patch ad some other files |
| 08:06:17 | CIA-8 | r27004 build result: All green |
| 08:06:21 | AzureSky | uclpack.exe, codepages.exe, convbdf.exe and others |
| 08:06:51 | S_a_i_n_t | "svn revert" doesn't "remove" anything, it just returns the source to an SVN state |
| 08:07:12 | JdGordon | which is damn annoying when new files are added |
| 08:07:15 | *** | Saving seen data "./dancer.seen" |
| 08:07:31 | S_a_i_n_t | convbdf shouldn't have been effected by saratoga's patch. |
| 08:07:39 | AzureSky | ah, well it found some files it changed them from what im seeing |
| 08:07:58 | S_a_i_n_t | you've installed something else I take it, some other patch? |
| 08:08:03 | AzureSky | will give it a test again now |
| 08:08:09 | AzureSky | nope one patch |
| 08:08:13 | S_a_i_n_t | pastebin the output of "svn stat" |
| 08:08:19 | AzureSky | just the volume pne |
| 08:08:37 | S_a_i_n_t | what is the FS# of saratoga's patch. |
| 08:08:44 | AzureSky | how am i to copy/paste the content of this window |
| 08:08:57 | S_a_i_n_t | right click the cygwin icon. |
| 08:09:02 | bluebrother | S_a_i_n_t: just did one yesterday: http://www.alice-dsl.net/dominik.riebeling/rockbox/themeeditor/ |
| 08:09:04 | S_a_i_n_t | it'll give option |
| 08:09:06 | AzureSky | FS #11364 |
| 08:09:39 | bluebrother | and I'm building native, wasn't too motivated to figure how to cross compile Qt. |
| 08:09:49 | bluebrother | VMs are nice for that. |
| 08:10:52 | bluebrother | bieber: there are a few new warnings in the theme editor you might want to give a look. Especially that "no return in non-void function". Such things caused crashes for me in C++ code in the past. |
| 08:11:00 | AzureSky | best i can offer http://i47.tinypic.com/2r5u5nl.jpg i cant select the text to copy |
| 08:11:07 | bluebrother | (no idea if that would occur in this case) |
| 08:11:26 | S_a_i_n_t | AzureSky: You've definitely not had a clean checkout... |
| 08:11:34 | bluebrother | AzureSky: click the window title icon, then Edit / Mark |
| 08:11:40 | S_a_i_n_t | that patch touches none of the files you mentioned. |
| 08:12:17 | AzureSky | yeah, so im going to test now after that svn stat see if it helped |
| 08:12:30 | S_a_i_n_t | svn stat changes nothing. |
| 08:12:35 | S_a_i_n_t | it's just an output. |
| 08:12:38 | AzureSky | oh |
| 08:12:49 | S_a_i_n_t | if you used svn revert, it should be in a "clean" state |
| 08:13:24 | AzureSky | Bain2k9@Bain2k9-PC ~/rockbox |
| 08:13:25 | AzureSky | $ svn stat |
| 08:13:25 | AzureSky | ? vol1.patch |
| 08:13:25 | DBUG | Enqueued KICK AzureSky |
| 08:13:25 | AzureSky | ? Build |
| 08:13:25 | AzureSky | ? trunk |
| 08:13:25 | *** | Alert Mode level 1 |
| 08:13:25 | AzureSky | ? tools/uclpack.exe |
| 08:13:25 | *** | Alert Mode level 2 |
| 08:13:25 | AzureSky | ? tools/codepages.exe |
| 08:13:26 | *** | Alert Mode level 3 |
| 08:13:26 | AzureSky | ? tools/convbdf.exe |
| 08:13:26 | *** | Alert Mode level 4 |
| 08:13:26 | AzureSky | ? tools/bmp2rb.exe |
| 08:13:27 | *** | Alert Mode level 5 |
| 08:13:27 | AzureSky | ? tools/scramble.exe |
| 08:13:27 | *** | Alert Mode level 6 |
| 08:13:27 | AzureSky | ? tools/rdf2binary.exe |
| 08:13:28 | *** | Alert Mode level 7 |
| 08:13:28 | AzureSky | ? tools/mkboot.exe |
| 08:13:28 | *** | Alert Mode level 8 |
| 08:13:28 | AzureSky | M firmware/drivers/audio/as3514.c |
| 08:13:31 | AzureSky | oops |
| 08:14:01 | AzureSky | Bain2k9@Bain2k9-PC ~/rockbox |
| 08:14:01 | AzureSky | $ svn revert |
| 08:14:01 | AzureSky | svn: Try 'svn help' for more info |
| 08:14:01 | *** | Alert Mode level 9 |
| 08:14:01 | AzureSky | svn: Not enough arguments provided |
| 08:14:09 | bluebrother | AzureSky: please use a pastebin for that. |
| 08:14:25 | AzureSky | help doesnt give me arguements that i see for revert..... |
| 08:14:48 | S_a_i_n_t | "svn revert" |
| 08:17:17 | AzureSky | http://pastebin.com/UgF6QPvT |
| 08:18:38 | S_a_i_n_t | "doh!" |
| 08:18:50 | S_a_i_n_t | use "svn rever -R" |
| 08:19:00 | Ctcp | Ignored 1 channel CTCP requests in 0 seconds at the last flood |
| 08:19:00 | * | S_a_i_n_t forgot he uses aliases for this. |
| 08:19:14 | S_a_i_n_t | "svn revert -R" |
| 08:19:55 | S_a_i_n_t | If it just suddenly "broke", my first thought is a local change. |
| 08:20:42 | S_a_i_n_t | if it compiled before, and nothing changed, it should compile again. this is simple logic. |
| 08:20:56 | AzureSky | http://pastebin.com/grQmPHNC |
| 08:21:30 | pixelma | bluebrother: just in case you didn't notice and for your info - I put the link to your manuals into the topic so that some are available (that's how I understood you yesterday) |
| 08:22:37 | CIA-8 | New commit by ranma (r27005): mps depends on endpoint, I wonder why gcc didn't warn about i being used uninitialized... |
| 08:22:57 | S_a_i_n_t | AzureSky: Sorry man, I'm on a roll it seems. |
| 08:23:03 | S_a_i_n_t | I forgot the "." |
| 08:23:17 | S_a_i_n_t | "svn revert -R ." |
| 08:23:44 | S_a_i_n_t | though, the svn stat output looked perfectly normal. |
| 08:24:02 | *** | Alert Mode OFF |
| 08:24:12 | S_a_i_n_t | I just don't see why it would suddenly not work, if nothing in the environment changed. |
| 08:24:14 | CIA-8 | r27005 build result: All green |
| 08:25:15 | bluebrother | pixelma: yeah, that was the idea. And I've seen the topic change :) |
| 08:26:04 | S_a_i_n_t | bluebrother: Thanks btw. |
| 08:26:12 | S_a_i_n_t | I'll have a play with it very soon. |
| 08:26:16 | bluebrother | AzureSky: is "usage: revert PATH..." not descriptive enought? ;-) |
| 08:26:40 | bluebrother | (that line might be a bit hard to spot) |
| 08:27:46 | | Join ender` [0] (krneki@foo.eternallybored.org) |
| 08:30:53 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
| 08:36:50 | S_a_i_n_t | AzureSky: I also noticed you're using the first edition of saratoga's volume patch. |
| 08:37:06 | S_a_i_n_t | It's not really important, but there are 3 more revisons after that. |
| 08:37:14 | S_a_i_n_t | http://www.rockbox.org/tracker/task/11364?getfile=22155 is the final installment. |
| 08:38:05 | S_a_i_n_t | JdGordon: bieber: IS there any documentaion about the themeeditor? (other than the wiki page) |
| 08:38:16 | S_a_i_n_t | simple instructions, howto's etc.? |
| 08:38:30 | bieber | Other than the README, no |
| 08:38:51 | S_a_i_n_t | bieber: Thanks. |
| 08:38:55 | bieber | I'm a little hesitant to put too much into documentation atm, as the ui is changing pretty frequently |
| 08:39:02 | bieber | Any specific questions I can answer for you? |
| 08:39:29 | S_a_i_n_t | Trial and error is important I guess, I'll try going in blind (as most users do) and see how daunting it is/isn't. |
| 08:39:46 | S_a_i_n_t | bieber: Not yet, just trying to look at it like a first time user. |
| 08:39:57 | S_a_i_n_t | (which I actually am ;)) |
| 08:41:27 | bieber | The one thing that's pretty much impossible to find out from poking around atm is that if you set the keys #screenwidth and #screenheight in your config file, it will set the screen size in previews to those dimensions. Of course, if you have a backdrop defined then it will just grab the dimensions from there and you don't need to worry about it |
| 08:41:46 | bieber | Although all the preview does atm is show the backdrop and the viewports blocked out in red |
| 08:42:00 | JdGordon | which is enough :) |
| 08:42:16 | | Quit stoffel (Remote host closed the connection) |
| 08:42:21 | bieber | JdGordon: While you're here, a couple basic questions about rendering |
| 08:42:28 | JdGordon | bieber: I've been thinking and I would prefer if only the actual theme settings were in your config drop list... only a set list are acceptable to the theme site |
| 08:42:31 | JdGordon | shoot |
| 08:42:59 | bieber | I'm assuming that when I'm rendering a viewport, it's transparent unless the user specifies a backdrop or a bg color? |
| 08:43:48 | JdGordon | umm, yeah, mostly |
| 08:44:02 | S_a_i_n_t | IIUC it will grab FG/BG colours defined from the config in this case |
| 08:44:05 | JdGordon | a hidden viewport clears the region when it becomes visible |
| 08:45:13 | bieber | Okay |
| 08:45:32 | | Quit hebz0rl (Quit: Ex-Chat) |
| 08:45:45 | bieber | I see a couple of ways to define a "Custom UI Viewport", is that where the SBS goes? |
| 08:46:16 | S_a_i_n_t | UI viewport is where the lists are drawn |
| 08:46:47 | S_a_i_n_t | .sbs UI viewport takes preference over the config UI viewport. |
| 08:47:00 | bieber | Lists as in playlists? |
| 08:47:09 | S_a_i_n_t | as in menu |
| 08:47:41 | S_a_i_n_t | "Bookmarks, Files, Playlists, Now Playing" etc. |
| 08:48:55 | | Join hebz0rl [0] (~hebz0rl@dslb-088-067-204-104.pools.arcor-ip.net) |
| 08:49:12 | bieber | Ohhh, SBS is for the menus? |
| 08:49:48 | S_a_i_n_t | basically. |
| 08:49:50 | | Quit Zarggg (Read error: Connection reset by peer) |
| 08:49:56 | bieber | Alright, now things make a lot more sense |
| 08:50:00 | | Join mitk [0] (~mitk@195.117.162.130) |
| 08:50:20 | | Join Zagor [0] (bjst@rockbox/developer/Zagor) |
| 08:50:24 | bieber | For some reason I was under the impression that sbs was for the status bar on the while playing screen |
| 08:50:27 | * | bieber facepalms |
| 08:51:03 | S_a_i_n_t | the important thing about the UI viewport is that although it can be defined in two places, only one is ever used. |
| 08:51:20 | bieber | Makes sense |
| 08:51:20 | S_a_i_n_t | If its defined in the config file and the .sbs, the sbs values will be used. |
| 08:53:47 | | Join flydutch [0] (~flydutch@host110-154-dynamic.14-87-r.retail.telecomitalia.it) |
| 08:54:03 | JdGordon | it can be in the sbs multiple times also :) |
| 08:54:09 | JdGordon | but only one is ever used |
| 08:54:22 | bieber | Just the last one loaded? |
| 08:54:33 | S_a_i_n_t | it can be conditional |
| 08:57:21 | | Join Zarggg [0] (~zarggg@2001:0:4137:9e76:0:fbf8:beb1:ba3d) |
| 08:58:54 | | Quit bieber (Remote host closed the connection) |
| 08:59:46 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
| 09:00 |
| 09:00:08 | S_a_i_n_t | bieber: Where do I set the screen dimensions? o_0 |
| 09:00:49 | bieber | You have to have a project open (File->Open Project and select the .cfg file) |
| 09:01:17 | bieber | Then set #screenwidth and #screenheight values in the .cfg files, save, and your screen sizes should be correct |
| 09:01:33 | S_a_i_n_t | this will change though, right? |
| 09:01:55 | bieber | JdGordon: You mentioned only allowing valid keys in the dropdown: do we have a list of those somewhere? I grabbed the current list out of an appendix in the manual |
| 09:01:56 | S_a_i_n_t | it seems a little odd to not be able to set screen dimensions immediately ina new project |
| 09:02:19 | bieber | S_a_i_n_t: Yes, soon I'll be adding a whole dialog full of options for skin preview |
| 09:02:53 | | Quit jennifur (Remote host closed the connection) |
| 09:04:13 | S_a_i_n_t | *whew*, is there an easier way currently than, new document, save, open project? I assume loading a backdrop will auto adjust the values but I don't have a backdrop set. |
| 09:05:19 | S_a_i_n_t | Hmmm...that doesn't even work how I inteded. |
| 09:05:30 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
| 09:05:38 | S_a_i_n_t | I have to manually create a project .cfg atm? |
| 09:06:06 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
| 09:06:15 | | Quit bieber (Read error: Connection reset by peer) |
| 09:06:39 | | Nick fxb__ is now known as fxb (~felixbrun@h1252615.stratoserver.net) |
| 09:08:02 | AzureSky | http://pastebin.com/mQ55XxJf |
| 09:08:07 | S_a_i_n_t | It seems quite useful for working on an existing theme, but unless I'm missing something creating one from scratch seems a bit of a PITA |
| 09:08:36 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
| 09:09:07 | bieber | S_a_i_n_t: So far, yes. I haven't decided just how to handle generating project files/directories yet |
| 09:09:29 | S_a_i_n_t | I see. |
| 09:09:43 | S_a_i_n_t | If I can help with suggestions, just shoot. |
| 09:10:11 | | Join einhirn [0] (~Miranda@bsod.rz.tu-clausthal.de) |
| 09:10:36 | S_a_i_n_t | "New Project" with a window to choose available targets would be great. |
| 09:11:01 | S_a_i_n_t | With predefined attributes for known targets, and a way to add new targets also. |
| 09:11:27 | bieber | Predefined targets are definitely coming |
| 09:11:39 | bieber | I'm more wondering, how would you prefer to have it set up a new project, filewise? |
| 09:11:41 | S_a_i_n_t | that would make creating a project from scratch relatively painless. |
| 09:12:03 | bieber | Just create all the likely directories on disk? Ask which directories you intend to use? Should it be left loose or all packed into an archive? |
| 09:12:04 | S_a_i_n_t | how do you mean, second question. |
| 09:12:40 | S_a_i_n_t | Loose would be best, as then you could access it with more ease outside of the editor. |
| 09:12:42 | S_a_i_n_t | (IMO) |
| 09:12:51 | bieber | Well, you have a lot of directories that could potentially be used... /fonts, /backdrop, etc., but only some will actually be needed |
| 09:13:08 | JdGordon | bieber: we should be able to get a list pretty easily |
| 09:13:29 | JdGordon | actually it would be best if it could be shared with the themesite so maybe a text file on the web would be good? |
| 09:13:29 | S_a_i_n_t | I would say create them all, them only archive the ones that aren't empty for the finished product. |
| 09:13:50 | bieber | S_a_i_n_t: That makes sense |
| 09:13:57 | bieber | JdGordon: Is the web stuff also in the SVN tree? |
| 09:14:00 | S_a_i_n_t | Happy to help. |
| 09:14:22 | JdGordon | its in the themes.rockbox.org branch.. but I tihnk its list is stored in the db |
| 09:14:31 | S_a_i_n_t | It would be simple to check if the dir has content, if not, don't include it in the final archive of the finiched theme |
| 09:15:52 | S_a_i_n_t | I *love* being able to pull things out of the mainwindow, resize them, reposition them etc. |
| 09:15:56 | S_a_i_n_t | good work on that. |
| 09:16:55 | S_a_i_n_t | I think it is great for now, but new project handling definitely needs some work. |
| 09:17:19 | AzureSky | still get 1 compile error, guess i can give it a try again tomarrow..... |
| 09:17:55 | S_a_i_n_t | AzureSky: If it fails today, it'll probabl;y fail tomorrow. |
| 09:18:03 | S_a_i_n_t | I'm looking at your pastebin now. |
| 09:18:24 | bieber | JdGordon: The DB on the themesite? Would we just manually sync it with the Theme Editor source from time to time? |
| 09:18:37 | JdGordon | no, the list is pretty static |
| 09:18:40 | AzureSky | i fully deleted my rockbox folder and remade it and redownloaded the svn and its still erroring |
| 09:19:15 | S_a_i_n_t | AzureSky: What target is this? |
| 09:19:30 | S_a_i_n_t | I can try on my environment. |
| 09:19:51 | AzureSky | fuzev2 |
| 09:19:56 | AzureSky | 63 |
| 09:21:06 | S_a_i_n_t | stick around for a while and I'll let you know. |
| 09:21:53 | AzureSky | k, can stick it out for like 30min, its 12:22 here(am) im gonna need some sleep for wurk tomarow :/ |
| 09:22:07 | S_a_i_n_t | np |
| 09:23:12 | AzureSky | if the compiler made better use of my hexa core compiles would go faster, as it stands, its not super slow but its far from blazing fast |
| 09:23:47 | | Quit bieber (Read error: Connection reset by peer) |
| 09:23:54 | S_a_i_n_t | there's an option to use multicore. |
| 09:24:02 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
| 09:24:10 | | Join webguest49 [0] (www-data@giant.haxx.se) |
| 09:24:12 | S_a_i_n_t | "make -j(number of cores)" IIRC |
| 09:24:33 | S_a_i_n_t | but I don't think it supports hex-core |
| 09:24:48 | | Join webguest18 [0] (www-data@giant.haxx.se) |
| 09:24:48 | | Quit webguest49 (Client Quit) |
| 09:26:50 | S_a_i_n_t | bieber: How do I save a project file? |
| 09:26:58 | S_a_i_n_t | Is this done automatically? |
| 09:27:10 | bieber | Just Ctrl-S, same as with a wps file |
| 09:27:18 | bieber | Be warned, though |
| 09:27:28 | S_a_i_n_t | 0_o |
| 09:27:37 | bieber | It trashes any formatting and any comments that aren't in #value:key form when saving |
| 09:27:51 | S_a_i_n_t | Hmmm. :/ |
| 09:28:24 | S_a_i_n_t | It's a great idea, but for now I think I'll only be using it to propose advancements to its development. |
| 09:28:40 | bieber | It's definitely not production yet ;) |
| 09:28:44 | S_a_i_n_t | I believe I can code faster by hand at the moment. |
| 09:28:49 | | Join mt [0] (~mtee@rockbox/developer/mt) |
| 09:29:05 | S_a_i_n_t | I understand its a WIP though, not ragging on it. |
| 09:29:22 | bieber | For now it's treating .cfg files just as machine readable. I need to stick preserving comments/order of options to my to-do list |
| 09:34:05 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 09:37:22 | | Quit hebz0rl (Quit: Ex-Chat) |
| 09:37:29 | | Join JdGordon| [0] (~jonno@123-243-140-31.static.tpgi.com.au) |
| 09:37:30 | | Quit JdGordon| (Changing host) |
| 09:37:30 | | Join JdGordon| [0] (~jonno@rockbox/developer/JdGordon) |
| 09:39:54 | | Join Kitr88 [0] (~Kitar_st@89.142.54.33) |
| 09:41:03 | TheSeven | pamaury: damn, readwrite() from file.c directly passes buffer pointers on large writes, which are likely to be misaligned :-/ |
| 09:41:38 | | Quit kugel (Ping timeout: 240 seconds) |
| 09:41:56 | pixelma | Zagor: you answered to the English voices mail and said you wanted to fix, seems the daily manuals are affected too (which is then why they are gone since today and bluebrother was so kind as to provide some) |
| 09:42:21 | pixelma | just a heads-up ;) |
| 09:42:26 | | Quit webguest18 (Quit: CGI:IRC (Ping timeout)) |
| 09:42:31 | pamaury | TheSeven: then perhaps we should enforce alignment on all big buffers used with file I/O |
| 09:42:37 | Zagor | pixelma: right, I hope to find time to fix that today |
| 09:43:02 | | Quit Kitar|st (Ping timeout: 252 seconds) |
| 09:43:10 | TheSeven | pamaury: that won't help if the file offset they're written to is not on a 16byte boundary |
| 09:43:17 | pixelma | Zagor: nice, thanks |
| 09:43:34 | pamaury | true |
| 09:43:36 | TheSeven | i don't think we can avoid copying-around to a sectorcache buffer in that case |
| 09:43:48 | pamaury | indeed |
| 09:44:04 | pixelma | having a little more than the last 3 or 4 dailies available would have given a bit more time :) |
| 09:44:47 | | Quit Kitr88 (Ping timeout: 264 seconds) |
| 09:46:18 | | Join hebz0rl [0] (~hebz0rl@dslb-088-067-204-104.pools.arcor-ip.net) |
| 09:48:39 | | Quit kaylinsigswort (Remote host closed the connection) |
| 09:49:30 | | Join Kitar|st [0] (Kitar_st@89.142.56.44) |
| 09:51:29 | S_a_i_n_t | AzureSky: Compiles fine here. |
| 09:51:34 | AzureSky | ok |
| 09:51:50 | S_a_i_n_t | I suggest your CygWin is sick. |
| 09:51:51 | AzureSky | i get rockbox.elf error, whatever that is |
| 09:52:39 | AzureSky | oh joy u mean i need to reinstall it......what a pain in the arse |
| 09:52:49 | S_a_i_n_t | bah, that's easy. |
| 09:53:25 | AzureSky | then wait forever for the cross compiler to compile :P |
| 09:53:55 | S_a_i_n_t | Install according to the wiki, don't install the precompiled toolchains though, download the source (you can just keep the checkout you already have), run rockboxdev.sh, done. |
| 09:54:16 | S_a_i_n_t | you can also just backup the toolchain. |
| 09:54:22 | S_a_i_n_t | No need to recompile it. |
| 09:55:08 | S_a_i_n_t | Just backup the user/local dir |
| 09:57:02 | AzureSky | ok will work on that tomarow after work, maybe at work if im really board :) |
| 09:57:07 | | Join JdGordon1 [0] (~jonno@123-243-140-31.static.tpgi.com.au) |
| 09:57:08 | S_a_i_n_t | Hmmm...I just checked no also, and it compiles fine with the patch you've been adding also. |
| 09:57:21 | S_a_i_n_t | your environment is definitely a bit ill. |
| 09:57:35 | AzureSky | wana zshare that compile for me ;) |
| 09:57:58 | S_a_i_n_t | leave your IRC open and I'll PM you the link shortly. |
| 09:58:04 | S_a_i_n_t | (-ish) |
| 09:58:07 | | Join petur [0] (d408b802@rockbox/developer/petur) |
| 09:58:13 | AzureSky | ok thanks :) |
| 09:58:18 | S_a_i_n_t | You can check it when you wake up or whatever |
| 09:58:23 | S_a_i_n_t | No worries. |
| 09:59:25 | AzureSky | im sure you know what its like to bang your head against a keyboard when something just wont work :) |
| 09:59:47 | TheSeven | pamaury: which function is responsible for closing directories? (freeing the structs) |
| 10:00 |
| 10:02:46 | | Quit petur (Client Quit) |
| 10:04:14 | TheSeven | what the hell. i replaced "unsigned char cache [SECTOR_SIZE]" in the file struct with "struct sectorcache_entry* cache" and that compiles without warnings! |
| 10:04:30 | TheSeven | (without changing the accesses to it) |
| 10:06:37 | | Quit DerPapst1 (Quit: Leaving.) |
| 10:07:17 | *** | Saving seen data "./dancer.seen" |
| 10:10:01 | | Quit bmbl (Quit: Bye!) |
| 10:10:35 | S_a_i_n_t | TheSeven: I blame black magic. |
| 10:11:07 | * | TheSeven blames the fact that there is not even a single type-safe access to it |
| 10:11:36 | pamaury | TheSeven: closedir you mean ? |
| 10:11:48 | pamaury | for fat dir, there is no such a function |
| 10:11:52 | pamaury | iirc |
| 10:12:08 | TheSeven | so those are just forgotten about? |
| 10:12:33 | TheSeven | that's bad, as I will need to unlock a buffer when the dir is released |
| 10:13:15 | pamaury | I guess you can add one and call it from closedir |
| 10:13:36 | pamaury | I always thought it was strange that is wasn't done this way |
| 10:13:53 | pamaury | But check my statement, perhaps I'm wrong |
| 10:18:37 | pamaury | TheSeven: iirc, there is no file close for fat, only a "closewrite" |
| 10:19:09 | TheSeven | but the fd must be marked free somewhere |
| 10:19:43 | pamaury | yes |
| 10:19:51 | pamaury | you want to have a look at file.c and dir_uncached.c |
| 10:20:08 | pamaury | this is were the fat driver is used |
| 10:20:19 | pamaury | and the file handles are hanled |
| 10:20:22 | pamaury | *handled |
| 10:20:40 | TheSeven | what I also don't get is all the seeking stuff in file.c |
| 10:20:57 | TheSeven | /* seek back one sector to put file position right */ arount line 600 |
| 10:21:14 | TheSeven | is that just to make a subsequent writeback end up in the right sector? |
| 10:21:40 | TheSeven | so the fat file pointer will be advanced only during writebacks? |
| 10:22:02 | pamaury | From what I remember, if you seek at the position where the current sector is not the same, this code write back the sector basically |
| 10:22:58 | pamaury | I don't have the code at hand, only the online browser so it's no great so if you could figure out yourself, that would be better :) |
| 10:24:06 | pamaury | Hum, the comment you state is not in seek code, it's in readwrite |
| 10:25:30 | pamaury | There are some strange pieces of code: |
| 10:25:50 | | Join petur [0] (d408b802@rockbox/developer/petur) |
| 10:25:56 | pamaury | within the "if ( file->fileoffset + nread < file->size )" condition, it checks for "if ( file->fileoffset > file->size )" |
| 10:27:06 | * | TheSeven considers the whole fat code a bit strange :-P |
| 10:27:27 | pamaury | well, it |
| 10:27:29 | pamaury | 's fat |
| 10:27:48 | pamaury | fs code is always a bit strange |
| 10:28:13 | pamaury | And the one sector caching stuff makes it tricky |
| 10:28:57 | TheSeven | ...and the LFN hell |
| 10:29:12 | TheSeven | yes, FAT is a bit strange by itself |
| 10:29:26 | pamaury | You are not allowed to make any comment on LFN ! I rewrote it recently :) |
| 10:32:53 | * | amiconn wonders what TheSeven and pamaury are trying to achieve |
| 10:33:19 | TheSeven | amiconn: getting rid of all sector buffers on the stack and all unaligned accesses to the storage api |
| 10:33:20 | pamaury | We trying to get rid of the sector buffers hanging there and there |
| 10:33:25 | amiconn | why? |
| 10:33:47 | amiconn | You cannot get rid of unaligned accesses |
| 10:33:54 | pamaury | Beacuse that's ugly and a waste of space and it does some poor caching |
| 10:34:06 | amiconn | Large reads and writes pretty much have to go through |
| 10:34:27 | TheSeven | those can be copied-around sector by sector if they're misaligned |
| 10:34:30 | pamaury | Yes but there are so big reads that go to stack buffers we could easily be aligned |
| 10:34:35 | amiconn | Better caching might be useful, if it's done in a way that doesn't waste ram or binsize |
| 10:34:36 | TheSeven | that's what the storage driver would do anyway a few levels deeper |
| 10:35:01 | amiconn | Copying around costs extra time. |
| 10:35:21 | TheSeven | the storage layer will have to do that anyway if it's misaligned, because it can't do DMA otherwise, at least on some targets |
| 10:35:21 | amiconn | Not all drivers require the data to be aligned, so forcing alignment would be a waste |
| 10:35:46 | TheSeven | that's why it will only be copied around if it's misaligned and STORAGE_WANTS_ALIGN is defined |
| 10:36:07 | TheSeven | actually, the more interesting part is getting rid of the on-stack buffers that lead to dozens of stack overflows on large-sector targets like nano2g |
| 10:37:25 | pamaury | And anyway, sector buffer on stack are evil |
| 10:37:50 | TheSeven | they're a source of unneccessarily misaligned accesses |
| 10:38:55 | amiconn | Imo they are not evil in general |
| 10:39:07 | amiconn | Buffers on the stack don't take memory permanently |
| 10:39:19 | TheSeven | buffers from a sector cache won't either |
| 10:39:39 | TheSeven | they'll be used as cache memory as long as they aren't locked by someone who needs them |
| 10:40:58 | amiconn | Hmm. If you can get this to work efficiently... |
| 10:41:22 | pamaury | And that will avoid the big waste of space due to buffers in file.c, dir_uncached.c, ... That might be a way to factor this out and even provide some proper caching |
| 10:41:26 | amiconn | Regarding the misaligned accesses - it may or may not help to copy them to/from a buffer |
| 10:41:57 | TheSeven | amiconn: hm? |
| 10:42:00 | amiconn | Sure, you can use dma then - but at the cost of extra copying. Doing the transfer without dma may be more efficient overall. |
| 10:42:16 | amiconn | That's target dependent, of course |
| 10:42:18 | pamaury | We don't want to copy anything |
| 10:42:37 | pamaury | We want the cache to responsible for read/write and buffer "allocation" |
| 10:42:46 | pamaury | * to be |
| 10:42:46 | TheSeven | amiconn: maybe we should add another define then |
| 10:42:54 | TheSeven | STORAGE_WANTS_ALIGN vs. STORAGE_NEEDS_ALIGN |
| 10:43:02 | amiconn | Why? |
| 10:43:35 | amiconn | Imo something that depends on the driver should be handled on the driver level |
| 10:43:37 | TheSeven | for example on nano2g, there is (currently) no way to do unaligned accesses, so they're caught by copying-around code in the nand driver |
| 10:44:02 | amiconn | Yes, and that's where it belongs imo |
| 10:44:08 | TheSeven | that code needs some otherwise unneccessary buffers at the driver level |
| 10:44:24 | TheSeven | i'd like to have the ability to align those at the sector cache level |
| 10:44:25 | pamaury | amiconn: so you would sacrifiece performance because the rest of the code is of poor design ? |
| 10:45:22 | amiconn | No, why? |
| 10:46:39 | pamaury | That's waht you are saying, "something that depends on the driver should be handled on the driver level" |
| 10:50:48 | amiconn | Yes, but I don't sacrifice performance that way |
| 10:51:11 | amiconn | I have to do the alignment somewhere, but only for some targets |
| 10:51:28 | TheSeven | amiconn: the cache layer can do it without the need for additional memory |
| 10:51:28 | amiconn | So I prefer doing that where the code is target dependent anyway - and that's the driver |
| 10:52:02 | amiconn | The cache can be aligned, sure. But large accesses shouldn't go through the cache, as that *reduces* performance |
| 10:52:26 | TheSeven | they need to go through the cache for consistency anyway |
| 10:53:26 | TheSeven | the cache will check for collisions and pass them (without copying) to the storage layer, as long as they're aligned |
| 10:53:28 | JdGordon | the rockbox UI is so not touch freidnly :p |
| 10:53:39 | pamaury | That is not only for alignment. We want to solve the issue of sector buffers being handled in a non-uniform way. If we do that correctly, there will be no penalty. |
| 10:53:42 | amiconn | I don't see why |
| 10:54:02 | TheSeven | pamaury: actually there will be a small penalty for some checks |
| 10:54:03 | amiconn | Why they have to go through the cache, that is |
| 10:54:16 | pamaury | amiconn: because that's a cache ! |
| 10:54:34 | | Join swilde [0] (~wilde@aktaia.intevation.org) |
| 10:54:41 | pamaury | If something is in the cache, you want to enforce consistency or take advantage of it |
| 10:54:55 | amiconn | Reading a large chunk of data from a file (e.g. an audio file) will almost for sure happen only once |
| 10:55:06 | pamaury | TheSeven: yes but the penalty is small considered to actual reading/writing |
| 10:55:38 | TheSeven | amiconn: that's why those will just be passed through by the cache after checking that there are no dirty pages in that area |
| 10:55:57 | amiconn | If it goes through the cache, it will throw out previously cached data (e.g. parts of the fat or directory blocks), which are likely to be needed again before reading another file |
| 10:56:15 | amiconn | -> performance loss |
| 10:56:25 | TheSeven | no, it won't |
| 10:57:04 | TheSeven | the cache will just pass very large accesses which are aligned and don't touch dirty pages |
| 10:57:18 | amiconn | Dirty pages? You want to do write caching? |
| 10:57:37 | TheSeven | only in a limited way |
| 10:58:23 | TheSeven | the cache would allow for arbitrary write caching, but the file system code will usually trigger page cleaning as soon as it has finished modifying the sector |
| 10:58:43 | amiconn | ewrh |
| 10:59:08 | pamaury | amiconn: we don't invest anything, all this caching stuff is already done |
| 10:59:13 | pamaury | * invent |
| 10:59:26 | pamaury | in more or less clever ways |
| 10:59:32 | TheSeven | it's just done in ugly on-stack or in-struct buffers right now |
| 11:00 |
| 11:01:04 | TheSeven | the only difference on the caching side right now is more clever read caching, as the buffer space for currently-unused files or directories will be used as an MRU cache for small accesses |
| 11:03:52 | amiconn | Well, let's see |
| 11:07:08 | | Join efyx [0] (~efyx@lap34-1-82-225-185-146.fbx.proxad.net) |
| 11:07:43 | TheSeven | pamaury: I'm about to make the file.c code always lock sectors exclusively, even when only reading. Any objections? |
| 11:08:24 | pamaury | Why exclusive ? |
| 11:08:43 | TheSeven | because I can't easily distinguish between reads and writes at the place where I need to lock it |
| 11:08:51 | TheSeven | and nobody else should be needing it anyway |
| 11:09:06 | pamaury | A file can have several read handles |
| 11:09:09 | pamaury | At least in theory |
| 11:09:37 | TheSeven | how can i find out if the handle may have write access? |
| 11:09:59 | TheSeven | the write flag in filedesc? |
| 11:10:08 | pamaury | Each knows if it needs write access or not I think |
| 11:10:18 | pamaury | Check in open |
| 11:10:46 | TheSeven | as far as i can tell, you can just issue writing request to read-only file handles |
| 11:10:52 | TheSeven | that doesn't seem to be checked anywhere |
| 11:13:50 | pamaury | huh ? |
| 11:13:54 | pamaury | I doubt it |
| 11:14:02 | TheSeven | at least not in readwrite() |
| 11:14:06 | pamaury | That's wrong then |
| 11:14:17 | pamaury | We don't want to enforce such a bad practice |
| 11:15:10 | pamaury | Perhaps thr fat code checks that ? |
| 11:15:29 | pamaury | In fact, I'm pretty sure it does |
| 11:15:29 | TheSeven | that would be evil |
| 11:16:00 | TheSeven | and might cause delayed failures, because you could still write to the sector cache |
| 11:16:39 | TheSeven | anyway, I'm gonna check it in readwrite, can't hurt if it's checked twice |
| 11:16:54 | pamaury | Agree |
| 11:17:10 | pamaury | This way, you don't need exclusive lock for read |
| 11:18:23 | TheSeven | yep |
| 11:23:45 | TheSeven | also, the current implementation seems to only detect an out of disk space situation after the first non-fitting sector was written, while the cached implementation will already complain when trying to lock the sector |
| 11:25:22 | TheSeven | hm, and it looks like reading past EOF in the tail will cause EIO |
| 11:30:37 | pamaury | reading past eof should truncate read, that's the expected behaviour but it should not trigger an error iirc |
| 11:42:21 | Zagor | pixelma: do you know if we can cut down on the output from the manual builds? it's awfully noisy. |
| 11:43:04 | pamaury | can't you redirect everything to /dev/null ? |
| 11:43:16 | Zagor | we still want to see any errors |
| 11:44:23 | pixelma | Zagor: yes, bluebrother provided a patch, (there must be some pastebin links from yesterday). Unfortunately it also filters errors on older tex installations |
| 11:44:40 | Zagor | ok |
| 11:45:18 | pamaury | latex is a pain about filtering errors |
| 11:45:35 | pixelma | although we adjusted it a bit later (no complete patch for that but it works even in older installations |
| 11:45:39 | pixelma | ) |
| 11:46:51 | pixelma | Zagor: all in the logs of yesterday, a bit sprinkled across the hours though. While you are at it, do you know which latex package the daily build server runs, how old it is? |
| 11:50:41 | Zagor | it's running texlive 2009-9 |
| 11:55:10 | TheSeven | hm. "Misaligned data buffer at 09090909 (sector 151587081, count 1)" |
| 11:55:17 | TheSeven | that address is a *bit* suspicious |
| 11:55:29 | | Join evildork [0] (www-data@giant.haxx.se) |
| 11:56:03 | TheSeven | given the fact that 0x09090909 = 151587081, it's even more weitd |
| 11:56:05 | TheSeven | weird* |
| 11:58:39 | pamaury | that's *really* suspicious |
| 12:00 |
| 12:01:18 | evildork | If I use Rockbox, will I be able to organize my music by custom tags such as Composer>Mood>Opus, etc.? |
| 12:01:47 | | Join giovanni [0] (~giovanni@93.37.247.158) |
| 12:02:21 | TheSeven | pamaury: someone overwrites my sectorcache_entry structs :-/ |
| 12:03:03 | pamaury | how is this possible ? |
| 12:03:21 | TheSeven | a bug somewhere |
| 12:03:46 | TheSeven | as I said, the compiler didn't even warn when I changed the type of the cache field in the fd struct |
| 12:03:56 | TheSeven | i probably missed an access to that one |
| 12:04:28 | | Quit thefirstM (Remote host closed the connection) |
| 12:04:32 | | Quit slck (Ping timeout: 276 seconds) |
| 12:04:39 | pamaury | I had a similar problem in tagcache. the code was using the wrong structure names ! It worked but that was an incredible mistake |
| 12:05:16 | pamaury | Isn't there a warning about pointer casts ? |
| 12:05:24 | TheSeven | nope |
| 12:05:38 | TheSeven | all accesses to that bugger are done in a type-unsafe way |
| 12:07:19 | *** | Saving seen data "./dancer.seen" |
| 12:07:28 | | Join watto [0] (~watto@193.203.81.165) |
| 12:07:58 | | Join slck [0] (Venci@Slackware.SlackPix.Com) |
| 12:09:02 | pamaury | TheSeven: try to compile with -Wc++-compat, perhaps it can help |
| 12:09:09 | JdGordon | quick arm question.. does this diff look correct? http://pastebin.com/AVWKmU2T I want to only boot the OF on hold or usb connected |
| 12:09:19 | JdGordon | currently it boots rockbox if hold is on |
| 12:10:29 | TheSeven | JdGordon: USB present = bit 0x4000000 *not* set? |
| 12:10:50 | JdGordon | 0x3c when not connected, 0x38 when it is connected |
| 12:11:32 | TheSeven | you mean 0x3c000000? |
| 12:11:35 | JdGordon | yes |
| 12:11:53 | TheSeven | looks sane then |
| 12:12:03 | JdGordon | thanks |
| 12:12:12 | JdGordon | that change shuoldnt be able to brick it right? |
| 12:12:52 | TheSeven | IIUC that's a cowon d2? those aren't brickable anyway IIRC |
| 12:13:40 | TheSeven | the only way to make that code fail would be a data abort during that read (wrong memory address for the GPIO) |
| 12:13:59 | TheSeven | otherwise, it will always boot one or the other, even if something is the wrong way round |
| 12:14:06 | JdGordon | sweet |
| 12:14:23 | TheSeven | and you should still be able to use the hold switch to boot it to the OF, even if the code below is broken |
| 12:18:28 | JdGordon | wierd... it seems to work, but wont boot rockbox at all... |
| 12:19:29 | | Quit AzureSky (Ping timeout: 276 seconds) |
| 12:21:01 | | Join Rob2222 [0] (~Miranda@p4FDC9202.dip.t-dialin.net) |
| 12:21:25 | TheSeven | maybe the usb detection isn't working yet at that point during boot? |
| 12:21:35 | TheSeven | is the gpio set to input? |
| 12:21:50 | JdGordon | do I need to adjust that second ldreq pc line? |
| 12:22:19 | JdGordon | make it -44 ? |
| 12:22:20 | TheSeven | *facepalm*, yes, of course |
| 12:23:00 | JdGordon | 44 or 48? |
| 12:23:09 | TheSeven | but that would only make OF boot because USB is plugged fail |
| 12:23:52 | TheSeven | 44 |
| 12:28:21 | JdGordon | grr... there is another ldr [pc, #-28] a bit further down... can I copy that to a reg to make it more sane? |
| 12:28:56 | TheSeven | if there is a free reg, probably yes |
| 12:29:04 | | Quit GeekShadow (Ping timeout: 260 seconds) |
| 12:29:14 | TheSeven | where did that -28 further down point to originally? |
| 12:30:17 | JdGordon | .word 0 /* Saved entrypoint of original firmware*/ I think |
| 12:30:27 | TheSeven | the upper one, yes |
| 12:30:37 | JdGordon | the line after that is the same but points to the rockbox bootloader bin |
| 12:30:46 | JdGordon | so I guess just make sure that second one points correctly? |
| 12:30:49 | TheSeven | but if there were multiple [pc, -28]s, they can't possibly point to the same value |
| 12:31:06 | JdGordon | two .word 0 lines together |
| 12:31:23 | TheSeven | is that second one immediately below the #endif? |
| 12:31:27 | TheSeven | if yes, just change it to -44 |
| 12:32:54 | | Join dfkt [0] (dfkt@unaffiliated/dfkt) |
| 12:33:12 | S_a_i_n_t | perhaps a silly question, but what is the difference between a standerd build (N) and a Debug build (D)? |
| 12:33:34 | JdGordon | woo! |
| 12:34:26 | JdGordon | usb doesnt work though.. its either pointing to the wrong one or logic is bad :) |
| 12:34:35 | JdGordon | thanks TheSeven :) |
| 12:35:03 | TheSeven | are you sure that you want to access 0xf005a080? |
| 12:35:57 | | Join DerPapst [0] (~Alexander@p4FE8F1F8.dip.t-dialin.net) |
| 12:35:58 | TheSeven | S_a_i_n_t: it enables some additional sanity checks in the kernel, and probably some DEBUGF output |
| 12:36:00 | JdGordon | tst doesnt does equality not logical &? |
| 12:36:33 | * | S_a_i_n_t has a parsing error |
| 12:36:47 | TheSeven | tst will set the flags on the result of r0 & #whatever |
| 12:37:03 | TheSeven | eq = the result was zero, ne = nonzero |
| 12:37:51 | JdGordon | ok, so it is only checking the one bit? |
| 12:37:55 | TheSeven | exactly. |
| 12:38:37 | TheSeven | however, it still looks like you ignored the #0x20 offset to the address |
| 12:39:01 | TheSeven | (no idea what it is good for in the first place) |
| 12:39:28 | JdGordon | AH, I see what that is doing |
| 12:39:49 | JdGordon | GPIOA is 0xf005a000 + 0x20 |
| 12:39:57 | JdGordon | is that second ldr needed? |
| 12:39:59 | TheSeven | yep |
| 12:40:08 | JdGordon | so I'm reading the wrong gio |
| 12:41:08 | TheSeven | if another reg is free, you might want to ldr the base address to r1 instead, use [r1, #0x20] and [r1, #0x60], remove the second ldr r0, =... and adjust the offsets again |
| 12:41:24 | TheSeven | that would save 8 bytes of binsize :-) |
| 12:41:32 | S_a_i_n_t | \o/ |
| 12:42:28 | JdGordon | it works :D |
| 12:43:36 | JdGordon | I'm getting nice scrambled lcd output though when rockbox.d2 eventually loads! |
| 12:48:36 | CIA-8 | New commit by mt (r27006): Wrote apps/codecs/wmapro.c and modified libwmapro to make the codec work in the sim. Neither libwmapro nor wmapro.c have been added to the main build ... |
| 12:50:37 | CIA-8 | r27006 build result: 92 errors, 296 warnings (mt committed) |
| 12:51:30 | mt | wow |
| 12:51:42 | CIA-8 | New commit by jdgordon (r27007): change the D2 bootloader to boot rockbox by default. boot OF is hold is on or usb is connected |
| 12:52:19 | Zagor | mt: it certainly looks like they have been added... |
| 12:52:29 | mt | Yeah :/ |
| 12:53:31 | | Quit petur (Quit: medic!) |
| 12:53:35 | CIA-8 | r27007 build result: 93 errors, 295 warnings (jdgordon committed) |
| 12:54:03 | JdGordon | how did a warning become an error ? :) |
| 12:55:44 | mt | aah .. the infamous nomsg commit comes back to bite me |
| 12:55:57 | | Quit mitk (Quit: Leaving) |
| 12:56:06 | | Quit retroj (Ping timeout: 240 seconds) |
| 12:56:31 | | Join ucchan [0] (~ucchan@softbank126102048036.bbtec.net) |
| 12:56:55 | * | TheSeven shouts at file.c |
| 12:57:18 | TheSeven | that thing keeps hiding its memory corruption bug :-/ |
| 12:59:10 | pamaury | TheSeven: did you try with the warning I told you ? |
| 12:59:44 | TheSeven | i checked them all manually |
| 13:00 |
| 13:00:18 | pamaury | TheSeven the warriori does everything himself :) |
| 13:00:32 | ucchan | funman login? |
| 13:00:43 | TheSeven | it weren't that many |
| 13:01:49 | | Join AsusFreak [0] (www-data@giant.haxx.se) |
| 13:02:01 | pamaury | ucchan: he is not online now it seems |
| 13:03:07 | pamaury | It would probably be a good thing to try to compile rockbox with strict pointer cast rules, just to check if there are more of these incorrect pointer types. But there would be hundreds of false positive |
| 13:04:05 | TheSeven | there are also some other checks that seem to be disabled, even though most of their hits seem to be actual bugs/quirks |
| 13:04:19 | pamaury | like ? |
| 13:06:33 | ucchan | pamaury I think so. sigh.. |
| 13:07:41 | pamaury | ucchan: is the problem related to his last commit on the text viewer ? |
| 13:08:35 | CIA-8 | New commit by mt (r27008): Fix reds. (Actually removed wmapro from main build, which was added in a previous commit) |
| 13:09:10 | TheSeven | haha |
| 13:09:14 | TheSeven | /data/rockbox-trunk/firmware/common/dircache.c:953: warning: identifier ânewâ conflicts with C++ keyword |
| 13:09:15 | ucchan | pamaury: yes. your commit is very wrong. |
| 13:09:30 | JdGordon | does anyone know how the d2 storage works? |
| 13:09:50 | JdGordon | trying to find how the internel/external drives kludge happens (they are swapped) |
| 13:10:18 | CIA-8 | r27008 build result: All green |
| 13:10:23 | pamaury | JdGordon: it's basically a hack. If you boot from sd card. The sd card is the main storage and the internal sotrage is seem as a sd card. If you boot from internal storage, it's the contrary I think. |
| 13:10:39 | JdGordon | right, I want to undo that hack :) |
| 13:10:54 | JdGordon | I also cant find where it finds which to boot from |
| 13:11:32 | pamaury | I think the bootloader does the selection but I might ne wrong. I didn't play a lot with the d2 yet |
| 13:12:45 | pamaury | ucchan: if you can wait, funman is usually online later on. I would say in 3 or 4 hours, he is likely to be there |
| 13:13:18 | pamaury | JdGordon: ask gevaerts, he probably knows that |
| 13:13:52 | TheSeven | pamaury: damn, -Wc++-compat didn't yield anything related |
| 13:14:01 | * | gevaerts denies everything |
| 13:14:06 | pamaury | That's a shame |
| 13:14:10 | JdGordon | aawww |
| 13:14:42 | JdGordon | anyone seen any rogue #ifdef COWOND2 lines in the file drivers? |
| 13:15:33 | | Quit Battousai (Read error: Operation timed out) |
| 13:15:55 | TheSeven | JdGordon: nope |
| 13:15:55 | gevaerts | JdGordon: the ordering is decided by the order in storage_init() in storage.c. Putting NAND before SD will change that, without impacting any other target (no other target has those two AFAIK). |
| 13:16:22 | JdGordon | thanks |
| 13:16:34 | gevaerts | JdGordon: I'm not sure if it's a good idea to suddenly change this though. This ordering wasn't really an accident, and people might depend on it |
| 13:16:52 | AsusFreak | Hi. Just saw that you're catching Cowon's D2. I just put a task in the bug tracker: absolute point mode still has problems (FS #11424). |
| 13:17:05 | | Join Battousai [0] (~bryan@gentoo/developer/battousai) |
| 13:17:34 | TheSeven | gevaerts: we still don't have R/W NAND access on the D2? |
| 13:17:36 | gevaerts | TheSeven: indeed not |
| 13:18:11 | JdGordon | AsusFreak: I've already replied to it |
| 13:18:14 | * | TheSeven wonders why people are so scared of FTLs |
| 13:18:30 | JdGordon | kugel, get online! :D |
| 13:18:55 | gevaerts | TheSeven: in this case shotofadds did spend lots of time on it. I wouldn't say he's scared of it |
| 13:19:26 | AsusFreak | JdGordon: ok, thanks for information. |
| 13:19:35 | TheSeven | what's holding back R/W access then? |
| 13:19:50 | gevaerts | bugs |
| 13:19:54 | pamaury | Perhaps he doesn't have a full understanding of it |
| 13:20:03 | gevaerts | You can enable it I think, but it doesn't survive long |
| 13:21:30 | TheSeven | debug it, then :-) |
| 13:21:38 | * | JdGordon dares pamaury to fix it :D |
| 13:22:04 | gevaerts | hm, or not |
| 13:22:19 | gevaerts | Actually, the write bit isn't there at all yet |
| 13:22:26 | pamaury | I can have a look, if the format is explained. |
| 13:22:30 | gevaerts | It still goes wrong every now and then even in readonly mode |
| 13:23:19 | gevaerts | shotofadds might have uncommitted write code though |
| 13:23:32 | TheSeven | well, R/O failures should be rather easy to debug, if you can get hold a flash dump of such a state |
| 13:23:34 | * | gevaerts recommends talking to shotofadds about this :) |
| 13:24:06 | pamaury | yes clearly |
| 13:25:17 | JdGordon | gevaerts: so the actual ordering problem is storage doesnt really let a target arbitrarily set the ordering? |
| 13:27:42 | gevaerts | JdGordon: well, yes and no. The sd vs nand ordering thing is a purely D2 issue (by coincidence, sure, but it's the case anyway). I've not seen a case yet where this fixed ordering causes problems, so I at least haven't spent time on it. Changing that shouldn't be too hard though if needed |
| 13:28:56 | JdGordon | isnt it a fairly safe assumption that if a target has sd it is likely to either use it for external? so it should almost certainly be last in the list |
| 13:29:32 | JdGordon | I mean nothing would ever use sd but have a ata port also |
| 13:29:55 | JdGordon | and even if it did, it owuldnt use sd as main storage |
| 13:30:21 | gevaerts | the Elio has both ATA and sd :) |
| 13:30:32 | gevaerts | but yes, it will use ATA as main storage |
| 13:30:32 | JdGordon | both internal? |
| 13:30:39 | gevaerts | no, a card slot |
| 13:31:06 | JdGordon | right, so ATA being before sd is a fairly safe assumption, ditto nand before sd :) |
| 13:31:20 | gevaerts | You're right of course. The only reason SD *isn't* last is the D2 situation, where at the time it was decided to use SD as main storage to have writable main storage |
| 13:34:08 | JdGordon | and the only situation where writable main storage is required is the .rockbox folder, and that needs to change for RaaA anyway, so maybe we wrap all ROCKBOX_DIR calls in something like open_rbdir(char* filename, int mode) where filename is relative to /.rockbox and it will work out where that actually is? |
| 13:34:41 | gevaerts | that could work |
| 13:34:45 | JdGordon | we could probably even do that transparently in the open() call? |
| 13:35:42 | gevaerts | It's not exactly simple I think. How would you handle it? File not present on NAND, open it on SD? |
| 13:35:44 | pamaury | If possible, it would be better to avoid the creation of a virtual folder by having a transparent redirection in open |
| 13:36:10 | JdGordon | gevaerts: yes |
| 13:36:32 | JdGordon | it would be messy if say config.cfg is in both, but as long as it isnt it would only write/read to the sd |
| 13:36:57 | gevaerts | What if e.g. a user does something like generate the database from a PC, and then update it in rockbox? |
| 13:37:27 | JdGordon | is there anything we could do with that? probably not |
| 13:37:40 | | Quit DerPapst (Quit: Leaving.) |
| 13:37:57 | gevaerts | Or what if a user tries to put a file in .rockbox from the file browser? Should it fail or transparently move it to the SD? What if the SD has a file of the same name? |
| 13:37:59 | JdGordon | or if internal is ro and we are opening for write we reverse the search order |
| 13:38:51 | JdGordon | ok, so using a sepeare open function would be safer |
| 13:39:15 | gevaerts | I think so. At least it would have less nonobvious side effects |
| 13:39:30 | gevaerts | Possibly its complexity might offset that of course |
| 13:40:05 | JdGordon | I cant image it being very complex |
| 13:40:19 | JdGordon | I hope kugel hasnt done anything for RaaA on this yet |
| 13:43:08 | JdGordon | only 61 uses of ROCKBOX_DIR in apps... easy :) |
| 13:47:02 | JdGordon | need to fiddle with open_internal() to make this work I think |
| 13:47:43 | JdGordon | unless there is a good safe way to prefix the disk mount point onto the path? |
| 13:52:40 | TheSeven | ah, finally |
| 13:53:04 | TheSeven | after adding about a hundred debug statements all over the place, one actually triggered |
| 13:53:16 | TheSeven | a null pointer is causing the whole mess |
| 13:55:09 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
| 13:57:43 | JdGordon | gevaerts: ok, you were right.. this isnt exactly trivial |
| 14:00 |
| 14:03:55 | TheSeven | pamaury: ok, the cache seems to mostly work |
| 14:04:18 | TheSeven | however, someone isn't unlocking some handles when entering usb mode or shutting down |
| 14:04:19 | pamaury | \o/ mostly ? |
| 14:04:56 | TheSeven | but having a database initialization succeed through the cache is at least a good sign :-) |
| 14:05:20 | pamaury | TheSeven: iirc, some handles are kept open when entering usb mode. This is not a problem if you are not using ums but when you are, it's embarassing |
| 14:06:51 | pamaury | That should probably be fixed if it's the case |
| 14:07:22 | *** | Saving seen data "./dancer.seen" |
| 14:07:30 | pamaury | You can try to dump the filenames which are still opened when entering usb screen |
| 14:08:15 | | Quit pjm0616 (Ping timeout: 260 seconds) |
| 14:08:33 | pamaury | Perhaps the playlist one. I can't remember its name |
| 14:08:43 | JdGordon | .playlist_control ? |
| 14:08:57 | pamaury | yes |
| 14:09:04 | JdGordon | very likely to be left open |
| 14:09:26 | JdGordon | also a .m3u if it is too large to fit in the playlist buffer |
| 14:09:27 | pamaury | I had this problelm once, I think it's always open, a bad thing for sure |
| 14:14:18 | CIA-8 | New commit by jdgordon (r27009): Fix the hotkey touch region in the wps |
| 14:16:10 | CIA-8 | r27009 build result: All green |
| 14:17:47 | TheSeven | what would happen if that file is modified via usb? |
| 14:17:54 | TheSeven | bad things(tm)? |
| 14:18:03 | JdGordon | which? |
| 14:18:09 | TheSeven | one of the opened ones |
| 14:18:23 | | Join DataGhost [0] (~dataghost@17-18-ftth.onsnetstudenten.nl) |
| 14:18:23 | | Quit DataGhost (Changing host) |
| 14:18:23 | | Join DataGhost [0] (~dataghost@unaffiliated/dataghost) |
| 14:18:40 | pamaury | really bad things(tm) |
| 14:19:12 | pixelma | someone else looked around for kugel too? |
| 14:19:39 | AsusFreak | JdGordon: thanks a lot for r27009. Your speed is great. |
| 14:20:41 | | Join pjm0616 [0] (~user@61.250.113.98) |
| 14:21:26 | JdGordon | bieber: I just noticed you have bmp displaying also... niiice :) |
| 14:26:40 | TheSeven | pamaury: nvram.bin is the first one that's hit |
| 14:26:41 | | Join CGL [0] (~CGL@190.207.202.226) |
| 14:27:09 | TheSeven | what is that file good for btw? |
| 14:27:28 | TheSeven | as far as i can tell it's an archos relic |
| 14:27:31 | pamaury | What is this fucking file ? Isn't this for fast preferences that are stored in a non volatile ram ? |
| 14:28:09 | TheSeven | stored in an NVRAM on archos, in a file on everything else IIRC |
| 14:28:28 | pamaury | What does it contain ? Is it used ? |
| 14:28:51 | JdGordon | it contains a few settings which arnt in config.cfg |
| 14:28:56 | JdGordon | it should be closed when not used |
| 14:28:56 | Torne | it contains the settings that are marked as being in nvram, which i realise isn't a helpful answer :) |
| 14:29:37 | Torne | it is slightly useful, perhaps, because iirc it's used to remember several things that change often, and storing then in nvram.bin is less effort than rewriting config.cfg on every shutdown.. |
| 14:29:49 | CIA-8 | New commit by mt (r27010): Remove the stubbed main() from wmaprodec.c since it's useless now and remove Makefile for the same reason. |
| 14:29:51 | Torne | iirc that's where things like the dircache size and so on go |
| 14:30:13 | JdGordon | yes |
| 14:30:23 | JdGordon | I'm looking at the code and it should be being closed on use |
| 14:30:38 | JdGordon | it will only stay open if it is empty on first call |
| 14:30:42 | JdGordon | which is a bug |
| 14:31:00 | JdGordon | settings.c line 125 |
| 14:31:35 | CIA-8 | r27010 build result: All green |
| 14:31:57 | TheSeven | that's unlikely to be the cause |
| 14:32:48 | | Quit hebz0rl (Ping timeout: 264 seconds) |
| 14:34:30 | CIA-8 | New commit by theseven (r27011): Don't leak a file handle if nvram.bin is <8 bytes in size |
| 14:36:07 | CIA-8 | r27011 build result: All green |
| 14:36:18 | TheSeven | but it's still open |
| 14:38:10 | CIA-8 | New commit by mt (r27012): Add a patch file in libwmapro to make it easier to add the library and the decoder to the main build and modify README.rockbox accordingly. |
| 14:38:54 | TheSeven | hm, that one was actually a lost buffer handle, not a lost file handle. fixed. |
| 14:39:50 | CIA-8 | r27012 build result: All green |
| 14:40:17 | TheSeven | oh well |
| 14:40:25 | TheSeven | the next one is my UI font :-/ |
| 14:42:09 | | Join AzureSky [0] (~biteme@ashentech.broker.freenet6.net) |
| 14:42:57 | | Quit AsusFreak (Quit: CGI:IRC (EOF)) |
| 14:43:10 | CIA-8 | New commit by jdgordon (r27013): Allow the progressbar and volume touch regions to be reverse by prepending them with ! (so full is the left/top instead of right/bottom) |
| 14:45:01 | CIA-8 | r27013 build result: All green |
| 14:45:12 | TheSeven | should we switch to the SYSFIXED font during USB? |
| 14:45:58 | JdGordon | we dont really need to do we? the text on screen is static |
| 14:45:58 | TheSeven | is there any text besides the HID mode being drawn during USB? |
| 14:47:00 | gevaerts | not *today* |
| 14:47:15 | gevaerts | There has been talk about themable USB screens though |
| 14:47:27 | gevaerts | hm, right |
| 14:47:31 | * | gevaerts sees the issue :) |
| 14:47:52 | TheSeven | even the HID mode could potentially cause trouble if there is a glyph cache miss |
| 14:47:57 | gevaerts | yes, indeed |
| 14:48:42 | * | gevaerts now claims that that is actually the problem with HID mode for some people, thereby pushing all responsibility for it to other people |
| 14:48:58 | TheSeven | now how do i get rid of that annoying handle? |
| 14:50:01 | | Join Ed__ [0] (~chatzilla@58.171.101.118) |
| 14:51:00 | Ed__ | hello, I have an h340 and the WPS doesn't change no matter what theme I pick, and I just used the rockbox utility to install the latest stable release, can anyone tell me why? |
| 14:51:46 | JdGordon | because the themes changed recently so rbtuil is installing the wrong themes |
| 14:52:10 | JdGordon | download the themes manually from http://themes.rockbox.org/oldsite/www/index.php |
| 14:52:25 | Ed__ | how do I get rid of the themes that are in there? |
| 14:52:49 | TheSeven | JdGordon: any thoughts on how to get rid of that handle? |
| 14:52:55 | TheSeven | how can I make sure the font is unloaded? |
| 14:53:13 | JdGordon | which handle? |
| 14:53:20 | TheSeven | to my GUI font |
| 14:53:34 | TheSeven | (and restore the old state again when leaving USB mode) |
| 14:53:55 | JdGordon | force an unload before going into usb mode, then settings_apply() when you come out |
| 14:54:18 | JdGordon | font_unload(FONT_UI); |
| 14:54:35 | TheSeven | will that automatically switch to the sysfixed font? |
| 14:54:41 | JdGordon | yep |
| 14:55:04 | TheSeven | may I do that from the USB thread, or does it need to be done from the GUI thread? |
| 14:55:12 | JdGordon | you need to wrap that in NB_SCREENS(i) also and unload FONT_UI+i |
| 14:56:06 | JdGordon | umm... I'm not sure what scrolling will do when you unload its font |
| 14:56:16 | JdGordon | otherwise it is probably safe from usb thread |
| 14:56:51 | TheSeven | hm, isn't the gui thread reacting to that USB event anyway? |
| 14:56:57 | TheSeven | can you point me to the handler for this? |
| 14:57:28 | JdGordon | qppw/gui/usb_screen.c |
| 14:57:32 | JdGordon | apps* |
| 14:57:49 | gevaerts | the gui thread jumps to the usb screen code |
| 15:00 |
| 15:03:23 | TheSeven | FOR_NB_SCREENS(i) font_unload(FONT_UI + i); doesn't seem to close the handles |
| 15:05:58 | | Quit antil33t (Read error: Connection reset by peer) |
| 15:06:05 | | Join antil33t [0] (~Mudkips@203-184-54-232.callplus.net.nz) |
| 15:09:09 | | Quit mikroflops (Ping timeout: 276 seconds) |
| 15:16:26 | | Join komputes [0] (~komputes@ubuntu/member/komputes) |
| 15:22:31 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
| 15:22:39 | kugel | who yelled after me? |
| 15:22:47 | | Quit bieber (Ping timeout: 240 seconds) |
| 15:23:04 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
| 15:24:03 | * | TheSeven doesn't understand that usb codfe |
| 15:24:05 | TheSeven | code* |
| 15:24:17 | * | kugel read coffee |
| 15:24:19 | pamaury | The usb code is not easy to follow |
| 15:24:46 | TheSeven | someone's calling sectorcache_invalidate, but I can't seem to figure out who |
| 15:25:19 | TheSeven | it's probably coming through a disk_mount_all() call, but why on earth would anybody call that on usb *insertion*? |
| 15:25:44 | pamaury | because on ums, noone can access the disk so the usb code unmount everything |
| 15:25:48 | pamaury | see usb_slave function I think |
| 15:26:15 | pamaury | (in usb.c) |
| 15:26:16 | TheSeven | yeah, but it does actually *mount*, not unmount it on insertion |
| 15:27:19 | | Join evilnick_B [0] (0c140464@rockbox/staff/evilnick) |
| 15:27:24 | pamaury | no, that doesn't make sense |
| 15:27:48 | JdGordon | kugel: I was wondering if you'd done anything about /.rockbox or ROCKBOX_DIR for RaaA yet? |
| 15:28:31 | | Join jgarvey [0] (~jgarvey@cpe-065-190-066-089.nc.res.rr.com) |
| 15:29:51 | kugel | not yet |
| 15:30:13 | pamaury | TheSeven: if I look at the code, it does mount when usb is unplugged but does not umount when usb is plugged |
| 15:30:18 | pamaury | gevaerts: can you confirm that ^ |
| 15:30:25 | JdGordon | kugel: do you know how you're going to do it yet? |
| 15:30:30 | | Quit jgarvey (Client Quit) |
| 15:30:56 | | Join jgarvey [0] (~jgarvey@cpe-065-190-066-089.nc.res.rr.com) |
| 15:30:58 | kugel | I hope I can replace ROCKBOX_DIR with something that's not a compile time constant for RaaA |
| 15:31:01 | TheSeven | pamaury: i've placed a panicf in front of all calls to disk_mount_all in usb.c, and none of those gets triggered |
| 15:31:13 | TheSeven | so it must be somebody else that's calling it, but who? |
| 15:31:23 | pamaury | no idea |
| 15:31:24 | TheSeven | the only other place i can find where it is called is on boot |
| 15:31:43 | kugel | then change PLUGIN_DIR etc to /usr/local/lib, CONFIG_DIR to $HOME, and so on |
| 15:31:54 | pamaury | did you grep the entire tree ? Even in app/ for the default handler ? |
| 15:32:15 | TheSeven | yep |
| 15:32:28 | kugel | pamaury: your sim usb driver would be helpful now, no? :) |
| 15:32:54 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
| 15:33:11 | TheSeven | pamaury: http://pastie.org/1013551 |
| 15:33:20 | pamaury | My usb sim driver behaves differently from a real target, and with all the modification you did to the tree, it would not compile :( |
| 15:33:29 | JdGordon | kugel: ok, damn... probably wont be helpful for the d2 then |
| 15:34:05 | kugel | JdGordon: I think it would be inconvinient for the d2 anyway, because you'd need the OF to update rockbox |
| 15:34:14 | kugel | (assuming rockbox usb works on that thing) |
| 15:35:02 | pamaury | TheSeven: and you said to put a panic on every call and no one gets triggered ? |
| 15:35:12 | TheSeven | both of the usb.c ones have one |
| 15:35:18 | TheSeven | the only other relevant one is |
| 15:35:18 | TheSeven | ../../apps/main.c:570: rc = disk_mount_all(); |
| 15:35:44 | pixelma | kugel: also me... you said somethng about the FM quickscreen - do you have plans to revive it? Your changes to the c200 keymap back then cause a slight bug (you also decrease volume now at the same time you would call the quickscreen from the radio), this bug stayed even after removing the FM quickscreen action and the fix is probably different depending on whether you won't need the quickscreen or you do |
| 15:35:49 | TheSeven | ah, indeed, that's usb-related |
| 15:35:53 | TheSeven | where is the boot one then? |
| 15:36:31 | TheSeven | hmm, nope, that's early usb |
| 15:36:37 | | Join DerPapst [0] (~Alexander@p5099d40e.dip0.t-ipconnect.de) |
| 15:36:59 | kugel | pixelma: didn't I use the same button as the wps quickscreen on? |
| 15:37:02 | kugel | one* |
| 15:37:08 | TheSeven | *facepalm* |
| 15:37:35 | | Join t0rc [0] (~t0rc@unaffiliated/t0rc/x-5233201) |
| 15:37:52 | pixelma | kugel: I think you did but that doesn't matter for the problem |
| 15:38:36 | | Join hebz0rl [0] (~hebz0rl@dslb-088-067-204-104.pools.arcor-ip.net) |
| 15:39:02 | pixelma | it's an action (or prevent another action being hit at the same time problem) and the fm has a context on its own, also some special actions on its own |
| 15:39:52 | pixelma | I didn't notice until yesterday as I prefer my other targets for radio |
| 15:41:59 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
| 15:43:29 | | Join chaos [0] (~chaos@gentoo/user/ch4os) |
| 15:43:30 | | Join panni_ [0] (hannes@ip-95-222-52-93.unitymediagroup.de) |
| 15:43:52 | TheSeven | JdGordon: where is FONT_UI declared? |
| 15:44:10 | JdGordon | font.h probably |
| 15:44:14 | | Quit pjm0616 (Ping timeout: 272 seconds) |
| 15:44:38 | | Join pjm0616 [0] (~user@61.250.113.98) |
| 15:44:42 | TheSeven | oh, right, an enum |
| 15:44:49 | TheSeven | didn't grep properly for that |
| 15:45:57 | TheSeven | judging from that, FONT_UI can't be unloaded at all |
| 15:46:54 | JdGordon | pardon? |
| 15:47:04 | TheSeven | if (font_id >= SYSTEMFONTCOUNT && pf) |
| 15:47:18 | TheSeven | SYSTEMFONTCOUNT is below FONT_UI in the enum |
| 15:47:30 | JdGordon | oh right |
| 15:48:06 | JdGordon | you could change that :) |
| 15:48:39 | JdGordon | I think you could also load a dummy font file |
| 15:48:49 | JdGordon | "doesnt_exist.fnt" |
| 15:49:25 | TheSeven | urgh |
| 15:49:31 | JdGordon | that will cause the fd to close at least |
| 15:49:35 | JdGordon | not too pretty though |
| 15:50:10 | | Join domonoky [0] (~Domonoky@rockbox/developer/domonoky) |
| 15:51:09 | JdGordon | does anyone have any suggestions how to go about replacing the skin engine? right now I'm commenting out pretty much anything which stops it compiling when I remove the old code, but that is huge and messy |
| 15:52:01 | | Quit dfkt (Quit: -= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.) |
| 15:52:46 | TheSeven | yay! |
| 15:53:45 | TheSeven | WTF |
| 15:54:00 | TheSeven | the next one is a directory handle on /.rockbox/wps/UniCatcher |
| 15:54:43 | TheSeven | well, that's the alphabetically very last folder on it - dircache? |
| 15:57:48 | TheSeven | indeed - that one seems to be coming from dircache |
| 15:58:01 | JdGordon | more likely last dir opened? |
| 15:58:11 | TheSeven | if i disable dircache it's gone |
| 15:58:38 | TheSeven | and i'm testing with usb plugged in during boot, so that it'll enter usb mode as early as possible |
| 16:00 |
| 16:01:56 | TheSeven | next one: /.rockbox/viewers.config |
| 16:02:28 | TheSeven | who could be holding a handle to that file? |
| 16:07:25 | *** | Saving seen data "./dancer.seen" |
| 16:09:19 | pamaury | hehe, file handle mess |
| 16:09:37 | pamaury | that was a trap |
| 16:09:41 | | Join MethoS- [0] (~clemens@134.102.106.250) |
| 16:14:11 | | Join halmi [0] (~netbook@93-82-39-232.adsl.highway.telekom.at) |
| 16:19:56 | | Quit bmbl (Quit: Bye!) |
| 16:20:16 | S_a_i_n_t | TheSeven: Can you confirm keyclick is broken on Nano2g? |
| 16:20:40 | S_a_i_n_t | It only "clicks" if playback is happening, silent otherwise. |
| 16:20:54 | | Join retroj [0] (~retroj@unaffiliated/retroj) |
| 16:20:58 | TheSeven | hm, actually i haven't heard a single click recently |
| 16:21:01 | S_a_i_n_t | (for me anyway...just noticed it now) |
| 16:21:03 | TheSeven | might be disabled, let me check |
| 16:21:29 | JdGordon | TheSeven: is that with dircache on? |
| 16:21:41 | TheSeven | JdGordon: what? |
| 16:21:49 | JdGordon | viewers.condfig |
| 16:22:03 | S_a_i_n_t | I also notice this stops the "hiss" during startup, which confirms my theory that something in keyclick makes that awful hiss before an audio event |
| 16:22:19 | TheSeven | JdGordon: no, with dircache disabled |
| 16:22:48 | * | TheSeven didn't notice a hiss |
| 16:22:56 | S_a_i_n_t | (explanation: when I boot my Nanos, there is an awful hissing noise until the first keyclick, or until playback starts) |
| 16:22:56 | JdGordon | woops... read_config() in filetypes.c is leaking a fd |
| 16:23:25 | S_a_i_n_t | quite noticable on my targets, better IEMS perhaps. |
| 16:23:45 | ucchan | sorry, I quit soon. I must go to bed. funman, if you read this comment, please read FS #11399. |
| 16:23:47 | TheSeven | i just get a loud click when it initializes the codec |
| 16:24:14 | TheSeven | JdGordon, pamaury: apparently that cache is a bug attractor :-) |
| 16:24:28 | TheSeven | it already caught four unrelated bugs |
| 16:24:40 | CIA-8 | New commit by jdgordon (r27014): close the viewers.config file fd |
| 16:24:44 | pamaury | that's because they are lots of evil things related to open handles that no one want to touch |
| 16:24:52 | S_a_i_n_t | TheSeven: You gots clicks? I checked myself, and keyclick is definitely enabled, yet I have not keyclick unless playback is happening :/ |
| 16:25:04 | | Quit ucchan (Quit: Leaving...) |
| 16:25:13 | JdGordon | we should be able to detect this in the sim no? |
| 16:26:43 | CIA-8 | r27014 build result: All green |
| 16:27:27 | TheSeven | S_a_i_n_t: worksforme, r26788:26996M-100621 |
| 16:28:00 | S_a_i_n_t | Hmmmm. |
| 16:28:21 | * | S_a_i_n_t scratches his head. |
| 16:28:57 | S_a_i_n_t | I shall try building that revision. |
| 16:28:58 | * | TheSeven is updating to r27014M |
| 16:29:12 | | Quit halmi (Quit: halmi) |
| 16:29:33 | | Join halmi [0] (~netbook@93-82-39-232.adsl.highway.telekom.at) |
| 16:30:04 | S_a_i_n_t | I'm quite annoyed you can't hear the hiss to be honest though :/ |
| 16:30:25 | S_a_i_n_t | its quite annoying, for me. |
| 16:30:41 | JdGordon | TheSeven: lsof shows the sim only leaving open the font files... so you might be almost done :) |
| 16:31:12 | S_a_i_n_t | After boot there's a static hiss that goes away as soon as there is an audio even like voice, keyclick, or playback. |
| 16:31:33 | S_a_i_n_t | but is constant until then. |
| 16:31:35 | S_a_i_n_t | s/even/event/ |
| 16:31:39 | TheSeven | JdGordon: I already extended that font thing: for (i = FONT_UI; i < MAXFONTS; i++) font_unload(i); |
| 16:31:55 | TheSeven | S_a_i_n_t: never had that |
| 16:32:25 | TheSeven | might have been caused by one of those pcm changes |
| 16:32:43 | JdGordon | .playlist_control stays open after stopping playback |
| 16:32:50 | S_a_i_n_t | I've had it through the history of the port. |
| 16:32:53 | S_a_i_n_t | (s) |
| 16:33:18 | S_a_i_n_t | At first I thought it was crappy IEMs, so I got better ones, and it got worse :/ |
| 16:33:30 | TheSeven | how loud is that? |
| 16:33:31 | S_a_i_n_t | s/worse/more noticable/ |
| 16:34:07 | S_a_i_n_t | quite noticable at -25dB, earsplitting at +6 |
| 16:34:22 | S_a_i_n_t | like a white noise, static. |
| 16:34:49 | | Join dfkt [0] (dfkt@unaffiliated/dfkt) |
| 16:35:42 | S_a_i_n_t | I tested it by cranking the volume to =^, turning the player off, booting and not touching anything. |
| 16:35:53 | S_a_i_n_t | argh... "+6" |
| 16:36:36 | S_a_i_n_t | it dissappears as soon as there's a keyclick, voice event, playback starts etc then doesn't come back. |
| 16:36:54 | JdGordon | building the DB doesnt leak any fd's :) |
| 16:36:57 | S_a_i_n_t | as if something isn't init'ed properly at boot. |
| 16:37:17 | TheSeven | hm, my codec doesn't seem to produce this |
| 16:37:34 | TheSeven | however, i have noticed increased power consumption until playback is started for the first time |
| 16:37:39 | S_a_i_n_t | weirs, all my Nanos do, regardless of the build. |
| 16:37:39 | TheSeven | could have the same root cause |
| 16:37:48 | S_a_i_n_t | Indeed. |
| 16:38:42 | TheSeven | yours are both rev. 7 devices? |
| 16:38:43 | S_a_i_n_t | It happens in my patched builds, in clean SVN, on both Nano 1st and 2nd Gen. |
| 16:38:47 | TheSeven | mine is a rev. 9 |
| 16:38:58 | S_a_i_n_t | how do I check this? |
| 16:38:59 | TheSeven | ah, even first gen? that's funny |
| 16:39:11 | S_a_i_n_t | yeah, even first gen. |
| 16:39:43 | S_a_i_n_t | (sorry to distract you from you cache stuff ;)) |
| 16:41:49 | S_a_i_n_t | However, this "hiss" isn't happening on the Nano2g with broken keyclick, which makes me suspicious of keyclick as being the cause. |
| 16:42:00 | JdGordon | TheSeven: calling playlist_shutdown() before starting usb is probably a good idea |
| 16:42:06 | JdGordon | that will close the control file |
| 16:42:14 | | Join toffe82 [0] (~chatzilla@12.169.218.14) |
| 16:42:23 | TheSeven | from which thread? |
| 16:43:13 | TheSeven | yay! your viewers.config fix killed the last one of them |
| 16:43:51 | JdGordon | main thread |
| 16:45:08 | JdGordon | that should maybe be added to clean_shutdown() also |
| 16:45:43 | | Join halmi_ [0] (~netbook@188-22-120-122.adsl.highway.telekom.at) |
| 16:46:08 | JdGordon | ah, it is there already, in sustem_flush() |
| 16:47:23 | | Join mikroflops [0] (~yogurt@90-224-31-157-no112.tbcn.telia.com) |
| 16:47:45 | | Quit halmi (Ping timeout: 245 seconds) |
| 16:51:57 | evildork | If I use Rockbox, will I be able to organize my music by custom tags such as Composer>Mood>Opus, etc.? |
| 16:52:00 | | Join Ed___ [0] (~chatzilla@120.156.68.25) |
| 16:53:26 | * | TheSeven glares at *very, very* outdated comments in apps/main.c |
| 16:54:24 | | Join n1s [0] (~n1s@rockbox/developer/n1s) |
| 16:54:30 | JdGordon | see! comments are a waste of time! |
| 16:55:05 | | Quit Ed__ (Ping timeout: 260 seconds) |
| 16:55:56 | pamaury | evildork: what is mood and opus ? And what do you mean by organize ? With the database, using a custom tagnavi file, you can navigate your music files as you want with the proper tags |
| 16:56:01 | TheSeven | does the main thread have an event queue? if yes, where are the handlers? |
| 16:56:37 | pamaury | You should have a look at the default handlers in misc.c |
| 16:56:44 | pamaury | Probably |
| 16:56:54 | pamaury | (apps/msic.c) |
| 16:57:19 | TheSeven | aha, misc.c. very descriptive file name. |
| 16:57:32 | pixelma | evildork: do you want to organise purely on a per tag basis (in a database) or could you imagine to just stuff your music into different subfolders? |
| 16:57:47 | CIA-8 | New commit by jdgordon (r27015): Make the title area go up one level in the lists again. This Doesnt work if you have the title in a seperate viewport in the sbs, but that isnt really ... |
| 16:58:29 | pamaury | TheSeven: that's the treasure file. If you don't know where to look, look in misc.c ^^ |
| 16:59:28 | CIA-8 | r27015 build result: All green |
| 17:00 |
| 17:00:03 | | Quit bieber (Ping timeout: 264 seconds) |
| 17:00:16 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
| 17:02:22 | TheSeven | S_a_i_n_t: btw, keyclick is fine on r27014M |
| 17:02:50 | | Quit Ed___ (Quit: ChatZilla 0.9.86 [Firefox 3.6.3/20100401080539]) |
| 17:02:53 | TheSeven | damn |
| 17:03:11 | TheSeven | sector cache exclusive access collision while trying to create a folder in the root directory |
| 17:04:19 | | Join lestatar [0] (~chatzilla@cpe-72-229-41-214.nyc.res.rr.com) |
| 17:06:00 | | Part Zagor |
| 17:06:52 | pamaury | can you identify who tries to open it ? |
| 17:07:36 | TheSeven | it'll probably be more interesting to know who has already locked it |
| 17:10:18 | | Join DerPapst1 [0] (~Alexander@dslb-088-069-150-210.pools.arcor-ip.net) |
| 17:11:01 | TheSeven | hmm, the root directory is already opened non-exclusively when exclusive access is requested to add an entry |
| 17:11:11 | TheSeven | that *might* be the very same directory handle |
| 17:12:09 | | Quit DerPapst (Ping timeout: 240 seconds) |
| 17:12:56 | lestatar | hello all...don't wanna crowd the channel - i have a few rb v2 fuze questions please... :-) |
| 17:13:25 | n1s | lestatar: just ask, if anyone can answer, they probably will |
| 17:13:35 | lestatar | great thanks... |
| 17:14:07 | lestatar | first time here, so let me get this outta the way...all you rb project folks deserve medals...awesome awesome work :-) |
| 17:14:35 | lestatar | ok, so v2 fuze works great for me so far, but at, online help is down |
| 17:14:49 | kugel | just ask... |
| 17:15:06 | lestatar | i loaded a few albums onto 16gb microsdhc card...not all songs showing up while browsing card |
| 17:15:11 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
| 17:15:27 | | Part retroj ("Channel buffer killed") |
| 17:15:30 | lestatar | however, all appear and play properly viewing artist file browsing for instance |
| 17:16:18 | n1s | so they don't show up in the dayabase? |
| 17:16:19 | lestatar | update of db seems to initiate [refreshing in background message], but after several hours the added songs still do not list under card browsing |
| 17:16:35 | lestatar | they show up fine under artist |
| 17:16:49 | lestatar | not while directly viewing card contents |
| 17:17:07 | lestatar | not an issue with the card or rb necessarily as i already have 1500 files on the card |
| 17:17:15 | lestatar | just these last few albums i added |
| 17:17:16 | n1s | so they show up in the database but *not* in the file browser? |
| 17:17:20 | S_a_i_n_t | are they "hidden"? |
| 17:17:25 | lestatar | correct |
| 17:17:35 | S_a_i_n_t | I mean the file properties |
| 17:17:40 | lestatar | not hidden file attribs |
| 17:17:44 | S_a_i_n_t | aha |
| 17:17:53 | lestatar | though by practice, all my mp3 files are set for readonly |
| 17:18:16 | n1s | lestatar: just to be on the safe side, set your file view setting to "all" and check again |
| 17:18:41 | lestatar | ok, thanks...will give it a shot... |
| 17:18:56 | lestatar | stupid sandisk of is refreshing media again... :-( |
| 17:19:15 | lestatar | another question re: playlists |
| 17:19:34 | n1s | if that doesn't show them, they either are in another dir or something is very wrong |
| 17:20:00 | lestatar | using of and a couple 3rd party apps, i was able to create mixed content playlists successfully on fuze, using songs from both main mem and card... |
| 17:20:07 | lestatar | is this possible in rb? |
| 17:20:33 | S_a_i_n_t | IIRC, yes. |
| 17:20:35 | lestatar | and is it possible for rb to read/import .pla playlists i have previosuly created with above mixed contents? |
| 17:20:46 | n1s | no, m2u only |
| 17:20:49 | n1s | m3u |
| 17:21:17 | lestatar | ah, ok...thought i skimmed somewhere in manual that import could be done...guess only for m3u |
| 17:21:46 | lestatar | no worries, thanks...so i should create my mixed content playlists directly on the fuze/rb? |
| 17:22:09 | S_a_i_n_t | probably easiest. |
| 17:22:42 | * | TheSeven is running into a problem |
| 17:22:55 | S_a_i_n_t | TheSeven: Try walking ;) |
| 17:22:55 | lestatar | ok, thanks again...will def review manual and try all suggestions. you folks are great :-D |
| 17:23:07 | lestatar | cheers all :-) |
| 17:23:51 | TheSeven | to fix that properly, we'll probably need a mutex protecting directory access |
| 17:25:51 | TheSeven | or just write with non-exclusive access |
| 17:30:30 | TheSeven | hmm |
| 17:30:54 | TheSeven | somehow I'm losing the settings during shutdown |
| 17:30:59 | gevaerts | pamaury: yes, I've seen that too |
| 17:31:07 | | Quit evildork (Quit: CGI:IRC) |
| 17:31:39 | pamaury | gevaerts: seen what ? |
| 17:31:57 | gevaerts | what you asked me two hours ago :) |
| 17:32:31 | JdGordon | TheSeven: probably means the disk idle callback thingamy isnt being called |
| 17:33:19 | pamaury | gevaerts: is this normal ? Shouldn't the code unmount everything on usb plug ? |
| 17:33:58 | | Join Jaigoda [0] (www-data@giant.haxx.se) |
| 17:34:02 | gevaerts | pamaury: in my not very informed opinion, yes |
| 17:34:19 | gevaerts | Not unmounting seems like asking for trouble sooner or later |
| 17:34:20 | pamaury | the same for me :) |
| 17:38:48 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
| 17:39:43 | * | TheSeven wonders how the cache can cause settings not to be saved |
| 17:39:43 | | Quit Jaigoda (Quit: CGI:IRC (EOF)) |
| 17:39:47 | | Join webguest44 [0] (www-data@giant.haxx.se) |
| 17:40:14 | | Quit webguest44 (Client Quit) |
| 17:40:25 | * | kugel grumbles |
| 17:40:43 | JdGordon | TheSeven: if the cache stops the disk spinning up the callback wont be called |
| 17:40:45 | kugel | I would like to commit the CONFIG_PLATFORM patch but I'm running into trouble with dependency generation |
| 17:40:58 | gevaerts | kugel: anything specific? |
| 17:41:13 | saratoga | funman: I've seen a few people report that their fuze hard locked when plugged into USB, and could only be recovered by letting the battery drain |
| 17:41:39 | saratoga | does the Fuzev2 enter some particular mode when trying to reboot into the OF ? |
| 17:41:56 | | Join petur [0] (~petur@rockbox/developer/petur) |
| 17:42:22 | TheSeven | JdGordon: shouldn't the settings be saved during shutdown anyway? |
| 17:42:30 | JdGordon | well yes |
| 17:43:02 | | Join stoffel [0] (~quassel@p57B4DF68.dip.t-dialin.net) |
| 17:45:06 | kugel | gevaerts: yea, font.h includes sysfont.h, but dep gen doesn't see that with my patch (so sysfont.h is created too late) |
| 17:45:16 | gevaerts | ah, fun |
| 17:45:31 | kugel | it looks to me as if dep gen would take config.h into account. I assume it works in svn because -DSIMULATOR comes from the command line |
| 17:45:41 | | Join bertrik [0] (www-data@giant.haxx.se) |
| 17:46:00 | kugel | would *not* |
| 17:46:13 | gevaerts | Is this in gsoc-app-target? |
| 17:46:21 | kugel | no, gsoc-ifdef-cleanup |
| 17:46:32 | | Quit bertrik (Client Quit) |
| 17:47:45 | | Join bertrik [0] (~bertrik@90-145-31-194.bbserv.nl) |
| 17:48:03 | kugel | not quite up-to-date, I can push the sync'd one |
| 17:49:03 | gevaerts | that might help |
| 17:49:28 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
| 17:49:51 | | Join funman [0] (~fun@rockbox/developer/funman) |
| 17:50:01 | kugel | hm no, even moving the include sysfont.h out of the ifdef doesn't add it to the dep list |
| 17:50:36 | bertrik | FlynDice, could you test FS #11423 for me on your ams targets? |
| 17:51:18 | kugel | hm, push doesn't work |
| 17:51:45 | FlynDice | bertrik: Not immediately but some time today should work.... what am I testing for? |
| 17:52:02 | funman | battery life has increased a bit on clipv1 : 20 minutes or so |
| 17:52:54 | kugel | gevaerts: I needed −−force for some reason, so pulling won't work for you I think (just re-checkout the whole branch) |
| 17:53:23 | funman | perhaps because of r26953/r26954; but then I hope we see that it's not causing all those skips/glitches in audio playback |
| 17:53:23 | bertrik | FlynDice just see if the radio still works (tunes successfully) |
| 17:53:51 | FlynDice | bertrik: sure |
| 17:54:33 | lestatar | hi folks...am back...the one with the missing files under microsd file view |
| 17:54:34 | kugel | hm, the bug is somewhere else |
| 17:54:42 | lestatar | changed settings to file, show all... |
| 17:54:47 | kugel | font.h doesn't add sysfont.h dependency in svn too |
| 17:55:06 | | Quit GeekShadow (Quit: The cake is a lie !) |
| 17:55:12 | lestatar | viewd the card...a few songs from addition last night still do not appear |
| 17:55:20 | gevaerts | kugel: I'm looking at any sim build I guess? |
| 17:55:29 | kugel | yea |
| 17:55:39 | lestatar | but again show fine under artist browing... any more ideas? :-/ |
| 17:56:11 | S_a_i_n_t | lestatar: How is the dir structure organised? |
| 17:56:19 | S_a_i_n_t | too many files in one dir perhaps? |
| 17:56:25 | lestatar | on card, music/tunes |
| 17:56:27 | lestatar | that is it |
| 17:56:43 | S_a_i_n_t | try sorting into subdirs. |
| 17:56:49 | gevaerts | or increase the limit |
| 17:57:14 | S_a_i_n_t | aha! bingo! |
| 17:57:28 | lestatar | my max entry was already 3000 |
| 17:57:36 | lestatar | i am nowhere near that |
| 17:57:41 | | Quit pamaury (Remote host closed the connection) |
| 17:57:42 | S_a_i_n_t | and, does it exceed 300 entries? |
| 17:57:46 | lestatar | prolly like 1500-1700 files on card |
| 17:57:48 | S_a_i_n_t | oh, ok. |
| 17:57:49 | S_a_i_n_t | Hmmm. |
| 17:57:56 | pixelma | they should still show up, just not appear until you scroll down enough |
| 17:58:19 | lestatar | been looking pretty carefully since i noticed this... |
| 17:58:30 | lestatar | 1 album has like 12 songs |
| 17:58:38 | lestatar | in artist view they all appear... |
| 17:58:41 | lestatar | all play properly |
| 17:58:56 | lestatar | just odd in microsd view only like 4 songs appear in list |
| 17:59:05 | pixelma | that sounds like you are browsing the database? |
| 17:59:08 | lestatar | and i would hate to do subdirectories... |
| 17:59:12 | S_a_i_n_t | I find it very odd that they show in the database, but not in the filebrowser :/ |
| 17:59:36 | lestatar | yes, am new, but this seems odd to me as well |
| 17:59:48 | lestatar | just increased limit to 5000 |
| 17:59:57 | lestatar | should i do update of rb db anyway? |
| 18:00 |
| 18:00:20 | lestatar | waitasec... |
| 18:00:27 | lestatar | what should directory cache setting be? |
| 18:00:29 | gevaerts | if you have less files than the limit, increasing the limit isn't going to help |
| 18:00:45 | gevaerts | kugel: "font.h doesn't add sysfont.h dependency". Where? |
| 18:01:13 | lestatar | i notice when i browse the card, a message says dir buffer full...is that a clue? |
| 18:01:16 | kugel | I'm getting compile errors in pdabox (which includes plugin.h, and font.h from there) |
| 18:01:17 | gevaerts | I mean, as a dependency of what file? |
| 18:01:32 | JdGordon | lestatar: are the filenames pretty long? |
| 18:01:32 | gevaerts | ah, ok. I'll look at that one |
| 18:01:47 | JdGordon | that message does suggest uppding the max files setting |
| 18:01:48 | kugel | from /home/kugel/rbdev/rockbox-git/apps/plugins/pdbox/PDa/src/d_arithmetic.c:11: |
| 18:02:08 | lestatar | not crazy long, but some yes, perhaps |
| 18:02:29 | lestatar | though again, i already had plenty of files viewable under card browsing with even longer filenames |
| 18:03:03 | lestatar | this is a minor issue...i just find it odd is all... :-) |
| 18:04:23 | | Quit bertrik (Quit: >home) |
| 18:04:27 | kugel | gevaerts: FIRMLIB depends on sysfont.[oh] as per firmware.make, maybe there's the problem |
| 18:05:18 | kugel | sysfont.h gets created with make bin |
| 18:05:30 | TheSeven | how does that storage idle callback work? |
| 18:07:27 | *** | Saving seen data "./dancer.seen" |
| 18:08:01 | | Quit Rob2222 (Ping timeout: 272 seconds) |
| 18:08:19 | | Nick fxb is now known as fxb__ (~felixbrun@h1252615.stratoserver.net) |
| 18:08:33 | | Join Rob2222 [0] (~Miranda@p4FDC9202.dip.t-dialin.net) |
| 18:08:53 | gevaerts | kugel: what's interesting is that othe files in pdbox/PDa/src do have sysfont.h in their dependencies |
| 18:09:13 | kugel | hm, I haven't looked after these files |
| 18:11:20 | | Quit bieber (Ping timeout: 260 seconds) |
| 18:11:32 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
| 18:13:04 | kugel | gevaerts: d_arithmetic.c should depend on m_pd.h (the only file it includes) but it doesn't |
| 18:13:20 | kugel | the files included by m_pd.h get lost |
| 18:14:53 | gevaerts | kugel: I think I see what's going on |
| 18:14:54 | kugel | gevaerts: ha! FIXEDPOINT is not defined in dep gen, but during compilation |
| 18:14:59 | gevaerts | right! |
| 18:15:06 | TheSeven | hmm |
| 18:15:20 | gevaerts | Now how to fix that... |
| 18:15:24 | TheSeven | i'm explicitly calling call_storage_idle_notifys(true); before flushing the cache, and the settings are still not saved! |
| 18:16:03 | kugel | pdbox.make adds it to the command line |
| 18:17:32 | kugel | gevaerts: I have no idea how dep gen works :( |
| 18:17:43 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
| 18:17:45 | kugel | I assume it takes config.h and runs gcc -E through all .c files? |
| 18:18:15 | kugel | no, not gcc -E, but it collects their #includes |
| 18:18:52 | funman | -M* |
| 18:18:54 | | Quit einhirn (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
| 18:19:06 | | Join webguest51 [0] (www-data@giant.haxx.se) |
| 18:19:20 | funman | see mkdepfile in tools/functions.make |
| 18:20:36 | | Join DerpDerp [0] (48cc921f@gateway/web/freenode/ip.72.204.146.31) |
| 18:20:46 | DerpDerp | Hopefully |
| 18:20:49 | DerpDerp | I did this right. |
| 18:20:56 | DerpDerp | This is the IRC for rockbox, right? |
| 18:21:20 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 18:21:42 | | Quit webguest51 (Client Quit) |
| 18:21:45 | | Join G4Oblivion [0] (www-data@giant.haxx.se) |
| 18:21:46 | DerpDerp | Well, if anyone would be willing to help me out, that'd be great. |
| 18:22:01 | funman | not if you don't tell the problem |
| 18:22:10 | Torne | Ask your question, don't speculate on whether people will help you :) |
| 18:22:54 | DerpDerp | Fair point. So, I unstalled rockbox on my Sansa Fuze v2 today, and it was working fine right up until I plugged it into the USB port, at which point, the screen went black. |
| 18:23:06 | DerpDerp | It hasn't turned back on since, so I'm wondering if there's anything I can do. |
| 18:23:26 | Torne | do you mean "installed"? :) |
| 18:23:36 | kugel | gevaerts: that bug is in svn too, but I wonder why my patch exposes it |
| 18:24:09 | gevaerts | kugel: bad luck I'd say |
| 18:24:40 | kugel | I guess I better wait for Zagor to look at it? |
| 18:24:41 | DerpDerp | Well, used the manual install instructions I found using mkamsboot. |
| 18:24:49 | Torne | DerpDerp: it's supposed to reboot to the original firmware when you insert the USB cable. |
| 18:24:56 | gevaerts | I hope to have a fix in a few minutes |
| 18:25:02 | kugel | oh great |
| 18:25:06 | Torne | where did you get your versoin of mkamsboot? |
| 18:25:27 | DerpDerp | From this link: http://download.rockbox.org/bootloader/sandisk-sansa/mkamsboot/ |
| 18:25:31 | | Join pamaury [0] (~quassel@p5DDEE734.dip.t-dialin.net) |
| 18:25:31 | | Quit pamaury (Changing host) |
| 18:25:31 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 18:25:31 | Torne | also, have you tried holding the power switch for a really long time |
| 18:25:43 | Torne | if it's crashed you may need to hard reset it that way, i believe it can take 30-40 seconds |
| 18:25:48 | DerpDerp | Yeah, I held it down for upward of a minute. |
| 18:26:41 | Torne | i assume plugging/unplugging the cable further doesn't do anything either? |
| 18:26:45 | | Quit G4Oblivion (Quit: CGI:IRC (Ping timeout)) |
| 18:26:49 | DerpDerp | Sadly, no. |
| 18:27:15 | Torne | do you know what version of Rockbox you installed? |
| 18:27:42 | funman | DerpDerp: http://www.rockbox.org/wiki/SansaAMS#Specific_problems see the last line |
| 18:27:43 | bertrik | Possibly there's still something running on the fuze and you need to wait until the battery discharges completely |
| 18:27:58 | TheSeven | damn |
| 18:28:11 | TheSeven | i just can't manage to make that stupid ipod save it's setting with the cache enabled |
| 18:28:17 | DerpDerp | I installed the most recent build, I think. |
| 18:28:30 | DerpDerp | http://build.rockbox.org/data/rockbox-sansafuzev2.zip |
| 18:28:32 | Torne | The most recent build changes many times per day, though.. |
| 18:29:17 | DerpDerp | Ah, I see. Well, I just did this a couple hours ago using the build I posted up there. |
| 18:29:19 | Torne | anyway, yes, you might have to wait for it to discharge before you can make it work again, unfortunately |
| 18:29:59 | DerpDerp | Gotcha. Thanks for the help! What usually causes something like this to happen? |
| 18:30:19 | funman | If you find a probable cause please tell us because we have no idea |
| 18:30:26 | DerpDerp | Will do! |
| 18:30:31 | funman | although plugging USB cable seems to be a somewhat common factor |
| 18:31:10 | DerpDerp | It was turned on when I plugged it in, so next time (if there is a next time for this particular player), I might power it down first. |
| 18:31:40 | funman | what does 'stmfd sp!, {r0-r2}' do exactly: sp -= 3; sp[0] = r2; sp[1] = r1; sp[2] = r0; ? |
| 18:32:28 | funman | and sp will point to the last item pushed on the stack : r2 ? |
| 18:32:49 | TheSeven | the other way round |
| 18:32:58 | TheSeven | r0 will be at the lowest address IIRC |
| 18:33:23 | TheSeven | but it's been some months since i last touched that kind of thing ;-) |
| 18:33:43 | funman | i have a memory dump of c200v2, i try to unwind the stack myselfd |
| 18:33:44 | Torne | Yes |
| 18:33:51 | Torne | lowest register goes to lowest address |
| 18:34:00 | funman | ok |
| 18:34:14 | CIA-8 | New commit by gevaerts (r27016): Remove plugin-specific defines from the pdbox makefile, and add them to m_pd.h, which is the global pdbox header. This makes sure that the defines are ... |
| 18:34:54 | funman | so sp always points to lowest register pushed? |
| 18:35:04 | Torne | if it's not been moved to make room for locals, yes |
| 18:35:19 | kugel | gevaerts: I had that idea as well, I hoped their would be a way to not ignore *.make |
| 18:35:32 | gevaerts | no easy way as far as I can see |
| 18:35:38 | funman | ok thanks! |
| 18:36:07 | CIA-8 | r27016 build result: 24 errors, 0 warnings (gevaerts committed) |
| 18:36:20 | gevaerts | :( |
| 18:37:58 | | Quit DerpDerp (Quit: Page closed) |
| 18:39:42 | | Join pamaury_ [0] (~quassel@p5DDEE2F3.dip.t-dialin.net) |
| 18:39:58 | CIA-8 | New commit by gevaerts (r27017): Fix include file |
| 18:40:01 | TheSeven | pamaury: uh oh. file system corruption. |
| 18:40:44 | pamaury_ | ah, bad news |
| 18:41:50 | CIA-8 | r27017 build result: All green |
| 18:42:03 | pamaury_ | TheSeven: is it due to the new cache system ? |
| 18:42:11 | TheSeven | yep |
| 18:42:26 | TheSeven | the last sector of every file written contains garbage |
| 18:42:39 | TheSeven | and it looks like some fat changes didn't get committed |
| 18:42:41 | * | pamaury_ thinks that have the simulator read/write into a file would be of great help here |
| 18:42:58 | pamaury_ | *having |
| 18:43:07 | | Quit anewuser (Quit: for SELL 2 by the price of 1 now!) |
| 18:43:36 | | Quit pamaury (Ping timeout: 264 seconds) |
| 18:43:40 | kugel | gevaerts: sim compiles now |
| 18:43:41 | pamaury_ | Are you sure there are no hanging buffers, that everything is committed to the cache ta leats ? |
| 18:43:50 | TheSeven | i doubt this |
| 18:43:59 | TheSeven | the problem is most probably somewhere in file.c |
| 18:44:02 | | Quit Strife89 (Quit: Reboot to Windows.) |
| 18:44:51 | pamaury_ | Did you check that rockbox flushes everything at poweroff ? |
| 18:45:02 | TheSeven | i enforced that |
| 18:45:17 | CIA-8 | New commit by bertrik (r27018): Fix FS #10474 - FM tuner on Gigabeat S fails to tune last frequency on start |
| 18:45:21 | TheSeven | that stuff should actually have been committed when the file was closed |
| 18:46:04 | pamaury_ | Are you sure all the files are closed ? |
| 18:46:15 | TheSeven | i hope so |
| 18:46:32 | | Join Strife89 [0] (~Strife89@adsl-67-57-76.mcn.bellsouth.net) |
| 18:46:53 | CIA-8 | r27018 build result: All green |
| 18:46:55 | TheSeven | i just copied 43 music files using the file browser, if those wouldn't have been closed, it would have ran out of handles for sure |
| 18:46:59 | pamaury_ | And the garbage at the end happens for every file written ? |
| 18:47:39 | funman | TheSeven: plugins check if any file is still opened when they exit (but file browser isn't a plugin) |
| 18:48:18 | pamaury_ | funman: TheSeven spent hours fixed leaking file handles in the code :) |
| 18:48:24 | pamaury_ | *fixing |
| 18:48:42 | | Part Viaken ("WeeChat 0.2.6") |
| 18:48:56 | pamaury_ | funman: I think uchida disapprove your last commit regarding the text viewer |
| 18:49:12 | pamaury_ | He went on channel but you weren't online |
| 18:49:14 | funman | pamaury_: thanks, i am subscribed to the flyspray task so i saw it |
| 18:49:30 | funman | i missed him by a few |
| 18:49:37 | TheSeven | And the garbage at the end happens for every file written ? < yes |
| 18:50:14 | pamaury_ | TheSeven: can you put your code on FS or on pastebin. I can have a look at it |
| 18:50:14 | funman | text_viewer crashes in gui_statusbar_draw() |
| 18:50:52 | | Nick pamaury_ is now known as pamaury (~quassel@p5DDEE2F3.dip.t-dialin.net) |
| 18:51:01 | | Quit pamaury (Changing host) |
| 18:51:01 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 18:51:35 | | Join BHSPitMonkey [0] (~stephen@unaffiliated/bhspitmonkey) |
| 18:52:00 | TheSeven | I'll commit some of it (fixes for bugs that surfaced while developing it) first |
| 18:52:25 | | Join domonoky1 [0] (~Domonoky@agsb-d9bdbbf7.pool.mediaWays.net) |
| 18:52:25 | pamaury | yes you are right to do so, you did a great job at tracking this issue it seems |
| 18:52:45 | funman | JdGordon: gui_statusbar_draw is an awfully large function :/ |
| 18:52:53 | TheSeven | i'm not sure if i should commit that font one though |
| 18:53:04 | CIA-8 | New commit by kugel (r27019): Rockbox as an application: Replace many occurences of #ifdef SIMULATOR with #if (CONFIG_PLATFORM & PLATFORM_HOSTED) (or equivalently). ... |
| 18:53:12 | pamaury | avoid it for now |
| 18:53:16 | | Quit domonoky (Ping timeout: 265 seconds) |
| 18:53:35 | kugel | next commit will add the application target |
| 18:53:38 | pamaury | That would probably cause a major confusion + bug reports + hangry people |
| 18:53:40 | | Quit swilde (Remote host closed the connection) |
| 18:54:11 | funman | hungry* |
| 18:54:28 | CIA-8 | New commit by theseven (r27020): Fix ata_idle_notify on iPod Nano 2G |
| 18:55:04 | pamaury | *angry |
| 18:55:13 | CIA-8 | r27019 build result: 72 errors, 88 warnings (kugel committed) |
| 18:55:41 | | Join Jerom [0] (~Jerom@95.171.137.241) |
| 18:57:00 | CIA-8 | r27020 build result: 72 errors, 88 warnings (theseven committed) |
| 19:00 |
| 19:00:57 | | Join Horscht [0] (~Horscht2@xbmc/user/horscht) |
| 19:08:15 | Jerom | I like the new feature of this bot |
| 19:08:27 | Bagder | 808 points! |
| 19:09:01 | funman | info gcc says : `-O' also turns on `-fomit-frame-pointer' on machines where doing |
| 19:09:04 | funman | so does not interfere with debugging. |
| 19:09:35 | | Join webguest43 [0] (www-data@giant.haxx.se) |
| 19:10:24 | CIA-8 | New commit by kugel (r27021): Fix yellows and reds (one was a type, the other was rather incorrect before already) |
| 19:10:35 | | Quit webguest43 (Client Quit) |
| 19:10:38 | | Join webguest57 [0] (www-data@giant.haxx.se) |
| 19:10:46 | funman | on ARM i don't see the frame pointer being saved with -O, I don't understand how doing so doesn't interfere with debugging |
| 19:10:47 | kugel | typo* |
| 19:10:51 | | Quit webguest57 (Client Quit) |
| 19:11:16 | | Join G4Oblivion [0] (www-data@giant.haxx.se) |
| 19:12:12 | CIA-8 | r27021 build result: 123 errors, 6 warnings (kugel committed) |
| 19:12:29 | G4Oblivion | Hello, I need help getting rockbox source. I'm trying to download with cygwin. I keep getting the .sig error. |
| 19:12:37 | Jerom | Nice :D |
| 19:12:45 | G4Oblivion | I tried cmd with -x and nothing happened |
| 19:13:09 | S_a_i_n_t | G4Oblivion: That is explained in the wiki IRC |
| 19:13:18 | S_a_i_n_t | add the -x to the .exe |
| 19:13:44 | kugel | :( |
| 19:14:37 | S_a_i_n_t | sorry, to the shortcut to the .bat file |
| 19:15:05 | funman | no dev have been using Sansa AMSv1/AMSv2 recently? |
| 19:15:17 | * | kugel uses it all day |
| 19:15:52 | G4Oblivion | Like this "setup.exe -x"? I'm using RB wiki. |
| 19:16:22 | funman | kugel: with a build >= r26954 ? no songs skipping or weird audio glitches? which model is it? |
| 19:16:30 | * | kugel wonders why he even changed backlight.h |
| 19:16:44 | kugel | funman: my build on it is older I think |
| 19:16:45 | | Join M3DLG [0] (~M3DLG@bb-87-81-252-83.ukonline.co.uk) |
| 19:17:04 | kugel | backlight.c* |
| 19:17:08 | funman | saratoga and a lot of people report problems, i think r26953/r26954 cause them |
| 19:17:13 | | Quit DerPapst1 (Quit: Leaving.) |
| 19:17:25 | funman | but i don't want to believe it because test_disk was fine :'( |
| 19:17:35 | Bagder | 1236 points and we have a HIGHSCORE! |
| 19:17:37 | funman | hm |
| 19:17:45 | Jerom | fuzev2, r26981 I had no problem this afternoon |
| 19:18:00 | funman | TheSeven: can cached sectors be read/written before storage_(read/write)_sectors() return ? |
| 19:18:00 | S_a_i_n_t | G4Oblivion: Easiest way I found to doit was make a shortcut, right click it, then change the "target to include " -x" |
| 19:18:41 | ranma | funman: I haven't been using it to listen to audio lately ^^; |
| 19:19:01 | G4Oblivion | Ok. Thanks, I will try it. |
| 19:19:04 | bertrik | funman, I haven't seen a problem on my clipv1 and clip+ |
| 19:19:06 | funman | i dump/clean the dcache for the provided buffer before doing the transfer so it should be safe unless some code loads/store to/from this buffer while the transfer is in progress |
| 19:19:13 | kugel | funman: I used invalidate_dcache_range (not clean_) in FS #10805, I can't remember audio glitches |
| 19:19:25 | funman | with mp3 i can hear one glitch per album perhaps |
| 19:19:52 | funman | but given the weird problem with text_viewer and some ipods, eabi transition could have broken something else |
| 19:20:45 | funman | ranma: btw do you have an idea on how gdb is unwinding the stack without the frame pointer? |
| 19:21:03 | funman | perhaps it uses some hints from the .elf file? |
| 19:21:40 | funman | see FS #11399, the backtrace I found is way off from what gdb tells |
| 19:22:25 | | Quit amiconn (Disconnected by services) |
| 19:22:27 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
| 19:22:30 | G4Oblivion | ugh, its doing the same thing cmd does. I start it but nothing pops up. I don't see it under processes either. without -x its fine. |
| 19:22:42 | G4Oblivion | I will try again tomorrow when I have more time. Thanks for the help. |
| 19:22:45 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
| 19:22:49 | funman | r11 is #0 so i wonder how it gives a backtrace at all |
| 19:23:16 | funman | saratoga: have you run test_codec again since the PLL@248MHz commit was reverted? |
| 19:23:46 | saratoga | no, should i? |
| 19:23:51 | S_a_i_n_t | G4Oblivion |
| 19:24:12 | S_a_i_n_t | argh, G4Oblivion: Are you using " -x" or "-X" |
| 19:24:22 | funman | S_a_i_n_t: i can test, reading ABI / rb forums I'm not sure if the problems are fixed; and also someone reported glitches with Fuzev1 |
| 19:24:28 | funman | saratoga: ^ |
| 19:24:49 | S_a_i_n_t | a: it needs a space, and b: it needs to be a capital X |
| 19:24:59 | Ctcp | Ping from gevaerts!~fg@rockbox/developer/gevaerts |
| 19:25:19 | S_a_i_n_t | "C:\Cygwin\Local_Package_Directory\Setup\setup.exe -X" is my example, woks fine. |
| 19:25:27 | funman | saratoga: did test_codec crash or did you just play the .ape file normally? |
| 19:25:36 | saratoga | i played them normally |
| 19:25:44 | saratoga | ape didn't crash, it jus tplayed static after a while |
| 19:25:51 | funman | they won't fit in audiobuffer anyway for test_codec |
| 19:26:01 | ranma | funman: I'd assume it's using debug information from the .elf |
| 19:26:27 | funman | perhaps we can use -O with -fno-omit-frame-pointer |
| 19:26:38 | kugel | funman: very possible, you can run gdb without debug info if you give it the .elf file |
| 19:26:52 | CIA-8 | New commit by kugel (r27022): I shouldn't have touched backlight.c at all :\ Also fix the yellow in peakmeter.c |
| 19:28:02 | | Quit G4Oblivion (Quit: CGI:IRC (Ping timeout)) |
| 19:28:33 | | Join G4Oblivion [0] (www-data@giant.haxx.se) |
| 19:28:37 | CIA-8 | r27022 build result: All green |
| 19:29:24 | G4Oblivion | oh, wow. My bad, I forgot to use a capitial X. Thank you very much. |
| 19:29:33 | S_a_i_n_t | No problem. |
| 19:30:02 | G4Oblivion | I tried that with CMD and it didn't work so I thought it was supposed to be a lower case. |
| 19:30:30 | S_a_i_n_t | I actually don;t need to add the -X at all, I have no idea why some people do. |
| 19:30:49 | ranma | funman: BTW http://svn.rockbox.org/viewvc.cgi?view=rev;revision=26946 was causing me problems, I haven't checked yet if it's the buggy mps change or the switch to the _range() variants |
| 19:30:54 | S_a_i_n_t | I just checked for your benifit that it indeed works fo rme. |
| 19:30:59 | funman | saratoga: i'm testing ape (starting from c1000) |
| 19:31:02 | funman | on fuzev2* |
| 19:31:30 | funman | -> noise on c1000 after 50 seconds |
| 19:31:56 | | Join dfkt_ [0] (~dfkt@unaffiliated/dfkt) |
| 19:32:26 | funman | ranma: why is mps buggy? |
| 19:32:35 | | Quit G4Oblivion (Client Quit) |
| 19:34:59 | ranma | It depends on the endpoint, you were using the uninitilized counter value |
| 19:35:16 | ranma | i.e. it should be in the for loop |
| 19:35:21 | funman | oh right i see it .. |
| 19:35:47 | | Quit dfkt (Ping timeout: 265 seconds) |
| 19:36:25 | | Join playing-the-fool [0] (~zic@lin67-4-78-239-42-148.fbx.proxad.net) |
| 19:36:54 | ranma | And my "I wonder why the compiler didn't warn" comment is stupid, of course the build system doesn't see it since it's not enabled by default yet :) |
| 19:38:59 | funman | i should've paid attention to it anyway :/ |
| 19:39:37 | funman | if i undefined STORAGE_WANTS_ALIGN i don't get white noise (so far), but it can also be due to the aligned path being taken less often |
| 19:40:15 | saratoga | funman: did you think the current AMSv2 volume patch was ok? |
| 19:40:57 | funman | i should ask you, not the reverse :p |
| 19:41:10 | funman | i see nothing wrong with it |
| 19:41:11 | saratoga | well i think its ok, but what do i know |
| 19:41:18 | funman | +1 |
| 19:41:28 | saratoga | ok i'll commit it later and see if anyone complains |
| 19:41:43 | | Quit playing-the-fool (Quit: Ухожу Ñ Ð¾Ñ‚ ваÑ) |
| 19:41:48 | saratoga | FWIW i think volume could probably be increased on AMSv1 using the same method, but i'm not sure and people aren't complaining |
| 19:42:05 | saratoga | i remember dfkt saying volume on the clipv1 was lower then the OF when he tested |
| 19:42:34 | Jerom | saratoga: Are you increasing the current volume on AMSv2 ? |
| 19:42:43 | saratoga | yeah |
| 19:42:53 | Jerom | May I ask why ? |
| 19:43:04 | saratoga | why not? |
| 19:43:13 | Jerom | Not deafen people |
| 19:43:18 | funman | it'll only increase maximal volume, you're free to make it lower |
| 19:43:41 | Jerom | I'm currently at -20 for normal usage... |
| 19:43:42 | saratoga | if you don't want loud volume don't turn it up |
| 19:43:59 | Jerom | But well not my problem ^^ |
| 19:44:34 | funman | i think i'll leave this probable SD bug in trunk while TheSeven is messing with storage |
| 19:44:46 | saratoga | i don't get the SD bug |
| 19:44:49 | saratoga | test disk runs forever |
| 19:44:56 | saratoga | but playback screws up in seconds |
| 19:44:56 | TheSeven | funman: I'm not about to commit anything soon |
| 19:45:10 | funman | TheSeven: well you might spot some bugs in the process |
| 19:45:11 | TheSeven | if whatever you fixed is in the target tree, it won't collide |
| 19:45:29 | funman | saratoga: i commented out STORAGE_WANTS_ALIGN and it's now playing c4000 ok |
| 19:45:48 | funman | TheSeven: no no, i added bugs, not fixed them :P |
| 19:45:54 | saratoga | ahhh so its a playback/buffering.c bug? |
| 19:46:10 | funman | saratoga: no i think it's a race condition |
| 19:46:11 | dfkt_ | saratoga, that was a very old test with RB on the clipv1, and yes, RB was quieter than the OF... should i repeat it with a new build? |
| 19:46:20 | | Nick dfkt_ is now known as dfkt (~dfkt@unaffiliated/dfkt) |
| 19:46:34 | saratoga | dfkt: if you want, I'm not sure if its even possible to increase :) |
| 19:46:41 | funman | i can play back a full album (in mp3) and hear only 1 small glitch (repeated pattern or skipping some seconds) on the full time of the album |
| 19:47:08 | saratoga | the clipv1 uses a different DAC, but it looks to me like its not set to max volume, though maybe it doesn't go any higher even if set higher |
| 19:47:47 | saratoga | FWIW it seems to me like the clipv1 is quite loud already |
| 19:47:49 | funman | c5000 isn't realtime :o) |
| 19:50:28 | n1s | it is... on the beast |
| 19:50:53 | | Join Buschel [0] (~~andree@p54A3B9BF.dip.t-dialin.net) |
| 19:51:42 | funman | n1s: this is on fuzev2 (armv5/240MHz) |
| 19:52:21 | Buschel | can an as3525 guru take a look at FS #11419 ? |
| 19:52:25 | n1s | then your observation is probably correct :) |
| 19:52:56 | funman | Buschel: i think it's the problem we were just talking about |
| 19:53:38 | Buschel | ah, ok :o) |
| 19:54:21 | | Join DerPapst [0] (~Alexander@p4FE8F1F8.dip.t-dialin.net) |
| 19:56:03 | funman | not sure what I should do then, perhaps revert and put these in a patch on flyspray? |
| 19:56:39 | | Quit CGL (Remote host closed the connection) |
| 19:57:15 | | Quit DerPapst (Client Quit) |
| 20:00 |
| 20:01:12 | dfkt | saratoga, actually, OF was as loud on max volume as RB was on +6 (and distorted accordingly) - http://www.head-fi.org/forum/thread/498535/ignore-not-working-properly#post_6728902 |
| 20:01:39 | dfkt | oops |
| 20:02:00 | dfkt | http://www.anythingbutipod.com/forum/showpost.php?p=343274&postcount=1 |
| 20:03:39 | | Join Buschel_ [0] (~~andree@p54A3B9BF.dip.t-dialin.net) |
| 20:04:49 | | Join itcheg [0] (www-data@giant.haxx.se) |
| 20:06:46 | | Quit Buschel (Ping timeout: 248 seconds) |
| 20:07:29 | *** | Saving seen data "./dancer.seen" |
| 20:09:47 | | Quit itcheg (Quit: CGI:IRC (Ping timeout)) |
| 20:11:55 | Strife89 | What happened to the links to voice files on the Current Build page? |
| 20:12:29 | kugel | funman: that would probably be better |
| 20:12:38 | Strife89 | Or rather, the Daily build page. |
| 20:12:47 | kugel | funman: add the patches to FS #11419 |
| 20:13:02 | kugel | funman: err, FS #10805 |
| 20:13:34 | funman | i should check these patches and find what was wrong then |
| 20:14:24 | kugel | funman: I wonder if the cache needs to be invalidated after the transfer on reads? |
| 20:14:44 | funman | after or before shouldn't matter afaiu |
| 20:14:52 | funman | as long as we don't access the range |
| 20:15:07 | kugel | also, why isn't the cache handled by the dma code? any dma access needs cache coherency |
| 20:16:08 | funman | hm yes it makes sense |
| 20:19:16 | | Quit mc2739 (Ping timeout: 260 seconds) |
| 20:19:53 | funman | i think we should manage to make Torne use his clipv2 more so he gets to fix all the problems |
| 20:20:49 | pixelma | Strife89: follow the link to the users' ml ;) Manuals were affected too hence the topic... both were fixed today but will only be back to normal tomorrow |
| 20:21:09 | TheSeven | hm, splashf()ing a lot during file i/o doesn't seem to be a good idea |
| 20:21:13 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
| 20:21:28 | funman | TheSeven: draw something in a corner of the screen and lcd_update_rect() should be faster |
| 20:21:30 | | Quit kugel (Remote host closed the connection) |
| 20:21:43 | TheSeven | i doubt that this would have helped much |
| 20:21:47 | | Join stripwax [0] (~Miranda@87-194-34-169.bethere.co.uk) |
| 20:21:51 | TheSeven | it somehow messed up the font loading |
| 20:21:57 | Strife89 | pixelma: Ah, didn't see that ... For some reason XChat is showing the tail end of the topic. :/ |
| 20:22:04 | Strife89 | pixelma: Thanks, though. :) |
| 20:22:10 | TheSeven | i ended up with a main menu where only about half of the letters were displayed |
| 20:22:15 | TheSeven | looked funny :-) |
| 20:24:32 | CIA-8 | New commit by bieber (r27023): Theme Editor: Fixed some compiler warnings |
| 20:26:28 | CIA-8 | r27023 build result: All green |
| 20:27:26 | * | ranma just successfully wrote a file via usb using rockbox for the first time |
| 20:28:27 | | Join Luca_S [0] (www-data@giant.haxx.se) |
| 20:28:35 | gevaerts | ranma: congratulations! |
| 20:28:36 | Luca_S | ranma: woot! congrats :D |
| 20:28:39 | funman | ranma: how large? |
| 20:28:42 | bertrik | nice |
| 20:28:45 | * | gevaerts remembers that feeling :) |
| 20:29:07 | * | gevaerts also remembers finding the corrupted bytes afterwards :) |
| 20:29:27 | simonrvn | woohoo |
| 20:29:32 | ranma | funman: 6084736 Bytes |
| 20:29:42 | funman | wow :) congrats :) |
| 20:30:37 | Jerom | ranma: please tell me it's for AMSv2 |
| 20:31:46 | | Quit funman (Quit: free(random());) |
| 20:32:05 | ranma | Jerom: AMSv1 ;) |
| 20:32:40 | Jerom | I wanna cry ^^ |
| 20:32:43 | | Quit stripwax (Quit: http://miranda-im.org) |
| 20:32:50 | ranma | md5sum is fine too |
| 20:33:04 | simonrvn | oh damn. well, it's a start. :) |
| 20:33:23 | Luca_S | Jerom: feel free to work on it :-) |
| 20:33:32 | Jerom | I'm trying |
| 20:33:59 | Jerom | But well I'm a "normal" developer I don't work on embedded devices |
| 20:34:05 | * | TheSeven thinks he spotted the bug |
| 20:34:17 | gevaerts | Jerom: you have to start with something :) |
| 20:34:36 | Jerom | Well I'm reading pamaury's code and I can also make sens out of it |
| 20:34:41 | Jerom | sense* |
| 20:34:49 | | Join stripwax_ [0] (~Miranda@87-194-34-169.bethere.co.uk) |
| 20:34:55 | | Quit t0rc (Quit: Leaving) |
| 20:35:55 | Jerom | But It seems that the setup packet is never comming |
| 20:36:03 | Luca_S | huh.. the patch for PLL@384MHz on AMSv2 was reverted? |
| 20:36:13 | Jerom | Yes Luca_S |
| 20:36:28 | | Quit MethoS- (Remote host closed the connection) |
| 20:37:06 | ranma | Jerom: I'm wondering if the clocking is set up correctly in the v2 usb driver |
| 20:37:25 | ranma | If the 48MHz usb clock is wrong that would explain why it won't receive the setup packet |
| 20:37:52 | Jerom | The only interrupts that we are receiving is bus reset |
| 20:37:55 | Jerom | And enum done |
| 20:38:18 | ranma | Well, those don't depend on the usb clock |
| 20:38:31 | ranma | bus reset is just when both lines are pulled to 0 |
| 20:38:37 | Jerom | Ok |
| 20:38:41 | Luca_S | the revert message says it caused problems with uSD... I didn't experience them :/ |
| 20:38:51 | ranma | enum done is based on timeouts in the controller I think |
| 20:38:53 | | Quit wombat23 (Remote host closed the connection) |
| 20:38:59 | Jerom | Luca_S: http://forums.rockbox.org/index.php?topic=25104.0 |
| 20:39:06 | Jerom | These people experienced problems |
| 20:40:02 | | Quit halmi_ (Quit: halmi_) |
| 20:40:18 | TheSeven | sweet :) |
| 20:40:24 | Strife89 | I suspect I'm having issues with disk corruption, but might anyone care to decipher this error? : |
| 20:40:24 | TheSeven | pamaury: it's working |
| 20:40:42 | Strife89 | "Prefetch abort at FE21A1DA (0)" |
| 20:40:57 | Jerom | Well I cant try to copy some of the nano2g init code over to amsv2 |
| 20:40:58 | ranma | Jerom: I was wonering that because I do "CGU_USB = 1<<5 /* enable */ | (CLK_DIV(AS3525_PLLB_FREQ, 48000000) / 2) << 2 | 2; /* source = PLLB */" |
| 20:41:00 | Strife89 | This occured while booting Rockbox. |
| 20:41:04 | pixelma | Strife89: is this an Ipod? |
| 20:41:13 | Strife89 | pixelma: An iPod Color. |
| 20:41:15 | ranma | And the amsv2 driver just sets the enable bit without setting up the divider settings |
| 20:41:16 | Strife89 | 4G. |
| 20:41:40 | Jerom | ranma you think I should try copying over your init line ? |
| 20:41:46 | bertrik | PLLB is not enabled by default as far as I know |
| 20:41:47 | pixelma | amiconn reported unstability issues with recent builds on a Color too yesterday (IIRC) |
| 20:41:54 | Luca_S | Jerom: I just read that thread. Rockboxed FuzeV2 have always been a bit picky regarding to uSD, I had some troubles myself at some point |
| 20:41:55 | pixelma | or insability |
| 20:42:12 | pixelma | Strife89: you know what I mean ;\ |
| 20:42:13 | ranma | Jerom: I think it can't hurt to try, but of course the bit meanings might be different... |
| 20:42:17 | bertrik | We do know now how to configure and enable it though |
| 20:42:19 | Strife89 | pixelma: :) |
| 20:42:24 | Jerom | Ok will try it |
| 20:42:27 | Luca_S | might just be that some wait need better tuning |
| 20:42:35 | TheSeven | hm, should i commit the "unload fonts during USB" fix? |
| 20:42:50 | Strife89 | pixelma: I still suspect disk corruption. Earlier the bootloader couldn't even FIND rockbox.ipod. |
| 20:43:16 | Strife89 | pixelma: I rebooted to Disk Mode and recopied the file. |
| 20:43:19 | Jerom | Time to drain the battery again :p |
| 20:43:21 | pamaury | TheSeven: what was the problem ? |
| 20:43:39 | Strife89 | And this time, I got the crash. |
| 20:43:40 | pixelma | same thing reported by amiconn yesterday or the day before it |
| 20:43:51 | TheSeven | pamaury: me being stupid :) |
| 20:44:17 | pamaury | nothing unusual then :P |
| 20:44:39 | S_a_i_n_t | amiconn reported issues with a 5G Video IIRC |
| 20:44:59 | | Join wombat23 [0] (~beuteltie@adsl-99-39-2-249.dsl.pltn13.sbcglobal.net) |
| 20:45:04 | Strife89 | pixelma: If it means anything, I'm still using the testing version of that bootloader from a couple of months back. |
| 20:45:21 | n1s | Strife89: a disk scan never hutrs of course |
| 20:45:36 | CIA-8 | New commit by Buschel (r27024): Disable unneeded parts of mpc's file-I/O interface. |
| 20:45:59 | S_a_i_n_t | Strife89: It shouldn't make a difference, that bootloader made it into SVN |
| 20:46:10 | Strife89 | n1s: Yeah, that was my next move. |
| 20:46:19 | * | TheSeven wonders why usb on nano2g is so unstable lately |
| 20:46:38 | S_a_i_n_t | TheSeven: I find it stable, but dog slow. |
| 20:46:43 | Strife89 | Care to refresh my memory? Isn't it "chkdsk /f X:" ? |
| 20:46:49 | Strife89 | Where X is the drive letter? |
| 20:47:01 | S_a_i_n_t | I mean, it doesn't drop out or anything (for me) but transfer is rather painfull |
| 20:47:01 | TheSeven | S_a_i_n_t: The drive keeps disappearing in the middle of a transfer |
| 20:47:17 | pixelma | Strife89: http://www.rockbox.org/irc/log-20100620#18:51:54 |
| 20:47:28 | CIA-8 | r27024 build result: All green |
| 20:47:29 | n1s | Strife89: i think most windows command line tools have this /? thing that tells you how to use them |
| 20:47:41 | TheSeven | S_a_i_n_t: how slow is it? which storage size? which flash variant? |
| 20:47:44 | CIA-8 | New commit by jethead71 (r27025): iPod 3G: Ensure wheel repeats are not generated unless the same keycode is posted more than once in a row, regardless of velocity. Remove 'was_hold' ... |
| 20:47:51 | TheSeven | I'm getting 4-5MB/s on my 8GB nano2g |
| 20:47:56 | Jerom | ranma: Where do you pull this 48Mhz ? |
| 20:49:02 | S_a_i_n_t | TheSeven: I'm getting about 3~4MBps on 4GB nano2g |
| 20:49:14 | TheSeven | that seems to be OK |
| 20:49:16 | S_a_i_n_t | same on the 2 and 8 GB ones |
| 20:49:17 | TheSeven | the OF can't be much faster |
| 20:49:25 | n1s | Strife89: anyway, as pixelma said this sounds more like some fun bug, prefetch abort is usually some weird bug |
| 20:49:26 | CIA-8 | r27025 build result: 19 errors, 79 warnings (jethead71 committed) |
| 20:49:30 | ranma | Jerom: 384MHz PLLB divided by 8 |
| 20:49:52 | ranma | You may have to enable the pllb first (see usb-drv-as3525.c) |
| 20:50:21 | S_a_i_n_t | the OF seems to easily do 4~5 to 5MBps flat, but, I'm really lucky to get 3~4MBps with RB |
| 20:50:39 | S_a_i_n_t | that is the peak speed, it can drop to less than 1MBps |
| 20:52:03 | Strife89 | n1s: So it would seem. chkdsk turned up some fragment in .rockbox, but that was it. I copied rockbox.ipod again and got the same result. |
| 20:52:25 | Strife89 | I hope this bug doesn't affect the iPod Videos. |
| 20:52:27 | | Join Xerion [0] (~xerion@82-170-197-160.ip.telfort.nl) |
| 20:52:37 | Jerom | You got to love embedded devices, registers, registers some more registers ^^ |
| 20:53:28 | bertrik | Jerom, pamaury need help setting up the PLL? |
| 20:53:57 | Strife89 | Might the prefetch problem have anything to do with the Database or dircache? |
| 20:54:02 | Jerom | Well it seems that USB needs PLLB for it's clock |
| 20:54:31 | Jerom | And it disabled by default so I copied some of ranma code over to see if I can enable it and receive something from the usb bus |
| 20:55:17 | bertrik | 240 MHz from PLL A / 5 = 48 MHz too, or do you really need PLL B? |
| 20:55:22 | pamaury | I don't know. On my clip+ it is correctly enumerated and responds to bus reset. If the clock was badly setup, it would not work at all |
| 20:55:49 | ranma | pamaury: No, the bus reset and enumeration don't depend on getting the clock right I think |
| 20:56:04 | ranma | Bus reset is just pulling both data lines low, clock is not relevant |
| 20:56:05 | Jerom | bertrik: Honestly I have no idea :p |
| 20:56:28 | pamaury | But enum speed is likely to require the clock |
| 20:56:29 | ranma | For enumeration it'll probably just fall back to FS or LS if the clock is wrong |
| 20:56:36 | bertrik | but enumeration (assuming you mean setting the USB device address) does send a packet, right? |
| 20:56:37 | pamaury | It reports HS |
| 20:57:10 | Jerom | enumeration happens before setup |
| 20:57:14 | pamaury | no, the "enumeration" is the speed detection |
| 20:57:16 | ranma | Hmm, if it reports HS that might be an indication the clock is alright |
| 20:57:47 | | Quit Buschel_ (Ping timeout: 276 seconds) |
| 20:57:49 | pamaury | It reports HS something like 10 times and after linux fallbacks to FS but still I can't make it work |
| 20:57:56 | pamaury | today I will try to work without DMA, doing PIO |
| 20:58:00 | | Join stripwax [0] (~Miranda@87-194-34-169.bethere.co.uk) |
| 20:58:29 | bertrik | sometimes it can also help to run without interrupts, just polling the interrupt routine |
| 20:59:08 | n1s | Strife89: as amiconn said it feels like something fundamental, and is probably either a bug in the new toolchain or a previously latent bug in rockbox, uncovered by the new toolchain |
| 20:59:28 | | Join Buschel [0] (~~andree@p54A3B9BF.dip.t-dialin.net) |
| 21:00 |
| 21:00:00 | ranma | http://www.beyondlogic.org/usbnutshell/usb2.htm just says it does a 'hs chirp' for enumeration, maybe it's too tolerant on that part? |
| 21:00:07 | | Quit stripwax_ (Ping timeout: 252 seconds) |
| 21:00:23 | pamaury | bertrik: but polling what ? I would poll the status register, that is the one which triggers interrupt :) |
| 21:00:34 | pamaury | Ok, let's try with the clock then |
| 21:00:58 | | Quit Luca_S (Quit: CGI:IRC (EOF)) |
| 21:02:10 | | Join Luca_S [0] (www-data@giant.haxx.se) |
| 21:02:29 | pamaury | The OF, it if does stick with the init sequence, sets CGU_USB to 0x20 which means usb phy clock enable, div=1,clk=clk_main |
| 21:02:40 | pamaury | IF the as3525 datasheet applies |
| 21:03:36 | pamaury | How can I change that ? |
| 21:03:45 | TheSeven | http://www.rockbox.org/tracker/task/11428 |
| 21:03:55 | TheSeven | should I commit that? |
| 21:04:24 | Jerom | pamaury: I'm currently trying ranma's clocking |
| 21:04:54 | S_a_i_n_t | TheSeven: S_a_i_n_t doesn't see why not. |
| 21:05:18 | ranma | pamaury: Do we use the same PLLA clock as OF? Maybe the controller has an internal register for clock division? |
| 21:05:34 | pamaury | I don't know |
| 21:05:35 | | Quit rasher (Read error: Connection reset by peer) |
| 21:06:01 | | Quit Luca_S (Client Quit) |
| 21:06:34 | pamaury | The PHY part of the controller is basically a mystery. No two controllers in the nature do the same thing :( The only thing I know is what I reversed engineered |
| 21:07:24 | Jerom | I forgot to remove uSB card... OF taking ages to scan the sd |
| 21:07:30 | Strife89 | Let's see what happens if I delete .glyphcache, .playlist_control, config.cfg, DATABA~8.TCD (looks like it got corrupted :/ ), and nvram.bin; then try to boot. |
| 21:08:16 | S_a_i_n_t | ETOOMUCHBOLD! |
| 21:08:19 | pamaury | It does not seem to change something |
| 21:08:28 | pamaury | (I tried ranman clock setup) |
| 21:08:46 | Strife89 | What do you know? It boots. :D |
| 21:08:52 | bertrik | pamaury, what clock frequency does the USB peripheral require? |
| 21:09:05 | pamaury | I don't know |
| 21:09:35 | | Part watto |
| 21:09:47 | bertrik | I thought it usually is 48 MHz for USB2.0 devices |
| 21:09:52 | | Join rasher [0] (~rasher@0x5550f5a3.adsl.cybercity.dk) |
| 21:09:52 | | Quit rasher (Changing host) |
| 21:09:52 | | Join rasher [0] (~rasher@rockbox/developer/rasher) |
| 21:09:59 | bertrik | *hi-speed devices I mean |
| 21:10:10 | Strife89 | n1s: I think I just discovered something. |
| 21:10:36 | gevaerts | Isn't it also often 60MHz? |
| 21:10:36 | | Join Luca_S [0] (www-data@giant.haxx.se) |
| 21:10:49 | pamaury | Yes but I can't be sure, I don't know what the usb controller *needs* |
| 21:10:53 | Strife89 | I deleted some of the cache/temporary files from.rockbox, as listed three minutes ago. |
| 21:11:05 | Strife89 | This time, Rockbox booted. |
| 21:11:12 | Strife89 | (This is on the iPod Color.) |
| 21:11:26 | pamaury | 48MHz seems to be common and sensical so let's stick with it |
| 21:11:29 | Strife89 | As a test, I started a song and then changed themes. |
| 21:11:34 | TheSeven | pamaury: http://www.rockbox.org/tracker/task/11429 |
| 21:11:38 | Strife89 | Then I shut down Rockbox and rebooted. |
| 21:11:52 | pamaury | I'll check with the usb analyzer if something changed |
| 21:12:00 | Strife89 | It was successful. |
| 21:12:08 | pamaury | TheSeven: thanks, I'll have a look at later |
| 21:12:13 | S_a_i_n_t | Strife89: None of those files are present in the .zip install thogh correct? |
| 21:12:21 | Strife89 | S_a_i_n_t: Correct. |
| 21:12:23 | S_a_i_n_t | something changed in their format? |
| 21:12:28 | Strife89 | Evidently. |
| 21:12:39 | Strife89 | Or in the way Rockbox handles them. |
| 21:12:39 | S_a_i_n_t | intersting... |
| 21:12:56 | S_a_i_n_t | Nice catch |
| 21:13:06 | pamaury | Jerom: any success ? |
| 21:13:12 | Jerom | No... |
| 21:13:34 | pamaury | bertrik: when did the main clock changed recently ? |
| 21:13:45 | pamaury | (on as3525v2) |
| 21:13:50 | Jerom | It changed forth than back again |
| 21:14:12 | Strife89 | S_a_i_n_t: Spread the word, then? I think this warrants plenty of investigation. |
| 21:14:15 | bertrik | indeed, now it's back at 240 MHz again |
| 21:14:36 | S_a_i_n_t | That's the awesome thing about IRC logs ;) |
| 21:15:01 | n1s | Strife89: ah, that could make sense, the new abi probably changes rules for struct layouts in memory so some of those binary files could get loaded wrongly |
| 21:15:31 | | Quit stoffel (Remote host closed the connection) |
| 21:15:51 | pamaury | Anyway, I will try with PIO, see if I have more luck. |
| 21:15:55 | n1s | if amiconn reads the logs it would be interesting if he still has said files and could investigate |
| 21:16:43 | * | S_a_i_n_t wonders if Strife89 can pull said files out of the recycler |
| 21:17:03 | n1s | TheSeven: it (FS #11428) sounds like The Right Thing (TM) |
| 21:17:14 | Strife89 | S_a_i_n_t: You're in luck, I backed up the entire .rockbox folder before updating it on the iPod. |
| 21:17:39 | n1s | Strife89: would be interestin gif you could work out which file(s) cause it |
| 21:17:54 | S_a_i_n_t | indeed. |
| 21:17:56 | TheSeven | n1s: it will make the HID mode text look a bit ugly |
| 21:18:21 | n1s | working and ugly > pretty and broken :) |
| 21:18:39 | Strife89 | n1s, S_a_i_n_t: I dunno how well I could do that. I'll give it a try, but what say I package those files and send 'em to you? |
| 21:19:06 | n1s | although i suppose we could preload needed glyphs for the usb screen... |
| 21:19:12 | S_a_i_n_t | easy to tell, try installing the build you backed up, then delete the files one by one. |
| 21:19:21 | S_a_i_n_t | or, one at a time, then try booting. |
| 21:19:25 | n1s | Strife89: i have no ipod 4g |
| 21:19:44 | | Join vmwareuser [0] (www-data@giant.haxx.se) |
| 21:20:04 | Strife89 | n1s: Ah. |
| 21:20:15 | Strife89 | S_a_i_n_t: Works for me. |
| 21:20:16 | | Quit stripwax (Ping timeout: 245 seconds) |
| 21:20:33 | Strife89 | I'll package a .zip with the files anyway, just in case. |
| 21:20:58 | S_a_i_n_t | yeah, be good to keep it aside to see exactly what broke, then why. |
| 21:21:10 | | Quit BHSPitMonkey (Ping timeout: 265 seconds) |
| 21:21:44 | | Join stripwax [0] (~Miranda@87-194-34-169.bethere.co.uk) |
| 21:21:54 | | Quit stripwax (Client Quit) |
| 21:22:06 | Strife89 | Here's the .zip. http://dl.dropbox.com/u/1905158/problem_files.zip |
| 21:22:11 | Strife89 | On to testing. |
| 21:22:29 | vmwareuser | Can someone help me to get Rockbox to compile for a 5.5G iPod on the Vmware platform? It worked for me when I last tried a few months ago, but with the current svn revision the make script just throws heaps of "arm-elf-eabi-gcc: command not found" errors ...? |
| 21:22:51 | TheSeven | you need to install the new (eabi) toolchain |
| 21:23:04 | TheSeven | we've switched compilers some days ago |
| 21:23:04 | Luca_S | vmwareuser: I had the same problems, and after a few hours of trying, I simply downloaded an ubuntu vm off the net |
| 21:23:18 | Luca_S | rockboxdev.sh fails there |
| 21:23:30 | | Join katg [0] (~quassel@cpc1-rdng15-0-0-cust791.winn.cable.ntl.com) |
| 21:24:15 | n1s | vmwareuser: run the rockboxdev.sh script in the tools dir of the checkout (as root) and select the arm-eabi toolchain |
| 21:24:53 | TheSeven | any pp ipod owners around? |
| 21:25:00 | | Quit Topy44 (Ping timeout: 260 seconds) |
| 21:25:05 | Luca_S | and if it works, please report how you did it :P |
| 21:25:06 | TheSeven | could anybody have a look why the hardware keyclick patch is failing on pp? |
| 21:25:07 | vmwareuser | nls: thanks a lot, I'll try that now. |
| 21:25:27 | saratoga | you'll probbaly have to install some other packages before it'll build |
| 21:25:39 | TheSeven | vmwareuser: also make sure to first remove the old toolchains from the PATH before running rockboxdev.sh |
| 21:25:51 | Jerom | install texinfo |
| 21:25:54 | Luca_S | saratoga: it failed saying 'please update config.guess' |
| 21:25:55 | Jerom | You will probably need it |
| 21:26:06 | | Join kaylinsigswort [0] (~Zigtown@CPE00259ce0fdb2-CM0014f8cc807a.cpe.net.cable.rogers.com) |
| 21:26:12 | TheSeven | if you don't do this, the toolchains will seem to build properly, but will fail in odd ways while compiling rockbox |
| 21:27:13 | saratoga | i didn't have to remove anything from the path when building under ubuntu FWIW |
| 21:27:36 | TheSeven | saratoga: but you probably didn't have the old arm-elf toolchain installed |
| 21:27:55 | saratoga | yes I do |
| 21:28:00 | TheSeven | it failed for me (and several others) if it finds those while building the new ones |
| 21:28:06 | saratoga | my machine has been a build server for a number of years |
| 21:28:07 | TheSeven | ubuntu karmic over here |
| 21:28:14 | saratoga | same |
| 21:28:22 | | Join Topy44 [0] (~topy@my.fastsh.it) |
| 21:28:31 | TheSeven | hm, interesting |
| 21:28:33 | vmwareuser | TheSeven: removing the old toolchain on the vmware machine means removing /usr/local/arm-elf/bin from PATH? |
| 21:28:54 | TheSeven | yes, and deleting that directory is also a good idea |
| 21:29:23 | TheSeven | the new toolchains will be installed to /usr/local/bin anyway |
| 21:29:42 | S_a_i_n_t | I was going to say that, not only did I need to remove the path, I needed to remove the old toolchain dir also |
| 21:30:03 | n1s | it failed for me too but i had a slightly older arm-elf-eabi in my PATH |
| 21:30:04 | * | S_a_i_n_t is glad TheSeven has this problem also. |
| 21:30:18 | S_a_i_n_t | that'd be the third person I've seen have that trouble |
| 21:30:33 | | Quit Luca_S (Quit: CGI:IRC) |
| 21:30:39 | S_a_i_n_t | woo! 4 then counting n1s |
| 21:30:46 | S_a_i_n_t | I'm not alone ;) |
| 21:31:37 | n1s | S_a_i_n_t: as long as the old toolchains aren't in your PATH you should be fine (i have tried that= |
| 21:32:21 | S_a_i_n_t | I needed to remove the toolchain dir as well. |
| 21:32:26 | TheSeven | n1s: I'm not completely sure, but it's very unlikely that they were in the path when I build the new ones, yet it failed |
| 21:32:46 | TheSeven | i ended up removing all *arm-elf*, *sh-elf*, ... stuff from /usr/local |
| 21:32:57 | n1s | weird |
| 21:33:22 | TheSeven | actually the old ones were never in the path for the root account |
| 21:33:24 | n1s | btw jhMikeS posted a thread in the forums with an updated vmware 'm not completely sure, but it's very unlikely that they were in the path when I build the new ones, yet it failed |
| 21:33:24 | n1s | <TheSeven> i ended up removing all *arm-elf*, *sh-elf*, ... stuff from /usr/local |
| 21:34:32 | n1s | eh, sorry, not intended paste |
| 21:38:33 | Jerom | is the PLLA static or can it change ? |
| 21:38:56 | saratoga | it can be adjusted but it doesn't reclock during normal use |
| 21:39:10 | Jerom | thx |
| 21:39:39 | | Quit katg (Read error: Connection reset by peer) |
| 21:43:16 | bieber | JdGordon: If an identifier is specified in a Vi tag, does that make it hidden to begin with? |
| 21:45:26 | TheSeven | liar: What's the status of FS #10894? |
| 21:46:53 | * | FlynDice reads about uSD problems after adjusting PLLA freq and wonders if saratoga might have a AMSv2 with uSD to check frequencies on like he did for v1's |
| 21:48:33 | amiconn | S_a_i_n_t: I have no G5. I noticed the problem on my iPod Photo |
| 21:48:52 | amiconn | It *might* actually be the database code |
| 21:49:22 | Strife89 | The database code ..... |
| 21:49:25 | Strife89 | Hmmm. |
| 21:49:29 | amiconn | While a complete rebuild is meant to be that, there have been reports that it might not under certain conditions |
| 21:50:06 | amiconn | The usual suggestion in such a case was to delete all .tcd files, then reboot and rebuild |
| 21:50:10 | saratoga | FlynDice: with the spec analyzer? would that tell us anything new? i think we have a good idea what the frequencies are |
| 21:51:13 | amiconn | Perhaps a somewhat broken .tcd file (in whatever way) makes the database code enter a weird state it won't recover from |
| 21:52:29 | amiconn | It would explain the various weird effects I've seen. First a data abort at boot. Then, after rebooting and starting said rebuild (with the old .tcd files in place) neverending database scan with the statistics in the debug menu acting up |
| 21:52:30 | FlynDice | saratoga: I don't think there's documentation for the divider we use to set the card clock rate, I just made an educated guess and it worked. |
| 21:52:40 | n1s | if that can be confirmed, isn't there some kind of version for these files that can be bumped? |
| 21:53:10 | amiconn | After reset, another try to rebuild the db finally worked |
| 21:54:02 | amiconn | Perhaps the db code needs some sanity checks when loading the .tcd files? |
| 21:54:23 | S_a_i_n_t | wouldn't the .tcd files mean (in some cases, most often in fact, unnecessarily) that the db would be rebuilt on every update? |
| 21:54:24 | amiconn | If there are pointers, a broken .tcd may potentially cause overwrites of arbitrary memory |
| 21:54:31 | saratoga | http://www.sparkfun.com/commerce/product_info.php?products_id=9419 |
| 21:54:35 | saratoga | could use one of these to do it |
| 21:54:43 | S_a_i_n_t | s/wouldn't the/wouldn't versioning the/ |
| 21:55:19 | saratoga | though is it that hard to pull apart a clip+ |
| 21:56:04 | * | bertrik was about to mention that sniffer |
| 21:56:09 | | Quit kaylinsigswort (Remote host closed the connection) |
| 21:56:11 | amiconn | Strife89: Eh, you also had a corrupted .tcd file (shortname only)? |
| 21:56:25 | Strife89 | amiconn: Yep. |
| 21:56:33 | * | amiconn had that as well |
| 21:56:55 | amiconn | Imo that's another hint towards the db code |
| 21:57:09 | * | S_a_i_n_t has seen his database go haywire when a .tcd was currupted. |
| 21:57:10 | * | TheSeven wonders why screendump could stkov |
| 21:57:15 | bertrik | saratoga, it mentions it sniffs SPI traffic, but I guess we can measure at least the clock rate with it |
| 21:57:30 | S_a_i_n_t | the file included a ~ after it got chewed by whatever chewed it. |
| 21:57:38 | saratoga | well it gives you all the pins, so i could clip a probe to it and measure the clock rate |
| 21:57:56 | amiconn | The ~ is part of the standard scheme for shortnames |
| 21:57:58 | n1s | S_a_i_n_t: it would trigger a rebuild only when the version was bumped |
| 21:58:02 | saratoga | all my equipment is analog anyway |
| 21:58:23 | amiconn | n1s: Versioning wouldn't protect from broken files |
| 21:58:46 | Strife89 | S_a_i_n_t, n1s: I can't seem to reproduce the error. :/ |
| 21:58:49 | amiconn | It might still be a good idea though |
| 21:58:55 | n1s | amiconn: true, but in this case it could have been used (if anyone would have anticipated the breakage) |
| 21:59:22 | S_a_i_n_t | Strife89: Even if you write the old backed up build back to the iPod? |
| 21:59:33 | Strife89 | S_a_i_n_t: Yup. :/ |
| 21:59:39 | S_a_i_n_t | weird. |
| 21:59:40 | bertrik | I guess the rockbox fund could pay for it (and pay for a full-size SD card sniffer too) |
| 21:59:50 | n1s | but then i don't know how we would deal with runtime data |
| 22:00 |
| 22:00:46 | Strife89 | S_a_i_n_t: I have a hunch that it had something to do with a theme failing to load, but that's all it is - a hunch. |
| 22:01:13 | saratoga | its 15 dollars i don't really mind, though it might be quicker to just pop open the player |
| 22:02:11 | Jerom | pamaury: I got an out ep int ! |
| 22:02:12 | saratoga | on the e200v2 you could get at the sd contacts just by opening the device |
| 22:02:33 | pamaury | Jerom: \o/ how ? |
| 22:02:44 | Jerom | /* PHY clock */ |
| 22:02:46 | Jerom | CGU_USB = 1<<5 /* enable */ |
| 22:02:48 | Jerom | | (CLK_DIV(AS3525_PLLA_FREQ, 60000000)) << 2 |
| 22:02:50 | Jerom | | AS3525_CLK_PLLA; /* source = PLLA */ |
| 22:02:55 | Jerom | sry for copy pasting |
| 22:03:16 | bertrik | nice |
| 22:03:46 | pamaury | where does this 60Mhz comes from ? |
| 22:04:01 | Jerom | Someone said it sooner ^^ |
| 22:04:08 | gevaerts | 480/8 |
| 22:04:26 | gevaerts | 60MHz makes lots of sense as a byte clock on usb |
| 22:07:06 | pamaury | hum, doesn't work on mine, perhaps I have bad modification on the usb code. You are using my svn code ? |
| 22:07:31 | Jerom | Yes |
| 22:07:32 | *** | Saving seen data "./dancer.seen" |
| 22:07:42 | | Join blu3t0oth [0] (~blu3t0oth@pD9577CBF.dip.t-dialin.net) |
| 22:09:15 | | Join toffe82_ [0] (~chatzilla@12.169.218.14) |
| 22:11:02 | * | TheSeven is wondering if he should commit FS #11428 right away or wait for further input |
| 22:11:06 | | Quit toffe82 (Ping timeout: 245 seconds) |
| 22:11:39 | Strife89 | Depends on what it is. :) |
| 22:12:03 | Strife89 | Ah. |
| 22:12:04 | CIA-8 | New commit by bieber (r27026): Theme Editor: Reworked information passing among render functions, now loads all viewports and shows Custom UI viewport in blue |
| 22:13:17 | TheSeven | hm, nobody complains |
| 22:13:33 | pamaury | Jerom: it works ! That's awesome |
| 22:13:41 | pamaury | I will finally be able to write this driver |
| 22:13:46 | Jerom | :) I'm happy to |
| 22:13:54 | blu3t0oth | Rockbox is awesome! |
| 22:13:59 | CIA-8 | r27026 build result: All green |
| 22:14:12 | Jerom | It will be a good day when we can finally drop the OF |
| 22:14:15 | pamaury | That's incredibly lucky that the svn code is ok, it would not work on my local version |
| 22:14:24 | pamaury | I need to figure this out |
| 22:15:51 | pamaury | hum, there was a as3525 clock change, I wasn't at head; perhaps it explain things |
| 22:17:22 | pamaury | yep |
| 22:17:27 | vmwareuser | TheSeven: I installed the new toolchain using tools/rockboxdev.sh (after I removed the arm-elf directory from /usr/local). I reran tools/configure, and now the make command fails after only two lines of output: "CC apps/action.c", followed by "/home/user/rockbox/apps/action.c:26:18: error: lang.h: no such file or directory". Any idea why this happens? I'm using a fresh SVN checkout of Rockbox. |
| 22:17:32 | saratoga | head is back to the way it was before I think |
| 22:17:41 | pamaury | Okay, I will commit my changes |
| 22:17:45 | saratoga | vmwareuser: make clean |
| 22:18:45 | vmwareuser | saratoga: Oops. Thanks : ) |
| 22:18:53 | CIA-8 | New commit by theseven (r27027): Allow UI fonts to be unloaded (needed for FS #11428) |
| 22:19:34 | CIA-8 | New commit by theseven (r27028): Unload fonts when entering UMS mode to close their file handles. Closes FS #11428 |
| 22:20:19 | pamaury | Jerom: you definitely need to be in the credits for that :) |
| 22:20:28 | pamaury | TheSeven: you finally committed it :) |
| 22:20:36 | CIA-8 | r27027 build result: All green |
| 22:20:39 | Jerom | :) |
| 22:20:47 | TheSeven | also a nice revision number :-) |
| 22:22:17 | CIA-8 | r27028 build result: 10 errors, 1 warnings (theseven committed) |
| 22:22:42 | | Quit n1s (Quit: Lämnar) |
| 22:22:51 | TheSeven | urgh. charcell. |
| 22:24:59 | | Join binaryhermit [0] (~binaryher@adsl-99-141-184-240.dsl.emhril.sbcglobal.net) |
| 22:24:59 | CIA-8 | New commit by theseven (r27029): Fix charcell red. |
| 22:25:33 | CIA-8 | New commit by pamaury (r27030): as3525v2-usb: add a few missing define for completeness, finish reorganization of the header |
| 22:25:40 | CIA-8 | New commit by pamaury (r27031): as3525v2-usb: add support to derive usb clock from pllb, correct endpoint listing, simplify a few things. |
| 22:25:48 | CIA-8 | New commit by pamaury (r27032): as3525-usb: change clock handling |
| 22:25:55 | CIA-8 | New commit by pamaury (r27033): as3525v2-usb: definitely switch to this clock handling, it is the good one |
| 22:26:45 | CIA-8 | r27029 build result: All green |
| 22:27:13 | pamaury | Jerom: what is your real name ? So I can add you to the credits |
| 22:27:45 | Jerom | Jérôme Heil |
| 22:28:14 | Jerom | Don't look up my name on google, it's not me :p |
| 22:28:29 | CIA-8 | r27033 build result: All green |
| 22:28:57 | pamaury | don't worry, it's just that we only put real name in credits |
| 22:34:42 | | Part blu3t0oth ("Verlassend") |
| 22:34:43 | CIA-8 | New commit by pamaury (r27034): Add Jérôme Heil to CREDITS for figuring out the usb clock problem on as3525v2. |
| 22:34:50 | pamaury | Now the hard work begins |
| 22:36:51 | CIA-8 | r27034 build result: All green |
| 22:37:45 | Jerom | I copied most of the code from usb-s3c6400.c over but now that you changed the defines ^^ |
| 22:38:28 | pamaury | At the end, we might merge the two drivers |
| 22:38:31 | simonrvn | woohoo Jerom |
| 22:38:51 | simonrvn | \o/ |
| 22:38:58 | Jerom | Thx \o/ |
| 22:39:08 | simonrvn | :) |
| 22:39:29 | pamaury | However, the two usb controllers have different versions. As we don't have the datasheet for the 2.2 nor the 2.6, we difference between them are unclear. The linux patch I have suggest they are really small but... |
| 22:39:42 | pamaury | *the |
| 22:41:41 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
| 22:41:42 | simonrvn | *nod* |
| 22:42:47 | Jerom | btw I noticed a strange behavior from the controller, now that we ACK the interrupt it wait 5s before it sends the default reply |
| 22:43:19 | | Join piggz [0] (~piggz@78.151.66.184) |
| 22:43:47 | piggz | hi..when the database is updating in the background, is there a way to tell its status, like, if its finished? |
| 22:44:00 | pamaury | We need to write a proper handling for this |
| 22:44:51 | Jerom | piggz: When the small hd or sdcard icon disappear |
| 22:45:30 | piggz | Jerom: beside the clock? |
| 22:45:55 | Jerom | Yes it show "disk" activity |
| 22:46:12 | Jerom | When there is no more activity the database as finish its work |
| 22:46:17 | Jerom | has* |
| 22:46:40 | | Quit Buschel (Ping timeout: 260 seconds) |
| 22:46:47 | Jerom | pamaury: You don't wanna use a struct wakeup ? |
| 22:48:13 | vmwareuser | For the record, the current SVN revision compiled just fine on the vmware image that is available for download on rockbox.org. After deleting the old toolchain, running rockboxdev.sh to install the new one and running a 'make clean' in the build directory, compilation worked without errors. Now I've got a current version of Rockbox with my beloved wikiviewer plugin installed. Thanks! |
| 22:48:39 | pamaury | I don't know what you call a struct wakeup but yes I will use a wakeup object. But for now, I need to write in/out handlers, see if it correctly handles controls transfers and also check both HS and FS |
| 22:49:25 | piggz | Jerom: cool, cheers, its gone off now |
| 22:49:35 | | Quit hebz0rl (Ping timeout: 276 seconds) |
| 22:49:49 | | Quit vmwareuser (Quit: CGI:IRC) |
| 22:55:41 | * | TheSeven will go to sleep |
| 22:55:52 | kisak | have a nice sleep |
| 22:55:55 | TheSeven | i hope there will be some comments in fs#11429 tomorrow :-) |
| 22:56:16 | simonrvn | g'night |
| 22:56:30 | Jerom | pamaury: When the usb core request a endpoint, the direction is from the host point of view ? |
| 22:56:42 | pamaury | yes |
| 22:56:43 | pamaury | always |
| 22:58:16 | | Quit evilnick_B (Quit: Page closed) |
| 22:58:22 | | Join Meekrab [0] (~Meekrab@S01060013468957f1.ed.shawcable.net) |
| 23:00 |
| 23:02:09 | | Quit bmbl (Quit: Bye!) |
| 23:05:06 | Meekrab | Hi! |
| 23:05:12 | | Join detaos [0] (~quassel@ip72-218-104-242.hr.hr.cox.net) |
| 23:05:23 | Meekrab | Who wants to pick up the zune project again? |
| 23:07:19 | kisak | isn't the easiest way to use Hide-a-pod to port to the zune? |
| 23:08:32 | kisak | I mean ... noone appears to be active at this time, feel free to find where someone else left off and dig in |
| 23:09:03 | Bagder | Meekrab: go ahead, lead the way |
| 23:09:21 | Bagder | afaik the zune was never even sold outside of north america |
| 23:09:31 | Meekrab | I was dicking with the ability to inject code into a running machine, but sofar i need a 4096bit RSA encrypted hash |
| 23:10:36 | | Quit Topy44 (Ping timeout: 260 seconds) |
| 23:12:34 | Meekrab | If |
| 23:12:43 | | Join Topy44 [0] (~topy@my.fastsh.it) |
| 23:13:17 | | Quit Jerom (Quit: Jerom) |
| 23:14:09 | Meekrab | If I designed a simple yet powerful OPEN SOURCE DMP, would I be allowed to sell the hardware with rockbox? |
| 23:14:23 | Bagder | yes |
| 23:14:31 | Bagder | as long as you also offer the source code to it |
| 23:14:39 | Meekrab | Well yeah, hence open source |
| 23:19:30 | | Join MethoS- [0] (~clemens@134.102.106.250) |
| 23:20:03 | | Join Jerom [0] (~jerome@95.171.137.241) |
| 23:20:40 | Meekrab | Basically I have the idea for an ARM based unit with a 320x480 screen, 30gb HDD, SD expansion, AV out, and a common USB B cable output and custom dock port (dubbed the geekport) |
| 23:21:21 | piggz | that would be an ipod video then? ;) |
| 23:21:25 | Meekrab | I'd call it the BeatBox |
| 23:21:30 | piggz | - the sd |
| 23:22:20 | domonoky1 | Meekrab: maybe you want to look into what the lyre project is trying todo.. |
| 23:22:23 | Meekrab | Basically it'll look like a zune with a bigass screen and buttons on the side of the unit |
| 23:23:08 | | Join AzureWurk [0] (opera@c-24-21-44-114.hsd1.wa.comcast.net) |
| 23:23:15 | Jerom | Meekrab: You should look at this thread for a start: http://forums.rockbox.org/index.php?topic=6751.0 |
| 23:23:18 | pamaury | TheSeven: shouldn't your usb driver call usb_core_transfer_complete even on transfer timeout ? |
| 23:24:38 | AzureWurk | question: is it normal for the I2SO clock on the fuze v2 to drp from 240mhz to 5mhz when playing ogg files? |
| 23:24:45 | Meekrab | I project that it'll cost about $200, with your basic stuf |
| 23:24:52 | Meekrab | like usb, headphones, etc |
| 23:25:05 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
| 23:25:51 | Meekrab | I'm thinking I'm going to base it around the zune, and use the toshiba HDD and familiar batteries except without the vendor lock in on firmware/Os |
| 23:27:08 | Meekrab | would 64mb for buffering ram be overkill? after all the hdd does suck alot out of the battery on spinup |
| 23:27:49 | Jerom | Well if you wanna use a hdd |
| 23:27:58 | Jerom | Then I don't think it is |
| 23:28:01 | dfkt | <Meekrab> Basically it'll look like a zune with a bigass screen <−− the current zune hd has the same screen resolution /// <Meekrab> Basically I have the idea for an ARM based unit with a 320x480 screen, 30gb HDD <−− nowadays i guess 32gb of flash memory would be more useful |
| 23:28:11 | Bagder | Meekrab: i'd suggest you join the lyre.sf.net people |
| 23:28:50 | Meekrab | Hard drives have a longer running life |
| 23:28:57 | Bagder | really? |
| 23:29:01 | Bagder | what do you base that on? |
| 23:29:17 | Meekrab | Flash memory has limited writing cycles and is then pooched |
| 23:29:27 | Meekrab | hard drives can be popped out and replaced |
| 23:29:28 | Bagder | hahaha |
| 23:29:35 | Bagder | do you know how many cycles? |
| 23:29:37 | Meekrab | plus their cheap :) |
| 23:29:43 | Bagder | do you know about wear leveling? |
| 23:29:44 | simonrvn | they're |
| 23:29:51 | Meekrab | A few billion, with wear leveling |
| 23:30:00 | Jerom | an 1.5" hdd isn't cheap imho |
| 23:30:02 | Meekrab | 10 years of constant access on a regular basis |
| 23:30:14 | Meekrab | Who said anything about the little HDDs? |
| 23:30:23 | Meekrab | I was thinking SATA 2.5 HDDs :P |
| 23:30:38 | Bagder | hdds are in fact much more likely to fail |
| 23:30:51 | simonrvn | more mechanical things to break |
| 23:31:05 | Jerom | I broke my ipod 5.5G that way |
| 23:31:08 | AzureWurk | but unlike hdd when flash cells fail they just cant erase data most tims, hdds die they are pooched and so is everything on them :P |
| 23:31:21 | Jerom | Slipped off my hand... |
| 23:31:23 | Bagder | Meekrab: but yes, lots of people consider building their own. for a while. then they realize the problems and stop... :-) |
| 23:31:40 | Meekrab | I guess Compactflash would be a viable replacement |
| 23:31:45 | Meekrab | it still uses the ATA spec |
| 23:32:05 | AzureWurk | i would use a par o swapable micro ssd's |
| 23:32:06 | Meekrab | (sorry to be biased but I really love programming for ATA spec storage like SD and CF and regular HDD0 |
| 23:32:08 | ender` | it's quite easy to find CF->IDE adapters of all sizes |
| 23:32:20 | ender` | but there's also SD->IDE adapters |
| 23:32:32 | Meekrab | but why not build my machine around CF for main storage? |
| 23:32:47 | ender` | CF cards are huge |
| 23:32:51 | Bagder | because CF is old and buried? |
| 23:33:02 | AzureWurk | cf is a bit large for capacity |
| 23:33:33 | Meekrab | but it has better hardware control in my opinion |
| 23:33:48 | ender` | CF spec is still being extended (IIRC, the last addition added SATA support) |
| 23:34:17 | Meekrab | If I place a CF card on a zune, it takes up about 1/3 of its shapespace |
| 23:34:24 | | Quit dfkt (Quit: -= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.) |
| 23:34:42 | Meekrab | a 1.8HDD, it takes up 2/3 its shapespace |
| 23:35:15 | Meekrab | I'm basing my idea on two good facts |
| 23:35:23 | AzureWurk | a drive like asus eeepc's use is alot smaller |
| 23:35:35 | Bagder | well, pick whatever you want. Rockbox should be able to use it. good luck! |
| 23:35:55 | Meekrab | Average End users want to have a customised player that looks cool and does exactly what they want, like play games, doom etc |
| 23:36:36 | AzureWurk | http://www.newegg.com/Product/Product.aspx?Item=N82E16820183253 |
| 23:36:56 | Meekrab | the other guy, who knows about the hardware will know what to do and probably took it apart lots of times, and he knows when and where to get replacment parts, for the storage it could be as easy as taking a trip to wally world and getting an even bigger CF |
| 23:38:06 | Meekrab | and even though it is a Warranty nightmare, it'll be completely modular like a standard desktop PC |
| 23:38:15 | Meekrab | hence why I'd sell it as DIY |
| 23:39:01 | AzureWurk | imho after having replaced 4-8gb of the above for ppl they are what i would use for a pmp |
| 23:39:29 | Meekrab | and I absolutely refuse to encorporate ZIF or ribbon cables of any sort in my machine |
| 23:39:56 | AzureWurk | naa those just snap in like ram |
| 23:40:10 | Bagder | can we get back to Rockbox talk now? |
| 23:40:14 | | Quit komputes (Remote host closed the connection) |
| 23:40:24 | AzureWurk | question: is it normal for the I2SO clock on the fuze v2 to drp from 240mhz to 5mhz when playing ogg files? |
| 23:41:10 | CIA-8 | New commit by bertrik (r27035): Apply FS #11423 - Use udelay in AMS driver for FM radio I2C |
| 23:41:38 | | Join Milardo2 [0] (~Milardo2@c-24-5-128-36.hsd1.ca.comcast.net) |
| 23:42:57 | CIA-8 | r27035 build result: All green |
| 23:43:14 | | Quit anewuser (Quit: for SELL 2 by the price of 1 now!) |
| 23:44:21 | Milardo2 | hey does anybody know what information if any is needed for the meizu port? |
| 23:45:42 | AzureWurk | whats the addy to the revision page: |
| 23:48:06 | | Quit M3DLG (Ping timeout: 248 seconds) |
| 23:50:16 | S_a_i_n_t | revision page? |
| 23:50:52 | simonrvn | recent changes or whatever? |
| 23:52:28 | S_a_i_n_t | AzureWurk: I trust you recieved the custom Fuze V2 build I compiled for you? You may (or may not) notice it includes a few other patches, but there is nothing in there that is too drastic. |
| 23:52:43 | | Quit detaos (Remote host closed the connection) |
| 23:56:04 | | Join detaos [0] (~quassel@ip72-218-104-242.hr.hr.cox.net) |
| 23:57:51 | S_a_i_n_t | It uses saratoga's volume patch (revision #4), shutdown in menu, a prettier graphic for the battery discharge curve in Debug, ...and a few more I can't remember. Most of the patches in my tree are iPod specific so won't show up in the build I gave you, but a few will so I thought I'd mention it in case you think "Hey! Where did 'X' go?!?" the next time you update your build. |
| 23:58:09 | S_a_i_n_t | AzureSky: AzureWurk: ^^ |
icon identifies that the person is a core developer (has commit access).