Previous day | Jump to hour: 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20 21 22 23 | Next day

Seconds: Show Hide | Joins: Show Hide | View raw
Font: Serif Sans-Serif Monospace | Size: Small Medium Large

Click in the nick column to highlight everything a person has said.
The Logo icon identifies that the person is a core developer (has commit access).

Notice: Only Gecko based browsers prior to FF4 support the multipart/mixed "server push" method used by this log reader to auto-update. Since you do not appear to use such a browser, this page will simply show the current log, and not automatically update.

#rockbox log for 2009-07-17

00:01:36pixelmacabbiev2 (I also believe cabbiev3) already display albumart. I don't understand what you want to tell with the tags you quoted - did you add them again? Also, I think you typed %C1 (as the number) but it should be %Cl (lower case L) and needs parameters like position and width and height
00:06:11 Join BdN3504 [0] (n=5ce22714@gateway/web/cgi-irc/
00:07:07BdN3504how do i use the info presented in dmesg to mount a usb device in the vmware image?
00:08:38Robert777w/o editing Cabbie 2 or 3 it does not display the converted bmp's
00:08:52Robert777Ipod 60gig
00:09:09BdN3504aaah, sorry i think i got it myself it says scsi 2 so that
00:09:17BdN3504will be sdb right?
00:17:41kugelgevaerts: Zagor didn't change anything yet w.r.t to the weird build round times, right?
00:18:12 Quit Robert777 ()
00:18:14kugelI see 277s now which seems more reasonable than the 400+ which were before
00:18:24kugelalthough, there's 37 clients now, not exactly comparable
00:23:40gevaertskugel: no. Maybe this runaway speculative build detection had some side effects?
00:23:55 Quit DarkDefender ("Leaving")
00:26:11 Join stephen_ [0] (
00:26:44 Quit BdN3504 ("CGI:IRC (EOF)")
00:27:51pixelmaabout the cygwin sim build warnings which showed up in the build table too - they are there for quite some time already and it was mentioned here when they started and at a later time I guess too. You can also see them when you crosscompile windows sims under Linux
00:32:30kugelgevaerts: possibly
00:35:12 Quit bertrik ("De groeten")
00:37:47 Quit kugel (Remote closed the connection)
00:41:29 Quit stephen_ ("Leaving")
00:42:15CIA-69New commit by 03mt (r21909): Remove an overlooked, unused float from struct COOKContext.
00:42:16*roolku discovered another problem with the build system: the mrobe build faild, but is not indicated in red:;type=mrobe100">;type=mrobe100
00:42:25roolkumrobe100 that is
00:43:41 Quit evilnick_7 ("Page closed")
00:45:12roolkuinfact all builds that have the pegbox plugin have failed?
00:51:52 Join funman_ [0] (n=fun@rockbox/developer/funman)
00:52:32 Nick funman_ is now known as funman (n=fun@rockbox/developer/funman)
00:53:47shotofaddsgevaerts: I've updated the storage rework and D2 SD patches, if you could give them a quick once-over for sanity. "works here for me"
00:54:45 Part wincent ("Kopete 0.12.7 :")
00:54:50funmanshotofadds: where is the SD code for D2 ?
00:55:20shotofaddsfunman: fs#10415. it's very much based on the PP driver.
00:55:42CIA-69New commit by 03roolku (r21910): remove references to pegbox bitmaps that were removed in">r21907 (problem not detected by build system)
00:56:30 Quit flydutch ("/* empty */")
00:58:24funmanshotofadds: are the functions at the end of the new file needed ? (sd_sleepnow, sd_disk_is_active, sd_soft_reset, sd_spinup_time)
00:58:52***Saving seen data "./dancer.seen"
01:00:08shotofaddsfunman: they're needed for the storage rework (fs#9545). whether they are needed at all is a whole different discussion...
01:00:40shotofaddsi'd like to see a way of getting rid of the nonsensical ones (sd_spin et al)
01:00:51roolkuzagor/bagder: 2009-07-16 23:58:44 Server refused connection: error duplicate name! <−− on atlas-roolku half way through">r21910
01:00:53funmanagain void functions ? :/
01:01:31shotofaddsfeel free to add your thoughts to fs#9545 ;-)
01:01:54funmansd_sleep() and sd_spin() should be removed as well
01:02:44funmanperhaps "#define sd_sleep()" in storage.h
01:03:06funmanor sd.h (more appropriate since it is included by storage.h)
01:03:09 Join stephen_ [0] (
01:03:26shotofaddsthat could work, yes.
01:03:37shotofaddsbut right now I must sleep :/
01:04:17 Quit shotofadds ("Leaving")
01:07:44funmanobo: you have problems reading thumb code?
01:09:33CIA-69New commit by 03mt (r21911): Remove a call to av_clip() which limits the PCM output of the decoder to 16-bit.
01:16:28funmanwhat is the difference between #defined HAVE_DISK_STORAGE and #if (CONFIG_STORAGE & STORAGE_ATA) ? (semantically)
01:18:05funmanthe ipod nano has a ATA controller but no disk storage ..
01:18:15 Quit roolku ()
01:18:57 Quit Thundercloud (Remote closed the connection)
01:19:04pixelmathe former is only used be the ifp IIRC
01:19:28pixelmaor maybe I'm confusing things
01:19:43funmanit has a nand storage
01:20:24Unhelpfulamiconn: ok... now i can work on this. i assume that since code has to be word-aligned i should only be changing the size of the padding a word at a time?
01:27:48 Quit jfc (Read error: 104 (Connection reset by peer))
01:28:10 Join jfc [0] (
01:28:43 Quit jfc (Read error: 54 (Connection reset by peer))
01:29:04 Join jfc [0] (
01:29:34 Quit jfc (Read error: 54 (Connection reset by peer))
01:29:55 Join jfc [0] (
01:40:48 Quit jfc (Read error: 54 (Connection reset by peer))
01:40:55 Quit mt (Read error: 113 (No route to host))
01:41:06 Join CaptainKwel [0] (
01:41:15 Join jfc [0] (
01:47:38 Join JdGordon| [0] (n=Miranda@
01:49:54Unhelpfulok, i went back to the regular arm asm in libtremor, and left the pad where it was in window.c... without the pad the codec is broken, as it was before in long-call-stub builds. a 4, 8, or 12-byte pad fixes it. 16 is broken again, 20 works again... so there's a 1-in
01:51:18Unhelpfula 1-in-4 chance of building a broken codec by "accident", and changing out some of the libtremor asm just happens to "fix" it. i'll start moving the pad around now and see if i can determine what exactly is breaking
01:54:16Unhelpfulit doesn't like we force a greater-than-word alignment at the start of the data section, so it could easily be something there that breaks, as well
02:00:00 Part toffe82
02:00:14 Quit JdGordon| ("Miranda IM! Smaller, Faster, Easier.")
02:04:48 Quit jfc (Read error: 104 (Connection reset by peer))
02:05:11 Join jfc [0] (
02:06:04 Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.")
02:09:19CIA-69New commit by 03funman (r21912): Storage API : remove undeeded target-specific functions ...
02:11:52 Quit n17ikh (
02:11:52 Quit preglow (
02:11:52 Quit JdGordon (
02:11:52 Quit Bagder (
02:11:52 Quit daurn| (
02:11:52 Quit rasher (
02:11:52 Quit jfc (
02:11:52 Quit Sajber^ (
02:11:52 Quit tom243 (
02:11:52 Quit Torne (
02:11:52 Quit saratoga (
02:11:52 Quit J-23 (
02:11:52 Quit rphillips (
02:11:52 Quit Llorean (
02:11:52 Quit cdleonard (
02:11:52 Quit Zarggg_ (
02:11:52 Quit trisiak (
02:11:52 Quit tmzt (
02:11:52 Quit safetydan (
02:11:52 Quit Horscht (
02:11:52 Quit AndyI (
02:11:52 Quit gibbon_ (
02:11:52 Quit FOAD (
02:11:52 Quit dionoea (
02:11:52 Quit avacore^ (
02:11:52 Quit karma (
02:11:52 Quit vedlith (
02:11:52 Quit advcomp2019 (
02:11:52 Quit dz (
02:11:52 Quit krazykit (
02:11:52 Quit Kohlrabi_ (
02:11:52 Quit soap (
02:11:52 Quit Byan (
02:11:52 Quit Hadaka (
02:11:52 Quit rvvs89 (
02:11:52 Quit jordan` (
02:11:52 Quit obo (
02:11:52 Quit lostlogic (
02:11:52 Quit dys (
02:11:52 Quit fred_2 (
02:11:52 Quit r00s (
02:11:53 Quit jon-kha (
02:11:53 Quit Slasheri (
02:11:53 Quit rwong (
02:11:53 Quit parafin (
02:11:53 Quit bubsy (
02:11:53 Quit Erant (
02:13:45NJoinjfc [0] (
02:13:45NJoinsafetydan [0] (n=deverton@rockbox/developer/safetydan)
02:13:45NJoinSajber^ [0] (
02:13:45NJointom243 [0] (
02:13:45NJoinHorscht [0] (n=Horscht2@xbmc/user/horscht)
02:13:45NJoinn17ikh [0] (
02:13:45NJoinrasher [50] (n=rasher@rockbox/developer/rasher)
02:13:45NJoinpreglow [0] (
02:13:45NJoinJdGordon [0] (n=jonno@rockbox/developer/JdGordon)
02:13:45NJoindaurn| [0] (
02:13:45NJoinBagder [241] (n=daniel@rockbox/developer/bagder)
02:13:45NJoinTorne [0] (
02:13:45NJoinJ-23 [0] (
02:13:45NJoinAndyI [0] (i=AndyI@
02:13:45NJoincdleonard [0] (n=cdleonar@
02:13:45NJoingibbon_ [0] (
02:13:45NJoinrphillips [0] (
02:13:45NJoindys [0] (
02:13:45NJoinFOAD [0] (
02:13:45NJoinsaratoga [0] (i=9803c6dd@gateway/web/freenode/x-9b73980ae5d89280)
02:13:45 Join Llorean [0] (n=DarkkOne@rockbox/user/Llorean)
02:13:45NJoinavacore^ [0] (
02:13:45NJoinZarggg_ [0] (
02:13:45NJoindionoea [0] (
02:13:45 Join bubsy [0] (i=Bubsy@unaffiliated/bubsy)
02:13:45 Join obo [0] (n=obo@rockbox/developer/obo)
02:13:45NJoinkarma [0] (
02:13:45NJoinvedlith [0] (
02:13:45NJoinadvcomp2019 [0] (n=advcomp2@unaffiliated/advcomp2019)
02:13:45NJoindz [0] (
02:13:45NJoinkrazykit [0] (
02:13:45NJointmzt [0] (
02:13:45NJointrisiak [0] (
02:13:45NJoinKohlrabi_ [0] (
02:13:45NJoinjon-kha [0] (
02:13:45NJoinsoap [50] (n=soap@rockbox/staff/soap)
02:13:45NJoinByan [0] (
02:13:45NJoinjordan` [0] (i=gromit@
02:13:45NJoinrvvs89 [0] (n=ivo@pdpc/supporter/base/rvvs89)
02:13:45NJoinHadaka [0] (
02:13:45NJoinparafin [0] (
02:13:45NJoinfred_2 [0] (
02:13:45NJoinErant [0] (
02:13:45NJoinlostlogic [50] (n=lostlogi@rockbox/developer/lostlogic)
02:13:45NJoinr00s [0] (
02:13:45NJoinrwong [0] (
02:13:45NJoinSlasheri [0] (i=miipekk@rockbox/developer/Slasheri)
02:13:45funmanlostlogic: hi
02:13:45funmani wonder if you want to work on sound for yh920 when you don't own the target
02:15:09 Join JdGordon_ [0] (i=ae914296@gateway/web/freenode/x-1ac3b52e465f9ae4)
02:16:57Unhelpful*bizarre* - the next section linked forces 16-byte alignment in at the section start and end. so it pretty much has to be something in in .data that can break the codec if it lands on the wrong alignment... and the only thing in the vorbis codec's .data is ci.
02:20:03 Join evilnick_home [0] (
02:20:13obofunman: I find the 32 bit code easier to read - i.e. immediate values > 8bit, registers defined for each function rather than being passed from one to the next.
02:20:32JdGordon_how many codecs have subtrack/cuesheet support built in?
02:20:38JdGordon_is it just ogg/flac?
02:20:42JdGordon_and .cue obviously
02:20:46funmanobo: what do you mean by registers passed between functions ?
02:20:57Unhelpfulobo: technically immediate values in arm ore still only 8-bit... you just have the option of rotating them
02:21:11funmanJdGordon_: i don't know but recently i listened to a .ape with its .cue, i suppose cuesheet support isn't codec dependant since it has to be enabled ?
02:21:56funmanis there a list of the steering board on ?
02:21:57JdGordon_i dont mean rockboxwise... I know flac has inbuilt cuesheet support (we dont support it yet), and ogg has subtracks...
02:22:18funmanmp4 has chapters, but i don't know if it's used in audio
02:24:44UnhelpfulJdGordon: ogg subtracks will be quite tricky to support in any meaningful manner, if you mean ogg chaining
02:24:59JdGordon_I'm trying to decide what to do with cuesheet handling... possibly making it always enabled and still use the audio buffer for the listing instead of a static buffer
02:25:01*linuxstb slaps JdGordon_ for even thinking about chained oggs
02:25:03amiconnUnhelpful: So if everything after .data is cache line aligned, and the breakage is 1-in-4 in a cache line, the problem must be either in .text, .rodata or .data
02:25:25Unhelpfulthere's no global table of tracks and offsets, just <stream1headers><stream1data><stream2headers><stream2data>
02:25:37obofunman: in the bootloader it tended to define a hardware register in one function, and then pass it onto other functions as a paramater, adding, subtracting and shifting as it went. In the firmware it tends to doesn't seem to do that.
02:25:40JdGordon_that sounds pretty stupid :p
02:27:16JdGordon_AA does conversion and resize on load right? would doing the same with .cue files be bad?
02:28:09Unhelpfulamiconn: i'm less sure now... i put the pad in the linker script so that i could poke around for the section that has broken alignment, and that's how i found that the alignement fix worked in .data - which appears to be after .text and .rodata.
02:28:26*JdGordon_ is referring to 9789 for the interested...
02:28:54Unhelpfulthe next section *defined* is ncdata, which is unused and forces cache alignment, then iram, which of course can't have its alignment changed by non-iram padding.
02:30:43 Quit tom243 ()
02:34:31amiconnIRAM doubles as part of .bss after init
02:35:08Unhelpfulamiconn: right... but the stuff that belongs in iram is moved there, and .bss is cleared.
02:35:47amiconnYes, but in either process there might be an error, although this is unlikely since the start of these sections is cacheline aligned
02:35:48Unhelpfulanyway, moving the pad to the end of .data fixes the codec as well - so apparently the alignment problem is triggered by something in .bss?
02:38:28Unhelpful.bss is cacheline aligned?
02:39:24amiconnIt is due to the preceding .ncdata section
02:40:44Unhelpfulso how can padding the end of .data fix this, then?
02:41:10amiconnIt can't - something else must be changing :\
02:42:59Unhelpfulperhaps i'm doing it wrong... but all i did was add the . += 0x4 in this:
02:44:47 Quit jfc (Read error: 54 (Connection reset by peer))
02:44:55 Join AJzer [0] (n=6171b58d@gateway/web/cgi-irc/
02:45:08 Join jfc [0] (
02:46:14amiconnDid you compared the .map files for padding of 0/8/8/12 bytes?
02:46:28 Quit jfc (Read error: 54 (Connection reset by peer))
02:46:49 Join jfc [0] (
02:47:20 Quit jfc (Read error: 54 (Connection reset by peer))
02:47:20AJzerSo if rbutilqt autodetects ann Unsupported apple player variant, was I bilked?
02:47:41 Join jfc [0] (
02:48:25JdGordon_jfc: if you are going to have connection issues again, please leave this chan untill you sort it all out
02:48:44amiconnThere's no symbol that would be influenced by this... almost looks like a linker bug (??)
02:49:47 Quit jfc (Read error: 54 (Connection reset by peer))
02:50:09 Join jfc [0] (
02:50:34funmanAJzer: i'm not sure what "bilked" means, but if rbutilqt says it's unsupported, it's unsupported by rockbox
02:50:55AJzerIt means I asked the sales guy if it was 5th gen, and he said yes. And it seems it was not.
02:51:51AJzerI just want to be sure there's no room for error on the part of Rockbox
02:52:41 Nick n00b81 is now known as taylor_ (n=n00b81@unaffiliated/n00b81)
02:52:42Unhelpfulamiconn: that'd be a shame... perhaps i should attempt to reproduce with our standard toolchain? if it's not a problem with the linker itself, i should be able to find a "bad" alignment in <= 4 tries
02:53:08Unhelpfulfunman: "cheated"
02:53:18amiconnI'd like to see the .map files for the 4 cases, together with the info which of the 4 worked
02:53:55Unhelpfulamiconn: not a problem... i'll throw them on pastebin :)
02:54:07funmanAJzer: unlikely, but you can check
02:54:37Unhelpfuli've only actually run two of these, though... the determination that there was only one broken alignment in four was made when the padding was still in window.c
02:55:02AJzerYeah, it's vague enough to not clarify anything. But thanks for your help.
02:55:16 Quit webmind (Read error: 60 (Operation timed out))
02:56:05evilnick_homeAJzer: That link shouldn't be vague at all
02:57:07Unhelpfulamiconn: 0 padding: 4: 8: 12:
02:58:55***Saving seen data "./dancer.seen"
02:58:56Unhelpfulthe 0 padding should be identical to bulding with this toolchain with an unmodified mapfile, and is the case that was broken originally. files skip immediately without any error splash, so they may be quitting due to some error or the codec thinks its reached end-of-track prematurely.
02:58:58 Join stettberger_ [0] (
02:59:32 Join karma_ [0] (
03:00:45AJzerevilnick_home: There is no visible way to tell the difference between an Ipod Classic 80GB (6th gen), and the Ipod Video 80GB (5.5 gen). That is what I mean by vague.
03:02:04 Join toffe82 [0] (
03:02:13funmanisn't the case different ? (polished for classic)
03:02:51amiconnUnhelpful: It is a linker bug.... Compare the .iram load address in the broken case with that of the other 3
03:03:05 Quit stettberger (Read error: 104 (Connection reset by peer))
03:03:11amiconn...and with a build produced by our standard toolchain
03:03:13evilnick_homeI thought that the Classics all had metallic fronts
03:03:31AJzerif I have a classic, it's got a black matte front.
03:03:33linuxstbAJzer: That page says "You can distinguish the iPod classic from the iPod (5th generation) by the last three digits of the serial number. The iPod classic serial number's last three digits will be one of the following: Y5N, YMU, YMV, and YMX."
03:03:43Unhelpfulamiconn: it's not aligned!
03:04:08 Join webmind [0] (
03:04:17amiconnIn the 0-pad case it *should* be 0x01e96f20, because that's what 'iramcopy' is set to (in the preceding section, due to alignment
03:04:23JdGordon_they arnt digits!
03:04:46AJzerlinuxstb: thanks. I've read through that damn page numerous times and missed this.
03:04:47amiconnBut somehow the linker manages to ignore the alignment
03:05:03amiconnThe wonders of using dev snapshots :\
03:05:15linuxstbAJzer: But rbutil won't lie about that either...
03:06:05Unhelpfulamiconn: blast... the call stubs aren't generated in any released binutils, though. :/
03:06:28 Join BHSPitMonkey [0] (n=stephen@unaffiliated/bhspitmonkey)
03:06:38*amiconn wonders what the linker will do when padding by 16 bytes
03:07:44 Quit karma (Connection reset by peer)
03:08:12amiconnIt is in fact interesting that the code didn't crash. The .icode functions all end up shifted in iram
03:08:14Unhelpfulwhen the padding was in window.c, 16 came back around to being broken.
03:08:35AJzerlinuxstb: software can be particular, and imperfect. I felt it was worth checking before going through the process of returns.
03:08:48amiconnYeah, but what about .data? This seems to be a bug in section handling
03:09:08 Quit JdGordon_ (Ping timeout: 180 seconds)
03:11:30Unhelpfuli'm trying it now...
03:14:31 Quit AJzer ("CGI:IRC")
03:15:40 Join notlistening [0] (
03:16:32Unhelpfulthe address moves to the next aligned value (0x01e96f30) at pad = 12. bumping the padding to 16 changes it to 0x01e96f28
03:16:35notlisteningHi guys any deve;p[ment on the MicroSD cpu boost sisue I have seen the post on the forums. Does that still need testing?
03:16:47 Quit jfc (Read error: 104 (Connection reset by peer))
03:17:08notlisteningSansaAMS related
03:17:09 Join jfc [0] (
03:17:42 Quit jfc (Read error: 104 (Connection reset by peer))
03:18:03 Join jfc [0] (
03:18:17Unhelpfuli guess i'd better 1) see if a fresh snapshot has fixed this then 2) find out what the binutils folks want for a bug report
03:18:34 Quit jfc (Read error: 104 (Connection reset by peer))
03:18:34funmannotlistening: ping flyndice
03:18:55 Join jfc [0] (
03:19:46 Quit jfc (Read error: 104 (Connection reset by peer))
03:20:06notlisteningfunman, now i am being an irc idiot how do i ping?
03:20:07 Join jfc [0] (
03:20:12Unhelpfulit being fixed in a newer snapshot isn't exactly a giant help for us, though... the snapshots are numbered, but it looks like they just use one number for a while, and then change it. i believe this because the timestamp on their 2.19.51 snapshot is *days* later than when i built it. :/
03:20:23 Quit DataGhost (Read error: 110 (Connection timed out))
03:20:38 Quit jfc (Read error: 104 (Connection reset by peer))
03:20:59 Join jfc [0] (
03:21:17Unhelpfulnotlistening: in all likelihood he will notice that his name has been said... most clients will highlight the channel if your nick is used, and i usually search the logs for my nick if i've been gone a bit, and most other devs probably do as well
03:21:30 Quit jfc (Read error: 104 (Connection reset by peer))
03:21:51 Join jfc [0] (
03:22:06amiconnIt's clearly a bug in that 'iramcopy' is calculated as 0x01e96f20, and section .iram says 'AT ( iramcopy)', yet the linker uses a slightly different load address
03:22:22 Quit jfc (Read error: 104 (Connection reset by peer))
03:22:43 Join jfc [0] (
03:23:06Mode"#rockbox +o JdGordon " by ChanServ (ChanServ@services.)
03:23:14 Quit jfc (Read error: 104 (Connection reset by peer))
03:23:16funmannotlistening: just someting like "flyndice: ping" (when he'll be here, but i hope he reads the logs)
03:23:18Unhelpfulthe previous snapshot is from november, and there are naturally quite a few daily and weekly diffs since then.
03:23:35 Join jfc [0] (
03:23:47Mode"#rockbox +b *! " by JdGordon (n=jonno@rockbox/developer/JdGordon)
03:24:48Unhelpfulor "flyndice: <description>", or "see my prior report" or such if you've just said what the problem was. it irritates me a little bit when i come back and there's an "unhelpful: PING" and i can't find anything about what somebody thought i'd need to know about.
03:25:39Mode"#rockbox -o JdGordon " by ChanServ (ChanServ@services.)
03:26:58notlisteningthanks for the tip Unhelpful
03:34:09Mode"#rockbox +o JdGordon " by ChanServ (ChanServ@services.)
03:34:15Mode"#rockbox -b *! " by JdGordon (n=jonno@rockbox/developer/JdGordon)
03:34:17Mode"#rockbox -o JdGordon " by ChanServ (ChanServ@services.)
03:35:29 Quit Sajber^ (Read error: 54 (Connection reset by peer))
03:37:48 Quit notlistening ("Leaving")
03:38:59 Join Lss__ [0] (
03:39:31 Quit Riku (Read error: 54 (Connection reset by peer))
03:40:54soap2009-07-16 20:14:37 Server refused connection: error duplicate name!
03:40:54soapAddress the above issue(s), then restart!
03:41:12soapEST - FWIW. Is this a known issue with the new build client or is there something I am doing wrong?
03:41:30Mikachuyou're probably using a -name that is already used?
03:41:50Mikachumaybe an old connection that didn't time out yet
03:45:05 Quit funman ("free(random());")
03:46:26Unhelpfulamiconn: i would think if we find some non-release binutils revision that makes this work we'd need to add support to for checking out that revision, in order to use these features... since the snapshot version numbers don't appear to represent a specific revision.
03:48:30 Join Xerion_ [0] (
04:05:31 Quit Xerion (Read error: 110 (Connection timed out))
04:05:31 Nick Xerion_ is now known as Xerion (
04:07:01 Quit TheSeven (Nick collision from services.)
04:07:17 Join The_Seven [0] (
04:07:21 Nick The_Seven is now known as TheSeven (
04:18:39 Join dys` [0] (
04:30:59 Quit dys (Connection timed out)
04:36:52 Quit efyx_ (Remote closed the connection)
04:38:46 Quit martian67 (Read error: 60 (Operation timed out))
04:38:58 Quit taylor_ ("Leaving")
04:41:33 Join martian67 [0] (n=martian6@about/linux/regular/martian67)
04:51:53 Nick karma_ is now known as karma (
04:55:29Unhelpfulamiconn: hrm... that binutils snapshot works correctly without the -meabi=4 hack. i wonder if the problem exists when it's linking eabi objects build with a proper eabi toolchain? i don't see how that case would differ, unless gcc inserts some attributes that change things...
04:56:23 Join martian67_ [0] (n=martian6@about/linux/regular/martian67)
04:57:07 Quit martian67_ (SendQ exceeded)
04:59:00***Saving seen data "./dancer.seen"
05:02:57 Quit KBH (Read error: 104 (Connection reset by peer))
05:03:10 Join HBK [0] (
05:03:55 Quit martian67 (Success)
05:06:32 Join martian67_ [0] (n=martian6@about/linux/regular/martian67)
05:14:30 Join _lifeless [0] (n=lifeless@
05:31:34 Join __lifeless [0] (n=lifeless@
05:34:35 Join dash32 [0] (n=dash32@
05:36:23 Quit patmulchrone (Remote closed the connection)
05:38:26 Quit cdleonard (Read error: 110 (Connection timed out))
05:40:46 Quit Horscht ("Verlassend")
05:44:04CIA-69New commit by 03kkurbjun (r21913): M:Robe 500: Start of interrupt support.
05:48:32 Quit _lifeless (Read error: 113 (No route to host))
05:49:32Unhelpfulbah! yes, this binutils has this problem with a proper eabi gcc, also. i wonder what the problem really is, though? it seems just bizarre that it would correctly calculate the value of iramcopy, but then use an incorrect address when something else refers to that address.
05:59:53 Join RandAl [0] (n=chatzill@
06:00:04 Nick RandAl is now known as Rand_Althor (n=chatzill@
06:00:11Rand_AlthorOff-topic question: Anyone ever re-soldered the headphone jack on a c200?
06:00:59 Nick adi|away is now known as aditya (n=aditya@
06:14:35 Quit dash32 (Remote closed the connection)
06:18:16 Join goffa [0] (n=goffa@
06:19:12 Join dash32 [0] (n=dash32@
06:22:56 Quit dash32 (Remote closed the connection)
06:23:07 Quit Rand_Althor ("ChatZilla 0.9.85 [Firefox 3.5/20090624025744]")
06:29:37CIA-69New commit by 03kkurbjun (r21914): M:Robe 500: Fix simulator build and include some small changes for the 640x480 setup.
06:52:26 Join cdleonard [0] (n=cdleonar@
06:59:02***Saving seen data "./dancer.seen"
07:00:13 Join courtc_ [0] (n=court@unaffiliated/courtc)
07:02:26 Quit courtc (Read error: 111 (Connection refused))
07:08:12CIA-69New commit by 03kkurbjun (r21915): Jewels: Simplify support for new target screen sizes. Just adding new bitmaps for the target should now be enough.
07:10:44cdleonardI'm trying to look at the iAudio7 port but it fails to compile with 'unsupported instruction on ldrd'
07:11:03Unhelpfulis that *exactly* what it says? "on ldrd"?
07:12:30cdleonardno; not exactly. I'll give you the exact message
07:15:27cdleonard /home/cdleonard/files/work/rockbox/trunk/apps/recorder/jpeg_idct_arm.S:197: Error: selected processor does not support `ldrd">r4,.Lpool4'
07:15:38cdleonardand more like that
07:16:06 Quit CaptainKwel (Remote closed the connection)
07:17:03Unhelpfulthat's interesting. the iaudio7 should, to my knowledge, support that instruction...
07:17:09cdleonardhere are the full messages:
07:17:57cdleonardit seems iAudio7 sets -mcpu=arm9. Maybe that's a bit too general?
07:19:16cdleonardthe architecture numbers seem very confusing
07:20:09cdleonardit sets -mcpu=arm9e; sorry
07:23:02Unhelpfulto the best of my knowledge arm9e CPUs should support that instruction. you might try setting it to the more-specific arm946e-s, which i *think* is the core in the iaudio7's SoC. i had a similar problem when working on our development toolchain, with it failing to build libgcc for -mcpu=arm9e because the assembler claimed that ldrd was not a supported instruction.
07:26:13cdleonardthis page says so">
07:27:43cdleonardbut now arm-elf-as fails with 'unknown cpu `arm946e-s''
07:29:04cdleonardand if I set it to just 'arm946e' (also listed as a supported cpu in the manual) I get errors in SOURCES files
07:29:48cdleonardlike this:
07:30:08cdleonardI don't get it; why would an error be printed for one of those files?
07:32:08Unhelpfulthose files are still run through GCC to use its preprocessor. and arm946e is not listed as a valid -mcpu option here, while arm946e-s is
07:33:32Unhelpfulah! i see the issue... gcc knows "arm946e-s", but not "arm946e"... and for the assembler it's the other way around.
07:33:41cdleonardbut as doesn't recognize arm946e-s; but it does recognize arm946e
07:34:07cdleonardsomething about this patch:"> ?
07:36:01Unhelpfulno, that patch is not in any way your problem, it only specifies the flags used to build libgcc. the reason that the accepted CPUs don't match between as and gcc is likely due to differences between gcc and binutils versions... perhaps a newer binutils would help? i can't promise that will *work* though ;)
07:40:00 Join einhirn [0] (
07:41:35 Nick stettberger_ is now known as stettberger (
07:42:41cdleonardare you sure I need different versions? that patch seems to list mcpu options
07:42:56cdleonard(i have no idea what multilibs are about)
07:44:02 Part toffe82
07:44:04Unhelpfuli am sure that as from binutils-2.16 doesn't support the same cpu options with the same names as gcc-4.0.3.
07:44:12Unhelpfulthe patch is really not your problem.
07:49:09cdleonardbut this version mismatch has been there for quite a while; right?
07:51:36Unhelpfulright, but we don't normally build with -mcpu=arm946e-s anywhere. we use -mcpu=arm9e, which works fine, apparently, unless you try to use certain instructions - the file where you're having the problem is relatively new, and it's quite possible nobody's tried to build for iaudo7 since it was introduced.
07:58:19 Quit amiconn (Nick collision from services.)
07:58:20cdleonardit seems the mismatch is real
07:58:22 Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn)
07:58:40 Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn)
07:59:00 Quit pixelma (Nick collision from services.)
07:59:00 Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma)
07:59:18 Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma)
07:59:29Unhelpfulbinutils-2.17 support arm946e-s
08:01:28cdleonardyes; and it seems less risky than upgrading gcc
08:02:45Unhelpfuli would suspect you'd actually need to *downgrade* gcc, since it seems to be *newer* versions of binutils that support arm946e-s
08:04:39 Quit amiconn (Nick collision from services.)
08:04:42 Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn)
08:04:44 Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma)
08:04:44 Quit pixelma (Nick collision from services.)
08:05:00 Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn)
08:05:02 Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma)
08:05:40 Quit J-23 (Read error: 113 (No route to host))
08:11:55 Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma)
08:11:55 Quit pixelma (Nick collision from services.)
08:11:56 Quit amiconn (Nick collision from services.)
08:11:59 Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn)
08:12:11 Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma)
08:12:17 Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn)
08:24:19 Join stoffel [0] (
08:28:13amiconnUnhelpful: Such is the nature of bugs - things don't work as expected... Did you try a fresh snapshot already?
08:28:50amiconnIf we have a known-good non-release binutils, we could just tarball it and host it ourselves
08:30:26 Join Harryy [0] (n=Harry@botters/harryy)
08:30:56HarryyHow is Rockbox for Zune going? I read around in 2006 that it was very possible, did not see anything else after that point.
08:31:17 Quit safetydan ("Leaving.")
08:32:26Unhelpfulamiconn: i tried a fresh snapshot, no luck. we could also conceivably patch in the needed bits (about the new relocation types and stub generation) to a released binutils
08:33:00amiconnSomehow I thought that the latest release would already support it...
08:33:27*amiconn wonders how fast the binutils people are at fixing reported bugs
08:33:28Unhelpfulany idea about this -mcpu=arm9e issue? i can't find anything *besides* binutils that says it doesn't support the ldrd instruction.
08:34:02Unhelpfuli don't know... i've been trying to come up with a synthetic test case for the bug without any luck. :/
08:34:24amiconnImo we should update the required binutils for arm, and then use arm946e-s
08:35:14markunHarryy: actually it's not looking so good
08:35:18Unhelpfuleven the newest does this with -mcpu=arm9e - i had the same thing happen while building libgcc
08:35:56markunHarryy: Internally it's very similar to the Gigabeat S, but the exploit to run our code has been fixed in the Zune.
08:37:32cdleonardok; I tried to build with binutils 2.17 but it fails to link; something about FPA instructions
08:38:34Unhelpfuli have what certainly seem to be the same elements that trigger it in our case - a low and high segment of memory, a section with a target address different from its load address, which is calculated in the same fashion... but nothing goes wrong. :/
08:38:58cdleonardthis is with -mcpu=arm946e-s
08:39:08Harryymarkun: could someone not hard-flash it? why does it get insta-protected by M$?
08:39:32 Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma)
08:39:32 Quit pixelma (Nick collision from services.)
08:39:32 Quit amiconn (Nick collision from services.)
08:39:33 Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn)
08:39:47 Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma)
08:39:51 Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn)
08:40:09amiconnUnhelpful: Yes, since arm9e obviously doesn't support that instruction, but the jpeg decoder asm stuff uses it for armv5
08:40:14amiconnBut since our arm9e is actually an arm946e-s which supports it, going for the latter option would be a good thing
08:42:06Unhelpfulamiconn: it doesn't? the quick reference sheet notes the instruction as supported on armv5e processors, and the infocenter docs for arm9e-s include it as well - i can't find anything there for just "arm9e"
08:42:08Harryymarkun: do you think Zune will work in the near future? Or if I should rip apart my zune and use it as a HDD? right now it is laying around as I never use Linux
08:43:18UnhelpfulHarryy: i don't think there's been any real progress in quite some time
08:43:19HarryyI meant never use Winblows
08:43:54HarryyUnhelpful: bah. So I should call Microfail and tell them to port it to Linux? :3
08:44:00Harryy"or else!"?
08:44:24Unhelpfulgood luck.
08:44:54Unhelpfulthey never said it would work with linux, or that you could run your own code on it.
08:45:13Harryyhrm :|
08:45:19 Join Rob2223 [0] (n=Miranda@
08:45:21Harryyit doesn't even work under Wine
08:45:24Harryymoney hunry pigs
08:46:27Unhelpfulwine doesn't support USB devices at all, does it? other than using platform drivers for things like mass storage?
08:46:32HarryyI dunno
08:46:40Harryythe Zune.exe will not even install
08:46:55Harryywon't even launch the install GUI :\
08:47:37Harryyso, what do you suggest? I break my zune and use it as an external HDD?
08:48:00 Join flydutch [0] (
08:48:27 Join bertrik [0] (
08:51:16amiconnUnhelpful: I think the 'arm9e' option is meant to produce generic code which should run on all arm9xx. Iirc someone said that some eearly arm9 revisions don't suppor ldrd/strd
08:51:25cdleonardany thoughts about my link issue?
08:51:45cdleonardmaybe I need to build libgcc differently?
08:56:47 Part Harryy
08:58:09Unhelpfuldid you build your gcc and binutils together? i would recommend you use the script and change the binutils version there. you may also want to edit the patch and replace the arm9e with arm946e-s, so that you'll have a processor-specific libgcc as well
08:59:04***Saving seen data "./dancer.seen"
08:59:14cdleonardedit the multilib patch?
08:59:51Unhelpfulif you just edit the downloaded copy in-place, the script should use it without downloading it again.
09:00:06JdGordonhave we got any way to lock the audio buffer so calls to bufgetdata() are guarenteed safe? or are they fine as long as there is no thread switching during the call?
09:01:14cdleonardok; I'll try that
09:02:25 Join petur [50] (n=petur@rockbox/developer/petur)
09:03:02 Quit stephen_ ("Leaving")
09:03:11 Quit Rob2222 (Read error: 110 (Connection timed out))
09:03:20JdGordongot all you beer lovers
09:04:03*petur is more a Murphy's fan
09:05:03*petur wonders what happened to the buildclient on his NAS after midnight
09:06:31amiconnUnhelpful: Check The ARM946E-S">r0 is an ARMv5TExP which lacks ldrd/strd, the ARM946E-S r1p1 is a full ARMv5TE and supports ldrd/strd
09:06:45amiconnWeird - we'd have to distinguish revisions...
09:08:01Unhelpfulah... that's horrid. the quickref sheet simply reads "ARM v5E, and 6 and above" :/
09:08:59Unhelpfuli had actually wondered about the "5E" business, though - we don't have any way to distinguish it from "5", but most of the interesting ARMv5 instructions are 5E-only
09:09:06amiconnSame applies to ARM966E-S revions, btw
09:09:58amiconnThe ARM reference manual states "Version 5TE and above, excluding ARMv5TExP" for ldrd/strd
09:12:40Unhelpfulew. perhaps we need to define an ARM_FEATURES or such... do we actually have any ARM9 targets *without* ldrd and friends?
09:13:07amiconnI don't know - I don't have *any* arm9 target, only arm7 and arm11
09:13:40Unhelpfulit's the same here, only a beast and e200 :/
09:13:47amiconnSince the SoC manufacturers probably won't tell what revision of the arm core is used, this will probably need on-target testing
09:14:24Unhelpfulit seems like that would be important to know if you're to generate code for it :/
09:15:14amiconnWe do have arm9 targets without ldrd/strd, but those I know are clearly distinguishable since they're armv5, not armv5 (Gigabeat F/X is ARM920T)
09:15:39amiconnerr I mean armv4, not armv5
09:16:28amiconnARM architecture versions vs. arm versions is messy like that. Even revisions are important :\
09:18:07 Join KBH [0] (
09:19:23 Join Grahack [0] (
09:23:21cdleonardit compiled! I still got errors in apps/plugins/flipit and brickmania; but I also got a
09:23:36cdleonardI guess the plugin errors are normal for platforms that are not fully supported?
09:24:48Unhelpfulplugins often need adaptations to new platforms - they may select a bitmap based on display size, and not find the right one at all, for example.
09:25:31 Nick courtc_ is now known as courtc (n=court@unaffiliated/courtc)
09:25:56Unhelpfulugh... trying to bisect binutils history for the link bug is going to be a nightmre. they still use cvs!
09:26:19 Join HBK- [0] (
09:29:12 Quit KBH (Read error: 60 (Operation timed out))
09:30:59 Join mt [0] (n=mt@rockbox/developer/mt)
09:31:47 Join Thundercloud [0] (
09:34:11 Quit HBK (Read error: 110 (Connection timed out))
09:34:34amiconnUnhelpful: We could play safe and not use ldrd/strd on all armv5
09:37:13Unhelpfulamiconn: yes, but i don't think that all support smulxy and friends, either
09:37:22amiconnThey do
09:37:34amiconnCheck the reference manual
09:38:46amiconnIt's ldrd, strd, mcrr, mrrc and pld which aren't supported in the v5TExP variant
09:41:11Unhelpfulwe could have a flag of some sort for that variant. ldrd is a fair saving vs two ldr, isn't it?
09:42:22 Quit bertrik (Read error: 113 (No route to host))
09:43:36Unhelpfulany ideas how best to report this binutils problem? they have a bugzilla, but i don't as yet have a test case that doesn't involve all of rockbox :/
09:44:42amiconnOn armv5, ldrd just saves an instruction, it doesn't save cycles (ldr is 1-cycle w/o interlocks, ldrd is 2 cycles w/o interlocks)
09:45:20Unhelpfulah... so it's only of benefit to code size on armv5
09:45:25amiconnOn armv6 this is no longer true, but that's not a problem
09:46:06amiconnYeah. Code size *might* have a slight effect on speed though, because of the cache
09:46:50 Quit MrDuck (Read error: 113 (No route to host))
09:46:54Unhelpfulit's going to be quite a small difference, compared to the difference it makes on armv6
09:48:00Unhelpfulam i reading wrong, or does armv6 have an interlock on a shifted input register, where early versions do not?
09:48:47 Quit Thundercloud (Remote closed the connection)
09:49:13amiconnarmv6 does much more pipelining (even ldm/stm are single cycle), hence there are more interlocks
09:50:09amiconnVariable shift input means an extra cycle on armv5 and earlier. Not so on armv6, there it means having an "early reg"
09:53:20Unhelpfulthe shifted register seems to be an "early reg" on armv6 even if it's shift-by-immediate
09:53:54Unhelpfulper the example interlocks section for arm1136jf-s
09:56:11 Join robin0800 [0] (n=robin080@
09:56:24 Join KBH [0] (n=hbk@
09:58:22 Join HBK [0] (
10:14:19 Quit HBK- (Read error: 110 (Connection timed out))
10:14:44 Quit KBH (Read error: 110 (Connection timed out))
10:21:23 Join icefest [0] (
10:21:45 Quit icefest (Client Quit)
10:30:50 Quit BHSPitMonkey (Remote closed the connection)
10:35:18amiconnUnhelpful: Seems we have only two places in rockbox which try to use ldrd/strd on armv5 - the libdemac predictor and the jpeg idct
10:35:38amiconnThe former is an easy fix - I'd just have to change a preprocessor conditional
10:36:01amiconnDo you think this should be the way to go?
10:36:22amiconn(actually two preprocessor conditionals - one for ldrd and one for strd)
10:39:38 Join Sajber^ [0] (
10:42:04 Quit __lifeless (Read error: 110 (Connection timed out))
10:44:16 Join m67_l3 [0] (n=martian6@about/linux/regular/martian67)
10:45:04 Quit m67_l3 (SendQ exceeded)
10:45:17cdleonardI managed to upload stuff to my iAudio7 with tcctool but nothing shows up (only the buttons light up)
10:45:54cdleonardthe wiki says the lcd should work
10:45:56 Join m67_l3 [0] (n=martian6@about/linux/regular/martian67)
10:46:18linuxstbWhat "stuff" did you upload? I think only one person has ever worked on the iAudio7 port, and he doesn't seem to be around any more...
10:46:50cdleonardI first uploaded rockbox.bin directly
10:47:15cdleonardthen I used tools/scramble -tcc=crc and uploaded the result
10:47:17 Quit Sajber^ (Read error: 104 (Connection reset by peer))
10:47:22cdleonardnothing happened both times
10:47:51 Join Sajber^ [0] (
10:48:21linuxstbYou could try a bootloader build, and upload that.
10:48:38cdleonardfrom what I understand using tcctool loads a binary into RAM which is lost on a device reboot
10:49:02cdleonardso it's perfectly safe.
10:49:06*linuxstb wrote tcctool
10:50:27cdleonardI don't understand what the bootloader does; how is it different from a normal build? The normal build is what you're supposed to use on a perfectly supported device, right?"> vitja mentioned replacing stuff in the original firmware
10:51:56linuxstbA bootloader build is simpler, and is designed to load rockbox from the main firmware partition. In a new port, the bootloader is normally developed work, so it more likely to be working.
10:52:30cdleonardmaybe that's the only way the lcd works?
10:53:00cdleonard"In a new port, the bootloader is normally developed work, so it more likely to be working." what?
10:53:27gevaertstry the bootloader first. I'm not sure if the main binary is always linked at the right addresses to work with tcctool
10:53:46 Quit martian67_ (Success)
10:56:20cdleonardok; and the bootloader can also be uploaded using tcctool?
10:57:43linuxstbIt should be able to, yes. Unless it's being built to be appended to the original firmware and flashed.
10:58:23 Join DarkDefender [0] (
10:59:08***Saving seen data "./dancer.seen"
11:01:34Unhelpfulamiconn: the latter isn't hard, either, really... but there may be a few parts where ldrd is used in shared armv5/armv6 code, and those might need checks against ARM_ARCH added
11:01:54amiconnThese are the only two places
11:02:26cdleonardso I can either use {scramble -tcc=crc and tcctool} or {mktccboot and flashing}
11:02:40amiconnThere is only one more file that uses ldrd/strd at all, and that is the armv6 idct for libmpeg2. But as said, that is used for armv6 only
11:02:49amiconngrep told me...
11:04:15Unhelpfulgrep is such a helpful friend to have :D
11:07:04linuxstbcdleonard: I've just built an iaudio7 bootloader, and it looks like it's being built to be used with mktccboot. You need to remove the line "#define TCCBOOT" from firmware/config-iaudio7.h in order to build a standalone file you can test with tcctool
11:07:49 Join MrDuck [0] (
11:08:27cdleonardwait; can't I upload a mktccboot image with tcctool?
11:09:31linuxstbYes, you can probably do that as well.
11:09:48 Part karma
11:13:53Unhelpfulamiconn: they're in one file, but the bulk of each idct function for sizes 4 and up is split for armv4, armv5, and armv6. should i go ahead and commit?
11:17:59CIA-69New commit by 03amiconn (r21916): Don't use ldrd/strd on ARMv5 since not all revisions support them and the gain from using them is minimal (basically code size only).
11:21:01CIA-69New commit by 03unhelpful (r21917): Remove ldrd from ARMv5 JPEG IDCT, remove old debug code selecting ARMv5 code for one function even when building for ARMv6.
11:24:40cdleonardyes; mktccboot and then tcctool worked
11:25:11cdleonardI get the OF if hold is off and otherwise a white screen
11:25:53 Join _lifeless [0] (n=lifeless@
11:27:19Unhelpfulcdleonard: we *said* unsupported! ;)
11:27:53gevaertscdleonard: if you unpack the you got earlier on the device, and you then run the bootloader with tcctool, *and* you're lucky, you may end up with rockbox running!
11:29:19cdleonardyou mean the bootloader tries to load rockbox from the flash drive?
11:30:28cdleonardI thought it was just a smaller build with no plugins
11:31:16Unhelpfulno, that is not what the bootloader is. :)
11:32:04cdleonardthis should be in the wiki
11:33:45cdleonardso what is the point in using a bootloader and a .bin file on the flash drive instead of just uploading the whole .bin file with tcctool?
11:35:48Unhelpfulthe bootloader is, on many targets, installed in places that are less convenient to update. the main rockbox binary is usually installed on whatever visible storage the device offers, so that you can update it with the device in mass-storage mode, while the bootloader may be in a hidden partition, or some bit of flash, or whatever makes sense for the device in question.
11:39:13cdleonardok; so the iAudio7 has a mass-storage partition and some tiny private buffer to boot from. And tcctool replaces that private buffer with something else without overwriting it
11:39:51Unhelpful"replaces without overwriting"?
11:40:33cdleonardmakes the device boot from a file on the PC instead
11:40:43cdleonardignoring it's own private magic
11:46:58 Join efyx_ [0] (">
11:48:53cdleonardthose links are broken; where can I go to RTFM for tcc77x?
11:54:27linuxstbcdleonard: The "tiny buffer" is the 1MB (I think) of NOR flash. When the device boots, the code in NOR flash is what is executed first.
11:59:28linuxstbcdleonard: So you still haven't got anything sensible displayed on the LCD?
12:01:36linuxstbThen I would suggest trying older SVN revisions - from the time the lcd driver was first committed for the iaudio 7. Maybe something has got broken.
12:01:55cdleonardI didn't try putting the normal build on the drive
12:01:56linuxstbThe iAudio7 port is essentially unmaintained now,..
12:02:18cdleonardI guess I could also try to play with button leds in the bootloader; see something moving
12:02:35 Nick fxb__ is now known as fxb (
12:03:15linuxstbLooking at the bootloader code (bootloader/telechips.c), you should see the Rockbox logo displayed when the bootloader runs. So if that isn't showing, something is wrong.
12:03:19amiconnHmm, something in the new build system is borked
12:04:11amiconnOne of my clients definitely did some builds in the last two rounds, but isn't listed at all??
12:06:26 Quit robin0800 ("Leaving")
12:07:04amiconnHmm, actually only in one of those rounds. In the other, it didn't receive any build command, but the client still thinks it's properly connected
12:07:40amiconnHmmm, and the other client died with 'duplicate name' before those two round, but *is* listed???
12:08:01*amiconn summons Zagor
12:08:42amiconnAlso, disconnects (for whatever reason) while a build is running doesn't clean up
12:09:09 Join robin0800 [0] (n=robin080@
12:09:37 Quit robin0800 (Client Quit)
12:10:34 Join AndyIL [0] (i=AndyI@
12:13:05gevaertsamiconn: the latest builds also don't appear on">
12:15:37amiconnOh, hmm, that explains it
12:15:57amiconnThere's still the 'duplicate name' issue though
12:16:34*amiconn didn't compare revisions - another reason why timestamps on dev.cgi would be A Good Thing
12:19:29 Join robin0800 [0] (n=robin080@
12:21:27 Quit AndyI (Read error: 110 (Connection timed out))
12:24:07 Quit robin0800 ("Leaving")
12:24:26 Join robin0800 [0] (n=robin080@
12:26:08 Join mcuelenaere [0] (
12:26:43mcuelenaereLambdaCalculus37 (logs): have you seen">FS #10445?
12:32:33 Quit robin0800 ("Leaving")
12:32:51 Join robin0800 [0] (n=robin080@
12:39:18 Join _zic [0] (
12:40:26 Join dfkt [0] (i=dfkt@unaffiliated/dfkt)
12:54:03Grahackmcuelenaere: remember"> ? in fact we need this but rather if(!rb->file_exists(filename) && *mode == ('a'|'w'))
12:54:06Unhelpfulamiconn: if you want to add anything to it or follow it, we have a bug report in for binutils now:
12:59:11***Saving seen data "./dancer.seen"
13:01:21 Join mc2739 [0] (
13:02:03Unhelpfulamiconn: hrm, also, moving the first ". = ALIGN(16)" in .ncdata into .data after the contents seems to fix the weird offset bug.
13:02:53mc2739Bagder: Zagor: The current builds page looks like it is stuck on">r21884
13:03:12amiconnBut then it will introduce unnecessary padding on targets which don't use .ncdata
13:03:31mcuelenaereGrahack: hmm you're right
13:03:50mcuelenaereyou can't do && *mode == ('a' | 'w') in C though ;)
13:04:36Unhelpfulamiconn: not saying that's a fix we should use, just documenting the behavior in case it means anything to somebody else :)
13:05:36Grahackmcuelenaere: it compiled but didn't work, maybe if(!rb->file_exists(filename) && (*mode == 'a' || *mode == 'w')) ?
13:06:06mcuelenaereGrahack: yes it compiles, but it doesn't do the intended behaviour; *mode == 'a' || *mode == 'w' will work indeed
13:07:46Torne('a'|'w') == 'w', in fact
13:08:09Torneby fluke
13:09:55CIA-69New commit by 03mcuelenaere (r21918): Lua IOlib: when opening files for writing/appending, check if they exist and if not, add O_CREAT.
13:10:13tmztTorne: binary?
13:10:34tmztI mean bitwise
13:10:41Grahackok, thanks for the C lesson guys. You see how I'm happy this Lua plugin exists ! and what is "by fluke" ?
13:12:21mcuelenaereGrahack: ;)
13:14:21 Join mc2739_ [0] (
13:14:49 Quit mc2739 (Nick collision from services.)
13:14:51 Nick mc2739_ is now known as mc2739 (
13:17:54 Join __lifeless [0] (n=lifeless@
13:24:05peturah, buildsystem is working again...
13:24:54peturbut"> still has old binaries
13:25:37peturerm.. buildsystem not ok, still missing 2 builds
13:26:24 Quit dmb (Read error: 113 (No route to host))
13:34:46 Quit _lifeless (Read error: 101 (Network is unreachable))
13:35:00 Join ucchan [0] (
13:37:00 Join mc2739_ [0] (
13:37:16 Quit mc2739 (Nick collision from services.)
13:37:18 Nick mc2739_ is now known as mc2739 (
13:45:02TorneGrahack: "by chance"
13:45:06Tornethough it's not chance, really
13:45:11Torneit's just bitwise or
13:49:25 Join junker [0] (
13:54:19tmztthat applies to all ascii character on certain boundries right?
13:56:20Unhelpfultmzt: if the set bits in one are a subset of those in the other
13:56:46tmztright, okay. sorry for the offtopic
13:56:58tmztthat would be a hard bug to track down though
13:58:34 Quit mc2739 (Read error: 110 (Connection timed out))
14:01:09 Quit aditya (Read error: 110 (Connection timed out))
14:01:17 Quit DarkDefender ("Leaving")
14:01:56 Join aditya [0] (n=aditya@
14:05:39 Join dfkt_ [0] (
14:05:52 Join _lifeless [0] (n=lifeless@
14:06:02 Quit junker (Read error: 110 (Connection timed out))
14:07:58 Quit dfkt_ (Client Quit)
14:08:49 Quit dfkt (Read error: 104 (Connection reset by peer))
14:09:56 Join MarcGuay [0] (
14:10:23 Quit MarcGuay (Client Quit)
14:10:54 Quit _zic ("Ухожу")
14:20:10 Quit robin0800 ("Leaving")
14:22:13 Quit __lifeless (Read error: 110 (Connection timed out))
14:22:57 Join LambdaCalculus37 [0] (i=44a0430d@rockbox/staff/LambdaCalculus37)
14:23:07 Join robin0800 [0] (n=robin080@
14:30:25GodEateris the build system supposed to remember scores for "come and go" clients that it's seen before ?
14:31:37 Join dash32 [0] (n=dash32@
14:31:41 Join ReKleSS [0] (n=ReKleSS@
14:35:20 Join dfkt [0] (i=dfkt@unaffiliated/dfkt)
14:36:59GodEatermy laptop keeps being told it has a score of zero the first time I bring it back into the build system
14:37:37ReKleSScould I get write permission on the wiki? (name is JeremyChin)
14:37:43ReKleSSI'd like to put up my H120 reflashing thing
14:37:57ReKleSSI sort of doubt anybody's actually going to attempt it, though...
14:38:42GodEaterReKleSS: done
14:38:54GodEateryou're welcome
14:40:05*GodEater prods linuxstb to update IPodNano2GPort
14:40:26linuxstbGodEater: Do I have to do _everything_.... ;)
14:40:43GodEaterno, but you understand most of what's been done better than me :)
14:41:05GodEaterI imagine you can probably copy and paste most of your forum post anyway
14:44:21 Join wincent [0] (
14:48:03linuxstbGodEater: OK, I'll have a look later. I think I've just figured out how to properly control my type of LCD, so things are looking good.
14:51:36 Join itcheg [0] (i=4117734b@gateway/web/freenode/x-d7d4b04603c63a02)
14:53:57 Join faemir [0] (n=faemir@
14:54:10GodEaterexcellent news
14:54:25*GodEater wishes we had a message forwarding bot on linux4nano to send dev news this way
14:54:37TorneGodEater: isn't it called GodEater?
14:54:55GodEaternot in the last few days it hasn't been
14:55:00GodEaterI've been snowed at work :(
14:56:27 Quit stoffel (Read error: 113 (No route to host))
14:59:12***Saving seen data "./dancer.seen"
14:59:38 Join GreatBeaver [0] (
15:00:11GreatBeaveri wrote a guide on using the mk8025gal in the iriver h120, if someone wants he can post it on the rockbox forum talking about the h120
15:00:37 Quit gevaerts (Nick collision from services.)
15:00:47 Join gevaerts [0] (n=fg@rockbox/developer/gevaerts)
15:01:41 Quit antil33t (Read error: 104 (Connection reset by peer))
15:01:55 Join antil33t [0] (n=Mudkips@
15:02:11 Quit obo (Read error: 110 (Connection timed out))
15:05:34GodEaterGreatBeaver: we encourage people to post static content like guides to our wiki, not the forums.
15:06:58 Join _zic [0] (
15:07:39 Quit _zic (Client Quit)
15:07:55 Join _zic [0] (
15:07:58linuxstbMaybe we just want a link from the HardDriveReplacement page to that forum thread?
15:14:05 Quit mt (Read error: 113 (No route to host))
15:16:29 Quit Sajber^ (Read error: 104 (Connection reset by peer))
15:16:29 Quit robin0800 ("Leaving")
15:24:23 Quit _zic (Remote closed the connection)
15:27:17*linuxstb wonders about the capital P in IPodNano2GPort
15:29:19 Join DarkDefender [0] (
15:30:53 Quit ucchan (Read error: 104 (Connection reset by peer))
15:31:27 Join robin0800 [0] (n=robin080@
15:32:17 Join evilnick [0] (i=0c140464@gateway/web/freenode/x-bdfd7fa976ec2ece)
15:35:30LambdaCalculus37linuxstb: Sorry, that was my mistake. :)
15:35:45*linuxstb glares at LambdaCalculus37 ;)
15:38:10linuxstbLambdaCalculus37: Tried anything on your Nano yet?
15:40:29LambdaCalculus37linuxstb: I got a logo on mine. :)
15:41:18linuxstbWas that before or after my change to support the second LCD type?
15:41:41LambdaCalculus37Before, it didn't work.
15:41:43GreatBeaverlinuxstb: could you do that for me please/
15:42:04GreatBeaveri know i had a lot of hard time finding info on harddrives for the H120, i think if its on wikia it would help a lot of people
15:42:28GodEaterWikia ?
15:43:04GreatBeaverrockbox's wiki?
15:43:08GreatBeaverdoes rockbox have a wiki?
15:43:28GodEateryes, but it's not called wikia :)
15:44:22GreatBeaverwhat is it called?
15:44:25gevaertsthe wiki
15:44:53cdleonardI managed to get my iAudio7 blinking whenever buttons are pressed; but lcd doesn't work
15:45:11GreatBeaveri cant find it
15:45:11cdleonardat least I know it doesn't hang or crash during init
15:45:25linuxstbcdleonard: Not even with older versions of Rockbox?
15:46:14cdleonarddidn't try that
15:46:18GreatBeavercan someone link the rockbox wiki?
15:46:22GreatBeaveri keep searching for it but cant find
15:47:17cdleonardfrom"> I guess I should try">r18435"> then click on "wiki" in the left menu of every page
15:48:47GreatBeaveri dont think my guide fits anywhere there
15:51:20 Join n00b81 [0] (n=n00b81@unaffiliated/n00b81)
15:53:52 Quit LambdaCalculus37 ()
15:54:42 Quit MrDuck (Connection timed out)
15:54:44*GodEater is amazed GreatBeaver managed to read the entire rockbox wiki in 1 and hald minutes
15:57:36 Join funman [0] (n=fun@rockbox/developer/funman)
16:03:27 Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.")
16:04:14 Join Ypsy [0] (
16:04:24YpsyHi there
16:04:45YpsyAny Sansa Fuze users here?
16:04:55linuxstbYpsy: You should just ask your question.
16:05:35YpsyWell, I cba to read through all that 24 pages about rockbox on the fuze and just wanted to ask if there is a beta version of rockbox for it :P
16:06:02linuxstbLook in the "testing builds" forum
16:06:04funmanthere is a test build which deadlocks and corrupt the microsd content
16:06:13linuxstbAnd you'll find that....
16:06:21*linuxstb doesn't think funman is selling it very well...
16:07:22GodEaterhe's just being honest
16:07:29GodEaterhe's clearly not cut out for a salesman job
16:07:40YpsyDont have a microsd yet so whatever :P
16:07:59GodEaterI imagine you'll enjoy just the deadlocks then
16:08:34 Join DerPapst [0] (
16:08:43 Join stripwax [0] (
16:08:46funmani'm not sure deadlocks (well, now they are panics) happen when using the internal storage
16:08:58 Join kugel [0] (n=kugel@rockbox/developer/kugel)
16:09:09GodEaterfunman: so it'll be an exciting journey for him then!
16:09:45 Part Grahack
16:09:52funmana forum user will send me his problematic microsd card, i'll see what i can do with it
16:12:41*mcuelenaere just permanently flashed Rockbox to his VX747 :)
16:16:57 Quit efyx_ (Remote closed the connection)
16:17:52cdleonardlinuxstb: I tried">r18435 and">r19597 (another vitje commit) and got the same result: white screen on startup
16:18:15cdleonardbacklight can be controlled but lcd doesn't seem to do anything useful
16:18:46linuxstbcdleonard: Is your iaudio 7 relatively new? It's not uncommon for manufacturers to change LCD types
16:19:45cdleonardit's a 16gb model; and it's reasonably new
16:19:59 Join jgarvey [0] (
16:20:04cdleonardback says CWS-iAUDIO-7(B); note the (B)
16:20:12YpsyIs there a stable of the fuze version in sight?
16:20:27 Join Zagor [242] (n=bjst@rockbox/developer/Zagor)
16:21:31 Join Horscht [0] (n=Horscht2@xbmc/user/horscht)
16:22:14GodEaterYpsy: maybe, but who can say ?
16:22:42GodEaterif the people working on it knew what to fix to get a stable one, they'd have fixed it already right ?
16:23:02GodEaterbut since they don't, they don't know how long it'll take to work out what the problems are.
16:23:14GodEaterso it's impossible to guess when a stable build might turn up
16:23:26Ypsyokies, but they're still working on it, are they?
16:23:38GodEaterfunman is working like a slave :)
16:23:48Ypsyhaha :) good to hear ;)
16:23:50cdleonardI used mktccboot with OF v1.18; while the original developer probably used 1.17; could that be a problem?
16:24:24cdleonardthis new version is almost twice as big; which seems strange:
16:25:04linuxstbcdleonard: Yes, it's worth trying older OFs as well.
16:25:40linuxstbAlthough maybe not... Our code should be run before the OF, so that should affect things.
16:26:03 Join aaron424 [0] (
16:26:27*GodEater hands linuxstb a extra "not"
16:27:13linuxstbGodEater: Thankyou.
16:27:37GodEaterany time
16:27:40GodEaterI'm here to help
16:28:27cdleonardother than that I guess I have to disassemble the OF; right?
16:29:24linuxstbcdleonard: Yes. Although it would be good to find someone who can get Rockbox working on their iAudio 7. As I said, I think vitja was the only person who ever did any work on it.
16:29:44linuxstbSo the code in SVN may not even be working...
16:30:45CIA-69New commit by 03mcuelenaere (r21919): Onda VX747: add dual-boot capability + make it possible to permanently 'stick' Rockbox to your DAP
16:31:25cdleonardbuttons and lights do work; so at least some code can run inside the device
16:31:28*GodEater notices his load average starting to climb
16:31:42GodEatercan I retire the old build client's directory now by the way ?
16:31:59*GodEater noticed no follow up email from Zagor to say the switch over had officially occured
16:32:13ZagorGodEater: yes you can
16:32:19GodEaterZagor: thanks
16:32:55 Quit funman ("free(random());")
16:32:56amiconnZagor: Any idea on the duplicate name issue? It still happens occasionally. Bad if you want to leave the client running unattended...
16:33:06 Join __lifeless [0] (n=lifeless@
16:33:25Zagoramiconn: do you have a time when it occurred last?
16:33:33amiconnOh, and the client doesn't clean up if the connection gets lost (for whatever reason) while a build is running
16:33:42gevaertsah yes, during last night all my clients got disconnected with duplicate-name issues
16:34:03GodEatermine too
16:34:10amiconnZagor: Today, 07:11:23 CEST on jupiter
16:34:21GodEater2009-07-17 06:13:48 Server refused connection: error duplicate name!
16:34:34GodEateron atlas
16:34:36 Join toffe82 [0] (n=chatzill@
16:34:53amiconnGodEater: Real GMT or GMT+1 for summertime?
16:35:04Zagorthe good news is I've found and fixed the ping problems
16:35:15GodEaterprolly GMT+1
16:35:22GodEaterhe guessed wildly
16:35:26amiconnThat's almost the same time as on mine, then
16:35:34amiconnDo you sync from NTP?
16:36:06*GodEater goes to check
16:36:38GodEaterdoesn't look like it
16:36:42*GodEater fixes
16:37:02amiconnSo the difference could be just clock drift on your side, and it happened at the same time
16:37:29GodEatervery likely
16:37:56toffe82I have a problem on my gigabeat X60, 2 times this week, the music stop (not the same) and I can't do anything, only have the lcd on and off ??
16:37:57GodEaterI have 15:37 here currently
16:38:07 Join mc2739 [0] (
16:38:21kugelGMT then
16:38:23toffe82the first time I had also a message , cannot read the playlist or something like this
16:38:37kugelGMT+1 (no summer time)
16:38:48gevaertsMy duplicates are: hal 00:45:50, monster 02:10:55, rb1 06:32:06, rb2 05:46:51, rb3 02:13:31, rb4 02:13:31, all GMT+2
16:39:02mc2739Zagor: The current builds page looks like it has not updated since">r21884
16:39:04kugeldoes GMT include summertime :?
16:39:37Zagormc2739: oh right, I'll fix that
16:39:50GodEaterkugel: it's technically BST
16:39:55GodEater"British Summer Time"
16:39:57toffe82my question is, why the power button doesn't work and pressing a button switch the lcd on and off ?
16:40:29kugelI guess we should refer to UTC instead?
16:43:00gevaertskugel: is your client gone?
16:43:25 Quit _lifeless (Read error: 110 (Connection timed out))
16:43:31kugelit's taken off, until late august
16:46:15kugelhaha, now other clients have the chance to feel how it is to have 100k points :)
16:47:55gevaertsthat would have happened anyway :) I decided to go back to a single client, and that machine had around 150k before, split over four clients
16:48:34 Join J-23_ [0] (
16:49:17Zagoramiconn: I'm not sure what we can do. the server didn't get the disconnection of your client until a full 10 seconds after the duplicate name
16:49:36 Nick J-23_ is now known as J-23 (
16:50:10Zagorthe only thing I can suggest is that we instead of exit 22 waits maybe 10 seconds and retry
16:50:37mcuelenaerehmm could the VX747 bootloader red be a false alarm?
16:51:19Zagormcuelenaere: the build log says "Build Status: Failed" so it definitely is something wrong. exactly what is unclear to me though,
16:51:36mcuelenaereZagor: perhaps the 'mipsel-elf-gcc: no input files ' line?
16:51:52Zagorfull unparsed log:">
16:52:07Zagormcuelenaere: probably
16:53:11Zagoractually, no. the "failed" is written by rbclient if the specified output file isn't produced
16:53:38mcuelenaereah, how does it determine the specified output?
16:53:38kugelgevaerts: why did youß
16:53:43mcuelenaere(I renamed the bootloader to ccpmp.bin)
16:53:55Zagormcuelenaere: it's listed in the "builds" file. in this case it is "rockboot.vx747"
16:54:05kugelgevaerts: what 150k do you mean (my machine had a rating of 230k)
16:54:40Zagormcuelenaere: if the rename is permanent, please change the builds file too
16:55:08mcuelenaereZagor: www/buildserver/builds ?
16:55:15Zagormcuelenaere: yes
16:56:49CIA-69New commit by 03mcuelenaere (r21920): Update builds file with new VX747 bootloader filename
16:56:58 Quit ReKleSS ("Leaving")
16:59:14***Saving seen data "./dancer.seen"
16:59:46 Quit Xerion (" ")
17:01:05amiconnZagor: Maybe double the retry interval every time? This way a real duplcate wouldn't cause too much server load
17:01:58amiconnHmm, or maybe exit if the client gets a "duplcate name" message on startup, but not on retry
17:02:32ZagorEvery 10 seconds won't cause any measurable server load either.
17:03:24ZagorI'm adding logging to the next client update, to easier see what the client is doing
17:04:20 Join mnk200 [0] (
17:05:55 Join Xerion [0] (
17:07:02 Join dfkt [0] (i=dfkt@unaffiliated/dfkt)
17:07:21gevaertskugel: that 230k was before rounds got faster
17:08:33kugelgevaerts: I don't mean the score on the client page
17:09:00kugelI mean the "230000 puts you in the fast category"
17:09:28gevaertskugel: yes, but that's the average score
17:11:13CIA-69New commit by 03zagor (r21921): Added logging. Added output file missing message. Added sleep+retry instead of exit 22 on HELLO failure (to cure duplicate name exits).
17:14:22gevaertskugel: don't worry. I also have no idea what we're actually talking about :)
17:15:55kugelgevaerts: so which 150k did you mean?
17:16:11gevaertskugel: the sum of rb*
17:16:59 Join roolku [0] (
17:18:05roolkuZagor: did you see my coment from last night: the build system didn't spot the failed builds when images are missing such as:;type=mrobe100">;type=mrobe100
17:19:08Zagorroolku: thanks, i'll look at tat
17:19:10roolkuZagor: also lots of clients failed because of the duplicate name issue (4 of mine and I others commented as well) - you can see the decline in numbers in the build table
17:19:27Zagoryes last nights server wasn't my best :)
17:19:34kugelgevaerts: I don't think they'll just sum up
17:19:53gevaertskugel: no, but they would have 100k even without your client stopping
17:20:00kugelsure ;)
17:20:05kugelblah blah :P
17:20:11CIA-69New commit by 03zagor (r21922): Bugfixed socket data reading. ...
17:20:42kugelWhy are you switching to single client? Didn't you "prove" yourself that multiple clients are better?
17:22:35 Join rockbox [0] (n=anilbpai@
17:22:53rockboxnewbie here
17:23:01GodEateroriginal name though
17:23:28 Quit petur ("beer time!")
17:23:32rockboxhow can i get started with ?
17:23:55GodEaterwhere "?" is what exactly?
17:25:18markunroolku: what are you trying to do?
17:25:34markuneh, rockbox ..
17:25:55markunroolku: sorry :)
17:36:39 Quit mc2739 ("ChatZilla 0.9.85 [Firefox 3.0.11/2009060215]")
17:36:42 Quit rockbox (Read error: 60 (Operation timed out))
17:37:53 Quit Zagor ("Leaving")
17:38:12 Join BryanJacobs [0] (
17:42:04kugelgevaerts: do you have an opinion on the dev-ml discussion about funman's commit? It's your code...
17:44:52gevaertsnot really, except that I'm not convinced that those functions belong in the plugin api
17:45:23gevaertsspinnung up a disk because you're going to read from an sd card soon is not optimal
17:45:59GodEateresp. if there's no disk there
17:47:18gevaertsthat's the better case, as it will just be a stub. The problem is that those plugin api things are based on the assumption that there's only one storage device
17:48:49amiconnata_spin() is not for spinning up a disk - that's done automatically. It is for keeping the disk spinning when the code knows the next access will happen very soon
17:49:19saratogalinuxstb: about DSP_SET_SAMPLE_DEPTH, do you happen to remember how that value is calculated?
17:49:28 Quit nls (Read error: 110 (Connection timed out))
17:49:32 Join funman [0] (n=fun@rockbox/developer/funman)
17:49:49amiconnAnd ata_sleep() is for quick sleep, without waiting for the timeout. Both make very much sense for plugins as well (mpegplayer, video, ...), and having an empty function for them for non-hdd is safe
17:49:50funmanwhat about disk_spin() ? (more explicit on its usage)
17:50:18gevaertsamiconn: sure, but what about a player with both a hard drive and some flash thing?
17:50:25amiconnWhat about it?
17:50:39gevaertswhat do the functions do?
17:51:09amiconnThe same as they do on single storage, I'd expect
17:51:18funmanin case of multi driver targets, only the internal storage would be a disk, we could separate ata_spin/ata_sleep from storage driver
17:51:35gevaertsfunman: not sure :) How about USB?
17:51:57amiconngevaerts: Can you control usb hdd spinup/spindown from the host?
17:52:12funmangevaerts: what's the link with USB ?
17:52:52gevaertsamiconn: not sure, but I'd expect so
17:53:20gevaertsfunman: usb host. More disks
17:53:41funmanI still don't get it, sorry
17:54:02gevaerts"only the internal storage would be a disk" is not true for those
17:54:46gevaertsamiconn: my point is that those functions apply to the drive having the files the plugin is working with. The way they are defined, the core doesn't know which drive this is
17:54:50funmanI meant the removable storage can not be a hard disk
17:56:13 Join CaptainKwel [0] (i=2669ecc2@gateway/web/freenode/x-7b57b702622b978a)
17:56:20amiconnNot atm, no. But I'd go KISS first, and just apply it to all drives. We can still add differentiation later
17:59:04funmanSo if I want to separate ata_sleep/ata_spin from storage driver, there should be a way (in the future) to know (dynamically) if said storage uses a disk or not
18:01:23amiconnNo, the purpose of the multi-storage layer is that the app doesn't need to know that
18:02:43gevaertsfunman: what's the point of separating it?
18:03:17amiconnWhat could be added later is a way to track which storage the app uses, and only apply *_spin() to that one
18:03:21funmani have explained that in my mail to rockbox-dev i think
18:03:46gevaertsfunman: not in a way that makes it clear to me
18:03:48funmanseparate unrelated code, because i think physical storage type is not related to the storage driver (ata/nand/sd/mmc)
18:04:03 Quit dash32 ("Verlassend")
18:04:19amiconnIn theory, yes (not counting cf mods)
18:04:44amiconnIt doesn't hurt to use ata_spin() and ata_sleep() on a cf - it just doesn't change anything
18:05:13funmanit annoys me when i read the code
18:06:01gevaertsfunman: so you want to basically rename storage_spin() to disk_spin()?
18:06:03funmanwhat is bad with making this code really dependant on HAVE_DISK_STORAGE ? (which is the only time when it has an effect)
18:06:17funmangevaerts: yes
18:06:23linuxstbsaratoga: Sorry, I don't. preglow might...
18:06:33amiconnfunman: More ifdefs for nothing
18:06:34gevaertsfunman: the problem with the disk_*() names I see is that it (IMHO) really doesn
18:06:55gevaertst belong in disk.c, while the names would imply that. I don't know how important this si
18:06:56linuxstbsaratoga: Or you could try searching the IRC logs - I'm sure it's been discussed here
18:06:57funmanamiconn: clear separation IMO
18:07:24amiconndisk_ is entirely different module, related to handling partitions
18:07:26gevaertsfunman: HAVE_DISK_STORAGE means that there is at least one disk, not that the driver you're looking at deals with disks
18:07:37funmangevaerts: well disk.c is wrongly named as well, i didn't know where to put these new disk_spin functions
18:07:53gevaertsfunman: spindle_spin()? :)
18:08:08amiconnAnd storage_ abstracts from the storage types. So having *_spin() and *_sleep() for all drivers actually provides better abstraction
18:08:25gevaertsAnyway, I think they do belong in storage.c, because they make use of the storage.c indirection layer
18:08:57funmanwhat about keeping this discussion on the ML where it started?
18:09:17funmani'm still not convinced, and we could better track each other arguments on the ML
18:10:18*gevaerts would like to see the stubs eliminated, but when he tried it turned out to be difficult
18:10:47amiconnWell you need those stubs anyway for the plugin api
18:11:01gevaertsthat was the sticking point, yes
18:11:03funmanonly when they are needed at all?
18:11:32*amiconn wonders why funman doesn't understand the advantage of less ifdefing
18:11:46gevaertsfunman: so you'd add #ifdef HAVE_DISK_STORAGE around the stubs?
18:12:04funmanamiconn: i do understand them, do you understand the advantage of less stubs and code separation ?
18:12:14amiconnWhy should the app care what storage is in use? It tells the lower layers whether it's okay to send the storage to sleep, or whether it wants it to stay quickly accessible
18:12:15funmangevaerts: no, i already did
18:12:27amiconnThis *is* code separation
18:12:46funmanthe storage, or the disk ?
18:12:57gevaertshow would the app know?
18:13:03amiconnWhether the storage layer actually needs to do something in response to this info simply doesn't matter for the app
18:13:23 Quit aditya (Read error: 110 (Connection timed out))
18:13:36*gevaerts agrees with amiconn. It would be nice if the app could give some hints about which bit of storage it cares about, but it's better not to try to handle too many changes at the same time
18:14:37amiconnYeah, that's another thing. I think that later the app could pass e.g. a path, which can be used by the storage layer to decide which driver is responsible
18:15:06amiconnIn fact that might not be necessary
18:15:17gevaertsthe storage rework patch is already quite complex. I'd like to separate it in different bits, not make it bigger
18:15:48amiconnThe main purpose of ata_spin() is to keep the gui responsive when browsing, and for that, it makes much sense to broadcast it to all underlying storage driver
18:16:29amiconnata_sleep() otoh is meant to be sent whenever some buffering ends, in order to save timeout_seconds of disk power
18:16:35funmani think removing this special case for hard disks makes things simpler
18:16:55amiconnSo that can be broadcast as well - it is very likely that only one disk is spinning if there are several
18:17:05amiconnWhat special case????
18:17:21funmandisk spinning / sleeping
18:17:31funmanremoving it, *from the storage layer*
18:17:51funmanI wonder if you have noticed">r21912 and its discussion on the dev ML
18:17:53mcuelenaereCan RBUtility debug builds still be builded?
18:17:56*gevaerts sees that funman missed some stubs anyway :)
18:17:57amiconnRockbox strives to save power, so it is *necessary* for the app layer to tell the storage layer about these cases
18:18:39funmanamiconn: not to the storage layer, but to the harddisk-specific driver
18:19:04gevaertsfunman: that means the apps need to know what sort of storage they are using
18:19:06*amiconn wonders whether he can't explain things anymore :\\
18:19:25amiconnfunman: The app layer shouldn't have to care whether this is hdd or some other storage
18:20:07funmanwell using storage_sleep() implies you know about hard disks
18:20:41amiconnYes, but you don't need to know whether it actually *is* a hard disk
18:21:23funmanI don't know what is "quick sleep", or "timeout"
18:22:32funmansince you are writing this code specifically for the case if there is a harddisk, why not mentioning it explicitely?
18:22:56amiconnBetter abstraction *and* less ifdefing - I already said that
18:23:33amiconnThe only "drawback" is calling an empty stub on non-hdd targets, which means a whopping 8 bytes of binsize
18:24:42funmanthere is also the drawback of seeing the stubs defined which was more my real concern, rather than saving 8 bytes
18:25:05amiconnWhat drawback is that?
18:25:18funmanSeeing the stubs annoy me
18:25:57amiconnThen don't look... why stubs that provide better abstraction would be annoying is beyond me, really
18:26:23amiconnThey are not the only stubs in rockbox - sometimes stubs are the best soltion
18:26:23funmanHere I don't see the need for abstraction
18:26:45funmanOk you convinced me
18:27:26*kugel finds it sad that amiconn never bothers to express his opinion on the ml
18:27:45funmanme too
18:28:13amiconnirc is so much more convenient. And it's logged, so no disadvantage related to that
18:28:31funmanmailing list is threaded
18:28:34 Join shotofadds [0] (n=rob@rockbox/developer/shotofadds)
18:29:03shotofaddsI think we should re-implement the whole damn lot in C++ and hide the 'stubs' in the abstract storage driver base class ;-)
18:29:20shotofadds*much* better
18:32:45kugelamiconn, funman: If we make it inline, it would expand to nothing for the core, but an empty function for plugins, or am I wrong here?
18:33:15kugelI saw that if a inline is non-static, gcc inlines but also creates the function body (so that it's addressable)
18:33:19funmani think you can't export a static function
18:33:33amiconnfunman: inline, not static inline
18:34:09amiconnHmm, actually it could be static inline in the .h. Gcc should include a body in plugin.c in this case
18:34:37CIA-69New commit by 03funman (r21923): Revert">r21912 : "Storage API : remove undeeded target-specific functions" ...
18:34:43CIA-69New commit by 03funman (r21924): remove nand_soft_reset and nand_disk_is_active already voided in storage.h
18:34:47kugelit doesn't create a body at all, IIUC, if it's static. At least that's what I saw in chopper
18:35:08amiconnata.c wants it to be an ordinary fucntion though, as does the storage layer for true multistorage if that either includes ata or usb
18:35:13*kugel looked in the disassembly due to related weirdnesses
18:36:12kugeloh, the front page is quick today
18:37:14amiconnkugel: It will create a body as soon as you take the function address
18:37:58kugelah, I understand. but it still wouldn't work for ata.c?
18:38:20amiconnata.h would define it non-static as normal
18:38:25funmanthe declaration could be different for each storage type
18:42:00 Join bertrik [0] (
18:49:13 Join obo [0] (n=obo@rockbox/developer/obo)
18:58:41funmanI'm looking at clipv2 again, I'm embedding a rockbox.sansa binary with mkamsboot but I see weird behaviour : no LCD, reboot on keypress
18:59:06funmanI believe this is a problem with the as3514 code, there could be incompatible differences with the audio codec used in as3531
18:59:13kugelfunman: maybe a panic before lcd init?
18:59:17***Saving seen data "./dancer.seen"
18:59:26funmanah thanks i didn't about that
18:59:49kugeldid the lcd actually work on the clipv2?
19:02:51 Join DarkSpectrum [0] (
19:03:45 Join readability [0] (n=chad@
19:05:42kugelfunman: I'm getting the fuzev2 in ~1 month, probably
19:06:07kugelI hope the lcd driver is similar to the v1, I might be able to implement it then
19:06:56funmanhave you looked in the OF if the init procedure differ
19:07:10kugelno, not yet
19:08:12 Join aditya [0] (n=aditya@
19:10:25 Quit obo (Remote closed the connection)
19:10:40 Join obo [0] (
19:12:54kugelfunman: so you're in touch with a uclinux guy that made code for AMS?
19:13:19shotofaddsregarding the storage_* functions, I think it'd be best to proceed with the multistorage patch as-is, and then look at removing the "redundant" stubs later. KISS, as was mentioned earlier..
19:13:43shotofaddsso... with that in mind, is there anything else that needs to be done to">FS #9545 before it can be committed?
19:13:45kugelcommit early, commit often :)
19:14:43*shotofadds wonders about the high-score possibilities of this patch :)
19:14:59shotofaddsmaybe I'll leave it to gevaerts ;-)
19:15:09kugelmany .h files, generally a good way to break things :)
19:15:23 Join FlynDice [0] (
19:15:52kugeldoes it remove the uber-guly IF_MV stuff also?
19:15:58*kugel looks
19:16:14kugelyea, IF_MV is IF_MD now :S
19:16:49gevaertskugel: not in all places :)
19:16:59gevaertsThere's IF_MV() and IF_MD() now!
19:18:52*gevaerts seems to remember that there were still some open issues
19:19:28funmankugel: i contacted the person working at AMS who works on linux and uboot (not uclinux) ports
19:19:45funmanHe told me he couldn't do anything and advised me to contact the product marketing manager
19:20:08funmanthe one who stopped answering my emails some month ago (and is still continuing, i have no answer to my yesterday's mail)
19:20:15kugelgevaerts: could drive be a part of struct storage_info?
19:20:59kugelfunman: I wonder if at least 1 sd controller is still the pl180 one
19:21:19kugelgevaerts: wouldn't it make sense and save a parameter to many functions?
19:21:30funmankugel: what do you mean ?
19:21:51gevaertskugel: what functions? struct storage_info isn't passed to any functions...
19:21:55kugeleither the internal or external one (that of course applies to the fuzev2=
19:22:12kugelgevaerts: "void mmc_get_info(IF_MD2(int drive,) struct storage_info *info)"
19:22:30funmankugel: the controller in the clipv2 is not a pl180 for sure, and I didn't see 2 different drivers in the fuzev2 OF either
19:22:31gevaertskugel: that's to get information *from* a drive
19:22:49funmanthe person from AMS let me know the SD controller is made by synopsys
19:22:53kugelgevaerts: so what?
19:23:00kugelfunman: both?
19:23:09gevaertskugel: what exactly are you proposing?
19:23:12kugelI mean, IIUC the 3525 ones use 2 controllers
19:23:26kugelgevaerts: have a int drive member in struct storage_info
19:23:36gevaertskugel: again, why?
19:23:43kugeland anyhow get rid of IF_MV :)
19:23:43funmankugel: he didn't give me any details, I suppose the 2nd controller was added by SanDisk to the AMS design like they did for the AS3525 (which only has 1 PL180 controller according to the datasheet)
19:23:53gevaertsHow does that help you do that?
19:24:07funmankugel: the storage_info struct isn't filled when given to this function
19:24:26kugelit will be filled, but the drive number could be filled before
19:24:30funmanonly allocated
19:24:42kugelwhatever, nevermind
19:24:59gevaertskugel: so yes, you could remove the IF_MD() from *_get_info(). What does that gain you?
19:25:10gevaertsIt's still everywhere else
19:25:12kugelless uglyness
19:25:13 Join LambdaCalculus37 [0] (i=44a0430d@gateway/web/freenode/x-c4584eed183a0db2)
19:25:22gevaertsmore inconsistency
19:25:23funmankugel: if you want to work on it, fs#10047 has a description of all registers i could understand, and working code for sending commands to the card
19:25:29gevaertsi.e. *more* uglyness
19:26:01kugelI'm also not seeing that saving a drive parameter is worth obfuscicating code (and pre-processing function paramters is obfuscicating imo)
19:26:17mcuelenaereLambdaCalculus37: seen">FS #10445?
19:26:59kugelfunman: if the microsd is still "powert by" pl180, I could hopefully boot rockbox off the microsd
19:28:38kugelgevaerts: what do you not understand?
19:28:51 Join nls [0] (
19:28:55funmankugel: i didn't see pl180 code
19:29:24funmanand since the RAM is large enough, you can skip the bootloader process and embed rockbox directly into the firmware file (for testing purposes)
19:29:25gevaertskugel: why you feel it's good to pass the drive parameter from *_get_info() in a different way than to all other storage functions
19:30:07kugelit would be part of the plan to get rid of IF_MV completely :)
19:30:24*gevaerts gives up
19:30:30kugelfunman: is the firmware part of the OF big enough?
19:30:35kugelgevaerts: still not clear?
19:30:46kugelget a coffee! :p
19:31:11LambdaCalculus37mcuelenaere: No, I haven't. What is it?
19:31:44mcuelenaereLambdaCalculus37: didn't you commit">r21908?
19:31:56LambdaCalculus37mcuelenaere: I did.
19:32:08kugelmcuelenaere: the guy filled his flyspray info wrongly
19:32:34LambdaCalculus37mcuelenaere: I see what he said. He should've just posted that on the mailing list.
19:32:40 Nick fxb is now known as fxb__ (
19:32:42mcuelenaerekugel: you mean his name? I think it's correct
19:32:43funmankugel: for clipv2 yes (80kb left)
19:32:56*bertrik has an idea
19:33:00kugelmcuelenaere: it ends with i in his flyspray info too
19:33:11mcuelenaereisn't it supposed to be ending with an i?
19:33:13bertrikI have this firmware image and I wonder if there's a filesystem image in it
19:33:27kugelmcuelenaere: oh, I read it the other way around
19:33:31kugelnevermind then
19:33:31bertrikI guess I could use one of those lost file / partition scanners to find it
19:33:54funmanbertrik: perhaps look at hal/mount for filesystem signatures detection
19:34:02mcuelenaereanyway, it's a minor thing :)
19:34:05kugelfunman: how would you load rockbox.sansa into ram?
19:34:05 Nick evilnick is now known as eviltroll (i=0c140464@gateway/web/freenode/x-bdfd7fa976ec2ece)
19:34:08 Quit faemir (Remote closed the connection)
19:34:21kugelor in the firmware file
19:34:22bertrikfunman, what is hal/mount?
19:34:27funmankugel: mkamsboot
19:34:43funmanbertrik: hal is a freedesktop library / daemon for hardware detection
19:34:53kugelis 80K big enough?
19:35:03funmanmount is a standard unix command to mount filesystems (it can detect filesystem types)
19:35:19bertrikbut mount can't scan a file, or can it?
19:35:47funmanbertrik: no but you could see how it detects filesystems and look in your file for known filesystem signatures (magic values)
19:35:55 Nick Ypsy is now known as YpsyZNC (
19:36:11bertriktoo much work, I'd rather try a program already made for that
19:37:14kugelfunman: ah, the firmware part is 400k
19:37:21*kugel didn't know
19:38:02 Quit wincent (Read error: 110 (Connection timed out))
19:38:54gevaertskugel: so if you pass drive always, and get rid of IF_MV that way, why do you want the drive in struct storage_info?
19:38:57 Join bmbl [0] (n=Miranda@unaffiliated/bmbl)
19:39:49kugelI had that idea before I knew it was only used for that single function (I thought was more, but it was just the same function for ata/nand/sd/mmc)
19:40:06gevaertsso you didn't even look at the code?>
19:40:20kugelI skimmed the diff and looked at the struct definition
19:40:23 Quit einhirn ("Miranda IM! Smaller, Faster, Easier.")
19:40:43kugeland grepped firmware/*
19:40:46 Join einhirn [0] (
19:41:09 Join fdinel [0] (
19:41:10gevaertsshotofadds: before it can be committed, things like NUM_VOLUMES, MAX_NUM_DRIVES need to be cleaned up
19:42:00bertrikok, can't find anything in the firmware image with testdisk
19:42:26LambdaCalculus37Argh, why am I having trouble restoring the HDD1630? I'm trying to put it into recovery mode to restore Rockbox onto it.
19:42:32LambdaCalculus37toffe82: Ping
19:42:41LambdaCalculus37Or lowlight: Ping
19:42:50toffe82LambdaCalculus37: pong
19:43:01kugelfunman: btw, before mkamsboot 1.1, the (c) should be updated
19:43:06 Quit einhirn (Client Quit)
19:43:24LambdaCalculus37toffe82: Do you know how to get service and recovery mode working on the HDD1630? I'm using the instructions here:">
19:43:27funmankugel: i'm not sure large modifications were made this year
19:43:28LambdaCalculus37But nothing works.
19:43:34gevaertskugel: I told you several times that it's only used in *get_info(). You ignored that and kept pretending to know better
19:43:36 Quit aaron424 (Remote closed the connection)
19:43:37kugelit doesn't matter
19:43:50kugelgevaerts: no, you read my sentences wrong then
19:44:01funmani would just remove the copyright notice if linuxstb agrees
19:44:35 Join stoffel [0] (
19:44:39funmanbecause it is useless
19:44:43gevaertskugel: and every time someone tries to tell you that you're wrong, you try to ridicule them or you accuse them of misreading what you say
19:45:37toffe82LambdaCalculus37: it is not on the wiki of the 1630 ?
19:45:42 Quit fdinel (Client Quit)
19:46:24toffe82I though there was 2 separate pages ???
19:46:30 Join fdinel [0] (
19:46:48LambdaCalculus37toffe82: Just the one page AFAIK.
19:47:29toffe82it is not in the service manual ?
19:47:33 Quit fdinel (Client Quit)
19:47:53 Join fdinel [0] (
19:48:23linuxstbfunman: I've no objection
19:48:32linuxstb(but no opinion either way...)
19:49:33*kugel is curious what funman is going to answer (test build thread)
19:51:06CIA-69New commit by 03funman (r21925): mkamsboot: remove runtime copyright notice ...
19:51:30funmankugel: i won't answer
19:51:33 Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.")
19:51:36 Quit fdinel (Client Quit)
19:52:01 Quit flydutch ("/* empty */")
19:52:10LambdaCalculus37toffe82: I have the USB mass storage image on the screen.
19:52:23LambdaCalculus37But what appears in Disk Utility is a read-only device.
19:52:55 Quit mnk200 (Client Quit)
19:53:02kugelfunman: can you explain it to me then? I mean, what you said makes sense, but it still seems to have fixed the problems
19:53:14LambdaCalculus37Wait... switching to my Windows box got it to appear as a mass storage device.
19:53:32 Join fdinel [0] (
19:53:35kugelI query'd with him, fsck reported errors on the internal partition, and crapped out due to 512 filesystems on the microsd
19:53:35LambdaCalculus37ARGH... no drive appears, though.
19:53:41toffe82LambdaCalculus37: :)
19:54:07toffe82I think it is really dependant of the time you press the buttons
19:54:24funmankugel: there is still no known cause to problems with SD driver, so anything might randomly affect the transfers
19:54:32 Quit nls (Read error: 60 (Operation timed out))
19:54:55 Join MrDuck [0] (
19:55:24funmanperhaps this could be related to bad blocks, I think SD can't handle bad blocks in every situation. The SD specification might have more details on this topic
19:57:00kugelSD as in what? the card, the controller or our driver?
19:57:20gevaertskugel: I think the IF_MV() thing is already a lot better than it was. Due to the way the storage_* macros work, it's confined to the actual drivers. Code that just calls storage_* doesn't need them anymore
19:58:15funmankugel: card and/or controller (hardware)
19:58:45CIA-69New commit by 03dave (r21926): Improvments to Nano 2G LCD driver. This now works reliably on both LCD types.
19:59:00kugelfunman: the guy got his FS corruption before installing rockbox IIUC
19:59:08kugelgevaerts: yea, that's true
19:59:22gevaertsthat was by accident actually :)
20:00:23 Join Thundercloud [0] (
20:03:23kugelfunman: he installed the bootloader but couldn't install because of read-only filesystem
20:08:54kugelsomething is definitely weird with our sd driver
20:09:47*gevaerts thinks about how to get rid of this explicit MAX_NUM_DRIVES definition. Maybe just set it in config-*? It's only applicable to multi-driver anyway, so it wouldn't appear in many files
20:09:51funmankugel: help welcome
20:10:08 Join stoffel_ [0] (
20:10:43kugelgevaerts: how different is a drive from a volume with that patch?
20:11:12gevaertskugel: totally. That's one of the points. A drive is really a physical drive, while a volume is a filesystem
20:11:42kugeldoes it already allow more than 2 filesystems in rockbox?
20:11:57gevaertssure, if you set NUM_VOLUMES high enough
20:12:17 Nick dys` is now known as dys (
20:12:36dysis there a way to disable the nvram.bin-feature without recompiling?
20:12:37gevaertswhich is one of the other issues. What's a good default setting for NUM_VOLUMES? It's not NUM_VOLUMES_PER_DRIVE...
20:12:47kugelre: MAX_NUM_DRIVES: Couldn't you simply merge HAVE_MULTIDRIVE and MAX_NUM_DRIVE, in some sort of #define CONFIG_NUM_DRIVES X (X being 1 for single drive targets)
20:13:09kugeldefined in config-*
20:13:26*dys wants to prevent unneccessary writes to his precious 32GB cf card
20:13:30gevaertskugel: it doesn't use MAX_NUM_DRIVES for single-driver targets, so that bit is easy :)
20:13:32kugelnot sure about volumes though
20:14:02gevaertsbut yes, defining the number of drives in config-* sounds reasonable
20:14:18kugelgevaerts: I was thinking to do it similar as with dual-core which has also a NUM_CORES define
20:14:59kugelI think for single core it's actually set in config.h (#ifndef NUM_CORES \n #define NUM_CORES 1 \n #endif)
20:15:26kugelthere's no HAVE_DUAL_CORE #define
20:15:59gevaertskugel: the main annoyance with NUM_VOLUMES on e.g. a two drive player (like the sansas) is that it needs to be at least 2. Now if someone partitions the internal flash with two filesystems, suddenly the microsd slot stops working (from the user point of view. It's still there over USB)
20:16:47gevaertsWe could add extra code to allow single-volume-per-drive multidrive, but I don't really like that either
20:16:54kugelI'm thinking that, at least for internal storage, NUM_VOLUME could be determined at runtime
20:17:16gevaertsyou can't. The space needs to be allocated at boot
20:17:30kugelyes, exactly
20:17:48kugeldetect volumes at boot and allocate space before showing the main menu
20:17:57kugeli.e. what tagcache and the like do
20:18:22gevaertskugel: you need the space before you can even mount the first filesystem. This gets tricky
20:18:54 Join JdGordon| [0] (n=Miranda@nat/microsoft/x-62adc1062d939e90)
20:19:11gevaertsMaybe just assume NUM_DRIVES*4? We don't do extended partitions IIRC, so 4 is always enough. The only problem is a bit of wasted RAM, especially on the ondios
20:19:18kugelthe first filesystem could be statically allocated (there must be at least 1 volume anyway)
20:19:50gevaertsyou'll need more complicated data structures then. Right now it's just an array
20:19:58kugelthe bootloader would probably doesn't support multivolume anyway
20:20:06*gevaerts decides to measure the RAM usage overhead per extra allowed volume
20:20:43kugelany further volume could be perfectly allocated from the audio buffer
20:20:44LambdaCalculus37mcuelenaere: So zenutils can extract the Dell DJ bootloader?
20:20:46mcuelenaereLambdaCalculus37: it can also split the .bin file
20:20:54LambdaCalculus37mcuelenaere: A good sign? :)
20:20:57funmanwhy would you need several partitions if they all are fat32 ?
20:21:14mcuelenaereLambdaCalculus37: let's see if it has the keys to decrypt the main firmware )
20:21:38gevaertskugel: I'll first see how much the simple solution costs. It doesn't make sense to write complex allocation code if it's only a few hundred bytes
20:21:47kugelI agree fully
20:22:32gevaertsno point in saving 200 bytes of RAM if you need 1K of code to do that
20:23:55kugeland external drives would still be problematic
20:24:11kugelthey would at least require stopping playback if it was detected at runtime
20:24:21gevaertsah yes. That too...
20:25:47 Quit kugel (Remote closed the connection)
20:26:04mcuelenaereLambdaCalculus37: seems like these older firmwares don't even use 'real' encryption
20:26:34mcuelenaereI can send you the decrypted/splitted firmware if you want
20:26:55LambdaCalculus37mcuelenaere: Sure, email it over.
20:27:04LambdaCalculus37Do you need my email address?
20:27:09mcuelenaereI'll upload it
20:29:01mcuelenaerefor a rough description of what most of the files mean, see">
20:29:12 Quit funman ("free(random());")
20:29:16mcuelenaere(FBOOT & FRESC aren't encrypted)
20:29:34mcuelenaereCENC.dec is the decrypted version of CENC
20:31:25 Quit BryanJacobs ("Java user signed off")
20:31:36gevaertsHow much RAM do the v2 sansas have?
20:31:44LloreanDepends on the target
20:32:03gevaertsI know. I want to know all of them :)
20:33:32*gevaerts is trying to evaluate the cost of various multivolume options
20:39:55LambdaCalculus37mcuelenaere: I need to get a disk dump of the Dell DJ as well.
20:40:17mcuelenaereLambdaCalculus37: why? the file system is probably the same/an older version as the ZVM
20:40:37mcuelenaereLambdaCalculus37: does the Dell DJ communicate with a PC over MTP or NJB?
20:40:51LambdaCalculus37mcuelenaere: NJB IIRC.
20:41:00mcuelenaereah then it uses the older file system I think
20:41:20LambdaCalculus37mcuelenaere: I wanted to be sure.
20:43:17gevaertsOK. The total cost per volume on targets that already have more than one drive is 104 bytes
20:44:25gevaertsThat would mean that if we want to keep multidrive handling simple without risking not finding any volumes at all on the second drive because the first has more than one, it costs exactly 624 bytes
20:48:04JdGordon|SCREW bin/ram costs.... keep the damn code simple and eat the other costs
20:48:19JdGordon|seriously... its less than 1KB...
20:48:32 Join kugel [0] (n=kugel@rockbox/developer/kugel)
20:49:02kugelJdGordon|: I agree
20:49:43*gevaerts tends to agree as well, but he still did want to know the numbers first
20:49:50kugelhow does the filebrowser present the volumes?
20:50:09gevaertsI don't remember :\
20:50:12 Join Zarggg [0] (
20:50:26kugelI hope no driver letters :)
20:50:37kugelC:\rockbox :p
20:51:06JdGordon|or <Violume%d> iirc
20:51:26 Quit n00b81 ("Leaving")
20:51:46gevaertsOK. For Ondio the unmodified">FS #9545 has zero ramsize cost
20:51:58gevaertsso those 624 bytes are really all of them
20:53:07kugelis this with num_drives*4?
20:53:23gevaertsyes, but done by hand for now
20:53:36gevaertsi.e. #define NUM_VOLUMES 8
20:54:20kugelyea, the 600bytes are damn worth it then
20:54:40pixelmawhat for?
20:55:09kugelpixelma: scroll back a bit
20:56:35pixelmawell... "multivolume options" is a bit too general and I thought someone could give me a short summary instead having to look it up myself... oh. well... :\
20:56:42gevaertspixelma: the storage rework patch (">FS #9545) has the interesting side effect of making one pool of volumes on a multi-drive device. That means that while on e.g. ondio without HAVE_MULTIVOLUME you have one partition per drive, with">FS #9545 you just have two partitions, no matter on which drive, so two partitions on the first drive will hide the second drive
20:57:22gevaertsSo one proposal is to just always use HAVE_MULTIVOLUME, and allow 8 volumes. That's the maximum rockbox can handle anyway. The cost of that is 624 bytes
20:58:12bertrikthe YP-S3 LCD init looks quite similar to other LCDs already in rockbox, but there's not an exact match it seems
20:58:30gevaertson bigmem targets this is a no-brainer I think. The only targets where I'm not entirely sure would be ondio and c200v2
20:58:50pixelmaas if someone would partition a 128MB disk, except accidentally.... ;)
20:59:18***Saving seen data "./dancer.seen"
20:59:22itchegkugel: any idea how/why using a FM transmitter on an Ipod via IAP can garble the next tag?
20:59:33gevaertspixelma: there is that :)
20:59:51kugelitcheg: No, no idea. I have never worked with any of that
21:00:20itchegok, was asking more from the wps perspective
21:00:33kugelitcheg: JdGordon| may have an idea though, he reworked next-track stuff a bit few month ago
21:00:58itcheghmm let me double check my wps then
21:01:35 Quit Zarggg_ (Read error: 110 (Connection timed out))
21:02:03 Join salty-horse [0] (
21:02:16itchegkugel: can you tell me off hand if there is anything wrong with this: %s%alNext: %Ia;%t4%s%alNext: %?It<%It|%Fn>
21:02:21salty-horseis there a reason why the clock plugin is so big? (144KB)
21:02:35JdGordon|pretty images
21:03:35itchegJdGordon|: any idea how next tag can be affected by an ipod accessory?
21:03:59itchegthat is when used total nonsense shows
21:04:13JdGordon|no idea
21:04:28JdGordon|are you actually changing the track? or just having it connect
21:04:50itchegI guess I can file a bug, although I have no idea what info to include to make it usefull
21:04:56gevaertspixelma: I'm probably extrapolating from how I would behave. I guess you're right, and this won't be a problem anyway
21:06:20JdGordon|itcheg: extaclty when is the next track info getting garbeled? always? or only after changing tracks with it? or?
21:06:36kugelitcheg: no a new bug report, we have a task that collects IAP bugs
21:07:07itchegIt's not always have not been able to figure out exactly, but it only happens when I'm using the accessory
21:07:56itchegI used the iap patch before it was committed some months back and never had an issue
21:08:15kugelJdGordon|: I suspect your playback events
21:08:29itchegso not sure if something changed in iap or (more likely?) the whole wps change
21:08:32itchegor other
21:08:58*gevaerts has seen a garbled next track on his gigabeat F a few days ago...
21:09:12JdGordon|... are you changing the track from the transmitter?
21:10:51 Join mirak [0] (
21:13:15 Join dash32 [0] (
21:15:20 Join _zic [0] (
21:18:01amiconngevaerts: The volume pool issue already exists in svn
21:18:30gevaertsoh indeed...
21:18:33 Nick eviltroll is now known as evilnick (i=0c140464@gateway/web/freenode/x-bdfd7fa976ec2ece)
21:18:33*gevaerts must be blind
21:18:38amiconnImo it would be a good solution to limit both the total number of volumes and the number of volumes per drive
21:18:54amiconnThe latter limit should default to 1 imo
21:19:36kugelIIUC the targets defines the drive count?
21:19:40gevaertsshouldn't be too hard. Just have disk_mount() stop when it reaches the limit
21:19:59kugeloh, volume per drive
21:20:19 Quit mirak ("Ex-Chat")
21:22:00gevaertshm, this is easier than I expected. Just add an "&& mounted < VOLUMES_PER_DRIVE" to the main for loop in disk_mount()
21:22:45gevaertsamiconn: what do you think about the global number of volumes being limited to just NUM_DRIVES*VOLUMES_PER_DRIVE?
21:22:52JdGordon|gevaerts: do you remember "when" you saw the garbled next track? like was it after a track change (auto or manual?) or near begining/end of playlist? or?
21:24:27gevaertsJdGordon|: the garbled one was the last one in the playlist. I restarted playing from a bookmark, but I don't remember if the info was garbled when I restarted, or only a while later. I think it was garbled as soon as it was shown (i.e. just after an automatic track change)
21:25:01gevaertsOr do we need a separate total maximum number of volumes?
21:26:15 Join Zagor [242] (n=bjst@rockbox/developer/Zagor)
21:31:42itchegJdGordon|: I'm not making invoking any changes from IAP, I'm not sure as to any pattern, I don't think it was end of playlist etc.
21:32:24 Quit Galois ("Leaving")
21:33:04itchegI will try and discern a pattern
21:33:58itchegit happens a lot but not always
21:34:00 Join nls [0] (
21:35:10 Quit stoffel (Remote closed the connection)
21:35:43bertrikgevaerts, markun do you remember which id you got from the meizu m6 lcd?
21:36:05gevaerts0x0154 IIRC
21:36:48bertrikok thanks
21:36:55*gevaerts can remember this because he spent a long time trying to read that number from a flashing backlight :)
21:37:48 Quit DerPapst ("Leaving.")
21:40:33gevaertsbertrik: this is on m6sl by the way
21:41:45 Join mt [0] (n=mt@
21:42:44Zagordoes the iriver g120 have car adapter mode?
21:43:15bertrikgevaerts, the LCD I'm looking at now seems to expect 0x1503 back from register 0
21:44:39nlsZagor: the h300 at least starts the OF on AC power if off, dunno about h100
21:45:34mtsaratoga: fixp_mult_su() which multiplies a 32-bit number by a 16-bit fraction, won't it be faster if the fraction was represented with 32 bits ? then we could use the values in the library's trig table directly.
21:45:54bertriknot completely sure if it's really the LCD id, could also be some kind of verification of a previously written data
21:45:55Zagornls: starts OF, i.e does not boot into rockbox?
21:46:18bertrikis it usual for LCDs to have an LCD ID register at address 0?
21:46:54GreatBeaverhow come running time doesnt work on my rockbox h120 atm?
21:47:05GreatBeaverproblem with 3.3?
21:47:09nlsZagor: yes, this is fixed in svn but not present in the official booltoader, still not sure it applies to h100
21:47:19 Join funman [0] (n=fun@rockbox/developer/funman)
21:47:19Zagornls: ok
21:47:37itchegJdGordon|: before I go crazy testing scenarios can you confirm that this is ok: %s%alNext: %Ia;%t4%s%alNext: %?It<%It|%Fn>
21:48:13JdGordon|i see nothing obviously wrong... I thought %s cant go in sublines... but i have no idea
21:49:06gevaertsWhat happened to sd_disk_is_active()? I'm sure it did exist half a year ago
21:49:52bertrikweren't the xxx_is_active calls made obsolete with the xxx_idle_notify framework?
21:50:03JdGordon|that was the idea
21:50:08 Join kugel_ [0] (
21:50:18 Quit kugel (Nick collision from services.)
21:50:20GreatBeaveranyone know why run timer isnt working on my rockbox for h120?
21:50:20JdGordon|it didnt quite work 100% though
21:50:22 Nick kugel_ is now known as kugel (
21:50:25bertrikmaybe it was me who removed them now I think about it
21:50:27GreatBeaverit was working for 5 hours and then it just froze
21:50:42GreatBeaveri am afraid i damaged something physically in the H120 while modding is that possible?
21:50:47itchegok so I'll add the %s to the subline as well just to make sure...
21:51:11gevaertsThings still call storage_is_active()
21:51:51gevaertsAnyway, I can work around that. I don't need them
21:51:54nlsGreatBeaver: yes it is possible to physically damage a player
21:52:06GreatBeaverthats not what imean
21:52:18GreatBeaveri mean could the timer have stopped because i damaged something in the player?
21:56:29CIA-69New commit by 03zagor (r21927): Added average score and % cancelled time.
21:56:37LloreanGreatBeaver: Maybe you could be more specific about what exactly is happening, and use the terms from the manual so we know exactly which feature you're talking about
21:57:23GreatBeaverLlorean: when you go to system you can see system info, click it and you can see how many hours, minutes, seconds it has been runing
21:57:27GreatBeaverits been stuck at 0 0 0
21:57:34LloreanIs the charger plugged in?
21:57:36GreatBeaverbut it was working for 5 hours of time since yesterday
21:57:40GreatBeaverhmmm yes
21:57:59GreatBeaverlet me try disconnecting
21:58:39 Join w1ll14m [0] (n=54685215@gateway/web/cgi-irc/
21:59:07GreatBeavercool thanks it works now
21:59:09 Quit LambdaCalculus37 ()
21:59:12GreatBeaverhow come it turns off when the charger is on?
21:59:21LloreanAgain, I reference you to the manual
21:59:46LloreanIt should be in there, I hope.
21:59:49GreatBeaveri did try looking in the manual
22:00:08w1ll14mis there a way to use strncpy from a codec (recent changes to svn removed strncpy from codeclib)
22:00:13kugelJdGordon|: could you implement a touchscreen region for going to the playlist from wps?
22:00:20*bertrik intends to commit">FS #10445 - Error in CREDITS (a mis-spelled name)
22:01:23LloreanHm, Running Time isn't in the menu
22:01:25LloreanEr, manual
22:01:37LloreanGreatBeaver: Running Time is meant to tell you how long it's been running since a charger was last detected.
22:01:39LloreanNothing's broken
22:01:45GreatBeaveroh ok
22:01:52kugelw1ll14m: use strlcpy
22:03:37w1ll14mkugel: can i call that function like strcpy or do i have to change some?
22:04:46CIA-69New commit by 03bertrik (r21928):">FS #10445 - Error in CREDITS
22:05:07kugelif you're dealing with proper strings, then no. You also don't have to set the last byte to '\0'. Additionally you can check the return value against the size for tuncation
22:05:16funmanw1ll14m: you can check the description on
22:05:24w1ll14mfunman, kugel: thanx ;)
22:07:14pixelmaI didn't even notice I added my buildclient in the middle of a build round
22:07:42pixelmanot that it will be any usefull now that the "easy" builds are done
22:09:18pixelmabut the system dealt with it correctly :)
22:10:10w1ll14mkugel: to define strlcpy i have to use #define strlcpy ci->strlcpy right ?
22:10:17 Join notlistening [0] (
22:12:22notlisteningFlynDice, is there still need for testers on the sansaAMS SD voltage issue? I applied the pathch from the forums but not sure i did everything required to test it
22:13:38bertrikwhat I find a bit weird about this SD voltage issue is that we're lowering the 1.2V CPU voltage, not the 3V peripheral voltage that the SD slot uses
22:16:09notlisteningyeah and from my experience is that i had no issue with my current hard but i do not have lots to test with
22:16:54 Quit stoffel_ (Remote closed the connection)
22:17:32notlisteningon my e2xxv2 it decreases my runtime from 18 hours or so to 10
22:20:24 Join mc2739 [0] (
22:22:06funmanw1ll14m: yes or you can use ci->srtlcpy directly
22:22:18w1ll14mfunman: thanx
22:23:10gevaertsthere's a bug in the storage rework. It doesn't work properly with sansa+ramdisk
22:24:07 Part salty-horse ("Leaving")
22:24:48 Quit funman ("kde4 sux")
22:27:23 Join B4gder [241] (n=daniel@rockbox/developer/bagder)
22:28:45mc2739Zagor: In the new build system, how can a client's total time be more than the total time for the build? For example, build round 21928 took 326 seconds, but there are three clients with total times that are higher.
22:29:01Zagormc2739: because uploads are done in the background
22:29:50mc2739Zagor: is the upload counted in the client total, but not the build total?
22:30:27mc2739ah, thanks
22:33:33 Join petur [50] (n=petur@rockbox/developer/petur)
22:37:48nlsw1ll14m: you'll need to add strlcpy to the codec api
22:38:40w1ll14mnls: thanx, i just found out, i copied strlcpy from plugin.h/c
22:39:04nlsyeah that should do it
22:39:20nlswhat codec needs this btw?
22:39:56w1ll14mthere is this mikmod patch on flyspray
22:40:15w1ll14mit used strncpy
22:40:28w1ll14malso a nice moment to learn a little more about c
22:40:43CIA-69New commit by 03robert (r21929): Add information gleaned from disassembling the main firmware image. Detect lcd type in use. Still no actual output
22:43:08 Join Jaykay [0] (
22:44:39*linuxstb wonders why a codec would need strncpy
22:45:49JdGordon|kugel: should be dead simple to add... as long as there is already an ACTION_ for it...
22:46:07JdGordon|otherwise is will need a bit of extra work which is why some are missing
22:47:36amiconngevaerts: Btw, I observed something USB related on the beast: Since it got USB charging, I cannot use it when it's connected to my hub in rockbox. It works when connecting directly to the laptop
22:48:00amiconnBootloader USB works both on the hub and laptop
22:48:19amiconnIt seems the beast is drawing more than it says (or maybe even more than 500mA) when charging
22:48:39amiconnThe hub doesn't like this - it goes connect-disconnect-connect-disconnect....
22:48:39gevaertsoh, does your hub have overcurrent protection?
22:48:56amiconnAfter a number of cycles, rockbox freezes
22:49:25kugelJdGordon|: there is
22:49:41JdGordon|so its a 1 line change in wps_parser.c if you want to do it
22:50:04amiconngevaerts: Looks like it does, as *should* and uplink port...
22:50:28gevaertsso there are two problems : (a) it draws more than the hub likes (probably), and (b) it freezes after enough connect-disconnect cycles
22:50:42gevaertsamiconn: well, like so many things it's optional in the USB spec
22:51:25gevaertsIt would be good to verify that it's really the charging that causes this
22:52:21amiconnIt is - started happening when charging went in, just that I didn't bother to test different ports until now. Bootloader usb works fine on the hub, and that's what I normally use for updating
22:52:26kugelJdGordon|: hm, I'll look into it
22:52:46gevaertsyes, but IIRC charging went in at the same time as USB init rework
22:52:54 Join LambdaCalculus37 [0] (n=rmenes@rockbox/staff/LambdaCalculus37)
22:53:00 Quit mcuelenaere ()
22:55:37Zagorbah, what did I muck up this time?
22:56:18pixelmaI wondered about the order things happened
22:57:30w1ll14mwhat are the chances that strlcpy will be added to codeclib in the future ?
22:57:54gevaertsw1ll14m: if you can show you need it, 100% I guess
22:59:22***Saving seen data "./dancer.seen"
22:59:38Zagorah, bad regex
22:59:47w1ll14mgevaerts: thanx
22:59:53JdGordon|bad is redundant :)
23:00:07B4gderregex rocks!1
23:02:45Zagorthere, all green again
23:02:46CIA-69New commit by 03gevaerts (r21930): Allow access to the last sector of the ramdisk as well
23:03:29itchegTheSeven: Thank you, I'm not much of a programmer, (can read some assembler) but am fascinated by the project
23:03:40itchegwoops wrong channel
23:03:49TheSevenitcheg: read it nevertheless
23:03:56 Join faemir [0] (n=faemir@
23:04:48*gevaerts can now access three drives on his sansa over USB. Now to do this in rockbox itself...
23:05:03 Join dmb [0] (n=Dmb@unaffiliated/dmb)
23:06:59 Quit itcheg (Ping timeout: 180 seconds)
23:07:39amiconngevaerts: ramdisk?
23:08:00gevaertseasiest way to test multidriver
23:08:08amiconnYes, atm
23:08:23*amiconn wants to see the tpj port going, and also usb host
23:08:27gevaertsunless it's buggy of course :)
23:08:36 Join r0b- [0] (
23:08:41saratogamt: yes that was the idea, but first I wanted to test i using the existing multiply function just to make sure I didn't have any sort of off-by-one errors or similar
23:09:03kugelgevaerts: doesn't it start with 0?
23:09:11r0b-ok according to the hardware info for the E200 the cpu that it uses runs 200MIPS is that accurate?
23:09:14saratogasome of the speed up will be using the IRAM based trig table in the codeclib, some from the fasted 32x32=64 ASM'ed multiply function
23:09:21saratogar0b: no
23:09:29r0b-whats ir run then
23:09:43kugelI'd think it would be sector 0 to sector MAX-1 (MAX would be 1 too much)
23:09:58r0b-i know that but im talking in MIPS :P
23:10:02 Quit evilnick ("Page closed")
23:10:04saratogaMIPS is meaningless
23:10:09r0b-oh ok
23:10:11saratogaparticularly for ARMv4
23:10:32saratogabut it can't do 100MIPS anyway no matter how badly you fudge the numbers
23:10:34linuxstbbertrik: ping
23:10:39 Quit mc2739 ("ChatZilla 0.9.85 [Firefox 3.0.11/2009060215]")
23:10:43r0b-thanks saratoga
23:10:48gevaertskugel: it does, yes. Assume NUM_SECTORS==2, and read(1,1). That's allowed!
23:11:27saratogar0b: for codecs and such, number of multiply or multiply-adds per clock is more interesting usually
23:11:57gevaertskugel: the difference is this "+count" bit
23:12:33kugelgevaerts: isn't it an array[NUM_SECTORS]?
23:13:00gevaertskugel: NUM_SECTORS*SECTOR_SIZE
23:13:24kugelstart = NUM_SECTORS-1 (the last element of that array), count = 1 would overflow the array
23:13:30kugel(1 being a sector)
23:13:35gevaertsi.e. a read(1,1) will read 512 bytes starting from 512
23:14:01r0b-well another thing is that unless the e200 changed hardware my rockbox says mine has the PP5022C chip
23:14:11gevaertskugel: work out some examples in the real code, not simplified cases that disregard SECTOR_SIZE
23:14:29saratogashotofadds: how hard would it be to release a supported build for the D2 using SD only? it might make more people interested in D2 development if they could use it as a player
23:14:47 Quit dash32 (Read error: 110 (Connection timed out))
23:14:53saratogar0b: I think thats normal
23:15:01 Quit _zic (Remote closed the connection)
23:15:06saratogathat field is just what the hardware reports its name as I htink
23:15:28 Join dash32 [0] (
23:15:29amiconnr0b-: The PP5024 is basically a PP5022 with tacked-on AS3514, and it reports itself as PP5022
23:15:38r0b-oh ok
23:15:59kugelgevaerts: in the if there's no sector size.
23:16:28gevaertskugel: no, but in the array declaration there is
23:16:42amiconnZagor: Ahem, current build page only has the sources?
23:16:44kugelstart and count are in sectors, aren't they?
23:16:50shotofaddssaratoga: I've been toying with the idea of a producing such a build to go in the testing forum (I wouldn't necessarily want to call it supported yet). I think it's a good idea, but it'd only be worth it if the multi-driver rework is going to take a long time to complete...
23:17:14Zagoramiconn: haha, lovely
23:17:17gevaertskugel: assume an array "char a[64]". Are you allowed to read one byte starting from offset 63?
23:17:35saratogashotofadds; which rework is this? i thought the problem was lack of a FTL?
23:17:41B4gder36 clients is quite nice
23:18:05kugelyou're actually allowing to read from offset 64 too
23:18:10gevaertskugel: that's basically what's happening here. start+count=64, even if you're 0 based
23:18:11Zagoramiconn: fixed
23:18:18 Quit jgarvey ("Leaving")
23:18:26 Join Galois [0] (
23:18:30r0b-i gotta see if i can get my friend to hack my e250 to get video out :)
23:18:40 Quit w1ll14m ("CGI:IRC")
23:18:43 Join w1ll14m [0] (n=54685215@gateway/web/cgi-irc/
23:19:03kugelah, now I understand
23:19:22gevaertskugel: it's not simple. that bug was there for a reason :)
23:19:39shotofaddssaratoga: I meant the multi-storage-driver rework (fs#9545) which is needed to provide access to both NAND and SD.
23:19:58shotofaddsobviously there's no need for this to produce an SD-only build
23:20:35kugelgevaerts: yea, sorry about my confusion. start+count is the end, not the start :/
23:21:05B4gderZagor: I think using total = total/6 + current/6 should work fine enough
23:21:31saratogashotofadds: ah ok
23:21:37B4gderhm well not exactly that, but you get mu point
23:21:48amiconnmoo point ;)
23:21:49ZagorB4gder: yes :)
23:22:17*B4gder might have gotten one too many glasses of wine to be spelling correctly ;-)
23:22:21ZagorB4gder: who makes"> ? it's not updated for the new build.
23:22:27kugelgevaerts: looks like we have a similar bug in our sd driver
23:22:35B4gderlemme check
23:23:06kugel(samsa sd driver)
23:23:35B4gderZagor: the previous script invoked ./mksource after each build round
23:23:43gevaertskugel: hm, that looks suspiciously similar indeed
23:24:01B4gderZagor: present in my ~/rockbox-distbuild
23:24:25 Quit Jaykay ("ChatZilla 0.9.85 [Firefox 3.5/20090624025744]")
23:24:53kugeland it looks like this part could actually have dangerious side effects
23:25:50ZagorB4gder: ok, I'll run it then
23:27:24 Quit stripwax (Read error: 110 (Connection timed out))
23:28:08linuxstbbertrik: Good news - your Meizu i2c driver is working perfectly on the Nano 2G.
23:28:23saratogaZagor: any reason why my build server's directory has 5700 directories in its root?
23:28:38LambdaCalculus37linuxstb: Can I pick the first song for audio playback? :)
23:28:43Zagorsaratoga: !!
23:28:44saratogathey're each small but it seems like this could pretty quickly begin to bog down the file system
23:29:02 Quit bmbl ("Bye!")
23:29:18B4gderI think perhaps the build client should create more unique dir names, so it can delete them all explicitly each time it starts or restarts
23:29:20linuxstbLambdaCalculus37: It could come soon - I've already located audio codec functions in the diagmode code, and bertrik got sound on his Meizu...
23:29:27saratogaoh hmm they're pretty old, so maybe this bug has been fixed
23:29:34saratogalooks like since a week ago they stopped being made
23:29:39saratogasorry for the false report!
23:29:43Zagorsaratoga: ah, that sounds good
23:29:54LambdaCalculus37linuxstb: Then in that case...
23:30:09Zagorthough amiconn still reports getting some now and again, so B4gders suggestion sounds like a good one
23:31:41 Quit mt (Remote closed the connection)
23:32:56amiconnZagor: The buildclient leaks builddir if a disconnect happens during a build. 100% reproducable (didn't have that happen on the latest version yet)
23:33:02saratogaZagor: just wondering, if I have a networked file system, can I run the same build dir on multiple machines concurrently or do I need more then 1 checkout?
23:33:21 Quit Galois ("Leaving")
23:33:25Zagorsaratoga: you need one per kernel. the build dir is named after the pid
23:35:13Zagoramiconn: ok
23:37:23kugelgevaerts: this could be a major bug
23:38:09kugelif it's crossing banks, transfer is 1 sector too small, i.e. this sector will not be filled with data
23:38:34gevaertsthat would provide some interesting data corruption issues
23:38:44kugelalthough, as the same thing happens on reading, this sector should be skipped as well.
23:39:03gevaertsnot better...
23:39:18gevaertsi.e. you get whatever was in the buffer
23:39:21 Quit nls ("Lmnar")
23:39:33kugelbut we would probably handle files created by the of (any music, rockbox itself, due to missing usb) incorrectly
23:40:44 Join Zarggg_ [0] (
23:41:33*kugel commits the fix
23:42:03 Join aaron424 [0] (
23:42:11gevaertskugel: were there still known data corruption issues?
23:42:29 Join PaulJam [0] (
23:42:57kugelwell, my fuze functions properly, but it also only has 2GB. I'm not even sure if the 2GB do bank switches at all
23:43:26kugelbut yea, some people are still reporting fs problems
23:43:42roolkuamiconn: re: "* amiconn wants to see the tpj port going" - if you are interested in a device, I can give you a unit. The HD is broken, so you would need a hitachi 1.8" drive
23:44:05amiconnHmm, not another device with broken hdd.
23:44:21amiconnMy iPod Photo just ate its drive today :(
23:44:34pixelmaCF mod?
23:44:34*linuxstb has a working Elio that I would be glad to donate to anyone....
23:44:40Zagorholy cow. rockbox-clip.7z is a whopping 63% of!
23:44:47gevaertsroolku: you mean there are actually two of those in existence?
23:44:58linuxstbroolku: We should reunite them...
23:45:08amiconnSame failure as the first drive - starts making *a lot* of noise when accessing certain areas, and becoming extremely slow (basically stalled with myriads of retries)
23:45:13roolkuyou think they might breed?
23:45:17ZagorI know saratoga mentioned yesterday 7z was more efficient, but I didn't think the difference was this huge.
23:45:30pixelmagevaerts: and you have the other one of the two Logik Dax(e)s?
23:45:52gevaertspixelma: yes, but those have cousins, the ATMT-170s!
23:45:57B4gderZagor: zip is crap and 7zip is crazy good
23:45:59amiconnAnd I'm not using it much - that is certainly not due to mechanical stress
23:46:12kugelgevaerts: thinking about it again, it > or >= doesn't make any difference
23:46:20kugelin this case
23:46:24ZagorB4gder: we should use 7z for the uploaded targets. it will save us quite a lot of time.
23:46:27amiconnlinuxstb: Iirc bootloader boots, but there is no lcd output yet?
23:46:46B4gderZagor: yes, at least if bandwidth is the limiting factor and not the cpu
23:46:47kugelfor the == case, transfer won't actually change
23:46:50amiconnZagor: Do we want to repack on the server? Or wait for rbutil to support 7z?
23:47:03Zagor(since we tend to often end up with a zip target as the last build despite our efforts to the contrary)
23:47:21*amiconn votes for the latter
23:47:28roolkuamiconn: if it is the same type of drive as the h120, I have a spare 20GB one
23:47:36Zagorbleh, I forgot about rbutil
23:47:42amiconnIt's a dual platter Photo
23:47:56B4gderswitching to 7z might work as a subtle push on the rbutil team ;-)
23:48:36*gevaerts wants to know why the file browser doesn't want to see his ramdisk
23:48:44amiconnB4gder, Zagor: Any idea how long repacking would take? The saved upload time might still make up for that
23:48:47PaulJamkugel: when trying to make a H300 build with the custom ui vp v21 patch i get some warnings similar to "apps/gui/list.c:72: warning: implicit declaration of function `printf'" and compiing fails.
23:49:03kugelyea, I always forget to take those out :(
23:49:15Zagoramiconn: it probably wouldn't take very long. it just feels a bit "wrong"
23:49:39*B4gder agrees with Zagor
23:49:57*gevaerts thinks that providing a transition period would be good
23:50:30pixelma7zip could potentially be a bit more inconvenient for the average user (especially if RbUtil doesn't support it yet)
23:50:30gevaertsi.e. make the download links 7z, but make zips for rbutil until there's been a new rbutil for a few weeks
23:50:32B4gderis it really that many people that install the bleeding edge builds with rbutil?
23:50:35 Quit faemir ("Leaving")
23:50:39CIA-69New commit by 03kugel (r21931): Apply the same fix as">r21930 did for the ramdisk for the AMS Sansa driver. ...
23:50:47linuxstbamiconn: Yes, quite a few things work (standard PP) - ATA, most of the buttons (simple gpio). But no LCD.
23:51:19linuxstbamiconn: And I seem to recall it's got a wolfson codec, so the audio driver should be pretty similar to the others.
23:51:35amiconnLCD is 220x176 iirc?
23:51:42linuxstbYes, I think so.
23:51:46*amiconn would expect a pretty standard controller
23:51:49 Quit Zarggg (Read error: 110 (Connection timed out))
23:52:00amiconnDo we have access to the OF for disassembly?
23:52:08linuxstbYes, do you want it?
23:52:15kugelsince when is the front page picking up new commits so quickly?
23:52:36*amiconn is interested in both
23:52:38gevaertskugel: it used to be a cron job, it's a post-commit hook now I think
23:53:04Zagorgevaerts: no it's still a cronjob, we just run it more often
23:53:24*gevaerts promises to never soread misinformation again
23:53:45B4gderthe trigger for starting builds is not used for updating the front page
23:54:05kugeldid you just not bother enough to use a post-commit hook or is it hard to do with the front page?
23:54:18pixelmagevaerts: what about spreading it though?
23:54:23ZagorB4gder: it could though. I just added a "roundstart" script which could do just that
23:54:29gevaertspixelma: I don't know about that yet :)
23:54:33B4gderyes it could
23:54:46B4gderbut then I'd need to care for starting before the previous ended etc
23:54:53Zagorkugel: the svn and web servers are different machines, so a post-hook needs to communicate with the www server somehow
23:54:58B4gderjust using a cronjob is much simpler
23:56:47shotofaddsgevaerts: I'm massively confused trying to build a sim with the multi-storage patch. LD fails on the D2 sim looking for card_detect_target (referenced at root_menu.c:1100) but e200 sim does not seem to need this. I can't work out the difference :/
23:57:00 Join Ubuntuxer [0] (
23:57:42 Join tarbo_ [0] (n=me@unaffiliated/tarbo)
23:57:44gevaertsshotofadds: D2 would be multidriver, while e200 would only be multidrive
23:57:54gevaertsAre you working on the patch now?
23:58:05gevaertsIf so, maybe we need to coordinate a bit
23:58:21shotofaddsno, I'm not changing it at the moment
23:58:32gevaertsok. I'm safe then

Previous day | Next day