sb0 changed the topic of #m-labs to: ARTIQ, Migen, MiSoC, Mixxeo & other M-Labs projects :: fka #milkymist :: Logs http://irclog.whitequark.org/m-labs
balrog has joined #m-labs
<mithro> sb0: Do you have any existing code to make the lm32 call the exception_handler code in libbase on things like bus errors? It seems the or1k is configured to do that but not the lm32
<GitHub72> [artiq] jonaskeller opened issue #805: kernel invariants: seconds to mu conversion not for free on kernel https://github.com/m-labs/artiq/issues/805
kmehall has quit [Ping timeout: 276 seconds]
kmehall has joined #m-labs
<cr1901_modern> mithro: Does lm32 even use the WB error signals?
raghu has quit [Quit: Page closed]
froggytoad has joined #m-labs
<mithro> The one I just spent forever tracking down was a divide by zero
<cr1901_modern> Oh, that's not a bus error I don't think? In any case, I think the exception handlers are declared as "weak" symbols, so you can override them in your code for custom handling
* cr1901_modern will check in a bit once his todo stack is done for the night ._. ...
froggytoad has quit [Ping timeout: 268 seconds]
<sb0> mithro, I don't
_whitelogger has joined #m-labs
rohitksingh_work has joined #m-labs
froggytoad has joined #m-labs
<GitHub121> [migen] sbourdeauducq pushed 1 new commit to master: https://git.io/v7G48
<GitHub121> migen/master f357077 Sebastien Bourdeauducq: kc705: add pullup on SD card MISO
<bb-m-labs> build #165 of migen is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/migen/builds/165
<mithro> sb0: is someone working on SD cards?
<GitHub52> [artiq] mntng opened pull request #806: add test_spi.py (master...master) https://github.com/m-labs/artiq/pull/806
<_florent_> mithro: key2 has been working on a core: https://github.com/lambdaconcept/litesdcard
rohitksingh_work has quit [Quit: Leaving.]
<_florent_> mithro: I think mlabs is using sdcard just as a way to validate spi in artiq
<GitHub179> [artiq] jordens commented on issue #806: Please add proper commit messages stating what was done and if possible why it was done and how it was done. The ones auto-generated by github are insufficient. https://github.com/m-labs/artiq/pull/806#issuecomment-318310116
<GitHub139> [artiq] jordens commented on issue #806: Please add proper commit messages stating what was done and if possible why it was done and how it was done. The ones auto-generated by github are insufficient. https://github.com/m-labs/artiq/pull/806#issuecomment-318310116
<Ishan_Bansal> I am getting the following error on running my code:
<Ishan_Bansal> and I am not able to get the reason for this error. Can somebody help me out in this.
<Ishan_Bansal> the code which I run is the following : https://github.com/ishan98/litejpeg/blob/huffman/test/new_huffman_tb.py
<Ishan_Bansal> and the code of the huffmancore is as follows : https://github.com/ishan98/litejpeg/blob/huffman/litejpeg/core/huffman/huffmancore.py
<Ishan_Bansal> Atleast the reason why the error comes is also a big help.
<Ishan_Bansal> attie : ping
<Ishan_Bansal> sb0 : ping
<Ishan_Bansal> mithro : ping
<mithro> Ishan_Bansal: the error is explaining exactly what if wrong
<_florent_> Ishan_Bansal: you can add some prints in structure.py if you want to know part of your code is causing that
<mithro> Ishan_Bansal: you have something that isn't a Migen statement in you comb/sync
<GitHub180> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: One too many blank lines. https://github.com/m-labs/artiq/pull/806#discussion_r129796708
<GitHub53> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: ``ns`` is also not used. https://github.com/m-labs/artiq/pull/806#discussion_r129796933
<GitHub55> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: Remove that blank line. https://github.com/m-labs/artiq/pull/806#discussion_r129796658
<GitHub32> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: ``portable`` is not used. https://github.com/m-labs/artiq/pull/806#discussion_r129797002
<GitHub95> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: ``os`` is not used, don't import it. https://github.com/m-labs/artiq/pull/806#discussion_r129796882
<GitHub97> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: This change (and others) is already merged. Please don't submit them again (try ``git pull``). https://github.com/m-labs/artiq/pull/806#discussion_r129797841
<GitHub110> [artiq] sbourdeauducq commented on pull request #806 c1ecac2: ``CARD`` → ``Card`` https://github.com/m-labs/artiq/pull/806#discussion_r129797129
<Ishan_Bansal> mithro : But those errors comes with the representation of which line is not migen, isn't this error different from them ?
<mithro> As Florent mentioned you can put a print statement where that error is raised to see more info
<mithro> Ishan_Bansal: does that make sense?
<GitHub181> [sinara] gkasprow pushed 1 new commit to master: https://github.com/m-labs/sinara/commit/a0b2533ae84444db101a9a538657379855c05695
<GitHub181> sinara/master a0b2533 Greg: fixed DRC bugs and some issues
<Ishan_Bansal> mithro : I will try that.
<sb0> Ishan_Bansal, you can also modify the comb+= and sync+= handling code so that it checks immediately that you are giving it a migen statement
<sb0> that would pinpoint the original error
<cr1901_modern> _florent_: I took the briefest of skims, but why does an SD card need IDDR/ODDR?
<_florent_> cr1901_modern: for high speeds
<cr1901_modern> IDDR/ODDR can be single ended?
<Ishan_Bansal> mithro : On making the print in the structure, I am getting the following statement.
<Ishan_Bansal> (<litex.gen.fhdl.structure._Assign object at 0x7f1fee798b00>, <litex.gen.fhdl.structure._Assign object at 0x7f1fee7a5940>, None, None, <litex.gen.fhdl.structure.If object at 0x7f1fee7a50f0>, <litex.gen.fhdl.structure.If object at 0x7f1fee7aa6a0>, <litex.gen.fhdl.structure.If object at 0x7f1fee7aaac8>, <litex.gen.fhdl.structure.If object at 0x7f1fee7af080>, <litex.gen.fhdl.structure.If object at 0x7f1fee7af2b0>)
<Ishan_Bansal> but this gives no clue where the error is.
<mithro> The none objects would be my guess
<mithro> Ishan_Bansal: my guess is you are calling functions which are returning none
<Ishan_Bansal> mithro, yes there are two such functions, but are they a problem.
<Ishan_Bansal> dc_rom,ac_rom
<mithro> Hrm?
<Ishan_Bansal> This is the link.
<Ishan_Bansal> I am using this to store and get the values of the huffman tables.
<mithro> Ishan_Bansal: that is a function and not a Migen module
<mithro> Ishan_Bansal: same error as you had last time
<Ishan_Bansal> but mithro, when I tested the rom by using new_rom_tb.py, this works properly.
<Ishan_Bansal> No such error is their at that time.
<mithro> Your new_rom_tb is wrong then
<mithro> I don't see any such file in your repo either
<mithro> Ishan_Bansal: the reason your test works is because you are calling the function in a very different way
<mithro> Details matter
<mithro> Ishan_Bansal: compare how you are using the dc_rom function in the test bench rather than in your huffmancore.py
<Ishan_Bansal> mithro : Ok, is it because in the huffmancore I am using the dc_rom inside the self.sync ?
<Ishan_Bansal> I think I will get the idea.
<mithro> But more generally, it is very rare to need functions - you should normal be using modules
<attie> Ishan_Bansal: I think it would really help if you acquired a better foundation in object oriented python, quite independent of migen
<attie> you make a lot of mistakes that seem to be rooted in not understanding how objects work.
<attie> having a function that is not a class method but whose first argument is "self" should be ringing so many alarm bells in your head that you wouldn't even finish the line.
<Ishan_Bansal> attie : I am working on that.
sb0 has quit [Remote host closed the connection]
<GitHub19> [smoltcp] whitequark pushed 4 new commits to master: https://git.io/v7Zfy
<GitHub19> smoltcp/master 10585ad whitequark: Rearrange errors and clarify their semantics.
<GitHub19> smoltcp/master 50ae82d whitequark: Update a stale docstring.
<GitHub19> smoltcp/master d422f45 whitequark: as_unspecified → to_unspecified
<travis-ci> m-labs/smoltcp#158 (master - 53a3875 : whitequark): The build passed.
rohitksingh has joined #m-labs
<GitHub8> [smoltcp] whitequark pushed 2 new commits to master: https://git.io/v7Zcl
<GitHub8> smoltcp/master 5bf6458 whitequark: Get rid of Result<_, ()>....
<GitHub8> smoltcp/master 1a11e4d whitequark: Remove useless redundant tests.
<travis-ci> m-labs/smoltcp#159 (master - 5bf6458 : whitequark): The build passed.
sb0 has joined #m-labs
<GitHub96> [artiq] sbourdeauducq commented on pull request #806 c053fbf: Now you have some spurious spaces at the end of your file... (and e.g. one line 56) https://github.com/m-labs/artiq/pull/806#discussion_r129867712
<GitHub162> [artiq] sbourdeauducq commented on pull request #806 c053fbf: Now you have some spurious spaces at the end of your file... (and e.g. one line 56) https://github.com/m-labs/artiq/pull/806#discussion_r129867712
<GitHub75> [artiq] mntng commented on pull request #806 c053fbf: ............??? https://github.com/m-labs/artiq/pull/806#discussion_r129871820
<GitHub158> [artiq] sbourdeauducq commented on pull request #806 c053fbf: Your file contains space characters (" ") that should not be there. https://github.com/m-labs/artiq/pull/806#discussion_r129872209
sb0 has quit [Quit: Leaving]
sb0 has joined #m-labs
froggytoad has quit [Ping timeout: 276 seconds]
knielsen has left #m-labs [#m-labs]
ohsix has quit [Ping timeout: 260 seconds]
rohitksingh has quit [Read error: Connection reset by peer]
rohitksingh has joined #m-labs
rohitksingh has quit [Quit: Leaving.]
<cr1901_modern> sb0: Have you seen/tried this (RISC-V 4-5 stage pipelined softcore in Verilog)? https://github.com/VectorBlox/orca
ohsix has joined #m-labs
froggytoad has joined #m-labs
shenki has quit [Ping timeout: 468 seconds]
Gurty has quit [Excess Flood]
zumbi has joined #m-labs
froggyto1d has joined #m-labs
kyak_ has joined #m-labs
kyak_ has joined #m-labs
Gurty has joined #m-labs
ohsix has quit [Excess Flood]
ohsix_ has joined #m-labs
whitequa1k has joined #m-labs
shenki has joined #m-labs
siruf has quit [Write error: Broken pipe]
zumbi_ has quit [Remote host closed the connection]
gric_ has quit [Remote host closed the connection]
whitequark has quit [Remote host closed the connection]
froggytoad has quit [Write error: Broken pipe]
kyak has quit [Remote host closed the connection]
froggyto1d has quit [Remote host closed the connection]
siruf has joined #m-labs
ohsix_ has quit [Quit: Reconnecting]
ohsix has joined #m-labs