Re: [PATCH v2] dt-bindings: display: panel: Make DSI panels port optional

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: display: panel: ilitek,ili9806e: port made optional
  for SPI panel
--

commit bcf90e9f5f8315b117fe2401d0e209382d33f585
Author: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

dt-bindings: display: panel: Make DSI panels port optional

This commit removes the port property from the required list in several
DSI display panel Device Tree binding schemas so that panels placed
directly under a DSI bus node do not require an OF graph port.

> diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9806e.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9806e.yaml
> index 2080d9e0ffac4..9a2602ddd9060 100644
> --- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9806e.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9806e.yaml
> @@ -33,7 +33,6 @@ required:
>    - vdd-supply
>    - reset-gpios
>    - backlight
> -  - port
>  
>  if:
>    properties:

[Severity: Low]
Should port remain required in the conditional then branch for
rocktech,rk050hr345-ct106a?

While densitron,dmt028vghmcmi-1d and ortustech,com35h3p70ulc are MIPI-DSI
panels, rocktech,rk050hr345-ct106a is an SPI-controlled DPI/RGB panel
(registered with DRM_MODE_CONNECTOR_DPI in ili9806e_spi_probe()).  Because
its node sits under an SPI controller rather than a DSI host, the RGB
display controller still needs the OF graph port to link to the panel.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261112150650.137216-2-krzysztof.kozlowski@oss.qualcomm.com?part=1