{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4854","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4854/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4854/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4854/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/4854","id":134925547,"number":4854,"title":"stm32l1 allow open drain","user":{"login":"lebrush","id":1437538,"avatar_url":"https://avatars.githubusercontent.com/u/1437538?v=3","gravatar_id":"","url":"https://api.github.com/users/lebrush","html_url":"https://github.com/lebrush","followers_url":"https://api.github.com/users/lebrush/followers","following_url":"https://api.github.com/users/lebrush/following{/other_user}","gists_url":"https://api.github.com/users/lebrush/gists{/gist_id}","starred_url":"https://api.github.com/users/lebrush/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lebrush/subscriptions","organizations_url":"https://api.github.com/users/lebrush/orgs","repos_url":"https://api.github.com/users/lebrush/repos","events_url":"https://api.github.com/users/lebrush/events{/privacy}","received_events_url":"https://api.github.com/users/lebrush/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Device%20driver","name":"Device driver","color":"0052cc"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/enhancement","name":"enhancement","color":"f17070"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Ready%20for%20CI%20build","name":"Ready for CI build","color":"bfdadc"}],"state":"open","locked":false,"assignee":{"login":"gebart","id":317506,"avatar_url":"https://avatars.githubusercontent.com/u/317506?v=3","gravatar_id":"","url":"https://api.github.com/users/gebart","html_url":"https://github.com/gebart","followers_url":"https://api.github.com/users/gebart/followers","following_url":"https://api.github.com/users/gebart/following{/other_user}","gists_url":"https://api.github.com/users/gebart/gists{/gist_id}","starred_url":"https://api.github.com/users/gebart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gebart/subscriptions","organizations_url":"https://api.github.com/users/gebart/orgs","repos_url":"https://api.github.com/users/gebart/repos","events_url":"https://api.github.com/users/gebart/events{/privacy}","received_events_url":"https://api.github.com/users/gebart/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2016-02-19T17:34:12Z","updated_at":"2016-02-20T14:48:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/4854","html_url":"https://github.com/RIOT-OS/RIOT/pull/4854","diff_url":"https://github.com/RIOT-OS/RIOT/pull/4854.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/4854.patch"},"body":"The current API configuration does not allow open-drain to be configured in the GPIOs. However, the i2c needs to be configured as open-drain otherwise the output is corrupted. \r\n\r\nIn the following image you can see the output of the pins with not configured as open-drain. This effect is also observed by [others](https://my.st.com/public/STe2ecommunities/mcu/Lists/cortex_mx_stm32/Flat.aspx?RootFolder=https%3a%2f%2fmy%2est%2ecom%2fpublic%2fSTe2ecommunities%2fmcu%2fLists%2fcortex_mx_stm32%2fSTM32L-Discovery%20communicate%20in%20I2C%20with%20M24LR64&FolderCTID=0x01200200770978C69A1141439FE559EB459D7580009C4E14902C3CDE46A77F0FFD06506F5B&currentviews=2824)\r\n![stm32l1_opendrain](https://cloud.githubusercontent.com/assets/1437538/13183375/ebf033f0-d736-11e5-902e-463ee9932f7b.png)\r\n\r\nThe patch just extends the behaviour and does not modify the existing one."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/186620153","html_url":"https://github.com/RIOT-OS/RIOT/pull/4854#issuecomment-186620153","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4854","id":186620153,"user":{"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},"created_at":"2016-02-20T14:48:49Z","updated_at":"2016-02-20T14:48:49Z","body":"I am working at a solution for #4472 as we speak (probably get it done today, but testing will take a while), so I would say we merge this first so at least the I2C for the l1 is fixed for now.\r\n\r\ncode looks valid -> untested ACK once Travis is green."}}