--- Log opened Tue Apr 25 00:00:46 2017 | ||
-!- heroux_ is now known as heroux | 03:41 | |
wallento | is anyone emotional about that one? https://github.com/openrisc/mor1kx/blob/master/rtl/verilog/mor1kx_bus_if_wb32.v#L57 | 10:24 |
---|---|---|
wallento | I don't see too much value in it as it just prints a parameter | 10:25 |
shorne_ | wallento: it looks like it was there since first commit | 10:35 |
shorne_ | I never had noticed it before | 10:35 |
wallento | probably for debugging | 10:35 |
shorne_ | yeah | 10:35 |
wallento | not? | 10:35 |
wallento | if I simulate with multiple cores it becomes quite annoying ;) | 10:35 |
wallento | I somehow even remember we removed it, but apparently not | 10:36 |
shorne_ | I think its not needed | 10:36 |
wallento | I will do a PR | 10:36 |
wallento | there is also my PR pending which kills verilator warnings | 10:38 |
mor1kx | [mor1kx] wallento force-pushed fixes from 32fa112 to 6234e3e: https://github.com/openrisc/mor1kx/commits/fixes | 10:41 |
mor1kx | mor1kx/fixes 48df64d Stefan Wallentowitz: Style: case variable width... | 10:41 |
mor1kx | mor1kx/fixes 3e1ba20 Stefan Wallentowitz: Style: Width fix | 10:41 |
mor1kx | mor1kx/fixes 146fe2a Stefan Wallentowitz: Style: Shift in wide register and result range... | 10:41 |
mor1kx | [mor1kx] wallento created remove-parameter-display (+1 new commit): https://github.com/openrisc/mor1kx/commit/cef715a7dde8 | 10:43 |
mor1kx | mor1kx/remove-parameter-display cef715a Stefan Wallentowitz: Remove debug parameter output... | 10:43 |
mor1kx | [mor1kx] wallento opened pull request #52: Remove debug parameter output (master...remove-parameter-display) https://github.com/openrisc/mor1kx/pull/52 | 10:43 |
shorne_ | oh, you have a few more PR's too | 10:53 |
shorne_ | I am happy if we can fix all those linter issues | 10:55 |
shorne_ | need to go to bed, hope olof can review those | 10:55 |
mor1kx | [mor1kx] wallento pushed 1 new commit to master: https://github.com/openrisc/mor1kx/commit/c562996f97926ee0d51f20a8f42a7481f834a425 | 15:07 |
mor1kx | mor1kx/master c562996 Stefan Wallentowitz: Merge pull request #52 from openrisc/remove-parameter-display... | 15:07 |
mor1kx | [mor1kx] wallento closed pull request #52: Remove debug parameter output (master...remove-parameter-display) https://github.com/openrisc/mor1kx/pull/52 | 15:07 |
mithro | shorne: I have not used it | 22:34 |
mithro | shorne: Probably stared it because someone else stared it :-P | 22:36 |
--- Log closed Wed Apr 26 00:00:48 2017 |
Generated by irclog2html.py 2.15.2 by Marius Gedminas - find it at mg.pov.lt!