Re: [PATCH 1/2] dt-bindings: display: panel: add Ilitek ILI7836A OLED driver
| Newsgroups | gmane.linux.drivers.devicetree,gmane.comp.video.dri.devel |
|---|---|
| Message-ID | <[email protected]> |
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Low] The vendor prefix `retroidpocket` used in the new compatible string is not documented in `vendor-prefixes.yaml`. -- --- Patch [1]: [PATCH 1/2] dt-bindings: display: panel: add Ilitek ILI7836A OLED driver --- commit aacb89a0d4e50af14a474c3071ac0dce394b199e Author: Aaron Kling <[email protected]> dt-bindings: display: panel: add Ilitek ILI7836A OLED driver This commit introduces a new device tree binding for the Ilitek ILI7836A OLED driver. It documents the properties and compatible strings required for the MIPI-DSI display used on the Retroid Pocket Nova. > diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili7836a.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili7836a.yaml > new file mode 100644 > index 0000000000000..962cf257ed8d3 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili7836a.yaml [ ... ] > +properties: > + compatible: > + items: > + - const: retroidpocket,nova-panel [Severity: Low] Is the retroidpocket vendor prefix documented in vendor-prefixes.yaml? Adding a compatible string with an undocumented vendor prefix will cause a build warning when running make dt_binding_check. > + - const: ilitek,ili7836a > + > + reg: [ ... ] -- Sashiko AI review · https://sashiko.dev/#/patchset/20260808-ili7836a-v1-0-07a7becc5636-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org?part=1