Skip to content

Conversation

@6by9
Copy link
Contributor

@6by9 6by9 commented Oct 22, 2020

Cherry-pick of ff93994 from rpi-5.4.y

One bug fix patch to vc4_hdmi that fixes incorrect hot plug behaviour on Pi3.
One debug patch to dump out all the HDMI registers on PI4 instead of only those that overlap with Pi0-3.

6by9 added 3 commits October 22, 2020 12:39
OV9281 appears to drop the clock to LP mode between frames, but
the overlay didn't define this at both ends of the CSI2 link.
The overlay also had an incorrect link frequency defined, not that
the driver ever checked for one.

Fix both issues.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
The order of precedence should be:
- hotplug GPIO being defined
- DDC probe
- hotplug register

In particular the hotplug register is not valid if a GPIO is defined
(eg on Pi0-3), but was being checked.

Fixes "943f078 vc4: cec: Restore cec physical address on reconnect"

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
The vc5 HDMI registers hadn't been added into the debugfs
register sets, therefore weren't dumped on request.
Add them in.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
@pelwell pelwell merged commit ae12e44 into raspberrypi:rpi-5.9.y Oct 22, 2020
@6by9
Copy link
Contributor Author

6by9 commented Oct 22, 2020

@popcornmix This HDMI hotplug error may link into your CEC failure on Pi3 as it invalidates settings and generally messes around with CEC state when connected/disconnected.

@popcornmix
Copy link
Collaborator

I'll give it a test.
I had been debugging and found firmware was overwriting some registers (e.g. HDMI_CEC_CNTRL_1) but hadn't worked out why this only affected pi0-pi3 and 5.9.

@6by9
Copy link
Contributor Author

6by9 commented Oct 22, 2020

The firmware is overwriting registers? Have we not managed a clean handover from VPU to kernel then? Eek.

arm_dt.c masks the VPU away from the interrupts of any DT nodes that start v3d hdmi pixelvalve hvs txp firmwarekms csi, so hopefully that would stop it servicing them, but has HDMI been started before that point?

popcornmix added a commit to raspberrypi/firmware that referenced this pull request Oct 27, 2020
See: raspberrypi/Raspberry-Pi-OS-64bit#4

kernel: ARM: dts: Expand PCIe space on BCM2711
See: raspberrypi/linux#3930

kernel: bcm2711_defconfig: Restore lost downstream char drivers
See: https://www.raspberrypi.org/forums/viewtopic.php?f=29&t=288234&start=100#p1746578

kernel: dt updates, and kms using bcm2711-hvs for dma range
See: raspberrypi/linux#3927

kernel: 5.9: ov9281 cherrypick and kms patches
See: raspberrypi/linux#3914
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Oct 27, 2020
See: raspberrypi/Raspberry-Pi-OS-64bit#4

kernel: ARM: dts: Expand PCIe space on BCM2711
See: raspberrypi/linux#3930

kernel: bcm2711_defconfig: Restore lost downstream char drivers
See: https://www.raspberrypi.org/forums/viewtopic.php?f=29&t=288234&start=100#p1746578

kernel: dt updates, and kms using bcm2711-hvs for dma range
See: raspberrypi/linux#3927

kernel: 5.9: ov9281 cherrypick and kms patches
See: raspberrypi/linux#3914
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.

3 participants