[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Lukasz Majewski]

From: Mans Rullgard <mans@mansr.com>

This change allows setting pre-defined eMMC boot partition for SPL eMMC
booting. It is necessary in the case when one wants to boot (through falcon
boot) from eMMC after loading SPL from other memory (like SPI-NOR).

Signed-off-by: Mans Rullgard <mans@mansr.com>
[lukma: Edit the commit message]
Signed-off-by: Lukasz Majewski <lukma@denx.de>
Acked-by: Andreas Dannenberg <dannenberg@ti.com>

---

 common/spl/Kconfig   | 6 ++++++
 common/spl/spl_mmc.c | 4 ++++
 2 files changed, 10 insertions(+)

diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index f467eca2be..5b7667ea16 100644
--- a/common/spl/Kconfig
+++ b/common/spl/Kconfig
@@ -314,6 +314,12 @@ config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
 	  Address on the MMC to load U-Boot from, when the MMC is being used
 	  in raw mode. Units: MMC sectors (1 sector = 512 bytes).
 
+config SYS_MMCSD_RAW_MODE_EMMC_BOOT_PARTITION
+	int "Number of the eMMC boot partition to use"
+	default 1
+	help
+	  eMMC boot partition number to use when the eMMC in raw mode.
+
 config SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION
 	bool "MMC Raw mode: by partition"
 	help
diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
index 2caceb5186..a4d2e63b82 100644
--- a/common/spl/spl_mmc.c
+++ b/common/spl/spl_mmc.c
@@ -334,6 +334,9 @@ int spl_mmc_load(struct spl_image_info *spl_image,
 	err = -EINVAL;
 	switch (boot_mode) {
 	case MMCSD_MODE_EMMCBOOT:
+#ifdef CONFIG_SYS_MMCSD_RAW_MODE_EMMC_BOOT_PARTITION
+		part = CONFIG_SYS_MMCSD_RAW_MODE_EMMC_BOOT_PARTITION;
+#else
 		/*
 		 * We need to check what the partition is configured to.
 		 * 1 and 2 match up to boot0 / boot1 and 7 is user data
@@ -343,6 +346,7 @@ int spl_mmc_load(struct spl_image_info *spl_image,
 
 		if (part == 7)
 			part = 0;
+#endif
 
 		if (CONFIG_IS_ENABLED(MMC_TINY))
 			err = mmc_switch_part(mmc, part);
-- 
2.11.0

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Marek Vasut]

On 9/3/19 4:16 PM, Lukasz Majewski wrote:
> From: Mans Rullgard <mans@mansr.com>
> 
> This change allows setting pre-defined eMMC boot partition for SPL eMMC
> booting. It is necessary in the case when one wants to boot (through falcon
> boot) from eMMC after loading SPL from other memory (like SPI-NOR).
> 
> Signed-off-by: Mans Rullgard <mans@mansr.com>
> [lukma: Edit the commit message]
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> Acked-by: Andreas Dannenberg <dannenberg@ti.com>

Would it be better if this came from /chosen node in DT ?

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Måns Rullgård]

Marek Vasut <marex@denx.de> writes:

> On 9/3/19 4:16 PM, Lukasz Majewski wrote:
>> From: Mans Rullgard <mans@mansr.com>
>> 
>> This change allows setting pre-defined eMMC boot partition for SPL eMMC
>> booting. It is necessary in the case when one wants to boot (through falcon
>> boot) from eMMC after loading SPL from other memory (like SPI-NOR).
>> 
>> Signed-off-by: Mans Rullgard <mans@mansr.com>
>> [lukma: Edit the commit message]
>> Signed-off-by: Lukasz Majewski <lukma@denx.de>
>> Acked-by: Andreas Dannenberg <dannenberg@ti.com>
>
> Would it be better if this came from /chosen node in DT ?

This patch was made for an imx28 based board.  Fitting DT support into
SPL on that chip won't be easy.

-- 
Måns Rullgård

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Lukasz Majewski]

Hi MÃ¥ns, Marek,

> Marek Vasut <marex@denx.de> writes:
> 
> > On 9/3/19 4:16 PM, Lukasz Majewski wrote:  
> >> From: Mans Rullgard <mans@mansr.com>
> >> 
> >> This change allows setting pre-defined eMMC boot partition for SPL
> >> eMMC booting. It is necessary in the case when one wants to boot
> >> (through falcon boot) from eMMC after loading SPL from other
> >> memory (like SPI-NOR).
> >> 
> >> Signed-off-by: Mans Rullgard <mans@mansr.com>
> >> [lukma: Edit the commit message]
> >> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> >> Acked-by: Andreas Dannenberg <dannenberg@ti.com>  
> >
> > Would it be better if this came from /chosen node in DT ?  
> 
> This patch was made for an imx28 based board.  Fitting DT support into
> SPL on that chip won't be easy.

Unfortunately, on purpose the DT support for SPL on this board is
disabled.

This is the case where the only eligible option is to use OF_PLATDATA,
to benefit from DM drivers, but also to keep the size as small as
possible.

(Please refer to xea board support patches for more details).

> 



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190903/1a21d908/attachment.sig>

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Marek Vasut]

On 9/3/19 5:49 PM, Lukasz Majewski wrote:
> Hi MÃ¥ns, Marek,
> 
>> Marek Vasut <marex@denx.de> writes:
>>
>>> On 9/3/19 4:16 PM, Lukasz Majewski wrote:  
>>>> From: Mans Rullgard <mans@mansr.com>
>>>>
>>>> This change allows setting pre-defined eMMC boot partition for SPL
>>>> eMMC booting. It is necessary in the case when one wants to boot
>>>> (through falcon boot) from eMMC after loading SPL from other
>>>> memory (like SPI-NOR).
>>>>
>>>> Signed-off-by: Mans Rullgard <mans@mansr.com>
>>>> [lukma: Edit the commit message]
>>>> Signed-off-by: Lukasz Majewski <lukma@denx.de>
>>>> Acked-by: Andreas Dannenberg <dannenberg@ti.com>  
>>>
>>> Would it be better if this came from /chosen node in DT ?  
>>
>> This patch was made for an imx28 based board.  Fitting DT support into
>> SPL on that chip won't be easy.
> 
> Unfortunately, on purpose the DT support for SPL on this board is
> disabled.
> 
> This is the case where the only eligible option is to use OF_PLATDATA,
> to benefit from DM drivers, but also to keep the size as small as
> possible.
> 
> (Please refer to xea board support patches for more details).

Hm, then shall we use platdata ? I don't think that fits common code
like this either.

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Lukasz Majewski]

Hi Marek,

> On 9/3/19 5:49 PM, Lukasz Majewski wrote:
> > Hi MÃ¥ns, Marek,
> >   
> >> Marek Vasut <marex@denx.de> writes:
> >>  
> >>> On 9/3/19 4:16 PM, Lukasz Majewski wrote:    
> >>>> From: Mans Rullgard <mans@mansr.com>
> >>>>
> >>>> This change allows setting pre-defined eMMC boot partition for
> >>>> SPL eMMC booting. It is necessary in the case when one wants to
> >>>> boot (through falcon boot) from eMMC after loading SPL from other
> >>>> memory (like SPI-NOR).
> >>>>
> >>>> Signed-off-by: Mans Rullgard <mans@mansr.com>
> >>>> [lukma: Edit the commit message]
> >>>> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> >>>> Acked-by: Andreas Dannenberg <dannenberg@ti.com>    
> >>>
> >>> Would it be better if this came from /chosen node in DT ?    
> >>
> >> This patch was made for an imx28 based board.  Fitting DT support
> >> into SPL on that chip won't be easy.  
> > 
> > Unfortunately, on purpose the DT support for SPL on this board is
> > disabled.
> > 
> > This is the case where the only eligible option is to use
> > OF_PLATDATA, to benefit from DM drivers, but also to keep the size
> > as small as possible.
> > 
> > (Please refer to xea board support patches for more details).  
> 
> Hm, then shall we use platdata ? 

Yes, correct. The OF_PLATDATA allows you to use DM/DTS converted driver
with platform data encoded in the C structures.

For example such patch adds support for OF_PLATDATA to DM/DTS driver:
https://patchwork.ozlabs.org/patch/1148906/

> I don't think that fits common code
> like this either.

Please see above link.


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190903/fa9297a0/attachment.sig>

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Lukasz Majewski]

Dear Tom,

> From: Mans Rullgard <mans@mansr.com>
> 
> This change allows setting pre-defined eMMC boot partition for SPL
> eMMC booting. It is necessary in the case when one wants to boot
> (through falcon boot) from eMMC after loading SPL from other memory
> (like SPI-NOR).
> 
> Signed-off-by: Mans Rullgard <mans@mansr.com>
> [lukma: Edit the commit message]
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> Acked-by: Andreas Dannenberg <dannenberg@ti.com>
> 

Tom, are there any issues preventing this patch from being applied to
-master?

> ---
> 
>  common/spl/Kconfig   | 6 ++++++
>  common/spl/spl_mmc.c | 4 ++++
>  2 files changed, 10 insertions(+)
> 
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index f467eca2be..5b7667ea16 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -314,6 +314,12 @@ config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
>  	  Address on the MMC to load U-Boot from, when the MMC is
> being used in raw mode. Units: MMC sectors (1 sector = 512 bytes).
>  
> +config SYS_MMCSD_RAW_MODE_EMMC_BOOT_PARTITION
> +	int "Number of the eMMC boot partition to use"
> +	default 1
> +	help
> +	  eMMC boot partition number to use when the eMMC in raw
> mode. +
>  config SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION
>  	bool "MMC Raw mode: by partition"
>  	help
> diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
> index 2caceb5186..a4d2e63b82 100644
> --- a/common/spl/spl_mmc.c
> +++ b/common/spl/spl_mmc.c
> @@ -334,6 +334,9 @@ int spl_mmc_load(struct spl_image_info *spl_image,
>  	err = -EINVAL;
>  	switch (boot_mode) {
>  	case MMCSD_MODE_EMMCBOOT:
> +#ifdef CONFIG_SYS_MMCSD_RAW_MODE_EMMC_BOOT_PARTITION
> +		part = CONFIG_SYS_MMCSD_RAW_MODE_EMMC_BOOT_PARTITION;
> +#else
>  		/*
>  		 * We need to check what the partition is configured
> to.
>  		 * 1 and 2 match up to boot0 / boot1 and 7 is user
> data @@ -343,6 +346,7 @@ int spl_mmc_load(struct spl_image_info
> *spl_image, 
>  		if (part == 7)
>  			part = 0;
> +#endif
>  
>  		if (CONFIG_IS_ENABLED(MMC_TINY))
>  			err = mmc_switch_part(mmc, part);




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20191009/4813b409/attachment.sig>

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Tom Rini]

On Wed, Oct 09, 2019 at 11:07:18AM +0200, Lukasz Majewski wrote:
> Dear Tom,
> 
> > From: Mans Rullgard <mans@mansr.com>
> > 
> > This change allows setting pre-defined eMMC boot partition for SPL
> > eMMC booting. It is necessary in the case when one wants to boot
> > (through falcon boot) from eMMC after loading SPL from other memory
> > (like SPI-NOR).
> > 
> > Signed-off-by: Mans Rullgard <mans@mansr.com>
> > [lukma: Edit the commit message]
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > Acked-by: Andreas Dannenberg <dannenberg@ti.com>
> > 
> 
> Tom, are there any issues preventing this patch from being applied to
> -master?

Keeping in mind that we do need to allow for SPL to do reasonable things
to retain a smaller footprint, this is fine.  If Peng doesn't pick it up
as an MMC change I'll get it soon.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20191010/be60964d/attachment.sig>

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Peng Fan]

Hi Tom,

> Subject: Re: [PATCH] spl: mmc: Add option to set eMMC HW boot partition
> 
> On Wed, Oct 09, 2019 at 11:07:18AM +0200, Lukasz Majewski wrote:
> > Dear Tom,
> >
> > > From: Mans Rullgard <mans@mansr.com>
> > >
> > > This change allows setting pre-defined eMMC boot partition for SPL
> > > eMMC booting. It is necessary in the case when one wants to boot
> > > (through falcon boot) from eMMC after loading SPL from other memory
> > > (like SPI-NOR).
> > >
> > > Signed-off-by: Mans Rullgard <mans@mansr.com>
> > > [lukma: Edit the commit message]
> > > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > > Acked-by: Andreas Dannenberg <dannenberg@ti.com>
> > >
> >
> > Tom, are there any issues preventing this patch from being applied to
> > -master?
> 
> Keeping in mind that we do need to allow for SPL to do reasonable things to
> retain a smaller footprint, this is fine.  If Peng doesn't pick it up as an MMC
> change I'll get it soon.

It not in my patchwork list, so I not pick it. Please directly pick it up since it is ok to you.

Thanks,
Peng.

> 
> --
> Tom

[U-Boot] [PATCH] spl: mmc: Add option to set eMMC HW boot partition

[Tom Rini]

On Tue, Sep 03, 2019 at 04:16:50PM +0200, Lukasz Majewski wrote:

> From: Mans Rullgard <mans@mansr.com>
> 
> This change allows setting pre-defined eMMC boot partition for SPL eMMC
> booting. It is necessary in the case when one wants to boot (through falcon
> boot) from eMMC after loading SPL from other memory (like SPI-NOR).
> 
> Signed-off-by: Mans Rullgard <mans@mansr.com>
> [lukma: Edit the commit message]
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> Acked-by: Andreas Dannenberg <dannenberg@ti.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20191012/c627b42d/attachment.sig>