mbox series

[GIT,PULL,7/7] i.MX defconfig changes for 5.3

Message ID 20190625075305.29082-7-shawnguo@kernel.org
State New
Headers show
Series [GIT,PULL,1/7] i.MX drivers change for 5.3 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git tags/imx-defconfig-5.3

Message

Shawn Guo June 25, 2019, 7:53 a.m. UTC
Hi,

The change set is a little bigger than expected.  This is because that
half of the changes were for 5.2, but unfortunately they missed the
merge window.  So we have changes for two development cycles combined
here.  Please pull, thanks!

Shawn


The following changes since commit a188339ca5a396acc588e5851ed7e19f66b0ebd9:

  Linux 5.2-rc1 (2019-05-19 15:47:09 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git tags/imx-defconfig-5.3

for you to fetch changes up to 8bbc2b3a7f80b61c05b87606c2acd82a62d304b0:

  arm64: defconfig: Enable CONFIG_KEYBOARD_SNVS_PWRKEY as module (2019-06-25 15:15:59 +0800)

----------------------------------------------------------------
i.MX defconfig changes for 5.3:
 * imx_v6_v7_defconfig:
   - PCF857X GPIO expander
   - SIOX bus driver
   - thermal statistics
   - TPM PWM driver
   - OV2680 camera driver
   - SNVS LPGPR NVMEM driver
   - i.MX DT based cpufreq driver
 * arm64 defconfig built-in:
   - i.MX8MM pinctrl and clock
   - i.MX LPI2C driver
   - ROHM_BD718XX PMIC
   - OCOTP NVMEM support
   - i.MX SCU based SoC bus driver
 * arm64 defconfig modules:
   - i.MX SPI driver
   - i.MX system controller watchdog
   - SNVS RTC driver
   - ISL29018 light and proximity sensor driver
   - MPL3115 pressure sensor driver
   - i.MX8 DT based cpufreq support
   - QorIQ Thermal Monitoring Unit driver
   - SNVS power key driver

----------------------------------------------------------------
Abel Vesa (1):
      arm64: defconfig: Enable RTC_DRV_SNVS

Adam Ford (1):
      ARM: imx_v6_v7_defconfig: Add GPIO_PCF857X

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
      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
      arm64: defconfig: ARM_IMX_CPUFREQ_DT=m
      ARM: imx_v6_v7_defconfig: Enable CONFIG_ARM_IMX_CPUFREQ_DT

Peng Fan (1):
      defconfig: arm64: enable i.MX8 SCU octop driver

Uwe Kleine-König (1):
      ARM: imx_v6_v7_defconfig: Enable SIOX bus

 arch/arm/configs/imx_v6_v7_defconfig |  9 +++++++++
 arch/arm64/configs/defconfig         | 17 +++++++++++++++++
 2 files changed, 26 insertions(+)

Comments

Olof Johansson June 25, 2019, 12:32 p.m. UTC | #1
On Tue, Jun 25, 2019 at 03:53:05PM +0800, Shawn Guo wrote:
> Hi,
> 
> The change set is a little bigger than expected.  This is because that
> half of the changes were for 5.2, but unfortunately they missed the
> merge window.  So we have changes for two development cycles combined
> here.  Please pull, thanks!
> 
> Shawn
> 
> 
> The following changes since commit a188339ca5a396acc588e5851ed7e19f66b0ebd9:
> 
>   Linux 5.2-rc1 (2019-05-19 15:47:09 -0700)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git tags/imx-defconfig-5.3
> 
> for you to fetch changes up to 8bbc2b3a7f80b61c05b87606c2acd82a62d304b0:
> 
>   arm64: defconfig: Enable CONFIG_KEYBOARD_SNVS_PWRKEY as module (2019-06-25 15:15:59 +0800)
> 
> ----------------------------------------------------------------
> i.MX defconfig changes for 5.3:
>  * imx_v6_v7_defconfig:
>    - PCF857X GPIO expander
>    - SIOX bus driver
>    - thermal statistics
>    - TPM PWM driver
>    - OV2680 camera driver
>    - SNVS LPGPR NVMEM driver
>    - i.MX DT based cpufreq driver
>  * arm64 defconfig built-in:
>    - i.MX8MM pinctrl and clock
>    - i.MX LPI2C driver
>    - ROHM_BD718XX PMIC
>    - OCOTP NVMEM support
>    - i.MX SCU based SoC bus driver
>  * arm64 defconfig modules:
>    - i.MX SPI driver
>    - i.MX system controller watchdog
>    - SNVS RTC driver
>    - ISL29018 light and proximity sensor driver
>    - MPL3115 pressure sensor driver
>    - i.MX8 DT based cpufreq support
>    - QorIQ Thermal Monitoring Unit driver
>    - SNVS power key driver
> 
> ----------------------------------------------------------------
> Abel Vesa (1):
>       arm64: defconfig: Enable RTC_DRV_SNVS
> 
> Adam Ford (1):
>       ARM: imx_v6_v7_defconfig: Add GPIO_PCF857X
> 
> 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.


>       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.

>       arm64: defconfig: ARM_IMX_CPUFREQ_DT=m
>       ARM: imx_v6_v7_defconfig: Enable CONFIG_ARM_IMX_CPUFREQ_DT
> 
> Peng Fan (1):
>       defconfig: arm64: enable i.MX8 SCU octop driver
> 
> Uwe Kleine-König (1):
>       ARM: imx_v6_v7_defconfig: Enable SIOX bus
> 
>  arch/arm/configs/imx_v6_v7_defconfig |  9 +++++++++
>  arch/arm64/configs/defconfig         | 17 +++++++++++++++++
>  2 files changed, 26 insertions(+)


Not merged until above is clarified.


Thanks!


-Olof
Shawn Guo June 26, 2019, 2:02 a.m. UTC | #2
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
Olof Johansson June 27, 2019, 2:14 a.m. UTC | #3
On Wed, Jun 26, 2019 at 10:02 AM Shawn Guo <shawnguo@kernel.org> wrote:
>
> 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.

Ah, yes of course. But don't we usually use compatible for this?

> >
> >
> > >       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.

Having hardware self-describe is nice, not having to populate DT
entries from the bootloader.

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

Thanks!


-Olof
Shawn Guo June 27, 2019, 1:25 p.m. UTC | #4
On Thu, Jun 27, 2019 at 10:14:25AM +0800, Olof Johansson wrote:
> On Wed, Jun 26, 2019 at 10:02 AM Shawn Guo <shawnguo@kernel.org> wrote:
> >
> > 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.
> 
> Ah, yes of course. But don't we usually use compatible for this?

Yes, we do.  But soc_device_match() is now preferred over compatible,
since it can even identify the SoC revision while compatible usually
cannot.

Shawn