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
