Re: (EXT) Re: [PATCH v3 3/7] arm64: defconfig: enable drivers for TQ TQMa8MxML-MBa8Mx

From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: Abel Vesa <abel.vesa@nxp.com>, Shawn Guo <shawnguo@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>, Fabio Estevam <festevam@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: (EXT) Re: [PATCH v3 3/7] arm64: defconfig: enable drivers for TQ TQMa8MxML-MBa8Mx
Date: Fri, 17 Dec 2021 09:32:53 +0100	[thread overview]
Message-ID: <97b2f5bb633ff2e25d4953eea63b31a5499ff433.camel@ew.tq-group.com> (raw)
In-Reply-To: <Ybug4acjSvCG+XxA@ryzen>

Am Donnerstag, dem 16.12.2021 um 22:26 +0200 schrieb Abel Vesa:
> On 21-12-01 08:29:44, Alexander Stein wrote:
> > With the device tree in place, enable missing drivers as modules,
> > if
> > possible. PHY driver needs built-in for interrupt support.
> > 
> > Signed-off-by: Alexander Stein <
> > alexander.stein@ew.tq-group.com
> > >
> > ---
> > Changes in v3:
> > * None
> > 
> > Changes in v2:
> > * Add interconnect driver for imx8mm and imx8mn
> > 
> >  arch/arm64/configs/defconfig | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/arch/arm64/configs/defconfig
> > b/arch/arm64/configs/defconfig
> > index 0da6a944d5cd..44ce56dd2638 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -363,6 +363,7 @@ CONFIG_MICROSEMI_PHY=y
> >  CONFIG_AT803X_PHY=y
> >  CONFIG_REALTEK_PHY=y
> >  CONFIG_ROCKCHIP_PHY=y
> > +CONFIG_DP83867_PHY=y
> >  CONFIG_VITESSE_PHY=y
> >  CONFIG_USB_PEGASUS=m
> >  CONFIG_USB_RTL8150=m
> > @@ -397,6 +398,7 @@ CONFIG_TOUCHSCREEN_EDT_FT5X06=m
> >  CONFIG_INPUT_MISC=y
> >  CONFIG_INPUT_PM8941_PWRKEY=y
> >  CONFIG_INPUT_PM8XXX_VIBRATOR=m
> > +CONFIG_INPUT_PWM_BEEPER=m
> >  CONFIG_INPUT_PWM_VIBRA=m
> >  CONFIG_INPUT_HISI_POWERKEY=y
> >  # CONFIG_SERIO_SERPORT is not set
> > @@ -556,6 +558,7 @@ CONFIG_BATTERY_MAX17042=m
> >  CONFIG_CHARGER_BQ25890=m
> >  CONFIG_CHARGER_BQ25980=m
> >  CONFIG_SENSORS_ARM_SCPI=y
> > +CONFIG_SENSORS_JC42=m
> >  CONFIG_SENSORS_LM90=m
> >  CONFIG_SENSORS_PWM_FAN=m
> >  CONFIG_SENSORS_RASPBERRYPI_HWMON=m
> > @@ -793,6 +796,7 @@ CONFIG_SND_SOC_RT5659=m
> >  CONFIG_SND_SOC_SIMPLE_AMPLIFIER=m
> >  CONFIG_SND_SOC_SIMPLE_MUX=m
> >  CONFIG_SND_SOC_TAS571X=m
> > +CONFIG_SND_SOC_TLV320AIC32X4_I2C=m
> >  CONFIG_SND_SOC_WCD934X=m
> >  CONFIG_SND_SOC_WM8904=m
> >  CONFIG_SND_SOC_WM8960=m
> > @@ -908,6 +912,7 @@ CONFIG_RTC_DRV_DS1307=m
> >  CONFIG_RTC_DRV_HYM8563=m
> >  CONFIG_RTC_DRV_MAX77686=y
> >  CONFIG_RTC_DRV_RK808=m
> > +CONFIG_RTC_DRV_PCF85063=m
> >  CONFIG_RTC_DRV_PCF85363=m
> >  CONFIG_RTC_DRV_M41T80=m
> >  CONFIG_RTC_DRV_RX8581=m
> > @@ -1175,6 +1180,8 @@ CONFIG_SLIM_QCOM_NGD_CTRL=m
> >  CONFIG_MUX_MMIO=y
> >  CONFIG_INTERCONNECT=y
> >  CONFIG_INTERCONNECT_IMX=m
> > +CONFIG_INTERCONNECT_IMX8MM=m
> > +CONFIG_INTERCONNECT_IMX8MN=m
> 
> Nope. The interconnect doesn't work without imx8m-ddrc
> and imx-bus devfreq drivers. Those drivers do not work
> yet. 
> 
> So please drop the interconnect configs for now.

They have been applied to next already, see [1]. Shawn, do you want a
revert for those 2 options then?

Best regards,
Alexander

[1] 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/arch/arm64/configs/defconfig?id=2ecc02a6b3f0e2a4339952c4f8b86a4d87876930