Message ID | 20190625075305.29082-7-shawnguo@kernel.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [GIT,PULL,1/7] i.MX drivers change for 5.3 | expand |
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
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
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
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