Skip to content

staging: bcm2835_codec: Pass framerate to the component if set late #6108

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

Merged
merged 1 commit into from
May 14, 2024

Conversation

6by9
Copy link
Contributor

@6by9 6by9 commented Apr 17, 2024

For video encoding, if the framerate was set after the component was created, then it wasn't set correctly on the port, and an old value was encoded in the bitstream.

Update the port status when the framerate is set.

raspberrypi/rpicam-apps#664

@6by9
Copy link
Contributor Author

6by9 commented May 13, 2024

@popcornmix @naushir Any chance of a review or comment?

Naush - it fixes the codec storing the wrong framerate that was raised in raspberrypi/rpicam-apps#664

@naushir
Copy link
Contributor

naushir commented May 14, 2024

Apart from the setting of ret commend, looks good to me.

For video encoding, if the framerate was set after the component
was created, then it wasn't set correctly on the port, and an
old value was encoded in the bitstream.

Update the port status when the framerate is set.

raspberrypi/rpicam-apps#664

Signed-off-by: Dave Stevenson <[email protected]>
@6by9 6by9 force-pushed the rpi-6.6.y-codecs branch from 08b96f2 to 02d5df1 Compare May 14, 2024 12:49
@pelwell pelwell merged commit 886f86f into raspberrypi:rpi-6.6.y May 14, 2024
12 checks passed
popcornmix added a commit to raspberrypi/firmware that referenced this pull request May 20, 2024
kernel: RGB order override for DPI
See: raspberrypi/linux#6156

kernel: Minor panel fixups
See: raspberrypi/linux#6176

kernel: dtoverlays: Fix noints mode of mcp23017
See: raspberrypi/linux#6174

kernel: DTS: bcm2712: set nonzero QoS values for PCIE1
See: raspberrypi/linux#6171

kernel: staging: bcm2835_codec: Pass framerate to the component if set late
See: raspberrypi/linux#6108

kernel: spi: dw: Handle any number of gpiod CS lines
See: raspberrypi/linux#6161
popcornmix added a commit to raspberrypi/rpi-firmware that referenced this pull request May 20, 2024
kernel: RGB order override for DPI
See: raspberrypi/linux#6156

kernel: Minor panel fixups
See: raspberrypi/linux#6176

kernel: dtoverlays: Fix noints mode of mcp23017
See: raspberrypi/linux#6174

kernel: DTS: bcm2712: set nonzero QoS values for PCIE1
See: raspberrypi/linux#6171

kernel: staging: bcm2835_codec: Pass framerate to the component if set late
See: raspberrypi/linux#6108

kernel: spi: dw: Handle any number of gpiod CS lines
See: raspberrypi/linux#6161
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