From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eugeniu Rosca <roscaeugeniu@gmail.com>
Date: Sat, 30 Mar 2019 22:01:22 +0100
Subject: [U-Boot] [PATCH] ARM: rmobile: rcar-gen2: Activate bootm_size
In-Reply-To: <20190330064734.11989-1-marek.vasut+renesas@gmail.com>
References: <20190330064734.11989-1-marek.vasut+renesas@gmail.com>
Message-ID: <20190330210122.GA6158@x230>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Marek,

On Sat, Mar 30, 2019 at 07:47:34AM +0100, Marek Vasut wrote:
> Commit d245059ff797 ("ARM: rmobile: rcar-gen3: Activate bootm_size")
> only fixed the superfluous CONFIG_SYS_BOOTMAPSZ for R-Car Gen3 [..]

I don't think 'superfluous' is the best wording here, since
(according to its README description) CONFIG_SYS_BOOTMAPSZ
invalidates/overrides the user-provided 'bootm_size' by taking
precedence over the latter.

But I agree with the contents of this patch, since it resembles the
aforementioned commit and brings more harmony between Gen2 and Gen3.

I don't have any Gen2 board to test it, but from code review standpoint:

Reviewed-by: Eugeniu Rosca <rosca.eugeniu@gmail.com>

Best regards,
Eugeniu.