* [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL
@ 2015-07-01 22:20 Paul Kocialkowski
2015-07-15 12:56 ` Paul Kocialkowski
2015-07-28 14:58 ` [U-Boot] " Tom Rini
0 siblings, 2 replies; 5+ messages in thread
From: Paul Kocialkowski @ 2015-07-01 22:20 UTC (permalink / raw)
To: u-boot
Palmas power support is required for OMAP5 devices such as the OMAP5 uEVM, that
need to e.g. enable MMC power at SPL stage.
This is especially important when booting from a peripheral (such as USB, UART),
where the bootrom will not enable power for the MMC device that will hold the
main U-Boot.
Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
---
include/configs/ti_armv7_common.h | 1 +
include/configs/ti_omap5_common.h | 2 --
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
index 0aea7d1..34756de 100644
--- a/include/configs/ti_armv7_common.h
+++ b/include/configs/ti_armv7_common.h
@@ -261,6 +261,7 @@
#define CONFIG_SPL_LIBCOMMON_SUPPORT
#define CONFIG_SPL_LIBGENERIC_SUPPORT
#define CONFIG_SPL_SERIAL_SUPPORT
+#define CONFIG_SPL_POWER_SUPPORT
#define CONFIG_SPL_GPIO_SUPPORT
#define CONFIG_SPL_BOARD_INIT
diff --git a/include/configs/ti_omap5_common.h b/include/configs/ti_omap5_common.h
index 4179a57..b33feb6 100644
--- a/include/configs/ti_omap5_common.h
+++ b/include/configs/ti_omap5_common.h
@@ -40,9 +40,7 @@
#define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
#endif
-#ifndef CONFIG_SPL_BUILD
#define CONFIG_PALMAS_POWER
-#endif
#include <asm/arch/cpu.h>
#include <asm/arch/omap.h>
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL
2015-07-01 22:20 [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL Paul Kocialkowski
@ 2015-07-15 12:56 ` Paul Kocialkowski
2015-07-16 13:21 ` Tom Rini
2015-07-28 14:58 ` [U-Boot] " Tom Rini
1 sibling, 1 reply; 5+ messages in thread
From: Paul Kocialkowski @ 2015-07-15 12:56 UTC (permalink / raw)
To: u-boot
Le jeudi 02 juillet 2015 ? 00:20 +0200, Paul Kocialkowski a ?crit :
> Palmas power support is required for OMAP5 devices such as the OMAP5 uEVM, that
> need to e.g. enable MMC power at SPL stage.
>
> This is especially important when booting from a peripheral (such as USB, UART),
> where the bootrom will not enable power for the MMC device that will hold the
> main U-Boot.
I sent this a while back and it hasn't been picked up yet. Anything
holding it back in particular?
> Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> ---
> include/configs/ti_armv7_common.h | 1 +
> include/configs/ti_omap5_common.h | 2 --
> 2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
> index 0aea7d1..34756de 100644
> --- a/include/configs/ti_armv7_common.h
> +++ b/include/configs/ti_armv7_common.h
> @@ -261,6 +261,7 @@
> #define CONFIG_SPL_LIBCOMMON_SUPPORT
> #define CONFIG_SPL_LIBGENERIC_SUPPORT
> #define CONFIG_SPL_SERIAL_SUPPORT
> +#define CONFIG_SPL_POWER_SUPPORT
> #define CONFIG_SPL_GPIO_SUPPORT
> #define CONFIG_SPL_BOARD_INIT
>
> diff --git a/include/configs/ti_omap5_common.h b/include/configs/ti_omap5_common.h
> index 4179a57..b33feb6 100644
> --- a/include/configs/ti_omap5_common.h
> +++ b/include/configs/ti_omap5_common.h
> @@ -40,9 +40,7 @@
> #define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
> #endif
>
> -#ifndef CONFIG_SPL_BUILD
> #define CONFIG_PALMAS_POWER
> -#endif
>
> #include <asm/arch/cpu.h>
> #include <asm/arch/omap.h>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150715/6f0fb42b/attachment.sig>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL
2015-07-15 12:56 ` Paul Kocialkowski
@ 2015-07-16 13:21 ` Tom Rini
2015-07-26 16:30 ` Paul Kocialkowski
0 siblings, 1 reply; 5+ messages in thread
From: Tom Rini @ 2015-07-16 13:21 UTC (permalink / raw)
To: u-boot
On Wed, Jul 15, 2015 at 02:56:23PM +0200, Paul Kocialkowski wrote:
> Le jeudi 02 juillet 2015 ? 00:20 +0200, Paul Kocialkowski a ?crit :
> > Palmas power support is required for OMAP5 devices such as the OMAP5 uEVM, that
> > need to e.g. enable MMC power at SPL stage.
> >
> > This is especially important when booting from a peripheral (such as USB, UART),
> > where the bootrom will not enable power for the MMC device that will hold the
> > main U-Boot.
>
> I sent this a while back and it hasn't been picked up yet. Anything
> holding it back in particular?
>
> > Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> > ---
> > include/configs/ti_armv7_common.h | 1 +
> > include/configs/ti_omap5_common.h | 2 --
> > 2 files changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
> > index 0aea7d1..34756de 100644
> > --- a/include/configs/ti_armv7_common.h
> > +++ b/include/configs/ti_armv7_common.h
> > @@ -261,6 +261,7 @@
> > #define CONFIG_SPL_LIBCOMMON_SUPPORT
> > #define CONFIG_SPL_LIBGENERIC_SUPPORT
> > #define CONFIG_SPL_SERIAL_SUPPORT
> > +#define CONFIG_SPL_POWER_SUPPORT
> > #define CONFIG_SPL_GPIO_SUPPORT
> > #define CONFIG_SPL_BOARD_INIT
> >
> > diff --git a/include/configs/ti_omap5_common.h b/include/configs/ti_omap5_common.h
> > index 4179a57..b33feb6 100644
> > --- a/include/configs/ti_omap5_common.h
> > +++ b/include/configs/ti_omap5_common.h
> > @@ -40,9 +40,7 @@
> > #define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
> > #endif
> >
> > -#ifndef CONFIG_SPL_BUILD
> > #define CONFIG_PALMAS_POWER
> > -#endif
> >
> > #include <asm/arch/cpu.h>
> > #include <asm/arch/omap.h>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150716/9cc980a6/attachment.sig>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL
2015-07-16 13:21 ` Tom Rini
@ 2015-07-26 16:30 ` Paul Kocialkowski
0 siblings, 0 replies; 5+ messages in thread
From: Paul Kocialkowski @ 2015-07-26 16:30 UTC (permalink / raw)
To: u-boot
Le jeudi 16 juillet 2015 ? 09:21 -0400, Tom Rini a ?crit :
> On Wed, Jul 15, 2015 at 02:56:23PM +0200, Paul Kocialkowski wrote:
> > Le jeudi 02 juillet 2015 ? 00:20 +0200, Paul Kocialkowski a ?crit :
> > > Palmas power support is required for OMAP5 devices such as the OMAP5 uEVM, that
> > > need to e.g. enable MMC power at SPL stage.
> > >
> > > This is especially important when booting from a peripheral (such as USB, UART),
> > > where the bootrom will not enable power for the MMC device that will hold the
> > > main U-Boot.
> >
> > I sent this a while back and it hasn't been picked up yet. Anything
> > holding it back in particular?
> >
> > > Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> > > ---
> > > include/configs/ti_armv7_common.h | 1 +
> > > include/configs/ti_omap5_common.h | 2 --
> > > 2 files changed, 1 insertion(+), 2 deletions(-)
> > >
> > > diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
> > > index 0aea7d1..34756de 100644
> > > --- a/include/configs/ti_armv7_common.h
> > > +++ b/include/configs/ti_armv7_common.h
> > > @@ -261,6 +261,7 @@
> > > #define CONFIG_SPL_LIBCOMMON_SUPPORT
> > > #define CONFIG_SPL_LIBGENERIC_SUPPORT
> > > #define CONFIG_SPL_SERIAL_SUPPORT
> > > +#define CONFIG_SPL_POWER_SUPPORT
> > > #define CONFIG_SPL_GPIO_SUPPORT
> > > #define CONFIG_SPL_BOARD_INIT
> > >
> > > diff --git a/include/configs/ti_omap5_common.h b/include/configs/ti_omap5_common.h
> > > index 4179a57..b33feb6 100644
> > > --- a/include/configs/ti_omap5_common.h
> > > +++ b/include/configs/ti_omap5_common.h
> > > @@ -40,9 +40,7 @@
> > > #define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
> > > #endif
> > >
> > > -#ifndef CONFIG_SPL_BUILD
> > > #define CONFIG_PALMAS_POWER
> > > -#endif
> > >
> > > #include <asm/arch/cpu.h>
> > > #include <asm/arch/omap.h>
>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Now that the merge window is open, I think we could merge that one.
What do you think?
Thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150726/911d0ecd/attachment.sig>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] config: ti_omap5_common: Palmas power support in SPL
2015-07-01 22:20 [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL Paul Kocialkowski
2015-07-15 12:56 ` Paul Kocialkowski
@ 2015-07-28 14:58 ` Tom Rini
1 sibling, 0 replies; 5+ messages in thread
From: Tom Rini @ 2015-07-28 14:58 UTC (permalink / raw)
To: u-boot
On Thu, Jul 02, 2015 at 12:20:25AM +0200, Paul Kocialkowski wrote:
> Palmas power support is required for OMAP5 devices such as the OMAP5 uEVM, that
> need to e.g. enable MMC power at SPL stage.
>
> This is especially important when booting from a peripheral (such as USB, UART),
> where the bootrom will not enable power for the MMC device that will hold the
> main U-Boot.
>
> Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150728/073943cf/attachment.sig>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-07-28 14:58 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-01 22:20 [U-Boot] [PATCH] config: ti_omap5_common: Palmas power support in SPL Paul Kocialkowski
2015-07-15 12:56 ` Paul Kocialkowski
2015-07-16 13:21 ` Tom Rini
2015-07-26 16:30 ` Paul Kocialkowski
2015-07-28 14:58 ` [U-Boot] " Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox