Message ID | 1523895110-19899-1-git-send-email-mark.jonas@de.bosch.com |
---|---|
State | Accepted |
Commit | 5ebf9ad96c31d88b6dedc6dd1efa043fd882d044 |
Delegated to: | Stefano Babic |
Headers | show |
Series | [U-Boot] arm,imx6: fix PAD_CTL_SPEED_LOW constant | expand |
Hello Mark, Am 16.04.2018 um 18:11 schrieb Mark Jonas: > For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. > Only the i.MX6 SoloLite is an exemption. So far the code did not > consider that. Additionally, for a few i.MX6 processors the code used > the wrong value for the constant. > > This patch fixes the PAD_CTL_SPEED_LOW constant for: > - i.MX6 Solo [1] > - i.MX6 DualLite [1] > - i.MX6 Dual [2] > - i.MX6 Quad [2] > - i.MX6 DualPlus [3] > - i.MX6 QuadPlus [3] > > Before, it was already correct for: > - i.MX6 SoloLite [4] > - i.MX6 SoloX [5] > - i.MX6 UtraLite [6] > - i.MX6 ULL [7] > > [1] https://www.nxp.com/docs/en/reference-manual/IMX6SDLRM.pdf > [2] https://www.nxp.com/docs/en/reference-manual/IMX6DQRM.pdf > [3] https://www.nxp.com/docs/en/reference-manual/iMX6DQPRM.pdf > [4] https://www.nxp.com/docs/en/reference-manual/IMX6SLRM.pdf > [5] https://www.nxp.com/docs/en/reference-manual/IMX6SXRM.pdf > [6] https://www.nxp.com/docs/en/reference-manual/IMX6ULRM.pdf > [7] https://www.nxp.com/docs/en/reference-manual/IMX6ULLRM.pdf > > Signed-off-by: Mark Jonas <mark.jonas@de.bosch.com> > --- > arch/arm/include/asm/mach-imx/iomux-v3.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Good catch, thanks! Reviewed-by: Heiko Schocher <hs@denx.de> bye, Heiko
Hello, > > For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. > > Only the i.MX6 SoloLite is an exemption. So far the code did not > > consider that. Additionally, for a few i.MX6 processors the code used > > the wrong value for the constant. [..] > Good catch, thanks! > > Reviewed-by: Heiko Schocher <hs@denx.de> I am wondering how this will continue now. Is there anything I still have to do? Regards, Mark Building Technologies, Panel Software Fire (BT-FIR/ENG1) Bosch Sicherheitssysteme GmbH | Postfach 11 11 | 85626 Grasbrunn | GERMANY | www.boschsecurity.com Sitz: Stuttgart, Registergericht: Amtsgericht Stuttgart HRB 23118 Aufsichtsratsvorsitzender: Stefan Hartung; Geschäftsführung: Gert van Iperen, Andreas Bartz, Thomas Quante, Bernhard Schuster
On 26/04/2018 08:44, Jonas Mark (BT-FIR/ENG1) wrote: > Hello, > >>> For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. >>> Only the i.MX6 SoloLite is an exemption. So far the code did not >>> consider that. Additionally, for a few i.MX6 processors the code used >>> the wrong value for the constant. > > [..] > >> Good catch, thanks! >> >> Reviewed-by: Heiko Schocher <hs@denx.de> > > I am wondering how this will continue now. Is there anything I still > have to do? You have not - I am quite busy and I know I have a long list of patches to apply. Please wait, I will take care of it in next days. Regards, Stefano
Hello Mark, Am 26.04.2018 um 08:44 schrieb Jonas Mark (BT-FIR/ENG1): > Hello, > >>> For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. >>> Only the i.MX6 SoloLite is an exemption. So far the code did not >>> consider that. Additionally, for a few i.MX6 processors the code used >>> the wrong value for the constant. > > [..] > >> Good catch, thanks! >> >> Reviewed-by: Heiko Schocher <hs@denx.de> > > I am wondering how this will continue now. Is there anything I still > have to do? No, just wait until Stefano has time to comment or pick up the patch. bye, Heiko > > Regards, > Mark > > Building Technologies, Panel Software Fire (BT-FIR/ENG1) > Bosch Sicherheitssysteme GmbH | Postfach 11 11 | 85626 Grasbrunn | GERMANY | www.boschsecurity.com > > Sitz: Stuttgart, Registergericht: Amtsgericht Stuttgart HRB 23118 > Aufsichtsratsvorsitzender: Stefan Hartung; Geschäftsführung: Gert van Iperen, Andreas Bartz, Thomas Quante, Bernhard Schuster >
On 26/04/2018 08:50, Heiko Schocher wrote: > Hello Mark, > > Am 26.04.2018 um 08:44 schrieb Jonas Mark (BT-FIR/ENG1): >> Hello, >> >>>> For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. >>>> Only the i.MX6 SoloLite is an exemption. So far the code did not >>>> consider that. Additionally, for a few i.MX6 processors the code used >>>> the wrong value for the constant. >> >> [..] >> >>> Good catch, thanks! >>> >>> Reviewed-by: Heiko Schocher <hs@denx.de> >> >> I am wondering how this will continue now. Is there anything I still >> have to do? > > No, just wait until Stefano has time to comment or pick up the patch. > The second you said :-). I have already applied most of patches delegated to me, but I have still to run full build and check the rest of patches before pushing to server. Regards, Stefano
On 26/04/2018 08:44, Jonas Mark (BT-FIR/ENG1) wrote: > Hello, > >>> For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. >>> Only the i.MX6 SoloLite is an exemption. So far the code did not >>> consider that. Additionally, for a few i.MX6 processors the code used >>> the wrong value for the constant. > > [..] > >> Good catch, thanks! >> >> Reviewed-by: Heiko Schocher <hs@denx.de> > > I am wondering how this will continue now. Is there anything I still > have to do? > I did my homework: Applied to u-boot-imx, thanks ! Best regards, Stefano Babic
diff --git a/arch/arm/include/asm/mach-imx/iomux-v3.h b/arch/arm/include/asm/mach-imx/iomux-v3.h index 0c0ccf44..09094c9 100644 --- a/arch/arm/include/asm/mach-imx/iomux-v3.h +++ b/arch/arm/include/asm/mach-imx/iomux-v3.h @@ -147,10 +147,10 @@ typedef u64 iomux_v3_cfg_t; #define PAD_CTL_ODE (1 << 11) -#if defined(CONFIG_MX6SX) || defined(CONFIG_MX6UL) || defined(CONFIG_MX6ULL) -#define PAD_CTL_SPEED_LOW (0 << 6) -#else +#if defined(CONFIG_MX6SL) #define PAD_CTL_SPEED_LOW (1 << 6) +#else +#define PAD_CTL_SPEED_LOW (0 << 6) #endif #define PAD_CTL_SPEED_MED (2 << 6) #define PAD_CTL_SPEED_HIGH (3 << 6)
For most i.MX6 processors the PAD_CTL_SPEED_LOW constant is the same. Only the i.MX6 SoloLite is an exemption. So far the code did not consider that. Additionally, for a few i.MX6 processors the code used the wrong value for the constant. This patch fixes the PAD_CTL_SPEED_LOW constant for: - i.MX6 Solo [1] - i.MX6 DualLite [1] - i.MX6 Dual [2] - i.MX6 Quad [2] - i.MX6 DualPlus [3] - i.MX6 QuadPlus [3] Before, it was already correct for: - i.MX6 SoloLite [4] - i.MX6 SoloX [5] - i.MX6 UtraLite [6] - i.MX6 ULL [7] [1] https://www.nxp.com/docs/en/reference-manual/IMX6SDLRM.pdf [2] https://www.nxp.com/docs/en/reference-manual/IMX6DQRM.pdf [3] https://www.nxp.com/docs/en/reference-manual/iMX6DQPRM.pdf [4] https://www.nxp.com/docs/en/reference-manual/IMX6SLRM.pdf [5] https://www.nxp.com/docs/en/reference-manual/IMX6SXRM.pdf [6] https://www.nxp.com/docs/en/reference-manual/IMX6ULRM.pdf [7] https://www.nxp.com/docs/en/reference-manual/IMX6ULLRM.pdf Signed-off-by: Mark Jonas <mark.jonas@de.bosch.com> --- arch/arm/include/asm/mach-imx/iomux-v3.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)