Subject: Re: [GIT PULL 7/7] i.MX defconfig changes for 5.3

From: Shawn Guo <[email-masked]>
Date: Wed, 26 Jun 2019 10:02:18 +0800

On Tue, Jun 25, 2019 at 05:32:34AM -0700, Olof Johansson wrote:
> > Anson Huang (6):
> >       arm64: defconfig: add support for i.MX system controller watchdog
> >       ARM: imx_v6_v7_defconfig: Enable CONFIG_THERMAL_STATISTICS
> >       ARM: imx_v6_v7_defconfig: Add TPM PWM support by default
> >       arm64: defconfig: Enable CONFIG_QORIQ_THERMAL
> >       arm64: defconfig: Add i.MX SCU SoC info driver
> 
> An SoC info driver shouldn't need to be =y, I think? It exports no in-kernel
> interfaces, so it should do just fine as a module.

Hmm, soc_device_match() comes out of SoC bus subsystem and will be used
by device drivers to identify SoC.

> 
> 
> >       arm64: defconfig: Enable CONFIG_KEYBOARD_SNVS_PWRKEY as module
> > 
> > Fabio Estevam (3):
> >       arm64: defconfig: Enable CONFIG_SPI_IMX
> >       ARM: imx_v6_v7_defconfig: Enable the OV2680 camera driver
> >       ARM: imx_v6_v7_defconfig: Select CONFIG_NVMEM_SNVS_LPGPR
> > 
> > Leonard Crestez (6):
> >       arm64: defconfig: Enable imx8mm clk/pinctrl
> >       arm64: defconfig: Enable lpi2c for imx8qxp and sensors
> >       arm64: defconfig: Enable ROHM_BD718XX PMIC for imx8mm-evk
> >       arm64: defconfig: NVMEM_IMX_OCOTP=y for imx8m
> 
> And why does this need to be =y? Patch doesn't say.

Sorry for not making it clear in the commit log.  This basically enables
the access to SoC fuses, where various system information are stored
there, like CPU speed grading which is needed by cpufreq support.

I will rework the patch by putting above info into commit log, and
resend the PR.

Shawn

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel