| 00:00:36 | | Quit bluebrother (Read error: 113 (No route to host)) |
| 00:01:27 | | Quit stripwax ("http://miranda-im.org") |
| 00:03:15 | | Join joshin [0] (n=joshin@VDSL-130-13-176-10.PHNX.QWEST.NET) |
| 00:04:38 | | Quit baunegaard () |
| 00:05:38 | | Part joshin |
| 00:07:23 | * | amiconn wants that big fat mega-trout for slapping kugel |
| 00:07:35 | | Quit dfkt_ ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.") |
| 00:08:16 | | Join stripwax [0] (n=Miranda@87.194.34.169) |
| 00:09:11 | | Quit ender` (" To err is human; to moo bovine.") |
| 00:09:29 | CIA-43 | New commit by amiconn (r22903): Fix crash in early USB introduced in r22365 by moving the gui_syncstatusbar_init() back to where it needs to be. The comment clearly stated that it ... |
| 00:10:11 | | Quit dfkt (Read error: 110 (Connection timed out)) |
| 00:12:20 | saratoga | stripwax: i just used the ffmpeg code directly for the wma unrolling optimization |
| 00:12:28 | saratoga | i'm not sure if memset would be any better |
| 00:12:46 | saratoga | yeah thats why i walked back to my office and am cooling off before driving home |
| 00:12:56 | saratoga | opps |
| 00:13:17 | amiconn | This bug is present in release 3.4, and it affects all targets with hardware usb, just with differing probability of showing up |
| 00:14:46 | | Join Blue_Dude [0] (n=chatzill@adsl-235-222-153.mco.bellsouth.net) |
| 00:15:29 | pixelma | I think there is a release notes page in the wiki, it should probably at least be put there |
| 00:16:06 | | Quit Blue_Dude (robinson.freenode.net irc.freenode.net) |
| 00:16:06 | NSplit | robinson.freenode.net irc.freenode.net |
| 00:16:06 | | Quit janne_ok1anen (robinson.freenode.net irc.freenode.net) |
| 00:16:06 | | Quit Slasheri (robinson.freenode.net irc.freenode.net) |
| 00:16:11 | | Quit janne_oksanen (Killed by sagan.freenode.net (Nick collision)) |
| 00:16:11 | | Join janne_oksanen [0] (n=janne@b01.ainola.jyu.fi) |
| 00:16:12 | NHeal | robinson.freenode.net irc.freenode.net |
| 00:16:12 | NJoin | janne_ok1anen [0] (n=janne@b01.ainola.jyu.fi) |
| 00:16:32 | NJoin | Blue_Dude [0] (n=chatzill@adsl-235-222-153.mco.bellsouth.net) |
| 00:18:14 | | Join froggyman_ [0] (n=chatzill@pool-72-69-88-109.chi01.dsl-w.verizon.net) |
| 00:18:14 | | Join Slasheri [0] (i=miipekk@xen.ihme.org) |
| 00:18:21 | | Quit froggyman (Nick collision from services.) |
| 00:18:34 | | Nick froggyman_ is now known as froggyman (n=chatzill@pool-72-69-88-109.chi01.dsl-w.verizon.net) |
| 00:27:15 | amiconn | b38-roolku seems to have a buggy native compiler |
| 00:27:27 | | Quit stripwax (Read error: 110 (Connection timed out)) |
| 00:27:33 | | Join faemir [0] (n=faemir@cpc2-cmbg2-0-0-cust30.cmbg.cable.ntl.com) |
| 00:27:50 | amiconn | Imo it would be best to disable it completely, but at least everything built natively should be disabled |
| 00:27:56 | amiconn | (sims, checkwps) |
| 00:29:01 | amiconn | The reason for disabling it completely is that the crosscompilers might not be reliable if they're built using this buggy native gcc. But they were probably built quite a while ago |
| 00:29:29 | mc2739 | 23:19:33CIA-43New commit by 03gevaerts (r22902): block roolku-b38, which seems to have trouble compiling sh targets |
| 00:30:14 | mc2739 | I think the build server needs to update for it to take |
| 00:30:21 | pixelma | it's only blocked for sh targets then, not native things (if I understand correctly) |
| 00:31:09 | amiconn | That doesn't make sense |
| 00:31:34 | mc2739 | The way I understand, the client will be blocked for all builds |
| 00:32:27 | | Quit DerPapst ("Leaving.") |
| 00:32:41 | amiconn | Yeah, looks like it can only do complete blocks |
| 00:32:51 | amiconn | I wonder why the buildserver doesn't grab it |
| 00:33:17 | mc2739 | I think Zagor needs to update / restart the buildmaster |
| 00:33:37 | amiconn | meh |
| 00:33:39 | pixelma | but why did the client get builds with r22903? |
| 00:33:41 | | Quit JackWinter (Read error: 104 (Connection reset by peer)) |
| 00:34:00 | amiconn | What good is that blocklist if one of the Swedes still needs to handle it manually? |
| 00:34:23 | mc2739 | good question |
| 00:34:50 | | Quit Topy44 (Read error: 104 (Connection reset by peer)) |
| 00:35:06 | pixelma | I also thought this system was introduced to have some power over the system without them being around |
| 00:35:34 | | Join JackWinter [0] (n=jack@vodsl-10804.vo.lu) |
| 00:37:59 | | Join Topy44 [0] (n=Topy44@f048211076.adsl.alicedsl.de) |
| 00:38:46 | *** | Saving seen data "./dancer.seen" |
| 00:38:57 | | Quit Sajber^ (Read error: 104 (Connection reset by peer)) |
| 00:43:49 | pixelma | maybe this system was broken with the server move and just no-one noticed so far because it wasn't used until today? |
| 00:45:50 | | Quit robin0800 (Remote closed the connection) |
| 00:47:45 | | Quit faemir (Read error: 110 (Connection timed out)) |
| 00:53:36 | | Join jpt9 [0] (n=jpt9@seel-25.dynamic.rpi.edu) |
| 00:53:37 | jpt9 | Hey. |
| 00:53:40 | jpt9 | 3.4 is *awesome*! |
| 00:53:46 | jpt9 | Love the USB HID support. |
| 00:56:12 | | Quit matsl (Read error: 110 (Connection timed out)) |
| 01:00 |
| 01:02:15 | | Join mt [0] (n=MTee@rockbox/developer/mt) |
| 01:03:49 | | Nick fxb is now known as fxb__ (n=felixbru@h1252615.stratoserver.net) |
| 01:08:06 | | Quit Blue_Dude ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
| 01:10:23 | | Quit AsaelReiter ("CGI:IRC") |
| 01:17:23 | | Join FlynDice [0] (n=FlynDice@239-126-174-206.static.gci.net) |
| 01:17:57 | | Quit bertrik (Read error: 113 (No route to host)) |
| 01:20:05 | | Quit mt2 (Read error: 104 (Connection reset by peer)) |
| 01:20:29 | | Join mt2 [0] (n=mtee@41.233.155.27) |
| 01:23:12 | | Quit Thundercloud (Remote closed the connection) |
| 01:30:42 | | Join faemir [0] (n=faemir@cpc2-cmbg2-0-0-cust30.cmbg.cable.ntl.com) |
| 01:34:53 | | Quit faemir (Client Quit) |
| 01:39:35 | | Quit bluebrot13r ("leaving") |
| 01:42:42 | | Part jpt9 ("Leaving") |
| 01:47:07 | | Quit arohtar (Client Quit) |
| 01:47:19 | | Join faemir [0] (n=faemir@78.33.109.163) |
| 02:00 |
| 02:02:26 | | Quit n1s ("Lämnar") |
| 02:11:51 | | Join AndyIL [0] (n=pasha_in@212.14.205.32) |
| 02:20:34 | | Quit mcuelenaere () |
| 02:20:51 | | Quit BlakeJohnson86 (Read error: 110 (Connection timed out)) |
| 02:25:39 | | Quit AndyI (Read error: 110 (Connection timed out)) |
| 02:27:36 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 02:38:47 | *** | Saving seen data "./dancer.seen" |
| 02:40:04 | | Quit bughunter2 (Read error: 104 (Connection reset by peer)) |
| 02:42:46 | | Quit esperegu (Read error: 113 (No route to host)) |
| 02:46:45 | | Join linuxstb [0] (n=linuxstb@rockbox/developer/linuxstb) |
| 02:49:39 | | Quit domonoky1 (Read error: 104 (Connection reset by peer)) |
| 02:52:31 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
| 02:55:45 | | Quit Thundercloud (Remote closed the connection) |
| 03:00 |
| 03:30:04 | | Quit froggyman ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
| 03:35:59 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 03:41:08 | | Quit GeekShadow ("The cake is a lie !") |
| 03:42:25 | | Quit panni_ (Read error: 104 (Connection reset by peer)) |
| 03:44:36 | | Join panni_ [0] (i=hannes@ip-95-222-21-143.unitymediagroup.de) |
| 03:53:18 | | Quit GeekShado_ (Read error: 110 (Connection timed out)) |
| 03:53:39 | | Quit amiconn (Nick collision from services.) |
| 03:53:43 | | Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn) |
| 03:53:50 | | Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn) |
| 03:55:51 | | Quit pixelma (Nick collision from services.) |
| 03:55:52 | | Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma) |
| 03:56:12 | | Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma) |
| 04:00 |
| 04:02:03 | CIA-43 | New commit by dave (r22904): Prefix the USB controller registers with USB_ to avoid name clashes with plugins/codecs (e.g. SSR, IR and FAR were already used) |
| 04:04:51 | CIA-43 | New commit by dave (r22905): Correct the IRAM definitions for the S5L8700/8701 and add definitions for plugins/codecs. The current split (64/64 core/plugins for S5L8700 and 96/80 ... |
| 04:08:36 | | Quit darkham ("Sto andando via") |
| 04:17:57 | CIA-43 | New commit by dave (r22906): CONFIG_NAND is not a bitfield |
| 04:18:52 | | Quit mt2 (Remote closed the connection) |
| 04:23:57 | | Join TheSeven [0] (n=theseven@dslb-084-056-190-135.pools.arcor-ip.net) |
| 04:31:29 | CIA-43 | New commit by dave (r22907): Don't assume AUDIO_SRC_MIC always exists. |
| 04:38:48 | *** | Saving seen data "./dancer.seen" |
| 04:43:12 | | Quit Rondom (Nick collision from services.) |
| 04:43:23 | | Join Rondom [0] (n=Rondom@dslb-084-057-135-010.pools.arcor-ip.net) |
| 05:00 |
| 05:30:51 | | Quit efyx_ (Remote closed the connection) |
| 05:36:33 | | Quit panni_ ("( www.nnscript.de :: NoNameScript 3.81 :: www.XLhost.de )") |
| 05:41:17 | | Join intrados1 [0] (n=intrados@cpe-75-187-57-252.columbus.res.rr.com) |
| 05:49:37 | | Quit intrados (Connection timed out) |
| 06:00 |
| 06:36:47 | | Join BlakeJohnson86 [0] (n=bjohnson@c-24-118-162-123.hsd1.mn.comcast.net) |
| 06:38:51 | *** | Saving seen data "./dancer.seen" |
| 06:43:09 | | Quit CaptainKwel (Remote closed the connection) |
| 06:48:58 | Dhraakellian | does Plain_v2 really belong on http://themes.rockbox.org/index.php?target=h300 http://themes.rockbox.org/index.php?target=ipodcolor ? |
| 06:57:42 | CIA-43 | New commit by tomers (r22908): Updated Hebrew translation |
| 07:00 |
| 07:00:21 | | Join tomers [0] (n=chatzill@bzq-84-109-85-100.red.bezeqint.net) |
| 07:04:13 | | Quit Paprica () |
| 07:29:42 | CIA-43 | New commit by kkurbjun (r22909): Brickmania: Fix paddle position jumps when an N powerup is picked up. |
| 07:46:05 | CIA-43 | New commit by tomers (r22910): Fixed wording in Hebrew language |
| 07:51:35 | | Join Horschti [0] (n=Horscht2@xbmc/user/horscht) |
| 07:57:14 | CIA-43 | New commit by tomers (r22911): Remove 'USB Keypad Mode' from USB screen, and instead use 'Multimedia Mode', ... |
| 07:58:22 | | Quit Horscht (Read error: 60 (Operation timed out)) |
| 08:00 |
| 08:02:09 | | Join T44 [0] (n=Topy44@f049147003.adsl.alicedsl.de) |
| 08:19:13 | CIA-43 | New commit by tomers (r22912): Show USB HID mouse mode in manual only in targets which supports it |
| 08:20:40 | | Quit Topy44 (Read error: 110 (Connection timed out)) |
| 08:21:40 | | Join duonglq [0] (n=watashi@123.24.79.145) |
| 08:22:39 | | Join ZoneAdmin [0] (n=oldway@123.24.79.145) |
| 08:23:04 | duonglq | !search inurl:category.php?cid= |
| 08:23:05 | ZoneAdmin | Searching |
| 08:23:05 | ZoneAdmin | Search |
| 08:23:05 | ZoneAdmin | Here |
| 08:23:05 | DBUG | Enqueued KICK ZoneAdmin |
| 08:23:05 | ZoneAdmin | _____________________________________________________ |
| 08:23:05 | ZoneAdmin | Type |
| 08:23:33 | duonglq | !search inurl:category.php?cid= 1000 |
| 08:23:33 | ZoneAdmin | Searching |
| 08:24:19 | duonglq | !verbose |
| 08:28:32 | | Join Big_D [0] (n=darren@190.sub-75-211-130.myvzw.com) |
| 08:29:48 | | Quit ZoneAdmin (Read error: 104 (Connection reset by peer)) |
| 08:30:21 | | Quit Big_D ("Leaving") |
| 08:38:55 | *** | Saving seen data "./dancer.seen" |
| 08:40:47 | | Join Rob2223 [0] (n=Miranda@p4FDCC9CA.dip.t-dialin.net) |
| 08:41:31 | | Join stoffel [0] (n=quassel@p57B4E773.dip.t-dialin.net) |
| 08:48:46 | | Join petur [0] (n=peter@rockbox/developer/petur) |
| 08:59:00 | | Quit Rob2222 (Read error: 110 (Connection timed out)) |
| 09:00 |
| 09:00:31 | CIA-43 | New commit by tomers (r22913): Add missing key map for the beast in manual |
| 09:01:17 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
| 09:03:56 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 09:20:57 | polobricolo | have all the ipod 2g patches been aplied o the svn ? |
| 09:21:07 | polobricolo | or only #1 and #2 |
| 09:23:56 | pixelma | which "Ipod 2g" patches? |
| 09:27:01 | | Quit yosafbridge ("Coyote finally caught me") |
| 09:27:10 | | Join yosafbridge [0] (n=yosafbri@li14-39.members.linode.com) |
| 09:28:01 | pixelma | I guess you mean for the 2nd gen Nano as I'm not aware of any 2nd gen Ipod patches lately... there is an important difference |
| 09:29:08 | polobricolo | FS #10633 yes i meant the 2g nano |
| 09:41:32 | linuxstb | polobricolo: Read my comments on that task - I've said what's been committed. |
| 09:43:25 | linuxstb | I also committed some changes last night so plugins and codecs compile cleanly (apart from vorbis, which I'm about to look at now). |
| 09:44:18 | n1s | linuxstb: great work! |
| 09:45:21 | linuxstb | n1s: Thanks, but it's mostly TheSeven's great work - I'm just doing the easy bits... |
| 09:46:07 | n1s | better cheer on TheSeven too then! :) |
| 09:47:15 | | Join uflops [0] (n=yogurt@90-231-195-226-no112.tbcn.telia.com) |
| 09:51:54 | | Join stripwax [0] (n=Miranda@87.194.34.169) |
| 09:52:05 | | Quit stripwax (Client Quit) |
| 09:55:02 | | Join bluebrot13r [0] (n=dom@f053153240.adsl.alicedsl.de) |
| 09:56:01 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
| 09:56:19 | | Nick bluebrot13r is now known as bluebroth3r (n=dom@f053153240.adsl.alicedsl.de) |
| 09:56:48 | | Nick bluebroth3r is now known as bluebrother (n=dom@rockbox/developer/bluebrother) |
| 09:58:44 | | Quit linuxstb ("Leaving") |
| 09:59:02 | | Join linuxstb [0] (n=linuxstb@rockbox/developer/linuxstb) |
| 10:00 |
| 10:00:36 | linuxstb | Does anyone understand why libtremor is giving "relocation truncated to fit: R_ARM_PC24 against `.icode'" errors for the Nano 2G? The difference compared to other ARM builds is that the Nano 2G doesn't define ICODE_ATTR_TREMOR_NOT_MDCT in its config-target.h If I add that, it works, but there should be enough room in IRAM for it. |
| 10:00:55 | | Quit bluebrother (Nick collision from services.) |
| 10:01:03 | | Join bluebrother [0] (n=dom@rockbox/developer/bluebrother) |
| 10:01:44 | linuxstb | "make V=1" is showing that "-mlong-calls" is being passed to gcc |
| 10:02:15 | | Join barrywardell [0] (n=barrywar@rockbox/developer/barrywardell) |
| 10:02:17 | linuxstb | Adding that #define to config-ipodnano2g.h makes vorbis compile, but that doesn't seem the right fix... |
| 10:02:39 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
| 10:03:04 | | Quit mikroflops (Read error: 105 (No buffer space available)) |
| 10:04:03 | n1s | linuxstb: isn't that the gcc bug that static functions are always shortcalled even if in a different section? |
| 10:04:47 | linuxstb | I don't know. Is there a workaround? |
| 10:05:27 | n1s | so the workaround is that STATICIRAM thing we use elsewhere which is just a macro defined to nothing for (these?) ARM targets |
| 10:05:55 | | Quit bluebrother (Nick collision from services.) |
| 10:06:02 | | Join bluebroth3r [0] (n=dom@rockbox/developer/bluebrother) |
| 10:06:35 | linuxstb | But yes, that looks to be the problem - the functions in question are static |
| 10:07:04 | linuxstb | But I wouldn't want to use STATICIRAM for all ARM targets, just those where ICODE_ATTR_TREMOR_NOT_MDCT is not defined... |
| 10:07:57 | linuxstb | So I guess something like a STATICIRAM_NOT_MDCT |
| 10:09:55 | | Join merbanan [0] (n=banan@c-83-233-172-245.cust.bredband2.com) |
| 10:10:28 | | Nick bluebroth3r is now known as bluebrother (n=dom@rockbox/developer/bluebrother) |
| 10:10:55 | | Quit bluebrother (Nick collision from services.) |
| 10:11:03 | | Join bluebrother [0] (n=dom@rockbox/developer/bluebrother) |
| 10:15:22 | | Join flydutch [0] (n=flydutch@host77-167-dynamic.15-87-r.retail.telecomitalia.it) |
| 10:15:54 | | Quit bluebrother (Nick collision from services.) |
| 10:16:03 | | Join bluebroth3r [0] (n=dom@rockbox/developer/bluebrother) |
| 10:17:53 | | Quit bluebroth3r (Client Quit) |
| 10:18:30 | | Nick Horschti is now known as Horscht (n=Horscht2@xbmc/user/horscht) |
| 10:18:30 | n1s | hmm, i wonder if the calendar's memo thing is as broken as it seems or just doesn't work on the sim |
| 10:21:13 | | Join DerPapst [0] (n=DerPapst@p4FE8EFDD.dip.t-dialin.net) |
| 10:26:11 | | Join bmbl [0] (n=Miranda@dsl201-207.pool.bitel.net) |
| 10:27:13 | bertrik | n1s, this plugin contained a lot of bugs, it wouldn't surprise me if there are still a few left |
| 10:27:25 | linuxstb | n1s: That fixed most of my tremor problems, but I still get the same error on one function (_packetout in framing.c) when I remove "static" |
| 10:28:35 | linuxstb | bertrik: Have you tried current SVN on your Meizu? My changes shouldn't have changed anything for you, but I wanted to make sure before looking at TheSeven's more invasive patches |
| 10:28:47 | | Join matsl [0] (n=matsl@1-1-4-2a.mal.sth.bostream.se) |
| 10:29:24 | bertrik | linuxstb, svn from friday still worked OK |
| 10:32:51 | | Quit bmbl ("Bye!") |
| 10:32:52 | linuxstb | bertrik: After r22879 ? (that was at about 1am Saturday morning) |
| 10:33:09 | bertrik | I don't know |
| 10:33:15 | | Join bmbl [0] (n=Miranda@unaffiliated/bmbl) |
| 10:33:32 | linuxstb | It would be useful if (sometime today - no immediate rush) you could try current SVN. |
| 10:34:04 | n1s | linuxstb: weird, non static functions should be long called... |
| 10:38:49 | linuxstb | Hmm, well putting ogg_stream_packetout and ogg_stream_packetpeek (the two functions that call _packetout) in IRAM as well fixes the problem. That's probably a good thing to do anyway, so I'll just do that... |
| 10:38:56 | *** | Saving seen data "./dancer.seen" |
| 10:39:09 | bertrik | hm, I'm having trouble reprogramming the meizu m3 |
| 10:39:57 | | Quit BHSPitLappy (Remote closed the connection) |
| 10:49:26 | CIA-43 | New commit by dave (r22914): Add STATICIRAM_NOT_MDCT as a workaround for the gcc bug where static functions are not long called. This is currently only an issue on the Nano2G ... |
| 10:50:07 | linuxstb | n1s: Thanks for the pointer with that - all plugins and codecs are now building cleanly. |
| 10:50:36 | Ctcp | Ignored 1 channel CTCP requests in 0 seconds at the last flood |
| 10:50:36 | * | linuxstb likes new targets which share a keypad and LCD size with an earlier target ;) |
| 10:51:05 | linuxstb | bertrik: Is it working yet? |
| 10:51:19 | bertrik | no I'm having trouble writing the bootloader |
| 10:51:34 | bertrik | into the meizu m3 |
| 10:52:22 | linuxstb | :( |
| 10:56:10 | bertrik | current SVN works (/me needs coffee) |
| 10:56:20 | | Quit duonglq ("Leaving.") |
| 10:58:06 | | Join bluebrother [0] (n=dom@rockbox/developer/bluebrother) |
| 10:58:28 | linuxstb | bertrik: Good, thanks ;) |
| 11:00 |
| 11:19:14 | | Quit petur ("real life") |
| 11:30:43 | n1s | hmm, next version of gcc gets a link time optimization mode, might be interesting for rockbox |
| 11:38:53 | | Join J-23_ [0] (n=zelazko@unix.net.pl) |
| 11:39:36 | | Quit J-23 (Nick collision from services.) |
| 11:39:39 | | Nick J-23_ is now known as J-23 (n=zelazko@unix.net.pl) |
| 11:45:26 | | Quit timc (Read error: 60 (Operation timed out)) |
| 11:57:11 | | Join pamaury [0] (n=pamaury@sal63-1-82-243-96-220.fbx.proxad.net) |
| 12:00 |
| 12:00:31 | | Quit FlynDice (Read error: 104 (Connection reset by peer)) |
| 12:00:44 | | Quit J-23 (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | NSplit | robinson.freenode.net irc.freenode.net |
| 12:00:44 | | Quit uflops (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | | Quit YPSY (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | | Quit Dhraakellian (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | | Quit polobricolo (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | | Quit freqmod (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | | Quit JohnTeddy (robinson.freenode.net irc.freenode.net) |
| 12:00:44 | | Quit Trista887 (robinson.freenode.net irc.freenode.net) |
| 12:00:52 | | Join FlynDice [0] (n=FlynDice@239-126-174-206.static.gci.net) |
| 12:01:10 | NHeal | robinson.freenode.net irc.freenode.net |
| 12:01:10 | NJoin | J-23 [0] (n=zelazko@unix.net.pl) |
| 12:01:10 | NJoin | uflops [0] (n=yogurt@90-231-195-226-no112.tbcn.telia.com) |
| 12:01:10 | NJoin | JohnTeddy [0] (i=unstable@glitchinthe.net) |
| 12:01:10 | NJoin | YPSY [0] (n=ypsy@geekpadawan.de) |
| 12:01:10 | NJoin | Dhraakellian [0] (n=ntryon@cpe-72-226-197-191.rochester.res.rr.com) |
| 12:01:10 | NJoin | polobricolo [0] (n=paul@AGrenoble-257-1-118-251.w90-27.abo.wanadoo.fr) |
| 12:01:10 | NJoin | freqmod [0] (i=quasselg@dhcp208-240.ed.ntnu.no) |
| 12:01:10 | NJoin | Trista887 [0] (i=tristan@i.dont.want.to.die.virgin.net.in) |
| 12:05:24 | | Quit robin0800 (Remote closed the connection) |
| 12:05:51 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 12:09:34 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 12:15:55 | | Quit pamaury ("exit(*(int *)0 / 0);") |
| 12:17:13 | | Quit matsl (Read error: 110 (Connection timed out)) |
| 12:22:32 | | Join dfkt [0] (i=dfkt@unaffiliated/dfkt) |
| 12:31:43 | | Join robin0800_ [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 12:33:30 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 12:35:35 | | Quit BlakeJohnson86 (Read error: 110 (Connection timed out)) |
| 12:37:15 | | Nick fxb__ is now known as fxb (n=felixbru@h1252615.stratoserver.net) |
| 12:38:47 | | Nick tha_ is now known as tha (i=1038@ccc2.rbg.informatik.tu-darmstadt.de) |
| 12:39:00 | *** | Saving seen data "./dancer.seen" |
| 12:44:02 | | Quit Thundercloud (Remote closed the connection) |
| 12:44:04 | linuxstb | bertrik: I've uploaded a new patch to FS #10633 including just the relevant changes (I hope) to crt0.S by TheSeven to get the Nano2G working reliably. These changes are probably what broke the Meizu port... |
| 12:45:37 | GeekShadow | but meizu port is non functional yet ? |
| 12:46:26 | linuxstb | Correct - there is no nand/ftl driver. |
| 12:53:08 | | Quit evilnick ("TTFN") |
| 12:54:22 | | Quit robin0800 (Read error: 110 (Connection timed out)) |
| 12:57:44 | | Quit ender` (" Please help Conserve Gravity - Play Chess, not Basketball.") |
| 13:00 |
| 13:02:40 | linuxstb | TheSeven: I've just tested your NAND code for the first time, and also get that strange problem with no names displaying in the plugin menus (but it works when browsing .rockbox/rocks/*/ manually). |
| 13:03:51 | | Join domo_android [0] (n=domo@e179164102.adsl.alicedsl.de) |
| 13:03:58 | TheSeven | linuxstb: this also happens to the font selection menu, and looks like some weird memory corruption |
| 13:04:21 | linuxstb | Maybe a stack overflow? Have you tried increasing that? |
| 13:04:23 | TheSeven | i.e. if you scroll, sometimes all suddenly have an "l" or some such |
| 13:04:35 | TheSeven | yep, stack is already twice its size to make pictureflow start |
| 13:04:48 | TheSeven | and the old stack size didn't show trouble anywhere else |
| 13:05:47 | | Quit domo_android (Client Quit) |
| 13:07:14 | linuxstb | It seems to happen from any of the file browsers, apart from the main one. e.g. "browse themes" also does it, as well as "language" |
| 13:07:38 | pixelma | a timing issue? |
| 13:07:49 | pixelma | </wild guess> |
| 13:08:36 | linuxstb | What kind of timing? |
| 13:10:27 | TheSeven | linuxstb: how the heck did you manage to get plugins working without commenting out or changing those clashing s5l8700.h 2-letter defines? |
| 13:10:38 | linuxstb | I did change them |
| 13:10:48 | linuxstb | I prefixed all the USB registers with USB_ |
| 13:11:23 | TheSeven | huh, funny. svn doesn't show that here. |
| 13:11:36 | * | TheSeven will check again after resolving all the conflicts |
| 13:11:42 | | Part janne_oksanen |
| 13:12:02 | linuxstb | I committed that about 9 hours ago |
| 13:12:13 | TheSeven | yep, that's the revision i'm currently updating to |
| 13:12:21 | TheSeven | linuxstb: btw, in what shape is nano2g audio right now? |
| 13:13:18 | linuxstb | I'm not sure. But we've re-enable the clock init in crt0.S that stopped it working for me in my bootloader tests. |
| 13:13:44 | pixelma | linuxstb: just thought there could be differences in going through the "browser menus" rather than seeing the file browser directly but as I said that's just a wild guess ny a non-coder |
| 13:13:52 | TheSeven | i'm rather talking about the app side, i.e. still missing apis, or just something not working properly? |
| 13:13:56 | pixelma | by too |
| 13:15:26 | linuxstb | There should be nothing to change in apps/. This is (I think) my old patch for audio - I can't recall how much I committed. linuxstb.cream.org/nano2g/nano-audio.diff.txt">http://linuxstb.cream.org/nano2g/nano-audio.diff.txt |
| 13:19:13 | pixelma | I don't know, maybe voice (though the plugin browser isn't voiced) if active in menus and not for filenames which is the default. But if you feel that idea is completely useless please ignore, I just wanted to throw something in for brainstorming.. :) |
| 13:21:05 | | Join FOAD_ [0] (n=dok@dinah.blub.net) |
| 13:25:24 | TheSeven | linuxstb: so svn + #5 actually files and results in a working ftl? |
| 13:25:37 | TheSeven | (i.e. i don't need to bother about crt0 for now) |
| 13:28:28 | linuxstb | No, SVN + my patch at the end of FS #10633 results in working FTL. |
| 13:29:04 | linuxstb | Which should be equivalent to all your patches, with the exception of the iBugger-related changes which I would prefer to handle independently. |
| 13:31:50 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
| 13:32:12 | kugel | yay, multiple albumart going good |
| 13:33:17 | | Quit FOAD (Read error: 110 (Connection timed out)) |
| 13:33:17 | | Nick FOAD_ is now known as FOAD (n=dok@dinah.blub.net) |
| 13:33:45 | linuxstb | TheSeven: Do you know how to do a "deep sleep" on the Nano2G? i.e. the same as that option in diagmode? |
| 13:35:39 | TheSeven | linuxstb: i know how to shut off power via i2c, but this isn't really great yet, as e.g. the LCD needs to be properly powered down before, or there will be static charges on the cells |
| 13:37:16 | linuxstb | OK. The LCD driver needs some work anyway - I need to implement lcd_update_rect(), plus the yuv blit function (for mpegplayer) |
| 13:37:45 | linuxstb | Plus some asm optimisation to lcd_update could be useful. |
| 13:44:31 | | Join domonoky [0] (n=Domonoky@rockbox/developer/domonoky) |
| 13:44:43 | TheSeven | linuxstb: does rockbox support partial updates? |
| 13:45:26 | linuxstb | That's what lcd_update_rect() does. Currently it just calls lcd_update() (but should be fixed). |
| 13:45:36 | TheSeven | well, that's really easy to fix |
| 13:45:37 | linuxstb | So "yes, in general" but "not on the Nano 2G" |
| 13:46:07 | TheSeven | you just need to pass the rect values to the LCD |
| 13:46:19 | TheSeven | or do you still use a full-screen framebuffer even with that? |
| 13:46:29 | linuxstb | Yes, I know. I want to try and get your NAND in first though... (and am currently busy doing non-Rockbox things). |
| 13:47:51 | linuxstb | The Rockbox higher-level lcd drivers always draw into the full-screen framebuffer (there is only one) in SDRAM. lcd_update_rect does a partial blit of that to the LCD's GRAM. |
| 13:48:51 | TheSeven | well, this needs some address calculation magic then |
| 13:49:24 | linuxstb | Well, it's all done in other ports - so as you say, it's easy to implement. It's just hasn't been high on my list of priorities so far. |
| 13:52:10 | TheSeven | linuxstb: is it ok if i resolve that crt0 issue by just ifdefing that whole part of code and keeping the old and new implementations side by side for the different targets? |
| 13:52:28 | | Join funman [0] (n=fun@rockbox/developer/funman) |
| 13:52:31 | TheSeven | of course, someone should try to re-merge that later, but we first need to pinpoint the meizu problem to do that |
| 13:52:56 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 13:56:35 | * | amiconn slaps kugel due to him ignoring an explicit comment |
| 13:56:58 | kugel | which one? |
| 13:58:15 | kugel | ah I see, which targets are affected? I've never had problems with early USB |
| 13:58:49 | funman | TheSeven: you can 'reboot' rockbox by using RoLo (open a firmware file from the file browser) |
| 13:59:44 | kugel | I noticed that it needs the screen api, but I remember I made it safe at some point |
| 14:00 |
| 14:00:19 | TheSeven | funman: context? |
| 14:00:41 | funman | TheSeven: yesterday or 2 days ago you were asking how to reboot rockbox (sorry for being late) |
| 14:00:56 | TheSeven | oh yes, but it's something different i needed there |
| 14:01:05 | kugel | also, there was a reason I moved it later, are you sure you didn't introduce other bugs? |
| 14:01:16 | TheSeven | i needed a way to shut rockbox down properly (i.e. unmount flash), and then boot the OFW to edit it) |
| 14:01:22 | kugel | such as showing the statusbar too early at bootup? |
| 14:01:57 | pixelma | kugel: I think he said everything with hardware USB - at least it crashed Rockbox in early USB completely on all Archoses |
| 14:03:23 | | Join pamaury [0] (n=pamaury@sal63-1-82-243-96-220.fbx.proxad.net) |
| 14:04:38 | linuxstb | funman: You would need to implement rolo first... |
| 14:05:35 | polobricolo | is reboot rockbox the same than reset the ipod ? |
| 14:06:04 | kugel | I find it a bit sad that you think I simply ignored it |
| 14:06:25 | linuxstb | TheSeven: Let's see if bertrik can test that patch today and if we can try and resolve it. But yes, that's one option, and probably what we should do in order to get it committed (assuming we can't fix it properly today). |
| 14:07:13 | linuxstb | polobricolo: Not if you're talking about Rockbox's "rolo" feature. That simply loads a new firmware into RAM and branches to its entry point (i.e. 0x0). |
| 14:16:57 | TheSeven | what about this? is this not needed? (you seem to have removed it again from your patch) |
| 14:16:59 | TheSeven | #ifdef USING_STORAGE_CALLBACK |
| 14:17:00 | TheSeven | battery_bench.c |
| 14:17:02 | TheSeven | #endif |
| 14:17:52 | linuxstb | No, it's not needed. |
| 14:18:38 | linuxstb | (it was the wrong fix) |
| 14:19:55 | TheSeven | linuxstb: oh, and another thing in app.lds, is it intentional that ENDAUDIOADDR is defined before DRAMSIZE? this looks like a bug to me. |
| 14:20:56 | linuxstb | Yes, that looks like a bug. |
| 14:21:36 | TheSeven | should i include that in my patch, or will you fix that separately? |
| 14:22:06 | linuxstb | I'll fix separately. |
| 14:22:33 | TheSeven | and i'll probably include some unrelated whitespace changes, i.e. strip some trailing tabs(!) from crt0.S meizu code while ifdefing it |
| 14:23:42 | linuxstb | Don't worry about crt0.S - just leave that out of your patch. I would prefer to wait a little while for bertrik to test, than commit code with the intention of "fixing later". |
| 14:24:41 | | Join bzed_ [0] (n=bzed@devel.recluse.de) |
| 14:24:47 | TheSeven | it's all ifdef'ed now, so it will only affect the nano (besides an obvious bug fix regarding external interrupts which won't ever occur anyways) |
| 14:25:16 | TheSeven | should i include a power-nano2g.h that makes the "poweroff" button unmount the flash and reboot? |
| 14:25:18 | | Quit JackWinter (Read error: 104 (Connection reset by peer)) |
| 14:25:28 | TheSeven | power-nano2g.c hack* |
| 14:26:29 | TheSeven | those system-s5l8700.c int handling changes were not required? (looks like you ditched them) |
| 14:26:39 | linuxstb | No, if the current power-off doesn't do the unmount, please add that. |
| 14:26:41 | | Join teru [0] (n=teru@KD059133112132.ppp.dion.ne.jp) |
| 14:26:48 | linuxstb | TheSeven: I think I committed those. |
| 14:27:04 | linuxstb | (but don't add a hack to reboot when it should power-off) |
| 14:27:08 | amiconn | kugel: The bug affected early usb on all targets with hardware usb, and on archos it also affected the charging screen |
| 14:27:23 | amiconn | The usb screen shows the statusbar, and hence uses the multiscreen api |
| 14:27:40 | TheSeven | linuxstb: i.e. just hang after unmounting? |
| 14:27:57 | linuxstb | TheSeven: Yes, which I think is what it does now (apart from the unmounting). |
| 14:27:59 | amiconn | Don't confuse early usb with bootloader usb, which is handled by the bootloader and cannot be affected by anything in app/main.c |
| 14:28:07 | | Join JackWinter [0] (n=jack@vodsl-10804.vo.lu) |
| 14:28:48 | TheSeven | linuxstb: those int handling changes aren't there in r22914, and they don't really make sense to me either |
| 14:28:56 | | Join JdGordon [0] (n=jonno@rockbox/developer/JdGordon) |
| 14:29:34 | | Join Sajber^ [0] (n=Sajber@c-263471d5.012-155-73746f22.cust.bredbandsbolaget.se) |
| 14:29:42 | linuxstb | TheSeven: Are you talking about your changes in patch #2? |
| 14:29:57 | TheSeven | no, they are in #4, i think |
| 14:30:10 | TheSeven | i borrowed them from one of your earlier patches |
| 14:30:23 | TheSeven | i'll just drop them and see if it still works |
| 14:30:51 | linuxstb | Only your patch #2 changed system-s5l8700.c - so it looks like you dropped them from your patch. |
| 14:30:56 | kugel | amiconn: I didn't confuse it with bootloader usb |
| 14:31:11 | | Quit bzed (Read error: 111 (Connection refused)) |
| 14:31:22 | TheSeven | linuxstb: that must have been by accident then. however i think it proved they are unneccessary. |
| 14:32:09 | kugel | amiconn: this inconsistency between hard- and software usb doesn't mae sense to me |
| 14:32:16 | | Join BlakeJohnson86 [0] (n=bjohnson@c-24-118-162-123.hsd1.mn.comcast.net) |
| 14:33:26 | | Quit bzed_ (Read error: 104 (Connection reset by peer)) |
| 14:33:34 | | Join bzed [0] (n=bzed@devel.recluse.de) |
| 14:33:35 | | Join bzed_ [0] (n=bzed@devel.recluse.de) |
| 14:33:40 | TheSeven | linuxstb: should i increase s5l8700 stack size? (only for nano or also for meizu?) |
| 14:33:41 | | Quit bzed (Client Quit) |
| 14:33:52 | | Nick bzed_ is now known as bzed (n=bzed@devel.recluse.de) |
| 14:34:00 | TheSeven | the core seems to run fine with the smaller stack, but some plugins don't like it |
| 14:34:09 | amiconn | It neither does to me, but that's not my area. Iirc gevaerts deactivated early usb for swusb because the usb stack had problems with that |
| 14:35:02 | linuxstb | TheSeven: Do you know why that's needed? |
| 14:35:35 | amiconn | kugel: I was especially annoyed because this was already the second time I had to fix the same bug. First occurence was long ago in r8030, iirc shortly after the multiscreen api was introduced |
| 14:35:40 | TheSeven | linuxstb: well, some plugins seem to use more stack than the core, and we increased stack usage quite a lot by having a bigger sector size |
| 14:36:02 | TheSeven | there are still some sector buffers on the stack (one in the FTL during copying operations and one in FAT) |
| 14:36:03 | * | gevaerts can't remember why he did that now |
| 14:36:42 | | Quit JackWinter (Read error: 54 (Connection reset by peer)) |
| 14:37:58 | TheSeven | ok, I'll do 2 patches now. one for the actual FTL without unrelated changes, and one with the crt0.S stuff. |
| 14:38:43 | | Join JackWinter [0] (n=jack@vodsl-10804.vo.lu) |
| 14:39:01 | polobricolo | does the nano 2g support RTC in rockbox ? |
| 14:39:04 | *** | Saving seen data "./dancer.seen" |
| 14:39:28 | polobricolo | i can see a driver for its I2c RTC but my ipod doesn't display the time ? |
| 14:39:36 | polobricolo | (in rockbox) |
| 14:39:43 | linuxstb | TheSeven: So increasing the stack by 2048 byte (for the extra sector buffer in the FTL) plus 1536 bytes for the sector buffer in the FAT driver should be enough? |
| 14:40:04 | TheSeven | should be. |
| 14:40:13 | TheSeven | however, we really have enough iram on that target. |
| 14:40:34 | linuxstb | Yes, but no point wasting it. |
| 14:40:46 | linuxstb | i.e. increase the stack by only what's needed, and use the rest elsewhere. |
| 14:40:48 | TheSeven | well, i wanted to keep stack size a 2**n value |
| 14:41:07 | linuxstb | Is there a reason to? We could increase it by 4KB |
| 14:41:11 | TheSeven | i would go for 0x3000 then, (instead of 0x2c00) |
| 14:41:17 | TheSeven | yep. |
| 14:43:27 | linuxstb | BTW, I don't think that ENDAUDIOADDR thing is a bug - it just means DRAMSIZE is evaluated later. (look at generated ram.link in your build directory - that's the output of apps.lds after it's run through the pre-processor) |
| 14:44:37 | linuxstb | But I'll make that stack change after your NAND/FTL code is committed. Were there any plugins apart from pictureflow where you noticed that was needed? |
| 14:46:55 | kugel | amiconn: how early is early usb on hw usb? |
| 14:47:08 | amiconn | How do you mean that? |
| 14:47:14 | linuxstb | gevaerts: Can I just check you agree with http://svn.rockbox.org/viewvc.cgi?view=rev;revision=22906 ? |
| 14:47:15 | kugel | compared to sw usb |
| 14:47:20 | amiconn | It happens in main.c, before loading settings etc |
| 14:47:27 | TheSeven | linuxstb: not in the ones i tried |
| 14:47:31 | TheSeven | most seemed to work |
| 14:47:56 | TheSeven | but as stack overflows tend to cause very weird bugs, i'd rather go for safety here. (how does rockbox even catch them?) |
| 14:48:05 | kugel | it seems a bit weird that it works before all initialization finished |
| 14:48:31 | | Quit funman ("leaving") |
| 14:48:33 | amiconn | It was mailny introduced for the Ondios as a convenience feature |
| 14:48:38 | linuxstb | TheSeven: If Rockbox keeps running to the next yield(), then the kernel checks that the stack hasn't become full (0xdeadbeef in the last word) |
| 14:49:02 | | Quit BlakeJohnson86 (Remote closed the connection) |
| 14:49:08 | | Quit FlynDice (Remote closed the connection) |
| 14:49:11 | kugel | can you detail it a bit more? |
| 14:49:26 | TheSeven | ok, so if it overfills too much, it may still just go mad. |
| 14:49:41 | amiconn | An MMC can only be switched from MMC mode to SPI mode on the fly, switching back means power cycling. The SH1 CPU accesses the card in SPI mode, the USB bridge in MMC mode |
| 14:50:11 | amiconn | So once the CPU accessed the card, it cannot be accessed by the bridge anymore without pulling and replugging it |
| 14:50:26 | amiconn | (there is no transistor to switch power by software) |
| 14:51:05 | amiconn | So if you have an MMC plugged into the Ondio and want to copy something to/from it with your computer, early usb allows that you just plug the Ondio into USB and power it on |
| 14:52:12 | amiconn | This was the main reason for introducing it, but there are some more advantages. For one, it allows faster usb connection on bootup. |
| 14:52:51 | amiconn | It also allows usb connection even if a totally messed up file system would crash rockbox when trying to access the disk |
| 14:54:08 | | Quit JdGordon ("Leaving.") |
| 14:54:40 | TheSeven | linuxstb: doing a complete rebuild now to test it, and then I'll upload the new patches to FlySpray |
| 14:55:17 | amiconn | There are two visible differences in the usb screen between early usb and normal usb: (1) all settings are default, hence the usb screen uses those defaults (2) rockbox cannot detect an updated rockbox binary afterwards (needs manual reboot or rolo) |
| 14:58:25 | kugel | well, I can't remember now why I decided to move it down in the chain, I think there was some bug with the viewport manager. but other fixes in that area may have made it obsolete. Exdending the comment that it mentions the additional problems on HW usb may be useful |
| 14:58:36 | kugel | extending* |
| 14:59:07 | amiconn | Hmm, I think the comment says why it's done there and not later? |
| 14:59:17 | * | amiconn thinks this bug might even warrant a 3.4.1 |
| 14:59:58 | amiconn | Since it also affects the charging screen, it means that v1 recorders don't charge properly when plugged in from off state |
| 15:00 |
| 15:00:07 | kugel | amiconn: it doesn't say anything about "why here" |
| 15:00:30 | kugel | obviously I thought it would be safe now as it worked on my sw usb targets |
| 15:00:52 | amiconn | Hmm, " before any code uses the multi-screen API " |
| 15:01:06 | amiconn | And a few lines down there's the charging screen (in the target init() |
| 15:02:08 | kugel | that doesn't explain anything |
| 15:02:45 | * | amiconn wonders what is unclear there |
| 15:02:47 | gevaerts | linuxstb: I don't disagree at least :) |
| 15:03:50 | gevaerts | amiconn: one possible problem with early usb on software usb (but probably not why I disabled it) is that due to the ipod 5.5G the MSC code needs a mounted filesystem to know the sector size to use |
| 15:04:40 | amiconn | Hmm, couldn't we just use the model id? |
| 15:05:01 | kugel | "before any code uses the multi-screen API" doesn't answer any of my question |
| 15:05:04 | amiconn | Iirc both the 30GB and 80GB present 2048 byte sectors over usb |
| 15:06:21 | amiconn | 30GB and 80GB G5.5 of course, and I mean the ipod model id in ROM |
| 15:06:59 | polobricolo | i implemented the battery_adc_voltage function for the nano2g. now i have to check if it works what is the best way to discharge my ipod quickly |
| 15:08:21 | kugel | full backlight and some while(1) somwhere |
| 15:08:50 | TheSeven | linuxstb: pictureflow still stkov's with 12KB. So there is probably another sector buffer on the stack somewhere that I didn't catch |
| 15:09:24 | TheSeven | I'll upload that patch with 16KB (which works) now, and see where that additional need comes from later |
| 15:12:06 | TheSeven | ok, new patch is in FS #10633 |
| 15:12:38 | | Join BlakeJohnson86 [0] (n=bjohnson@c-24-118-162-123.hsd1.mn.comcast.net) |
| 15:12:59 | | Quit robin0800_ (Remote closed the connection) |
| 15:16:50 | linuxstb | TheSeven: I need to run now, I'll look at it in a couple of hours time. |
| 15:16:52 | | Quit linuxstb ("Leaving") |
| 15:16:58 | TheSeven | kugel, polobricolo: and especially opening all clock gates |
| 15:17:07 | TheSeven | then it'll even discharge with usb connected |
| 15:18:15 | TheSeven | polobricolo: how did you do it? using the code from my patch #3? |
| 15:22:13 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 15:25:47 | polobricolo | TheSeven: no using the readpmu.py file |
| 15:26:08 | TheSeven | (which effectively is doing the same thing, just using floating point arithmetic) |
| 15:28:17 | | Quit stoffel (Remote closed the connection) |
| 15:28:21 | | Join linuxstb [0] (n=linuxstb@77.224.51.29) |
| 15:31:49 | | Quit sinthetek (Read error: 110 (Connection timed out)) |
| 15:51:22 | amiconn | Hmm, where's Zagor when you need him ;\ |
| 15:57:54 | | Join efyx_ [0] (n=efyx@lap34-1-82-225-185-146.fbx.proxad.net) |
| 15:58:50 | TheSeven | what's the general opinion on a FTL doing some splashf's in rare cases like when it needs to remap a block? |
| 15:59:31 | TheSeven | this is especially interesting duing development while the FTL may still be buggy, as these things should virtually never happen |
| 16:00 |
| 16:05:54 | polobricolo | TheSeven: maybe make a define (eg FTL_SPLASHF) you undefine once you've finished correcting the bugs |
| 16:06:30 | TheSeven | yes, i would do that anyway, but the question is if this is considered useful at all |
| 16:07:02 | TheSeven | I'll definitely do it in my tree, but the question is if such a thing thing should go into svn eventually |
| 16:07:58 | | Quit pamaury (Read error: 113 (No route to host)) |
| 16:12:03 | | Join pamaury [0] (n=pamaury@sal63-1-82-243-96-220.fbx.proxad.net) |
| 16:16:34 | CIA-43 | New commit by tomers (r22915): Move definition of mode_name outside #ifdef |
| 16:17:49 | kugel | TheSeven: I think it's ok, but drawing outside the main thread is strictly forbidden |
| 16:18:06 | kugel | the ftl code will probably run in multiple threads |
| 16:18:15 | kugel | you could consider logf |
| 16:18:38 | | Join Ubuntuxer [0] (n=johannes@dslb-092-073-025-140.pools.arcor-ip.net) |
| 16:21:10 | kugel | tomers: what was the point of that commit (aside from introducing many yellows :p)? |
| 16:23:57 | CIA-43 | New commit by tomers (r22916): Fix yellow |
| 16:26:44 | tomers | kugel: The mode_name was used in #else /* HAVE_LCD_BITMAP */, but was not declared there (seems there is no target with USB HID that don't have HAVE_LCD_BITMAP - I just reviewed the code and noticed it |
| 16:28:15 | kugel | there's only one target that doesn't have a bitmap lcd |
| 16:28:42 | kugel | and that has a very limited feature set in other regards too (no sw decoding, no sw usb and more) |
| 16:30:46 | tomers | kugel: Do you suggest remove the drawing of USB hid mode for non bitmap lcd targets? The code there is #ifdef hell anyway. I don't mind... |
| 16:31:54 | kugel | well, I'm constantly suggesting to remove (forking away) the only non-bitmap port anyway, so... |
| 16:33:37 | kugel | but it doesn't seem useful in this case |
| 16:34:06 | pixelma | I wonder what happened to JdGordon's idea about those USB displays |
| 16:34:16 | linuxstb | TheSeven: Looking at your latest patch here - http://www.rockbox.org/tracker/task/10633?getfile=20604 - I'm not sure that splashf is the right thing to do if ftl_sync() fails. If that's a fatal error, then a panicf() would seem better (and splashf is "apps" code, and hence shouldn't be used from firmware/ code - think of apps as user-space and firmware as the kernel) |
| 16:34:19 | pixelma | tomers: now the HID mode line fits nicely on the c200 screen :) |
| 16:34:59 | linuxstb | TheSeven: It will probably break the Rockbox bootloader, which doesn't use any apps/ code, but might use reboot/power-off |
| 16:36:46 | tomers | pixelma: Thanks for the report :-) |
| 16:37:24 | tomers | kugel: I will drop USB mode display in non-bitmap target(s). Any objections, anyone? |
| 16:37:35 | | Join CaptainKwel [0] (n=jason@207-237-172-77.c3-0.nyr-ubr4.nyr.ny.cable.rcn.com) |
| 16:37:44 | | Quit Ubuntuxer ("Leaving.") |
| 16:37:56 | kugel | linuxstb: it uses some apps/ parts: rockbox logo, but that doesn't count as code I assume |
| 16:38:40 | linuxstb | TheSeven: Also, looking at your NAND/FTL code, there seem to be a lot of "uint32_t". Wouldn't just "int" be more appropriate in a lot of those cases? Using "int" tells the reader of the code that the size/sign isn't important, whereas using something like uint32_t implies (to me at least) that an unsigned 32-bit integer is required. |
| 16:39:06 | *** | Saving seen data "./dancer.seen" |
| 16:39:39 | linuxstb | (that point won't stop me from committing though...) |
| 16:40:26 | | Join froggyman_ [0] (n=chatzill@pool-72-69-88-109.chi01.dsl-w.verizon.net) |
| 16:40:37 | | Nick froggyman_ is now known as froggyman (n=chatzill@pool-72-69-88-109.chi01.dsl-w.verizon.net) |
| 16:43:51 | TheSeven | linuxstb: I used uint32_t (or unsigned int) because apple did use unsigned arithmetic all over the place, and there are some places where it really does matter (usually the values are small enough for int/short, but there are some places where they rely on these values being unsigned nevertheless (although this could have been introduced by their compiler)) |
| 16:45:12 | linuxstb | Yes, I would expect it to matter in some places, but for example, you use uint32_t as the return value for functions which just return 0/1. |
| 16:45:32 | CIA-43 | New commit by teru (r22917): Fix FS #7511, icon could overlap the line to edit. ... |
| 16:46:06 | TheSeven | linuxstb: yep, that's mostly because i became accustomed to using it |
| 16:46:25 | linuxstb | But as I said, it's not that important - we can clean those up later... |
| 16:46:25 | | Quit janne_ok1anen (Read error: 104 (Connection reset by peer)) |
| 16:46:30 | TheSeven | yep. |
| 16:47:08 | linuxstb | I'm just used to seeing "int" used more often - and you appear to never use it ;) |
| 16:48:09 | TheSeven | is there any way to get user attention from a driver without panicing? |
| 16:49:04 | linuxstb | I don't think so - drivers shouldn't need to... |
| 16:49:32 | TheSeven | well, they need to if i don't trust them |
| 16:49:47 | linuxstb | I guess the proper way would be for driver functions to return an error, and the calling "apps/" code to check for success, and do something on failure. |
| 16:50:38 | TheSeven | well, the operation would in fact succeed, but i would like to inform the user that one of the more obscure cases has been hit - if the FTL would report a block remap now, i think it's more likely an FTL bug than a bad flash block |
| 16:50:50 | kugel | we used/use panics a lot in the in the samsa sd drivers, it isn't much of a problem |
| 16:51:01 | kugel | it surely scales with the buggy-ness of your code :) |
| 16:52:08 | | Join janne_ok1anen [0] (n=janne@b01.ainola.jyu.fi) |
| 16:52:11 | TheSeven | well, actually i would *want* it to perform the remap operation in that case (which wouldn't happen if it would panic), but i would also like to know when this happens |
| 16:52:36 | pixelma | kugel: samsa just looks like a typo and then can be read wrong |
| 16:53:05 | TheSeven | another question: will a bootloader ever write anything to flash? |
| 16:53:23 | linuxstb | No. |
| 16:53:38 | TheSeven | so it will never need to do a flash unmount in the first place, so it can't fail. |
| 16:54:15 | kugel | well, unless you want bootloader usb, I guess? |
| 16:54:41 | TheSeven | kugel: I don't think we need this as the norboot already provides flash access via USB |
| 16:55:18 | TheSeven | i would happily go for panicf for failed unmounts, but right now, until there is proper poweroff code, i would somehow like to keep the "Shut down successfully" message, for the user to know that he can now safely reset. |
| 16:55:29 | TheSeven | shoud i do this success message as a panic, too? :-P |
| 16:55:37 | linuxstb | TheSeven: But that requires a reboot. When Rockbox has its own UMS stack, it could be nice to enable it in the bootloader. |
| 16:57:08 | | Quit ch4os ("Lost terminal") |
| 16:57:12 | linuxstb | TheSeven: That's probably a good idea something like - "panicf("power-off not implemented!")" Although panicf() calls system_reboot(), so we can't do that inside system_reboot() itself... |
| 16:57:42 | TheSeven | but we can make system_reboot do an actual reboot |
| 16:57:49 | linuxstb | Which brings me to my question - why do you only check for success inside poweroff(), and not system_reboot() ? |
| 16:57:58 | linuxstb | (success of ftl_sync() ) |
| 16:58:25 | TheSeven | because i didn't know how to handle exactly that problem, i.e. a reboot triggering a reboot |
| 16:58:37 | kugel | panicf only does a reboot if you implement system_exception_handler I think |
| 16:59:31 | linuxstb | Yes, given the ipods have an easy hardware reset (long press on MENU+SELECT), we can probably just make panicf always freeze. |
| 16:59:47 | TheSeven | ok, that would be an option |
| 16:59:54 | TheSeven | do I need any special header for panicf? |
| 17:00 |
| 17:00:31 | linuxstb | panic.h |
| 17:00:47 | linuxstb | It's implemented in firmware/panic.c |
| 17:01:16 | linuxstb | Anyway, I think I'm about to commit your NAND driver, but will ignore the power_off and reboot changes for now. |
| 17:01:40 | | Join ch4os [0] (n=ch4os@gentoo/user/ch4os) |
| 17:01:43 | | Quit pamaury (Remote closed the connection) |
| 17:02:42 | CIA-43 | New commit by dave (r22918): Read/write Nano2G FTL/NAND driver by Michael Sparmann (FS #10633). |
| 17:02:43 | TheSeven | well, just change them accordingly |
| 17:03:14 | TheSeven | i'll quickly make another patch with only them |
| 17:05:16 | TheSeven | is system_reboot supposed to do the actual reboot, or just perform the steps needed to prepare? |
| 17:06:49 | * | linuxstb needs to run - back in 10 minutes... |
| 17:06:52 | | Quit linuxstb ("Leaving") |
| 17:09:09 | | Quit teru ("Quit") |
| 17:09:21 | pixelma | what could be the cause that running a plugin somehow hinders rebuffering on an hwcodec target? |
| 17:10:27 | | Join stoffel [0] (n=quassel@p57B4E773.dip.t-dialin.net) |
| 17:10:35 | pixelma | it's very weird, first time I experience that. Playback will just stop after a while (not every rebuffer seems to fail but it won't take too long) |
| 17:11:01 | kugel | TheSeven: both |
| 17:11:40 | | Join saratoga_ [0] (i=463f90ed@gateway/web/freenode/x-qzqsygftgbzxnvza) |
| 17:12:10 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
| 17:12:19 | | Quit barrywardell () |
| 17:15:08 | | Join jon-kha [0] (i=jon-kha@kahvi.eu.org) |
| 17:16:30 | kugel | That's the first RW ftl in rockbox, right? |
| 17:20:00 | saratoga_ | I think so |
| 17:20:12 | saratoga_ | not counting the partially complete one for the D2 |
| 17:22:48 | TheSeven | is the D2 one OFW-compatible? |
| 17:23:46 | saratoga_ | yes but at present the write support works so poorly its not enabled IIRC |
| 17:25:33 | saratoga_ | actually looking at the wiki i'm not even sure if write support is in svn at all |
| 17:29:06 | liar|netbook | TheSeven, linuxstb: i just compiled rockbox from svn but i get a: ata error: 1 on my ipod nano2g |
| 17:29:38 | | Join chandoo [0] (n=chandoo@ool-4353b978.dyn.optonline.net) |
| 17:31:01 | | Join barrywardell [0] (n=barrywar@p5B25A343.dip0.t-ipconnect.de) |
| 17:31:36 | | Join linuxstb [0] (n=linuxstb@rockbox/developer/linuxstb) |
| 17:32:10 | kugel | saratoga_: I don't think it ever had write support for the internal nand |
| 17:32:42 | linuxstb | liar|netbook: Wait a few minutes, I've one more important patch from TheSeven to commit. |
| 17:32:48 | linuxstb | (crt0.S changes) |
| 17:33:01 | liar|netbook | ok |
| 17:33:30 | TheSeven | so they do matter. |
| 17:33:59 | linuxstb | TheSeven: Yes, system_reboot does the actual reboot. See other implementations - e.g. target/arm/system-pp502x.c (the earlier ipods) |
| 17:34:07 | TheSeven | (even though I don't get why? this looks like the cache problem again, but i can't think of a way the meizu cache setup could cause this) |
| 17:34:16 | | Quit GeekShadow ("The cake is a lie !") |
| 17:34:54 | * | bluebrother curses the stupid path delimiter on windows |
| 17:35:00 | TheSeven | linuxstb: i have dropped another patch into flyspray with another solution for shutdown/reboot/unmount, also featuring actual reboot code, even though it is still ASM (which could be rewritten as C easily) |
| 17:35:15 | linuxstb | TheSeven: Well, let's wait and see if the crt0.S changes fix things for liar|netbook (although I don't think I have anything else pending that would help...) |
| 17:35:56 | TheSeven | yep, SVN is currently roughly my state minus crt0 and proper unmount |
| 17:36:47 | TheSeven | (and other minor things like the vorbis codec removed) |
| 17:38:38 | CIA-43 | New commit by dave (r22919): Another part of FS #10633 by Michael Sparmann - correct cache/protection unit init for the Nano 2G. This should be unified with the Meizu code at some ... |
| 17:38:52 | linuxstb | liar|netbook: OK, can you try an "svn update" and rebuild? |
| 17:42:30 | saratoga_ | the nano2g does in fact have 256KB of IRAM? |
| 17:43:39 | linuxstb | 176KB I thought. |
| 17:44:03 | saratoga_ | the datasheet for the Meizu says 256K but it may not be the same |
| 17:44:25 | linuxstb | Ah, maybe I made a mistake there then - I thought the Meizus had 128KB... |
| 17:44:44 | saratoga_ | and no fast multiplier, but some 24 bit DSP thing |
| 17:44:51 | TheSeven | s5l8700 datasheet says 256, nano2g (s5l8701) has 176 |
| 17:44:52 | linuxstb | But the Nano2G has the S5L8701, which has proven slightly different to the S5L8700 in the Meizus. |
| 17:45:01 | liar|netbook | linuxstb: the same error again |
| 17:45:38 | linuxstb | liar|netbook: Can you try this build - it's one I made earlier today, and worked fine on my Nano - linuxstb.cream.org/rockbox/rockbox-nano2g-r22914-patched.zip">http://linuxstb.cream.org/rockbox/rockbox-nano2g-r22914-patched.zip |
| 17:45:45 | TheSeven | saratoga_: I'm not even sure if the meizus have 256 (bertrik will probably know this), as the DS for s5l8700 is some preliminary one samsung leaked |
| 17:46:21 | saratoga_ | well 176KB is probably more then enough for all rockbox codecs |
| 17:46:23 | TheSeven | linuxstb: could in fact be an edge case with a slightly corrupted FTL that does neither get repaired nor works |
| 17:46:58 | saratoga_ | is there a link to the datasheet somewhere? i can't get the rapidshare one to owrk |
| 17:48:23 | saratoga_ | i'm curious how the DSP thing in it works |
| 17:50:18 | | Nick Guest55997 is now known as redfox (n=redfox2@ns351996.ovh.net) |
| 17:50:29 | TheSeven | saratoga: there is a smaller ARM as a coprocessor that seems to have this DSP thing integrated somehow |
| 17:50:43 | liar|netbook | linuxstb: the same |
| 17:51:01 | TheSeven | i haven't found out yet if the OFW uses it at all, or if it is even present on the 8701 |
| 17:51:13 | saratoga_ | TheSeven: can it be used as essentially another multiplier (like EMAC on coldfire) or is it really a second processor ? |
| 17:51:19 | saratoga_ | ok |
| 17:51:21 | TheSeven | liar|netbook: I'll take that over, and propose to move it to #linux4nano |
| 17:51:31 | TheSeven | i'll send you some test files to try through ibugger |
| 17:51:37 | TheSeven | i guess there's something weird with your ftl |
| 17:51:40 | | Quit robin0800 (Read error: 60 (Operation timed out)) |
| 17:51:41 | liar|netbook | ok |
| 17:51:52 | linuxstb | TheSeven: Discussing it is fine here (but it's your choice...) |
| 17:52:35 | TheSeven | linuxstb: i suspect it's just some garbage lying around on the flash, already had such a case before where the OFW would mount the flash and my FTL wouldn't for some reason |
| 17:52:50 | TheSeven | debugging that would be quite noisy on IRC though |
| 17:54:11 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 17:54:53 | TheSeven | saratoga_: "CalmADM2E, a CalmRISC based audio DSP module, is designed for high-quality audio processing. It includes Samsung's 16-bit microprocessor, CalmRISC16E, and 24-bit DSP engine, CalmMAC24E." |
| 17:55:08 | saratoga_ | yeah reading it now |
| 17:56:09 | saratoga_ | apparently its clocked differently then the main core and has its own cache, so i guess its run as a seperate CPU and not as an extra multiplier for the arm core |
| 17:58:07 | TheSeven | yep. |
| 17:58:46 | saratoga_ | heh it can do hardware ring buffering |
| 17:59:12 | saratoga_ | so i guess the idea of this thing is that you map its address space onto some part of DRAM and give it a simple math kernel to grind through |
| 17:59:25 | saratoga_ | eventually it signals its done and the main CPU takes over |
| 17:59:33 | saratoga_ | sounds like a huge pain to use for codecs |
| 18:00 |
| 18:00:03 | linuxstb | What about running the entire codec on it? Or at least the "decode_frame()" function? |
| 18:00:15 | saratoga_ | its only 24 bit so that might be tricky |
| 18:01:15 | linuxstb | Well, first thing would be to find a compiler for it... |
| 18:01:22 | saratoga_ | that too |
| 18:01:26 | saratoga_ | or even the "CalmMAC24 DSP Coprocessor Architecture Reference Manual" |
| 18:02:26 | | Join lifeless_ [0] (n=lifeless@188.18.103.55) |
| 18:02:56 | saratoga_ | http://www.lw23.com/lunwen_815285917/ |
| 18:03:08 | saratoga_ | apparently someone once tried to implement vorbis on it |
| 18:04:06 | | Quit krazykit ("bbiab") |
| 18:05:56 | | Quit J-23 (Read error: 104 (Connection reset by peer)) |
| 18:06:35 | | Join mitk [0] (n=mitk@chello089077247055.chello.pl) |
| 18:07:05 | | Part mitk ("Leaving") |
| 18:08:27 | | Join J-23 [0] (n=zelazko@unix.net.pl) |
| 18:08:35 | CIA-43 | New commit by dave (r22920): Another patch from FS #10633 by Michael Sparmann (with a couple of minor cosmetic tweaks by me) - cleanly unmount the flash when powering off or ... |
| 18:08:52 | saratoga_ | i wonder why they used this stupid DSP core but didn't even bother to give the ARM cpu a fast MAC unit |
| 18:11:19 | | Join mitk [0] (n=mitk@chello089077247055.chello.pl) |
| 18:12:31 | | Join krazykit [0] (n=kkit@c-24-218-166-241.hsd1.ma.comcast.net) |
| 18:12:36 | CIA-43 | New commit by tomers (r22921): Remove USB mode displaying in non lcd-bitmap targets, as there are no such target which use USB HID |
| 18:14:08 | TheSeven | saratoga_: probably because they just didn't need it as the DSP is doing all the hard work |
| 18:15:46 | | Quit Thundercloud (Remote closed the connection) |
| 18:20:45 | | Quit ender` (" I know you think you understand what you thought I said, but I am not sure that what you heard is not what I meant.") |
| 18:21:20 | linuxstb | TheSeven: Some other screens (apart from the menu items which are file browsers) also seem to have the same problem - e.g. "System -> Debug -> View OS stacks" only displays a scrollbar and no information... |
| 18:21:45 | * | linuxstb wonders if it's a strange Rockbox bug due to lcd_update_rect() doing a full-screen update |
| 18:22:31 | saratoga_ | theres an op code chart here for the 16 bit core: http://www.donat.org/archos/documents/opcodes.html |
| 18:24:13 | * | bluebrother finally has an updated rbutil deployment script working as expected |
| 18:25:03 | kugel | linuxstb: possibly. can you try deactivating the statusbar? |
| 18:25:31 | linuxstb | kugel: I'm just trying a sim build with lcd_update_rect == lcd_update |
| 18:25:44 | | Join esperegu [0] (n=quassel@145.116.11.103) |
| 18:26:03 | linuxstb | But yes, I can try disabling the status bar whilst that's compiling... |
| 18:27:24 | linuxstb | Hmm, the actual status bar setting doesn't show any options, but if I try to select it blindly, then the settings change. Disabling the statusbar doesn't help though. |
| 18:27:27 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
| 18:27:30 | | Quit TheSeven (Read error: 145 (Connection timed out)) |
| 18:28:04 | linuxstb | The sim seems to be fine |
| 18:28:20 | * | linuxstb decides to just go ahead and implement lcd_update_rect, and see if that works... |
| 18:28:21 | kugel | several parts do set_viewport(NULL); before returning. it may lead to an empty screen if some following code does a clear_viewport(); update_viewport() without setting the proper vp |
| 18:28:42 | froggyman | If I post a compiled build for lolsack (from the forum) do I have to post the source? |
| 18:28:56 | kugel | I would suspect do_menu and/or list code |
| 18:29:49 | saratoga_ | i found a datahseet with a bit more about the 24 bit MAC, and it can do a single cycle 24x24 mul, shift and then 52 bit accumulate |
| 18:30:05 | saratoga_ | so you could fake a 32x32 fixped point mul in 2 clocks |
| 18:31:02 | saratoga_ | verese 5-6 on the main CPU, so its definately a little faster |
| 18:33:38 | saratoga_ | its also got a hardware bit reverse operation for doing ffts |
| 18:35:53 | saratoga_ | i guess if someone was really motivated they could reuse my old dual core mp3 code with this by running the synthesis filterbank on the DSP |
| 18:39:10 | *** | Saving seen data "./dancer.seen" |
| 18:41:27 | CIA-43 | New commit by bluebrother (r22922): Add config option to disable ccache during build. ... |
| 18:45:29 | pixelma | I'd like to help finding out what happens with Rockbox USB on MacOS 10.4 (using my c200). A build that has USB_HID disabled works correctly (I can access the device after it appeared on the desktop), with HID enabled this doesn't happen and on first connect the OS tries to identify a keyboard. Does anybody have any idea what I could look for (and where, I don't know Mac OS too well) |
| 18:45:49 | pixelma | ? |
| 18:46:01 | CIA-43 | New commit by bluebrother (r22923): Rework rbutil deployment script. ... |
| 18:47:49 | pixelma | system profiler (that's called in a German version) still shows a "Rockbox media player" connected on the USB port |
| 18:50:31 | | Quit saratoga_ ("Page closed") |
| 18:52:13 | | Quit antil33t (Read error: 104 (Connection reset by peer)) |
| 18:52:20 | | Join antil33t [0] (n=Mudkips@119.224.12.185) |
| 18:53:17 | | Join TheSeven [0] (n=theseven@dslb-084-056-154-236.pools.arcor-ip.net) |
| 18:54:27 | | Join Omlet [0] (n=omlet05@133.99-240-81.adsl-dyn.isp.belgacom.be) |
| 18:54:32 | | Join Omlet05 [0] (n=omlet05@133.99-240-81.adsl-dyn.isp.belgacom.be) |
| 18:55:12 | | Quit Omlet05 (Client Quit) |
| 18:55:18 | | Quit pixelma (Read error: 104 (Connection reset by peer)) |
| 18:55:19 | | Quit amiconn (Read error: 54 (Connection reset by peer)) |
| 18:55:25 | | Quit Omlet (Client Quit) |
| 18:55:36 | | Join pixelma [0] (i=quassel@rockbox/staff/pixelma) |
| 18:55:38 | | Join amiconn [0] (i=quassel@rockbox/developer/amiconn) |
| 18:57:22 | | Join darkham [0] (n=darkham@host172-36-dynamic.50-79-r.retail.telecomitalia.it) |
| 19:00 |
| 19:00:23 | | Quit amiconn (Remote closed the connection) |
| 19:00:24 | | Quit pixelma (Remote closed the connection) |
| 19:02:30 | | Join amiconn [0] (i=quassel@rockbox/developer/amiconn) |
| 19:02:30 | | Join pixelma [0] (i=quassel@rockbox/staff/pixelma) |
| 19:03:26 | | Quit kugel (Read error: 110 (Connection timed out)) |
| 19:05:04 | | Nick YPSY is now known as Ypsy (n=ypsy@geekpadawan.de) |
| 19:06:23 | | Join kugel_ [0] (n=kugel@e178097144.adsl.alicedsl.de) |
| 19:06:31 | | Nick kugel_ is now known as kugel (n=kugel@e178097144.adsl.alicedsl.de) |
| 19:08:52 | | Quit TheSeven (Read error: 104 (Connection reset by peer)) |
| 19:09:19 | | Join TheSeven [0] (n=theseven@dslb-084-056-154-236.pools.arcor-ip.net) |
| 19:11:10 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
| 19:12:08 | | Quit robin0800 (Remote closed the connection) |
| 19:24:06 | linuxstb | TheSeven: Can you quickly test linuxstb.cream.org/rockbox.ipod">http://linuxstb.cream.org/rockbox.ipod ? It implements lcd_update_rect(). If the menus still work, then the function works... |
| 19:24:26 | linuxstb | (it works on my lcd type, but needs a quick test on your's). It doesn't fix the menu problem with plugins etc though. |
| 19:24:31 | domonoky | pixelma: what does dmesg say when you connect rockbox usb hid to the mac ? |
| 19:24:58 | linuxstb | TheSeven: (or I can post a patch) |
| 19:25:12 | pixelma | domonoky: just checked: lots of lines saying "AppleUSBEHCI[0x2568800]::Found a transaction past the completion deadline on bus 75, timing out!" which I already found in the system.log |
| 19:25:28 | kugel | linuxstb: maybe there's a threading problem? IIRC there are a few splashes outside the main thread |
| 19:25:56 | | Nick DaveDavenport is now known as Qt (n=qball@unaffiliated/qball) |
| 19:26:10 | | Nick Qt is now known as Ma (n=qball@unaffiliated/qball) |
| 19:26:20 | | Join funman [0] (n=fun@rockbox/developer/funman) |
| 19:26:28 | linuxstb | kugel: The problem is that some lists are not displaying the items or the selection bar. It seems unrelated to any splashes happening. |
| 19:27:03 | | Nick Ma is now known as DaveDavenport (n=qball@unaffiliated/qball) |
| 19:27:14 | kugel | does deactivating the scrollbar and/or changing the line selector change anything? |
| 19:27:43 | | Quit stoffel (Read error: 113 (No route to host)) |
| 19:27:45 | linuxstb | In such lists, the icons and (if needed) scrollbar are showing. Some examples are the "show OS stacks" screen, and the plugin/themes/wps/font etc file browsers. The same directories display fine when accessed via the normal file browser. |
| 19:29:31 | linuxstb | No, I don't think those settings help. But those settings themselves are examples that don't display... |
| 19:29:38 | kugel | the latter use a slightly different filebrowser (a wrapper which does additional stuff) IIRC |
| 19:30:07 | kugel | maybe still a stack problem? |
| 19:30:29 | linuxstb | Possibly, although I would have thought the kernel should detect it and give a stkov |
| 19:30:50 | kugel | I'm not sure if it the detection works for the main thread |
| 19:30:56 | linuxstb | I'm also running with a 16KB stack. |
| 19:31:19 | pixelma | domonoky: any other idea what I could check? |
| 19:31:27 | amiconn | The stkov detection works for all threads |
| 19:31:29 | TheSeven | linuxstb: your .ipod is running fine for me |
| 19:31:40 | linuxstb | TheSeven: OK, thanks. I'll commit it then. |
| 19:32:01 | TheSeven | and it's *way*faster than the old one |
| 19:32:10 | TheSeven | (especially in some plugins) |
| 19:32:19 | kugel | that's the point of update_rect() :P |
| 19:32:53 | CIA-43 | New commit by dave (r22924): Implement lcd_update_rect() |
| 19:33:01 | TheSeven | linuxstb: have a look at the plasma demo... any idea where that wandering vsync-like line comes from? or is this also occurring on other targets? |
| 19:33:05 | linuxstb | TheSeven: Good, so it's definitely working! I also noticed the speed difference. |
| 19:33:21 | linuxstb | TheSeven: I just tried plasma a few minutes ago and didn't notice that. I'll look again... |
| 19:33:57 | | Quit efyx_ (Remote closed the connection) |
| 19:34:03 | amiconn | TheSeven: It also happens on other targets. It's just that the lcd update is not (and can not) be synchronised with the internal timing of the lcd controller |
| 19:34:11 | TheSeven | i'll *try* to capture a shot of that issue |
| 19:34:12 | linuxstb | Ah yes, it is there, but quite hard to see. |
| 19:34:20 | domonoky | pixelma: there is no mention of hid in dmesg ? but it dont have other ideas to check.. |
| 19:34:35 | TheSeven | linuxstb: it's quite noticable on mine, as it's wandering pretty slowly |
| 19:34:55 | | Quit barrywardell () |
| 19:35:25 | funman | AlexP: did you put your e200v2 manual work on flyspray? |
| 19:35:36 | | Quit faemir (Read error: 131 (Connection reset by peer)) |
| 19:35:44 | amiconn | It's most visible if (by chance) the update frequency almost matches the internal refresh rate of the controller |
| 19:35:47 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 19:35:57 | | Quit froggyman ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
| 19:36:08 | pixelma | domonoky: no the only other line I can see sometimes reads "AppleUSBEHCI[0x2568800]::DoIOTransfer - error 0xe000404f queueing request" |
| 19:36:47 | | Join faemir [0] (n=faemir@78.33.109.163) |
| 19:37:25 | domonoky | pixelma: looks like macosx does something different with usb hid then, which rockbox doesnt correctly handle. |
| 19:37:45 | linuxstb | TheSeven: BTW, Rockbox has a "debug" menu - would it be useful to have a "debug NAND" screen displaying various info? |
| 19:38:45 | pixelma | it's "only" 10.4. At least I saw no complaints otherwise - and which is the reason I wanted to help because I have access to a Mac running this version |
| 19:38:48 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 19:41:27 | mc2739 | funman: I think AlexP has committed his work on the manual and requested Bagder to place it on the download server. But, Bagder is on vacation, so not sure if that part was accomplished before he left. |
| 19:41:31 | * | bluebrother would like to see someone test FS #10100 but can't provide binaries :( |
| 19:42:07 | domonoky | pixelma: some webpage suggest "HID Explorer" to debug usb hid issues on mac. dont know if that will help, if the system fails to communicate with rockbox. |
| 19:44:27 | linuxstb | domonoky: BTW, I was looking at your ipodpatcher change for ramsize detection, and it means that ipodpatcher itself now gets the XML twice (in the place you added, and in main.c). Plus you didn't put windows-only code (the requirement to remount r/w) within __WIN32_. But don't worry too much - I'll probably clean it up... |
| 19:45:09 | pixelma | by the way - Rockbox changes into the USB screen |
| 19:45:17 | domonoky | linuxstb: i think bluebrother did this :-) but thanks for fixing anyway :-) |
| 19:45:31 | linuxstb | Too many Dominics... |
| 19:45:32 | * | bluebrother remembers doing that. Oops :) |
| 19:45:44 | linuxstb | Or even Dominiks |
| 19:45:47 | bluebrother | linuxstb: just use the nicknames ;-) |
| 19:45:58 | * | linuxstb is sure he saw "domonoky" in the svn log |
| 19:46:32 | linuxstb | No, I obviously didn't... |
| 19:46:36 | | Quit funman ("leaving") |
| 19:46:38 | TheSeven | picture of "vsync" behavior i meant (line at the bottom): http://img524.imageshack.us/img524/9494/img0966m.jpg |
| 19:48:01 | pixelma | interesting, I sometimes get a reaction when changing to "Browser mode" for the HID mode and randomly pressing buttons, looks like it is detected as a continous press of "up". When I disconnect then Rokbox hangs in the USB screen and can only be turned off by a hard power off |
| 19:48:25 | linuxstb | bluebrother: The xml reading should probably be done elsewhere (not in either of the places you and I added it). I've a lot of ipodpatcher work to do (to support the Nano2G fully), so will have a think about it when I'm working on that. |
| 19:48:37 | | Quit mitk ("Leaving") |
| 19:48:47 | pixelma | domonoky: I'm more concerned that I can't get access to the disk, not the functionality of the HID modes :\ |
| 19:49:45 | bluebrother | linuxstb: ok. I was thinking about other solutions as well, but that was the easiest to at least use it from within rbutil without too much changes. Hopefully I'll get around looking into the autodetection stuff in a while, as beastpatcher also requires changes over there. |
| 19:49:49 | domonoky | pixelma: uh so rockbox-usb is broken completly on mac ? |
| 19:50:02 | amiconn | pixelma: But it does work if you build without hid support, right? |
| 19:50:36 | pixelma | domonoky: on all (3) 10.4 boxes I tried - amiconn: yes |
| 19:51:12 | | Join stoffel [0] (n=quassel@p57B4E773.dip.t-dialin.net) |
| 19:51:18 | | Part toffe82 |
| 19:51:20 | pixelma | 10.5 is probably more common nowadays, even 10.6 is out for a short while |
| 19:52:12 | amiconn | Do 10.5 (or even 10.6) still support PPC? |
| 19:52:21 | amiconn | *Does |
| 19:52:56 | pixelma | I'm quite sure 10.6 does not, don't know about 10.5 |
| 19:53:52 | domonoky | afaic you can build universal binarys on 10.5, so it should support PPC.. |
| 19:55:54 | | Quit darkham ("Sto andando via") |
| 19:59:24 | TheSeven | linuxstb: where should i start if i want to have a shot at making audio work? |
| 20:00 |
| 20:00:01 | linuxstb | Firstly at my patch for audio - to check that everything in there that looks important was committed. |
| 20:00:36 | linuxstb | And then also at the pcm-s5l8700.c file to see if that code makes sense. |
| 20:01:14 | TheSeven | it does, from a first glance |
| 20:01:56 | linuxstb | Maybe the problem is the same as the one which stopped my bootloader test working if I left that clock-related code in crt0.S |
| 20:02:06 | TheSeven | how were you doing that audio test? was that using the same apis like playing music, or did you inject the data in a more lowlevel place? |
| 20:02:06 | linuxstb | So Rockbox just freezes when you try and play audio? |
| 20:02:22 | TheSeven | well, playback just stays at 0 seconds |
| 20:02:35 | linuxstb | Yes, so I guess nothing is being transferred to the DAC |
| 20:02:35 | TheSeven | and no audible reaction on the output |
| 20:02:44 | linuxstb | Same symptoms as I had. |
| 20:03:03 | TheSeven | i just wanted to know if i need to dig into that or if there's just something still lacking that's preventing it from working at all |
| 20:03:11 | | Join webguest78 [0] (i=www-data@giant.haxx.se) |
| 20:03:25 | | Join webguest74 [0] (i=www-data@giant.haxx.se) |
| 20:03:36 | linuxstb | I was just using the API that firmware/ provides to apps/ - i.e. the "pcm_play_data()" (I think) function that takes a pointer to the first chunk of audio, and a pointer to a "get_more()" function, that is called in the DMA ISR to get a pointer to the next chunk of data to transfer. |
| 20:04:48 | linuxstb | TheSeven: I don't think anything is missing. |
| 20:05:10 | TheSeven | ok, then i'll do another test build without the clock setup and see if that changes something |
| 20:06:06 | | Join FOAD_ [0] (n=dok@dinah.blub.net) |
| 20:07:06 | linuxstb | TheSeven: BTW, are you happy for me to close FS #10633 now? As I wrote on a comment there, I think the only things not now committed are the stack change and the iBugger stuff. I don't need a patch for the stack change, and iBugger belongs as a new task. |
| 20:07:52 | | Quit flydutch ("/* empty */") |
| 20:08:10 | | Quit webguest78 ("CGI:IRC (Ping timeout)") |
| 20:08:23 | TheSeven | linuxstb: I don't think the ibugger thing is really relevant any more at all (as we can only debug the bootloader using it), it was just that it was connected pretty closely to the cache fixes |
| 20:09:29 | linuxstb | TheSeven: OK. Could that kind of debugger be integrated into Rockbox? It may be nice to port it to other targets. |
| 20:10:04 | linuxstb | And we probably can't debug the bootloader for much longer - I want to change that to run from IRAM (so it can load Rockbox directly to SDRAM). |
| 20:10:14 | TheSeven | i think it deserves a rewrite, and it's hardly portable (at least the device side) |
| 20:10:21 | TheSeven | most of it is lowlevel USB ASM code |
| 20:10:37 | TheSeven | once the driver is up and running, the rest is done quickly |
| 20:11:14 | TheSeven | and concerning IRAM/DRAM: That's exactly what this patch was addressing |
| 20:11:20 | TheSeven | if DEBUG, then run drom DRAM, else IRAM |
| 20:12:45 | linuxstb | BTW, do you have any ipod accessories? Rockbox has basic support for those on ipods (remotes etc), which communicate via the UART, so that seems relatively straightforward to port to the Nano2G |
| 20:16:20 | | Quit stoffel (Read error: 104 (Connection reset by peer)) |
| 20:18:35 | | Join gregzx [0] (n=chatzill@dtm222.neoplus.adsl.tpnet.pl) |
| 20:19:03 | | Join Lear [0] (i=chatzill@rockbox/developer/lear) |
| 20:21:10 | | Quit webguest74 ("CGI:IRC (Ping timeout)") |
| 20:21:17 | | Quit JackWinter (Remote closed the connection) |
| 20:21:49 | | Join webguest67 [0] (i=www-data@giant.haxx.se) |
| 20:22:14 | | Join JackWinter [0] (n=jack@vodsl-10804.vo.lu) |
| 20:22:21 | | Quit FOAD (Read error: 110 (Connection timed out)) |
| 20:22:22 | | Nick FOAD_ is now known as FOAD (n=dok@dinah.blub.net) |
| 20:26:08 | TheSeven | linuxstb: nope, i don't have addons |
| 20:26:13 | TheSeven | accessories* |
| 20:26:36 | TheSeven | and i don't have audio either (still just hanging at 0 seconds), even with disabled clock setup |
| 20:26:51 | | Nick intrados1 is now known as intrados (n=intrados@cpe-75-187-57-252.columbus.res.rr.com) |
| 20:27:40 | linuxstb | Hmm, let me look at my old patch again... |
| 20:30:03 | | Join biengo [0] (n=quassel@xdsl-84-44-231-43.netcologne.de) |
| 20:30:25 | linuxstb | One change I see is to replace the call to wmcodec_write(SAMPCTRL, WM8975_44100HZ); with wmcodec_write(SAMPCTRL, 0); in firmware/drivers/audio/wm8975.c:audiohw_preinit() |
| 20:30:33 | linuxstb | (I can't remember why that was there though...) |
| 20:31:18 | TheSeven | linuxstb: another point could be that the ofw was initializing it for us, as i think we booted through notes-ibugger back then |
| 20:32:19 | linuxstb | Hmm, do we actually implement a handler for the DMA interrupt? |
| 20:32:42 | linuxstb | Looks like my test code wasn't doing that... |
| 20:33:08 | TheSeven | linuxstb: I'm gonna try my "yippie" sample again through iloader-ibugger, to see if this depends on the ofw setup |
| 20:33:13 | | Join AsaelReiter [0] (i=www-data@giant.haxx.se) |
| 20:33:40 | linuxstb | (it looks like it was just playing a 1MB chunk in a single DMA transfer) |
| 20:34:34 | TheSeven | shoudln't i at least get *anything* on the output then? |
| 20:34:38 | TheSeven | it just stays silent |
| 20:35:01 | | Quit esperegu (Read error: 104 (Connection reset by peer)) |
| 20:35:04 | linuxstb | If it's working, you would just get the first 32KB of PCM data - which is likely to be silent for most tracks. |
| 20:35:18 | TheSeven | oops, clickwheel just stopped responding (hold switch still does) after some time on the playback screen |
| 20:35:57 | | Join esperegu [0] (n=quassel@145.116.11.103) |
| 20:37:10 | linuxstb | TheSeven: But yes, testing your code in iloader would be useful, to rule out the OF setup. |
| 20:37:27 | TheSeven | it's failing :-/ |
| 20:37:36 | TheSeven | let's install notes-ibugger again |
| 20:39:12 | *** | Saving seen data "./dancer.seen" |
| 20:39:35 | polobricolo | linuxstb:i have a dock if you want me to try something |
| 20:39:49 | polobricolo | (with my nano2g) |
| 20:40:26 | linuxstb | polobricolo: I haven't looked at Rockbox's "Apple Accessories" code at all (so can't really help), but if you wanted, it might be fun to try and get that working on your Nano. |
| 20:41:04 | linuxstb | Does your dock have buttons to control the ipod? |
| 20:41:21 | linuxstb | (although it might be hard to test when Rockbox playback isn't working...) |
| 20:41:30 | TheSeven | skipping will work |
| 20:42:22 | TheSeven | linuxstb: re-verified: on notes-ibugger it works |
| 20:42:36 | TheSeven | i'll now upload a rockbox build through notes-iloader and check that |
| 20:42:47 | TheSeven | notes-ibugger* |
| 20:42:59 | TheSeven | even though notes-ibugger-iloader would also be a nice (and working!) chain |
| 20:43:42 | TheSeven | "ATA error 1" - shit |
| 20:44:08 | TheSeven | yes, of course the OFW can't unmount cleanly if notes crashes it |
| 20:45:47 | TheSeven | let's try the trivial first: clock gates |
| 20:46:18 | polobricolo | linuxstb: yes. it also has a remote |
| 20:46:46 | polobricolo | where is it implented in rockbox ? |
| 20:47:17 | linuxstb | polobricolo: I've no idea... |
| 20:48:14 | linuxstb | apps/iap.c seems to be the main (high-level) code |
| 20:48:50 | liar|netbook | TheSeven: you got that ata error 1 too? |
| 20:49:56 | TheSeven | liar|netbook: yep, when I'm trying to mount a flash that wasn't unmounted cleanly before |
| 20:50:00 | linuxstb | polobricolo: Seems to also be some lower-level code in firmware/drivers/serial.c (search for IPOD - it seems badly hard-coded to assume all ipods are PortalPlayer ;) ) |
| 20:50:29 | TheSeven | linuxstb: clock gates didn't help |
| 20:51:01 | TheSeven | maybe some clock thing again, or i2s controller setup? |
| 20:51:32 | linuxstb | polobricolo: This is the original patch to implement accessory support - scroll to the end to see what was eventually committed, as that patch was a long time in the development. http://www.rockbox.org/tracker/task/8624 |
| 20:51:32 | TheSeven | or DMA... |
| 20:52:13 | linuxstb | Audio worked on the Meizu, and that was being booted from DFU mode (i.e. no OFW initialisations) |
| 20:53:36 | saratoga | any one have an idea how a codec returning an error can deadlock playback on target? |
| 20:54:03 | saratoga | that MPC SV8 sample is handled correctly by MPC as far as I can tell, just playback doesn't seem to like having the codec return an error |
| 20:56:01 | | Join robin0800 [0] (n=robin080@cpc3-brig8-0-0-cust436.brig.cable.ntl.com) |
| 20:56:02 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 20:57:04 | | Quit AsaelReiter ("CGI:IRC") |
| 20:57:32 | * | TheSeven will do an FTL hack that allows mounting unclean FTLs now |
| 20:58:30 | saratoga | huh seems to only happen once and a while |
| 20:58:56 | | Join GeekShado_ [0] (n=Antoine@APoitiers-552-1-27-19.w86-217.abo.wanadoo.fr) |
| 21:00 |
| 21:01:15 | TheSeven | linuxstb: rockbox playback doesn't work either if i boot it through notes-ibugger |
| 21:04:03 | biengo | AlexP: I've uploaded new versions of the Cassette and Mountain-Fog theme to the e200 theme site, could you remove the old versions, eventually? thanks. |
| 21:04:23 | TheSeven | liar|netbook: can you try /tmp/rockbox.ipod-forcemount.7z and see if that fails, too? |
| 21:07:20 | | Quit faemir (Read error: 54 (Connection reset by peer)) |
| 21:09:03 | | Quit biengo (Dead socket) |
| 21:09:03 | | Quit bluebrother (Read error: 104 (Connection reset by peer)) |
| 21:09:20 | | Join biengo [0] (n=quassel@xdsl-84-44-231-43.netcologne.de) |
| 21:12:07 | linuxstb | TheSeven: Have you looked at the code in dma-s5l8700.c ? Could that conflict in some way with the NAND driver's use of DMA? |
| 21:12:32 | liar|netbook | TheSeven: it fails too |
| 21:12:36 | TheSeven | oh, didn't think of this |
| 21:12:42 | TheSeven | i'll have a look asap |
| 21:12:52 | TheSeven | liar|netbook: then it's not just a dirty mark but something more obscure |
| 21:13:22 | TheSeven | liar|netbook: if you have a build environment at hand, try splashf'ing it |
| 21:13:32 | | Join bluebrother [0] (n=dom@rockbox/developer/bluebrother) |
| 21:13:47 | | Join funman [0] (n=fun@rockbox/developer/funman) |
| 21:14:10 | | Quit GeekShadow (Read error: 113 (No route to host)) |
| 21:14:24 | linuxstb | saratoga: Does that happen on both the sim and a target? |
| 21:14:36 | liar|netbook | TheSeven: splashf'ing? |
| 21:14:51 | | Join faemir [0] (n=faemir@78.33.109.163) |
| 21:15:12 | saratoga | linuxstb: it happens on my sansa and on the ipod apparently |
| 21:15:15 | saratoga | didn't try it elsewhere |
| 21:15:24 | TheSeven | add debug output (or better return codes as linuxstb suggested) and see what's going on |
| 21:16:12 | TheSeven | linuxstb: when I read the I2S regs, I get all zeroes? :-/ |
| 21:16:20 | CIA-43 | New commit by kkurbjun (r22925): Brickmania - Fix and enable the absolute touchscreen mode. |
| 21:16:39 | linuxstb | TheSeven: Hmm, they're write-only? |
| 21:16:56 | linuxstb | When are you reading them? |
| 21:16:57 | TheSeven | wouldn't expect that |
| 21:17:00 | polobricolo | would it be possible to disable the automatic shutdown on the nano2g as it does not shutdown (causes apanic) |
| 21:17:01 | stripwax | Does definiing MAX_PHYS_SECTOR_SIZE give worse disk performance than not defining it? Thinking that all ipod videos have this define but only 80GB 5.5G needs it |
| 21:17:16 | TheSeven | i just tried to do some reg dumps from iloader-ibugger and notes-ibugger and compare |
| 21:17:22 | linuxstb | stripwax: Well, we don't have a separate build for 80GB ipods... |
| 21:17:30 | linuxstb | (only 32MB/64MB) |
| 21:17:30 | stripwax | and I'm wondering why not .. :) |
| 21:18:00 | linuxstb | Wouldn't that give us four builds for the video? |
| 21:18:07 | stripwax | Only three. |
| 21:18:28 | linuxstb | Why? Which combination can't exist? |
| 21:18:30 | TheSeven | there is no 80gb 32mb one |
| 21:18:41 | TheSeven | (iirc) |
| 21:18:49 | stripwax | unless we decide to provide a build for use with 32MB ipods that have an upgraded disk |
| 21:19:05 | linuxstb | There shouldn't be, but refurbished 80GBs sometimes have a 32MB main board (IIRC) |
| 21:19:12 | stripwax | ... yuk |
| 21:19:26 | | Quit BHSPitLappy (Remote closed the connection) |
| 21:19:39 | linuxstb | That what causes the problems with people trying a 64MB build on 80GB ipods - it turns out they only have 32MB RAM. |
| 21:19:44 | | Join JackWinter2 [0] (n=jack@vodsl-10804.vo.lu) |
| 21:20:11 | stripwax | they should take them back and demand a replacement replacement :) |
| 21:20:13 | linuxstb | Although ipodpatcher can detect both sector size and RAM size now, so in theory rbutil can support detection. (although no RAM size detection on OS X yet). |
| 21:20:15 | | Join sig11 [0] (n=1049A2D8@ypokhilko.vn.ua) |
| 21:21:37 | TheSeven | linuxstb: I'm using DMA channel 3 for NAND. which one does i2s use? |
| 21:21:41 | * | domonoky would prever if this detection is done in rockbox itself. only one ipod video build would be nice, and less confusing for users :-) |
| 21:23:38 | gevaerts | Does the 5.5G OF and EDM *always* present 2k sectors over USB, even with other disks? |
| 21:24:37 | * | n1s agrees with domonoky |
| 21:25:08 | pixelma | domonoky: the plan is there sinde DevCon 2007... |
| 21:25:16 | pixelma | since too |
| 21:26:12 | gevaerts | if it does, basing sector size on model id would have huge advantages : no more guessing in disk.c, so no more bootloaders getting confused by remnants of 512-byte MBRs, no more guessing in usb_storage.c |
| 21:26:16 | | Join matsl [0] (n=matsl@1-1-4-2a.mal.sth.bostream.se) |
| 21:27:13 | domonoky | pixelma: plans dont count.. 2007 is long ago. :-) |
| 21:27:18 | | Quit esperegu (Read error: 104 (Connection reset by peer)) |
| 21:27:41 | saratoga | if no one minds I'd like to disable prebuffering of *.codec files on low memory targets |
| 21:27:56 | saratoga | it seems to improve stability and buffering them on low memory targets is completely broken anyway |
| 21:28:21 | pixelma | domonoky: that was my point (that it's long ago) |
| 21:32:20 | | Quit JackWinter (Read error: 110 (Connection timed out)) |
| 21:32:33 | | Join Grahack [0] (n=chri@ip-222.net-82-216-222.rev.numericable.fr) |
| 21:33:23 | | Join bughunter2 [0] (n=bughunte@unaffiliated/bughunter2) |
| 21:34:05 | | Quit Sajber^ (Read error: 131 (Connection reset by peer)) |
| 21:36:17 | | Quit GeekShado_ (Read error: 104 (Connection reset by peer)) |
| 21:36:32 | | Quit biengo (Remote closed the connection) |
| 21:36:40 | | Join GeekShado_ [0] (n=Antoine@APoitiers-552-1-27-19.w86-217.abo.wanadoo.fr) |
| 21:36:48 | | Quit matsl ("Leaving") |
| 21:36:57 | pixelma | btw. I found one box running MacOS 10.5.8 and tried with my c200 again - it also wanted to identify a keyboard first which I then interrupted, it mounted the two disks in the end but it took quite a while (had trouble ejecting too) |
| 21:37:03 | | Join matsl [0] (n=matsl@1-1-4-2a.mal.sth.bostream.se) |
| 21:38:26 | pixelma | and while the HID functions also seemed to work it didn't work as nicely as it does on my XP laptop. The keys "stick" sometimes and in mouse mode there is always a jump into a 90° different direction in the end |
| 21:41:42 | | Quit Lear ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
| 21:43:27 | | Join benime [0] (n=benime@c-68-40-80-75.hsd1.mi.comcast.net) |
| 21:44:53 | | Quit janne_ok1anen ("leaving") |
| 21:45:36 | | Join skyhunter [0] (n=quassel@f054011045.adsl.alicedsl.de) |
| 21:48:16 | | Join janne_oksanen [0] (n=janne@b01.ainola.jyu.fi) |
| 21:48:32 | polobricolo | i have to go i'll have a look at iap tomorrow |
| 21:50:05 | | Quit robin0800 (Remote closed the connection) |
| 21:50:20 | | Join robin0800 [0] (n=robin080@cpc3-brig8-0-0-cust436.brig.cable.ntl.com) |
| 21:51:14 | | Join MethoS- [0] (n=clemens@134.102.106.250) |
| 21:51:35 | | Quit robin0800 (Remote closed the connection) |
| 21:51:40 | | Join mt2 [0] (n=mtee@41.233.155.27) |
| 21:58:46 | | Join pamaury [0] (n=pamaury@91-164-186-177.rev.libertysurf.net) |
| 22:00 |
| 22:00:31 | | Join barrywardell [0] (n=barrywar@p5B25A343.dip0.t-ipconnect.de) |
| 22:00:37 | | Quit skyhunter (Read error: 104 (Connection reset by peer)) |
| 22:01:02 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 22:01:30 | | Quit GeekShadow (Read error: 104 (Connection reset by peer)) |
| 22:04:18 | | Quit Grahack ("Leaving.") |
| 22:05:22 | mc2739 | pixelma: ping |
| 22:06:16 | pixelma | pong? |
| 22:07:03 | mc2739 | pixelma: do you know what is needed to fix this error while building the manual? −− ! LaTeX Error: File `xcolor.sty' not found. |
| 22:08:23 | funman | mc2739: you must install the tex package providing xcolor (it's in the description of ubuntu packages) |
| 22:08:26 | pixelma | if I understand correctly, you are missing the xcolor package (it was included by default in my LaTeX packages so can't tell how it is called exactly) |
| 22:08:29 | gevaerts | you need to install the package containing xcolor.sty. On debian-based systems, it's probably latex-xcolor |
| 22:09:06 | mc2739 | ok, thanks |
| 22:09:23 | pixelma | mc2739: which packages did you install? |
| 22:09:40 | mc2739 | Is this something new? I'm sure I could build manuals before, although it has been a while |
| 22:09:50 | linuxstb | TheSeven: (sorry, I was away from my computer). Looking at target/arm/s5l8700/dma-target.h, I2S is using channel 0. |
| 22:10:41 | mc2739 | I installed using the wiki directions, except texlive was installed |
| 22:12:07 | domonoky | mc2739: yes the use of xcolor in the manual is pretty new. |
| 22:12:29 | mc2739 | found it - it is latex-xcolor |
| 22:13:21 | linuxstb | saratoga: Any idea if disabling pre-buffering of codecs would break gapless (i.e. by introducing a delay when reading the codec binary from disk) ? But I guess that's not important, as inter-codec gapless isn't that useful... |
| 22:14:40 | pixelma | mc2739: yes, it's new (well quite) and is used to colour the tables a bit to make them better readable |
| 22:15:12 | saratoga | linuxstb: i'll do some testing but i doubt it given the length of the audio buffer |
| 22:15:45 | mc2739 | thanks to all - manual build is working now |
| 22:15:48 | linuxstb | How big is that? The same 512KB as large-mem targets? |
| 22:17:54 | | Quit GeekShado_ (Read error: 113 (No route to host)) |
| 22:19:00 | saratoga | linuxstb: 176KB |
| 22:19:08 | pixelma | barrywardell: are you really around (and still using a Mac)? |
| 22:19:25 | barrywardell | pixelma: yes, and yes ;) |
| 22:19:55 | funman | 176kB is the size of pcm buffer |
| 22:19:58 | linuxstb | saratoga: Hmm, only 1 second. But I guess that's long enough to load a codec and start decoding... |
| 22:20:18 | saratoga | linuxstb: codec files are less then 100KB which should take very little time to load on a flash target |
| 22:20:24 | saratoga | but of course that needs testing |
| 22:20:27 | TheSeven | linuxstb: I think I'm close. |
| 22:20:45 | TheSeven | it's clocking again |
| 22:20:46 | pixelma | barrywardell: you have an e200, right.. how's Roclbox USB working for you (and which OSX version exactly)? |
| 22:20:58 | pixelma | Rockbox too... |
| 22:21:22 | barrywardell | pixelma: yeah, I have an e200 and currently have OSX 10.5.8 |
| 22:21:42 | linuxstb | TheSeven: Hurray! ;) When the time comes - http://www.rockbox.org/wiki/GentlemenMails (I think proper audio playback inside Rockbox itself deserves a second email) |
| 22:21:48 | TheSeven | linuxstb: I managed to make the DMA transfer work (and the I2S status bits behave like expected), but still no audio... power to the WMCODEC shut off? |
| 22:22:10 | barrywardell | pixelma: usb was working ok last time I tried, but will check again now |
| 22:22:22 | funman | TheSeven: or wrong audio output selected |
| 22:23:09 | linuxstb | TheSeven: So is Rockbox showing an elapsed time other than zero? |
| 22:24:14 | TheSeven | linuxstb: didn't play around with rockbox yet, but made DMACTCNT0 run down in the audiotest sample |
| 22:24:20 | pixelma | barrywardell: I'm asking (don't know if you read logs) because I have trouble with it on MacOS 10.4.11 (and although it worked on the one 10.5.8 box, it seems to work better on my XP laptop for me) |
| 22:25:04 | linuxstb | TheSeven: Ah, OK. As funman said, it could be that you haven't configured the WM8975's mixer correctly. |
| 22:25:08 | TheSeven | CLKCON &= ~0x80; does the trick, still need to check what this is in the first place |
| 22:25:47 | TheSeven | hah. MCLK_MASK |
| 22:25:52 | TheSeven | we should have known that... |
| 22:26:01 | TheSeven | "Mask audio clock into SPDIF or IIS block" |
| 22:26:14 | linuxstb | ;) Yes, that sounds useful... |
| 22:26:30 | TheSeven | should I just unmask that in crt0.S? |
| 22:26:49 | TheSeven | (this is probably what screwed audio as soon as the clock stuff was enabled) |
| 22:27:03 | linuxstb | Probably somewhere in the audio init, let me look... |
| 22:27:05 | kugel | audio on nanog2? |
| 22:27:34 | TheSeven | only after running through the OFW once, but we're close to getting it working inside rockbox |
| 22:27:35 | linuxstb | kugel: We had that a while ago in test apps, but now TheSeven is debugging why it's not working in Rockbox itself. |
| 22:27:48 | kugel | ah alright |
| 22:27:52 | gevaerts | What's the status of the manual and rbutil integration? ;) |
| 22:28:13 | linuxstb | "cp manual-ipodnano.pdf manual-ipodnano2g.pdf" - done! |
| 22:28:24 | pixelma | screenshots and button tables etc. for the manual will be easy |
| 22:28:50 | linuxstb | I still have a little more work on ipodpatcher, but when that's done, the install will be identical to earlier ipods. |
| 22:28:52 | pixelma | maybe some other installation instruction could be usefull |
| 22:29:01 | linuxstb | (from a user's point of view) |
| 22:29:05 | * | pixelma takes that back |
| 22:29:52 | TheSeven | linuxstb: we're *explicitly* masking MCLK in the clock setup |
| 22:29:54 | | Quit gevaerts (Nick collision from services.) |
| 22:30:03 | | Join gevaerts [0] (n=fg@rockbox/developer/gevaerts) |
| 22:30:09 | barrywardell | pixelma: current svn in "multimedia mode" seems to work ok for me |
| 22:31:57 | linuxstb | TheSeven: pcm_play_dma_init() in pcm-s5l8700.c seems the right place. Unless you think there's no harm in just enabling it in crt0.S |
| 22:32:09 | barrywardell | pixelma: copying a 700mb file is quite fast (~9MB/s) |
| 22:32:12 | TheSeven | it is - but it didn't help either |
| 22:32:47 | barrywardell | pixelma: but pushing buttons while copying seems to make the copying freeze |
| 22:33:06 | linuxstb | TheSeven: Does your test app do the same inits as crt0.S ? |
| 22:33:19 | pixelma | barrywardell: does it always take such a long time to mount (appear on the desktop), thought it takes longer than on XP but that was the first connect |
| 22:33:36 | TheSeven | linuxstb: it doesn't do any |
| 22:33:49 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 22:33:51 | TheSeven | but it's not only the mask, MCLK gets screwed in some further ways |
| 22:34:03 | barrywardell | pixelma: no, it was quite quick to mount. |
| 22:34:18 | TheSeven | and we're using PLL2 on the nano, not sure if the meizus even have that |
| 22:34:37 | linuxstb | I thought I changed that to PLL1? |
| 22:34:54 | linuxstb | I remember testing it first with PLL2, and then when it was working, trying PLL1 and that worked too. |
| 22:36:09 | TheSeven | where is that? I can't see this getting set up at all. |
| 22:36:13 | pixelma | barrywardell: I tried changing the volume in Multimedia Mode with my c200 and one time a simple button press was like continuously pressing the button, maybe it's a tad bit different with the scroll wheel |
| 22:36:46 | linuxstb | TheSeven: pcm_dma_apply_settings() in pcm-s5l8700.c |
| 22:36:58 | barrywardell | pixelma: volume control works great on the e200 with the scroll wheel |
| 22:37:19 | TheSeven | linuxstb: oh yes, you're setting up the PLL itself, but where do you set which PLL to use? |
| 22:37:24 | barrywardell | pixelma: the only problem i have encountered so far is trying to change volume while copying files |
| 22:37:55 | pixelma | but more importantly - on 10.4 the disks wouldn't mount at all with USB HID enabled. Do you have any idea what to check for etc.? Although I can only try again some time tomorrow... |
| 22:38:59 | barrywardell | pixelma: usb prober can be quite useful |
| 22:38:59 | TheSeven | linuxstb: hmm, looks like you're already setting it up correctly further down |
| 22:39:09 | gevaerts | pixelma, barrywardell: I suspect that we still have some basic driver issues that cause some lost usb packets every now and then. Error detection and handling should be solid enough to not have them cause data loss, but they do cause these bus resets (which appear to the user as freezes), which could explain occasional gliches |
| 22:39:14 | *** | Saving seen data "./dancer.seen" |
| 22:39:54 | barrywardell | gevaerts: ok, although for me, the copy operation didn't ever recover from the freeze. It got stuck and I had to pull the usb cable |
| 22:40:15 | pixelma | barrywardell: is that on board in a usual installation or an external program... though I guess I could also google |
| 22:41:07 | barrywardell | pixelma: I think it's included when you install XCode |
| 22:41:11 | TheSeven | linuxstb: what's the best way to check if DMACTCNT0 is running in rockbox? |
| 22:41:20 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
| 22:41:33 | barrywardell | pixelma: in /Developer/Applications/Utilities |
| 22:42:05 | pixelma | barrywardell: have to check tomorrow if it's installed, this is a work Mac |
| 22:43:02 | linuxstb | TheSeven: Probably by writing a debug screen - see the hell in apps/debug_menu.c for that code (the System -> Debug menu) |
| 22:43:29 | pixelma | barrywardell: thanks for the hint so far. Are you around a bit more often again the next days? |
| 22:43:37 | amiconn | mc2739, pixelma, gevaerts: packages.debian.org can search package contents if you want to know what package provides a certain file |
| 22:43:54 | TheSeven | linuxstb: well, how does the player app determine if it's running? |
| 22:44:09 | barrywardell | pixelma: yes, I'll be online during the day (CET) most of this week, so I'll login here too |
| 22:44:26 | linuxstb | TheSeven: I'm not sure what you mean. There is no "player app" as such. |
| 22:44:28 | TheSeven | return DMATCNT0 * 2; in pcm_get_bytes_waiting() looks like a bug to me, shouldn't this be return DMACTCNT0 * 2;? |
| 22:44:47 | TheSeven | linuxstb: I mean the usual player screen if you open a music file |
| 22:45:06 | | Nick uflops is now known as mikroflops (n=yogurt@90-231-195-226-no112.tbcn.telia.com) |
| 22:45:23 | linuxstb | There's already a "debug audio" screen - maybe that helps. |
| 22:45:50 | TheSeven | where is this? |
| 22:45:54 | pixelma | barrywardell: nice :) I really want to help out debugging this as I seem to be the only one around here who was affected but don't know much about MacOSX, so maybe get back to you then ... :) |
| 22:46:06 | amiconn | linuxstb: Why would you want to disable codec buffering? |
| 22:46:09 | mc2739 | amiconn: That is good to know information! |
| 22:46:12 | linuxstb | TheSeven: Start playback, then go to the main menu and System -> Debug |
| 22:46:24 | TheSeven | you mean that buffering thread screen? |
| 22:46:25 | linuxstb | amiconn: Ask saratoga that question - it was his suggestion. |
| 22:46:54 | barrywardell | pixelma: for reference, here's the output I get: http://pastebin.com/m44ebc85d |
| 22:46:57 | * | amiconn prefers actually fixing bugs over working around them |
| 22:47:02 | linuxstb | TheSeven: I thought there was another... But to be honest it's been a while since I looked. It should show you the amount of bytes in the PCM buffer (which is what is DMA'd to the DAC). |
| 22:47:31 | TheSeven | is this the return value of pcm_get_bytes_waiting(void)? |
| 22:47:50 | linuxstb | Sounds likely. |
| 22:49:36 | | Quit GeekShadow (Read error: 54 (Connection reset by peer)) |
| 22:49:38 | | Join Grahack [0] (n=chri@ip-222.net-82-216-222.rev.numericable.fr) |
| 22:49:56 | pixelma | barrywardell: great, thanks. By the way, I got the "Rockbox media player" at an USB port in the system information but then nothing (and also quite rarely some reactions on the input in some HID mode)... does that tell you something |
| 22:49:58 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 22:50:08 | pixelma | = |
| 22:50:20 | pixelma | oops, sorry |
| 22:51:13 | barrywardell | pixelma: I'm not too sure. USB prober should tell you quite a bit more, especially if you can install the debug usb driver for logging |
| 22:53:25 | | Quit Grahack (Client Quit) |
| 22:53:27 | saratoga | amiconn: besides the fact that it works poorly, i'm not sure what sense it makes to buffer codecs when the buffer is so small |
| 22:53:39 | pixelma | barrywardell: ok, that has to wait a bit then. I will try during the next days, hope I'll be able to use it correctly. Many thanks for your help so far :) |
| 22:53:40 | amiconn | Less special casing |
| 22:55:54 | | Quit funman ("free(random());") |
| 22:56:16 | | Quit Thundercloud (Read error: 104 (Connection reset by peer)) |
| 22:56:20 | saratoga | but yes I would like to figure out why it doesn't work correctly |
| 22:57:01 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 22:57:22 | saratoga | i observed erratic behavior (rebuffering codecs) at 8MB of RAM, though I didn't test larger |
| 22:57:48 | saratoga | i suspect that the checks it uses to decide if it can reuse teh current codec no longer apply |
| 22:58:00 | saratoga | svn shows they're nearly as old as SWCODEC itself |
| 23:00 |
| 23:02:38 | | Join DataGhost [0] (n=dataghos@unaffiliated/dataghost) |
| 23:05:25 | linuxstb | TheSeven: I guess you've found it, but the buffering debug screen is dbg_buffering_thread() in debug_menu.c Does that show anything interesting? |
| 23:05:49 | TheSeven | yep, that DMA still isn't running :-/ |
| 23:06:23 | TheSeven | another question: did that ever work? the GPIO setup was wrong. |
| 23:06:35 | linuxstb | So the PCM buffer fills up, but is never emptied? |
| 23:06:41 | TheSeven | yes |
| 23:07:09 | linuxstb | DMA should have been working - otherwise I wouldn't have got sound. But I guess I just relied on the OF's setup if my code was wrong... |
| 23:07:13 | | Quit merbanan (Read error: 110 (Connection timed out)) |
| 23:07:36 | TheSeven | well, you set the mode of the pins to 2 instead of 7. |
| 23:08:01 | TheSeven | the OF set them to 2, and then to 7 immediately before starting playback |
| 23:11:53 | CIA-43 | New commit by kkurbjun (r22926): Brickmania: Fix the flip-sides powerup with touchscreen and make the whole gameboard available to position the paddle |
| 23:12:12 | | Join domonoky1 [0] (n=Domonoky@e179163077.adsl.alicedsl.de) |
| 23:15:24 | CIA-43 | New commit by kkurbjun (r22927): Brickmania: Add new, finer detail images for the bricks and the ball for 640x480 screens. |
| 23:15:58 | * | gevaerts thinks that any code that provides a better bricking experience is good :) |
| 23:16:37 | | Join GeekShado_ [0] (n=Antoine@APoitiers-552-1-27-19.w86-217.abo.wanadoo.fr) |
| 23:16:49 | kkurbjun | :) |
| 23:17:58 | TheSeven | linuxstb: wrong MCLK speed won't hurt - it will just speed up or slow down the audio |
| 23:18:01 | CIA-43 | New commit by bluebrother (r22928): Add command line option for adding additional files to the build folder. |
| 23:23:04 | | Join GeekShad__ [0] (n=Antoine@APoitiers-552-1-27-19.w86-217.abo.wanadoo.fr) |
| 23:24:13 | | Quit GeekShad__ (Connection reset by peer) |
| 23:24:32 | * | kugel wonders if kkurbjun can get in touch with chopper after finishing brickmania. It could be so much smoother |
| 23:24:35 | | Join GeekShad__ [0] (n=Antoine@APoitiers-552-1-27-19.w86-217.abo.wanadoo.fr) |
| 23:24:59 | * | kkurbjun thinks chopper is named that for a reason :) |
| 23:25:11 | kugel | :'( |
| 23:25:27 | | Join Omlet [0] (n=omlet05@133.99-240-81.adsl-dyn.isp.belgacom.be) |
| 23:25:30 | kkurbjun | :-D |
| 23:25:33 | | Join Omlet05 [0] (n=omlet05@133.99-240-81.adsl-dyn.isp.belgacom.be) |
| 23:25:36 | | Quit Omlet05 (Remote closed the connection) |
| 23:27:07 | * | bluebrother wonders if it's feasible to merge the OSX deploy script with deploy-release.py |
| 23:28:04 | | Quit domonoky (Read error: 110 (Connection timed out)) |
| 23:31:02 | * | TheSeven suspects that something with DMA is just broken |
| 23:33:13 | | Quit Omlet ("( www.nnscript.com :: NoNameScript 4.22 :: www.esnation.com )") |
| 23:34:54 | | Quit GeekShadow (Connection timed out) |
| 23:35:44 | | Quit chandoo (Read error: 110 (Connection timed out)) |
| 23:39:28 | | Join JdGordon [0] (n=jonno@rockbox/developer/JdGordon) |
| 23:40:27 | * | TheSeven was right |
| 23:40:50 | TheSeven | linuxstb: "audiotest" works now also when booted through iloader-ibugger |
| 23:41:15 | | Quit GeekShado_ (Read error: 110 (Connection timed out)) |
| 23:41:17 | linuxstb | Cool. So now for Rockbox... |
| 23:41:18 | | Nick Ypsy is now known as YPSY (n=ypsy@geekpadawan.de) |
| 23:42:34 | * | TheSeven just can't spot a difference |
| 23:42:53 | TheSeven | i've fixed the target of the DMA data, but that didn't help |
| 23:42:59 | TheSeven | the peripheral ID was just wrong |
| 23:43:11 | TheSeven | (probably right according to the DS, but apple clearly uses a different one |
| 23:43:26 | * | bertrik remembers an ambiguous DMA table from the ds |
| 23:43:40 | * | linuxstb wonders how this worked at all in his bootloader tests... |
| 23:43:48 | * | TheSeven too |
| 23:43:57 | linuxstb | Honestly, I did hear audio! |
| 23:44:41 | * | TheSeven trusts linuxstb, but doesn't trust the DMA any more :-) |
| 23:45:35 | * | TheSeven spotted another wrong define |
| 23:47:17 | * | TheSeven can't calculate bit positions today |
| 23:49:11 | * | TheSeven notices that not setting up the wmcodec properly will stall the DMA |
| 23:51:01 | * | TheSeven doesn't really understand how all this rockbox audio setup code works |
| 23:51:38 | bertrik | hm, I thought the IIS port on the s5l8700 was IIS master and the codec IIS slave, which should make it basically impossible for the wmcodec to stall audio |
| 23:51:59 | TheSeven | bertrik: on the meizu, yes, on the nano, no |
| 23:52:03 | TheSeven | the wmcodec is a master |
| 23:55:55 | CIA-43 | New commit by tomers (r22929): Have codepage name show in a comment |
| 23:55:59 | | Quit jordan`` (Read error: 104 (Connection reset by peer)) |
| 23:57:53 | TheSeven | linuxstb: any chance the wmcodec setup is at fault? |
| 23:58:26 | linuxstb | TheSeven: Anything is possible... Did you make that change I made in my code? |
| 23:58:33 | TheSeven | which one? |
| 23:58:47 | linuxstb | The one I mentioned earlier - in the wmcodec init |
| 23:58:48 | | Join jordan` [0] (i=gromit@78.235.252.137) |
icon identifies that the person is a core developer (has commit access).