All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM
@ 2016-08-11  6:09 Peng Fan
  2016-08-26 13:32 ` Stefano Babic
  2016-08-26 14:03 ` Stefano Babic
  0 siblings, 2 replies; 5+ messages in thread
From: Peng Fan @ 2016-08-11  6:09 UTC (permalink / raw)
  To: u-boot

To i.MX7, different boot medias' IVT offset is fixed at 0x400.
So consolidate BOOT_FROM to use FLASH_OFFSET_STANDARD.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Cc: Stefano Babic <sbabic@denx.de>
---
 board/freescale/mx7dsabresd/imximage.cfg | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/board/freescale/mx7dsabresd/imximage.cfg b/board/freescale/mx7dsabresd/imximage.cfg
index 76574ff..7314316 100644
--- a/board/freescale/mx7dsabresd/imximage.cfg
+++ b/board/freescale/mx7dsabresd/imximage.cfg
@@ -16,11 +16,8 @@
 
 IMAGE_VERSION 2
 
-/*
- * Boot Device : sd
- */
-
-BOOT_FROM	sd
+/* valid for various boot medias */
+BOOT_FROM	FLASH_OFFSET_STANDARD
 
 /*
  * Secure boot support
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM
  2016-08-11  6:09 [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM Peng Fan
@ 2016-08-26 13:32 ` Stefano Babic
  2016-08-26 14:03 ` Stefano Babic
  1 sibling, 0 replies; 5+ messages in thread
From: Stefano Babic @ 2016-08-26 13:32 UTC (permalink / raw)
  To: u-boot

On 11/08/2016 08:09, Peng Fan wrote:
> To i.MX7, different boot medias' IVT offset is fixed at 0x400.
> So consolidate BOOT_FROM to use FLASH_OFFSET_STANDARD.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
>  board/freescale/mx7dsabresd/imximage.cfg | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/board/freescale/mx7dsabresd/imximage.cfg b/board/freescale/mx7dsabresd/imximage.cfg
> index 76574ff..7314316 100644
> --- a/board/freescale/mx7dsabresd/imximage.cfg
> +++ b/board/freescale/mx7dsabresd/imximage.cfg
> @@ -16,11 +16,8 @@
>  
>  IMAGE_VERSION 2
>  
> -/*
> - * Boot Device : sd
> - */
> -
> -BOOT_FROM	sd
> +/* valid for various boot medias */
> +BOOT_FROM	FLASH_OFFSET_STANDARD
>  
>  /*
>   * Secure boot support
> 

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: sbabic at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM
  2016-08-11  6:09 [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM Peng Fan
  2016-08-26 13:32 ` Stefano Babic
@ 2016-08-26 14:03 ` Stefano Babic
  2016-08-28 11:52   ` Peng Fan
  1 sibling, 1 reply; 5+ messages in thread
From: Stefano Babic @ 2016-08-26 14:03 UTC (permalink / raw)
  To: u-boot

Hi Peng,

On 11/08/2016 08:09, Peng Fan wrote:
> To i.MX7, different boot medias' IVT offset is fixed at 0x400.
> So consolidate BOOT_FROM to use FLASH_OFFSET_STANDARD.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
>  board/freescale/mx7dsabresd/imximage.cfg | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/board/freescale/mx7dsabresd/imximage.cfg b/board/freescale/mx7dsabresd/imximage.cfg
> index 76574ff..7314316 100644
> --- a/board/freescale/mx7dsabresd/imximage.cfg
> +++ b/board/freescale/mx7dsabresd/imximage.cfg
> @@ -16,11 +16,8 @@
>  
>  IMAGE_VERSION 2
>  
> -/*
> - * Boot Device : sd
> - */
> -
> -BOOT_FROM	sd
> +/* valid for various boot medias */
> +BOOT_FROM	FLASH_OFFSET_STANDARD
>  

But this is not:


BOOT_OFFSET	FLASH_OFFSET_STANDARD

