{"action":"created","issue":{"url":"https://api.github.com/repos/cleanflight/cleanflight/issues/1638","repository_url":"https://api.github.com/repos/cleanflight/cleanflight","labels_url":"https://api.github.com/repos/cleanflight/cleanflight/issues/1638/labels{/name}","comments_url":"https://api.github.com/repos/cleanflight/cleanflight/issues/1638/comments","events_url":"https://api.github.com/repos/cleanflight/cleanflight/issues/1638/events","html_url":"https://github.com/cleanflight/cleanflight/pull/1638","id":124312358,"number":1638,"title":"AlienFlightF3 V2 support(MPU6500/9250 via SPI)","user":{"login":"MJ666","id":8107952,"avatar_url":"https://avatars.githubusercontent.com/u/8107952?v=3","gravatar_id":"","url":"https://api.github.com/users/MJ666","html_url":"https://github.com/MJ666","followers_url":"https://api.github.com/users/MJ666/followers","following_url":"https://api.github.com/users/MJ666/following{/other_user}","gists_url":"https://api.github.com/users/MJ666/gists{/gist_id}","starred_url":"https://api.github.com/users/MJ666/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MJ666/subscriptions","organizations_url":"https://api.github.com/users/MJ666/orgs","repos_url":"https://api.github.com/users/MJ666/repos","events_url":"https://api.github.com/users/MJ666/events{/privacy}","received_events_url":"https://api.github.com/users/MJ666/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cleanflight/cleanflight/labels/enhancement","name":"enhancement","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cleanflight/cleanflight/milestones/12","html_url":"https://github.com/cleanflight/cleanflight/milestones/1.13.0","labels_url":"https://api.github.com/repos/cleanflight/cleanflight/milestones/12/labels","id":1458885,"number":12,"title":"1.13.0","description":null,"creator":{"login":"hydra","id":57075,"avatar_url":"https://avatars.githubusercontent.com/u/57075?v=3","gravatar_id":"","url":"https://api.github.com/users/hydra","html_url":"https://github.com/hydra","followers_url":"https://api.github.com/users/hydra/followers","following_url":"https://api.github.com/users/hydra/following{/other_user}","gists_url":"https://api.github.com/users/hydra/gists{/gist_id}","starred_url":"https://api.github.com/users/hydra/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hydra/subscriptions","organizations_url":"https://api.github.com/users/hydra/orgs","repos_url":"https://api.github.com/users/hydra/repos","events_url":"https://api.github.com/users/hydra/events{/privacy}","received_events_url":"https://api.github.com/users/hydra/received_events","type":"User","site_admin":false},"open_issues":22,"closed_issues":29,"state":"open","created_at":"2015-12-14T10:45:04Z","updated_at":"2016-04-11T20:16:12Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2015-12-30T07:02:29Z","updated_at":"2016-04-13T07:43:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cleanflight/cleanflight/pulls/1638","html_url":"https://github.com/cleanflight/cleanflight/pull/1638","diff_url":"https://github.com/cleanflight/cleanflight/pull/1638.diff","patch_url":"https://github.com/cleanflight/cleanflight/pull/1638.patch"},"body":"Enable Gyro interrupt for all AlienFlight F3 board revisions \r\nAK8963 mag implementation (part of MPU9250)\r\nUpdate LED driver for alternative LED set (AlienFlightF3 V2)\r\nAdd SPI port 3 for STM32F303 targets\r\nreintroduce SPI driver timeout to avoid blockage\r\nAlienFlight/AlienWii documentation update\r\nRename AlienWii to AlienFlight"},"comment":{"url":"https://api.github.com/repos/cleanflight/cleanflight/issues/comments/209286050","html_url":"https://github.com/cleanflight/cleanflight/pull/1638#issuecomment-209286050","issue_url":"https://api.github.com/repos/cleanflight/cleanflight/issues/1638","id":209286050,"user":{"login":"martinbudden","id":194586,"avatar_url":"https://avatars.githubusercontent.com/u/194586?v=3","gravatar_id":"","url":"https://api.github.com/users/martinbudden","html_url":"https://github.com/martinbudden","followers_url":"https://api.github.com/users/martinbudden/followers","following_url":"https://api.github.com/users/martinbudden/following{/other_user}","gists_url":"https://api.github.com/users/martinbudden/gists{/gist_id}","starred_url":"https://api.github.com/users/martinbudden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinbudden/subscriptions","organizations_url":"https://api.github.com/users/martinbudden/orgs","repos_url":"https://api.github.com/users/martinbudden/repos","events_url":"https://api.github.com/users/martinbudden/events{/privacy}","received_events_url":"https://api.github.com/users/martinbudden/received_events","type":"User","site_admin":false},"created_at":"2016-04-13T07:43:05Z","updated_at":"2016-04-13T07:43:05Z","body":"@MJ666 , I hadn't seen this PR. A couple of comments:\r\n\r\nIn `config.c`, could you change\r\n```\r\nrxConfig()->serialrx_provider = 1;\r\n```\r\nto\r\n```\r\nrxConfig()->serialrx_provider = SERIALRX_SPEKTRUM2048;\r\n```\r\nand\r\n```\r\npidProfile()->pidController = 2;\r\n```\r\nto\r\n```\r\npidProfile()->pidController = PID_CONTROLLER_LUX_FLOAT;\r\n```\r\nSorry to be a bit pedantic.\r\n\r\nAlso you might like to merge in my PR https://github.com/cleanflight/cleanflight/pull/2038, which changes\r\n```\r\ncustomMotorMixer(0)->throttle = 1.0f;\r\ncustomMotorMixer(0)->roll = -0.414178f;\r\ncustomMotorMixer(0)->pitch = 1.0f;\r\ncustomMotorMixer(0)->yaw = -1.0f;\r\n```\r\netc, to\r\n```\r\n*customMotorMixer(0) = (motorMixer_t){ 1.0f, -0.414178f,  1.0f, -1.0f };    // REAR_R\r\n```\r\nThis saves @hyrdra having to do the PR. My PR reduces the number of lines of code under the `ALIENFLIGHT` build flag, helping keep `config.c` compact. It also saves a bit of ROM space."}}