Re: [PATCH 3/3] ARM: dts: rockchip: rv1108-elgin-r1: Do not describe unexisting DAC device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jul 19, 2024 at 10:02:56AM +0200, Heiko Stübner wrote:
> Am Donnerstag, 18. Juli 2024, 22:05:40 CEST schrieb Fabio Estevam:
> > There is no DAC connected to the SPI bus of the Elgin RV1108 R1 board.
> > 
> > There is an LCD controlled via SPI though.
> > 
> > Properly describe it by adding the "elgin,spi-lcd" compatible
> > string.
> > 
> > Reported-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> > Closes: https://lore.kernel.org/linux-arm-kernel/20240717-parrot-malt-83cc04bf6b36@spud/
> > Signed-off-by: Fabio Estevam <festevam@xxxxxxxxx>
> > ---
> > Hi Heiko,
> > 
> > Conor sent a patch removing the rohm,dh2228fv entry from rv1108-elgin-r1.dts:
> > 
> > https://lore.kernel.org/linux-arm-kernel/20240717-parrot-malt-83cc04bf6b36@spud/
> > 
> > If you prefer, I can send a patch on top of his.
> 
> The decision is interesting, one less patch vs. keeping Conor's separate
> patch history. Maybe Conor voices his opinion while we're still in the merge
> window :-) .

Adding the correct device is an improvement over my patch, I don't see a
reason (other than vanity) for applying both. However, I don't think
that "elgin,spi-lcd" is a suitable compatible as it is far too generic.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux