Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: make both HDMI ports work on RPi 4 #3430

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

AlekSi
Copy link
Contributor

@AlekSi AlekSi commented Apr 4, 2021

Closes #3414.

Should be tested manually before merge.

  • you linked an issue (if applicable)
  • you included tests (if applicable)
  • you ran conformance (make conformance)
  • you formatted your code (make fmt)
  • you linted your code (make lint)
  • you generated documentation (make docs)
  • you ran unit-tests (make unit-tests)

@AlekSi
Copy link
Contributor Author

AlekSi commented Apr 4, 2021

/approve

andrewrynhard
andrewrynhard previously approved these changes Apr 4, 2021
@@ -5,8 +5,8 @@
package rpi4

import (
_ "embed" //nolint:gci
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

//nolint:gci is required by the combination of blank import, golang/lint issues (and abandonment) and gci being gci.

@AlekSi AlekSi marked this pull request as ready for review April 4, 2021 17:40
@AlekSi
Copy link
Contributor Author

AlekSi commented Apr 5, 2021

/rebase

Closes siderolabs#3414.

Signed-off-by: Alexey Palazhchenko <alexey.palazhchenko@gmail.com>
Copy link
Member

@andrewrynhard andrewrynhard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AlekSi This is a simple enough change that you can force merge. I don't think you need to rebase.

@andrewrynhard andrewrynhard merged commit 3dc8462 into siderolabs:master Apr 5, 2021
@AlekSi AlekSi deleted the rpi-hdmi branch April 6, 2021 11:29
@AlekSi
Copy link
Contributor Author

AlekSi commented Apr 6, 2021

That PR wasn't merged because I wanted to re-test it manually :) And yeah, it does not solve the issue.

@AlekSi AlekSi mentioned this pull request Apr 6, 2021
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make both HDMI ports work on Raspberry Pi 4
3 participants