| 00:06:57 | | Quit speachy (Ping timeout: 240 seconds) |
| 00:08:44 | | Quit JanC (Remote host closed the connection) |
| 00:08:57 | | Join JanC [0] (~janc@user/janc) |
| 01:00 |
| 01:04:38 | *** | Saving seen data "./dancer.seen" |
| 02:00 |
| 02:04:04 | | Join IgorCh [0] (~chernokal@128-74-95-94.broadband.corbina.ru) |
| 03:00 |
| 03:04:40 | *** | No seen item changed, no save performed. |
| 03:46:25 | | Quit pixelma (Quit: .) |
| 03:46:25 | | Quit amiconn (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.) |
| 03:47:33 | | Join amiconn [0] (jens@p200300ea8717ee00305e95fffec66ff3.dip0.t-ipconnect.de) |
| 03:47:33 | | Join pixelma [0] (marianne@p200300ea8717ee00305e95fffec66ff3.dip0.t-ipconnect.de) |
| 04:00 |
| 04:17:04 | | Join lebellium [0] (~lebellium@2a01cb04012c0900b80be005e271fa99.ipv6.abo.wanadoo.fr) |
| 05:00 |
| 05:04:42 | *** | Saving seen data "./dancer.seen" |
| 05:06:40 | | Quit S|h|a|w|n (Read error: Connection reset by peer) |
| 05:22:11 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 05:39:39 | | Join ircs [0] (~ircs@2-111-85-72-cable.dk.customer.tdc.net) |
| 05:49:08 | | Join ZincAlloy [0] (~Adium@ip5f5abcae.dynamic.kabel-deutschland.de) |
| 05:52:20 | | Quit ircs (Read error: Connection reset by peer) |
| 05:53:48 | | Quit ZincAlloy (Ping timeout: 256 seconds) |
| 06:00 |
| 06:22:34 | | Join ZincAlloy [0] (~Adium@2a02:8108:943f:d824:edde:dd9e:468b:9f44) |
| 07:00 |
| 07:04:44 | *** | Saving seen data "./dancer.seen" |
| 07:13:42 | | Quit demap (Ping timeout: 256 seconds) |
| 07:19:22 | | Join amachronic [0] (~amachroni@user/amachronic) |
| 07:24:33 | amachronic | dconrad: whoops! my bad. seems weird that the AXP churn broke the LCD though. |
| 07:50:32 | | Quit ufdm (Ping timeout: 256 seconds) |
| 08:00 |
| 08:00:22 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 08:04:37 | | Quit demap (Ping timeout: 240 seconds) |
| 08:25:44 | | Join ufdm [0] (~ufdm@c-68-46-16-107.hsd1.mn.comcast.net) |
| 08:38:21 | | Join dconrad [0] (~dconrad@152.117.104.224) |
| 08:39:24 | dconrad | amachronic, no worries, but do you have any thoughts on what might be going on? |
| 08:40:42 | dconrad | my thought is maybe power to the lcd was dropping after initialization, so it loses all the initialization we do |
| 08:41:42 | amachronic | nothing immediately springs to mind but I'd have to look at the old code to see what's changed |
| 08:47:39 | amachronic | looks like the biggest change is setting enabled supplies, which wasn't done at all before. |
| 08:50:21 | amachronic | maybe you should just remove that call and see what happens? |
| 08:50:28 | dconrad | yeah I can try that |
| 08:51:14 | dconrad | I think I noticed that before, we were reading the existing state and adding a supply to it to make sure it was on, maybe something we don't know about needs to be turned on? |
| 08:51:40 | amachronic | that was the ADC |
| 08:51:48 | dconrad | ah shoot haha |
| 08:52:01 | dconrad | you flip flopped them in the code |
| 08:52:06 | amachronic | ironically that was one of the buggy operations that prompted me to do the rewrite |
| 08:52:53 | amachronic | the axp's i2c multi register writes are strange and I screwed it up the first time around |
| 08:53:39 | amachronic | I probably should've stuck to fixing that and left the rest of that crap alone :P |
| 08:56:01 | dconrad | haha oh well, thats what happens when you have auto-translated documentation at best |
| 09:00 |
| 09:01:24 | amachronic | perhaps it's a timing issue? I see the erosq lcd code doesn't have a reset+delay at first like the M3K does. |
| 09:02:31 | amachronic | rather oddly there's a 120ms delay after the first 'sleep out' command... I guess that weirdness comes from the OF? |
| 09:04:46 | *** | Saving seen data "./dancer.seen" |
| 09:04:51 | amachronic | if setting the supplies causes a glitch in the power supplies maybe the LCD module or controller sees it as a reset. |
| 09:08:55 | rb-bluebot | Build Server message: New build round started. Revision f1ddd6f014, 303 builds, 10 clients. |
| 09:11:19 | dconrad | well, the old code did set PWROUTPUTCTRL2 to 0x5c, though I don't quite know what that maps to |
| 09:12:17 | amachronic | that's basically everything |
| 09:12:26 | dconrad | yeah, no change commenting out axp_set_enabled_supplies() |
| 09:14:26 | dconrad | I could certainly try copying the m3k's reset and sleep out commands for the lcd |
| 09:14:41 | dconrad | I imagine they use the same command set |
| 09:14:54 | amachronic | yeah, the MIPI standard stuff |
| 09:21:01 | dconrad | no change there either |
| 09:21:13 | rb-bluebot | Build Server message: Build round completed after 738 seconds. |
| 09:21:18 | rb-bluebot | Build Server message: Revision f1ddd6f014 result: All green |
| 09:21:43 | rb-bluebot | Build Server message: New build round started. Revision 3adeae2026, 303 builds, 9 clients. |
| 09:22:07 | amachronic | I could just revert the AXP rewrite since there's nothing too critical in it |
| 09:23:01 | amachronic | but I don't understand why the LCD broke to begin with? |
| 09:23:24 | dconrad | yeah I don't get why either |
| 09:24:15 | dconrad | ok, so 5c maps exactly to dcdc1-3, ldo2-3, + exten according to your register document |
| 09:24:54 | dconrad | sorry, 5f |
| 09:25:39 | dconrad | we are no longer setting the sample rate to 25hz, though I don't see why that would have an effect |
| 09:26:27 | amachronic | i found that exten & dcdc2 were in _both_ PWRCTL1 and the upper 4 bits in PWRCTL2 |
| 09:26:50 | amachronic | the new code has some shenanigans to account for that but perhaps it's misguided. |
| 09:27:09 | dconrad | perhaps its an "and" relationship to turn them on? |
| 09:27:48 | amachronic | iirc updates to one will reflect in the other. |
| 09:27:57 | dconrad | hmm |
| 09:28:51 | amachronic | did you put back the old PWRCTL2 setting when you removed set_enabled_supplies? |
| 09:29:16 | amachronic | i somehow totally missed that the old code was setting supplies too |
| 09:29:20 | dconrad | oh, no I didn't |
| 09:29:25 | dconrad | I can try that |
| 09:30:13 | amachronic | if the ErosQ is anything like the M3K & Q1 some of those non-default supplies will be needed for the SD card etc. |
| 09:31:11 | rb-bluebot | Build Server message: Build round completed after 568 seconds. |
| 09:31:12 | rb-bluebot | Build Server message: Revision 3adeae2026 result: All green |
| 09:31:13 | dconrad | what's the name of the register, is it AXP_REG_PWRCTRL2? |
| 09:33:07 | amachronic | AXP_REG_PWRCTL2 |
| 09:33:42 | dconrad | ah whoops |
| 09:36:59 | rb-bluebot | Build Server message: New build round started. Revision 8f063d49c2, 303 builds, 10 clients. |
| 09:37:08 | dconrad | yeah that doesn't seem to be it either |
| 09:37:28 | dconrad | just for fun I am going to put those two lines down after setting up the adcs though |
| 09:38:40 | dconrad | ok that didn't do anything |
| 09:43:32 | dconrad | is the adc sample rate still being set to 25hz somewhere? |
| 09:44:17 | amachronic | no, it seems I forgot about that |
| 09:45:39 | amachronic | can't see why that would affect it though. |
| 09:46:05 | dconrad | I don't either, but it's a change and I'm just kind of trying stuff |
| 09:46:33 | rb-bluebot | Build Server message: Build round completed after 574 seconds. |
| 09:46:34 | rb-bluebot | Build Server message: Revision 8f063d49c2 result: All green |
| 09:48:41 | dconrad | I imagine it's set by ADCCTL SAMPLE_RATE, are valid values written down anywhere? |
| 09:51:39 | amachronic | it's in the datasheet reg 0x84 |
| 09:51:51 | amachronic | 25 Hz is 0 |
| 09:51:57 | dconrad | ah, thanks |
| 09:54:39 | amachronic | did you ever figure out what the LCD controller was on this thing? |
| 09:54:52 | dconrad | no, it remains a mystery |
| 09:55:53 | dconrad | although there seem to be some commands that aren't strictly defined in the standard, so someone who is familiar might be able to make a good guess |
| 09:56:09 | dconrad | or at least commands I couldn't find what they do |
| 10:00 |
| 10:04:55 | dconrad | assuming I wrote to it correctly, |
| 10:05:10 | dconrad | int readaxp = axp_read(AXP_REG_ADCCTL); |
| 10:05:10 | dconrad | axp_write(AXP_REG_ADCCTL, (readaxp & 0x3f)); |
| 10:05:14 | dconrad | no change |
| 10:05:41 | dconrad | which... is expected I guess |
| 10:09:16 | amachronic | oh well. I'll try a revert (at least for you to test) though it's a bit of a hack job since I've added stuff |
| 10:10:09 | dconrad | sounds good, I appreciate it |
| 10:12:13 | amachronic | g#4166 |
| 10:12:16 | rb-bluebot | Gerrit review #4166 at https://gerrit.rockbox.org/r/c/rockbox/+/4166 : Revert "AXP PMU rewrite (again)" by Aidan MacDonald |
| 10:12:27 | amachronic | it compiles, but I haven't tested it. |
| 10:12:48 | dconrad | I'll check it out and see how it goes |
| 10:18:44 | amachronic | i have to step away from the computer for a bit but I'll check back in later. hopefully the revert will mysteriously fix the problem. |
| 10:19:07 | dconrad | well I can't speak to any regressions but it does work this way |
| 10:19:35 | dconrad | yeah have a good one, we'll get to the bottom of it |
| 10:20:04 | amachronic | alright, at least it works again in the meantime :) |
| 10:20:08 | | Quit amachronic (Quit: amachronic) |
| 10:34:15 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 10:52:37 | | Quit ufdm (Ping timeout: 240 seconds) |
| 11:00 |
| 11:04:50 | *** | Saving seen data "./dancer.seen" |
| 11:09:32 | | Join ufdm [0] (~ufdm@c-68-46-16-107.hsd1.mn.comcast.net) |
| 11:47:25 | | Quit demap (Ping timeout: 240 seconds) |
| 11:51:21 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 11:55:54 | | Quit demap (Ping timeout: 256 seconds) |
| 13:00 |
| 13:04:54 | *** | Saving seen data "./dancer.seen" |
| 14:00 |
| 14:27:12 | | Quit IgorCh (Ping timeout: 256 seconds) |
| 14:43:10 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 14:47:27 | | Quit demap (Ping timeout: 256 seconds) |
| 15:00 |
| 15:04:56 | *** | Saving seen data "./dancer.seen" |
| 15:16:00 | | Join amachronic [0] (~amachroni@user/amachronic) |
| 15:20:31 | amachronic | dconrad: i pushed a series to revert the axp changes - tip is g4170 if you want to test. |
| 15:20:56 | dconrad | oh, yeah I can definitely do that |
| 15:22:07 | dconrad | it looks like it's applying some of the things you changed onto the preexisting axp code instead of completely rewriting it? |
| 15:22:40 | amachronic | yep |
| 15:22:49 | amachronic | the 1st in the series is a pure revert |
| 15:23:08 | amachronic | next 2 are pure cleanups, the last is an actual bugfix |
| 15:25:48 | dconrad | sounds good, it's building now |
| 15:29:26 | dconrad | looks like that whole chain works well, so whatever that bug was isn't in these changes |
| 15:30:44 | dconrad | are there any specific tests or things to look at with these changes? |
| 15:31:26 | amachronic | if it charges at the right rate and the enabled ADCs seem to work it should be fine |
| 15:32:06 | dconrad | it seems to be reading the battery ok, let me see how much current it will pull |
| 15:32:43 | dconrad | should "Charge During USB Connection" still be set to force? |
| 15:32:50 | dconrad | or, is that still default? |
| 15:32:55 | amachronic | yeah it is |
| 15:35:25 | amachronic | I've tried to rework USB charging, but I'm starting to think it's not worth trying to comply with the USB specs, since the original firmwares don't and USB hosts don't seem to care. |
| 15:36:52 | dconrad | it seems to be happily pulling 450 mA from both an AC adapter or my computer (in charge-only mode), and all the ADCs seem to be working correctly in the debug page |
| 15:37:19 | dconrad | I imagine approximately complying with USB is about the closest any of these get |
| 15:38:25 | dconrad | of course, now the battery is full so I_chrg is 0 |
| 15:38:38 | amachronic | good! I'll merge it then since afaik there should be no regressions. |
| 15:39:32 | rb-bluebot | Build Server message: New build round started. Revision c62c323ebc, 303 builds, 11 clients. |
| 15:41:01 | dconrad | it's a bummer reverting a nice clean rewrite though :-\ |
| 15:42:53 | amachronic | yeah, but functionally it didn't do much except shave 3k off the binary. |
| 15:45:59 | | Quit amachronic (Quit: amachronic) |
| 15:49:02 | rb-bluebot | Build Server message: Build round completed after 570 seconds. |
| 15:49:04 | rb-bluebot | Build Server message: Revision c62c323ebc result: All green |
| 17:00 |
| 17:04:57 | *** | Saving seen data "./dancer.seen" |
| 17:32:30 | | Quit _bilgus_ (Ping timeout: 256 seconds) |
| 17:46:57 | | Join speachy [0] (~speachy@gw-hs.shaftnet.org) |
| 17:46:57 | | Quit speachy (Changing host) |
| 17:46:57 | | Join speachy [0] (~speachy@rockbox/developer/speachy) |
| 17:46:57 | Mode | "#rockbox +v speachy" by ChanServ (ChanServ@services.libera.chat) |
| 17:47:05 | | Quit lebellium (Quit: Leaving) |
| 17:51:03 | | Quit emacsomancer (Ping timeout: 256 seconds) |
| 18:00 |
| 18:43:40 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 18:47:49 | | Quit demap (Ping timeout: 240 seconds) |
| 19:00 |
| 19:04:58 | *** | Saving seen data "./dancer.seen" |
| 19:13:46 | | Join emacsomancer [0] (~emacsoman@136.60.128.68) |
| 19:19:19 | | Join skipwich [0] (~skipwich@user/skipwich) |
| 19:19:40 | | Quit skipwich (Read error: Connection reset by peer) |
| 19:20:40 | | Join skipwich [0] (~skipwich@user/skipwich) |
| 19:34:07 | | Quit ZincAlloy (Quit: Leaving.) |
| 20:00 |
| 20:29:22 | | Join _bilgus_ [0] (~bilgus@162.154.213.134) |
| 21:00 |
| 21:05:01 | *** | Saving seen data "./dancer.seen" |
| 21:54:59 | | Quit dconrad (Remote host closed the connection) |
| 22:00 |
| 22:06:01 | | Join dconrad [0] (~dconrad@152.117.104.224) |
| 22:13:36 | | Quit dconrad () |
| 22:14:18 | | Join demap [0] (~jelle@58-253-250-62.ftth.glasoperator.nl) |
| 22:18:40 | | Quit demap (Ping timeout: 256 seconds) |
| 23:00 |
| 23:05:04 | *** | Saving seen data "./dancer.seen" |
icon identifies that the person is a core developer (has commit access).