Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
On 26/11/2015 14:08, Marek Vasut wrote: > The following patch changed the PFUZE100 swbst register bit definitions > and broke PMIC configuration on multiple boards, at least on the novena > and gw_ventana. This patch fixes it. > > commit 8fa46350a4c7dca7710362f6c871098557b934ad > Author: Peng Fan > Date: Fri Aug 7 16:43:45 2015 +0800 > > power: regulator: add pfuze100 support > > Signed-off-by: Marek Vasut > Cc: Fabio Estevam > Cc: Peng Fan > Cc: Przemyslaw Marczak > Cc: Stefano Babic > Cc: Tim Harvey > Cc: Vagrant Cascadian > --- Applied to u-boot-imx, thanks! Best regards, Stefano Babic -- = DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
On Thu, Nov 26, 2015 at 5:08 AM, Marek Vasut wrote: > The following patch changed the PFUZE100 swbst register bit definitions > and broke PMIC configuration on multiple boards, at least on the novena > and gw_ventana. This patch fixes it. > > commit 8fa46350a4c7dca7710362f6c871098557b934ad > Author: Peng Fan > Date: Fri Aug 7 16:43:45 2015 +0800 > > power: regulator: add pfuze100 support > > Signed-off-by: Marek Vasut > Cc: Fabio Estevam > Cc: Peng Fan > Cc: Przemyslaw Marczak > Cc: Stefano Babic > Cc: Tim Harvey > Cc: Vagrant Cascadian > --- > board/gateworks/gw_ventana/common.c | 2 +- > board/kosagi/novena/novena.c| 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/board/gateworks/gw_ventana/common.c > b/board/gateworks/gw_ventana/common.c > index d406c83..a20190e 100644 > --- a/board/gateworks/gw_ventana/common.c > +++ b/board/gateworks/gw_ventana/common.c > @@ -806,7 +806,7 @@ void setup_pmic(void) > /* Set SWBST to 5.0V and enable */ > pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); > reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); > - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); > + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << > SWBST_MODE_SHIFT)); > pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); > } > } > diff --git a/board/kosagi/novena/novena.c b/board/kosagi/novena/novena.c > index babba85..b3159d3 100644 > --- a/board/kosagi/novena/novena.c > +++ b/board/kosagi/novena/novena.c > @@ -216,7 +216,7 @@ int power_init_board(void) > /* Set SWBST to 5.0V and enable (for USB) */ > pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); > reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); > - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); > + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); > pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); > > return 0; > -- > 2.1.4 > Acked-by: Tim Harvey Thanks Marek! Tim ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
On 11/30/2015 11:00 AM, Stefano Babic wrote: On 30/11/2015 10:53, Przemyslaw Marczak wrote: Hello Stefano, On 11/26/2015 02:08 PM, Marek Vasut wrote: The following patch changed the PFUZE100 swbst register bit definitions and broke PMIC configuration on multiple boards, at least on the novena and gw_ventana. This patch fixes it. commit 8fa46350a4c7dca7710362f6c871098557b934ad Author: Peng Fan Date: Fri Aug 7 16:43:45 2015 +0800 power: regulator: add pfuze100 support Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Peng Fan Cc: Przemyslaw Marczak Cc: Stefano Babic Cc: Tim Harvey Cc: Vagrant Cascadian --- board/gateworks/gw_ventana/common.c | 2 +- board/kosagi/novena/novena.c| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) Will you pick-up this patch, or should it go through PMIC tree? They are i.MXes, I pick them up. Best regards, Stefano Ok, thank you. Best regards, -- Przemyslaw Marczak Samsung R&D Institute Poland Samsung Electronics p.marc...@samsung.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
On 30/11/2015 10:53, Przemyslaw Marczak wrote: > Hello Stefano, > > On 11/26/2015 02:08 PM, Marek Vasut wrote: >> The following patch changed the PFUZE100 swbst register bit definitions >> and broke PMIC configuration on multiple boards, at least on the novena >> and gw_ventana. This patch fixes it. >> >> commit 8fa46350a4c7dca7710362f6c871098557b934ad >> Author: Peng Fan >> Date: Fri Aug 7 16:43:45 2015 +0800 >> >> power: regulator: add pfuze100 support >> >> Signed-off-by: Marek Vasut >> Cc: Fabio Estevam >> Cc: Peng Fan >> Cc: Przemyslaw Marczak >> Cc: Stefano Babic >> Cc: Tim Harvey >> Cc: Vagrant Cascadian >> --- >> board/gateworks/gw_ventana/common.c | 2 +- >> board/kosagi/novena/novena.c| 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) > > Will you pick-up this patch, or should it go through PMIC tree? They are i.MXes, I pick them up. Best regards, Stefano -- = DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
Hello Stefano, On 11/26/2015 02:08 PM, Marek Vasut wrote: The following patch changed the PFUZE100 swbst register bit definitions and broke PMIC configuration on multiple boards, at least on the novena and gw_ventana. This patch fixes it. commit 8fa46350a4c7dca7710362f6c871098557b934ad Author: Peng Fan Date: Fri Aug 7 16:43:45 2015 +0800 power: regulator: add pfuze100 support Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Peng Fan Cc: Przemyslaw Marczak Cc: Stefano Babic Cc: Tim Harvey Cc: Vagrant Cascadian --- board/gateworks/gw_ventana/common.c | 2 +- board/kosagi/novena/novena.c| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) Will you pick-up this patch, or should it go through PMIC tree? Best regards, -- Przemyslaw Marczak Samsung R&D Institute Poland Samsung Electronics p.marc...@samsung.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
On Thu, Nov 26, 2015 at 02:08:50PM +0100, Marek Vasut wrote: >The following patch changed the PFUZE100 swbst register bit definitions >and broke PMIC configuration on multiple boards, at least on the novena >and gw_ventana. This patch fixes it. > >commit 8fa46350a4c7dca7710362f6c871098557b934ad >Author: Peng Fan >Date: Fri Aug 7 16:43:45 2015 +0800 > >power: regulator: add pfuze100 support > >Signed-off-by: Marek Vasut >Cc: Fabio Estevam >Cc: Peng Fan >Cc: Przemyslaw Marczak >Cc: Stefano Babic >Cc: Tim Harvey >Cc: Vagrant Cascadian >--- > board/gateworks/gw_ventana/common.c | 2 +- > board/kosagi/novena/novena.c| 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > >diff --git a/board/gateworks/gw_ventana/common.c >b/board/gateworks/gw_ventana/common.c >index d406c83..a20190e 100644 >--- a/board/gateworks/gw_ventana/common.c >+++ b/board/gateworks/gw_ventana/common.c >@@ -806,7 +806,7 @@ void setup_pmic(void) > /* Set SWBST to 5.0V and enable */ > pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); > reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); >- reg |= (SWBST_5_00V | SWBST_MODE_AUTO); >+ reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << >SWBST_MODE_SHIFT)); > pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); > } > } >diff --git a/board/kosagi/novena/novena.c b/board/kosagi/novena/novena.c >index babba85..b3159d3 100644 >--- a/board/kosagi/novena/novena.c >+++ b/board/kosagi/novena/novena.c >@@ -216,7 +216,7 @@ int power_init_board(void) > /* Set SWBST to 5.0V and enable (for USB) */ > pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); > reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); >- reg |= (SWBST_5_00V | SWBST_MODE_AUTO); >+ reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); > pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); > > return 0; Reviewed-by: Peng Fan >-- >2.1.4 > -- ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
On 2015-11-26, Marek Vasut wrote: > The following patch changed the PFUZE100 swbst register bit definitions > and broke PMIC configuration on multiple boards, at least on the novena > and gw_ventana. This patch fixes it. Works on novena. Tested-by: Vagrant Cascadian live well, vagrant > commit 8fa46350a4c7dca7710362f6c871098557b934ad > Author: Peng Fan > Date: Fri Aug 7 16:43:45 2015 +0800 > > power: regulator: add pfuze100 support > > Signed-off-by: Marek Vasut > Cc: Fabio Estevam > Cc: Peng Fan > Cc: Przemyslaw Marczak > Cc: Stefano Babic > Cc: Tim Harvey > Cc: Vagrant Cascadian > --- > board/gateworks/gw_ventana/common.c | 2 +- > board/kosagi/novena/novena.c| 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/board/gateworks/gw_ventana/common.c > b/board/gateworks/gw_ventana/common.c > index d406c83..a20190e 100644 > --- a/board/gateworks/gw_ventana/common.c > +++ b/board/gateworks/gw_ventana/common.c > @@ -806,7 +806,7 @@ void setup_pmic(void) > /* Set SWBST to 5.0V and enable */ > pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); > reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); > - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); > + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << > SWBST_MODE_SHIFT)); > pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); > } > } > diff --git a/board/kosagi/novena/novena.c b/board/kosagi/novena/novena.c > index babba85..b3159d3 100644 > --- a/board/kosagi/novena/novena.c > +++ b/board/kosagi/novena/novena.c > @@ -216,7 +216,7 @@ int power_init_board(void) > /* Set SWBST to 5.0V and enable (for USB) */ > pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); > reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); > - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); > + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); > pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); > > return 0; > -- > 2.1.4 signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
Hello Marek, On 11/26/2015 02:08 PM, Marek Vasut wrote: The following patch changed the PFUZE100 swbst register bit definitions and broke PMIC configuration on multiple boards, at least on the novena and gw_ventana. This patch fixes it. commit 8fa46350a4c7dca7710362f6c871098557b934ad Author: Peng Fan Date: Fri Aug 7 16:43:45 2015 +0800 power: regulator: add pfuze100 support Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Peng Fan Cc: Przemyslaw Marczak Cc: Stefano Babic Cc: Tim Harvey Cc: Vagrant Cascadian --- board/gateworks/gw_ventana/common.c | 2 +- board/kosagi/novena/novena.c| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/board/gateworks/gw_ventana/common.c b/board/gateworks/gw_ventana/common.c index d406c83..a20190e 100644 --- a/board/gateworks/gw_ventana/common.c +++ b/board/gateworks/gw_ventana/common.c @@ -806,7 +806,7 @@ void setup_pmic(void) /* Set SWBST to 5.0V and enable */ pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); } } diff --git a/board/kosagi/novena/novena.c b/board/kosagi/novena/novena.c index babba85..b3159d3 100644 --- a/board/kosagi/novena/novena.c +++ b/board/kosagi/novena/novena.c @@ -216,7 +216,7 @@ int power_init_board(void) /* Set SWBST to 5.0V and enable (for USB) */ pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); return 0; Thanks for fixing :) Reviewed-by: Przemyslaw Marczak Best regards, -- Przemyslaw Marczak Samsung R&D Institute Poland Samsung Electronics p.marc...@samsung.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] arm: imx6: novena, gw_ventana: Fix use of pfuze100 bit definitions
The following patch changed the PFUZE100 swbst register bit definitions and broke PMIC configuration on multiple boards, at least on the novena and gw_ventana. This patch fixes it. commit 8fa46350a4c7dca7710362f6c871098557b934ad Author: Peng Fan Date: Fri Aug 7 16:43:45 2015 +0800 power: regulator: add pfuze100 support Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Peng Fan Cc: Przemyslaw Marczak Cc: Stefano Babic Cc: Tim Harvey Cc: Vagrant Cascadian --- board/gateworks/gw_ventana/common.c | 2 +- board/kosagi/novena/novena.c| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/board/gateworks/gw_ventana/common.c b/board/gateworks/gw_ventana/common.c index d406c83..a20190e 100644 --- a/board/gateworks/gw_ventana/common.c +++ b/board/gateworks/gw_ventana/common.c @@ -806,7 +806,7 @@ void setup_pmic(void) /* Set SWBST to 5.0V and enable */ pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); } } diff --git a/board/kosagi/novena/novena.c b/board/kosagi/novena/novena.c index babba85..b3159d3 100644 --- a/board/kosagi/novena/novena.c +++ b/board/kosagi/novena/novena.c @@ -216,7 +216,7 @@ int power_init_board(void) /* Set SWBST to 5.0V and enable (for USB) */ pmic_reg_read(p, PFUZE100_SWBSTCON1, ®); reg &= ~(SWBST_MODE_MASK | SWBST_VOL_MASK); - reg |= (SWBST_5_00V | SWBST_MODE_AUTO); + reg |= (SWBST_5_00V | (SWBST_MODE_AUTO << SWBST_MODE_SHIFT)); pmic_reg_write(p, PFUZE100_SWBSTCON1, reg); return 0; -- 2.1.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot