{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3180","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3180/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3180/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3180/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/3180","id":86716838,"number":3180,"title":"cpu/sam3s: initial support for the Atmel SAM3S4B","user":{"login":"sgso","id":3769182,"avatar_url":"https://avatars.githubusercontent.com/u/3769182?v=3","gravatar_id":"","url":"https://api.github.com/users/sgso","html_url":"https://github.com/sgso","followers_url":"https://api.github.com/users/sgso/followers","following_url":"https://api.github.com/users/sgso/following{/other_user}","gists_url":"https://api.github.com/users/sgso/gists{/gist_id}","starred_url":"https://api.github.com/users/sgso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sgso/subscriptions","organizations_url":"https://api.github.com/users/sgso/orgs","repos_url":"https://api.github.com/users/sgso/repos","events_url":"https://api.github.com/users/sgso/events{/privacy}","received_events_url":"https://api.github.com/users/sgso/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/arm","name":"arm","color":"dddddd"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/major","name":"major","color":"ff0000"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/NEEDS%20SQUASHING","name":"NEEDS SQUASHING","color":"009800"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/new%20feature","name":"new feature","color":"e11d21"}],"state":"open","locked":false,"assignee":{"login":"haukepetersen","id":620834,"avatar_url":"https://avatars.githubusercontent.com/u/620834?v=3","gravatar_id":"","url":"https://api.github.com/users/haukepetersen","html_url":"https://github.com/haukepetersen","followers_url":"https://api.github.com/users/haukepetersen/followers","following_url":"https://api.github.com/users/haukepetersen/following{/other_user}","gists_url":"https://api.github.com/users/haukepetersen/gists{/gist_id}","starred_url":"https://api.github.com/users/haukepetersen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/haukepetersen/subscriptions","organizations_url":"https://api.github.com/users/haukepetersen/orgs","repos_url":"https://api.github.com/users/haukepetersen/repos","events_url":"https://api.github.com/users/haukepetersen/events{/privacy}","received_events_url":"https://api.github.com/users/haukepetersen/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2015-06-09T19:59:13Z","updated_at":"2015-07-23T13:12:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/3180","html_url":"https://github.com/RIOT-OS/RIOT/pull/3180","diff_url":"https://github.com/RIOT-OS/RIOT/pull/3180.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/3180.patch"},"body":"This adds support for the SAM3S4B but the includes contain the necessary\r\nheaders for all MCUs of the SAM3S family (except the SAM3S8/SD8).\r\n\r\n- include/ contains a pristine copy of sam/utils/cmsis/sam3s/include/ from\r\nthe latest ASF standalone archive as of commit date (version 3.24.1).\r\n- periph/spi.c: copy from sam3x8e with `spi_init_slave()` removed because it differs but I can't test it (device is a USB stick).\r\n- periph/gpio.c: depends on #3095. It's my own implementation because sam3x8e version is broken anyway.\r\n- periph/timer.c: unchanged copy from sam3x8e. I do not understand this timer stuff (yet). It does something but I'd really like to have more information how timers shall behave. I'm clueless.\r\n- periph/uart.c: shortened copy from sam3x8e.\r\n- startup.c: I #ifdef'd it to be compatible with all SAM3S cpus. Of course not tested.\r\n\r\nWe should probably have some kind of sam3-common."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/124092751","html_url":"https://github.com/RIOT-OS/RIOT/pull/3180#issuecomment-124092751","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3180","id":124092751,"user":{"login":"aethaniel","id":699018,"avatar_url":"https://avatars.githubusercontent.com/u/699018?v=3","gravatar_id":"","url":"https://api.github.com/users/aethaniel","html_url":"https://github.com/aethaniel","followers_url":"https://api.github.com/users/aethaniel/followers","following_url":"https://api.github.com/users/aethaniel/following{/other_user}","gists_url":"https://api.github.com/users/aethaniel/gists{/gist_id}","starred_url":"https://api.github.com/users/aethaniel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aethaniel/subscriptions","organizations_url":"https://api.github.com/users/aethaniel/orgs","repos_url":"https://api.github.com/users/aethaniel/repos","events_url":"https://api.github.com/users/aethaniel/events{/privacy}","received_events_url":"https://api.github.com/users/aethaniel/received_events","type":"User","site_admin":false},"created_at":"2015-07-23T13:12:10Z","updated_at":"2015-07-23T13:12:10Z","body":"atsam3s and atsam3x share same architecture and have also lot of common peripherals with most of all atsam4 and atsamg5x (but not with samd series). \r\nCouldn't they have a common folder (ie atsam34 or similar) to host their peripheral drivers?\r\nDevice specific actions could be done in dedicated (sub)folders like atsam3x, atsam3s, atsamg55, etc...\r\n"}}