{"action":"created","issue":{"url":"https://api.github.com/repos/lowRISC/socip/issues/1","repository_url":"https://api.github.com/repos/lowRISC/socip","labels_url":"https://api.github.com/repos/lowRISC/socip/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/lowRISC/socip/issues/1/comments","events_url":"https://api.github.com/repos/lowRISC/socip/issues/1/events","html_url":"https://github.com/lowRISC/socip/issues/1","id":164688886,"number":1,"title":"[Suggestion] Consider make aclk and aresetn interface input","user":{"login":"nbdd0121","id":4065244,"avatar_url":"https://avatars.githubusercontent.com/u/4065244?v=3","gravatar_id":"","url":"https://api.github.com/users/nbdd0121","html_url":"https://github.com/nbdd0121","followers_url":"https://api.github.com/users/nbdd0121/followers","following_url":"https://api.github.com/users/nbdd0121/following{/other_user}","gists_url":"https://api.github.com/users/nbdd0121/gists{/gist_id}","starred_url":"https://api.github.com/users/nbdd0121/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nbdd0121/subscriptions","organizations_url":"https://api.github.com/users/nbdd0121/orgs","repos_url":"https://api.github.com/users/nbdd0121/repos","events_url":"https://api.github.com/users/nbdd0121/events{/privacy}","received_events_url":"https://api.github.com/users/nbdd0121/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-07-09T22:34:25Z","updated_at":"2016-07-11T12:56:56Z","closed_at":null,"body":"Current `aclk` and `aresetn` are connected separately.  Instead of doing this, we can simply add `aclk` and `aresetn` as input of interface `nasti_channel`. After this we will only need to hook up these two signals once when we instantiate interface, no need to hook it up separately. These two signals are part of NASTI, so combining them with the interface definition makes more sense than current implementation."},"comment":{"url":"https://api.github.com/repos/lowRISC/socip/issues/comments/231726317","html_url":"https://github.com/lowRISC/socip/issues/1#issuecomment-231726317","issue_url":"https://api.github.com/repos/lowRISC/socip/issues/1","id":231726317,"user":{"login":"nbdd0121","id":4065244,"avatar_url":"https://avatars.githubusercontent.com/u/4065244?v=3","gravatar_id":"","url":"https://api.github.com/users/nbdd0121","html_url":"https://github.com/nbdd0121","followers_url":"https://api.github.com/users/nbdd0121/followers","following_url":"https://api.github.com/users/nbdd0121/following{/other_user}","gists_url":"https://api.github.com/users/nbdd0121/gists{/gist_id}","starred_url":"https://api.github.com/users/nbdd0121/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nbdd0121/subscriptions","organizations_url":"https://api.github.com/users/nbdd0121/orgs","repos_url":"https://api.github.com/users/nbdd0121/repos","events_url":"https://api.github.com/users/nbdd0121/events{/privacy}","received_events_url":"https://api.github.com/users/nbdd0121/received_events","type":"User","site_admin":false},"created_at":"2016-07-11T12:56:56Z","updated_at":"2016-07-11T12:56:56Z","body":"I don't really see any flexibility loss here, since master and slaves connecting to the same channel must use same clock and reset signal, as suggested in AXI spec. By including them in interface input, we are actually enforcing necessary requirements."}}
{"actor":{"display_login":"nbdd0121"}}