Re: [PATCH] ARM: dts: exynos5422-odroidxu3: reduce total RAM by 22 MiB

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2015-01-23 at 15:35 +0900, Kukjin Kim wrote:
> Marek Szyprowski wrote:
> > 
> > Last 22 MiB is RAM is reserved by secure monitor code and cannot be
> > accessed from Linux kernel, so adjust total RAM size to 0x7EA00000
> > (2 GiB - 22 MiB). This fixes random 'imprecise kernel abort' kernel
> > failures.
> > 
> > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> > ---
> >  arch/arm/boot/dts/exynos5422-odroidxu3.dts | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3.dts b/arch/arm/boot/dts/exynos5422-odroidxu3.dts
> > index f6fc9442f631..50843208860d 100644
> > --- a/arch/arm/boot/dts/exynos5422-odroidxu3.dts
> > +++ b/arch/arm/boot/dts/exynos5422-odroidxu3.dts
> > @@ -18,7 +18,7 @@
> >  	compatible = "hardkernel,odroid-xu3", "samsung,exynos5800", "samsung,exynos5";
> > 
> >  	memory {
> > -		reg = <0x40000000 0x80000000>;
> > +		reg = <0x40000000 0x7EA00000>;
> >  	};
> > 
> >  	chosen {
> > --

Just checked hardkernels bootloader which defines:
  #define CONFIG_TRUSTZONE_RESERVED_DRAM  0x1600000

So looks good,

Reviewed-By: Sjoerd Simons <sjoerd.simons@xxxxxxxxxxxxxxx>

> Hi,
> 
> Maybe is it related to the SoC not only for odriodxu3 board. If so, following
> would be better?
> 
> ---8<------------8<---
> diff --git a/arch/arm/boot/dts/exynos5800.dtsi b/arch/arm/boot/dts/exynos5800.dtsi
> index c0bb356..54840e3 100644
> --- a/arch/arm/boot/dts/exynos5800.dtsi
> +++ b/arch/arm/boot/dts/exynos5800.dtsi
> @@ -13,6 +13,8 @@
>   * published by the Free Software Foundation.
>   */
> 
> +/memreserve/ 0x80000000 0x1600000;
> +
>  #include "exynos5420.dtsi"
> 
>  / {
> ---8<------------8<---
> 
> - Kukjin
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


Attachment: smime.p7s
Description: S/MIME cryptographic signature


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux