Skip to content

docs: NA51089 pad-mux verified on hardware (#237) - #242

Merged
widgetii merged 1 commit into
masterfrom
novatek-hw-verified
Oct 10, 2026
Merged

widgetii merged 1 commit into
masterfrom
novatek-hw-verified

Conversation

@widgetii

Copy link
Copy Markdown
Member

Docs only. #237's reporter ran the merged build on an NT98566: reginfo --pads agreed with the kernel's /proc/nvt_info/nvt_pinmux/pinmux_summary on every pad, and claimed no pad the kernel had not configured. The comparison is in #237 (comment) and the replies after it.

docs/padmux.md said no Novatek die had been run on hardware. It now records NA51089 as verified. NA51055, NA51084 and the Novatek GPIO controller remain unverified.

The reporter's NT98566 agreed with the kernel's pinmux_summary on every
pad, and showed nothing the kernel had not configured. NA51055, NA51084
and the Novatek GPIO controller remain unverified.
@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can copy the agent prompt from any finding and feed it to your IDE agent

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Document NA51089 pad-mux hardware verification

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Record that NA51089 pad-mux output matched the kernel summary on a reporter’s NT98566.
• Clarify that NA51055, NA51084, and Novatek GPIO operations remain unverified.
Diagram

graph TD
  R["Live registers"] --> I["reginfo pads"] --> C{"Pad comparison"} --> D["Documented status"]
  R --> K["Kernel summary"] --> C
Loading
High-Level Assessment

Recording the result in the existing hardware-coverage section is appropriate: reviewers can find the verified die, comparison method, and unverified areas together. Leaving the evidence only in the issue would make the documentation’s previous blanket claim inaccurate.

Files changed (1) +10 / -3

Documentation (1) +10 / -3
padmux.mdRecord NA51089 hardware verification and remaining gaps +10/-3

Record NA51089 hardware verification and remaining gaps

• Replaces the blanket statement that Novatek dies lack hardware testing with the reporter’s NT98566 pad-by-pad comparison against the kernel summary. Specifies that NA51055, NA51084, and the Novatek GPIO controller remain unverified.

docs/padmux.md

@widgetii
widgetii merged commit ab796ca into master Oct 10, 2026
5 checks passed
@widgetii
widgetii deleted the novatek-hw-verified branch October 10, 2026 10:45
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.

1 participant