Skip to content

feat(uvc): add NV12 format parsing#492

Merged
tore-espressif merged 1 commit into
masterfrom
feature/add_nv12_format
May 13, 2026
Merged

feat(uvc): add NV12 format parsing#492
tore-espressif merged 1 commit into
masterfrom
feature/add_nv12_format

Conversation

@TDA-2030
Copy link
Copy Markdown
Collaborator

Description

This PR adds NV12 format support to the USB Host UVC descriptor parser.

Changes:

  • Add UVC_VS_FORMAT_NV12 to the public stream format enum.
  • Parse uncompressed UVC format descriptors with NV12 GUID.
  • Treat NV12 as an uncompressed frame format when populating frame info.
  • Fix unsupported format handling by checking the parser result as int before casting to enum, preventing -1 from leaking into
    frame info.

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@TDA-2030 TDA-2030 deployed to esp-docs preview May 12, 2026 03:38 — with GitHub Actions Active
Copy link
Copy Markdown
Collaborator

@tore-espressif tore-espressif left a comment

Choose a reason for hiding this comment

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

@TDA-2030 Thank you!

Do you have a USB descriptor with NV12 format so we could add it to our tests?

@tore-espressif tore-espressif added this to the usb_host_uvc 2.5.1 milestone May 12, 2026
@TDA-2030
Copy link
Copy Markdown
Collaborator Author

@TDA-2030 Thank you!

Do you have a USB descriptor with NV12 format so we could add it to our tests?

Yes, Here it is.
cam-desc.txt

@tore-espressif tore-espressif merged commit 906962c into master May 13, 2026
62 checks passed
@tore-espressif tore-espressif deleted the feature/add_nv12_format branch May 13, 2026 06:57
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.

2 participants