All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] exynos: Drop large alignment for SDRAM parameters
@ 2017-02-08 12:43 ` Simon Glass
  2017-02-09  8:31   ` Jaehoon Chung
  0 siblings, 1 reply; 3+ messages in thread
From: Simon Glass @ 2017-02-08 12:43 UTC (permalink / raw)
  To: u-boot

We don't ever search for these so there is no need for a 4KB alignment.
It just wastes space.

Drop this and use the standard 4-byte alignment.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 board/samsung/common/exynos-uboot-spl.lds | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/board/samsung/common/exynos-uboot-spl.lds b/board/samsung/common/exynos-uboot-spl.lds
index 4a933c87e9..8c497792ee 100644
--- a/board/samsung/common/exynos-uboot-spl.lds
+++ b/board/samsung/common/exynos-uboot-spl.lds
@@ -38,8 +38,7 @@ SECTIONS
 	} >.sram
 	. = ALIGN(4);
 
-	/* Align .machine_param on 256 byte boundary for easier searching */
-	.machine_param ALIGN(0x100) : { *(.machine_param) } >.sram
+	.machine_param : { *(.machine_param) } >.sram
 	. = ALIGN(4);
 
 	__image_copy_end = .;
-- 
2.11.0.483.g087da7b7c-goog

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

* [U-Boot] [PATCH] exynos: Drop large alignment for SDRAM parameters
  2017-02-08 12:43 ` [U-Boot] [PATCH] exynos: Drop large alignment for SDRAM parameters Simon Glass
@ 2017-02-09  8:31   ` Jaehoon Chung
  2017-02-11 13:38     ` Minkyu Kang
  0 siblings, 1 reply; 3+ messages in thread
From: Jaehoon Chung @ 2017-02-09  8:31 UTC (permalink / raw)
  To: u-boot

On 02/08/2017 09:43 PM, Simon Glass wrote:
> We don't ever search for these so there is no need for a 4KB alignment.
> It just wastes space.
> 
> Drop this and use the standard 4-byte alignment.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>

Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>

Best Regards,
Jaehoon Chung

> ---
> 
>  board/samsung/common/exynos-uboot-spl.lds | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/board/samsung/common/exynos-uboot-spl.lds b/board/samsung/common/exynos-uboot-spl.lds
> index 4a933c87e9..8c497792ee 100644
> --- a/board/samsung/common/exynos-uboot-spl.lds
> +++ b/board/samsung/common/exynos-uboot-spl.lds
> @@ -38,8 +38,7 @@ SECTIONS
>  	} >.sram
>  	. = ALIGN(4);
>  
> -	/* Align .machine_param on 256 byte boundary for easier searching */
> -	.machine_param ALIGN(0x100) : { *(.machine_param) } >.sram
> +	.machine_param : { *(.machine_param) } >.sram
>  	. = ALIGN(4);
>  
>  	__image_copy_end = .;
> 

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

* [U-Boot] [PATCH] exynos: Drop large alignment for SDRAM parameters
  2017-02-09  8:31   ` Jaehoon Chung
@ 2017-02-11 13:38     ` Minkyu Kang
  0 siblings, 0 replies; 3+ messages in thread
From: Minkyu Kang @ 2017-02-11 13:38 UTC (permalink / raw)
  To: u-boot

Dear Simon Glass,

On 9 February 2017 at 17:31, Jaehoon Chung <jh80.chung@samsung.com> wrote:

> On 02/08/2017 09:43 PM, Simon Glass wrote:
> > We don't ever search for these so there is no need for a 4KB alignment.
> > It just wastes space.
> >
> > Drop this and use the standard 4-byte alignment.
> >
> > Signed-off-by: Simon Glass <sjg@chromium.org>
>
> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
>
>
applied to u-boot-samsung.

Thanks,
Minkyu Kang.
-- 
from. prom.
www.promsoft.net

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

end of thread, other threads:[~2017-02-11 13:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20170208124440epcas1p17535f39c83b15f4c1fb3325f9b3e896a@epcas1p1.samsung.com>
2017-02-08 12:43 ` [U-Boot] [PATCH] exynos: Drop large alignment for SDRAM parameters Simon Glass
2017-02-09  8:31   ` Jaehoon Chung
2017-02-11 13:38     ` Minkyu Kang

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.