istead of BOTT_FROM ? It looks to me wrong.

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: sbabic at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM
  2016-08-26 14:03 ` Stefano Babic
@ 2016-08-28 11:52   ` Peng Fan
  2016-08-28 14:26     ` Stefano Babic
  0 siblings, 1 reply; 5+ messages in thread
From: Peng Fan @ 2016-08-28 11:52 UTC (permalink / raw)
  To: u-boot

Hi Stefano,

On Fri, Aug 26, 2016 at 04:03:00PM +0200, Stefano Babic wrote:
>Hi Peng,
>
>On 11/08/2016 08:09, Peng Fan wrote:
>> To i.MX7, different boot medias' IVT offset is fixed at 0x400.
>> So consolidate BOOT_FROM to use FLASH_OFFSET_STANDARD.
>> 
>> Signed-off-by: Peng Fan <peng.fan@nxp.com>
>> Cc: Stefano Babic <sbabic@denx.de>
>> ---
>>  board/freescale/mx7dsabresd/imximage.cfg | 7 ++-----
>>  1 file changed, 2 insertions(+), 5 deletions(-)
>> 
>> diff --git a/board/freescale/mx7dsabresd/imximage.cfg b/board/freescale/mx7dsabresd/imximage.cfg
>> index 76574ff..7314316 100644
>> --- a/board/freescale/mx7dsabresd/imximage.cfg
>> +++ b/board/freescale/mx7dsabresd/imximage.cfg
>> @@ -16,11 +16,8 @@
>>  
>>  IMAGE_VERSION 2
>>  
>> -/*
>> - * Boot Device : sd
>> - */
>> -
>> -BOOT_FROM	sd
>> +/* valid for various boot medias */
>> +BOOT_FROM	FLASH_OFFSET_STANDARD
>>  
>
>But this is not:
>
>
>BOOT_OFFSET	FLASH_OFFSET_STANDARD
>
>istead of BOTT_FROM ? It looks to me wrong.

My bad. You are right. I missed to do a compile test for this.

Thanks,
Peng.

>
>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: sbabic at denx.de
>=====================================================================

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM
  2016-08-28 11:52   ` Peng Fan
@ 2016-08-28 14:26     ` Stefano Babic
  0 siblings, 0 replies; 5+ messages in thread
From: Stefano Babic @ 2016-08-28 14:26 UTC (permalink / raw)
  To: u-boot

Hi Peng,

On 28/08/2016 13:52, Peng Fan wrote:
> Hi Stefano,
> 
> On Fri, Aug 26, 2016 at 04:03:00PM +0200, Stefano Babic wrote:
>> Hi Peng,
>>
>> On 11/08/2016 08:09, Peng Fan wrote:
>>> To i.MX7, different boot medias' IVT offset is fixed at 0x400.
>>> So consolidate BOOT_FROM to use FLASH_OFFSET_STANDARD.
>>>
>>> Signed-off-by: Peng Fan <peng.fan@nxp.com>
>>> Cc: Stefano Babic <sbabic@denx.de>
>>> ---
>>>  board/freescale/mx7dsabresd/imximage.cfg | 7 ++-----
>>>  1 file changed, 2 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/board/freescale/mx7dsabresd/imximage.cfg b/board/freescale/mx7dsabresd/imximage.cfg
>>> index 76574ff..7314316 100644
>>> --- a/board/freescale/mx7dsabresd/imximage.cfg
>>> +++ b/board/freescale/mx7dsabresd/imximage.cfg
>>> @@ -16,11 +16,8 @@
>>>  
>>>  IMAGE_VERSION 2
>>>  
>>> -/*
>>> - * Boot Device : sd
>>> - */
>>> -
>>> -BOOT_FROM	sd
>>> +/* valid for various boot medias */
>>> +BOOT_FROM	FLASH_OFFSET_STANDARD
>>>  
>>
>> But this is not:
>>
>>
>> BOOT_OFFSET	FLASH_OFFSET_STANDARD
>>
>> istead of BOTT_FROM ? It looks to me wrong.
> 
> My bad. You are right. I missed to do a compile test for this.

Ok, no problem. I fix it myself and I apply it.

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: sbabic at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-08-28 14:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-11  6:09 [U-Boot] [PATCH] imx: mx7dsabresd: consolidate BOOT_FROM Peng Fan
2016-08-26 13:32 ` Stefano Babic
2016-08-26 14:03 ` Stefano Babic
2016-08-28 11:52   ` Peng Fan
2016-08-28 14:26     ` Stefano Babic

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.