Re: [GIT PULL 6/6] i.MX defconfig update for 5.2

From: Shawn Guo <shawnguo@kernel.org>
To: Olof Johansson <olof@lixom.net>,
	Fabio Estevam <festevam@gmail.com>, Abel Vesa <abel.vesa@nxp.com>,
	Leonard Crestez <leonard.crestez@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>, Li Yang <leoyang.li@nxp.com>,
	arm@kernel.org, linux-imx@nxp.com, kernel@pengutronix.de,
	Fabio Estevam <festevam@gmail.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [GIT PULL 6/6] i.MX defconfig update for 5.2
Date: Tue, 30 Apr 2019 10:26:38 +0800	[thread overview]
Message-ID: <20190430022636.GA25377@dragon> (raw)
In-Reply-To: <20190429162910.l27ewbfvvestb474@localhost>

On Mon, Apr 29, 2019 at 09:29:10AM -0700, Olof Johansson wrote:
> On Mon, Apr 22, 2019 at 04:28:36PM +0800, Shawn Guo wrote:
> > The following changes since commit 9e98c678c2d6ae3a17cb2de55d17f69dddaa231b:
> > 
> >   Linux 5.1-rc1 (2019-03-17 14:22:26 -0700)
> > 
> > are available in the Git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git tags/imx-defconfig-5.2
> > 
> > for you to fetch changes up to a3f12aac21848796f5b5c6e9ce2f04e411ec2129:
> > 
> >   arm64: defconfig: NVMEM_IMX_OCOTP=y for imx8m (2019-04-22 10:33:53 +0800)
> > 
> > ----------------------------------------------------------------
> > i.MX defconfig update for 5.2:
> >  - Enable PCF857X GPIO expander and SIOX bus driver support for
> >    imx_v6_v7_defconfig.
> >  - Enable more drivers for i.MX8 platform support in arm64 defconfig:
> >    SNVS RTC, SCU Watchdog, SPI, i.MX8MM pinctrl and clock, LPI2C,
> >    ROHM_BD718XX PMIC and OCOTP NVMEM.
> 
> All of these are enabled as =y. I don't think all of them are critical for
> booting the system up to a rootfs. Can you please switch over any non-critical
> drivers to =m?

Okay, let's check them one by one.

- CONFIG_SPI_IMX=y

  @Fabio, do you have a critical use case for this to be 'y'?  Or can we
  change this to 'm'?

- CONFIG_RTC_DRV_SNVS=y

  @Abel, I guess this can be 'm'? Or you have a good reason for this to
  be 'y'?

- CONFIG_PINCTRL_IMX8MM=y, CONFIG_CLK_IMX8MM=y

  @Olof, I think we shouldn't have problem to have pinctrl and clock
  driver to be 'y', as they are pretty fundamental for a platform
  support.

- CONFIG_GPIO_MAX732X=y

  @Olof, this is essentially a GPIO controller, just like
  CONFIG_GPIO_PCA953X=y currently sitting in defconfig.

- CONFIG_I2C_IMX_LPI2C=y

  @Olof, this is a driver for I2C bus controller which is used for many
   I2C devices, and above GPIO expander is one example.  So we would
   treat it as fundamental one.

- CONFIG_MFD_ROHM_BD718XX=y, CONFIG_REGULATOR_BD718XX=y

  @Leonard, are they critical for booting imx8mm-evk board up? Or can
  they be installed as module after system is up?

- CONFIG_NVMEM_IMX_OCOTP=y

  @Olof, this is the driver for fuse access on i.MX platform.  Drivers
  like cpufreq will need to access fuse for cpu grade data.  So we would
  take this driver as fundamental infrastructural.

Shawn

> 
> (Not merged)

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel