{"action":"created","issue":{"url":"https://api.github.com/repos/armbian/build/issues/954","repository_url":"https://api.github.com/repos/armbian/build","labels_url":"https://api.github.com/repos/armbian/build/issues/954/labels{/name}","comments_url":"https://api.github.com/repos/armbian/build/issues/954/comments","events_url":"https://api.github.com/repos/armbian/build/issues/954/events","html_url":"https://github.com/armbian/build/pull/954","id":321335887,"number":954,"title":"allow BPi-M2-Zero to build bootable images with bootloader as binary blob","user":{"login":"chwe17","id":21057365,"avatar_url":"https://avatars1.githubusercontent.com/u/21057365?v=4","gravatar_id":"","url":"https://api.github.com/users/chwe17","html_url":"https://github.com/chwe17","followers_url":"https://api.github.com/users/chwe17/followers","following_url":"https://api.github.com/users/chwe17/following{/other_user}","gists_url":"https://api.github.com/users/chwe17/gists{/gist_id}","starred_url":"https://api.github.com/users/chwe17/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chwe17/subscriptions","organizations_url":"https://api.github.com/users/chwe17/orgs","repos_url":"https://api.github.com/users/chwe17/repos","events_url":"https://api.github.com/users/chwe17/events{/privacy}","received_events_url":"https://api.github.com/users/chwe17/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-05-08T20:30:16Z","updated_at":"2018-05-09T21:38:42Z","closed_at":"2018-05-09T06:13:18Z","author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/armbian/build/pulls/954","html_url":"https://github.com/armbian/build/pull/954","diff_url":"https://github.com/armbian/build/pull/954.diff","patch_url":"https://github.com/armbian/build/pull/954.patch"},"body":"The following changes should allow to build BPi-M2-Zero Images without affecting the build of other H2+/H3 Images by using a patched u-boot as a binary blob (u-boot from stable branch with the SD-Card fix from https://github.com/BPI-SINOVOIP/BPI-files/blob/master/others/armbian/build/patch/u-boot/u-boot-sunxi/fix-sdcard-detect-bpi-m2z.patch as userpatch). Tested with 2 Images:\r\n\r\nBPi M2 Zero Debian Jessie: http://ix.io/19P1\r\nOPi Zero Debian Jessie: http://ix.io/19P6 \r\n\r\nIt's still a nasty hack, but at least BPi M2 Zero csc images can be built without issues. In case you think it's too nasty, I'm not disappointed if it doesn't get merged into the buildscript. \r\n\r\nChWe"},"comment":{"url":"https://api.github.com/repos/armbian/build/issues/comments/387883662","html_url":"https://github.com/armbian/build/pull/954#issuecomment-387883662","issue_url":"https://api.github.com/repos/armbian/build/issues/954","id":387883662,"user":{"login":"chwe17","id":21057365,"avatar_url":"https://avatars1.githubusercontent.com/u/21057365?v=4","gravatar_id":"","url":"https://api.github.com/users/chwe17","html_url":"https://github.com/chwe17","followers_url":"https://api.github.com/users/chwe17/followers","following_url":"https://api.github.com/users/chwe17/following{/other_user}","gists_url":"https://api.github.com/users/chwe17/gists{/gist_id}","starred_url":"https://api.github.com/users/chwe17/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chwe17/subscriptions","organizations_url":"https://api.github.com/users/chwe17/orgs","repos_url":"https://api.github.com/users/chwe17/repos","events_url":"https://api.github.com/users/chwe17/events{/privacy}","received_events_url":"https://api.github.com/users/chwe17/received_events","type":"User","site_admin":false},"created_at":"2018-05-09T21:38:42Z","updated_at":"2018-05-09T21:38:42Z","author_association":"NONE","body":"Cause it doesn't work?! It's not that I've chosen the most ugly approach to solve something which could be solved with a simple patch... I tried several approaches to implement Icenowys dts for u-boot none of them worked on my board and it ends with:\r\n```\r\nU-Boot SPL 2017.11-armbian (May 09 2018 - 21:23:13)\r\nDRAM: 512 MiB\r\nTrying to boot from MMC1\r\nMMC: no card present\r\nmmc_init: -123, time 2\r\nspl: mmc init failed with error: -123\r\nSPL: failed to boot from all boot devices\r\n### ERROR ### Please RESET the board ###\r\n```\r\nalso thought about her comment:\r\n```\r\n/*\r\n--\r\n  | * On the production batch of this board the card detect GPIO is\r\n  | * high active (card inserted), although on the early samples it's\r\n  | * low active.\r\n  | */\r\n```\r\nand tested this too, which also doesn't do 'the magic'...\r\nCause the sdcard socket doesn't allow to check whenever it's active high or active low there's IMO no solution to figure out what's correct (despite that there's no chance to be sure it ends on PF6). I'm fine when you decide to decline the change, but then you should be consistent and remove the build-option for BPi-M2-Zero at all. It's known that the current active patches don't allow to build an image, there's IMO no possibility solve it 'proper' and you don't like the ugly approach so why keep the rest here and upset people? They build an image don't see it booting, ask in the forum and we tell them that they should use the search engine... And the fact that serial debug isn't populated (you've to solder it first) makes it even worse.\r\n\r\n\r\n"}}
{"actor":{"display_login":"chwe17"}}