{"action":"created","issue":{"url":"https://api.github.com/repos/optimsoc/optimsoc/issues/99","repository_url":"https://api.github.com/repos/optimsoc/optimsoc","labels_url":"https://api.github.com/repos/optimsoc/optimsoc/issues/99/labels{/name}","comments_url":"https://api.github.com/repos/optimsoc/optimsoc/issues/99/comments","events_url":"https://api.github.com/repos/optimsoc/optimsoc/issues/99/events","html_url":"https://github.com/optimsoc/optimsoc/pull/99","id":288319929,"number":99,"title":"Systems: Configurable system with compute tiles only","user":{"login":"wallento","id":299017,"avatar_url":"https://avatars3.githubusercontent.com/u/299017?v=4","gravatar_id":"","url":"https://api.github.com/users/wallento","html_url":"https://github.com/wallento","followers_url":"https://api.github.com/users/wallento/followers","following_url":"https://api.github.com/users/wallento/following{/other_user}","gists_url":"https://api.github.com/users/wallento/gists{/gist_id}","starred_url":"https://api.github.com/users/wallento/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wallento/subscriptions","organizations_url":"https://api.github.com/users/wallento/orgs","repos_url":"https://api.github.com/users/wallento/repos","events_url":"https://api.github.com/users/wallento/events{/privacy}","received_events_url":"https://api.github.com/users/wallento/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-01-13T10:49:12Z","updated_at":"2018-03-21T07:35:42Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/optimsoc/optimsoc/pulls/99","html_url":"https://github.com/optimsoc/optimsoc/pull/99","diff_url":"https://github.com/optimsoc/optimsoc/pull/99.diff","patch_url":"https://github.com/optimsoc/optimsoc/pull/99.patch"},"body":"Work in progress for a base system that is based on a mesh and has only compute tiles.\r\n\r\nWork items:\r\n\r\n- [x] Replace existing system_2x2_cccc\r\n- [x] Verilator toplevel handling of memories\r\n- [x] Change CTLIST config paramter to a bitmask\r\n- [ ] Route debug ring as meander (currently sequential by ID)\r\n- [ ] Allow extension points (extra ports to the NoC that are made external)\r\n- [ ] Attach host tile to extension point\r\n- [ ] Update documentation accordingly\r\n"},"comment":{"url":"https://api.github.com/repos/optimsoc/optimsoc/issues/comments/374853362","html_url":"https://github.com/optimsoc/optimsoc/pull/99#issuecomment-374853362","issue_url":"https://api.github.com/repos/optimsoc/optimsoc/issues/99","id":374853362,"user":{"login":"wallento","id":299017,"avatar_url":"https://avatars3.githubusercontent.com/u/299017?v=4","gravatar_id":"","url":"https://api.github.com/users/wallento","html_url":"https://github.com/wallento","followers_url":"https://api.github.com/users/wallento/followers","following_url":"https://api.github.com/users/wallento/following{/other_user}","gists_url":"https://api.github.com/users/wallento/gists{/gist_id}","starred_url":"https://api.github.com/users/wallento/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wallento/subscriptions","organizations_url":"https://api.github.com/users/wallento/orgs","repos_url":"https://api.github.com/users/wallento/repos","events_url":"https://api.github.com/users/wallento/events{/privacy}","received_events_url":"https://api.github.com/users/wallento/received_events","type":"User","site_admin":false},"created_at":"2018-03-21T07:35:41Z","updated_at":"2018-03-21T07:35:41Z","author_association":"OWNER","body":"Hi @imphil,\r\n\r\nIn a larger design the debug interconnect can become critical, because the mesh is actually mapped quite nicely as you would expect. Problem is that if you map them sequentially row by row, there is a very long trace. My suggestion is to wire them differently, but the IDs of the debug modules stay the same. So the logical view remains intact, but timing is improved.\r\n\r\nThe reason I did not put XDIM and YDIM into the CONFIG structure is that I still want to add other topologies soon. My general take is all common architecture configuration should go into the CONFIG structure, but so far it was mainly based on a gut feeling. I am open to moving stuff around, such as splitting it further, or drop it entirely (in favor of individual parameters or a new templating approach.\r\n\r\nCheers,\r\nStefan"}}
{"actor":{"display_login":"wallento"}}