Repository navigation
Conversation
Contributor
|
See comment about separate repo for this: #71 (comment) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
My Vulkan and RetroArch presentation paths use VideoOut functions whose link stubs are already in this SDK, but whose declarations have to be repeated in each project. This adds
<ps5/videoout.h>with the 12 functions and supporting buffer record/constants used by those paths.The declarations are extracted from the shared downstream header. They cover opening and closing the output, registering buffers, flip submission/status, vblank waits, and output configuration. Attribute storage and flip status remain opaque rather than assigning unverified meanings to every field. The high-frame-rate selector still requires title and display support.
This is a header-only contribution. It introduces no new stub exports and is installed by the existing header installation rule.
Validation
Compiled as both C11 and C++11 for the PS5 target with Clang 23.1.1 and
-Wall -Wextra -Werror. Checked the buffer record's 32-byte size and metadata offset of eight bytes, the 80-byte attribute-storage constant, and the 128-byte status-storage constant. These checks match the existing downstream layout.A test object references every function address. All 12 names remain unmangled in C++, match the existing
libSceVideoOutstub, and link through the normal SDK wrapper.Draft: no fresh presentation run was performed for this extraction. The declarations reflect the existing homebrew implementation, not a claim that every signature, reserved argument, or display mode has been independently validated across firmware. I would like review of the API names and opaque-record conventions before making this a stable SDK interface.
Refs #70.