00:01:36 | pixelma | cabbiev2 (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/labb.contactor.se/x-5978cc22848336b2) |
00:07:07 | BdN3504 | how do i use the info presented in dmesg to mount a usb device in the vmware image? |
00:08:38 | Robert777 | w/o editing Cabbie 2 or 3 it does not display the converted bmp's |
00:08:52 | Robert777 | Ipod 60gig |
00:09:09 | BdN3504 | aaah, sorry i think i got it myself it says scsi 2 so that |
00:09:17 | BdN3504 | will be sdb right? |
00:17:41 | kugel | gevaerts: Zagor didn't change anything yet w.r.t to the weird build round times, right? |
00:18:12 | | Quit Robert777 () |
00:18:14 | kugel | I see 277s now which seems more reasonable than the 400+ which were before |
00:18:24 | kugel | although, there's 37 clients now, not exactly comparable |
00:23:40 | gevaerts | kugel: no. Maybe this runaway speculative build detection had some side effects? |
00:23:55 | | Quit DarkDefender ("Leaving") |
00:26:11 | | Join stephen_ [0] (n=stephen@86-40-168-77-dynamic.b-ras2.srl.dublin.eircom.net) |
00:26:44 | | Quit BdN3504 ("CGI:IRC (EOF)") |
00:27:51 | pixelma | about 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:30 | kugel | gevaerts: 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:15 | CIA-69 | New commit by mt (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: rockbox.org/shownewlog.cgi?rev=21908;type=mrobe100">http://build.rockbox.org/shownewlog.cgi?rev=21908;type=mrobe100 |
00:42:25 | roolku | mrobe100 that is |
00:43:41 | | Quit evilnick_7 ("Page closed") |
00:45:12 | roolku | infact 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:47 | shotofadds | gevaerts: 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 : http://kopete.kde.org") |
00:54:50 | funman | shotofadds: where is the SD code for D2 ? |
00:55:20 | shotofadds | funman: rockbox.org/tracker/task/10415">fs#10415. it's very much based on the PP driver. |
00:55:42 | CIA-69 | New commit by roolku (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:24 | funman | shotofadds: 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 |
01:00:08 | shotofadds | funman: they're needed for the storage rework (rockbox.org/tracker/task/9545">fs#9545). whether they are needed at all is a whole different discussion... |
01:00:40 | shotofadds | i'd like to see a way of getting rid of the nonsensical ones (sd_spin et al) |
01:00:51 | roolku | zagor/bagder: 2009-07-16 23:58:44 Server refused connection: error duplicate name! <−− on atlas-roolku half way through r21910 |
01:00:53 | funman | again void functions ? :/ |
01:01:31 | shotofadds | feel free to add your thoughts to rockbox.org/tracker/task/9545">fs#9545 ;-) |
01:01:54 | funman | sd_sleep() and sd_spin() should be removed as well |
01:02:44 | funman | perhaps "#define sd_sleep()" in storage.h |
01:03:06 | funman | or sd.h (more appropriate since it is included by storage.h) |
01:03:09 | | Join stephen_ [0] (n=stephen@86-40-168-77-dynamic.b-ras2.srl.dublin.eircom.net) |
01:03:26 | shotofadds | that could work, yes. |
01:03:37 | shotofadds | but right now I must sleep :/ |
01:04:17 | | Quit shotofadds ("Leaving") |
01:07:44 | funman | obo: you have problems reading thumb code? |
01:09:33 | CIA-69 | New commit by mt (r21911): Remove a call to av_clip() which limits the PCM output of the decoder to 16-bit. |
01:16:28 | funman | what is the difference between #defined HAVE_DISK_STORAGE and #if (CONFIG_STORAGE & STORAGE_ATA) ? (semantically) |
01:18:05 | funman | the 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:04 | pixelma | the former is only used be the ifp IIRC |
01:19:28 | pixelma | or maybe I'm confusing things |
01:19:43 | funman | it has a nand storage |
01:20:24 | Unhelpful | amiconn: 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] (n=john@dpc6682208002.direcpc.com) |
01:28:43 | | Quit jfc (Read error: 54 (Connection reset by peer)) |
01:29:04 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
01:29:34 | | Quit jfc (Read error: 54 (Connection reset by peer)) |
01:29:55 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
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] (n=jason@207-237-172-77.c3-0.nyr-ubr4.nyr.ny.cable.rcn.com) |
01:41:15 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
01:47:38 | | Join JdGordon| [0] (n=Miranda@131.107.0.69) |
01:49:54 | Unhelpful | ok, 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:18 | Unhelpful | a 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:16 | Unhelpful | it 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 |
02:00:00 | | Part toffe82 |
02:00:14 | | Quit JdGordon| ("Miranda IM! Smaller, Faster, Easier. http://miranda-im.org") |
02:04:48 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
02:05:11 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
02:06:04 | | Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.") |
02:09:19 | CIA-69 | New commit by funman (r21912): Storage API : remove undeeded target-specific functions ... |
02:11:52 | | Quit n17ikh (simmons.freenode.net irc.freenode.net) |
02:11:52 | NSplit | simmons.freenode.net irc.freenode.net |
02:11:52 | | Quit preglow (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit JdGordon (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Bagder (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit daurn| (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit rasher (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit jfc (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Sajber^ (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit tom243 (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Torne (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit saratoga (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit J-23 (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit rphillips (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Llorean (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit cdleonard (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Zarggg_ (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit trisiak (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit tmzt (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit safetydan (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Horscht (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit AndyI (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit gibbon_ (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit FOAD (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit dionoea (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit avacore^ (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit karma (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit vedlith (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit advcomp2019 (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit dz (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit krazykit (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Kohlrabi_ (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit soap (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Byan (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit Hadaka (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit rvvs89 (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit jordan` (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit obo (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit lostlogic (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit dys (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit fred_2 (simmons.freenode.net irc.freenode.net) |
02:11:52 | | Quit r00s (simmons.freenode.net irc.freenode.net) |
02:11:53 | | Quit jon-kha (simmons.freenode.net irc.freenode.net) |
02:11:53 | | Quit Slasheri (simmons.freenode.net irc.freenode.net) |
02:11:53 | | Quit rwong (simmons.freenode.net irc.freenode.net) |
02:11:53 | | Quit parafin (simmons.freenode.net irc.freenode.net) |
02:11:53 | | Quit bubsy (simmons.freenode.net irc.freenode.net) |
02:11:53 | | Quit Erant (simmons.freenode.net irc.freenode.net) |
02:13:45 | NHeal | simmons.freenode.net irc.freenode.net |
02:13:45 | NJoin | jfc [0] (n=john@dpc6682208002.direcpc.com) |
02:13:45 | NJoin | safetydan [0] (n=deverton@rockbox/developer/safetydan) |
02:13:45 | NJoin | Sajber^ [0] (n=Sajber@h-142-120.A213.priv.bahnhof.se) |
02:13:45 | NJoin | tom243 [0] (i=chris243@c-24-118-138-250.hsd1.mn.comcast.net) |
02:13:45 | NJoin | Horscht [0] (n=Horscht2@xbmc/user/horscht) |
02:13:45 | NJoin | n17ikh [0] (n=n17ikh@c-68-59-19-150.hsd1.sc.comcast.net) |
02:13:45 | NJoin | rasher [50] (n=rasher@rockbox/developer/rasher) |
02:13:45 | NJoin | preglow [0] (i=thomj@tvilling2.pvv.ntnu.no) |
02:13:45 | NJoin | JdGordon [0] (n=jonno@rockbox/developer/JdGordon) |
02:13:45 | NJoin | daurn| [0] (n=daurnima@ppp118-208-169-5.lns10.mel4.internode.on.net) |
02:13:45 | NJoin | Bagder [241] (n=daniel@rockbox/developer/bagder) |
02:13:45 | NJoin | Torne [0] (i=torne@lowell.wolfpuppy.org.uk) |
02:13:45 | NJoin | J-23 [0] (n=zelazko@unix.net.pl) |
02:13:45 | NJoin | AndyI [0] (i=AndyI@212.14.205.32) |
02:13:45 | NJoin | cdleonard [0] (n=cdleonar@86.121.203.175) |
02:13:45 | NJoin | gibbon_ [0] (i=gibbon_@could.become.a.servant4you.org) |
02:13:45 | NJoin | rphillips [0] (n=rphillip@66-90-184-91.dyn.grandenetworks.net) |
02:13:45 | NJoin | dys [0] (n=andreas@krlh-5f736ea8.pool.einsundeins.de) |
02:13:45 | NJoin | FOAD [0] (n=dok@dinah.blub.net) |
02:13:45 | NJoin | saratoga [0] (i=9803c6dd@gateway/web/freenode/x-9b73980ae5d89280) |
02:13:45 | | Join Llorean [0] (n=DarkkOne@rockbox/user/Llorean) |
02:13:45 | NJoin | avacore^ [0] (i=nobody@1008ds1-rdo.0.fullrate.dk) |
02:13:45 | NJoin | Zarggg_ [0] (n=zarggg@65-78-69-194.c3-0.eas-ubr6.atw-eas.pa.cable.rcn.com) |
02:13:45 | NJoin | dionoea [0] (n=dionoea@yop.chewa.net) |
02:13:45 | | Join bubsy [0] (i=Bubsy@unaffiliated/bubsy) |
02:13:45 | | Join obo [0] (n=obo@rockbox/developer/obo) |
02:13:45 | NJoin | karma [0] (i=amg@host193-123-47-78-dhcp.bshellz.net) |
02:13:45 | NJoin | vedlith [0] (n=ved2@137-mi2-1.acn.waw.pl) |
02:13:45 | NJoin | advcomp2019 [0] (n=advcomp2@unaffiliated/advcomp2019) |
02:13:45 | NJoin | dz [0] (n=dz@alt.dissonance.nl) |
02:13:45 | NJoin | krazykit [0] (n=kkit@c-24-218-166-241.hsd1.ma.comcast.net) |
02:13:45 | NJoin | tmzt [0] (n=tmzt@adsl-99-164-52-98.dsl.akrnoh.sbcglobal.net) |
02:13:45 | NJoin | trisiak [0] (n=tree@chello089078243195.chello.pl) |
02:13:45 | NJoin | Kohlrabi_ [0] (n=Kohlrabi@frustrum.nosebud.de) |
02:13:45 | NJoin | jon-kha [0] (i=jon-kha@kahvi.eu.org) |
02:13:45 | NJoin | soap [50] (n=soap@rockbox/staff/soap) |
02:13:45 | NJoin | Byan [0] (n=notByan@logo.csl.mtu.edu) |
02:13:45 | NJoin | jordan` [0] (i=gromit@78.235.252.137) |
02:13:45 | NJoin | rvvs89 [0] (n=ivo@pdpc/supporter/base/rvvs89) |
02:13:45 | NJoin | Hadaka [0] (n=naked@kiiro.naked.iki.fi) |
02:13:45 | NJoin | parafin [0] (i=parafin@paraf.in) |
02:13:45 | NJoin | fred_2 [0] (i=fred@hpc-cluster.hamburgnet.de) |
02:13:45 | NJoin | Erant [0] (i=erant@plz.stfu.kthnx.org) |
02:13:45 | NJoin | lostlogic [50] (n=lostlogi@rockbox/developer/lostlogic) |
02:13:45 | NJoin | r00s [0] (n=ru@zentrale.profitables.biz) |
02:13:45 | NJoin | rwong [0] (n=ricky@www.roflwaffle.com) |
02:13:45 | NJoin | Slasheri [0] (i=miipekk@rockbox/developer/Slasheri) |
02:13:45 | funman | lostlogic: hi |
02:13:45 | funman | i 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:57 | Unhelpful | *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] (n=evilnick@pool-173-52-144-203.nycmny.east.verizon.net) |
02:20:13 | obo | funman: 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:32 | JdGordon_ | how many codecs have subtrack/cuesheet support built in? |
02:20:38 | JdGordon_ | is it just ogg/flac? |
02:20:42 | JdGordon_ | and .cue obviously |
02:20:46 | funman | obo: what do you mean by registers passed between functions ? |
02:20:57 | Unhelpful | obo: technically immediate values in arm ore still only 8-bit... you just have the option of rotating them |
02:21:11 | funman | JdGordon_: 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:56 | funman | is there a list of the steering board on rockbox.org ? |
02:21:57 | JdGordon_ | i dont mean rockboxwise... I know flac has inbuilt cuesheet support (we dont support it yet), and ogg has subtracks... |
02:22:11 | JdGordon_ | RockboxSteeringBoard |
02:22:18 | funman | mp4 has chapters, but i don't know if it's used in audio |
02:24:44 | Unhelpful | JdGordon: ogg subtracks will be quite tricky to support in any meaningful manner, if you mean ogg chaining |
02:24:59 | JdGordon_ | 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:03 | amiconn | Unhelpful: 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:25 | Unhelpful | there's no global table of tracks and offsets, just <stream1headers><stream1data><stream2headers><stream2data> |
02:25:37 | obo | funman: 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:40 | JdGordon_ | that sounds pretty stupid :p |
02:27:16 | JdGordon_ | AA does conversion and resize on load right? would doing the same with .cue files be bad? |
02:28:09 | Unhelpful | amiconn: 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:54 | Unhelpful | the 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:31 | amiconn | IRAM doubles as part of .bss after init |
02:35:08 | Unhelpful | amiconn: right... but the stuff that belongs in iram is moved there, and .bss is cleared. |
02:35:47 | amiconn | Yes, but in either process there might be an error, although this is unlikely since the start of these sections is cacheline aligned |
02:35:48 | Unhelpful | anyway, 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:28 | Unhelpful | .bss is cacheline aligned? |
02:39:24 | amiconn | It is due to the preceding .ncdata section |
02:40:44 | Unhelpful | so how can padding the end of .data fix this, then? |
02:41:10 | amiconn | It can't - something else must be changing :\ |
02:42:59 | Unhelpful | perhaps i'm doing it wrong... but all i did was add the . += 0x4 in this: http://pastie.org/548902 |
02:44:47 | | Quit jfc (Read error: 54 (Connection reset by peer)) |
02:44:55 | | Join AJzer [0] (n=6171b58d@gateway/web/cgi-irc/labb.contactor.se/x-79b0e921365b6503) |
02:45:08 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
02:45:52 | amiconn | weird |
02:46:14 | amiconn | Did you compared the .map files for padding of 0/8/8/12 bytes? |
02:46:18 | amiconn | *compare |
02:46:28 | | Quit jfc (Read error: 54 (Connection reset by peer)) |
02:46:49 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
02:47:20 | | Quit jfc (Read error: 54 (Connection reset by peer)) |
02:47:20 | AJzer | So if rbutilqt autodetects ann Unsupported apple player variant, was I bilked? |
02:47:41 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
02:48:25 | JdGordon_ | jfc: if you are going to have connection issues again, please leave this chan untill you sort it all out |
02:48:44 | amiconn | There'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] (n=john@dpc6682208002.direcpc.com) |
02:50:34 | funman | AJzer: i'm not sure what "bilked" means, but if rbutilqt says it's unsupported, it's unsupported by rockbox |
02:50:55 | AJzer | It means I asked the sales guy if it was 5th gen, and he said yes. And it seems it was not. |
02:51:51 | AJzer | I 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:42 | Unhelpful | amiconn: 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:08 | Unhelpful | funman: "cheated" |
02:53:18 | amiconn | I'd like to see the .map files for the 4 cases, together with the info which of the 4 worked |
02:53:55 | Unhelpful | amiconn: not a problem... i'll throw them on pastebin :) |
02:54:07 | funman | AJzer: unlikely, but you can check http://support.apple.com/kb/HT1353 |
02:54:37 | Unhelpful | i'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:02 | AJzer | Yeah, 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:05 | evilnick_home | AJzer: That link shouldn't be vague at all |
02:57:07 | Unhelpful | amiconn: 0 padding: http://pastie.org/548910 4: http://pastie.org/548911 8: http://pastie.org/548912 12: http://pastie.org/548913 |
02:58:55 | *** | Saving seen data "./dancer.seen" |
02:58:56 | Unhelpful | the 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] (n=stettber@peer.zerties.org) |
02:59:32 | | Join karma_ [0] (i=amg@host193-123-47-78-dhcp.bshellz.net) |
03:00 |
03:00:45 | AJzer | evilnick_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] (n=chatzill@ppp-71-140-90-107.dsl.frs2ca.pacbell.net) |
03:02:13 | funman | isn't the case different ? (polished for classic) |
03:02:51 | amiconn | Unhelpful: 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:11 | amiconn | ...and with a build produced by our standard toolchain |
03:03:13 | evilnick_home | I thought that the Classics all had metallic fronts |
03:03:31 | AJzer | if I have a classic, it's got a black matte front. |
03:03:33 | linuxstb | AJzer: 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:43 | Unhelpful | amiconn: it's not aligned! |
03:04:08 | | Join webmind [0] (n=webmind@shell.puscii.nl) |
03:04:17 | amiconn | In 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:23 | JdGordon_ | they arnt digits! |
03:04:46 | AJzer | linuxstb: thanks. I've read through that damn page numerous times and missed this. |
03:04:47 | amiconn | But somehow the linker manages to ignore the alignment |
03:05:03 | amiconn | The wonders of using dev snapshots :\ |
03:05:15 | linuxstb | AJzer: But rbutil won't lie about that either... |
03:06:05 | Unhelpful | amiconn: 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:12 | amiconn | It is in fact interesting that the code didn't crash. The .icode functions all end up shifted in iram |
03:08:14 | Unhelpful | when the padding was in window.c, 16 came back around to being broken. |
03:08:35 | AJzer | linuxstb: software can be particular, and imperfect. I felt it was worth checking before going through the process of returns. |
03:08:48 | amiconn | Yeah, but what about .data? This seems to be a bug in section handling |
03:09:08 | | Quit JdGordon_ (Ping timeout: 180 seconds) |
03:11:30 | Unhelpful | i'm trying it now... |
03:14:31 | | Quit AJzer ("CGI:IRC") |
03:15:40 | | Join notlistening [0] (n=tom@94-195-105-95.zone9.bethere.co.uk) |
03:16:32 | Unhelpful | the address moves to the next aligned value (0x01e96f30) at pad = 12. bumping the padding to 16 changes it to 0x01e96f28 |
03:16:35 | notlistening | Hi 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:08 | notlistening | SansaAMS related |
03:17:09 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
03:17:42 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
03:18:03 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
03:18:17 | Unhelpful | i 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:34 | funman | notlistening: ping flyndice |
03:18:55 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
03:19:46 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
03:20:06 | notlistening | funman, now i am being an irc idiot how do i ping? |
03:20:07 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
03:20:12 | Unhelpful | it 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] (n=john@dpc6682208002.direcpc.com) |
03:21:17 | Unhelpful | notlistening: 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] (n=john@dpc6682208002.direcpc.com) |
03:22:06 | amiconn | It'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] (n=john@dpc6682208002.direcpc.com) |
03:23:06 | Mode | "#rockbox +o JdGordon " by ChanServ (ChanServ@services.) |
03:23:14 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
03:23:16 | funman | notlistening: just someting like "flyndice: ping" (when he'll be here, but i hope he reads the logs) |
03:23:18 | Unhelpful | the previous snapshot is from november, and there are naturally quite a few daily and weekly diffs since then. |
03:23:35 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
03:23:47 | Mode | "#rockbox +b *!john@dpc6682208002.direcpc.com " by JdGordon (n=jonno@rockbox/developer/JdGordon) |
03:24:48 | Unhelpful | or "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:39 | Mode | "#rockbox -o JdGordon " by ChanServ (ChanServ@services.) |
03:26:58 | notlistening | thanks for the tip Unhelpful |
03:34:09 | Mode | "#rockbox +o JdGordon " by ChanServ (ChanServ@services.) |
03:34:15 | Mode | "#rockbox -b *!john@dpc6682208002.direcpc.com " by JdGordon (n=jonno@rockbox/developer/JdGordon) |
03:34:17 | Mode | "#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] (n=Lss@cm40.delta91.maxonline.com.sg) |
03:39:31 | | Quit Riku (Read error: 54 (Connection reset by peer)) |
03:40:54 | soap | 2009-07-16 20:14:37 Server refused connection: error duplicate name! |
03:40:54 | soap | Address the above issue(s), then restart! |
03:41:12 | soap | EST - FWIW. Is this a known issue with the new build client or is there something I am doing wrong? |
03:41:30 | Mikachu | you're probably using a -name that is already used? |
03:41:50 | Mikachu | maybe an old connection that didn't time out yet |
03:45:05 | | Quit funman ("free(random());") |
03:46:26 | Unhelpful | amiconn: i would think if we find some non-release binutils revision that makes this work we'd need to add support to rockboxdev.sh 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] (i=xerion@82-170-197-160.ip.telfort.nl) |
04:00 |
04:05:31 | | Quit Xerion (Read error: 110 (Connection timed out)) |
04:05:31 | | Nick Xerion_ is now known as Xerion (i=xerion@82-170-197-160.ip.telfort.nl) |
04:07:01 | | Quit TheSeven (Nick collision from services.) |
04:07:17 | | Join The_Seven [0] (n=theseven@dslb-084-056-176-104.pools.arcor-ip.net) |
04:07:21 | | Nick The_Seven is now known as TheSeven (n=theseven@dslb-084-056-176-104.pools.arcor-ip.net) |
04:18:39 | | Join dys` [0] (n=andreas@krlh-5f7069bf.pool.einsundeins.de) |
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 (i=amg@host193-123-47-78-dhcp.bshellz.net) |
04:55:29 | Unhelpful | amiconn: 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:00 |
05:02:57 | | Quit KBH (Read error: 104 (Connection reset by peer)) |
05:03:10 | | Join HBK [0] (n=hbk@pool-71-96-74-73.dfw.dsl-w.verizon.net) |
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@188.16.87.95) |
05:31:34 | | Join __lifeless [0] (n=lifeless@188.16.87.95) |
05:34:35 | | Join dash32 [0] (n=dash32@84.171.84.70) |
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:04 | CIA-69 | New commit by kkurbjun (r21913): M:Robe 500: Start of interrupt support. |
05:48:32 | | Quit _lifeless (Read error: 113 (No route to host)) |
05:49:32 | Unhelpful | bah! 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@76.235.56.133) |
06:00 |
06:00:04 | | Nick RandAl is now known as Rand_Althor (n=chatzill@76.235.56.133) |
06:00:11 | Rand_Althor | Off-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@59.95.5.10) |
06:14:35 | | Quit dash32 (Remote closed the connection) |
06:18:16 | | Join goffa [0] (n=goffa@216.220.23.105) |
06:19:12 | | Join dash32 [0] (n=dash32@84.171.84.70) |
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:37 | CIA-69 | New commit by kkurbjun (r21914): M:Robe 500: Fix simulator build and include some small changes for the 640x480 setup. |
06:52:26 | | Join cdleonard [0] (n=cdleonar@86.121.201.241) |
06:59:02 | *** | Saving seen data "./dancer.seen" |
07:00 |
07:00:13 | | Join courtc_ [0] (n=court@unaffiliated/courtc) |
07:02:26 | | Quit courtc (Read error: 111 (Connection refused)) |
07:08:12 | CIA-69 | New commit by kkurbjun (r21915): Jewels: Simplify support for new target screen sizes. Just adding new bitmaps for the target should now be enough. |
07:10:21 | cdleonard | hello! |
07:10:44 | cdleonard | I'm trying to look at the iAudio7 port but it fails to compile with 'unsupported instruction on ldrd' |
07:11:03 | Unhelpful | is that *exactly* what it says? "on ldrd"? |
07:12:30 | cdleonard | no; not exactly. I'll give you the exact message |
07:15:27 | cdleonard | /home/cdleonard/files/work/rockbox/trunk/apps/recorder/jpeg_idct_arm.S:197: Error: selected processor does not support `ldrd r4,.Lpool4' |
07:15:38 | cdleonard | and more like that |
07:16:06 | | Quit CaptainKwel (Remote closed the connection) |
07:17:03 | Unhelpful | that's interesting. the iaudio7 should, to my knowledge, support that instruction... |
07:17:09 | cdleonard | here are the full messages: http://pastebin.com/mb53ee3f |
07:17:57 | cdleonard | it seems iAudio7 sets -mcpu=arm9. Maybe that's a bit too general? |
07:19:16 | cdleonard | the architecture numbers seem very confusing |
07:20:09 | cdleonard | it sets -mcpu=arm9e; sorry |
07:23:02 | Unhelpful | to 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:13 | cdleonard | this page says so rockbox.org/twiki/bin/view/Main/CowonIaudio7Info">http://www.rockbox.org/twiki/bin/view/Main/CowonIaudio7Info |
07:27:43 | cdleonard | but now arm-elf-as fails with 'unknown cpu `arm946e-s'' |
07:29:04 | cdleonard | and if I set it to just 'arm946e' (also listed as a supported cpu in the manual) I get errors in SOURCES files |
07:29:48 | cdleonard | like this: http://pastebin.com/ma3940cc |
07:30:08 | cdleonard | I don't get it; why would an error be printed for one of those files? |
07:32:08 | Unhelpful | those 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:32 | Unhelpful | ah! i see the issue... gcc knows "arm946e-s", but not "arm946e"... and for the assembler it's the other way around. |
07:33:41 | cdleonard | but as doesn't recognize arm946e-s; but it does recognize arm946e |
07:33:45 | cdleonard | yes |
07:34:07 | cdleonard | something about this patch: rockbox.org/gcc/rockbox-multilibs-arm-elf-gcc-4.0.3.diff">http://www.rockbox.org/gcc/rockbox-multilibs-arm-elf-gcc-4.0.3.diff ? |
07:36:01 | Unhelpful | no, 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] (n=Miranda@bsod.rz.tu-clausthal.de) |
07:41:35 | | Nick stettberger_ is now known as stettberger (n=stettber@peer.zerties.org) |
07:42:41 | cdleonard | are you sure I need different versions? that patch seems to list mcpu options |
07:42:56 | cdleonard | (i have no idea what multilibs are about) |
07:44:02 | | Part toffe82 |
07:44:04 | Unhelpful | i 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:12 | Unhelpful | the patch is really not your problem. |
07:49:09 | cdleonard | but this version mismatch has been there for quite a while; right? |
07:51:36 | Unhelpful | right, 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:08 | cdleonard | http://sourceware.org/binutils/docs-2.16/as/ARM-Options.html#ARM-Options |
07:58:12 | cdleonard | http://gcc.gnu.org/onlinedocs/gcc-4.0.4/gcc/ARM-Options.html#ARM-Options |
07:58:19 | | Quit amiconn (Nick collision from services.) |
07:58:20 | cdleonard | it 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:29 | Unhelpful | binutils-2.17 support arm946e-s |
08:00 |
08:01:28 | cdleonard | yes; and it seems less risky than upgrading gcc |
08:02:45 | Unhelpful | i 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] (n=quassel@p57B4DA3B.dip.t-dialin.net) |
08:28:13 | amiconn | Unhelpful: Such is the nature of bugs - things don't work as expected... Did you try a fresh snapshot already? |
08:28:50 | amiconn | If 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:56 | Harryy | How 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:26 | Unhelpful | amiconn: 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:00 | amiconn | Somehow 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:28 | Unhelpful | any idea about this -mcpu=arm9e issue? i can't find anything *besides* binutils that says it doesn't support the ldrd instruction. |
08:34:02 | Unhelpful | i don't know... i've been trying to come up with a synthetic test case for the bug without any luck. :/ |
08:34:24 | amiconn | Imo we should update the required binutils for arm, and then use arm946e-s |
08:35:14 | markun | Harryy: actually it's not looking so good |
08:35:18 | Unhelpful | even the newest does this with -mcpu=arm9e - i had the same thing happen while building libgcc |
08:35:56 | markun | Harryy: Internally it's very similar to the Gigabeat S, but the exploit to run our code has been fixed in the Zune. |
08:37:32 | cdleonard | ok; I tried to build with binutils 2.17 but it fails to link; something about FPA instructions |
08:38:22 | cdleonard | http://pastebin.com/m7f478d97 |
08:38:34 | Unhelpful | i 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:58 | cdleonard | this is with -mcpu=arm946e-s |
08:39:08 | Harryy | markun: 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:09 | amiconn | Unhelpful: Yes, since arm9e obviously doesn't support that instruction, but the jpeg decoder asm stuff uses it for armv5 |
08:40:14 | amiconn | But since our arm9e is actually an arm946e-s which supports it, going for the latter option would be a good thing |
08:42:06 | Unhelpful | amiconn: 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:08 | Harryy | markun: 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:18 | Unhelpful | Harryy: i don't think there's been any real progress in quite some time |
08:43:19 | Harryy | I meant never use Winblows |
08:43:54 | Harryy | Unhelpful: bah. So I should call Microfail and tell them to port it to Linux? :3 |
08:44:00 | Harryy | "or else!"? |
08:44:24 | Unhelpful | good luck. |
08:44:54 | Unhelpful | they never said it would work with linux, or that you could run your own code on it. |
08:45:13 | Harryy | hrm :| |
08:45:19 | | Join Rob2223 [0] (n=Miranda@79.220.200.191) |
08:45:21 | Harryy | it doesn't even work under Wine |
08:45:24 | Harryy | money hunry pigs |
08:46:27 | Unhelpful | wine doesn't support USB devices at all, does it? other than using platform drivers for things like mass storage? |
08:46:32 | Harryy | I dunno |
08:46:40 | Harryy | the Zune.exe will not even install |
08:46:55 | Harryy | won't even launch the install GUI :\ |
08:47:37 | Harryy | so, what do you suggest? I break my zune and use it as an external HDD? |
08:48:00 | | Join flydutch [0] (n=flydutch@host87-202-dynamic.15-87-r.retail.telecomitalia.it) |
08:48:27 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
08:51:16 | amiconn | Unhelpful: 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:25 | cdleonard | any thoughts about my link issue? |
08:51:45 | cdleonard | maybe I need to build libgcc differently? |
08:56:47 | | Part Harryy |
08:58:09 | Unhelpful | did you build your gcc and binutils together? i would recommend you use the rockboxdev.sh 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:14 | cdleonard | edit the multilib patch? |
08:59:24 | Unhelpful | yes. |
08:59:51 | Unhelpful | if you just edit the downloaded copy in-place, the script should use it without downloading it again. |
09:00 |
09:00:06 | JdGordon | have 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:14 | cdleonard | ok; 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:14 | JdGordon | http://www.overclockers.com.au/image.php?pic=images/newspics/17jul9/12.jpg |
09:03:20 | JdGordon | got 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:31 | amiconn | Unhelpful: Check infocenter.arm.com. 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:45 | amiconn | Weird - we'd have to distinguish revisions... |
09:08:01 | Unhelpful | ah... that's horrid. the quickref sheet simply reads "ARM v5E, and 6 and above" :/ |
09:08:59 | Unhelpful | i 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:06 | amiconn | Same applies to ARM966E-S revions, btw |
09:09:58 | amiconn | The ARM reference manual states "Version 5TE and above, excluding ARMv5TExP" for ldrd/strd |
09:12:40 | Unhelpful | ew. perhaps we need to define an ARM_FEATURES or such... do we actually have any ARM9 targets *without* ldrd and friends? |
09:13:07 | amiconn | I don't know - I don't have *any* arm9 target, only arm7 and arm11 |
09:13:40 | Unhelpful | it's the same here, only a beast and e200 :/ |
09:13:47 | amiconn | Since the SoC manufacturers probably won't tell what revision of the arm core is used, this will probably need on-target testing |
09:14:24 | Unhelpful | it seems like that would be important to know if you're to generate code for it :/ |
09:15:14 | amiconn | We 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:39 | amiconn | err I mean armv4, not armv5 |
09:16:28 | amiconn | ARM architecture versions vs. arm versions is messy like that. Even revisions are important :\ |
09:18:07 | | Join KBH [0] (n=hbk@pool-71-96-74-73.dfw.dsl-w.verizon.net) |
09:19:23 | | Join Grahack [0] (n=chri@stc92-1-82-227-106-100.fbx.proxad.net) |
09:23:21 | cdleonard | it compiled! I still got errors in apps/plugins/flipit and brickmania; but I also got a rockbox.zip |
09:23:36 | cdleonard | I guess the plugin errors are normal for platforms that are not fully supported? |
09:24:48 | Unhelpful | plugins 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:56 | Unhelpful | ugh... 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] (n=hbk@pool-71-96-74-73.dfw.dsl-w.verizon.net) |
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] (i=thunderc@persistence.flat.devzero.co.uk) |
09:34:11 | | Quit HBK (Read error: 110 (Connection timed out)) |
09:34:34 | amiconn | Unhelpful: We could play safe and not use ldrd/strd on all armv5 |
09:37:13 | Unhelpful | amiconn: yes, but i don't think that all support smulxy and friends, either |
09:37:22 | amiconn | They do |
09:37:34 | amiconn | Check the reference manual |
09:38:46 | amiconn | It's ldrd, strd, mcrr, mrrc and pld which aren't supported in the v5TExP variant |
09:41:11 | Unhelpful | we 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:36 | Unhelpful | any 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:42 | amiconn | On 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:20 | Unhelpful | ah... so it's only of benefit to code size on armv5 |
09:45:25 | amiconn | On armv6 this is no longer true, but that's not a problem |
09:46:06 | amiconn | Yeah. 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:54 | Unhelpful | it's going to be quite a small difference, compared to the difference it makes on armv6 |
09:48:00 | Unhelpful | am 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:13 | amiconn | armv6 does much more pipelining (even ldm/stm are single cycle), hence there are more interlocks |
09:50:09 | amiconn | Variable shift input means an extra cycle on armv5 and earlier. Not so on armv6, there it means having an "early reg" |
09:53:20 | Unhelpful | the shifted register seems to be an "early reg" on armv6 even if it's shift-by-immediate |
09:53:54 | Unhelpful | per the example interlocks section for arm1136jf-s |
09:56:11 | | Join robin0800 [0] (n=robin080@81.98.157.181) |
09:56:24 | | Join KBH [0] (n=hbk@71.96.74.73) |
09:58:22 | | Join HBK [0] (n=hbk@pool-71-96-74-73.dfw.dsl-w.verizon.net) |
10:00 |
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] (n=chatzill@d58-110-115-114.meb3.vic.optusnet.com.au) |
10:21:29 | icefest | Hello |
10:21:42 | icefest | ceebs |
10:21:45 | | Quit icefest (Client Quit) |
10:30:50 | | Quit BHSPitMonkey (Remote closed the connection) |
10:35:18 | amiconn | Unhelpful: 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:38 | amiconn | The former is an easy fix - I'd just have to change a preprocessor conditional |
10:36:01 | amiconn | Do you think this should be the way to go? |
10:36:22 | amiconn | (actually two preprocessor conditionals - one for ldrd and one for strd) |
10:39:38 | | Join Sajber^ [0] (n=Sajber@h-142-120.A213.priv.bahnhof.se) |
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:17 | cdleonard | I managed to upload stuff to my iAudio7 with tcctool but nothing shows up (only the buttons light up) |
10:45:54 | cdleonard | the wiki says the lcd should work |
10:45:56 | | Join m67_l3 [0] (n=martian6@about/linux/regular/martian67) |
10:46:18 | linuxstb | What "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:50 | cdleonard | I first uploaded rockbox.bin directly |
10:47:15 | cdleonard | then I used tools/scramble -tcc=crc and uploaded the result |
10:47:17 | | Quit Sajber^ (Read error: 104 (Connection reset by peer)) |
10:47:22 | cdleonard | nothing happened both times |
10:47:51 | | Join Sajber^ [0] (n=Sajber@h-142-120.A213.priv.bahnhof.se) |
10:48:21 | linuxstb | You could try a bootloader build, and upload that. |
10:48:38 | cdleonard | from what I understand using tcctool loads a binary into RAM which is lost on a device reboot |
10:48:46 | linuxstb | Yes |
10:49:02 | cdleonard | so it's perfectly safe. |
10:49:06 | * | linuxstb wrote tcctool |
10:50:27 | cdleonard | I 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? |
10:51:50 | cdleonard | rockbox.org/index.php?topic=15360.0">http://forums.rockbox.org/index.php?topic=15360.0 vitja mentioned replacing stuff in the original firmware |
10:51:56 | linuxstb | A 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:30 | cdleonard | maybe that's the only way the lcd works? |
10:53:00 | cdleonard | "In a new port, the bootloader is normally developed work, so it more likely to be working." what? |
10:53:19 | linuxstb | s/work/first/ |
10:53:27 | gevaerts | try the bootloader first. I'm not sure if the main binary is always linked at the right addresses to work with tcctool |
10:53:28 | tmzt | ah |
10:53:46 | | Quit martian67_ (Success) |
10:56:20 | cdleonard | ok; and the bootloader can also be uploaded using tcctool? |
10:57:43 | linuxstb | It 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] (n=rob@78-69-30-229-no36.tbcn.telia.com) |
10:59:08 | *** | Saving seen data "./dancer.seen" |
11:00 |
11:01:34 | Unhelpful | amiconn: 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:54 | amiconn | These are the only two places |
11:02:26 | cdleonard | so I can either use {scramble -tcc=crc and tcctool} or {mktccboot and flashing} |
11:02:40 | amiconn | There 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:49 | amiconn | grep told me... |
11:04:15 | Unhelpful | grep is such a helpful friend to have :D |
11:07:04 | linuxstb | cdleonard: 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] (n=kachna@r3g248.net.upc.cz) |
11:08:27 | cdleonard | wait; can't I upload a mktccboot image with tcctool? |
11:09:31 | linuxstb | Yes, you can probably do that as well. |
11:09:48 | | Part karma |
11:13:53 | Unhelpful | amiconn: 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:59 | CIA-69 | New commit by amiconn (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:01 | CIA-69 | New commit by unhelpful (r21917): Remove ldrd from ARMv5 JPEG IDCT, remove old debug code selecting ARMv5 code for one function even when building for ARMv6. |
11:24:40 | cdleonard | yes; mktccboot and then tcctool worked |
11:25:11 | cdleonard | I get the OF if hold is off and otherwise a white screen |
11:25:15 | cdleonard | yay! |
11:25:53 | | Join _lifeless [0] (n=lifeless@188.16.69.242) |
11:27:19 | Unhelpful | cdleonard: we *said* unsupported! ;) |
11:27:53 | gevaerts | cdleonard: if you unpack the rockbox.zip 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:19 | cdleonard | you mean the bootloader tries to load rockbox from the flash drive? |
11:30:28 | cdleonard | I thought it was just a smaller build with no plugins |
11:31:16 | Unhelpful | no, that is not what the bootloader is. :) |
11:32:04 | cdleonard | this should be in the wiki |
11:33:45 | cdleonard | so 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:48 | Unhelpful | the 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:13 | cdleonard | ok; 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:51 | Unhelpful | "replaces without overwriting"? |
11:40:33 | cdleonard | makes the device boot from a file on the PC instead |
11:40:43 | cdleonard | ignoring it's own private magic |
11:46:58 | | Join efyx_ [0] (n=efyx@lap34-1-82-224-140-171.fbx.proxad.net) |
11:48:27 | cdleonard | rockbox.org/index.php?topic=10164.msg109942#msg109942">http://forums.rockbox.org/index.php?topic=10164.msg109942#msg109942 |
11:48:53 | cdleonard | those links are broken; where can I go to RTFM for tcc77x? |
11:54:27 | linuxstb | cdleonard: 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:28 | linuxstb | cdleonard: So you still haven't got anything sensible displayed on the LCD? |
12:00 |
12:01:02 | cdleonard | no |
12:01:36 | linuxstb | Then 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:55 | cdleonard | I didn't try putting the normal build on the drive |
12:01:56 | linuxstb | The iAudio7 port is essentially unmaintained now,.. |
12:02:18 | cdleonard | I 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 (n=felixbru@h1252615.stratoserver.net) |
12:03:15 | linuxstb | Looking 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:19 | amiconn | Hmm, something in the new build system is borked |
12:04:11 | amiconn | One 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:04 | amiconn | Hmm, 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:40 | amiconn | Hmmm, and the other client died with 'duplicate name' before those two round, but *is* listed??? |
12:08:01 | * | amiconn summons Zagor |
12:08:42 | amiconn | Also, disconnects (for whatever reason) while a build is running doesn't clean up |
12:09:09 | | Join robin0800 [0] (n=robin080@81.98.157.181) |
12:09:37 | | Quit robin0800 (Client Quit) |
12:10:34 | | Join AndyIL [0] (i=AndyI@212.14.205.32) |
12:13:05 | gevaerts | amiconn: the latest builds also don't appear on rockbox.org/dev.cgi">http://build.rockbox.org/dev.cgi |
12:15:37 | amiconn | Oh, hmm, that explains it |
12:15:57 | amiconn | There'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@81.98.157.181) |
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@81.98.157.181) |
12:26:08 | | Join mcuelenaere [0] (n=mcuelena@78-21-191-122.access.telenet.be) |
12:26:43 | mcuelenaere | LambdaCalculus37 (logs): have you seen rockbox.org/tracker/task/10445">FS #10445? |
12:32:33 | | Quit robin0800 ("Leaving") |
12:32:51 | | Join robin0800 [0] (n=robin080@81.98.157.181) |
12:39:18 | | Join _zic [0] (n=user@83-156-153-127.rev.libertysurf.net) |
12:40:26 | | Join dfkt [0] (i=dfkt@unaffiliated/dfkt) |
12:54:03 | Grahack | mcuelenaere: remember rockbox.org/viewvc.cgi/trunk/apps/plugins/lua/liolib.c?r1=21715&r2=21714&pathrev=21715">http://svn.rockbox.org/viewvc.cgi/trunk/apps/plugins/lua/liolib.c?r1=21715&r2=21714&pathrev=21715 ? in fact we need this but rather if(!rb->file_exists(filename) && *mode == ('a'|'w')) |
12:54:06 | Unhelpful | amiconn: if you want to add anything to it or follow it, we have a bug report in for binutils now: http://sourceware.org/bugzilla/show_bug.cgi?id=10409 |
12:59:11 | *** | Saving seen data "./dancer.seen" |
13:00 |
13:01:21 | | Join mc2739 [0] (n=mc2739@cpe-67-10-234-29.satx.res.rr.com) |
13:02:03 | Unhelpful | amiconn: hrm, also, moving the first ". = ALIGN(16)" in .ncdata into .data after the contents seems to fix the weird offset bug. |
13:02:53 | mc2739 | Bagder: Zagor: The current builds page looks like it is stuck on r21884 |
13:03:12 | amiconn | But then it will introduce unnecessary padding on targets which don't use .ncdata |
13:03:31 | mcuelenaere | Grahack: hmm you're right |
13:03:50 | mcuelenaere | you can't do && *mode == ('a' | 'w') in C though ;) |
13:04:36 | Unhelpful | amiconn: not saying that's a fix we should use, just documenting the behavior in case it means anything to somebody else :) |
13:05:36 | Grahack | mcuelenaere: it compiled but didn't work, maybe if(!rb->file_exists(filename) && (*mode == 'a' || *mode == 'w')) ? |
13:06:06 | mcuelenaere | Grahack: yes it compiles, but it doesn't do the intended behaviour; *mode == 'a' || *mode == 'w' will work indeed |
13:07:46 | Torne | ('a'|'w') == 'w', in fact |
13:08:09 | Torne | by fluke |
13:09:55 | CIA-69 | New commit by mcuelenaere (r21918): Lua IOlib: when opening files for writing/appending, check if they exist and if not, add O_CREAT. |
13:10:13 | tmzt | Torne: binary? |
13:10:34 | tmzt | I mean bitwise |
13:10:41 | Grahack | ok, thanks for the C lesson guys. You see how I'm happy this Lua plugin exists ! and what is "by fluke" ? |
13:12:21 | mcuelenaere | Grahack: http://www.google.com/search?q=define:fluke&hl=en ;) |
13:14:21 | | Join mc2739_ [0] (n=mc2739@cpe-67-10-234-29.satx.res.rr.com) |
13:14:49 | | Quit mc2739 (Nick collision from services.) |
13:14:51 | | Nick mc2739_ is now known as mc2739 (n=mc2739@cpe-67-10-234-29.satx.res.rr.com) |
13:17:54 | | Join __lifeless [0] (n=lifeless@188.16.115.214) |
13:24:05 | petur | ah, buildsystem is working again... |
13:24:54 | petur | but rockbox.org/">http://build.rockbox.org/ still has old binaries |
13:25:37 | petur | erm.. 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] (n=ucchan@FLA1Adp241.kng.mesh.ad.jp) |
13:37:00 | | Join mc2739_ [0] (n=mc2739@cpe-67-10-234-29.satx.res.rr.com) |
13:37:16 | | Quit mc2739 (Nick collision from services.) |
13:37:18 | | Nick mc2739_ is now known as mc2739 (n=mc2739@cpe-67-10-234-29.satx.res.rr.com) |
13:45:02 | Torne | Grahack: "by chance" |
13:45:06 | Torne | though it's not chance, really |
13:45:11 | Torne | it's just bitwise or |
13:49:25 | | Join junker [0] (n=chatzill@host-233-121-3-96.midco.net) |
13:54:19 | tmzt | that applies to all ascii character on certain boundries right? |
13:56:20 | Unhelpful | tmzt: if the set bits in one are a subset of those in the other |
13:56:46 | tmzt | right, okay. sorry for the offtopic |
13:56:58 | tmzt | that would be a hard bug to track down though |
13:58:34 | | Quit mc2739 (Read error: 110 (Connection timed out)) |
14:00 |
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@59.96.92.46) |
14:05:39 | | Join dfkt_ [0] (i=dfkt@chello062178002170.1.11.univie.teleweb.at) |
14:05:52 | | Join _lifeless [0] (n=lifeless@188.16.121.178) |
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] (n=chatzill@ip216-239-79-254.vif.net) |
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@81.98.157.181) |
14:30:25 | GodEater | is 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@84.171.84.70) |
14:31:41 | | Join ReKleSS [0] (n=ReKleSS@114.78.149.10) |
14:35:20 | | Join dfkt [0] (i=dfkt@unaffiliated/dfkt) |
14:35:25 | petur | yes |
14:36:46 | GodEater | hrm |
14:36:59 | GodEater | my laptop keeps being told it has a score of zero the first time I bring it back into the build system |
14:37:37 | ReKleSS | could I get write permission on the wiki? (name is JeremyChin) |
14:37:43 | ReKleSS | I'd like to put up my H120 reflashing thing |
14:37:57 | ReKleSS | I sort of doubt anybody's actually going to attempt it, though... |
14:38:42 | GodEater | ReKleSS: done |
14:38:45 | ReKleSS | thanks |
14:38:54 | GodEater | you're welcome |
14:40:05 | * | GodEater prods linuxstb to update IPodNano2GPort |
14:40:26 | linuxstb | GodEater: Do I have to do _everything_.... ;) |
14:40:43 | GodEater | no, but you understand most of what's been done better than me :) |
14:41:05 | GodEater | I imagine you can probably copy and paste most of your forum post anyway |
14:44:21 | | Join wincent [0] (n=wincent@host-091-097-067-213.ewe-ip-backbone.de) |
14:48:03 | linuxstb | GodEater: 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@78.33.109.163) |
14:54:10 | GodEater | excellent news |
14:54:25 | * | GodEater wishes we had a message forwarding bot on linux4nano to send dev news this way |
14:54:37 | Torne | GodEater: isn't it called GodEater? |
14:54:43 | Torne | :) |
14:54:49 | GodEater | :P |
14:54:55 | GodEater | not in the last few days it hasn't been |
14:55:00 | GodEater | I'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] (n=chatzill@c-71-59-18-236.hsd1.ga.comcast.net) |
14:59:42 | GreatBeaver | hi |
15:00 |
15:00:11 | GreatBeaver | i 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 http://www.head-fi.org/forums/f6/iriver-h120-80gb-hdd-guide-434921/ |
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@119.224.12.185) |
15:02:11 | | Quit obo (Read error: 110 (Connection timed out)) |
15:05:34 | GodEater | GreatBeaver: we encourage people to post static content like guides to our wiki, not the forums. |
15:06:58 | | Join _zic [0] (n=user@83-156-153-127.rev.libertysurf.net) |
15:07:39 | | Quit _zic (Client Quit) |
15:07:55 | | Join _zic [0] (n=user@83-156-153-127.rev.libertysurf.net) |
15:07:58 | linuxstb | Maybe 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] (n=rob@78-69-30-229-no36.tbcn.telia.com) |
15:30:53 | | Quit ucchan (Read error: 104 (Connection reset by peer)) |
15:31:27 | | Join robin0800 [0] (n=robin080@81.98.157.181) |
15:32:17 | | Join evilnick [0] (i=0c140464@gateway/web/freenode/x-bdfd7fa976ec2ece) |
15:35:30 | LambdaCalculus37 | linuxstb: Sorry, that was my mistake. :) |
15:35:45 | * | linuxstb glares at LambdaCalculus37 ;) |
15:38:10 | linuxstb | LambdaCalculus37: Tried anything on your Nano yet? |
15:40:29 | LambdaCalculus37 | linuxstb: I got a logo on mine. :) |
15:41:18 | linuxstb | Was that before or after my change to support the second LCD type? |
15:41:36 | LambdaCalculus37 | After. |
15:41:41 | LambdaCalculus37 | Before, it didn't work. |
15:41:43 | GreatBeaver | linuxstb: could you do that for me please/ |
15:42:04 | GreatBeaver | i 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:28 | GodEater | Wikia ? |
15:43:04 | GreatBeaver | rockbox's wiki? |
15:43:08 | GreatBeaver | does rockbox have a wiki? |
15:43:28 | GodEater | yes, but it's not called wikia :) |
15:44:22 | GreatBeaver | what is it called? |
15:44:25 | gevaerts | the wiki |
15:44:53 | cdleonard | I managed to get my iAudio7 blinking whenever buttons are pressed; but lcd doesn't work |
15:45:11 | GreatBeaver | i cant find it |
15:45:11 | cdleonard | at least I know it doesn't hang or crash during init |
15:45:25 | linuxstb | cdleonard: Not even with older versions of Rockbox? |
15:46:14 | cdleonard | didn't try that |
15:46:18 | GreatBeaver | can someone link the rockbox wiki? |
15:46:22 | GreatBeaver | i keep searching for it but cant find |
15:47:17 | cdleonard | from rockbox.org/tracker/task/9245">http://www.rockbox.org/tracker/task/9245 I guess I should try r18435 |
15:47:18 | linuxstb | rockbox.org">http://www.rockbox.org then click on "wiki" in the left menu of every page |
15:48:47 | GreatBeaver | i 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:54:49 | GodEater | *half |
15:57:36 | | Join funman [0] (n=fun@rockbox/developer/funman) |
16:00 |
16:03:27 | | Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.") |
16:04:14 | | Join Ypsy [0] (n=ypsy@geekpadawan.de) |
16:04:24 | Ypsy | Hi there |
16:04:45 | Ypsy | Any Sansa Fuze users here? |
16:04:55 | linuxstb | Ypsy: You should just ask your question. |
16:05:35 | Ypsy | Well, 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:02 | linuxstb | Look in the "testing builds" forum |
16:06:04 | funman | there is a test build which deadlocks and corrupt the microsd content |
16:06:13 | linuxstb | And you'll find that.... |
16:06:21 | * | linuxstb doesn't think funman is selling it very well... |
16:07:22 | GodEater | he's just being honest |
16:07:29 | GodEater | he's clearly not cut out for a salesman job |
16:07:40 | Ypsy | Dont have a microsd yet so whatever :P |
16:07:59 | GodEater | I imagine you'll enjoy just the deadlocks then |
16:08:34 | | Join DerPapst [0] (n=DerPapst@p4FE8F00E.dip.t-dialin.net) |
16:08:43 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
16:08:46 | funman | i'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:09 | GodEater | funman: so it'll be an exciting journey for him then! |
16:09:45 | | Part Grahack |
16:09:52 | funman | a 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:52 | cdleonard | linuxstb: I tried r18435 and r19597 (another vitje commit) and got the same result: white screen on startup |
16:18:15 | cdleonard | backlight can be controlled but lcd doesn't seem to do anything useful |
16:18:46 | linuxstb | cdleonard: Is your iaudio 7 relatively new? It's not uncommon for manufacturers to change LCD types |
16:19:45 | cdleonard | it's a 16gb model; and it's reasonably new |
16:19:59 | | Join jgarvey [0] (n=jgarvey@cpe-098-026-065-013.nc.res.rr.com) |
16:20:04 | cdleonard | back says CWS-iAUDIO-7(B); note the (B) |
16:20:12 | Ypsy | Is 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:14 | GodEater | Ypsy: maybe, but who can say ? |
16:22:42 | GodEater | if the people working on it knew what to fix to get a stable one, they'd have fixed it already right ? |
16:23:02 | GodEater | but since they don't, they don't know how long it'll take to work out what the problems are. |
16:23:14 | GodEater | so it's impossible to guess when a stable build might turn up |
16:23:26 | Ypsy | okies, but they're still working on it, are they? |
16:23:38 | GodEater | funman is working like a slave :) |
16:23:48 | Ypsy | haha :) good to hear ;) |
16:23:50 | cdleonard | I used mktccboot with OF v1.18; while the original developer probably used 1.17; could that be a problem? |
16:24:24 | cdleonard | this new version is almost twice as big; which seems strange: http://www.cowonamerica.com/download/iaudio_7_jsfw.html |
16:25:04 | linuxstb | cdleonard: Yes, it's worth trying older OFs as well. |
16:25:40 | linuxstb | Although maybe not... Our code should be run before the OF, so that should affect things. |
16:26:03 | | Join aaron424 [0] (n=chatzill@adsl-065-013-002-216.sip.asm.bellsouth.net) |
16:26:27 | * | GodEater hands linuxstb a extra "not" |
16:27:13 | linuxstb | GodEater: Thankyou. |
16:27:37 | GodEater | any time |
16:27:40 | GodEater | I'm here to help |
16:28:27 | cdleonard | other than that I guess I have to disassemble the OF; right? |
16:29:24 | linuxstb | cdleonard: 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:44 | linuxstb | So the code in SVN may not even be working... |
16:30:45 | CIA-69 | New commit by mcuelenaere (r21919): Onda VX747: add dual-boot capability + make it possible to permanently 'stick' Rockbox to your DAP |
16:31:25 | cdleonard | buttons 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:42 | GodEater | can 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:13 | Zagor | GodEater: yes you can |
16:32:19 | GodEater | Zagor: thanks |
16:32:55 | | Quit funman ("free(random());") |
16:32:56 | amiconn | Zagor: 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@188.16.117.104) |
16:33:25 | Zagor | amiconn: do you have a time when it occurred last? |
16:33:33 | amiconn | Oh, and the client doesn't clean up if the connection gets lost (for whatever reason) while a build is running |
16:33:42 | gevaerts | ah yes, during last night all my clients got disconnected with duplicate-name issues |
16:34:03 | GodEater | mine too |
16:34:10 | amiconn | Zagor: Today, 07:11:23 CEST on jupiter |
16:34:21 | GodEater | 2009-07-17 06:13:48 Server refused connection: error duplicate name! |
16:34:31 | GodEater | GMT |
16:34:34 | GodEater | on atlas |
16:34:36 | | Join toffe82 [0] (n=chatzill@74.0.180.178) |
16:34:53 | amiconn | GodEater: Real GMT or GMT+1 for summertime? |
16:35:03 | GodEater | erm |
16:35:04 | Zagor | the good news is I've found and fixed the ping problems |
16:35:05 | GodEater | er |
16:35:09 | GodEater | uhm |
16:35:15 | GodEater | prolly GMT+1 |
16:35:22 | GodEater | he guessed wildly |
16:35:26 | amiconn | That's almost the same time as on mine, then |
16:35:31 | GodEater | yes |
16:35:34 | amiconn | Do you sync from NTP? |
16:36:06 | * | GodEater goes to check |
16:36:38 | GodEater | doesn't look like it |
16:36:42 | * | GodEater fixes |
16:37:02 | amiconn | So the difference could be just clock drift on your side, and it happened at the same time |
16:37:29 | GodEater | very likely |
16:37:56 | toffe82 | I 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:57 | GodEater | I have 15:37 here currently |
16:38:07 | | Join mc2739 [0] (n=mc2739@adsl-71-149-166-159.dsl.snantx.sbcglobal.net) |
16:38:21 | kugel | GMT then |
16:38:23 | toffe82 | the first time I had also a message , cannot read the playlist or something like this |
16:38:37 | kugel | GMT+1 (no summer time) |
16:38:48 | gevaerts | My 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:02 | mc2739 | Zagor: The current builds page looks like it has not updated since r21884 |
16:39:04 | kugel | does GMT include summertime :? |
16:39:37 | Zagor | mc2739: oh right, I'll fix that |
16:39:50 | GodEater | kugel: it's technically BST |
16:39:55 | GodEater | "British Summer Time" |
16:39:57 | toffe82 | my question is, why the power button doesn't work and pressing a button switch the lcd on and off ? |
16:40:29 | kugel | I guess we should refer to UTC instead? |
16:43:00 | gevaerts | kugel: is your client gone? |
16:43:07 | kugel | apparently |
16:43:25 | | Quit _lifeless (Read error: 110 (Connection timed out)) |
16:43:31 | kugel | it's taken off, until late august |
16:46:15 | kugel | haha, now other clients have the chance to feel how it is to have 100k points :) |
16:47:55 | gevaerts | that 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] (n=zelazko@unix.net.pl) |
16:49:17 | Zagor | amiconn: 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 (n=zelazko@unix.net.pl) |
16:50:10 | Zagor | the only thing I can suggest is that we instead of exit 22 waits maybe 10 seconds and retry |
16:50:37 | mcuelenaere | hmm could the VX747 bootloader red be a false alarm? |
16:51:19 | Zagor | mcuelenaere: the build log says "Build Status: Failed" so it definitely is something wrong. exactly what is unclear to me though, |
16:51:36 | mcuelenaere | Zagor: perhaps the 'mipsel-elf-gcc: no input files ' line? |
16:51:52 | Zagor | full unparsed log: rockbox.org/data/21919-ondavx747boot.log">http://build.rockbox.org/data/21919-ondavx747boot.log |
16:52:07 | Zagor | mcuelenaere: probably |
16:53:11 | Zagor | actually, no. the "failed" is written by rbclient if the specified output file isn't produced |
16:53:38 | mcuelenaere | ah, how does it determine the specified output? |
16:53:38 | kugel | gevaerts: why did youß |
16:53:43 | mcuelenaere | (I renamed the bootloader to ccpmp.bin) |
16:53:55 | Zagor | mcuelenaere: it's listed in the "builds" file. in this case it is "rockboot.vx747" |
16:54:05 | kugel | gevaerts: what 150k do you mean (my machine had a rating of 230k) |
16:54:40 | Zagor | mcuelenaere: if the rename is permanent, please change the builds file too |
16:55:08 | mcuelenaere | Zagor: www/buildserver/builds ? |
16:55:15 | Zagor | mcuelenaere: yes |
16:56:49 | CIA-69 | New commit by mcuelenaere (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:00 |
17:01:05 | amiconn | Zagor: Maybe double the retry interval every time? This way a real duplcate wouldn't cause too much server load |
17:01:58 | amiconn | Hmm, or maybe exit if the client gets a "duplcate name" message on startup, but not on retry |
17:02:32 | Zagor | Every 10 seconds won't cause any measurable server load either. |
17:03:24 | Zagor | I'm adding logging to the next client update, to easier see what the client is doing |
17:04:20 | | Join mnk200 [0] (n=mankind@bl13-17-153.dsl.telepac.pt) |
17:05:55 | | Join Xerion [0] (i=xerion@82-170-197-160.ip.telfort.nl) |
17:07:02 | | Join dfkt [0] (i=dfkt@unaffiliated/dfkt) |
17:07:21 | gevaerts | kugel: that 230k was before rounds got faster |
17:08:33 | kugel | gevaerts: I don't mean the score on the client page |
17:09:00 | kugel | I mean the "230000 puts you in the fast category" |
17:09:28 | gevaerts | kugel: yes, but that's the average score |
17:10:20 | kugel | ? |
17:11:13 | CIA-69 | New commit by zagor (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:22 | gevaerts | kugel: don't worry. I also have no idea what we're actually talking about :) |
17:15:55 | kugel | gevaerts: so which 150k did you mean? |
17:16:11 | gevaerts | kugel: the sum of rb* |
17:16:59 | | Join roolku [0] (n=roolku@77-99-113-75.cable.ubr16.sgyl.blueyonder.co.uk) |
17:18:05 | roolku | Zagor: did you see my coment from last night: the build system didn't spot the failed builds when images are missing such as: rockbox.org/shownewlog.cgi?rev=21908;type=mrobe100">http://build.rockbox.org/shownewlog.cgi?rev=21908;type=mrobe100 |
17:19:08 | Zagor | roolku: thanks, i'll look at tat |
17:19:09 | Zagor | that |
17:19:10 | roolku | Zagor: 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:27 | Zagor | yes last nights server wasn't my best :) |
17:19:34 | kugel | gevaerts: I don't think they'll just sum up |
17:19:53 | gevaerts | kugel: no, but they would have 100k even without your client stopping |
17:20:00 | kugel | sure ;) |
17:20:05 | kugel | blah blah :P |
17:20:11 | CIA-69 | New commit by zagor (r21922): Bugfixed socket data reading. ... |
17:20:42 | kugel | Why are you switching to single client? Didn't you "prove" yourself that multiple clients are better? |
17:22:35 | | Join rockbox [0] (n=anilbpai@59.92.203.122) |
17:22:49 | rockbox | hi |
17:22:53 | rockbox | newbie here |
17:23:01 | GodEater | original name though |
17:23:28 | | Quit petur ("beer time!") |
17:23:32 | rockbox | how can i get started with ? |
17:23:55 | GodEater | where "?" is what exactly? |
17:25:18 | markun | roolku: what are you trying to do? |
17:25:34 | markun | eh, rockbox .. |
17:25:55 | markun | roolku: 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] (n=bryanjac@e33.cs.rochester.edu) |
17:42:04 | kugel | gevaerts: do you have an opinion on the dev-ml discussion about funman's commit? It's your code... |
17:44:52 | gevaerts | not really, except that I'm not convinced that those functions belong in the plugin api |
17:45:23 | gevaerts | spinnung up a disk because you're going to read from an sd card soon is not optimal |
17:45:59 | GodEater | esp. if there's no disk there |
17:47:18 | gevaerts | that'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:49 | amiconn | ata_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:19 | saratoga | linuxstb: 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:49 | amiconn | And 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:50 | funman | what about disk_spin() ? (more explicit on its usage) |
17:50:18 | gevaerts | amiconn: sure, but what about a player with both a hard drive and some flash thing? |
17:50:25 | amiconn | What about it? |
17:50:39 | gevaerts | what do the functions do? |
17:51:09 | amiconn | The same as they do on single storage, I'd expect |
17:51:18 | funman | in 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:35 | gevaerts | funman: not sure :) How about USB? |
17:51:57 | amiconn | gevaerts: Can you control usb hdd spinup/spindown from the host? |
17:52:12 | funman | gevaerts: what's the link with USB ? |
17:52:52 | gevaerts | amiconn: not sure, but I'd expect so |
17:53:20 | gevaerts | funman: usb host. More disks |
17:53:41 | funman | I still don't get it, sorry |
17:54:02 | gevaerts | "only the internal storage would be a disk" is not true for those |
17:54:46 | gevaerts | amiconn: 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:50 | funman | I 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:20 | amiconn | Not atm, no. But I'd go KISS first, and just apply it to all drives. We can still add differentiation later |
17:59:04 | funman | So 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:00 |
18:01:23 | amiconn | No, the purpose of the multi-storage layer is that the app doesn't need to know that |
18:02:43 | gevaerts | funman: what's the point of separating it? |
18:03:17 | amiconn | What could be added later is a way to track which storage the app uses, and only apply *_spin() to that one |
18:03:21 | funman | i have explained that in my mail to rockbox-dev i think |
18:03:46 | gevaerts | funman: not in a way that makes it clear to me |
18:03:48 | funman | separate 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:19 | amiconn | In theory, yes (not counting cf mods) |
18:04:44 | amiconn | It doesn't hurt to use ata_spin() and ata_sleep() on a cf - it just doesn't change anything |
18:05:13 | funman | it annoys me when i read the code |
18:06:01 | gevaerts | funman: so you want to basically rename storage_spin() to disk_spin()? |
18:06:03 | funman | what is bad with making this code really dependant on HAVE_DISK_STORAGE ? (which is the only time when it has an effect) |
18:06:17 | funman | gevaerts: yes |
18:06:23 | linuxstb | saratoga: Sorry, I don't. preglow might... |
18:06:33 | amiconn | funman: More ifdefs for nothing |
18:06:34 | gevaerts | funman: the problem with the disk_*() names I see is that it (IMHO) really doesn |
18:06:55 | gevaerts | t belong in disk.c, while the names would imply that. I don't know how important this si |
18:06:56 | linuxstb | saratoga: Or you could try searching the IRC logs - I'm sure it's been discussed here |
18:06:57 | funman | amiconn: clear separation IMO |
18:07:24 | amiconn | disk_ is entirely different module, related to handling partitions |
18:07:26 | gevaerts | funman: HAVE_DISK_STORAGE means that there is at least one disk, not that the driver you're looking at deals with disks |
18:07:37 | funman | gevaerts: well disk.c is wrongly named as well, i didn't know where to put these new disk_spin functions |
18:07:53 | gevaerts | funman: spindle_spin()? :) |
18:08:08 | amiconn | And storage_ abstracts from the storage types. So having *_spin() and *_sleep() for all drivers actually provides better abstraction |
18:08:25 | gevaerts | Anyway, I think they do belong in storage.c, because they make use of the storage.c indirection layer |
18:08:57 | funman | what about keeping this discussion on the ML where it started? |
18:09:17 | funman | i'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:47 | amiconn | Well you need those stubs anyway for the plugin api |
18:11:01 | gevaerts | that was the sticking point, yes |
18:11:03 | funman | only when they are needed at all? |
18:11:32 | * | amiconn wonders why funman doesn't understand the advantage of less ifdefing |
18:11:46 | gevaerts | funman: so you'd add #ifdef HAVE_DISK_STORAGE around the stubs? |
18:12:04 | funman | amiconn: i do understand them, do you understand the advantage of less stubs and code separation ? |
18:12:14 | amiconn | Why 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:15 | funman | gevaerts: no, i already did |
18:12:27 | amiconn | This *is* code separation |
18:12:46 | funman | the storage, or the disk ? |
18:12:57 | gevaerts | how would the app know? |
18:13:03 | amiconn | Whether 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:37 | amiconn | Yeah, 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:06 | amiconn | In fact that might not be necessary |
18:15:17 | gevaerts | the storage rework patch is already quite complex. I'd like to separate it in different bits, not make it bigger |
18:15:48 | amiconn | The 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:29 | amiconn | ata_sleep() otoh is meant to be sent whenever some buffering ends, in order to save timeout_seconds of disk power |
18:16:35 | funman | i think removing this special case for hard disks makes things simpler |
18:16:55 | amiconn | So that can be broadcast as well - it is very likely that only one disk is spinning if there are several |
18:17:05 | amiconn | What special case???? |
18:17:21 | funman | disk spinning / sleeping |
18:17:31 | amiconn | ahem... |
18:17:31 | funman | removing it, *from the storage layer* |
18:17:51 | funman | I wonder if you have noticed r21912 and its discussion on the dev ML |
18:17:53 | mcuelenaere | Can RBUtility debug builds still be builded? |
18:17:56 | * | gevaerts sees that funman missed some stubs anyway :) |
18:17:57 | amiconn | Rockbox strives to save power, so it is *necessary* for the app layer to tell the storage layer about these cases |
18:18:39 | funman | amiconn: not to the storage layer, but to the harddisk-specific driver |
18:19:04 | gevaerts | funman: 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:25 | amiconn | funman: The app layer shouldn't have to care whether this is hdd or some other storage |
18:20:07 | funman | well using storage_sleep() implies you know about hard disks |
18:20:41 | amiconn | Yes, but you don't need to know whether it actually *is* a hard disk |
18:21:23 | funman | I don't know what is "quick sleep", or "timeout" |
18:22:32 | funman | since you are writing this code specifically for the case if there is a harddisk, why not mentioning it explicitely? |
18:22:56 | amiconn | Better abstraction *and* less ifdefing - I already said that |
18:23:33 | amiconn | The only "drawback" is calling an empty stub on non-hdd targets, which means a whopping 8 bytes of binsize |
18:24:42 | funman | there is also the drawback of seeing the stubs defined which was more my real concern, rather than saving 8 bytes |
18:25:05 | amiconn | What drawback is that? |
18:25:18 | funman | Seeing the stubs annoy me |
18:25:57 | amiconn | Then don't look... why stubs that provide better abstraction would be annoying is beyond me, really |
18:26:23 | amiconn | They are not the only stubs in rockbox - sometimes stubs are the best soltion |
18:26:23 | funman | Here I don't see the need for abstraction |
18:26:45 | funman | Ok you convinced me |
18:27:26 | * | kugel finds it sad that amiconn never bothers to express his opinion on the ml |
18:27:45 | funman | me too |
18:28:13 | amiconn | irc is so much more convenient. And it's logged, so no disadvantage related to that |
18:28:31 | funman | mailing list is threaded |
18:28:34 | | Join shotofadds [0] (n=rob@rockbox/developer/shotofadds) |
18:29:03 | shotofadds | I think we should re-implement the whole damn lot in C++ and hide the 'stubs' in the abstract storage driver base class ;-) |
18:29:15 | amiconn | eurgh |
18:29:20 | shotofadds | *much* better |
18:32:45 | kugel | amiconn, 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:15 | kugel | I saw that if a inline is non-static, gcc inlines but also creates the function body (so that it's addressable) |
18:33:19 | funman | i think you can't export a static function |
18:33:33 | amiconn | funman: inline, not static inline |
18:34:09 | amiconn | Hmm, actually it could be static inline in the .h. Gcc should include a body in plugin.c in this case |
18:34:37 | CIA-69 | New commit by funman (r21923): Revert r21912 : "Storage API : remove undeeded target-specific functions" ... |
18:34:43 | CIA-69 | New commit by funman (r21924): remove nand_soft_reset and nand_disk_is_active already voided in storage.h |
18:34:47 | kugel | it doesn't create a body at all, IIUC, if it's static. At least that's what I saw in chopper |
18:35:08 | amiconn | ata.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:12 | kugel | oh, the front page is quick today |
18:37:14 | amiconn | kugel: It will create a body as soon as you take the function address |
18:37:58 | kugel | ah, I understand. but it still wouldn't work for ata.c? |
18:38:20 | amiconn | ata.h would define it non-static as normal |
18:38:25 | funman | the declaration could be different for each storage type |
18:42:00 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
18:49:13 | | Join obo [0] (n=obo@rockbox/developer/obo) |
18:58:41 | funman | I'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:06 | funman | I believe this is a problem with the as3514 code, there could be incompatible differences with the audio codec used in as3531 |
18:59:13 | kugel | funman: maybe a panic before lcd init? |
18:59:17 | *** | Saving seen data "./dancer.seen" |
18:59:26 | funman | ah thanks i didn't about that |
18:59:49 | kugel | did the lcd actually work on the clipv2? |
18:59:59 | funman | yes |
19:00 |
19:02:51 | | Join DarkSpectrum [0] (n=ZX@c-67-167-179-42.hsd1.mi.comcast.net) |
19:03:45 | | Join readability [0] (n=chad@206.248.173.89) |
19:05:42 | kugel | funman: I'm getting the fuzev2 in ~1 month, probably |
19:06:07 | kugel | I hope the lcd driver is similar to the v1, I might be able to implement it then |
19:06:56 | funman | have you looked in the OF if the init procedure differ |
19:06:57 | funman | ? |
19:07:10 | kugel | no, not yet |
19:08:12 | | Join aditya [0] (n=aditya@59.96.92.46) |
19:10:25 | | Quit obo (Remote closed the connection) |
19:10:40 | | Join obo [0] (n=obo@77-99-230-49.cable.ubr04.trow.blueyonder.co.uk) |
19:12:54 | kugel | funman: so you're in touch with a uclinux guy that made code for AMS? |
19:13:19 | shotofadds | regarding 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:31 | kugel | agree |
19:13:43 | shotofadds | so... with that in mind, is there anything else that needs to be done to rockbox.org/tracker/task/9545">FS #9545 before it can be committed? |
19:13:45 | kugel | commit early, commit often :) |
19:14:43 | * | shotofadds wonders about the high-score possibilities of this patch :) |
19:14:59 | shotofadds | maybe I'll leave it to gevaerts ;-) |
19:15:09 | kugel | many .h files, generally a good way to break things :) |
19:15:23 | | Join FlynDice [0] (n=FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
19:15:52 | kugel | does it remove the uber-guly IF_MV stuff also? |
19:15:58 | * | kugel looks |
19:16:14 | kugel | yea, IF_MV is IF_MD now :S |
19:16:49 | gevaerts | kugel: not in all places :) |
19:16:59 | gevaerts | There's IF_MV() and IF_MD() now! |
19:17:04 | kugel | ughh... |
19:18:52 | * | gevaerts seems to remember that there were still some open issues |
19:19:28 | funman | kugel: i contacted the person working at AMS who works on linux and uboot (not uclinux) ports |
19:19:45 | funman | He told me he couldn't do anything and advised me to contact the product marketing manager |
19:20:08 | funman | the one who stopped answering my emails some month ago (and is still continuing, i have no answer to my yesterday's mail) |
19:20:15 | kugel | gevaerts: could drive be a part of struct storage_info? |
19:20:45 | gevaerts | why? |
19:20:59 | kugel | funman: I wonder if at least 1 sd controller is still the pl180 one |
19:21:19 | kugel | gevaerts: wouldn't it make sense and save a parameter to many functions? |
19:21:30 | funman | kugel: what do you mean ? |
19:21:51 | gevaerts | kugel: what functions? struct storage_info isn't passed to any functions... |
19:21:55 | kugel | either the internal or external one (that of course applies to the fuzev2= |
19:21:56 | kugel | ) |
19:22:12 | kugel | gevaerts: "void mmc_get_info(IF_MD2(int drive,) struct storage_info *info)" |
19:22:30 | funman | kugel: 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:31 | gevaerts | kugel: that's to get information *from* a drive |
19:22:49 | funman | the person from AMS let me know the SD controller is made by synopsys |
19:22:53 | kugel | gevaerts: so what? |
19:23:00 | kugel | funman: both? |
19:23:09 | gevaerts | kugel: what exactly are you proposing? |
19:23:12 | kugel | I mean, IIUC the 3525 ones use 2 controllers |
19:23:26 | kugel | gevaerts: have a int drive member in struct storage_info |
19:23:36 | gevaerts | kugel: again, why? |
19:23:43 | kugel | and anyhow get rid of IF_MV :) |
19:23:43 | funman | kugel: 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:53 | gevaerts | How does that help you do that? |
19:24:07 | funman | kugel: the storage_info struct isn't filled when given to this function |
19:24:26 | kugel | it will be filled, but the drive number could be filled before |
19:24:30 | funman | only allocated |
19:24:42 | kugel | whatever, nevermind |
19:24:59 | gevaerts | kugel: so yes, you could remove the IF_MD() from *_get_info(). What does that gain you? |
19:25:10 | gevaerts | It's still everywhere else |
19:25:12 | kugel | less uglyness |
19:25:13 | | Join LambdaCalculus37 [0] (i=44a0430d@gateway/web/freenode/x-c4584eed183a0db2) |
19:25:22 | gevaerts | more inconsistency |
19:25:23 | funman | kugel: if you want to work on it, rockbox.org/tracker/task/10047">fs#10047 has a description of all registers i could understand, and working code for sending commands to the card |
19:25:29 | gevaerts | i.e. *more* uglyness |
19:26:01 | kugel | I'm also not seeing that saving a drive parameter is worth obfuscicating code (and pre-processing function paramters is obfuscicating imo) |
19:26:07 | kugel | obfuscating* |
19:26:17 | mcuelenaere | LambdaCalculus37: seen rockbox.org/tracker/task/10445">FS #10445? |
19:26:59 | kugel | funman: if the microsd is still "powert by" pl180, I could hopefully boot rockbox off the microsd |
19:28:38 | kugel | gevaerts: what do you not understand? |
19:28:51 | | Join nls [0] (n=n1s@cm-84.215.127.139.getinternet.no) |
19:28:55 | funman | kugel: i didn't see pl180 code |
19:29:24 | funman | and 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:25 | gevaerts | kugel: 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:07 | kugel | it would be part of the plan to get rid of IF_MV completely :) |
19:30:24 | * | gevaerts gives up |
19:30:30 | kugel | funman: is the firmware part of the OF big enough? |
19:30:35 | kugel | gevaerts: still not clear? |
19:30:46 | kugel | get a coffee! :p |
19:31:11 | LambdaCalculus37 | mcuelenaere: No, I haven't. What is it? |
19:31:44 | mcuelenaere | LambdaCalculus37: didn't you commit r21908? |
19:31:56 | LambdaCalculus37 | mcuelenaere: I did. |
19:32:08 | kugel | mcuelenaere: the guy filled his flyspray info wrongly |
19:32:34 | LambdaCalculus37 | mcuelenaere: 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__ (n=felixbru@h1252615.stratoserver.net) |
19:32:42 | mcuelenaere | kugel: you mean his name? I think it's correct |
19:32:43 | funman | kugel: for clipv2 yes (80kb left) |
19:32:56 | * | bertrik has an idea |
19:33:00 | kugel | mcuelenaere: it ends with i in his flyspray info too |
19:33:11 | mcuelenaere | isn't it supposed to be ending with an i? |
19:33:13 | bertrik | I have this firmware image and I wonder if there's a filesystem image in it |
19:33:27 | kugel | mcuelenaere: oh, I read it the other way around |
19:33:31 | kugel | nevermind then |
19:33:31 | bertrik | I guess I could use one of those lost file / partition scanners to find it |
19:33:54 | funman | bertrik: perhaps look at hal/mount for filesystem signatures detection |
19:34:02 | mcuelenaere | anyway, it's a minor thing :) |
19:34:05 | kugel | funman: 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:21 | kugel | or in the firmware file |
19:34:22 | bertrik | funman, what is hal/mount? |
19:34:27 | funman | kugel: mkamsboot |
19:34:43 | funman | bertrik: hal is a freedesktop library / daemon for hardware detection |
19:34:53 | kugel | is 80K big enough? |
19:35:03 | funman | mount is a standard unix command to mount filesystems (it can detect filesystem types) |
19:35:19 | bertrik | but mount can't scan a file, or can it? |
19:35:28 | funman | kugel: http://pastie.org/549620 |
19:35:47 | funman | bertrik: 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 (n=ypsy@geekpadawan.de) |
19:36:11 | bertrik | too much work, I'd rather try a program already made for that |
19:37:14 | kugel | funman: 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:54 | gevaerts | kugel: 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:49 | kugel | I 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:06 | gevaerts | so you didn't even look at the code?> |
19:40:20 | kugel | I skimmed the diff and looked at the struct definition |
19:40:23 | | Quit einhirn ("Miranda IM! Smaller, Faster, Easier. http://miranda-im.org") |
19:40:43 | kugel | and grepped firmware/* |
19:40:46 | | Join einhirn [0] (n=Miranda@bsod.rz.tu-clausthal.de) |
19:41:09 | | Join fdinel [0] (n=Miranda@modemcable204.232-203-24.mc.videotron.ca) |
19:41:10 | gevaerts | shotofadds: before it can be committed, things like NUM_VOLUMES, MAX_NUM_DRIVES need to be cleaned up |
19:42:00 | bertrik | ok, can't find anything in the firmware image with testdisk |
19:42:26 | LambdaCalculus37 | Argh, why am I having trouble restoring the HDD1630? I'm trying to put it into recovery mode to restore Rockbox onto it. |
19:42:32 | LambdaCalculus37 | toffe82: Ping |
19:42:41 | LambdaCalculus37 | Or lowlight: Ping |
19:42:50 | toffe82 | LambdaCalculus37: pong |
19:43:01 | kugel | funman: btw, before mkamsboot 1.1, the (c) should be updated |
19:43:06 | | Quit einhirn (Client Quit) |
19:43:24 | LambdaCalculus37 | toffe82: Do you know how to get service and recovery mode working on the HDD1630? I'm using the instructions here: rockbox.org/twiki/bin/view/Main/GoGearHDD6330#Service_Mode_I_found_2_methods_I">http://www.rockbox.org/twiki/bin/view/Main/GoGearHDD6330#Service_Mode_I_found_2_methods_I |
19:43:27 | funman | kugel: i'm not sure large modifications were made this year |
19:43:28 | LambdaCalculus37 | But nothing works. |
19:43:34 | gevaerts | kugel: 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:37 | kugel | it doesn't matter |
19:43:50 | kugel | gevaerts: no, you read my sentences wrong then |
19:44:01 | funman | i would just remove the copyright notice if linuxstb agrees |
19:44:20 | kugel | why? |
19:44:35 | | Join stoffel [0] (n=quassel@p57B4DA3B.dip.t-dialin.net) |
19:44:39 | funman | because it is useless |
19:44:43 | gevaerts | kugel: 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:37 | toffe82 | LambdaCalculus37: it is not on the wiki of the 1630 ? |
19:45:42 | | Quit fdinel (Client Quit) |
19:46:24 | toffe82 | I though there was 2 separate pages ??? |
19:46:30 | | Join fdinel [0] (n=Miranda@modemcable204.232-203-24.mc.videotron.ca) |
19:46:48 | LambdaCalculus37 | toffe82: Just the one page AFAIK. |
19:47:29 | toffe82 | it is not in the service manual ? |
19:47:33 | | Quit fdinel (Client Quit) |
19:47:53 | | Join fdinel [0] (n=Miranda@modemcable204.232-203-24.mc.videotron.ca) |
19:48:23 | linuxstb | funman: I've no objection |
19:48:32 | linuxstb | (but no opinion either way...) |
19:49:33 | * | kugel is curious what funman is going to answer (test build thread) |
19:51:06 | CIA-69 | New commit by funman (r21925): mkamsboot: remove runtime copyright notice ... |
19:51:30 | funman | kugel: 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:10 | LambdaCalculus37 | toffe82: I have the USB mass storage image on the screen. |
19:52:23 | LambdaCalculus37 | But what appears in Disk Utility is a read-only device. |
19:52:55 | | Quit mnk200 (Client Quit) |
19:53:02 | kugel | funman: 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:14 | LambdaCalculus37 | Wait... switching to my Windows box got it to appear as a mass storage device. |
19:53:32 | | Join fdinel [0] (n=Miranda@modemcable204.232-203-24.mc.videotron.ca) |
19:53:35 | kugel | I query'd with him, fsck reported errors on the internal partition, and crapped out due to 512 filesystems on the microsd |
19:53:35 | LambdaCalculus37 | ARGH... no drive appears, though. |
19:53:41 | toffe82 | LambdaCalculus37: :) |
19:54:07 | toffe82 | I think it is really dependant of the time you press the buttons |
19:54:24 | funman | kugel: 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] (n=kachna@r4ax178.net.upc.cz) |
19:55:24 | funman | perhaps 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:00 | kugel | SD as in what? the card, the controller or our driver? |
19:57:20 | gevaerts | kugel: 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:15 | funman | kugel: card and/or controller (hardware) |
19:58:45 | CIA-69 | New commit by dave (r21926): Improvments to Nano 2G LCD driver. This now works reliably on both LCD types. |
19:59:00 | kugel | funman: the guy got his FS corruption before installing rockbox IIUC |
19:59:08 | kugel | gevaerts: yea, that's true |
19:59:22 | gevaerts | that was by accident actually :) |
20:00 |
20:00:16 | kugel | hehe |
20:00:23 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
20:03:23 | kugel | funman: he installed the bootloader but couldn't install rockbox.zip because of read-only filesystem |
20:08:54 | kugel | something 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:51 | funman | kugel: help welcome |
20:10:08 | | Join stoffel_ [0] (n=quassel@p57B4DA3B.dip.t-dialin.net) |
20:10:43 | kugel | gevaerts: how different is a drive from a volume with that patch? |
20:11:12 | gevaerts | kugel: totally. That's one of the points. A drive is really a physical drive, while a volume is a filesystem |
20:11:42 | kugel | does it already allow more than 2 filesystems in rockbox? |
20:11:57 | gevaerts | sure, if you set NUM_VOLUMES high enough |
20:12:17 | | Nick dys` is now known as dys (n=andreas@krlh-5f7069bf.pool.einsundeins.de) |
20:12:36 | dys | is there a way to disable the nvram.bin-feature without recompiling? |
20:12:37 | gevaerts | which is one of the other issues. What's a good default setting for NUM_VOLUMES? It's not NUM_VOLUMES_PER_DRIVE... |
20:12:47 | kugel | re: 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:09 | kugel | defined in config-* |
20:13:26 | * | dys wants to prevent unneccessary writes to his precious 32GB cf card |
20:13:30 | gevaerts | kugel: it doesn't use MAX_NUM_DRIVES for single-driver targets, so that bit is easy :) |
20:13:32 | kugel | not sure about volumes though |
20:14:02 | gevaerts | but yes, defining the number of drives in config-* sounds reasonable |
20:14:18 | kugel | gevaerts: I was thinking to do it similar as with dual-core which has also a NUM_CORES define |
20:14:59 | kugel | I think for single core it's actually set in config.h (#ifndef NUM_CORES \n #define NUM_CORES 1 \n #endif) |
20:15:26 | kugel | there's no HAVE_DUAL_CORE #define |
20:15:59 | gevaerts | kugel: 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:47 | gevaerts | We could add extra code to allow single-volume-per-drive multidrive, but I don't really like that either |
20:16:54 | kugel | I'm thinking that, at least for internal storage, NUM_VOLUME could be determined at runtime |
20:17:16 | gevaerts | you can't. The space needs to be allocated at boot |
20:17:30 | kugel | yes, exactly |
20:17:48 | kugel | detect volumes at boot and allocate space before showing the main menu |
20:17:57 | kugel | i.e. what tagcache and the like do |
20:18:22 | gevaerts | kugel: 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:11 | gevaerts | Maybe 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:18 | kugel | the first filesystem could be statically allocated (there must be at least 1 volume anyway) |
20:19:50 | gevaerts | you'll need more complicated data structures then. Right now it's just an array |
20:19:58 | kugel | the 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:43 | kugel | any further volume could be perfectly allocated from the audio buffer |
20:20:44 | LambdaCalculus37 | mcuelenaere: So zenutils can extract the Dell DJ bootloader? |
20:20:46 | mcuelenaere | LambdaCalculus37: it can also split the .bin file |
20:20:54 | LambdaCalculus37 | mcuelenaere: A good sign? :) |
20:20:57 | funman | why would you need several partitions if they all are fat32 ? |
20:21:14 | mcuelenaere | LambdaCalculus37: let's see if it has the keys to decrypt the main firmware ) |
20:21:15 | mcuelenaere | :) |
20:21:38 | gevaerts | kugel: 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:47 | kugel | I agree fully |
20:22:32 | gevaerts | no point in saving 200 bytes of RAM if you need 1K of code to do that |
20:23:55 | kugel | and external drives would still be problematic |
20:24:11 | kugel | they would at least require stopping playback if it was detected at runtime |
20:24:21 | gevaerts | ah yes. That too... |
20:25:47 | | Quit kugel (Remote closed the connection) |
20:26:04 | mcuelenaere | LambdaCalculus37: seems like these older firmwares don't even use 'real' encryption |
20:26:34 | mcuelenaere | I can send you the decrypted/splitted firmware if you want |
20:26:55 | LambdaCalculus37 | mcuelenaere: Sure, email it over. |
20:27:04 | LambdaCalculus37 | Do you need my email address? |
20:27:09 | mcuelenaere | I'll upload it |
20:27:19 | LambdaCalculus37 | Okay. |
20:28:41 | mcuelenaere | http://www.mediafire.com/?zxgkingmntu |
20:29:01 | mcuelenaere | for a rough description of what most of the files mean, see rockbox.org/twiki/bin/view/Main/CreativeZVMPort">http://www.rockbox.org/twiki/bin/view/Main/CreativeZVMPort |
20:29:12 | | Quit funman ("free(random());") |
20:29:16 | mcuelenaere | (FBOOT & FRESC aren't encrypted) |
20:29:34 | mcuelenaere | CENC.dec is the decrypted version of CENC |
20:31:25 | | Quit BryanJacobs ("Java user signed off") |
20:31:36 | gevaerts | How much RAM do the v2 sansas have? |
20:31:44 | Llorean | Depends on the target |
20:32:03 | gevaerts | I know. I want to know all of them :) |
20:33:32 | * | gevaerts is trying to evaluate the cost of various multivolume options |
20:39:55 | LambdaCalculus37 | mcuelenaere: I need to get a disk dump of the Dell DJ as well. |
20:40:17 | mcuelenaere | LambdaCalculus37: why? the file system is probably the same/an older version as the ZVM |
20:40:37 | mcuelenaere | LambdaCalculus37: does the Dell DJ communicate with a PC over MTP or NJB? |
20:40:51 | LambdaCalculus37 | mcuelenaere: NJB IIRC. |
20:41:00 | mcuelenaere | ah then it uses the older file system I think |
20:41:20 | LambdaCalculus37 | mcuelenaere: I wanted to be sure. |
20:43:17 | gevaerts | OK. The total cost per volume on targets that already have more than one drive is 104 bytes |
20:44:25 | gevaerts | That 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:04 | JdGordon| | SCREW bin/ram costs.... keep the damn code simple and eat the other costs |
20:48:19 | JdGordon| | seriously... its less than 1KB... |
20:48:32 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
20:49:02 | kugel | JdGordon|: I agree |
20:49:43 | * | gevaerts tends to agree as well, but he still did want to know the numbers first |
20:49:50 | kugel | how does the filebrowser present the volumes? |
20:50:09 | gevaerts | I don't remember :\ |
20:50:12 | | Join Zarggg [0] (n=zarggg@65-78-69-194.c3-0.eas-ubr6.atw-eas.pa.cable.rcn.com) |
20:50:26 | kugel | I hope no driver letters :) |
20:50:37 | kugel | C:\rockbox :p |
20:50:52 | JdGordon| | <microSD%d> |
20:51:06 | JdGordon| | or <Violume%d> iirc |
20:51:26 | | Quit n00b81 ("Leaving") |
20:51:46 | gevaerts | OK. For Ondio the unmodified rockbox.org/tracker/task/9545">FS #9545 has zero ramsize cost |
20:51:58 | gevaerts | so those 624 bytes are really all of them |
20:53:07 | kugel | is this with num_drives*4? |
20:53:23 | gevaerts | yes, but done by hand for now |
20:53:36 | gevaerts | i.e. #define NUM_VOLUMES 8 |
20:54:20 | kugel | yea, the 600bytes are damn worth it then |
20:54:40 | pixelma | what for? |
20:55:09 | kugel | pixelma: scroll back a bit |
20:56:35 | pixelma | well... "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:42 | gevaerts | pixelma: the storage rework patch (rockbox.org/tracker/task/9545">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 rockbox.org/tracker/task/9545">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:22 | gevaerts | So 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:12 | bertrik | the YP-S3 LCD init looks quite similar to other LCDs already in rockbox, but there's not an exact match it seems |
20:58:30 | gevaerts | on 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:50 | pixelma | as if someone would partition a 128MB disk, except accidentally.... ;) |
20:59:18 | *** | Saving seen data "./dancer.seen" |
20:59:22 | itcheg | kugel: any idea how/why using a FM transmitter on an Ipod via IAP can garble the next tag? |
20:59:33 | gevaerts | pixelma: there is that :) |
20:59:51 | kugel | itcheg: No, no idea. I have never worked with any of that |
21:00 |
21:00:20 | itcheg | ok, was asking more from the wps perspective |
21:00:33 | kugel | itcheg: JdGordon| may have an idea though, he reworked next-track stuff a bit few month ago |
21:00:58 | itcheg | hmm 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] (n=ori@bzq-79-178-135-87.red.bezeqint.net) |
21:02:16 | itcheg | kugel: can you tell me off hand if there is anything wrong with this: %s%alNext: %Ia;%t4%s%alNext: %?It<%It|%Fn> |
21:02:21 | salty-horse | is there a reason why the clock plugin is so big? (144KB) |
21:02:35 | JdGordon| | pretty images |
21:03:35 | itcheg | JdGordon|: any idea how next tag can be affected by an ipod accessory? |
21:03:59 | itcheg | that is when used total nonsense shows |
21:04:13 | JdGordon| | no idea |
21:04:28 | JdGordon| | are you actually changing the track? or just having it connect |
21:04:50 | itcheg | I guess I can file a bug, although I have no idea what info to include to make it usefull |
21:04:56 | gevaerts | pixelma: I'm probably extrapolating from how I would behave. I guess you're right, and this won't be a problem anyway |
21:06:20 | JdGordon| | itcheg: extaclty when is the next track info getting garbeled? always? or only after changing tracks with it? or? |
21:06:36 | kugel | itcheg: no a new bug report, we have a task that collects IAP bugs |
21:07:07 | itcheg | It's not always have not been able to figure out exactly, but it only happens when I'm using the accessory |
21:07:56 | itcheg | I used the iap patch before it was committed some months back and never had an issue |
21:08:15 | kugel | JdGordon|: I suspect your playback events |
21:08:20 | kugel | +wor |
21:08:23 | kugel | work* |
21:08:29 | itcheg | so not sure if something changed in iap or (more likely?) the whole wps change |
21:08:32 | itcheg | or other |
21:08:58 | * | gevaerts has seen a garbled next track on his gigabeat F a few days ago... |
21:09:12 | JdGordon| | ... are you changing the track from the transmitter? |
21:10:51 | | Join mirak [0] (n=mirak@85-169-192-191.rev.numericable.fr) |
21:13:15 | | Join dash32 [0] (n=dash32@p54AB5446.dip.t-dialin.net) |
21:15:20 | | Join _zic [0] (n=user@83-156-153-127.rev.libertysurf.net) |
21:18:01 | amiconn | gevaerts: The volume pool issue already exists in svn |
21:18:30 | gevaerts | oh 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:38 | amiconn | Imo it would be a good solution to limit both the total number of volumes and the number of volumes per drive |
21:18:54 | amiconn | The latter limit should default to 1 imo |
21:19:36 | kugel | IIUC the targets defines the drive count? |
21:19:40 | gevaerts | shouldn't be too hard. Just have disk_mount() stop when it reaches the limit |
21:19:59 | kugel | oh, volume per drive |
21:20:19 | | Quit mirak ("Ex-Chat") |
21:22:00 | gevaerts | hm, this is easier than I expected. Just add an "&& mounted < VOLUMES_PER_DRIVE" to the main for loop in disk_mount() |
21:22:45 | gevaerts | amiconn: what do you think about the global number of volumes being limited to just NUM_DRIVES*VOLUMES_PER_DRIVE? |
21:22:52 | JdGordon| | 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:27 | gevaerts | JdGordon|: 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:01 | gevaerts | Or do we need a separate total maximum number of volumes? |
21:26:15 | | Join Zagor [242] (n=bjst@rockbox/developer/Zagor) |
21:31:42 | itcheg | JdGordon|: 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:04 | itcheg | I will try and discern a pattern |
21:33:58 | itcheg | it happens a lot but not always |
21:34:00 | | Join nls [0] (n=n1s@cm-84.215.127.139.getinternet.no) |
21:35:10 | | Quit stoffel (Remote closed the connection) |
21:35:43 | bertrik | gevaerts, markun do you remember which id you got from the meizu m6 lcd? |
21:36:05 | gevaerts | 0x0154 IIRC |
21:36:48 | bertrik | ok 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:33 | gevaerts | bertrik: this is on m6sl by the way |
21:41:45 | | Join mt [0] (n=mt@41.233.138.250) |
21:42:44 | Zagor | does the iriver g120 have car adapter mode? |
21:42:47 | Zagor | h120 |
21:43:15 | bertrik | gevaerts, the LCD I'm looking at now seems to expect 0x1503 back from register 0 |
21:44:39 | nls | Zagor: the h300 at least starts the OF on AC power if off, dunno about h100 |
21:45:34 | mt | saratoga: 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:54 | bertrik | not completely sure if it's really the LCD id, could also be some kind of verification of a previously written data |
21:45:55 | Zagor | nls: starts OF, i.e does not boot into rockbox? |
21:46:18 | bertrik | is it usual for LCDs to have an LCD ID register at address 0? |
21:46:54 | GreatBeaver | how come running time doesnt work on my rockbox h120 atm? |
21:47:05 | GreatBeaver | problem with 3.3? |
21:47:09 | nls | Zagor: 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:19 | Zagor | nls: ok |
21:47:37 | itcheg | JdGordon|: before I go crazy testing scenarios can you confirm that this is ok: %s%alNext: %Ia;%t4%s%alNext: %?It<%It|%Fn> |
21:48:13 | JdGordon| | i see nothing obviously wrong... I thought %s cant go in sublines... but i have no idea |
21:49:06 | gevaerts | What happened to sd_disk_is_active()? I'm sure it did exist half a year ago |
21:49:52 | bertrik | weren't the xxx_is_active calls made obsolete with the xxx_idle_notify framework? |
21:50:03 | JdGordon| | that was the idea |
21:50:08 | | Join kugel_ [0] (n=kugel@e178114251.adsl.alicedsl.de) |
21:50:18 | | Quit kugel (Nick collision from services.) |
21:50:20 | GreatBeaver | anyone know why run timer isnt working on my rockbox for h120? |
21:50:20 | JdGordon| | it didnt quite work 100% though |
21:50:22 | | Nick kugel_ is now known as kugel (n=kugel@e178114251.adsl.alicedsl.de) |
21:50:25 | bertrik | maybe it was me who removed them now I think about it |
21:50:27 | GreatBeaver | it was working for 5 hours and then it just froze |
21:50:42 | GreatBeaver | i am afraid i damaged something physically in the H120 while modding is that possible? |
21:50:47 | itcheg | ok so I'll add the %s to the subline as well just to make sure... |
21:51:11 | gevaerts | Things still call storage_is_active() |
21:51:51 | gevaerts | Anyway, I can work around that. I don't need them |
21:51:54 | nls | GreatBeaver: yes it is possible to physically damage a player |
21:52:06 | GreatBeaver | thats not what imean |
21:52:18 | GreatBeaver | i mean could the timer have stopped because i damaged something in the player? |
21:56:29 | CIA-69 | New commit by zagor (r21927): Added average score and % cancelled time. |
21:56:37 | Llorean | GreatBeaver: 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:23 | GreatBeaver | Llorean: 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:27 | GreatBeaver | its been stuck at 0 0 0 |
21:57:34 | Llorean | Is the charger plugged in? |
21:57:36 | GreatBeaver | but it was working for 5 hours of time since yesterday |
21:57:40 | GreatBeaver | hmmm yes |
21:57:59 | GreatBeaver | let me try disconnecting |
21:58:39 | | Join w1ll14m [0] (n=54685215@gateway/web/cgi-irc/labb.contactor.se/x-dde9f832bda86e0b) |
21:59:07 | GreatBeaver | cool thanks it works now |
21:59:09 | | Quit LambdaCalculus37 () |
21:59:12 | GreatBeaver | how come it turns off when the charger is on? |
21:59:21 | Llorean | Again, I reference you to the manual |
21:59:46 | Llorean | It should be in there, I hope. |
21:59:49 | GreatBeaver | i did try looking in the manual |
22:00 |
22:00:08 | w1ll14m | is there a way to use strncpy from a codec (recent changes to svn removed strncpy from codeclib) |
22:00:13 | kugel | JdGordon|: could you implement a touchscreen region for going to the playlist from wps? |
22:00:20 | * | bertrik intends to commit rockbox.org/tracker/task/10445">FS #10445 - Error in CREDITS (a mis-spelled name) |
22:01:23 | Llorean | Hm, Running Time isn't in the menu |
22:01:25 | Llorean | Er, manual |
22:01:37 | Llorean | GreatBeaver: Running Time is meant to tell you how long it's been running since a charger was last detected. |
22:01:39 | Llorean | Nothing's broken |
22:01:45 | GreatBeaver | oh ok |
22:01:52 | kugel | w1ll14m: use strlcpy |
22:03:37 | w1ll14m | kugel: can i call that function like strcpy or do i have to change some? |
22:04:46 | CIA-69 | New commit by bertrik (r21928): rockbox.org/tracker/task/10445">FS #10445 - Error in CREDITS |
22:05:07 | kugel | if 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:16 | funman | w1ll14m: you can check the description on http://www.manpagez.com/man/3/strlcpy/ |
22:05:24 | w1ll14m | funman, kugel: thanx ;) |
22:07:14 | pixelma | I didn't even notice I added my buildclient in the middle of a build round |
22:07:42 | pixelma | not that it will be any usefull now that the "easy" builds are done |
22:09:18 | pixelma | but the system dealt with it correctly :) |
22:10:10 | w1ll14m | kugel: to define strlcpy i have to use #define strlcpy ci->strlcpy right ? |
22:10:17 | | Join notlistening [0] (n=tom@94-195-105-95.zone9.bethere.co.uk) |
22:12:22 | notlistening | FlynDice, 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:38 | bertrik | what 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:09 | notlistening | yeah 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:24 | notlistening | *card |
22:16:54 | | Quit stoffel_ (Remote closed the connection) |
22:17:32 | notlistening | on my e2xxv2 it decreases my runtime from 18 hours or so to 10 |
22:20:24 | | Join mc2739 [0] (n=mc2739@rrcs-71-40-9-100.sw.biz.rr.com) |
22:22:06 | funman | w1ll14m: yes or you can use ci->srtlcpy directly |
22:22:18 | w1ll14m | funman: thanx |
22:23:10 | gevaerts | there'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:45 | mc2739 | Zagor: 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:01 | Zagor | mc2739: because uploads are done in the background |
22:29:50 | mc2739 | Zagor: is the upload counted in the client total, but not the build total? |
22:30:08 | Zagor | exactly |
22:30:27 | mc2739 | ah, thanks |
22:33:33 | | Join petur [50] (n=petur@rockbox/developer/petur) |
22:37:48 | nls | w1ll14m: you'll need to add strlcpy to the codec api |
22:38:40 | w1ll14m | nls: thanx, i just found out, i copied strlcpy from plugin.h/c |
22:39:04 | nls | yeah that should do it |
22:39:20 | nls | what codec needs this btw? |
22:39:56 | w1ll14m | there is this mikmod patch on flyspray |
22:40:14 | nls | ah |
22:40:15 | w1ll14m | it used strncpy |
22:40:28 | w1ll14m | also a nice moment to learn a little more about c |
22:40:43 | CIA-69 | New commit by robert (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] (n=chatzill@p5DDC7DFB.dip.t-dialin.net) |
22:44:39 | * | linuxstb wonders why a codec would need strncpy |
22:45:49 | JdGordon| | kugel: should be dead simple to add... as long as there is already an ACTION_ for it... |
22:46:07 | JdGordon| | otherwise is will need a bit of extra work which is why some are missing |
22:47:36 | amiconn | gevaerts: 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:00 | amiconn | Bootloader USB works both on the hub and laptop |
22:48:08 | gevaerts | hm |
22:48:19 | amiconn | It seems the beast is drawing more than it says (or maybe even more than 500mA) when charging |
22:48:39 | amiconn | The hub doesn't like this - it goes connect-disconnect-connect-disconnect.... |
22:48:39 | gevaerts | oh, does your hub have overcurrent protection? |
22:48:56 | amiconn | After a number of cycles, rockbox freezes |
22:49:25 | kugel | JdGordon|: there is |
22:49:41 | JdGordon| | so its a 1 line change in wps_parser.c if you want to do it |
22:50:04 | amiconn | gevaerts: Looks like it does, as *should* and uplink port... |
22:50:28 | gevaerts | so there are two problems : (a) it draws more than the hub likes (probably), and (b) it freezes after enough connect-disconnect cycles |
22:50:35 | amiconn | yes |
22:50:42 | gevaerts | amiconn: well, like so many things it's optional in the USB spec |
22:51:25 | gevaerts | It would be good to verify that it's really the charging that causes this |
22:52:21 | amiconn | It 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:26 | kugel | JdGordon|: hm, I'll look into it |
22:52:46 | gevaerts | yes, 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:52:56 | amiconn | hmm |
22:53:00 | | Quit mcuelenaere () |
22:55:37 | Zagor | bah, what did I muck up this time? |
22:56:18 | pixelma | I wondered about the order things happened |
22:57:30 | w1ll14m | what are the chances that strlcpy will be added to codeclib in the future ? |
22:57:54 | gevaerts | w1ll14m: if you can show you need it, 100% I guess |
22:59:22 | *** | Saving seen data "./dancer.seen" |
22:59:38 | Zagor | ah, bad regex |
22:59:47 | w1ll14m | gevaerts: thanx |
22:59:53 | JdGordon| | bad is redundant :) |
23:00 |
23:00:00 | Zagor | :) |
23:00:07 | B4gder | regex rocks!1 |
23:02:45 | Zagor | there, all green again |
23:02:46 | CIA-69 | New commit by gevaerts (r21930): Allow access to the last sector of the ramdisk as well |
23:03:29 | itcheg | TheSeven: Thank you, I'm not much of a programmer, (can read some assembler) but am fascinated by the project |
23:03:40 | itcheg | woops wrong channel |
23:03:49 | TheSeven | itcheg: read it nevertheless |
23:03:56 | | Join faemir [0] (n=faemir@78.33.109.163) |
23:03:59 | itcheg | haha |
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:39 | amiconn | gevaerts: ramdisk? |
23:07:44 | gevaerts | yes |
23:08:00 | gevaerts | easiest way to test multidriver |
23:08:08 | amiconn | Yes, atm |
23:08:23 | * | amiconn wants to see the tpj port going, and also usb host |
23:08:27 | gevaerts | unless it's buggy of course :) |
23:08:36 | | Join r0b- [0] (n=rob@adsl-76-235-176-134.dsl.klmzmi.sbcglobal.net) |
23:08:41 | saratoga | mt: 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:03 | kugel | gevaerts: doesn't it start with 0? |
23:09:11 | r0b- | ok according to the hardware info for the E200 the cpu that it uses runs 200MIPS is that accurate? |
23:09:14 | saratoga | some 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:21 | saratoga | r0b: no |
23:09:29 | r0b- | whats ir run then |
23:09:31 | r0b- | it* |
23:09:38 | saratoga | 2x80MHz |
23:09:43 | kugel | I'd think it would be sector 0 to sector MAX-1 (MAX would be 1 too much) |
23:09:58 | r0b- | i know that but im talking in MIPS :P |
23:10:02 | | Quit evilnick ("Page closed") |
23:10:04 | saratoga | MIPS is meaningless |
23:10:09 | r0b- | oh ok |
23:10:11 | saratoga | particularly for ARMv4 |
23:10:32 | saratoga | but it can't do 100MIPS anyway no matter how badly you fudge the numbers |
23:10:34 | linuxstb | bertrik: ping |
23:10:39 | | Quit mc2739 ("ChatZilla 0.9.85 [Firefox 3.0.11/2009060215]") |
23:10:43 | r0b- | thanks saratoga |
23:10:48 | gevaerts | kugel: it does, yes. Assume NUM_SECTORS==2, and read(1,1). That's allowed! |
23:11:27 | saratoga | r0b: for codecs and such, number of multiply or multiply-adds per clock is more interesting usually |
23:11:57 | gevaerts | kugel: the difference is this "+count" bit |
23:12:33 | kugel | gevaerts: isn't it an array[NUM_SECTORS]? |
23:13:00 | gevaerts | kugel: NUM_SECTORS*SECTOR_SIZE |
23:13:24 | kugel | start = NUM_SECTORS-1 (the last element of that array), count = 1 would overflow the array |
23:13:30 | kugel | (1 being a sector) |
23:13:35 | gevaerts | i.e. a read(1,1) will read 512 bytes starting from 512 |
23:14:01 | r0b- | well another thing is that unless the e200 changed hardware my rockbox says mine has the PP5022C chip |
23:14:11 | gevaerts | kugel: work out some examples in the real code, not simplified cases that disregard SECTOR_SIZE |
23:14:29 | saratoga | shotofadds: 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:53 | saratoga | r0b: I think thats normal |
23:15:01 | | Quit _zic (Remote closed the connection) |
23:15:06 | saratoga | that field is just what the hardware reports its name as I htink |
23:15:28 | | Join dash32 [0] (n=dash32@p54AB6AF3.dip.t-dialin.net) |
23:15:29 | amiconn | r0b-: The PP5024 is basically a PP5022 with tacked-on AS3514, and it reports itself as PP5022 |
23:15:38 | r0b- | oh ok |
23:15:59 | kugel | gevaerts: in the if there's no sector size. |
23:16:28 | gevaerts | kugel: no, but in the array declaration there is |
23:16:42 | amiconn | Zagor: Ahem, current build page only has the sources? |
23:16:44 | kugel | start and count are in sectors, aren't they? |
23:16:47 | gevaerts | yes |
23:16:50 | shotofadds | saratoga: 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:14 | Zagor | amiconn: haha, lovely |
23:17:17 | gevaerts | kugel: assume an array "char a[64]". Are you allowed to read one byte starting from offset 63? |
23:17:35 | saratoga | shotofadds; which rework is this? i thought the problem was lack of a FTL? |
23:17:36 | kugel | yes |
23:17:41 | B4gder | 36 clients is quite nice |
23:18:05 | kugel | you're actually allowing to read from offset 64 too |
23:18:10 | gevaerts | kugel: that's basically what's happening here. start+count=64, even if you're 0 based |
23:18:11 | Zagor | amiconn: fixed |
23:18:18 | | Quit jgarvey ("Leaving") |
23:18:26 | | Join Galois [0] (i=djao@efnet.math.uwaterloo.ca) |
23:18:30 | r0b- | 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/labb.contactor.se/x-790e25ec920775c8) |
23:19:03 | kugel | ah, now I understand |
23:19:22 | gevaerts | kugel: it's not simple. that bug was there for a reason :) |
23:19:36 | kugel | hehe |
23:19:39 | shotofadds | saratoga: I meant the multi-storage-driver rework (rockbox.org/tracker/task/9545">fs#9545) which is needed to provide access to both NAND and SD. |
23:19:58 | shotofadds | obviously there's no need for this to produce an SD-only build |
23:20:35 | kugel | gevaerts: yea, sorry about my confusion. start+count is the end, not the start :/ |
23:21:05 | B4gder | Zagor: I think using total = total/6 + current/6 should work fine enough |
23:21:31 | saratoga | shotofadds: ah ok |
23:21:37 | B4gder | hm well not exactly that, but you get mu point |
23:21:48 | amiconn | moo point ;) |
23:21:49 | Zagor | B4gder: yes :) |
23:22:17 | * | B4gder might have gotten one too many glasses of wine to be spelling correctly ;-) |
23:22:21 | Zagor | B4gder: who makes rockbox.org/dist/build-source/rockbox.7z">http://build.rockbox.org/dist/build-source/rockbox.7z ? it's not updated for the new build. |
23:22:27 | kugel | gevaerts: looks like we have a similar bug in our sd driver |
23:22:35 | B4gder | lemme check |
23:22:53 | kugel | http://pastie.org/549926 |
23:23:06 | kugel | (samsa sd driver) |
23:23:35 | B4gder | Zagor: the previous buildmaster.pl script invoked ./mksource after each build round |
23:23:43 | gevaerts | kugel: hm, that looks suspiciously similar indeed |
23:24:01 | B4gder | Zagor: present in my ~/rockbox-distbuild |
23:24:25 | | Quit Jaykay ("ChatZilla 0.9.85 [Firefox 3.5/20090624025744]") |
23:24:53 | kugel | and it looks like this part could actually have dangerious side effects |
23:25:50 | Zagor | B4gder: ok, I'll run it then |
23:27:24 | | Quit stripwax (Read error: 110 (Connection timed out)) |
23:28:08 | linuxstb | bertrik: Good news - your Meizu i2c driver is working perfectly on the Nano 2G. |
23:28:23 | saratoga | Zagor: any reason why my build server's directory has 5700 directories in its root? |
23:28:38 | LambdaCalculus37 | linuxstb: Can I pick the first song for audio playback? :) |
23:28:43 | Zagor | saratoga: !! |
23:28:44 | saratoga | they'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:18 | B4gder | I 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:20 | linuxstb | LambdaCalculus37: It could come soon - I've already located audio codec functions in the diagmode code, and bertrik got sound on his Meizu... |
23:29:27 | saratoga | oh hmm they're pretty old, so maybe this bug has been fixed |
23:29:34 | saratoga | looks like since a week ago they stopped being made |
23:29:39 | saratoga | sorry for the false report! |
23:29:43 | Zagor | saratoga: ah, that sounds good |
23:29:54 | LambdaCalculus37 | linuxstb: Then in that case... |
23:30:09 | Zagor | though 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:56 | amiconn | Zagor: 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:02 | saratoga | Zagor: 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:25 | Zagor | saratoga: you need one per kernel. the build dir is named after the pid |
23:35:13 | Zagor | amiconn: ok |
23:37:23 | kugel | gevaerts: this could be a major bug |
23:38:09 | kugel | if it's crossing banks, transfer is 1 sector too small, i.e. this sector will not be filled with data |
23:38:34 | gevaerts | that would provide some interesting data corruption issues |
23:38:44 | kugel | although, as the same thing happens on reading, this sector should be skipped as well. |
23:39:03 | gevaerts | not better... |
23:39:18 | gevaerts | i.e. you get whatever was in the buffer |
23:39:21 | | Quit nls ("Lämnar") |
23:39:33 | kugel | but we would probably handle files created by the of (any music, rockbox itself, due to missing usb) incorrectly |
23:40:44 | | Join Zarggg_ [0] (n=zarggg@65-78-69-194.c3-0.eas-ubr6.atw-eas.pa.cable.rcn.com) |
23:41:33 | * | kugel commits the fix |
23:42:03 | | Join aaron424 [0] (n=chatzill@adsl-065-013-002-216.sip.asm.bellsouth.net) |
23:42:11 | gevaerts | kugel: were there still known data corruption issues? |
23:42:29 | | Join PaulJam [0] (i=Paule@vpn-3027.gwdg.de) |
23:42:57 | kugel | well, 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:26 | kugel | but yea, some people are still reporting fs problems |
23:43:42 | roolku | amiconn: 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:05 | amiconn | Hmm, not another device with broken hdd. |
23:44:10 | roolku | :) |
23:44:21 | amiconn | My iPod Photo just ate its drive today :( |
23:44:34 | pixelma | CF mod? |
23:44:34 | * | linuxstb has a working Elio that I would be glad to donate to anyone.... |
23:44:40 | Zagor | holy cow. rockbox-clip.7z is a whopping 63% of rockbox-clip.zip! |
23:44:47 | gevaerts | roolku: you mean there are actually two of those in existence? |
23:44:58 | linuxstb | roolku: We should reunite them... |
23:45:08 | amiconn | Same 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:13 | roolku | you think they might breed? |
23:45:17 | Zagor | I know saratoga mentioned yesterday 7z was more efficient, but I didn't think the difference was this huge. |
23:45:30 | pixelma | gevaerts: and you have the other one of the two Logik Dax(e)s? |
23:45:50 | kugel | hmm |
23:45:52 | gevaerts | pixelma: yes, but those have cousins, the ATMT-170s! |
23:45:57 | B4gder | Zagor: zip is crap and 7zip is crazy good |
23:45:59 | amiconn | And I'm not using it much - that is certainly not due to mechanical stress |
23:46:12 | kugel | gevaerts: thinking about it again, it > or >= doesn't make any difference |
23:46:20 | kugel | in this case |
23:46:24 | Zagor | B4gder: we should use 7z for the uploaded targets. it will save us quite a lot of time. |
23:46:27 | amiconn | linuxstb: Iirc bootloader boots, but there is no lcd output yet? |
23:46:46 | B4gder | Zagor: yes, at least if bandwidth is the limiting factor and not the cpu |
23:46:47 | kugel | for the == case, transfer won't actually change |
23:46:50 | amiconn | Zagor: Do we want to repack on the server? Or wait for rbutil to support 7z? |
23:47:03 | Zagor | (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:28 | roolku | amiconn: if it is the same type of drive as the h120, I have a spare 20GB one |
23:47:36 | Zagor | bleh, I forgot about rbutil |
23:47:42 | amiconn | It's a dual platter Photo |
23:47:56 | B4gder | switching to 7z might work as a subtle push on the rbutil team ;-) |
23:48:10 | Zagor | haha |
23:48:36 | * | gevaerts wants to know why the file browser doesn't want to see his ramdisk |
23:48:44 | amiconn | B4gder, Zagor: Any idea how long repacking would take? The saved upload time might still make up for that |
23:48:47 | PaulJam | kugel: 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:03 | kugel | yea, I always forget to take those out :( |
23:49:15 | Zagor | amiconn: 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:30 | pixelma | 7zip could potentially be a bit more inconvenient for the average user (especially if RbUtil doesn't support it yet) |
23:50:30 | gevaerts | i.e. make the download links 7z, but make zips for rbutil until there's been a new rbutil for a few weeks |
23:50:32 | B4gder | is it really that many people that install the bleeding edge builds with rbutil? |
23:50:35 | | Quit faemir ("Leaving") |
23:50:39 | CIA-69 | New commit by kugel (r21931): Apply the same fix as r21930 did for the ramdisk for the AMS Sansa driver. ... |
23:50:47 | linuxstb | amiconn: Yes, quite a few things work (standard PP) - ATA, most of the buttons (simple gpio). But no LCD. |
23:51:19 | linuxstb | amiconn: And I seem to recall it's got a wolfson codec, so the audio driver should be pretty similar to the others. |
23:51:27 | amiconn | yeah |
23:51:35 | amiconn | LCD is 220x176 iirc? |
23:51:42 | linuxstb | Yes, 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:00 | amiconn | Do we have access to the OF for disassembly? |
23:52:06 | kugel | weeh |
23:52:08 | linuxstb | Yes, do you want it? |
23:52:15 | kugel | since when is the front page picking up new commits so quickly? |
23:52:36 | * | amiconn is interested in both |
23:52:38 | gevaerts | kugel: it used to be a cron job, it's a post-commit hook now I think |
23:53:04 | Zagor | gevaerts: no it's still a cronjob, we just run it more often |
23:53:09 | gevaerts | ah |
23:53:24 | * | gevaerts promises to never soread misinformation again |
23:53:34 | Zagor | :) |
23:53:45 | B4gder | the trigger for starting builds is not used for updating the front page |
23:54:05 | kugel | did you just not bother enough to use a post-commit hook or is it hard to do with the front page? |
23:54:18 | pixelma | gevaerts: what about spreading it though? |
23:54:23 | Zagor | B4gder: it could though. I just added a "roundstart" script which could do just that |
23:54:29 | gevaerts | pixelma: I don't know about that yet :) |
23:54:33 | B4gder | yes it could |
23:54:46 | B4gder | but then I'd need to care for starting before the previous ended etc |
23:54:53 | Zagor | kugel: the svn and web servers are different machines, so a post-hook needs to communicate with the www server somehow |
23:54:58 | B4gder | just using a cronjob is much simpler |
23:56:47 | shotofadds | gevaerts: 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] (n=johannes@dslb-094-220-231-176.pools.arcor-ip.net) |
23:57:42 | | Join tarbo_ [0] (n=me@unaffiliated/tarbo) |
23:57:44 | gevaerts | shotofadds: D2 would be multidriver, while e200 would only be multidrive |
23:57:54 | gevaerts | Are you working on the patch now? |
23:58:05 | gevaerts | If so, maybe we need to coordinate a bit |
23:58:21 | shotofadds | no, I'm not changing it at the moment |
23:58:32 | gevaerts | ok. I'm safe then |