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

core/linux-raspberrypi: re-enable BCM_VC_SM #1520

Closed
wants to merge 1 commit into from

Conversation

invisiblek
Copy link

  • This has been fixed in the raspberry pi kernel for multi-platform
  • Also fix building with a non-clean tree (firmware/brcm already exists)

* This has been fixed in the raspberry pi kernel for multi-platform.
* Also fix building with a non-clean tree (firmware/brcm already exists)
@invisiblek
Copy link
Author

Let's try this again ;)

@kmihelich
Copy link
Member

You've tested this on both architectures?

@invisiblek
Copy link
Author

I've only tested on ARMv7 (pi3), but it'll now run the ARMv6 assembly on v6 devices, which it hadn't been doing. I do have an original model B somewhere around that's not set up, but could test on that sometime later this week.

@kmihelich
Copy link
Member

I'd like to see this tested on both to be absolutely sure.

@invisiblek
Copy link
Author

Well, it builds, boots, /dev/vcsm exists, so its loading the driver correctly (ARMv6). But it doesn't appear the ARMv6 pi is affected by not having this driver in kodi. Kodi will crash on the pi3 if this driver isn't loaded, but not the case on the 1st gen. I suppose I'd have to dig into the userspace vc_sm stuff to know why.

Regardless, this should be good to go. The driver is now arch-agnostic upstream.

@jonnybarnes
Copy link

Any idea when this might make it into the repos? I believe this will fix Kodi video playback by bringing back /dev/vcsm

@graysky2
Copy link
Member

graysky2 commented Sep 29, 2017

Upstream and 4.9.52 has this present. I building for v7 and v6 now. I cannot test v6 on hardware though...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants