Skip to content
Snippets Groups Projects

add support for xiaomi-ido (Xiaomi Redmi 3 [Pro])

Merged Imported Administrator requested to merge lawl:device-ido into master

Created by: lawl

  • Boots
  • USB Networking
  • osk-sdl works to unlock root partition
  • battery
  • weston starts and displays shit, couldn't be fucked to configure eudev for it yet, so touchscreen works only in osk-sdl, not weston yet

Merge request reports

Merged by avatar (Mar 29, 2025 6:54pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Administrator
  • Administrator
  • Administrator
  • Author Owner

    Created by: lawl

    good points, fixed. would it make sense to add all of these checks as kind of a 'linter' to pmbootstrap?

    I'll fully admit this is/was a "fuck all of this" PR that wasn't/isn't ready for upstreaming. Yes, I'm still pissed because you know why, I'll try to not let it influence a great project.

    By * postmarketOS Bot on 2017-12-26T23:33:38

    Edited by Ghost User
  • Administrator
    Administrator @root started a thread on commit 35c044ae
  • 1 #!/bin/sh
    2
    3 # postmarketos-mkinitfs's init script does already have a framebuffer workaround script
    4 # but that script checks if fb0/mode is empty. in our case, it already has the correct value
    5 # but the framebuffer is nevertheless not doing anything.
    6 # additionally if we set the mode again the at time the postmarketos init script does,
    7 # absolutely nothing happens. It seems to work if we sleep 5 seconds to give whatever needs
    8 # more time, more time and then set the framebuffer mode again.
    9
    10
    11 set_xiaomi_ido_framebuffer_mode() {
    • Author Owner

      @lawl you could source init_functions.sh file and call set_framebuffer_mode function again instead of creating a new function

      By Daniele Debernardi on 2017-12-26T23:27:57

      Edited by Ghost User
  • Administrator
    Administrator @root started a thread on commit 35c044ae
  • 1 #!/bin/sh
    2
    3 # postmarketos-mkinitfs's init script does already have a framebuffer workaround script
    4 # but that script checks if fb0/mode is empty. in our case, it already has the correct value
    5 # but the framebuffer is nevertheless not doing anything.
    6 # additionally if we set the mode again the at time the postmarketos init script does,
    7 # absolutely nothing happens. It seems to work if we sleep 5 seconds to give whatever needs
    8 # more time, more time and then set the framebuffer mode again.
    9
    10
    11 set_xiaomi_ido_framebuffer_mode() {
  • Administrator
    Administrator @root started a thread on commit 35c044ae
  • 1 #!/bin/sh
    2
    3 # postmarketos-mkinitfs's init script does already have a framebuffer workaround script
    4 # but that script checks if fb0/mode is empty. in our case, it already has the correct value
    5 # but the framebuffer is nevertheless not doing anything.
    6 # additionally if we set the mode again the at time the postmarketos init script does,
    7 # absolutely nothing happens. It seems to work if we sleep 5 seconds to give whatever needs
    8 # more time, more time and then set the framebuffer mode again.
    9
    10
    11 set_xiaomi_ido_framebuffer_mode() {
    • Author Owner

      Created by: lawl

      nope. it's subtly different and has to be.

      [ -e "/sys/class/graphics/fb0/mode" ]

      vs.

      [ -z "$(cat /sys/class/graphics/fb0/mode)" ]

      otherwise set_framebuffer_mode actually does nothing here. /sys/class/graphics/fb0/mode DOES have the correct value already, we still need to rewrite it because I don't fucking know why. No output otherwise.

      By * postmarketOS Bot on 2017-12-26T23:27:57

      Edited by Ghost User
  • Administrator
    Administrator @root started a thread on commit 35c044ae
  • 1 #!/bin/sh
    2
    3 # postmarketos-mkinitfs's init script does already have a framebuffer workaround script
    4 # but that script checks if fb0/mode is empty. in our case, it already has the correct value
    5 # but the framebuffer is nevertheless not doing anything.
    6 # additionally if we set the mode again the at time the postmarketos init script does,
    7 # absolutely nothing happens. It seems to work if we sleep 5 seconds to give whatever needs
    8 # more time, more time and then set the framebuffer mode again.
    9
    10
    11 set_xiaomi_ido_framebuffer_mode() {
    • Author Owner

      Created by: lawl

      arguably we could remove all checks and just cat ..> mode without any checks, which is what i had in the beginning, but changed it to this. not sure if it makes too much of a difference really. i guess it kind of highlights the difference on why set_framebuffer_mode doesn't work for this device?

      By * postmarketOS Bot on 2017-12-26T23:27:57

      Edited by Ghost User
  • Author Owner

    Created by: lawl

    also i couldn't get it to boot without qcdt, and the twrp i looked at seemed to be qcdt too, so there does not seem to be an immidiate way around it. the kernel didn't build with the whateveritsnamewas for building the dtbs.

    By * postmarketOS Bot on 2017-12-26T23:31:25

  • Author Owner

    Created by: ollieparanoid

    good points, fixed. would it make sense to add all of these checks as kind of a 'linter' to pmbootstrap?

    Yes it would! #854 would have catched the pkgver at least (this PR only needs to be reviewed before it can be merged). In the future we might also be able to do linting checks for shell scripts/APKBUILDs like we do it now with PEP8 for python (#110).

    I'll fully admit this is/was a "fuck all of this" PR that wasn't/isn't ready for upstreaming. Yes, I'm still pissed because you know why, I'll try to not let it influence a great project.

    I hope that you still enjoy the project and I wouldn't mind discussing this more (#postmarketOS-offtopic or a private message chat?).

    By * postmarketOS Bot on 2017-12-26T23:34:30

  • Author Owner

    Created by: lawl

    regarding the udev rule for touch, again, i just can't be fucked, i want to get radio working. if that means i can't upstream it, so be it.

    By * postmarketOS Bot on 2017-12-26T23:34:35

  • Author Owner

    Created by: ollieparanoid

    To upstream it into master, it only needs to boot. As said before, the device page would be nice if you have time for it. Otherwise just tell me and I'll add it.

    So... shall we merge it?

    By * postmarketOS Bot on 2017-12-26T23:40:26

  • Author Owner

    Created by: lawl

    Device page added and added into the feature matrix.

    By * postmarketOS Bot on 2017-12-27T00:32:41

  • Author Owner

    Created by: ollieparanoid

    Awesome, thank you very much!

    By * postmarketOS Bot on 2017-12-27T01:56:28

  • Please register or sign in to reply
    Loading