Re: [PATCH v3 1/2] dt-bindings: display: panel: Add RAiO RA8875 display controller

From: Adam Azuddin

Date: Thu Oct 01 2026 - 18:52:28 EST


On Thu, Oct 01, 2026 at 07:54:19PM +0100, Conor Dooley wrote:
> On Thu, Oct 01, 2026 at 10:41:25PM +0800, Adam Azuddin wrote:
> > +description:
> > + The RA8875 is a TFT LCD controller for panels up to 800x480 pixels.
> > + It can be connected to the controller via 8080/6800 parallel interface,
> > + SPI or I2C. This binding describes the SPI-connected device.
>
> This binding describes all three, or at least it should describe both
> i2c and spi in the current state. I have no idea what the parallel
> interfaces look like in dt, but if what you've got covers spi it should
> also cover i2c. Probably remove this last sentence.
> pw-bot: changes-requested

Understood, I'll drop that last sentence.

> > + panel: display@0 {
>
> Drop the unused panel label if there's a respin.

Will do.

Thanks for the review,
Adam