{"action":"created","issue":{"url":"https://api.github.com/repos/ARM-software/CMSIS_5/issues/406","repository_url":"https://api.github.com/repos/ARM-software/CMSIS_5","labels_url":"https://api.github.com/repos/ARM-software/CMSIS_5/issues/406/labels{/name}","comments_url":"https://api.github.com/repos/ARM-software/CMSIS_5/issues/406/comments","events_url":"https://api.github.com/repos/ARM-software/CMSIS_5/issues/406/events","html_url":"https://github.com/ARM-software/CMSIS_5/pull/406","id":352111394,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA5NTAzNjAx","number":406,"title":"Update TPI_FFCR definition","user":{"login":"Kochise","id":23077417,"node_id":"MDQ6VXNlcjIzMDc3NDE3","avatar_url":"https://avatars0.githubusercontent.com/u/23077417?v=4","gravatar_id":"","url":"https://api.github.com/users/Kochise","html_url":"https://github.com/Kochise","followers_url":"https://api.github.com/users/Kochise/followers","following_url":"https://api.github.com/users/Kochise/following{/other_user}","gists_url":"https://api.github.com/users/Kochise/gists{/gist_id}","starred_url":"https://api.github.com/users/Kochise/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kochise/subscriptions","organizations_url":"https://api.github.com/users/Kochise/orgs","repos_url":"https://api.github.com/users/Kochise/repos","events_url":"https://api.github.com/users/Kochise/events{/privacy}","received_events_url":"https://api.github.com/users/Kochise/received_events","type":"User","site_admin":false},"labels":[{"id":678727485,"node_id":"MDU6TGFiZWw2Nzg3Mjc0ODU=","url":"https://api.github.com/repos/ARM-software/CMSIS_5/labels/CORE","name":"CORE","color":"fef2c0","default":false},{"id":358597288,"node_id":"MDU6TGFiZWwzNTg1OTcyODg=","url":"https://api.github.com/repos/ARM-software/CMSIS_5/labels/review","name":"review","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":{"login":"JonatanAntoni","id":25795816,"node_id":"MDQ6VXNlcjI1Nzk1ODE2","avatar_url":"https://avatars1.githubusercontent.com/u/25795816?v=4","gravatar_id":"","url":"https://api.github.com/users/JonatanAntoni","html_url":"https://github.com/JonatanAntoni","followers_url":"https://api.github.com/users/JonatanAntoni/followers","following_url":"https://api.github.com/users/JonatanAntoni/following{/other_user}","gists_url":"https://api.github.com/users/JonatanAntoni/gists{/gist_id}","starred_url":"https://api.github.com/users/JonatanAntoni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JonatanAntoni/subscriptions","organizations_url":"https://api.github.com/users/JonatanAntoni/orgs","repos_url":"https://api.github.com/users/JonatanAntoni/repos","events_url":"https://api.github.com/users/JonatanAntoni/events{/privacy}","received_events_url":"https://api.github.com/users/JonatanAntoni/received_events","type":"User","site_admin":false},"assignees":[{"login":"JonatanAntoni","id":25795816,"node_id":"MDQ6VXNlcjI1Nzk1ODE2","avatar_url":"https://avatars1.githubusercontent.com/u/25795816?v=4","gravatar_id":"","url":"https://api.github.com/users/JonatanAntoni","html_url":"https://github.com/JonatanAntoni","followers_url":"https://api.github.com/users/JonatanAntoni/followers","following_url":"https://api.github.com/users/JonatanAntoni/following{/other_user}","gists_url":"https://api.github.com/users/JonatanAntoni/gists{/gist_id}","starred_url":"https://api.github.com/users/JonatanAntoni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JonatanAntoni/subscriptions","organizations_url":"https://api.github.com/users/JonatanAntoni/orgs","repos_url":"https://api.github.com/users/JonatanAntoni/repos","events_url":"https://api.github.com/users/JonatanAntoni/events{/privacy}","received_events_url":"https://api.github.com/users/JonatanAntoni/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2018-08-20T12:22:52Z","updated_at":"2018-09-05T07:26:01Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/ARM-software/CMSIS_5/pulls/406","html_url":"https://github.com/ARM-software/CMSIS_5/pull/406","diff_url":"https://github.com/ARM-software/CMSIS_5/pull/406.diff","patch_url":"https://github.com/ARM-software/CMSIS_5/pull/406.patch"},"body":"See http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ddi0337e/BABIFBCI.html (for Cortex-M3, but not in core_cm3.h) vs. http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ddi0439b/BABCCJHJ.html (for Cortex-M4)"},"comment":{"url":"https://api.github.com/repos/ARM-software/CMSIS_5/issues/comments/418627357","html_url":"https://github.com/ARM-software/CMSIS_5/pull/406#issuecomment-418627357","issue_url":"https://api.github.com/repos/ARM-software/CMSIS_5/issues/406","id":418627357,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODYyNzM1Nw==","user":{"login":"JonatanAntoni","id":25795816,"node_id":"MDQ6VXNlcjI1Nzk1ODE2","avatar_url":"https://avatars1.githubusercontent.com/u/25795816?v=4","gravatar_id":"","url":"https://api.github.com/users/JonatanAntoni","html_url":"https://github.com/JonatanAntoni","followers_url":"https://api.github.com/users/JonatanAntoni/followers","following_url":"https://api.github.com/users/JonatanAntoni/following{/other_user}","gists_url":"https://api.github.com/users/JonatanAntoni/gists{/gist_id}","starred_url":"https://api.github.com/users/JonatanAntoni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JonatanAntoni/subscriptions","organizations_url":"https://api.github.com/users/JonatanAntoni/orgs","repos_url":"https://api.github.com/users/JonatanAntoni/repos","events_url":"https://api.github.com/users/JonatanAntoni/events{/privacy}","received_events_url":"https://api.github.com/users/JonatanAntoni/received_events","type":"User","site_admin":false},"created_at":"2018-09-05T07:26:01Z","updated_at":"2018-09-05T07:26:01Z","author_association":"MEMBER","body":"Hi @Kochise,\r\n\r\nYour contribution is very appreciated. My matter is just to keep the structure clean, comprehensible and easy to use. The core header files are intended to be used as-is by all the different device vendors. I admit we are not always consistent to that rule, but the core header should only contain core specific definitions. From core perspective only two bits of this register are defined and all others are reserved, i.e. implementation defined. Its completely up to the device vendor to implement the TPIU, either by (a) using the CoreSight SoC-400 subsystem as-is provided by Arm or (b) a custom(ized) solution. I assume the majority of vendors use solution (a).\r\n\r\nThat's why I proposed to have additional subsystem header files. Furthermore such a solution would make it easier to reuse the same subsystem for devices with different cores without duplicating all the information.\r\n\r\nIts hard to estimate the demand of having such defines. I expect only a minority of advanced users (like you?) ever configuring the debug and trace peripherals from inside the device. Typically these registers are under control of the debugger, e.g. uVision. That's why I am a bit reluctant of assigning development capacities (we are only a few guys here) to this topic at the moment.\r\n\r\nWhat do you think about developing/providing this CoreSight SoC-400 subsystem definitions (and perhaps others) separately for the time being, e.g. CoreSight git repo and pack? That might be easier to maintain and allow us keeping track of the popularity.\r\n\r\nCheers,\r\nJonatan"}}
{"actor":{"display_login":"JonatanAntoni"}}