sb0 changed the topic of #m-labs to: https://m-labs.hk :: Mattermost https://chat.m-labs.hk :: Logs http://irclog.whitequark.org/m-labs
lkcl has joined #m-labs
lkcl has quit [Ping timeout: 240 seconds]
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to vendor.altera [+1/-0/±2] https://git.io/JeWFf
<_whitenotifier> [m-labs/nmigen] whitequark 8538da3 - vendor.intel: add Quartus support. (WIP)
<_whitenotifier> [nmigen] whitequark synchronize pull request #178: vendor.intel: add Quartus support (WIP) - https://git.io/fjbbr
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to 7dfd7fb - https://codecov.io/gh/m-labs/nmigen/compare/7dfd7fb12a722398358869ebe5fdbd6788c1c0c6...ace10a6b29fa1e778d4bcfe14fb8c6981cdc8184
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to vendor.altera [+1/-0/±2] https://git.io/JeWFU
<_whitenotifier> [m-labs/nmigen] whitequark 53d86f5 - vendor.intel: add Quartus support.
<_whitenotifier> [nmigen] whitequark synchronize pull request #178: vendor.intel: add Quartus support (WIP) - https://git.io/fjbbr
<_whitenotifier> [nmigen] whitequark closed pull request #178: vendor.intel: add Quartus support - https://git.io/fjbbr
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to master [+1/-0/±2] https://git.io/JeWFT
<_whitenotifier> [m-labs/nmigen] whitequark 8021e2d - vendor.intel: add Quartus support.
<_whitenotifier> [nmigen] whitequark deleted branch vendor.altera - https://git.io/fhUU5
<_whitenotifier> [m-labs/nmigen] whitequark deleted branch vendor.altera
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/595885511?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/595885511?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Error. The Travis CI build could not complete due to an error - https://travis-ci.org/m-labs/nmigen/builds/595885760?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Error. The Travis CI build could not complete due to an error - https://travis-ci.org/m-labs/nmigen/builds/595885760?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Error. The Travis CI build could not complete due to an error - https://travis-ci.org/m-labs/nmigen/builds/595885764?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Error. The Travis CI build could not complete due to an error - https://travis-ci.org/m-labs/nmigen/builds/595885505?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Error. The Travis CI build could not complete due to an error - https://travis-ci.org/m-labs/nmigen/builds/595885764?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/595885776?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. 82.52% (+0.18%) compared to 7dfd7fb - https://codecov.io/gh/m-labs/nmigen/commit/8021e2dd76a6a98e54434333aa2e3d0e17f30d1f
<_whitenotifier> [nmigen] Failure. 20% of diff hit (target 82.33%) - https://codecov.io/gh/m-labs/nmigen/commit/8021e2dd76a6a98e54434333aa2e3d0e17f30d1f
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to 7dfd7fb - https://codecov.io/gh/m-labs/nmigen/commit/8021e2dd76a6a98e54434333aa2e3d0e17f30d1f
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/595885776?utm_source=github_status&utm_medium=notification
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±1] https://git.io/JeWFO
<_whitenotifier> [m-labs/nmigen] whitequark 730eff5 - README: add device support matrix.
<_whitenotifier> [nmigen] whitequark commented on pull request #40: WIP: Expand and document lib.cdc - https://git.io/JeWFZ
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/595888921?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to 8021e2d - https://codecov.io/gh/m-labs/nmigen/commit/730eff5d574674e7d29f9a7848635801ff78a481
<_whitenotifier> [nmigen] Success. Coverage not affected when comparing 8021e2d...730eff5 - https://codecov.io/gh/m-labs/nmigen/commit/730eff5d574674e7d29f9a7848635801ff78a481
harryho has joined #m-labs
electronic_eel has quit [Ping timeout: 240 seconds]
electronic_eel has joined #m-labs
electronic_eel has quit [Ping timeout: 265 seconds]
electronic_eel has joined #m-labs
sb0 has joined #m-labs
<sb0> whitequark: what about those USB "isochronous" endpoints, are they any better latency-wise?
cr1901_modern has quit [Ping timeout: 268 seconds]
cr1901_modern has joined #m-labs
cr1901_modern has quit [Read error: Connection reset by peer]
cr1901_modern has joined #m-labs
<mtrbot-ml_> [mattermost] <sb10q> https://github.com/lambdaconcept/lambdaUSB
<cr1901_modern> Can it do full speed (more than sufficient for my potential use case and doesn't require me to interface to a pesky PHY)?
<mtrbot-ml_> [mattermost] <sb10q> ULPI doesn't have "special features" afaik
<mtrbot-ml_> [mattermost] <sb10q> There's a soft full-speed phy in milkymist soc if you need one
<Dar1us> sb0: BTW USB transfer latency is impacted by how the USB host controller is programmed (based on an email from the FreeBSD USB stack maintainer)
<key2> i could add fs by rewriting the phy
<cr1901_modern> key2: I have to leave for a bit, but since you're here... I know ULPI PHYs handle FS, but the idea was I wanted to opt out of HS support and just have FS, so the PHY can be implemented purely digitally on 3.3V I/O pins.
<cr1901_modern> (Well until azonenberg adapts his Ethernet PHYs using resistors to USB.
<key2> yea it’s possible
<key2> and thought this way
cr1901_modern has quit [Read error: Connection reset by peer]
cr1901_modern has joined #m-labs
harryho has quit [Remote host closed the connection]
_whitelogger has joined #m-labs
<whitequark> sb0: isochronous endpoints drop packets on overflow
<whitequark> that'sthe whole point
<whitequark> so that is not any better than dropping packets inside the FPGA
<_whitenotifier> [nmigen-boards] ZirconiumX opened pull request #32: Add Terasic DE10-Nano - https://git.io/JeWjQ
<_whitenotifier> [m-labs/nmigen-boards] whitequark pushed 1 commit to master [+1/-0/±0] https://git.io/JeWjd
<_whitenotifier> [m-labs/nmigen-boards] ZirconiumX 4e8d894 - Add Terasic DE10-Nano (#32)
<_whitenotifier> [nmigen-boards] whitequark closed pull request #32: Add Terasic DE10-Nano - https://git.io/JeWjQ
<Dar1us> the amount of bandwidth reserved for isochronous is pretty low too
<ZirconiumX> wq: So, programming the DE10-Nano is painful due to the stupidity of the Quartus programmer
<whitequark> oh yeah, also that
<whitequark> iirc you get about half of HS bandwidth for ISO?
<whitequark> at most
<ZirconiumX> The SoC presents itself as a JTAG chain of the ARM core and the FPGA
<ZirconiumX> If the ARM core comes first in the chain, the programmer will fail
<ZirconiumX> Despite the autodetect code *finding the chip*
<whitequark> i think you need to change parameters to quartus_pgm
<whitequark> "--operation", "P;" + bitstream_filename + "@2"
<whitequark> or something lik that
<whitequark> oh yeah you also have to skip the device at the start
<ZirconiumX> Sure, except that the chain is not deterministic
<whitequark> oh?
<whitequark> why?
<ZirconiumX> Some board/chip revisions put the ARM first, some put the FPGA first
<whitequark> wtf
<ZirconiumX> wq: Holy shit it works
<whitequark> wait really? with no bypass?
<whitequark> just the @2?
<ZirconiumX> Yep
<ZirconiumX> I don't know if it works for all boards, but it seems to work for this one
<whitequark> so should this be added to de10_nano board?
<ZirconiumX> Yes please
<_whitenotifier> [m-labs/nmigen-boards] whitequark pushed 1 commit to master [+0/-0/±1] https://git.io/JeleA
<_whitenotifier> [m-labs/nmigen-boards] whitequark 8a688c3 - de10_nano: fix programming.
<ZirconiumX> wq: So, the MiSTer I/O board goes over the top of the DE10-Nano, making things like switches impossible to reach. As a practical matter, do I still put them in resources?
<whitequark> do you want to add mister as a subclass of de10-nano or as a completely new board?
<ZirconiumX> I thought we agreed on subclass
<_whitenotifier> [m-labs/nmigen-boards] whitequark pushed 1 commit to master [+1/-0/±0] https://git.io/JelvI
<_whitenotifier> [m-labs/nmigen-boards] whitequark d07839a - de0cv→de0_cv, for consistency with de10_nano
<whitequark> then you would add more resources, never remove any, no?
<ZirconiumX> Hmm, okay
* ZirconiumX is installing EAGLE just to read the schematics of the I/O and SDRAM boards
<whitequark> do they not have PDFs?
<ZirconiumX> Nope
<ZirconiumX> I'm not even sure if it *is* EAGLE actually
<ZirconiumX> I'm going on the assumption it is
<whitequark> link?
<whitequark> don't think that's eagle
<whitequark> altium
<ZirconiumX> This is an obvious place to put them
* ZirconiumX grumbles about this not being mentioned in the page
<ZirconiumX> wq: https://puu.sh/EqVI0/3e57b4eeb3.png <--- remind me again which direction the earth is again
<ZirconiumX> :P
<whitequark> does it matter?
<ZirconiumX> Mmm, the schematic isn't great
<_whitenotifier> [m-labs/nmigen-boards] whitequark pushed 1 commit to master [+1/-1/±0] https://git.io/Jelvj
<_whitenotifier> [m-labs/nmigen-boards] whitequark dcf7c47 - de0cv→de0_cv, for consistency with de10_nano
<ZirconiumX> wq: I feel like converting from the GPIO connector pins back to the FPGA pin is going to be error prone
<whitequark> is it described in terms of FPGA connector pins?
<whitequark> i mean, you could just add MiSTer as a new board entirely
<whitequark> if thats easier
<whitequark> there's no real policy
<ZirconiumX> It's described in terms of GPIO connector pins
<ZirconiumX> So I have to look at what the FPGA pin connected to that GPIO pin is and enter that as the given pin location
<ZirconiumX> Unless I'm being an idiot
<ZirconiumX> What I'm looking for is, I dunno, self.connectors["j0"][4] or whatever
<whitequark> sure? you can use that?
<whitequark> instead of eg AA10 specify j_0:4
<ZirconiumX> But connectors is a list, right?
<whitequark> in the resource Pins()
<ZirconiumX> Ah, I see, thank you
<cr1901_modern> Re: adding as a new board, I think most accurately MiSTer is an add-on that is by design only usable with a single DE0-variant board?
<ZirconiumX> Yeah
<ZirconiumX> Bikeshedding: the I/O Board has three LEDs with intended purposes; one for board power, one for disk access, and another as general user access
<ZirconiumX> Is this special enough to give names? Would that be three Resources, or one Resource with three subsignals?
<whitequark> you can have both
<whitequark> Resource("led", 0)
<whitequark> Resource("led_disk", 0)
<ZirconiumX> Do resources with the same name get appended?
<whitequark> nope, they're just independent
<whitequark> you can only request one at a time
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±4] https://git.io/JelUL
<_whitenotifier> [m-labs/nmigen] whitequark a7cc88f - vendor: yosys is a required tool for all Verilog-based flows.
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596149552?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to 730eff5 - https://codecov.io/gh/m-labs/nmigen/commit/a7cc88f3d4dacfd871d5e0f3c53bde63d8508d4f
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596149552?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. Coverage not affected when comparing 730eff5...a7cc88f - https://codecov.io/gh/m-labs/nmigen/commit/a7cc88f3d4dacfd871d5e0f3c53bde63d8508d4f
mumptai has joined #m-labs
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to master [+1/-0/±1] https://git.io/JelTF
<_whitenotifier> [m-labs/nmigen] whitequark b630d75 - vendor.lattice_machxo2: new supported family.
<_whitenotifier> [m-labs/nmigen-boards] whitequark pushed 1 commit to master [+2/-0/±0] https://git.io/JelTb
<_whitenotifier> [m-labs/nmigen-boards] whitequark e567899 - Add TinyFPGA AX{1,2} boards.
<_whitenotifier> [nmigen] Error. The Travis CI build could not complete due to an error - https://travis-ci.org/m-labs/nmigen/builds/596177581?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596177581?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to a7cc88f - https://codecov.io/gh/m-labs/nmigen/commit/b630d75a40156be2f893dc68b99ac634325f7c37
<_whitenotifier> [nmigen] Success. Coverage not affected when comparing a7cc88f...b630d75 - https://codecov.io/gh/m-labs/nmigen/commit/b630d75a40156be2f893dc68b99ac634325f7c37
sb0 has quit [Quit: Leaving]
<_whitenotifier> [m-labs/nmigen] whitequark pushed 1 commit to master [+1/-0/±2] https://git.io/JelkA
<_whitenotifier> [m-labs/nmigen] whitequark ef74159 - vendor.lattice_machxo2: new supported family.
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596197212?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. 82.52% (+0.18%) compared to a7cc88f - https://codecov.io/gh/m-labs/nmigen/commit/ef741594b51d2ce2dd0d90d0acf8346b10384947
<_whitenotifier> [nmigen] Success. Coverage not affected when comparing a7cc88f...ef74159 - https://codecov.io/gh/m-labs/nmigen/commit/ef741594b51d2ce2dd0d90d0acf8346b10384947
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to a7cc88f - https://codecov.io/gh/m-labs/nmigen/commit/ef741594b51d2ce2dd0d90d0acf8346b10384947
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596197212?utm_source=github_status&utm_medium=notification
<_whitenotifier> [m-labs/nmigen] whitequark pushed 2 commits to master [+1/-0/±2] https://git.io/JelIl
<_whitenotifier> [m-labs/nmigen] whitequark bfd4538 - xilinx_7series: add grade platform property.
<_whitenotifier> [m-labs/nmigen] whitequark 9fe27a1 - vendor.xilinx_ultrascale: new supported family.
<_whitenotifier> [m-labs/nmigen-boards] whitequark pushed 1 commit to master [+1/-0/±0] https://git.io/JelIB
<_whitenotifier> [m-labs/nmigen-boards] whitequark 01638af - Add KCU105 board stub.
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596202639?utm_source=github_status&utm_medium=notification
<_whitenotifier> [nmigen] Success. 82.52% (+0.18%) compared to ef74159 - https://codecov.io/gh/m-labs/nmigen/commit/9fe27a15ad0fe31a7d2a8d4f876780c13a434b56
<_whitenotifier> [nmigen] Success. Coverage not affected when comparing ef74159...9fe27a1 - https://codecov.io/gh/m-labs/nmigen/commit/9fe27a15ad0fe31a7d2a8d4f876780c13a434b56
<_whitenotifier> [nmigen] Success. 82.33% remains the same compared to ef74159 - https://codecov.io/gh/m-labs/nmigen/commit/9fe27a15ad0fe31a7d2a8d4f876780c13a434b56
<_whitenotifier> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/596202639?utm_source=github_status&utm_medium=notification
acathla has quit [Remote host closed the connection]
gnufan_home has joined #m-labs
acathla has joined #m-labs
<ZirconiumX> wq: Can you pass conn=("foo", 1) to SDRAMResource/SDCardResources?
<whitequark> no
<whitequark> you have to use foo_1:pin currently
<ZirconiumX> This is going to make SDRAM fun
<whitequark> well, we can add that
_florent_ has left #m-labs [#m-labs]
<ZirconiumX> whitequark: So, SDRAMResource requires DQM to be specified, but the MiSTer's SDRAM board doesn't actually contain those pins
<ZirconiumX> What should I do?
<whitequark> hm
<whitequark> link to schematic pdf?
<ZirconiumX> wq: It also seems like inheriting from a board definition like this is pretty painful
<whitequark> sure, you can make it from scratch then
<whitequark> not really an issue
mumptai has quit [Quit: Verlassend]
<ZirconiumX> wq: for now I have dqm="", but I'm not sure if that's really a valid solution
<whitequark> what SDRAM is that?
<ZirconiumX> It varies from board to board
<whitequark> any of them?
<ZirconiumX> Alliance AS4C32M16SB
<ZirconiumX> That's one chip used
<whitequark> ZirconiumX: neat
<whitequark> the alliance chip has dqm
<whitequark> i guess they just didn't connect it on mister
<whitequark> leave it as ="" for now
<ZirconiumX> Or hold it low
<whitequark> yes, that's what i mean
<_whitenotifier> [nmigen-boards] ZirconiumX opened pull request #33: Add DE10-Nano-based MiSTer - https://git.io/JelOb
<whitequark> ZirconiumX: didn't you say that you don't want to use the gpio* mapping?
<ZirconiumX> I did, but I wanted feedback on it in general
<ZirconiumX> while it's maybe a bit painful stylistically, it's functionally identical
<ZirconiumX> https://puu.sh/Er6fM/1197b915f1.png <--- Twitter's having fun :P
<whitequark> the LEDs should be separate resources
<whitequark> other than that looks fine
<whitequark> feel free to fix stylistic issues, or not
<Dar1us> ZirconiumX: nice work BTW
<cr1901_modern> Potential non verilog arcade ports in the future
<ZirconiumX> For that I'll wait for you to add conn as a kwarg for SDRAMResource (at least)
<ZirconiumX> cr1901_modern: We can dream :P
<whitequark> no I mean, didn't you want to avoid using indirection through the connector at all?
<ZirconiumX> No?
<whitequark> ah
<ZirconiumX> I actually prefer using the connector
<whitequark> ok
<whitequark> I misunderstood
<whitequark> feel free to add conn to SDRAMResource, SDCardResource, etc in a separate PR
<ZirconiumX> wq: I'm partly considering deprecating/removing DE10NanoPlatform for MisterPlatform, because you can use a subset of MisterPlatform for the same functionality as DE10NanoPlatform
<whitequark> uhh, I don't like that at all
<ZirconiumX> <whitequark> the LEDs should be separate resources <--- but the switches are OK?
<whitequark> switches too
<whitequark> they're all separate resources in nmigen
<_whitenotifier> [nmigen-boards] ZirconiumX synchronize pull request #33: Add DE10-Nano-based MiSTer - https://git.io/JelOb
gnufan_home has quit [Quit: Leaving.]
harryho has joined #m-labs