Re: [PATCH] arm64: defconfig: Enable modules for arm displays

From: Liviu Dudau <liviu.dudau@arm.com>
To: Sudeep Holla <sudeep.holla@arm.com>
Cc: carsten.haitzler@foss.arm.com, carsten.haitzler@arm.com,
	soc@kernel.org, linux-arm-kernel@lists.infradead.org,
	Robin Murphy <robin.murphy@arm.com>
Subject: Re: [PATCH] arm64: defconfig: Enable modules for arm displays
Date: Tue, 10 May 2022 13:55:13 +0100	[thread overview]
Message-ID: <YnpgsZAfUAN3apQO@e110455-lin.cambridge.arm.com> (raw)
In-Reply-To: <20220510111545.uqatdfthc6lv5p7i@bogus>

On Tue, May 10, 2022 at 12:15:45PM +0100, Sudeep Holla wrote:
> + Liviu, Robin
> 
> On Mon, May 09, 2022 at 01:55:10PM +0100, carsten.haitzler@foss.arm.com wrote:
> > Key devices that support displays on SoCs like the Komeda DRM driver, the
> > older HDLCD were not enabled by default and should be so displays can work
> > out of the box on defconfig. Also Candence I2C support should be enabled so
> > the PHY and thus displays can work too.
> > 
> 
> While at it, can we get these too enabled which are needed on JUNO IIUC.
> CONFIG_DRM_I2C_NXP_TDA998X
> CONFIG_DRM_I2C_NXP_TDA9950

I would not enable by default TDA9950 as that one is for CEC, which Juno doesn't
implement so you get warnings everytime time tda998x probes.

Best regards,
Liviu

> 
> Liviu/Robin who use HDLCD more than me on Juno can confirm if these are
> needed or not. I seem to have these for some HDLCD clock testing using SCMI.
> 
> -- 
> Regards,
> Sudeep

-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ¯\_(ツ)_/¯