* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
@ 2019-08-08 18:14 Robert Hancock
2019-08-08 18:36 ` Adam Ford
0 siblings, 1 reply; 8+ messages in thread
From: Robert Hancock @ 2019-08-08 18:14 UTC (permalink / raw)
To: u-boot
Previously the SPL size on all iMX6 platforms was restricted to 68KB
because the OCRAM size on iMX6SL/DL parts is only 128KB. However, the
other iMX6 variants have 256KB of OCRAM. Add an option
CONFIG_MX6_OCRAM_256KB which allows using the full size on boards which
don't need to support the SL/DL variants. This allows for an SPL size of
196KB, which makes it much easier to use configurations such as SPL with
driver model and FDT control.
Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
---
Changed since v2: Remove "default n" in Kconfig since that is the default.
arch/arm/mach-imx/mx6/Kconfig | 9 +++++++++
common/spl/Kconfig | 3 ++-
include/configs/imx6_spl.h | 28 ++++++++++++++++++++++++++--
3 files changed, 37 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig
index fe5991e..9c27176 100644
--- a/arch/arm/mach-imx/mx6/Kconfig
+++ b/arch/arm/mach-imx/mx6/Kconfig
@@ -87,6 +87,15 @@ config MX6ULL
select SYSCOUNTER_TIMER
select SYS_L2CACHE_OFF
+config MX6_OCRAM_256KB
+ bool "Support 256KB OCRAM"
+ depends on MX6D || MX6Q
+ help
+ Allows using the full 256KB size of the OCRAM on the MX6Q/MX6D series
+ of chips, such as for SPL. The OCRAM of the Lite series of chips is
+ only 128KB, so using this option will prevent the resulting code from
+ working on those chips.
+
config MX6_DDRCAL
bool "Include dynamic DDR calibration routines"
depends on SPL
diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index 54154b9..c5ad047 100644
--- a/common/spl/Kconfig
+++ b/common/spl/Kconfig
@@ -28,7 +28,8 @@ config SPL_FRAMEWORK
config SPL_SIZE_LIMIT
int "Maximum size of SPL image"
depends on SPL
- default 69632 if ARCH_MX6
+ default 69632 if ARCH_MX6 && !MX6_OCRAM_256KB
+ default 200704 if ARCH_MX6 && MX6_OCRAM_256KB
default 0
help
Specifies the maximum length of the U-Boot SPL image.
diff --git a/include/configs/imx6_spl.h b/include/configs/imx6_spl.h
index 212dee7..a223930 100644
--- a/include/configs/imx6_spl.h
+++ b/include/configs/imx6_spl.h
@@ -7,10 +7,32 @@
#define __IMX6_SPL_CONFIG_H
#ifdef CONFIG_SPL
+
+#ifdef CONFIG_MX6_OCRAM_256KB
/*
- * see Figure 8-3 in IMX6DQ/IMX6SDL Reference manuals:
+ * see Figure 8.4.1 in IMX6DQ Reference manuals:
+ * - IMX6DQ OCRAM (IRAM) is from 0x00907000 to 0x0093FFFF
+ * - BOOT ROM stack is at 0x0093FFB8
+ * - if icache/dcache is enabled (eFuse/strapping controlled) then the
+ * IMX BOOT ROM will setup MMU table at 0x00938000, therefore we need to
+ * fit between 0x00907000 and 0x00938000.
+ * - Additionally the BOOT ROM loads what they consider the firmware image
+ * which consists of a 4K header in front of us that contains the IVT, DCD
+ * and some padding thus 'our' max size is really 0x00908000 - 0x00938000
+ * or 192KB
+ */
+#define CONFIG_SPL_MAX_SIZE 0x30000
+#define CONFIG_SPL_STACK 0x0093FFB8
+/*
+ * Pad SPL to 196KB (4KB header + 192KB max size). This allows to write the
+ * SPL/U-Boot combination generated with u-boot-with-spl.imx directly to a
+ * boot media (given that boot media specific offset is configured properly).
+ */
+#define CONFIG_SPL_PAD_TO 0x31000
+#else
+/*
+ * see Figure 8-3 in IMX6SDL Reference manuals:
* - IMX6SDL OCRAM (IRAM) is from 0x00907000 to 0x0091FFFF
- * - IMX6DQ has 2x IRAM of IMX6SDL but we intend to support IMX6SDL as well
* - BOOT ROM stack is at 0x0091FFB8
* - if icache/dcache is enabled (eFuse/strapping controlled) then the
* IMX BOOT ROM will setup MMU table at 0x00918000, therefore we need to
@@ -29,6 +51,8 @@
*/
#define CONFIG_SPL_PAD_TO 0x11000
+#endif
+
/* MMC support */
#if defined(CONFIG_SPL_MMC_SUPPORT)
#define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION 1
--
1.8.3.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-08-08 18:14 [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ Robert Hancock
@ 2019-08-08 18:36 ` Adam Ford
2019-08-26 13:00 ` Adam Ford
0 siblings, 1 reply; 8+ messages in thread
From: Adam Ford @ 2019-08-08 18:36 UTC (permalink / raw)
To: u-boot
On Thu, Aug 8, 2019 at 1:14 PM Robert Hancock <hancock@sedsystems.ca> wrote:
>
> Previously the SPL size on all iMX6 platforms was restricted to 68KB
> because the OCRAM size on iMX6SL/DL parts is only 128KB. However, the
> other iMX6 variants have 256KB of OCRAM. Add an option
> CONFIG_MX6_OCRAM_256KB which allows using the full size on boards which
> don't need to support the SL/DL variants. This allows for an SPL size of
> 196KB, which makes it much easier to use configurations such as SPL with
> driver model and FDT control.
>
Tested-by: Adam Ford <aford173@gmail.com> #imx6q_logic
> Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
> ---
>
> Changed since v2: Remove "default n" in Kconfig since that is the default.
>
> arch/arm/mach-imx/mx6/Kconfig | 9 +++++++++
> common/spl/Kconfig | 3 ++-
> include/configs/imx6_spl.h | 28 ++++++++++++++++++++++++++--
> 3 files changed, 37 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig
> index fe5991e..9c27176 100644
> --- a/arch/arm/mach-imx/mx6/Kconfig
> +++ b/arch/arm/mach-imx/mx6/Kconfig
> @@ -87,6 +87,15 @@ config MX6ULL
> select SYSCOUNTER_TIMER
> select SYS_L2CACHE_OFF
>
> +config MX6_OCRAM_256KB
> + bool "Support 256KB OCRAM"
> + depends on MX6D || MX6Q
> + help
> + Allows using the full 256KB size of the OCRAM on the MX6Q/MX6D series
> + of chips, such as for SPL. The OCRAM of the Lite series of chips is
> + only 128KB, so using this option will prevent the resulting code from
> + working on those chips.
> +
> config MX6_DDRCAL
> bool "Include dynamic DDR calibration routines"
> depends on SPL
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index 54154b9..c5ad047 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -28,7 +28,8 @@ config SPL_FRAMEWORK
> config SPL_SIZE_LIMIT
> int "Maximum size of SPL image"
> depends on SPL
> - default 69632 if ARCH_MX6
> + default 69632 if ARCH_MX6 && !MX6_OCRAM_256KB
> + default 200704 if ARCH_MX6 && MX6_OCRAM_256KB
> default 0
> help
> Specifies the maximum length of the U-Boot SPL image.
> diff --git a/include/configs/imx6_spl.h b/include/configs/imx6_spl.h
> index 212dee7..a223930 100644
> --- a/include/configs/imx6_spl.h
> +++ b/include/configs/imx6_spl.h
> @@ -7,10 +7,32 @@
> #define __IMX6_SPL_CONFIG_H
>
> #ifdef CONFIG_SPL
> +
> +#ifdef CONFIG_MX6_OCRAM_256KB
> /*
> - * see Figure 8-3 in IMX6DQ/IMX6SDL Reference manuals:
> + * see Figure 8.4.1 in IMX6DQ Reference manuals:
> + * - IMX6DQ OCRAM (IRAM) is from 0x00907000 to 0x0093FFFF
> + * - BOOT ROM stack is at 0x0093FFB8
> + * - if icache/dcache is enabled (eFuse/strapping controlled) then the
> + * IMX BOOT ROM will setup MMU table at 0x00938000, therefore we need to
> + * fit between 0x00907000 and 0x00938000.
> + * - Additionally the BOOT ROM loads what they consider the firmware image
> + * which consists of a 4K header in front of us that contains the IVT, DCD
> + * and some padding thus 'our' max size is really 0x00908000 - 0x00938000
> + * or 192KB
> + */
> +#define CONFIG_SPL_MAX_SIZE 0x30000
> +#define CONFIG_SPL_STACK 0x0093FFB8
> +/*
> + * Pad SPL to 196KB (4KB header + 192KB max size). This allows to write the
> + * SPL/U-Boot combination generated with u-boot-with-spl.imx directly to a
> + * boot media (given that boot media specific offset is configured properly).
> + */
> +#define CONFIG_SPL_PAD_TO 0x31000
> +#else
> +/*
> + * see Figure 8-3 in IMX6SDL Reference manuals:
> * - IMX6SDL OCRAM (IRAM) is from 0x00907000 to 0x0091FFFF
> - * - IMX6DQ has 2x IRAM of IMX6SDL but we intend to support IMX6SDL as well
> * - BOOT ROM stack is at 0x0091FFB8
> * - if icache/dcache is enabled (eFuse/strapping controlled) then the
> * IMX BOOT ROM will setup MMU table at 0x00918000, therefore we need to
> @@ -29,6 +51,8 @@
> */
> #define CONFIG_SPL_PAD_TO 0x11000
>
> +#endif
> +
> /* MMC support */
> #if defined(CONFIG_SPL_MMC_SUPPORT)
> #define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION 1
> --
> 1.8.3.1
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-08-08 18:36 ` Adam Ford
@ 2019-08-26 13:00 ` Adam Ford
2019-09-27 12:17 ` Adam Ford
0 siblings, 1 reply; 8+ messages in thread
From: Adam Ford @ 2019-08-26 13:00 UTC (permalink / raw)
To: u-boot
On Thu, Aug 8, 2019 at 1:36 PM Adam Ford <aford173@gmail.com> wrote:
>
> On Thu, Aug 8, 2019 at 1:14 PM Robert Hancock <hancock@sedsystems.ca> wrote:
> >
> > Previously the SPL size on all iMX6 platforms was restricted to 68KB
> > because the OCRAM size on iMX6SL/DL parts is only 128KB. However, the
> > other iMX6 variants have 256KB of OCRAM. Add an option
> > CONFIG_MX6_OCRAM_256KB which allows using the full size on boards which
> > don't need to support the SL/DL variants. This allows for an SPL size of
> > 196KB, which makes it much easier to use configurations such as SPL with
> > driver model and FDT control.
> >
> Tested-by: Adam Ford <aford173@gmail.com> #imx6q_logic
>
> > Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
> > ---
Stefano,
Any chance this can be adopted? I have a bunch of patches pending
this, because my SPL is tight and this patch gives it some breathing
room.
thanks,
adam
> >
> > Changed since v2: Remove "default n" in Kconfig since that is the default.
> >
> > arch/arm/mach-imx/mx6/Kconfig | 9 +++++++++
> > common/spl/Kconfig | 3 ++-
> > include/configs/imx6_spl.h | 28 ++++++++++++++++++++++++++--
> > 3 files changed, 37 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig
> > index fe5991e..9c27176 100644
> > --- a/arch/arm/mach-imx/mx6/Kconfig
> > +++ b/arch/arm/mach-imx/mx6/Kconfig
> > @@ -87,6 +87,15 @@ config MX6ULL
> > select SYSCOUNTER_TIMER
> > select SYS_L2CACHE_OFF
> >
> > +config MX6_OCRAM_256KB
> > + bool "Support 256KB OCRAM"
> > + depends on MX6D || MX6Q
> > + help
> > + Allows using the full 256KB size of the OCRAM on the MX6Q/MX6D series
> > + of chips, such as for SPL. The OCRAM of the Lite series of chips is
> > + only 128KB, so using this option will prevent the resulting code from
> > + working on those chips.
> > +
> > config MX6_DDRCAL
> > bool "Include dynamic DDR calibration routines"
> > depends on SPL
> > diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> > index 54154b9..c5ad047 100644
> > --- a/common/spl/Kconfig
> > +++ b/common/spl/Kconfig
> > @@ -28,7 +28,8 @@ config SPL_FRAMEWORK
> > config SPL_SIZE_LIMIT
> > int "Maximum size of SPL image"
> > depends on SPL
> > - default 69632 if ARCH_MX6
> > + default 69632 if ARCH_MX6 && !MX6_OCRAM_256KB
> > + default 200704 if ARCH_MX6 && MX6_OCRAM_256KB
> > default 0
> > help
> > Specifies the maximum length of the U-Boot SPL image.
> > diff --git a/include/configs/imx6_spl.h b/include/configs/imx6_spl.h
> > index 212dee7..a223930 100644
> > --- a/include/configs/imx6_spl.h
> > +++ b/include/configs/imx6_spl.h
> > @@ -7,10 +7,32 @@
> > #define __IMX6_SPL_CONFIG_H
> >
> > #ifdef CONFIG_SPL
> > +
> > +#ifdef CONFIG_MX6_OCRAM_256KB
> > /*
> > - * see Figure 8-3 in IMX6DQ/IMX6SDL Reference manuals:
> > + * see Figure 8.4.1 in IMX6DQ Reference manuals:
> > + * - IMX6DQ OCRAM (IRAM) is from 0x00907000 to 0x0093FFFF
> > + * - BOOT ROM stack is at 0x0093FFB8
> > + * - if icache/dcache is enabled (eFuse/strapping controlled) then the
> > + * IMX BOOT ROM will setup MMU table at 0x00938000, therefore we need to
> > + * fit between 0x00907000 and 0x00938000.
> > + * - Additionally the BOOT ROM loads what they consider the firmware image
> > + * which consists of a 4K header in front of us that contains the IVT, DCD
> > + * and some padding thus 'our' max size is really 0x00908000 - 0x00938000
> > + * or 192KB
> > + */
> > +#define CONFIG_SPL_MAX_SIZE 0x30000
> > +#define CONFIG_SPL_STACK 0x0093FFB8
> > +/*
> > + * Pad SPL to 196KB (4KB header + 192KB max size). This allows to write the
> > + * SPL/U-Boot combination generated with u-boot-with-spl.imx directly to a
> > + * boot media (given that boot media specific offset is configured properly).
> > + */
> > +#define CONFIG_SPL_PAD_TO 0x31000
> > +#else
> > +/*
> > + * see Figure 8-3 in IMX6SDL Reference manuals:
> > * - IMX6SDL OCRAM (IRAM) is from 0x00907000 to 0x0091FFFF
> > - * - IMX6DQ has 2x IRAM of IMX6SDL but we intend to support IMX6SDL as well
> > * - BOOT ROM stack is at 0x0091FFB8
> > * - if icache/dcache is enabled (eFuse/strapping controlled) then the
> > * IMX BOOT ROM will setup MMU table at 0x00918000, therefore we need to
> > @@ -29,6 +51,8 @@
> > */
> > #define CONFIG_SPL_PAD_TO 0x11000
> >
> > +#endif
> > +
> > /* MMC support */
> > #if defined(CONFIG_SPL_MMC_SUPPORT)
> > #define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION 1
> > --
> > 1.8.3.1
> >
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-08-26 13:00 ` Adam Ford
@ 2019-09-27 12:17 ` Adam Ford
2019-09-27 12:31 ` Fabio Estevam
0 siblings, 1 reply; 8+ messages in thread
From: Adam Ford @ 2019-09-27 12:17 UTC (permalink / raw)
To: u-boot
On Mon, Aug 26, 2019 at 8:00 AM Adam Ford <aford173@gmail.com> wrote:
>
> On Thu, Aug 8, 2019 at 1:36 PM Adam Ford <aford173@gmail.com> wrote:
> >
> > On Thu, Aug 8, 2019 at 1:14 PM Robert Hancock <hancock@sedsystems.ca> wrote:
> > >
> > > Previously the SPL size on all iMX6 platforms was restricted to 68KB
> > > because the OCRAM size on iMX6SL/DL parts is only 128KB. However, the
> > > other iMX6 variants have 256KB of OCRAM. Add an option
> > > CONFIG_MX6_OCRAM_256KB which allows using the full size on boards which
> > > don't need to support the SL/DL variants. This allows for an SPL size of
> > > 196KB, which makes it much easier to use configurations such as SPL with
> > > driver model and FDT control.
> > >
> > Tested-by: Adam Ford <aford173@gmail.com> #imx6q_logic
> >
> > > Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
> > > ---
>
> Stefano,
>
> Any chance this can be adopted? I have a bunch of patches pending
> this, because my SPL is tight and this patch gives it some breathing
> room.
>
It's been over a month. Is there any reason this cannot be applied?
thanks
adam
> thanks,
>
> adam
>
> > >
> > > Changed since v2: Remove "default n" in Kconfig since that is the default.
> > >
> > > arch/arm/mach-imx/mx6/Kconfig | 9 +++++++++
> > > common/spl/Kconfig | 3 ++-
> > > include/configs/imx6_spl.h | 28 ++++++++++++++++++++++++++--
> > > 3 files changed, 37 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig
> > > index fe5991e..9c27176 100644
> > > --- a/arch/arm/mach-imx/mx6/Kconfig
> > > +++ b/arch/arm/mach-imx/mx6/Kconfig
> > > @@ -87,6 +87,15 @@ config MX6ULL
> > > select SYSCOUNTER_TIMER
> > > select SYS_L2CACHE_OFF
> > >
> > > +config MX6_OCRAM_256KB
> > > + bool "Support 256KB OCRAM"
> > > + depends on MX6D || MX6Q
> > > + help
> > > + Allows using the full 256KB size of the OCRAM on the MX6Q/MX6D series
> > > + of chips, such as for SPL. The OCRAM of the Lite series of chips is
> > > + only 128KB, so using this option will prevent the resulting code from
> > > + working on those chips.
> > > +
> > > config MX6_DDRCAL
> > > bool "Include dynamic DDR calibration routines"
> > > depends on SPL
> > > diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> > > index 54154b9..c5ad047 100644
> > > --- a/common/spl/Kconfig
> > > +++ b/common/spl/Kconfig
> > > @@ -28,7 +28,8 @@ config SPL_FRAMEWORK
> > > config SPL_SIZE_LIMIT
> > > int "Maximum size of SPL image"
> > > depends on SPL
> > > - default 69632 if ARCH_MX6
> > > + default 69632 if ARCH_MX6 && !MX6_OCRAM_256KB
> > > + default 200704 if ARCH_MX6 && MX6_OCRAM_256KB
> > > default 0
> > > help
> > > Specifies the maximum length of the U-Boot SPL image.
> > > diff --git a/include/configs/imx6_spl.h b/include/configs/imx6_spl.h
> > > index 212dee7..a223930 100644
> > > --- a/include/configs/imx6_spl.h
> > > +++ b/include/configs/imx6_spl.h
> > > @@ -7,10 +7,32 @@
> > > #define __IMX6_SPL_CONFIG_H
> > >
> > > #ifdef CONFIG_SPL
> > > +
> > > +#ifdef CONFIG_MX6_OCRAM_256KB
> > > /*
> > > - * see Figure 8-3 in IMX6DQ/IMX6SDL Reference manuals:
> > > + * see Figure 8.4.1 in IMX6DQ Reference manuals:
> > > + * - IMX6DQ OCRAM (IRAM) is from 0x00907000 to 0x0093FFFF
> > > + * - BOOT ROM stack is at 0x0093FFB8
> > > + * - if icache/dcache is enabled (eFuse/strapping controlled) then the
> > > + * IMX BOOT ROM will setup MMU table at 0x00938000, therefore we need to
> > > + * fit between 0x00907000 and 0x00938000.
> > > + * - Additionally the BOOT ROM loads what they consider the firmware image
> > > + * which consists of a 4K header in front of us that contains the IVT, DCD
> > > + * and some padding thus 'our' max size is really 0x00908000 - 0x00938000
> > > + * or 192KB
> > > + */
> > > +#define CONFIG_SPL_MAX_SIZE 0x30000
> > > +#define CONFIG_SPL_STACK 0x0093FFB8
> > > +/*
> > > + * Pad SPL to 196KB (4KB header + 192KB max size). This allows to write the
> > > + * SPL/U-Boot combination generated with u-boot-with-spl.imx directly to a
> > > + * boot media (given that boot media specific offset is configured properly).
> > > + */
> > > +#define CONFIG_SPL_PAD_TO 0x31000
> > > +#else
> > > +/*
> > > + * see Figure 8-3 in IMX6SDL Reference manuals:
> > > * - IMX6SDL OCRAM (IRAM) is from 0x00907000 to 0x0091FFFF
> > > - * - IMX6DQ has 2x IRAM of IMX6SDL but we intend to support IMX6SDL as well
> > > * - BOOT ROM stack is at 0x0091FFB8
> > > * - if icache/dcache is enabled (eFuse/strapping controlled) then the
> > > * IMX BOOT ROM will setup MMU table at 0x00918000, therefore we need to
> > > @@ -29,6 +51,8 @@
> > > */
> > > #define CONFIG_SPL_PAD_TO 0x11000
> > >
> > > +#endif
> > > +
> > > /* MMC support */
> > > #if defined(CONFIG_SPL_MMC_SUPPORT)
> > > #define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION 1
> > > --
> > > 1.8.3.1
> > >
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-09-27 12:17 ` Adam Ford
@ 2019-09-27 12:31 ` Fabio Estevam
2019-09-27 12:32 ` Adam Ford
0 siblings, 1 reply; 8+ messages in thread
From: Fabio Estevam @ 2019-09-27 12:31 UTC (permalink / raw)
To: u-boot
On Fri, Sep 27, 2019 at 9:17 AM Adam Ford <aford173@gmail.com> wrote:
> It's been over a month. Is there any reason this cannot be applied?
It has been applied in Stefano's tree:
https://gitlab.denx.de/u-boot/custodians/u-boot-imx/commit/90fb571d61a8967f5e56c5b03db5e44889325bde
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-09-27 12:31 ` Fabio Estevam
@ 2019-09-27 12:32 ` Adam Ford
2019-09-27 12:53 ` Stefano Babic
0 siblings, 1 reply; 8+ messages in thread
From: Adam Ford @ 2019-09-27 12:32 UTC (permalink / raw)
To: u-boot
On Fri, Sep 27, 2019 at 7:31 AM Fabio Estevam <festevam@gmail.com> wrote:
>
> On Fri, Sep 27, 2019 at 9:17 AM Adam Ford <aford173@gmail.com> wrote:
>
> > It's been over a month. Is there any reason this cannot be applied?
>
> It has been applied in Stefano's tree:
> https://gitlab.denx.de/u-boot/custodians/u-boot-imx/commit/90fb571d61a8967f5e56c5b03db5e44889325bde
Fantastic! I just looked at master. Hopefully it'll get pulled after
the next pull request.
Thank you,
adam
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-09-27 12:32 ` Adam Ford
@ 2019-09-27 12:53 ` Stefano Babic
2019-09-27 13:15 ` Adam Ford
0 siblings, 1 reply; 8+ messages in thread
From: Stefano Babic @ 2019-09-27 12:53 UTC (permalink / raw)
To: u-boot
On 27/09/19 14:32, Adam Ford wrote:
> On Fri, Sep 27, 2019 at 7:31 AM Fabio Estevam <festevam@gmail.com> wrote:
>>
>> On Fri, Sep 27, 2019 at 9:17 AM Adam Ford <aford173@gmail.com> wrote:
>>
>>> It's been over a month. Is there any reason this cannot be applied?
>>
>> It has been applied in Stefano's tree:
>> https://gitlab.denx.de/u-boot/custodians/u-boot-imx/commit/90fb571d61a8967f5e56c5b03db5e44889325bde
>
> Fantastic! I just looked at master. Hopefully it'll get pulled after
> the next pull request.
My last PR was rejected (at least, discouraged) due to the number of
changes and release is approaching. But I will not try to pick up random
patches from u-boot-imx master and try to push it just to get it in
release. I am starting to pick up all pacthes that are ready in -next,
starting from u-boot-imx, -master.
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: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ
2019-09-27 12:53 ` Stefano Babic
@ 2019-09-27 13:15 ` Adam Ford
0 siblings, 0 replies; 8+ messages in thread
From: Adam Ford @ 2019-09-27 13:15 UTC (permalink / raw)
To: u-boot
On Fri, Sep 27, 2019 at 7:53 AM Stefano Babic <sbabic@denx.de> wrote:
>
> On 27/09/19 14:32, Adam Ford wrote:
> > On Fri, Sep 27, 2019 at 7:31 AM Fabio Estevam <festevam@gmail.com> wrote:
> >>
> >> On Fri, Sep 27, 2019 at 9:17 AM Adam Ford <aford173@gmail.com> wrote:
> >>
> >>> It's been over a month. Is there any reason this cannot be applied?
> >>
> >> It has been applied in Stefano's tree:
> >> https://gitlab.denx.de/u-boot/custodians/u-boot-imx/commit/90fb571d61a8967f5e56c5b03db5e44889325bde
> >
> > Fantastic! I just looked at master. Hopefully it'll get pulled after
> > the next pull request.
>
> My last PR was rejected (at least, discouraged) due to the number of
> changes and release is approaching. But I will not try to pick up random
> patches from u-boot-imx master and try to push it just to get it in
> release. I am starting to pick up all pacthes that are ready in -next,
> starting from u-boot-imx, -master.
>
That makes sense since this is an improvement and not a bug fix.
adam
> 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: sbabic at denx.de
> =====================================================================
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-09-27 13:15 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-08 18:14 [U-Boot] [PATCH v2] ARM: imx: Support larger SPL size on IMX6DQ Robert Hancock
2019-08-08 18:36 ` Adam Ford
2019-08-26 13:00 ` Adam Ford
2019-09-27 12:17 ` Adam Ford
2019-09-27 12:31 ` Fabio Estevam
2019-09-27 12:32 ` Adam Ford
2019-09-27 12:53 ` Stefano Babic
2019-09-27 13:15 ` Adam Ford
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.