| *** tpb has joined #litex | 00:00 | |
| *** jaseg has quit IRC | 02:30 | |
| *** jaseg has joined #litex | 02:32 | |
| *** Degi has quit IRC | 03:16 | |
| *** Degi has joined #litex | 03:16 | |
| *** MyGreenBalloon has joined #litex | 09:22 | |
| *** MyGreenBalloon has quit IRC | 09:30 | |
| *** awe00 has joined #litex | 12:53 | |
| *** tumbleweed has quit IRC | 12:54 | |
| *** tumbleweed has joined #litex | 12:54 | |
| *** tumbleweed has joined #litex | 12:54 | |
| *** feldim2425_ has joined #litex | 13:57 | |
| *** bubble_buster_ has joined #litex | 13:57 | |
| *** y2kbugger_ has joined #litex | 13:57 | |
| *** flammit_ has joined #litex | 13:57 | |
| *** tucanae47__ has joined #litex | 13:57 | |
| *** gruetze_ has joined #litex | 13:58 | |
| *** bubble_buster has quit IRC | 14:00 | |
| *** flammit has quit IRC | 14:00 | |
| *** tucanae47 has quit IRC | 14:00 | |
| *** feldim2425 has quit IRC | 14:00 | |
| *** midnight has quit IRC | 14:00 | |
| *** CarlFK[m] has quit IRC | 14:00 | |
| *** lambda has quit IRC | 14:00 | |
| *** y2kbugger has quit IRC | 14:00 | |
| *** gruetzkopf has quit IRC | 14:00 | |
| *** bubble_buster_ is now known as bubble_buster | 14:00 | |
| *** tucanae47__ is now known as tucanae47 | 14:00 | |
| *** flammit_ is now known as flammit | 14:00 | |
| *** y2kbugger_ is now known as y2kbugger | 14:00 | |
| *** Guest43925 has joined #litex | 14:01 | |
| *** the_JinX has quit IRC | 14:06 | |
| *** midnight has joined #litex | 14:07 | |
| *** CarlFK[m] has joined #litex | 14:07 | |
| *** lambda has joined #litex | 14:10 | |
| *** gruetze_ is now known as gruetzkopf | 14:37 | |
| shorne | somlo: FYI, this is the patch to fix LiteETH for big endian machines like openrisc: https://github.com/stffrdhrn/linux/commit/0223cdd5b77a4f05b2824c1bf6493c04ee0ad76e | 14:40 |
|---|---|---|
| tpb | Title: Litex: LiteETH: Fixups for compat with BE, also properly detect unset… · stffrdhrn/linux@0223cdd · GitHub (at github.com) | 14:40 |
| shorne | I am not sure if you saw the LKML mails/messages. I am planning to upstream via my openrisc tree a few of the litex drivers in the 5.10 merge window. | 14:41 |
| shorne | those are here: https://github.com/stffrdhrn/linux/commits/or1k-5.10-litex | 14:42 |
| tpb | Title: Commits · stffrdhrn/linux · GitHub (at github.com) | 14:42 |
| shorne | Now, that liteETH is working I can do some work. | 14:43 |
| somlo | shorne: afaik, the stuff being pushed on lkml is 32-bit cpu and 8-bit csr-data-width specific, and I promised Mateusz to shut up and let it happen, and start making noise about 64-bit CPUs and 32-bit csr-data-width only after the initial upstreaming pass :) | 15:29 |
| somlo | shorne: I'll test your patch with Rocket (64-bit LE); if it doesn't break that, I'll post it to the litex-rocket-rebase branch (which in spite of its name *should* be backward compatible to 32-bit CPUs and whatever csr data width litex is built with) | 15:32 |
| somlo | shorne: so the rocket branch I maintain already has the accessors (ioread/iowrite) updated to work across widths and endianness | 15:45 |
| somlo | so all that's different in your version is the `readx_poll_timeout_atomic` thing, and some of the logic around `of_get_mac_address()` | 15:46 |
| somlo | I'll apply those changes manually (as your patch is not itself based on litex-*-rebase in litex-hub, as far as I can tell) | 15:47 |
| _florent_ | Hi shorne, nice to see you here and great to see you are testing/helping with the linux drivers. | 15:49 |
| _florent_ | If you think the dts generation is missing some features (like the IRQ you reported), feel free to create issue on LiteX repo for that, i'll try to implement it | 15:49 |
| _florent_ | Otherwise, as you are probably already aware, Mateusz and somlo are working on the drivers, but if have issues with the FPGA or trouble understanding how it works, feel free to ping me, i would be happy to help and do some tests | 15:52 |
| somlo | shorne: https://imgur.com/a/HEGz3CT -- is `inreg8` an or1k-ism ? Any way we can make this line portable across everything? https://github.com/stffrdhrn/linux/commit/0223cdd5b77a4f05b2824c1bf6493c04ee0ad76e#diff-9ec47ad4f74cb589e1024ed04c60670dR245 | 15:57 |
| tpb | Title: Imgur: The magic of the Internet (at imgur.com) | 15:58 |
| somlo | shorne: the of_get_mac_address() thing works perfectly, and fixes an annoyance I had commented out and was hoping to return to one day, but never did -- so thanks for that! :) | 16:06 |
| somlo | oh no, inreg8 is the silly liteeth-driver-specific accessor wrapper I thought I'd gotten rid of :) | 16:12 |
| somlo | it's `litex_reg_readb` in the litex-rocket-rebase branch, and seems to work fine when I test it on rocket | 16:16 |
| somlo | shorne: https://github.com/litex-hub/linux/commit/35b0a4c900cbd3c0176306043b54c82539942d57 -- thanks! | 16:42 |
| tpb | Title: Litex: LiteETH: Fixups for compat with BE, also properly detect unset… · litex-hub/linux@35b0a4c · GitHub (at github.com) | 16:42 |
| *** TMM has quit IRC | 17:42 | |
| *** TMM has joined #litex | 17:43 | |
| *** tcal has joined #litex | 17:49 | |
| somlo | _florent_: https://github.com/enjoy-digital/litex/commit/e28e808c244ce1d57146c92a270ce4d08c9967cc#diff-051c0188ba9860fa5fa5242889764189R34 | 18:16 |
| tpb | Title: software/liblitedram: allow forced write leveling delays, improve del… · enjoy-digital/litex@e28e808 · GitHub (at github.com) | 18:16 |
| somlo | it results in a build error (probably the `int sdrwl_delays[16];` array should be in sdram.c, not sdram.h (from where it gets included in multiple places, and ends up conflicting with itself) | 18:17 |
| somlo | and ouch, the entire memspeed thing once again violently assumes that a pointer is "unsigned int" :( | 18:29 |
| somlo | _florent_: https://github.com/enjoy-digital/litex/pull/649 | 18:48 |
| tpb | Title: misc. fixes for a few recent commits by gsomlo · Pull Request #649 · enjoy-digital/litex · GitHub (at github.com) | 18:48 |
| _florent_ | somlo: thanks for the PR | 19:15 |
| somlo | Thanks for applying! (I'm the 64-bit CI testing bot, but I catch the occasional unrelated compile warning or error ;) | 19:27 |
| somlo | _florent_: the memspeed/memtest thing turned out to be a lot less of a problem than I initially thought (there's lots of `unsigned int *` that's fine, only a few instances of `(unsigned int)addr` that needed to be fixed | 19:37 |
| somlo | so I ended up more freaked out about it than I needed to be :D | 19:37 |
| *** awe001 has joined #litex | 19:47 | |
| *** awe00 has quit IRC | 19:49 | |
| *** acathla has quit IRC | 19:56 | |
| *** acathla has joined #litex | 20:16 | |
| *** acathla has quit IRC | 20:16 | |
| *** acathla has joined #litex | 20:16 | |
| shorne | somlo: inreg8 is/was a helper in the liteeth8 driver. Let me try with your latest driver. The problem with using readb_.. I think is that readb is not for iomem, so for me the read returned nothing | 20:27 |
| shorne | _florent_: thanks, I will let you know, I have been able to get by reading the code on my own so far. I'll report the missing interrupts | 20:29 |
| somlo | shorne: I remembered getting rid of local-to-the-driver accessors such as inreg8 :) I think the "adapted" version of your patch in litex-rocket-rebase should work (the accessors there should be endianness and csr-data-width agnostic) | 21:15 |
| shorne | somlo: yeah, I figured they had to be removed, but I couldn't find the branch that had any newer liteeth driver. Now I have your's thanks | 22:19 |
| *** awe001 has quit IRC | 22:33 | |
| *** lf has quit IRC | 23:31 | |
| *** lf has joined #litex | 23:32 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!