{"action":"created","issue":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/13909","repository_url":"https://api.github.com/repos/MarlinFirmware/Marlin","labels_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/13909/labels{/name}","comments_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/13909/comments","events_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/13909/events","html_url":"https://github.com/MarlinFirmware/Marlin/pull/13909","id":440370991,"node_id":"MDExOlB1bGxSZXF1ZXN0Mjc1OTQ4NzI4","number":13909,"title":"[2.0.x]Reduce LCD flicker by adding 60Hz servo option - implemented on DUE, STM32F1 and AVR","user":{"login":"Bob-the-Kuhn","id":20692583,"node_id":"MDQ6VXNlcjIwNjkyNTgz","avatar_url":"https://avatars0.githubusercontent.com/u/20692583?v=4","gravatar_id":"","url":"https://api.github.com/users/Bob-the-Kuhn","html_url":"https://github.com/Bob-the-Kuhn","followers_url":"https://api.github.com/users/Bob-the-Kuhn/followers","following_url":"https://api.github.com/users/Bob-the-Kuhn/following{/other_user}","gists_url":"https://api.github.com/users/Bob-the-Kuhn/gists{/gist_id}","starred_url":"https://api.github.com/users/Bob-the-Kuhn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bob-the-Kuhn/subscriptions","organizations_url":"https://api.github.com/users/Bob-the-Kuhn/orgs","repos_url":"https://api.github.com/users/Bob-the-Kuhn/repos","events_url":"https://api.github.com/users/Bob-the-Kuhn/events{/privacy}","received_events_url":"https://api.github.com/users/Bob-the-Kuhn/received_events","type":"User","site_admin":false},"labels":[{"id":92233140,"node_id":"MDU6TGFiZWw5MjIzMzE0MA==","url":"https://api.github.com/repos/MarlinFirmware/Marlin/labels/Needs:%20Testing","name":"Needs: Testing","color":"fbca04","default":false},{"id":92052497,"node_id":"MDU6TGFiZWw5MjA1MjQ5Nw==","url":"https://api.github.com/repos/MarlinFirmware/Marlin/labels/PR:%20Improvement","name":"PR: Improvement","color":"c2e0c6","default":false},{"id":281783768,"node_id":"MDU6TGFiZWwyODE3ODM3Njg=","url":"https://api.github.com/repos/MarlinFirmware/Marlin/labels/S:%20Don't%20Merge","name":"S: Don't Merge","color":"b60205","default":false},{"id":342950962,"node_id":"MDU6TGFiZWwzNDI5NTA5NjI=","url":"https://api.github.com/repos/MarlinFirmware/Marlin/labels/S:%20Experimental","name":"S: Experimental","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2019-05-04T21:05:05Z","updated_at":"2019-05-06T17:41:43Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/pulls/13909","html_url":"https://github.com/MarlinFirmware/Marlin/pull/13909","diff_url":"https://github.com/MarlinFirmware/Marlin/pull/13909.diff","patch_url":"https://github.com/MarlinFirmware/Marlin/pull/13909.patch"},"body":"EXPERIMENTAL\r\n\r\nNEEDS TESTING\r\n\r\nReduce the LED flicker by adding an option to set the servo rate to 60Hz.\r\n\r\nDevices like the BLTouch are compatible.  It is unknown how other servo based devices will react.\r\n\r\n---\r\n\r\nIn file **Conditionals_post.h** the macro **SERVO_FREQUENCY** is set to 60 if **SERVO_60HZ** is enabled in **configuration.h**.  It is set to 50 if not enabled.\r\n\r\n**SERVO_FREQUENCY** is then used to set the servo frequency/period in the various HALs.\r\n\r\nI put the **SERVO_60HZ** in the  **#if EITHER(RGB_LED, RGBW_LED)** section so that it was obvious why it would be used.\r\n\r\n---\r\n\r\nIMPLEMENTATION STATUS:\r\n- DUE - tested on BLTouch\r\n- STM32F1 - not tested\r\n- AVR - tested on BLTouch\r\n- LPC1768 - tested on BLTouch but with a manually edited **pwm.h** file.  Have not been able to automate the selection.\r\n- all others - couldn't find where the servo frequency was set\r\n"},"comment":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/comments/489708700","html_url":"https://github.com/MarlinFirmware/Marlin/pull/13909#issuecomment-489708700","issue_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/13909","id":489708700,"node_id":"MDEyOklzc3VlQ29tbWVudDQ4OTcwODcwMA==","user":{"login":"Bob-the-Kuhn","id":20692583,"node_id":"MDQ6VXNlcjIwNjkyNTgz","avatar_url":"https://avatars0.githubusercontent.com/u/20692583?v=4","gravatar_id":"","url":"https://api.github.com/users/Bob-the-Kuhn","html_url":"https://github.com/Bob-the-Kuhn","followers_url":"https://api.github.com/users/Bob-the-Kuhn/followers","following_url":"https://api.github.com/users/Bob-the-Kuhn/following{/other_user}","gists_url":"https://api.github.com/users/Bob-the-Kuhn/gists{/gist_id}","starred_url":"https://api.github.com/users/Bob-the-Kuhn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bob-the-Kuhn/subscriptions","organizations_url":"https://api.github.com/users/Bob-the-Kuhn/orgs","repos_url":"https://api.github.com/users/Bob-the-Kuhn/repos","events_url":"https://api.github.com/users/Bob-the-Kuhn/events{/privacy}","received_events_url":"https://api.github.com/users/Bob-the-Kuhn/received_events","type":"User","site_admin":false},"created_at":"2019-05-06T17:41:43Z","updated_at":"2019-05-06T17:41:43Z","author_association":"CONTRIBUTOR","body":"@xC0000005 - what changes did you make in the **HAL_STM32** directory?  Have you verified the period is now 16.67mS?"}}
{"actor":{"display_login":"Bob-the-Kuhn"}}