00:00:02 | | Join [7] [0] (~quassel@rockbox/developer/TheSeven) |
00:07:19 | *** | Saving seen data "./dancer.seen" |
00:58:17 | braewoods | speachy: first i'd ask if the port is worth saving |
00:59:42 | braewoods | some of these are so old i kinda wonder a bit |
01:00 |
01:00:06 | braewoods | not to mention super rare |
01:00:33 | braewoods | seems like the irivers are the most common coldfire targets |
01:01:39 | braewoods | like the mpio units seem completely unavailable |
01:02:31 | braewoods | and the only cowon i see for sale is the x5 |
01:02:43 | braewoods | the other models seem to be really rare these days |
01:03:49 | braewoods | anyway i'm going to do research on MTP in the near future, work on that after i finish the bootloader project |
01:04:13 | braewoods | but it appears i will have to write our own support code for it since all the MTP libraries of relevance are C++ |
01:04:23 | braewoods | we need a C port |
01:05:20 | braewoods | i'll also need to figure out what ports can even use MTP... i assume you need something like OTG. |
01:23:37 | braewoods | _bilgus: ok, the bootloader i just build from master works again. i think it's fixed now for sure. |
01:23:47 | braewoods | i'll test H120 next |
01:24:08 | braewoods | just want to confirm it working before i start tinkering |
01:28:07 | braewoods | speachy: though i think you've got some oddness in the new usb code. |
01:28:36 | braewoods | speachy: when i first connect from reset settings... i see a menu selection appear briefly and then disappear |
01:28:41 | braewoods | it basically behaves as charge only |
01:29:01 | braewoods | i go into the settings to select mass storage |
01:29:19 | braewoods | that fixes it though it was already set to that apparently |
01:29:23 | braewoods | no idea what's going on there |
01:29:28 | braewoods | this is on the iriver h320 i have |
01:29:31 | braewoods | latest master build |
01:48:04 | braewoods | H120 still works on master |
02:00 |
02:07:23 | *** | No seen item changed, no save performed. |
02:07:59 | | Quit massiveH (Quit: Leaving) |
02:16:03 | braewoods | hm. |
02:16:08 | braewoods | interesting. |
02:16:21 | braewoods | the H300 bootloader starts when usb is connected but not when DC is connected |
02:16:40 | braewoods | this code suggests it should start when DC is connected |
02:17:55 | braewoods | speachy: i noticed the original author had partially implemented a charging screen in the h300 bootloader which is not present in h100. do you think i should keep it and fix it up or rip it out? i can look into backporting this into H100 bootloader too. |
02:31:11 | braewoods | for my own sanity i think i'm just going to disable the check in iriver_flash for valid BL |
02:31:25 | braewoods | PITA to keep updating it during development |
02:38:37 | braewoods | wow. found another bug. |
02:38:50 | braewoods | charger_inserted() returns false whether i am connected to AC or not in the bootloader |
02:47:26 | | Join prof_wolfff [0] (~prof_wolf@148.red-83-49-157.dynamicip.rima-tde.net) |
02:47:44 | braewoods | strange. |
02:59:42 | braewoods | seems i found the bug but let me confirm |
03:00 |
03:04:42 | braewoods | confirmed |
03:05:39 | braewoods | speachy: i found an issue at least in H300 bootloader for charger detection. it's because the power_thread_inputs variable is not initialized when the BL calls charger_inserted() from firmware/powermgmt.c |
03:06:07 | braewoods | but when i get the most recent data directly from power_input_status() |
03:06:10 | braewoods | it works fine |
03:06:21 | braewoods | so the code for initializing it is not being run |
03:08:01 | braewoods | i can think of 2 solutions |
03:08:24 | braewoods | either we patch the convenience functions to just use the function directly when compiling for the bootloader |
03:08:46 | braewoods | or we need to initialize it somehow, perhaps when power_init is called |
03:21:50 | braewoods | on second though |
03:21:52 | braewoods | i'll try this |
03:30:12 | | Quit advcomp2019_ (Ping timeout: 260 seconds) |
03:40:37 | | Quit prof_wolfff (Ping timeout: 260 seconds) |
03:43:18 | braewoods | strange. |
03:43:25 | braewoods | ok the thread init doesn't work at all |
03:43:44 | braewoods | i think we're going to need some custom code for the BLs |
03:44:33 | braewoods | i tried launching the powermgmt_init but it doesn't seem to work for the iriver h300 |
03:44:39 | braewoods | in bootloader mode anyway |
03:46:47 | braewoods | not to mention using it makes the BL grow by 2K |
03:57:14 | braewoods | _bilgus: is there some way to override the stock charger_inserted() and charger_inserted() functions? i get linking errors, not surprisingly |
03:57:27 | braewoods | err |
03:57:32 | braewoods | power_input_present() function |
03:58:07 | braewoods | i'm thinking of creating a version of these 2 that is basically a custom version of the detection code |
03:58:22 | braewoods | to keep the size down of the BL and such |
04:00 |
04:03:48 | | Join lebellium [0] (~lebellium@89-92-69-66.hfc.dyn.abo.bbox.fr) |
04:06:51 | | Join advcomp2019 [0] (~advcomp20@65-131-145-101.sxct.qwest.net) |
04:06:51 | | Quit advcomp2019 (Changing host) |
04:06:51 | | Join advcomp2019 [0] (~advcomp20@unaffiliated/advcomp2019) |
04:07:26 | *** | Saving seen data "./dancer.seen" |
04:32:11 | | Join TheLemonMan [0] (~lemonboy@irssi/staff/TheLemonMan) |
04:51:40 | | Quit S|h|a|w|n (Read error: Connection reset by peer) |
05:00 |
05:14:46 | braewoods | ok. implemented a proposed solution. |
05:14:49 | braewoods | g#2973 |
05:14:51 | fs-bluebot_ | Gerrit review #2973 at http://gerrit.rockbox.org/r/2973 : rockbox: implement an alternative to detect_charger() for bootloaders by James Buren |
05:16:31 | | Join advcomp2019_ [0] (~advcomp20@65-131-145-101.sxct.qwest.net) |
05:16:31 | | Quit advcomp2019_ (Changing host) |
05:16:31 | | Join advcomp2019_ [0] (~advcomp20@unaffiliated/advcomp2019) |
05:19:16 | | Quit advcomp2019 (Ping timeout: 258 seconds) |
05:23:03 | | Join prof_wolfff [0] (~prof_wolf@148.red-83-49-157.dynamicip.rima-tde.net) |
05:32:28 | braewoods | ah. so that's why the unit turns on when AC is connected. |
05:32:32 | braewoods | and has charging screen |
05:32:38 | braewoods | the OF of the H300 did that |
05:33:42 | braewoods | let's find out what the H120 does. |
05:40:51 | braewoods | ok. nothing. |
05:41:01 | braewoods | the H120 is hardware managed evidently |
05:41:10 | braewoods | so not much point to a charger screen |
05:54:53 | | Quit t0mato (Quit: Ping timeout (120 seconds)) |
06:00 |
06:04:43 | | Join MrZeus [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
06:07:28 | *** | Saving seen data "./dancer.seen" |
06:10:49 | | Join t0mato [0] (t0mato@gateway/vpn/mullvad/t0mato) |
06:21:37 | | Quit prof_wolfff (Ping timeout: 260 seconds) |
06:29:30 | _bilgus | braewoods, i'm ok with that I think it might be cleaner to mark the function weak and override in the BL |
06:36:10 | braewoods | _bilgus: i already found a solution. |
06:36:21 | braewoods | it's honestly not worth doing that for a single line function |
06:36:23 | braewoods | see gerrit |
06:36:36 | braewoods | unless you have a better idea? |
06:36:46 | braewoods | i just implemented a feature on the H300 TODO list |
06:36:56 | braewoods | usb charging now works in the bootloader |
06:42:12 | _bilgus | only thing I don't like about it is that it kinda pollutes the namespace with a bootloader function I'd rtaher see it redefined for all bootloaders |
06:43:11 | _bilgus | I saw gerrit lol |
06:43:24 | braewoods | ok. |
06:43:27 | braewoods | i can see what i can do. |
06:43:47 | braewoods | _bilgus: can you review my other commit? |
06:43:47 | _bilgus | i'd probably just #ifdef BOOTLOADER |
06:44:00 | braewoods | thing is i don't think all BLs need it |
06:44:21 | braewoods | there's some that manage to call powermgmt_init just fine evidently |
06:44:27 | braewoods | but not for coldfire |
06:44:28 | _bilgus | but it won't hurt them and except for hosted I doubt the power thread is even running |
06:45:06 | braewoods | i think that's incorrect. it is running in the regular native firmware |
06:45:14 | braewoods | i know that |
06:45:16 | braewoods | much |
06:45:29 | braewoods | because nothing else can initialize the internal variable |
06:45:44 | braewoods | but eh |
06:45:44 | _bilgus | the regular native fw? |
06:45:47 | braewoods | yea |
06:45:55 | braewoods | i know that since the debug menu app has |
06:46:03 | braewoods | calls to charger_inserted() |
06:46:21 | _bilgus | do you mean the the actualy running RB firmware? |
06:46:21 | braewoods | but ok. i'll redo the commit. |
06:46:26 | braewoods | ... YES |
06:46:32 | braewoods | rofl |
06:46:39 | braewoods | the battery debug options use it |
06:46:44 | braewoods | and it works correctly |
06:46:45 | braewoods | so |
06:46:51 | braewoods | the power thread must be running |
06:47:28 | braewoods | since the thread's function is the only thing that writes to the internal variable |
06:47:32 | braewoods | power_thread_inputs |
06:47:49 | braewoods | the issue is this can't run for w/e reason in the bootloader |
06:47:53 | braewoods | so the convenience functions don't work |
06:48:14 | braewoods | i first tried using powermgmt_init but no good |
06:48:21 | braewoods | so ye |
06:48:23 | braewoods | yea |
06:48:26 | braewoods | ok gimme a sec |
06:49:45 | _bilgus | I'll look at the bootloaders and see how many others are using a power thread in the bootloader |
06:50:37 | braewoods | ok go ahead |
06:50:43 | braewoods | it's probably best to wait a bit to find out |
06:51:16 | braewoods | i just wodnered why the charging mode didn't trigger in h300 and it was because charger_inserted() wasn't working correctly |
06:51:25 | braewoods | in bootloader mode |
06:51:27 | braewoods | only |
06:51:37 | braewoods | the workaround i found was to bypass it in bootloader mode |
06:52:09 | braewoods | it probably worked at one time but it doesn't now |
06:52:14 | _bilgus | so just ipod6g and imx233 |
06:52:14 | braewoods | question is how do we want to fix it? |
06:52:26 | braewoods | yea |
06:52:30 | braewoods | i tried adding it to h300 |
06:52:33 | braewoods | but it didn't help |
06:52:57 | _bilgus | let me see what they are doing with it idealy I'd like them to not use the thread |
06:53:14 | braewoods | not sure they really need it honestly |
06:53:50 | _bilgus | they shouldn't after your change I think |
06:54:35 | braewoods | i'll finish implementing it then |
06:58:06 | | Quit Misanthropos (Ping timeout: 256 seconds) |
07:00 |
07:02:25 | braewoods | _bilgus: sent new one. g#2975 |
07:02:27 | fs-bluebot_ | Gerrit review #2975 at http://gerrit.rockbox.org/r/2975 : rockbox: revise charger_inserted and power_input_present functions by James Buren |
07:03:21 | _bilgus | yes... |
07:03:48 | _bilgus | on the ipod and imx they both need the thread to set up charging |
07:04:06 | braewoods | so, undo the change? |
07:04:26 | braewoods | or how shall we implement it? |
07:04:30 | _bilgus | its not going to hurt them to bypass |
07:04:43 | braewoods | ok |
07:05:06 | braewoods | i'll revise my other patch |
07:05:35 | _bilgus | they can keep calling the thread to do their init its not going to hurt to give a reacharound on the insert funcs |
07:12:30 | braewoods | _bilgus: ok. both are done. review if you got time? |
07:12:49 | _bilgus | k |
07:21:04 | fs-bluebot_ | Build Server message: New build round started. Revision d5a2aeb, 293 builds, 9 clients. |
07:21:10 | _bilgus | looks good |
07:22:44 | braewoods | well that's another one off the TODO list |
07:23:49 | _bilgus | SO the part of it probably used to work I wonder what got removed from that player that it no longer is able to do the thread |
07:25:14 | _bilgus | or was it that the power thread evolved and someone forgot it |
07:26:15 | braewoods | i could trace it but i don't think it's worth it. |
07:28:15 | _bilgus | nah just rhetorical |
07:28:58 | braewoods | ok what else does the bootloader need |
07:30:13 | braewoods | "Add possibility to control USB hub setting"? |
07:30:33 | braewoods | i can't see the value in that though i know the OF has an option for setting 2 different USB modes |
07:30:43 | braewoods | only one of them makes disk mode work in the OF |
07:30:55 | braewoods | what good is the other mode? |
07:31:02 | braewoods | RB seems to use the one that works already |
07:32:08 | braewoods | "Option to remove the bootloader from the flash" ? |
07:32:20 | braewoods | no point. that basically means returning to OF |
07:32:33 | braewoods | if people want that they can use iriver_flash to reflash it then reflash from the OF |
07:32:55 | braewoods | low value; already achieveable via another set of options |
07:33:06 | braewoods | let the user do it if they want to go back to pure stock OF |
07:35:39 | braewoods | next up... |
07:35:47 | fs-bluebot_ | Build Server message: Build round completed after 883 seconds. |
07:35:51 | braewoods | backlight turn off during usb mode it appears |
07:36:00 | fs-bluebot_ | Build Server message: Revision d5a2aeb result: All green |
07:36:36 | _bilgus | I never like removing OF |
07:36:48 | _bilgus | I think we talked about this prior |
07:37:09 | braewoods | indeed |
07:37:21 | braewoods | iriver_flash allows for return to OF |
07:37:29 | braewoods | but it is mainly used for updating bootloader |
07:38:06 | braewoods | right now i'm just exhausting the todo left the previous bootloader author left behind |
07:38:19 | braewoods | then i'll see what else i want to do |
07:38:37 | braewoods | at the very least i would like to see if i can port over the H100 boot from rom stuff |
07:41:08 | _bilgus | braewoods, I think we might have missed something thats would shrink the h300 bootloader |
07:41:27 | braewoods | _bilgus: what now? |
07:41:41 | _bilgus | let me double check but I think its still pulling in the power thread stuff |
07:43:24 | braewoods | _bilgus: doesn't seem likely. powermgmt_init is not present in the final elf binary. |
07:43:49 | braewoods | it's currently sitting at 55k |
07:44:00 | _bilgus | nope its in the removed list |
07:44:03 | braewoods | still around 10k left to expand |
07:44:43 | braewoods | i'll see what h120 does. i recall it turns off backlight in usb mode. |
07:55:43 | | Join MrZeus_ [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
07:56:24 | | Quit MrZeus (Ping timeout: 256 seconds) |
07:59:28 | braewoods | hm. this seems good. let's build and test it. |
08:00 |
08:00:44 | _bilgus | the h300 can get the codepage_table removed I think odd that its getting pulled in |
08:00:53 | _bilgus | bootloader |
08:07:19 | braewoods | _bilgus: ok. how would we do that? |
08:07:30 | *** | Saving seen data "./dancer.seen" |
08:07:34 | braewoods | i'm currently working on adding a small amount of code to implement a backlight timeout |
08:07:46 | braewoods | it'll turn back on if buttons are pressed |
08:08:44 | braewoods | the loops already sleep for 1 second between updates in some manner |
08:08:58 | braewoods | so it shuld be simple to define a timeout in terms of seconds past |
08:16:36 | | Join sakax [0] (~r0b0t@unaffiliated/r0b0t) |
08:27:47 | | Join prof_wolfff [0] (~prof_wolf@148.red-83-49-157.dynamicip.rima-tde.net) |
08:27:58 | | Quit Moarc_ (Quit: i znowu NADMUCHAŁ BALONA) |
08:28:56 | braewoods | ok. this seems to be done. |
08:29:46 | | Join Moarc [0] (~chujko@a105.net128.okay.pl) |
08:32:20 | | Join mendel_munkis_ [0] (~mendelmun@ool-435680b7.dyn.optonline.net) |
08:32:56 | braewoods | g#2976 |
08:32:59 | fs-bluebot_ | Gerrit review #2976 at http://gerrit.rockbox.org/r/2976 : h300: implement a backlight timeout function when in charge or disk mode by James Buren |
08:34:49 | | Quit MrZeus_ (*.net *.split) |
08:34:49 | | Quit mendel_munkis (*.net *.split) |
08:34:49 | | Quit kugel (*.net *.split) |
08:34:50 | | Quit vup (*.net *.split) |
08:35:08 | | Join vup [0] (~~~~@46.101.193.235) |
08:35:46 | braewoods | _bilgus: 2 commits still waiting for merge. |
08:36:05 | braewoods | anyway the timeout function seems to work just fine |
08:36:14 | braewoods | though perhaps it could be better optimized |
08:36:20 | braewoods | i only found one issue that i have already fixed |
08:36:33 | braewoods | i forgot to force the backlight back on when exitting them ode |
08:36:35 | braewoods | mode |
08:37:14 | fs-bluebot_ | Build Server message: New build round started. Revision 5a22308, 293 builds, 9 clients. |
08:37:43 | | Quit danielp3344 (*.net *.split) |
08:37:43 | | Quit kadoban (*.net *.split) |
08:37:43 | | Quit prg318 (*.net *.split) |
08:37:43 | | Quit Strife89 (*.net *.split) |
08:37:43 | | Quit fauweh (*.net *.split) |
08:37:44 | | Join MrZeus_ [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
08:37:44 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
08:38:20 | _bilgus | why do we need to control the backlight timeout in bootloader? |
08:39:41 | braewoods | _bilgus: no idea but it was on the original author's TODO list in the wiki. see here: https://www.rockbox.org/wiki/IriverBoot |
08:39:46 | braewoods | "H300: Shut off backlight properly in USB mode " |
08:39:53 | braewoods | not sure what else they could mean by that honestly. |
08:40:09 | _bilgus | I think that is a bad idea personally |
08:40:21 | braewoods | why? |
08:40:39 | braewoods | i mean, we can just not implement it but i was trying to honor their last known intentions. |
08:40:59 | | Join danielp3344 [0] (danielp334@gateway/shell/matrix.org/x-fambbpxdzrniwmyn) |
08:40:59 | | Join kadoban [0] (kadobanmat@gateway/shell/matrix.org/x-jymgxfawqsikjoeo) |
08:40:59 | | Join prg318 [0] (~prg@deadcodersociety/prg318) |
08:40:59 | | Join Strife89 [0] (sid399903@gateway/web/irccloud.com/x-qyolaomvadnswwzo) |
08:40:59 | | Join fauweh [0] (~root@ithaqua.unzane.com) |
08:41:16 | _bilgus | if you are in bootloader its there for a specific reason either you can't get into fw or you are going around a bug in the fw |
08:42:22 | braewoods | _bilgus: i honestly don't care either way but... |
08:42:29 | braewoods | i was just going off their notes. |
08:42:39 | braewoods | i'll scrap it then. |
08:44:15 | braewoods | ok that's all on the TODO list then |
08:44:18 | | Quit blbro[m] (Ping timeout: 246 seconds) |
08:44:18 | | Quit kadoban (Ping timeout: 246 seconds) |
08:45:10 | braewoods | just one last feature |
08:45:15 | braewoods | or so i think |
08:45:24 | braewoods | let's compare the bootloaders now |
08:47:47 | | Quit danielp3344 (Ping timeout: 246 seconds) |
08:52:02 | fs-bluebot_ | Build Server message: Build round completed after 888 seconds. |
08:52:03 | fs-bluebot_ | Build Server message: Revision 5a22308 result: All green |
08:52:15 | _bilgus | ok as long as this works it should free 64k from the bootloaders |
09:00 |
09:06:38 | | Join kadoban [0] (kadobanmat@gateway/shell/matrix.org/x-kttvbwikdhecuywx) |
09:08:52 | braewoods | g#2977 |
09:08:54 | fs-bluebot_ | Gerrit review #2977 at http://gerrit.rockbox.org/r/2977 : h1x0/h300: clean up the bootloader source a bit by James Buren |
09:09:02 | braewoods | small change to prepare for the next step i have in mind |
09:10:04 | | Join ac_laptop [0] (~ac_laptop@186.2.247.129) |
09:11:08 | _bilgus | when you are doing stuff like that just do another commit on top and submit both not sure why that last one didn't do both |
09:11:28 | braewoods | No idea. |
09:12:06 | _bilgus | also sorry but can you separate the WS changes from the code changes |
09:12:34 | braewoods | as you wish. |
09:12:48 | _bilgus | two ways bot use a patch with only WS changes either make it the parent or do it as a child |
09:13:07 | _bilgus | long standing rule bitten me MANY times |
09:13:34 | _bilgus | having to clean up my commits not in a code fail kinda way lol |
09:15:43 | _bilgus | ok so shrinking codepage_table freed 1K in my bootloader but its 10k in the bin |
09:15:54 | _bilgus | oh yeah FFS clipplus is compressed |
09:17:50 | braewoods | _bilgus: https://gerrit.rockbox.org/r/#/c/rockbox/+/2978/ |
09:18:00 | braewoods | first one. |
09:18:06 | _bilgus | braewoods, are your bootloader compressed or decompressed in any step? |
09:18:13 | braewoods | _bilgus: not to my knowledge. |
09:18:21 | braewoods | the main difference is optimizations |
09:18:42 | braewoods | it's too big if you disable them entirely |
09:19:57 | _bilgus | ok so this would truly help there let me make sure this work on the clip+ and then I'll pass it on to you should give you all kinds of room |
09:20:53 | braewoods | i already got plenty for the foreseeable future |
09:20:58 | braewoods | but i guess it can't hurt. |
09:21:17 | braewoods | the H1x0 and H300 BLs use the last 64K of the ROM chip |
09:21:26 | braewoods | so roughyl 65,536 bytes |
09:21:33 | braewoods | it currently is around 55,000 bytes |
09:22:14 | braewoods | still the previous BLs they'd be replacing were around 45k |
09:23:51 | | Join blbro[m] [0] (blbrostrat@gateway/shell/matrix.org/x-mjpyubehgdijutir) |
09:25:34 | _bilgus | appears to work fine its a simple one |
09:29:29 | _bilgus | g#2979 |
09:29:31 | fs-bluebot_ | Gerrit review #2979 at http://gerrit.rockbox.org/r/2979 : Bootloaders −− Remove codepage_table by William Wilgus |
09:29:47 | _bilgus | could you try that on your bootloader? |
09:30:27 | fs-bluebot_ | Build Server message: New build round started. Revision 37eabbf, 293 builds, 9 clients. |
09:30:29 | braewoods | _bilgus: sure. review my changes while i do that? |
09:30:41 | _bilgus | already was |
09:30:43 | braewoods | i rebased the oriignal commit |
09:30:51 | braewoods | g#2977 |
09:30:53 | fs-bluebot_ | Gerrit review #2977 at http://gerrit.rockbox.org/r/2977 : h1x0/h300: clean up the bootloader source a bit by James Buren |
09:31:18 | _bilgus | I was gonna mention that but gerrit takes care of simple stuff |
09:31:37 | braewoods | ok |
09:31:59 | braewoods | sometimes i've had to do manual rebasing though |
09:32:01 | braewoods | so eh |
09:32:49 | _bilgus | yeah it sucks when it fails |
09:33:32 | braewoods | gimme a bit to test this |
09:33:45 | | Join danielp3344 [0] (danielp334@gateway/shell/matrix.org/x-htimkuhjycbzzooe) |
09:34:05 | braewoods | new size |
09:34:07 | braewoods | 51808 |
09:34:13 | braewoods | saved about 3.2k |
09:34:39 | _bilgus | thats a smaller than the map too its 64k of bytes |
09:34:59 | _bilgus | do you have a patching step as well? |
09:35:08 | braewoods | what's that? |
09:35:22 | _bilgus | or do you patch bootloader.iriver directly? |
09:35:55 | _bilgus | meh for 3.2 k i'll leave it till there is a size issue |
09:36:06 | braewoods | _bilgus: the tools do it for me? |
09:36:12 | | Quit Soap (Ping timeout: 256 seconds) |
09:36:39 | _bilgus | so there is compression for yours too just not as good as the clip+ apparently |
09:37:25 | | Join Soap [0] (~Soap@rockbox/staff/soap) |
09:37:27 | braewoods | _bilgus: bootloader still boots |
09:37:43 | _bilgus | yeah they don't use it |
09:38:06 | _bilgus | hosted might be one that would |
09:38:24 | braewoods | maybe just limit it to native targets for now |
09:38:40 | _bilgus | I'll put this in the abandoned pile till a time comes when we need room again' |
09:38:48 | braewoods | ok. |
09:38:56 | braewoods | i've still got 10k to spare |
09:39:15 | braewoods | i don't see myself using anywhere near that just to port over the one remaining feature |
09:39:20 | _bilgus | clip+ gains 1k with it it has 1k to spare currently |
09:39:49 | _bilgus | 1-1.5 forget the exact amount but its not much |
09:40:01 | braewoods | what else does the clip+ need? |
09:40:07 | braewoods | isn't it effectively complete by now? |
09:40:44 | braewoods | anyway i'm going to finish this up and try to cut a production bootloader candidate |
09:40:58 | braewoods | next week if this works out |
09:41:38 | _bilgus | no it is finished for now but its also not the only bootloader close to the edge like that |
09:41:42 | braewoods | only once i'm pretty sure they're ready |
09:42:12 | _bilgus | we will find more if given more coverage I think |
09:42:40 | braewoods | the iriver coldfire bootloaders still got plenty of room for common code expansion |
09:43:12 | braewoods | another thing that could help is toolchain upgrade |
09:43:23 | braewoods | but that should probably wait until after the next stable release |
09:44:27 | speachy | what does the CP tabble actually do? translate filenames/user strings into display strings? |
09:44:52 | fs-bluebot_ | Build Server message: Build round completed after 865 seconds. |
09:44:53 | fs-bluebot_ | Build Server message: Revision 37eabbf result: All green |
09:45:02 | speachy | if so, I don't think there's anything in the hosted BLs that would care. |
09:45:09 | fs-bluebot_ | Build Server message: New build round started. Revision aaa9a2c, 293 builds, 9 clients. |
09:45:39 | speachy | assuming the filenames the BL needs to hit are limited to ASCII. Which they are, in the case of RB. |
09:47:38 | _bilgus | speachy pretty much |
09:48:26 | _bilgus | and my exact thought but since they are bootloaders I don't really want to mess with them unless its needed or profound |
09:52:16 | braewoods | _bilgus: thanks. now i'm going to port over the last remaining feature from the h1x0 bootloader |
09:52:20 | braewoods | the failsafe mode |
10:00 |
10:01:22 | _bilgus | mendel_munkis_, g#2290 might give you another avenue if removing apply glabal settings :P doesn't work |
10:01:24 | fs-bluebot_ | Gerrit review #2290 at http://gerrit.rockbox.org/r/2290 : synclist add method for setting selection color by William Wilgus |
10:02:09 | fs-bluebot_ | Build Server message: Build round completed after 1020 seconds. |
10:02:54 | fs-bluebot_ | Build Server message: Revision aaa9a2c result: All green |
10:02:54 | _bilgus | All green... |
10:02:58 | _bilgus | lol |
10:07:34 | *** | Saving seen data "./dancer.seen" |
10:24:59 | | Join efqw [0] (uid412670@gateway/web/irccloud.com/x-xjazaakkfackctyc) |
10:26:18 | speachy | the remove global settings apply thing probably needs to go back in once the underlying issue is fixed (in 2290 maybe?) |
10:28:57 | | Quit Acou_Bass (Ping timeout: 260 seconds) |
10:33:15 | | Join Acou_Bass [0] (~eddie@cpc96070-bolt17-2-0-cust175.10-3.cable.virginm.net) |
10:40:02 | _bilgus | 2290 only enables the pieces I think it'd still need to be wired for plugins |
10:40:41 | _bilgus | or conversely set up in the quickscrren |
10:43:32 | fs-bluebot_ | Build Server message: New build round started. Revision be1be79, 293 builds, 9 clients. |
10:44:08 | | Join MrZeus [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
10:47:01 | | Quit MrZeus_ (Ping timeout: 258 seconds) |
10:57:23 | fs-bluebot_ | Build Server message: Build round completed after 831 seconds. |
10:57:28 | fs-bluebot_ | Build Server message: Revision be1be79 result: All green |
11:00 |
11:12:20 | | Quit sakax (Quit: Leaving) |
11:19:33 | | Quit TheLemonMan (Quit: "It's now safe to turn off your computer.") |
12:00 |
12:07:36 | *** | Saving seen data "./dancer.seen" |
13:00 |
13:06:57 | braewoods | _bilgus: i discovered a small issue with iriver_flash, not a real bug but something i'd like to correct |
13:07:06 | braewoods | the screen goes dark on the h300 during inactivity |
13:07:10 | braewoods | and that may freak some people out |
13:07:25 | braewoods | so how might i force it to stay on until iriver_flash exits? |
13:15:14 | braewoods | ok. g#2980 is the last major change i expect to make to the bootloaders for these. |
13:15:16 | fs-bluebot_ | Gerrit review #2980 at http://gerrit.rockbox.org/r/2980 : h1x0/h300: bring the two bootloaders more in sync by James Buren |
13:46:26 | | Join amsomniac [0] (~nadya@2601:181:8300:4db::b45) |
13:47:14 | amsomniac | is there a way to enable debug info on the bootloader? I'm trying to figure out this ATA error |
13:50:46 | braewoods | amsomniac: which bootloader? |
14:00 |
14:03:25 | amsomniac | The ipod6g bootloader |
14:04:22 | amsomniac | If people normally just modify the bootloader to get some more info that's cool too, I really like that the configure script lets me build a version of rockbox with more debugging info but until I can get it to boot I can't see it |
14:05:11 | braewoods | so you can't even boot with the stable release? |
14:05:15 | amsomniac | I'm going to try putting DEBUG=1 in the Makefile |
14:05:40 | braewoods | let me guess. you're using an ipod modded with some iflash thing? |
14:05:42 | amsomniac | Nope, I'm starting to think it's a hardware problem, not a rockbox one−− I haven't been able to boot anything except the stock firmware |
14:05:47 | amsomniac | Yep. |
14:06:21 | braewoods | not supported. https://forums.rockbox.org/index.php/topic,52560.0.html |
14:07:28 | amsomniac | thanks−− I might buy another SSD and then if that doesn't work give up |
14:07:39 | *** | No seen item changed, no save performed. |
14:08:06 | braewoods | i've been working to solve a similar issue with the iriver coldfire ports |
14:09:01 | braewoods | the OF they have tends to hate CF mods which are common for people upgrading theirs |
14:09:19 | braewoods | and the existing RB bootloaders don't really like them either |
14:09:30 | braewoods | but it was fixed in master a long time ago |
14:09:35 | braewoods | just it never got released |
14:09:38 | braewoods | lol |
14:22:01 | amsomniac | there must be a lot of configurations of mods and media players, it's amazing rockbox supports as much as it does. |
14:23:48 | braewoods | the iriver ones are probably one of the better non-apple ports |
14:23:56 | braewoods | and also somewhat unique |
14:24:30 | braewoods | the h100 and h300 series both support common types of data/charging cables. |
14:24:58 | braewoods | both have the RB bootloader in ROM; same as the OF |
14:25:07 | braewoods | so the OF is not stored on your storage device |
14:25:09 | amsomniac | Nice. My sansa e200 and fuze are very happy with rockbox but I haven't used either in a while because I've lost their weird charging cables |
14:25:26 | braewoods | the h100 and h300 use barrel jacks |
14:25:38 | braewoods | though the h300 also supports usb charging |
14:25:48 | braewoods | the h300 also has color screen |
14:25:55 | braewoods | h100 has a grayscale one |
14:26:11 | braewoods | but it also has LEDs indicating power or disk activity |
14:26:22 | braewoods | the h300 does not |
14:26:26 | braewoods | just kinda interesting |
14:26:40 | braewoods | you can mod them with CF cards |
14:26:53 | braewoods | almost any capacity |
14:27:00 | braewoods | up to 2TB |
14:27:03 | braewoods | if you're that inclined |
14:27:42 | braewoods | i just use 64GB CF cards myself |
14:27:54 | braewoods | they make the player around 15% lighter |
14:29:31 | amsomniac | I'm glad they're still making CF cards in higher capacities, I thought they stopped for some reason. |
14:30:17 | braewoods | most of the ones i get are used |
14:30:34 | braewoods | even if demand dries up though there's CF adapters to SD cards |
14:30:47 | braewoods | they're not too fast but the main bottleneck isn't the CF card |
14:31:07 | braewoods | e.g., i only get around 13 MB/s at maximum in my CF modded players |
14:31:50 | braewoods | they're the most cost effective option if you care more about price per GB |
14:32:02 | braewoods | but the write endurance is shit |
14:32:13 | braewoods | so best only use if you don't plan to do a ton of writing |
15:00 |
15:01:50 | | Quit _bilgus (Ping timeout: 264 seconds) |
15:07:52 | | Quit amsomniac (Quit: Leaving.) |
15:09:15 | | Quit amiconn (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.) |
15:09:15 | | Quit pixelma (Quit: .) |
15:11:50 | | Join amiconn [0] (jens@rockbox/developer/amiconn) |
15:11:51 | | Join pixelma [0] (marianne@rockbox/staff/pixelma) |
15:13:21 | | Join _bilgus [0] (~bilgus@2605:a000:1301:89f6:1f3:380e:6252:f251) |
15:40:44 | braewoods | speachy: i've been researching mtp more. no real libraries that i can see. they're all tied to specific usb stacks. |
15:41:07 | braewoods | mtp responders that is |
15:41:38 | braewoods | speachy: for what targets would it be useful anyway? from what i've seen it's mostly native targets with OTG support. |
15:41:55 | braewoods | the android ports already have MTP through android |
15:42:37 | braewoods | which leaves the Linux hosted ports but those can use something like umtp-responder |
15:43:04 | braewoods | we need our own implementation for native |
15:55:50 | braewoods | for another day though |
15:55:59 | braewoods | i still need to figure out how to make the usb communication work |
16:00 |
16:03:49 | | Quit MrZeus (Ping timeout: 264 seconds) |
16:06:39 | | Join MrZeus [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
16:07:40 | *** | Saving seen data "./dancer.seen" |
16:10:31 | | Join Misanthropos [0] (~Misanthro@91.240.64.196) |
16:14:13 | | Join MrZeus_ [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
16:17:01 | | Quit MrZeus (Ping timeout: 264 seconds) |
16:38:19 | bluebrother^ | speachy: I'm currently reworking how build-info is handled. What do you think about changing the format of the daily / dev builds? We would need to keep the [bleeding] for compatibility, but could add a new [development] that uses pretty much the same format as [release] but have <player>=<revision>,<timestamp> |
16:39:27 | bluebrother^ | and the same with a [daily] section for daily builds. We already support a [release-candidate] section with a format <player>=<version>,<url> so we could also add an optional ,<url> for [development] and [daily] |
16:40:08 | bluebrother^ | that would simplify handling in Rockbox Utility a bit, since now I'm assuming that we always have daily / development builds (unless status = 0) |
16:40:57 | bluebrother^ | but moving it to a similar format we have for [release] we could simply leave the ones out that don't have daily builds, and if it's not present we know the build doesn't exist. |
16:46:04 | braewoods | hm. i seem to have figured out how to tap into rockbox's usb stack |
16:48:00 | braewoods | part one of MTP support... figuring out how to integrate into the usb stack |
16:48:13 | _bilgus | braewoods, there is an override backlight function in the plugin.h list |
16:52:14 | braewoods | _bilgus: i figured it out. there's helpers to do what i needed. |
16:52:49 | braewoods | also started working on my WIP for MTP support |
16:52:56 | braewoods | got a skeleton drafted for future work |
16:54:11 | braewoods | but i see it being useful for safer file transfers |
16:54:17 | braewoods | how much slower it'll be i don't know |
16:54:39 | braewoods | though since we use fat32 the argument about portability doesn't really fly |
16:57:48 | speachy | bluebrother^: it's all generated by a couple of scripts in the www repo |
16:58:42 | speachy | bluebrother^: is that a complete URL to the actual file, or a template of some sort? |
17:00 |
17:00:19 | speachy | and <revision> is only used for display purposes, so for [daily] we'd use a datestamp (eg '20201113') and [development] we'd use a shortened git id? |
17:00:57 | speachy | braewoods: pamaury's partially-completed MTP stuff is probably a better place to start for native targets |
17:01:17 | braewoods | speachy: i can reference it. we'll see where i get with it all. |
17:01:53 | speachy | on android base (pretty much only the dx50/dx90) we get it for free but on other hosted stuff I don't know if we can make any assumptions about it being there. |
17:02:31 | braewoods | no idea but there are libraries we can use for Linux |
17:02:45 | braewoods | provided the target has a suitable kernel or so |
17:03:13 | braewoods | the easiest place to start though is native targets |
17:03:43 | braewoods | _bilgus: care to review my iriver_flash commits? they should be easy enough to review. |
17:14:30 | speachy | bluebrother^: and do you only want the latest daily listed? |
17:14:55 | speachy | (I'm assuming you still want the [dailies] section with the timestamp / revision broken out? |
17:48:02 | fs-bluebot_ | Build Server message: New build round started. Revision 33d42c2, 293 builds, 9 clients. |
17:49:03 | | Quit Rower (Ping timeout: 260 seconds) |
18:00 |
18:00:49 | fs-bluebot_ | Build Server message: Build round completed after 768 seconds. |
18:00:51 | fs-bluebot_ | Build Server message: Revision 33d42c2 result: All green |
18:07:42 | *** | Saving seen data "./dancer.seen" |
18:20:44 | fs-bluebot_ | Build Server message: New build round started. Revision 7d9ac02, 293 builds, 9 clients. |
18:28:58 | speachy | would anyone mind terribly if I took down git+gerrit to do another upgrade? |
18:30:42 | speachy | downtime shouldn't be more than 10-15min. |
18:33:08 | fs-bluebot_ | Build Server message: Build round completed after 744 seconds. |
18:33:12 | fs-bluebot_ | Build Server message: Revision 7d9ac02 result: All green |
18:33:13 | fs-bluebot_ | Build Server message: New build round started. Revision c10eea4, 293 builds, 9 clients. |
18:43:59 | mendel_munkis_ | speachy: I am okay with that. |
18:44:11 | | Nick mendel_munkis_ is now known as mendel_munkis (~mendelmun@ool-435680b7.dyn.optonline.net) |
18:44:14 | speachy | once the builders all finally go idle anyway. |
18:44:16 | | Join amsomniac [0] (~nadya@c-75-68-250-81.hsd1.vt.comcast.net) |
18:44:19 | mendel_munkis | yeah |
18:44:28 | mendel_munkis | _bilgus, |
18:45:10 | mendel_munkis | now I need to figure how to port a plugin off of memset(rb->lcd_framebuffer) |
18:45:57 | fs-bluebot_ | Build Server message: Build round completed after 765 seconds. |
18:45:58 | fs-bluebot_ | Build Server message: Revision c10eea4 result: All green |
18:54:50 | _bilgus | mendel have a look at the vp test program |
18:55:09 | mendel_munkis | will do. thanks |
19:00 |
19:05:41 | _bilgus | https://github.com/Rockbox/rockbox/blob/master/apps/plugins/fire.c#L383 |
19:06:05 | _bilgus | thats probably closer to what you want just a raw fb ptr.. |
19:10:46 | speachy | ok, gerrit's going offline for a bit. |
19:13:06 | | Quit amsomniac (Quit: Leaving.) |
19:18:35 | | Quit lebellium (Quit: Leaving) |
19:24:28 | speachy | it's online again. |
19:26:58 | mendel_munkis | how is PLUGIN_BUFFER_SIZE determined at port time? |
19:28:52 | mendel_munkis | I'm looking at specs and I see the mrobe500 has P_B_S of 0x100000 with 256mbit of ram as opposed tothe fuze+ which has 0x80000 and 512. |
19:32:31 | fs-bluebot_ | Build Server message: New build round started. Revision 3ef8455, 293 builds, 9 clients. |
19:39:50 | speachy | lovely, broken. |
19:41:27 | mendel_munkis | what exactly is broken? |
19:41:34 | speachy | buildmaster |
19:44:44 | speachy | or rather, it's handing out a URL to the builders that's not working properly |
19:45:20 | mendel_munkis | by the way I like the page at buildmaster.rockbox.org |
19:50:23 | fs-bluebot_ | Build Server message: New build round started. Revision c10eea4, 293 builds, 9 clients. |
19:51:29 | speachy | there. |
19:57:46 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
19:58:49 | | Join fs-bluebot [0] (~fs-bluebo@55d416be.access.ecotel.net) |
19:58:58 | speachy | I'd appreciate it if folks could try out the new PolyGerrit UI |
20:00 |
20:00:25 | | Quit bluebrother^ (Ping timeout: 240 seconds) |
20:00:45 | | Quit fs-bluebot_ (Ping timeout: 240 seconds) |
20:02:37 | fs-bluebot | Build Server message: Build round completed after 734 seconds. |
20:02:38 | fs-bluebot | Build Server message: Revision c10eea4 result: All green |
20:05:17 | _bilgus | speachy it gives 404 |
20:05:45 | speachy | so it's not just my browser then. :/ |
20:06:17 | speachy | looking at the www logs it's requesting clearly wrong URLs. |
20:07:17 | speachy | it's "working except for pulling up diffs |
20:07:46 | *** | Saving seen data "./dancer.seen" |
20:09:03 | | Quit prof_wolfff (Ping timeout: 260 seconds) |
20:10:19 | speachy | appears to be a problem with the reverse proxy. |
20:18:57 | | Join lemon_jesus3 [0] (~lemon_jes@208.59.79.137) |
20:20:45 | | Quit lemon_jesus (Ping timeout: 240 seconds) |
20:20:45 | | Nick lemon_jesus3 is now known as lemon_jesus (~lemon_jes@208.59.79.137) |
20:21:30 | _bilgus | lemon_jesus is looking for some help with the nand in the ipod 3g? |
20:21:54 | lemon_jesus | yes yes yes |
20:23:11 | lemon_jesus | I'm having a hard time figuring out the FIL |
20:23:32 | __builtin | you want to ping [7], I think |
20:24:15 | lemon_jesus | is that what TheSeven goes by nowadays, or is that someone else? |
20:24:25 | __builtin | that's TheSeven, yes |
20:24:56 | __builtin | I think he's on european time so you'll have to be patient |
20:26:14 | lemon_jesus | that's cool, I've been picking away at understanding the interface. it's been fun but also infuriating! |
20:26:57 | __builtin | what exactly are you doing? |
20:30:17 | speachy | ...wtf.. the javascript code isn't encoding the URLs properly. |
20:34:16 | lemon_jesus | __builtin good question - the project started with me trying to double the internal storage size of the n3g by replacing the NAND. obviously this didn't work and now it's months later and I'm down this rabbit hole where I've been trying to understand how the S5L8702 NAND interface works and I figure as I investigate this problem, I can contribute |
20:34:16 | lemon_jesus | to a Rockbox n3g NAND driver. |
20:36:36 | lemon_jesus | I've primarily been doing some stuff in Ghidra to attempt to fully understand the interactions of the software with the S5L8702, but I'm stuck at understanding what these registers do. I'm at a point where I'm not even sure what to Google anymore, and the datasheets for other samsung/nand related chips don't really have analogs to what I'm seeing |
20:36:37 | lemon_jesus | in the code. |
20:40:42 | __builtin | well, I wish you luck :) |
20:40:49 | __builtin | FTLs are notoriously difficult to RE |
20:42:25 | fs-bluebot | Build Server message: New build round started. Revision 3ef8455, 293 builds, 9 clients. |
20:42:44 | lemon_jesus | heh yeah I've been getting that sense |
20:43:07 | __builtin | any chance you could share what you've got? |
20:43:24 | __builtin | I wouldn't mind poking around in ghidra |
20:49:13 | mendel_munkis | well I switched to new gui and now I cant see anything |
20:50:32 | | Quit ac_laptop (Ping timeout: 246 seconds) |
21:00 |
21:02:34 | fs-bluebot | Build Server message: Build round completed after 1209 seconds. |
21:02:35 | fs-bluebot | Build Server message: Revision 3ef8455 result: All green |
21:03:35 | lemon_jesus | __builtin yeah I'll send you what I got. I'll DM you. |
21:05:16 | speachy | FIXED IT. GAAH. |
21:08:11 | speachy | _bilgus: new UI should work properly finally |
21:08:54 | speachy | (apparently the old UI was more forgiving of a config error that was there before I took things over) |
21:09:53 | _bilgus | I can't resize the review panes and they no longer center in the screen |
21:10:28 | speachy | _bilgus: that's beyond my purview. I'm just glad it _works_ finally |
21:10:55 | _bilgus | ok well the new ui sucks in the nicesest way possible lol |
21:10:56 | speachy | the next version drops the old UI entirely |
21:11:23 | speachy | but they do keep improving things so perhaps your complaint will become moot in the next upgrade. |
21:11:24 | _bilgus | I hope tey polished it then |
21:11:52 | speachy | In this version (2.16) they officially deprecated the old UI, considering the new UI feature complete |
21:12:14 | speachy | spent _far_ too much time on this @#%@$ thing |
21:13:36 | speachy | heh, one thing I want to do is add a commit hook that will reject things if there's messed-up whitespace |
21:13:49 | speachy | or inappropriate tabs |
21:14:02 | | Join massiveH [0] (~massiveH@ool-18e4e82f.dyn.optonline.net) |
21:16:34 | _bilgus | that would be nice |
21:19:16 | | Join ac_laptop [0] (~ac_laptop@186.2.247.129) |
21:23:18 | | Quit ufdm (Read error: Connection reset by peer) |
21:34:40 | | Join ufdm [0] (~ufdm@c-73-164-63-214.hsd1.mn.comcast.net) |
21:42:40 | | Join MrZeus__ [0] (MrZeus@gateway/vpn/mullvad/mrzeus) |
21:45:29 | | Quit MrZeus_ (Read error: Connection reset by peer) |
22:00 |
22:03:05 | | Quit MrZeus__ (Ping timeout: 240 seconds) |
22:04:59 | | Quit ufdm (Read error: Connection reset by peer) |
22:05:02 | | Join ufdm_ [0] (~ufdm@c-73-164-63-214.hsd1.mn.comcast.net) |
22:07:47 | *** | Saving seen data "./dancer.seen" |
22:08:39 | | Quit ufdm_ (Client Quit) |
22:08:51 | | Join ufdm [0] (~ufdm@c-73-164-63-214.hsd1.mn.comcast.net) |
22:42:39 | | Quit TheEaterOfSouls (Remote host closed the connection) |
22:46:18 | mendel_munkis | it's not as bad as the old new UI |
22:52:24 | _bilgus | speachy, https://gerrit.rockbox.org/oauth?error=redirect_uri_mismatch&error_description=The+redirect_uri+MUST+match+the+registered+callback+URL+for+this+application.&error_uri=https%3A%2F%2Fdocs.github.com%2Fapps%2Fmanaging-oauth-apps%2Ftroubleshooting-authorization-request-errors%2F%23redirect-uri-mismatch |
22:53:23 | speachy | _bilgus: d'oh. |
22:53:25 | speachy | gimme a few. |
23:00 |
23:02:41 | speachy | _bilgus: github access restored |
23:03:36 | speachy | google auth pending. |
23:03:43 | speachy | (they need to verify my changes) |
23:05:31 | | Quit tchan (Quit: WeeChat 2.8) |
23:13:23 | speachy | oh ffs. Google mandates having a privacy policy |
23:14:05 | _bilgus | rabbit holes |
23:14:15 | speachy | ...or rather, now they do |
23:15:04 | _bilgus | ok so singing in allows me to set preferences that make it better but it still is off a bit but bearable |
23:16:09 | _bilgus | hope the next one has a dark theme w/o resorting to css tricks |
23:17:02 | _bilgus | I like the show blame feature |
23:34:49 | | Join tchan [0] (~tchan@lunar-linux/developer/tchan) |
23:35:09 | speachy | https://www.rockbox.org/wiki/PrivacyPolicy |
23:58:37 | | Quit [7] (Ping timeout: 260 seconds) |
23:59:00 | | Join TheSeven [0] (~quassel@rockbox/developer/TheSeven) |