stekern | _franck_: did you see that message on the ml? have you been able to single step with openocd + adv_debug_sys? | 05:22 |
---|---|---|
stekern | I thought LoneTech's patch fixed that, but perhaps those are seperate issues... | 05:23 |
stekern | olofk: yes, one user per repo (they could of course be named the same). You can also choose to monitor specific branches (i.e. only master) | 05:46 |
stekern | if we set -n on the channel the user wouldn't need to join/quit for the announcements neither | 05:47 |
stekern | but it seems we lost op over this channel ;) | 05:47 |
-!- _franck_2_ is now known as _franck_web_ | 11:25 | |
root` | Hello everyone | 11:45 |
root` | ^_^ | 11:46 |
jeremybennett | stekern: juliusb has op on this channel I think | 12:04 |
jeremybennett | I'm not a channel operator. Perhaps we ought to get juliusb to add a few of us. | 12:04 |
juliusb | mmm | 12:11 |
juliusb | I have to remember how to get it | 12:11 |
-!- mode/#openrisc [+o juliusb] by ChanServ | 12:12 | |
@juliusb | that's it | 12:12 |
-!- mode/#openrisc [+o stekern] by ChanServ | 12:14 | |
-!- mode/#openrisc [+o jeremybennett] by ChanServ | 12:15 | |
-!- mode/#openrisc [+o olofk] by ChanServ | 12:15 | |
ConX | hello all! Can you please tell me why I am getting the following when trying to compile or1200's trunk version: or1200/rtl/verilog/or1200_genpc.v" line 281 'pcreg_boot' has not been declared | 12:45 |
@stekern | ConX: what are you trying to compile it with? | 13:36 |
ConX | I am using minsoc's build system | 13:39 |
@stekern | ok, but what is the target? xilinx fpga, altera, simulation (which simulator)? | 13:40 |
ConX | xilinx | 13:42 |
@stekern | olofk: you are using pcreg_boot before you have declared it | 13:42 |
@stekern | ConX: quick fix: add a | 13:43 |
@stekern | wire [31:0] pcreg_boot; | 13:43 |
@stekern | around line 125 | 13:44 |
@stekern | and change line 294 from | 13:44 |
@stekern | wire [31:0] pcreg_boot = boot_adr; | 13:44 |
@stekern | to | 13:44 |
@stekern | assign pcreg_boot = boot_adr; | 13:44 |
ConX | did the first and I was getting errors at the second (I am still learning Verilog) | 13:45 |
ConX | thank you stekern ! | 13:45 |
@stekern | ... and then diff it and send it as a patch to the mailinglists ;) | 13:45 |
ConX | :) | 13:46 |
ConX | ;) | 13:46 |
@stekern | although, now I see the simpler fix would be to just move line 294 to line 267 | 13:54 |
_franck_ | stekern: no, I didn't try to single step with openocd + adv_debug_sys | 19:51 |
_franck_ | I'll give it a try | 19:52 |
_franck_ | but like you I was sure LoneTech fixed this issue with his patch | 19:52 |
@olofk | Sorry about the genpc problem. I discovered it myself yesterday when I was adding modelsim support for orpsocv3 | 20:00 |
@olofk | Icarus didn't complain, so I missed that one | 20:00 |
@jeremybennett | olofk: Did it get through Verilator - it can be quite picky. | 20:16 |
@stekern | olofk: no worries, we are as guilty, we should have picked it up in the review ;) luckily we have good testers reporting back (ConX) | 20:17 |
@stekern | I think quartus allows that as well, but not 100% sure | 20:18 |
_franck_ | stekern: I confirm, stepi after a breakpoint on a branch instruction doesn't branch... | 20:34 |
_franck_ | I'm using openOCD | 20:34 |
_franck_ | LoneTech patch was about a stall not a wrong branch behavior | 20:35 |
@jeremybennett | _franck_: Do you believe that is a hardware fault or a fault in the GDB logic? | 20:40 |
_franck_ | I would say it's hardware because (I never did it) I think gdb has worked in some hardware configuration | 20:40 |
@jeremybennett | I don't recall seeing this behaviour before, but it is quite specific, so I may not have tested it. | 20:42 |
@jeremybennett | I remember the logic for breakpoints on branches or in branch delay slots is quite tricky. | 20:42 |
_franck_ | static int or1k_resume_or_step(struct target *target, int current, uint32_t address, int handle_breakpoints, int debug_execution, int step) | 20:49 |
_franck_ | current ? continue on current pc : continue at <address> | 20:49 |
_franck_ | <address> is coming from gdb... | 20:49 |
_franck_ | need to put some printfs :) | 20:49 |
_franck_ | stupid me, I didn't apply LoneTech's patch on the RTL I'm using ! (However, it souldn't change anything) | 20:52 |
LoneTech | sorry, fixing one bug does not imply the absence of others | 21:43 |
@jeremybennett | LoneTech: Damn - I knew there was something I had been missing all these years :) | 21:59 |
Generated by irclog2html.py 2.15.2 by Marius Gedminas - find it at mg.pov.lt!