{"action":"created","issue":{"url":"https://api.github.com/repos/JMRI/JMRI/issues/3187","repository_url":"https://api.github.com/repos/JMRI/JMRI","labels_url":"https://api.github.com/repos/JMRI/JMRI/issues/3187/labels{/name}","comments_url":"https://api.github.com/repos/JMRI/JMRI/issues/3187/comments","events_url":"https://api.github.com/repos/JMRI/JMRI/issues/3187/events","html_url":"https://github.com/JMRI/JMRI/pull/3187","id":211974100,"number":3187,"title":"RaspberryPi Input provisioning","user":{"login":"n3ix","id":14222051,"avatar_url":"https://avatars3.githubusercontent.com/u/14222051?v=3","gravatar_id":"","url":"https://api.github.com/users/n3ix","html_url":"https://github.com/n3ix","followers_url":"https://api.github.com/users/n3ix/followers","following_url":"https://api.github.com/users/n3ix/following{/other_user}","gists_url":"https://api.github.com/users/n3ix/gists{/gist_id}","starred_url":"https://api.github.com/users/n3ix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/n3ix/subscriptions","organizations_url":"https://api.github.com/users/n3ix/orgs","repos_url":"https://api.github.com/users/n3ix/repos","events_url":"https://api.github.com/users/n3ix/events{/privacy}","received_events_url":"https://api.github.com/users/n3ix/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-03-05T19:01:34Z","updated_at":"2017-03-06T03:24:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/JMRI/JMRI/pulls/3187","html_url":"https://github.com/JMRI/JMRI/pull/3187","diff_url":"https://github.com/JMRI/JMRI/pull/3187.diff","patch_url":"https://github.com/JMRI/JMRI/pull/3187.patch"},"body":"The RaspberryPiSensor.java init() code currently provisions all Pi input\r\npins to have PULL_DOWN termination.  While most pins can be configured\r\nto have either PullDown or PullUp configuration, there are some pins\r\nthat have fixed hardware PullUp termination.  As a result attempting to\r\ncreate a Sensor in JMRI using a pin with fixed hardware PullUp\r\ntermination fails.  This applies to IO 8 and 9 for all Pi models and\r\nalso to IO 30 and 31 for models that support those pins.\r\n\r\nSince all Pi input pins appear to support PullUp termination, my\r\nrecommendation is to change the init() method to provision the pins\r\nusing the PinPullResistance.PULL_UP argument rather than the\r\nPinPullResistance.PULL_DOWN.  This change will allow use of all the GPIO\r\npins as Sensor inputs and will provide a the same termination on all\r\npins (PullUp) for consistency.\r\n\r\nSee\r\nhttps://github.com/Pi4J/pi4j/blob/master/pi4j-core/src/main/java/com/pi4j/io/gpio/RaspiPin.java#L82\r\nfor the pin definitions."},"comment":{"url":"https://api.github.com/repos/JMRI/JMRI/issues/comments/284294073","html_url":"https://github.com/JMRI/JMRI/pull/3187#issuecomment-284294073","issue_url":"https://api.github.com/repos/JMRI/JMRI/issues/3187","id":284294073,"user":{"login":"pabender","id":14183416,"avatar_url":"https://avatars1.githubusercontent.com/u/14183416?v=3","gravatar_id":"","url":"https://api.github.com/users/pabender","html_url":"https://github.com/pabender","followers_url":"https://api.github.com/users/pabender/followers","following_url":"https://api.github.com/users/pabender/following{/other_user}","gists_url":"https://api.github.com/users/pabender/gists{/gist_id}","starred_url":"https://api.github.com/users/pabender/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pabender/subscriptions","organizations_url":"https://api.github.com/users/pabender/orgs","repos_url":"https://api.github.com/users/pabender/repos","events_url":"https://api.github.com/users/pabender/events{/privacy}","received_events_url":"https://api.github.com/users/pabender/received_events","type":"User","site_admin":false},"created_at":"2017-03-06T03:24:54Z","updated_at":"2017-03-06T03:24:54Z","body":"closing in favor of #3190 (which is still a work in progress...)"}}
{"actor":{"display_login":"pabender"}}