{"action":"created","issue":{"url":"https://api.github.com/repos/rilmodem/ofono/issues/231","repository_url":"https://api.github.com/repos/rilmodem/ofono","labels_url":"https://api.github.com/repos/rilmodem/ofono/issues/231/labels{/name}","comments_url":"https://api.github.com/repos/rilmodem/ofono/issues/231/comments","events_url":"https://api.github.com/repos/rilmodem/ofono/issues/231/events","html_url":"https://github.com/rilmodem/ofono/pull/231","id":139212207,"number":231,"title":"fix misaligned dual sim behavior compared to AOSP","user":{"login":"vicamo","id":159864,"avatar_url":"https://avatars.githubusercontent.com/u/159864?v=3","gravatar_id":"","url":"https://api.github.com/users/vicamo","html_url":"https://github.com/vicamo","followers_url":"https://api.github.com/users/vicamo/followers","following_url":"https://api.github.com/users/vicamo/following{/other_user}","gists_url":"https://api.github.com/users/vicamo/gists{/gist_id}","starred_url":"https://api.github.com/users/vicamo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vicamo/subscriptions","organizations_url":"https://api.github.com/users/vicamo/orgs","repos_url":"https://api.github.com/users/vicamo/repos","events_url":"https://api.github.com/users/vicamo/events{/privacy}","received_events_url":"https://api.github.com/users/vicamo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-03-08T08:32:19Z","updated_at":"2016-03-10T17:14:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rilmodem/ofono/pulls/231","html_url":"https://github.com/rilmodem/ofono/pull/231","diff_url":"https://github.com/rilmodem/ofono/pull/231.diff","patch_url":"https://github.com/rilmodem/ofono/pull/231.patch"},"body":"Two things addressed in this pull request. One commit for each.\r\n\r\nThis first thing is ril daemon socket path for the 2nd instance should be _/dev/socket/rild2_ instead of _/dev/socket/rild1_. See https://android.googlesource.com/platform/hardware/ril/+/android-5.0.0_r1/libril/ril.cpp#59\r\n\r\nThe other thing is some rild implementation issues radio state change event with state __UNAVAILABLE__ after __RIL_CONNECTED__. After that, rild is expecting a reset by __RADIO_POWER__ request to __OFF__. This is actually etched in Android telephony framework and for aforementioned ril implementations, they simply cannot transit its internal state to on without such reset. See https://android.googlesource.com/platform/frameworks/opt/telephony/+/android-5.1.1_r1/src/java/com/android/internal/telephony/RIL.java#3115 and http://git.mozilla.org/?p=releases/gecko.git;a=blob;f=dom/system/gonk/ril_worker.js;h=330030036500b0ecc44473738ddb78d8171936a4;hb=HEAD#l5333"},"comment":{"url":"https://api.github.com/repos/rilmodem/ofono/issues/comments/194962659","html_url":"https://github.com/rilmodem/ofono/pull/231#issuecomment-194962659","issue_url":"https://api.github.com/repos/rilmodem/ofono/issues/231","id":194962659,"user":{"login":"tonyespy","id":5264713,"avatar_url":"https://avatars.githubusercontent.com/u/5264713?v=3","gravatar_id":"","url":"https://api.github.com/users/tonyespy","html_url":"https://github.com/tonyespy","followers_url":"https://api.github.com/users/tonyespy/followers","following_url":"https://api.github.com/users/tonyespy/following{/other_user}","gists_url":"https://api.github.com/users/tonyespy/gists{/gist_id}","starred_url":"https://api.github.com/users/tonyespy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tonyespy/subscriptions","organizations_url":"https://api.github.com/users/tonyespy/orgs","repos_url":"https://api.github.com/users/tonyespy/repos","events_url":"https://api.github.com/users/tonyespy/events{/privacy}","received_events_url":"https://api.github.com/users/tonyespy/received_events","type":"User","site_admin":false},"created_at":"2016-03-10T17:14:16Z","updated_at":"2016-03-10T17:14:16Z","body":"I also agree with Alfonso, I'd also prefer that you setup a cbd instance in ril_connected instead of re-working ril_send_power.  One idea we stick to when making changes is to change as little code as possible."}}