| --- Log opened Tue Apr 16 00:00:17 2013 | ||
| stekern | juliusb: could still be some bus related problem still, I've sometimes had funny behaviour when reading the flash from u-boot | 01:27 |
|---|---|---|
| stekern | juliusb: ok, you treehugger ;) let's keep the gate on the operands, that will keep it from "wiggling": http://pastie.org/7595269 | 01:38 |
| mor1kx | [mor1kx] skristiansson pushed 3 new commits to master: https://github.com/openrisc/mor1kx/compare/5d1dd717c2ac...6c7d7d3ee656 | 03:47 |
| mor1kx | mor1kx/master 685982b Stefan Kristiansson: execute_alu: make branch destination calculation optional... | 03:47 |
| mor1kx | mor1kx/master 7f3670e Stefan Kristiansson: cappuccino/cpu: disable branch destination calculation in ALU | 03:47 |
| mor1kx | mor1kx/master 6c7d7d3 Stefan Kristiansson: execute_alu: make alu_result default to adder_result... | 03:47 |
| jemarch | hi | 11:11 |
| mor1kx | [mor1kx] skristiansson pushed 3 new commits to master: https://github.com/openrisc/mor1kx/compare/6c7d7d3ee656...f88dde32385c | 12:38 |
| mor1kx | mor1kx/master 1547a64 Stefan Kristiansson: cappuccino/rf: combine 'last_wb_result_b' and 'wb_to_decode_result'... | 12:38 |
| mor1kx | mor1kx/master a88e42b Stefan Kristiansson: cappuccino/execute_ctrl: remove unused ctrl_op_jal_o signal | 12:38 |
| mor1kx | mor1kx/master f88dde3 Stefan Kristiansson: cappuccino/rf: remove unnecessary decode to decode bypass... | 12:38 |
| mor1kx | [mor1kx] skristiansson pushed 3 new commits to master: https://github.com/openrisc/mor1kx/compare/f88dde32385c...f0a01bedbd3d | 14:34 |
| mor1kx | mor1kx/master 2a8547c Stefan Kristiansson: cappuccino/rf: reset wb_hazard_a/b instead of wb_hazard_result... | 14:34 |
| mor1kx | mor1kx/master ef862d1 Stefan Kristiansson: cappuccino/fetch: stop flushing on du_restart_i | 14:34 |
| mor1kx | mor1kx/master f0a01be Stefan Kristiansson: cappuccino/ctrl: clear branch_step when npc is written... | 14:34 |
| stekern | juliusb: you're not the only one fixing debugging-related bugs =P | 14:36 |
| @juliusb | :) | 15:48 |
| @juliusb | it's quite good actually, I hadn't noticed these bugs in simulation, or thought about them | 15:48 |
| @juliusb | but when I was finally using the design ont he board a little bit those things annoyed me greatly (when we step into or break on a l.j 0 it couldn't resume because the fetcher had locked up so had to reset the whoel thing) | 15:49 |
| @juliusb | it was a 2 line fix to stop that, but also added tests to recreate that situation | 15:52 |
| @juliusb | but usability issues like that really piss you off\ | 15:52 |
| stekern | yeah, I should fix bp support in cappuccino and then really try using the debugger with it | 16:04 |
| stekern | when he comes back from fat-camp | 16:04 |
| stekern | juliusb: is the 2nd try on the mul patch something you could live with btw? | 16:05 |
| @juliusb | oh didn't see it yet... | 16:51 |
| @juliusb | does it work OK with having only mul_op? | 16:52 |
| stekern | yup, I ran the 200000 test against espresso and cappuccino | 17:05 |
| stekern | and a and b should stay stable as long as you have the mul in there, otherwise I reckon you have another bug going on | 17:06 |
| stekern | but the main purpose of the patch was actually to remove the !rst in this (slightly restructured) original code: http://pastie.org/7601624 | 17:25 |
| stekern | then I got carried away with the logic removing ;) | 17:25 |
| stekern | but I think the latest patch should match the previous behaviour, non-wiggling and everything | 17:26 |
| @juliusb | oh right, fair enough, !rst is pretty dirty | 17:36 |
| @juliusb | soudns good | 17:36 |
| @juliusb | 200000 test? you mean GCC? | 17:36 |
| stekern | no, the mul test | 17:37 |
| @juliusb | ahh sorry im with you | 17:39 |
| @juliusb | yeah cool | 17:39 |
| @juliusb | how'd you run that? | 17:39 |
| @juliusb | on HW? | 17:39 |
| stekern | in verilator | 17:40 |
| stekern | I might have a 0 too much there though | 17:40 |
| @juliusb | :) | 17:40 |
| @juliusb | nps | 17:40 |
| stekern | the default or1k-mul | 17:41 |
| @juliusb | ya think it's 20k | 17:41 |
| stekern | cappuccino is doing well at fatcamp btw, he's already lost ~300 LC (in the altera way of measuring things) | 17:58 |
| stekern | still a chubby boy though, weighs in at 5300 LC | 17:59 |
| stekern | I want that to be < 4000 before I can say that I'm proud of him | 18:00 |
| @juliusb | wow, that's ambitious! | 20:49 |
| @juliusb | you reckon you can? | 20:49 |
| @juliusb | and why 4000? | 20:49 |
| @juliusb | OK I can read the contents of the SPI now | 22:16 |
| @juliusb | they look good | 22:16 |
| @juliusb | but that's from a C-code app | 22:16 |
| @juliusb | so why isn't the bootrom working..... | 22:16 |
| @juliusb | ? | 22:16 |
| @juliusb | works 100% in sim | 22:17 |
| @juliusb | i've updated it to support no-delay slot stuff | 22:17 |
| @juliusb | as I said, works fine in sim | 22:17 |
| @juliusb | bootrom.S is getting assembled with -mno-delay | 22:17 |
| @juliusb | hmmm | 22:18 |
| @juliusb | now it works!! | 22:18 |
| @juliusb | anoying.... | 22:18 |
| @juliusb | I made it delay before attempting to read the flash | 22:18 |
| mor1kx | [mor1kx] juliusbaxter pushed 2 new commits to master: https://github.com/openrisc/mor1kx/compare/f0a01bedbd3d...6ad603b83599 | 22:41 |
| mor1kx | mor1kx/master 809a1cc Julius Baxter: ctrl_pronto: fix bug in PC checker... | 22:41 |
| mor1kx | mor1kx/master 6ad603b Julius Baxter: fetch_pronto: fix sleep behaviour when debugging... | 22:41 |
| @juliusb | stekern: had to do this to get to compile in modelsim: http://pastie.org/7620998 | 22:51 |
| @juliusb | (didn't strictly have to make a whitespace mess, though) | 22:52 |
| --- Log closed Wed Apr 17 00:00:19 2013 | ||
Generated by irclog2html.py 2.15.2 by Marius Gedminas - find it at mg.pov.lt!