Message ID | 20210805135035.124938-1-pzimmermann@dh-electronics.com |
---|---|
State | New |
Headers | show |
Series | ARM: imx_v6_v7_defconfig: enable driver of the LTC3676 PMIC | expand |
On 8/5/21 3:50 PM, Pascal Zimmermann wrote: > The LTC3676 is a PMIC which is used on some i.MX6 based boards (like the > DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, enable > LTC3676 driver in imx_v6_v7_defconfig. > > Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") > Signed-off-by: Pascal Zimmermann <pzimmermann@dh-electronics.com> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: Sascha Hauer <s.hauer@pengutronix.de> > Cc: Pengutronix Kernel Team <kernel@pengutronix.de> > Cc: Fabio Estevam <festevam@denx.de> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: Marek Vasut <marex@denx.de> > Cc: kernel@dh-electronics.com > To: linux-arm-kernel@lists.infradead.org > --- > arch/arm/configs/imx_v6_v7_defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig > index 079fcd8d1d11..f94e3d11c3e9 100644 > --- a/arch/arm/configs/imx_v6_v7_defconfig > +++ b/arch/arm/configs/imx_v6_v7_defconfig > @@ -252,6 +252,7 @@ CONFIG_REGULATOR_DA9052=y > CONFIG_REGULATOR_DA9062=y > CONFIG_REGULATOR_DA9063=y > CONFIG_REGULATOR_GPIO=y > +CONFIG_REGULATOR_LTC3676=y > CONFIG_REGULATOR_MC13783=y > CONFIG_REGULATOR_MC13892=y > CONFIG_REGULATOR_PFUZE100=y Reviewed-by: Marek Vasut <marex@denx.de> Tested-by: Marek Vasut <marex@denx.de> # On DH iMX6Q DHCOM PDK2 Neither board can work well without a PMIC driver.
On Tue, Aug 10, 2021 at 07:36:24AM +0200, Marek Vasut wrote: > On 8/5/21 3:50 PM, Pascal Zimmermann wrote: > > The LTC3676 is a PMIC which is used on some i.MX6 based boards (like the > > DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, enable > > LTC3676 driver in imx_v6_v7_defconfig. > > > > Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") I don't think the Fixes tag is appropriate here. Dropped it and applied patch. Shawn > > Signed-off-by: Pascal Zimmermann <pzimmermann@dh-electronics.com> > > Cc: Shawn Guo <shawnguo@kernel.org> > > Cc: Sascha Hauer <s.hauer@pengutronix.de> > > Cc: Pengutronix Kernel Team <kernel@pengutronix.de> > > Cc: Fabio Estevam <festevam@denx.de> > > Cc: NXP Linux Team <linux-imx@nxp.com> > > Cc: Marek Vasut <marex@denx.de> > > Cc: kernel@dh-electronics.com > > To: linux-arm-kernel@lists.infradead.org > > --- > > arch/arm/configs/imx_v6_v7_defconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig > > index 079fcd8d1d11..f94e3d11c3e9 100644 > > --- a/arch/arm/configs/imx_v6_v7_defconfig > > +++ b/arch/arm/configs/imx_v6_v7_defconfig > > @@ -252,6 +252,7 @@ CONFIG_REGULATOR_DA9052=y > > CONFIG_REGULATOR_DA9062=y > > CONFIG_REGULATOR_DA9063=y > > CONFIG_REGULATOR_GPIO=y > > +CONFIG_REGULATOR_LTC3676=y > > CONFIG_REGULATOR_MC13783=y > > CONFIG_REGULATOR_MC13892=y > > CONFIG_REGULATOR_PFUZE100=y > > Reviewed-by: Marek Vasut <marex@denx.de> > Tested-by: Marek Vasut <marex@denx.de> # On DH iMX6Q DHCOM PDK2 > > Neither board can work well without a PMIC driver.
On 8/14/21 7:40 AM, Shawn Guo wrote: > On Tue, Aug 10, 2021 at 07:36:24AM +0200, Marek Vasut wrote: >> On 8/5/21 3:50 PM, Pascal Zimmermann wrote: >>> The LTC3676 is a PMIC which is used on some i.MX6 based boards (like the >>> DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, enable >>> LTC3676 driver in imx_v6_v7_defconfig. >>> >>> Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") > > I don't think the Fixes tag is appropriate here. Dropped it and applied > patch. Can you please elaborate why do you think the Fixes tag is not appropriate ? Neither system can work properly without the PMIC driver, so I believe the Fixes tag is correct and this should be backported to older LTS kernel releases. [...] >>> arch/arm/configs/imx_v6_v7_defconfig | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig >>> index 079fcd8d1d11..f94e3d11c3e9 100644 >>> --- a/arch/arm/configs/imx_v6_v7_defconfig >>> +++ b/arch/arm/configs/imx_v6_v7_defconfig >>> @@ -252,6 +252,7 @@ CONFIG_REGULATOR_DA9052=y >>> CONFIG_REGULATOR_DA9062=y >>> CONFIG_REGULATOR_DA9063=y >>> CONFIG_REGULATOR_GPIO=y >>> +CONFIG_REGULATOR_LTC3676=y >>> CONFIG_REGULATOR_MC13783=y >>> CONFIG_REGULATOR_MC13892=y >>> CONFIG_REGULATOR_PFUZE100=y >> >> Reviewed-by: Marek Vasut <marex@denx.de> >> Tested-by: Marek Vasut <marex@denx.de> # On DH iMX6Q DHCOM PDK2 >> >> Neither board can work well without a PMIC driver. [...]
On Sat, Aug 14, 2021 at 09:45:03AM +0200, Marek Vasut wrote: > On 8/14/21 7:40 AM, Shawn Guo wrote: > > On Tue, Aug 10, 2021 at 07:36:24AM +0200, Marek Vasut wrote: > > > On 8/5/21 3:50 PM, Pascal Zimmermann wrote: > > > > The LTC3676 is a PMIC which is used on some i.MX6 based boards (like the > > > > DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, enable > > > > LTC3676 driver in imx_v6_v7_defconfig. > > > > > > > > Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") > > > > I don't think the Fixes tag is appropriate here. Dropped it and applied > > patch. > > Can you please elaborate why do you think the Fixes tag is not appropriate ? > > Neither system can work properly without the PMIC driver, so I believe the > Fixes tag is correct and this should be backported to older LTS kernel > releases. It's enabling a feature (driver) rather than fixing a bug or regression. If you want to enable it for LTS kernel, go ahead to back port it. Do not abuse the Fixes tag. Shawn
On 8/14/21 1:24 PM, Shawn Guo wrote: > On Sat, Aug 14, 2021 at 09:45:03AM +0200, Marek Vasut wrote: >> On 8/14/21 7:40 AM, Shawn Guo wrote: >>> On Tue, Aug 10, 2021 at 07:36:24AM +0200, Marek Vasut wrote: >>>> On 8/5/21 3:50 PM, Pascal Zimmermann wrote: >>>>> The LTC3676 is a PMIC which is used on some i.MX6 based boards (like the >>>>> DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, enable >>>>> LTC3676 driver in imx_v6_v7_defconfig. >>>>> >>>>> Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") >>> >>> I don't think the Fixes tag is appropriate here. Dropped it and applied >>> patch. >> >> Can you please elaborate why do you think the Fixes tag is not appropriate ? >> >> Neither system can work properly without the PMIC driver, so I believe the >> Fixes tag is correct and this should be backported to older LTS kernel >> releases. > > It's enabling a feature (driver) rather than fixing a bug or regression. > If you want to enable it for LTS kernel, go ahead to back port it. > Do not abuse the Fixes tag. I disagree with this. Without the PMIC driver, the system is basically unusable. That is because the internal regulators in the MX6 depend on the PMIC, so those fail to probe, and in turn all the other components which depend on those internal regulators also fail to probe (that is everything which depends on anatop regulators). If I take linux-stable, compile it for a system that is supported in there, with the imx_v6_v7_defconfig which is part of it, I would expect the result to work properly, not just barely boot and be barely usable. Therefore, I believe the Fixes tag here is justified.
On 8/14/21 4:31 PM, Marek Vasut wrote: > On 8/14/21 1:24 PM, Shawn Guo wrote: >> On Sat, Aug 14, 2021 at 09:45:03AM +0200, Marek Vasut wrote: >>> On 8/14/21 7:40 AM, Shawn Guo wrote: >>>> On Tue, Aug 10, 2021 at 07:36:24AM +0200, Marek Vasut wrote: >>>>> On 8/5/21 3:50 PM, Pascal Zimmermann wrote: >>>>>> The LTC3676 is a PMIC which is used on some i.MX6 based boards >>>>>> (like the >>>>>> DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, >>>>>> enable >>>>>> LTC3676 driver in imx_v6_v7_defconfig. >>>>>> >>>>>> Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") >>>> >>>> I don't think the Fixes tag is appropriate here. Dropped it and >>>> applied >>>> patch. >>> >>> Can you please elaborate why do you think the Fixes tag is not >>> appropriate ? >>> >>> Neither system can work properly without the PMIC driver, so I >>> believe the >>> Fixes tag is correct and this should be backported to older LTS kernel >>> releases. >> >> It's enabling a feature (driver) rather than fixing a bug or regression. >> If you want to enable it for LTS kernel, go ahead to back port it. >> Do not abuse the Fixes tag. > > I disagree with this. Without the PMIC driver, the system is basically > unusable. That is because the internal regulators in the MX6 depend on > the PMIC, so those fail to probe, and in turn all the other components > which depend on those internal regulators also fail to probe (that is > everything which depends on anatop regulators). > > If I take linux-stable, compile it for a system that is supported in > there, with the imx_v6_v7_defconfig which is part of it, I would expect > the result to work properly, not just barely boot and be barely usable. > > Therefore, I believe the Fixes tag here is justified. I am curious, is there any further clarification ?
diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig index 079fcd8d1d11..f94e3d11c3e9 100644 --- a/arch/arm/configs/imx_v6_v7_defconfig +++ b/arch/arm/configs/imx_v6_v7_defconfig @@ -252,6 +252,7 @@ CONFIG_REGULATOR_DA9052=y CONFIG_REGULATOR_DA9062=y CONFIG_REGULATOR_DA9063=y CONFIG_REGULATOR_GPIO=y +CONFIG_REGULATOR_LTC3676=y CONFIG_REGULATOR_MC13783=y CONFIG_REGULATOR_MC13892=y CONFIG_REGULATOR_PFUZE100=y
The LTC3676 is a PMIC which is used on some i.MX6 based boards (like the DHCOM i.MX6 Quad SoM), it was first used on the GW Ventana board, enable LTC3676 driver in imx_v6_v7_defconfig. Fixes: e3946fe805053 ("ARM: dts: add Gateworks Ventana support") Signed-off-by: Pascal Zimmermann <pzimmermann@dh-electronics.com> Cc: Shawn Guo <shawnguo@kernel.org> Cc: Sascha Hauer <s.hauer@pengutronix.de> Cc: Pengutronix Kernel Team <kernel@pengutronix.de> Cc: Fabio Estevam <festevam@denx.de> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: Marek Vasut <marex@denx.de> Cc: kernel@dh-electronics.com To: linux-arm-kernel@lists.infradead.org --- arch/arm/configs/imx_v6_v7_defconfig | 1 + 1 file changed, 1 insertion(+)