From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Mon, 17 Aug 2015 11:05:01 +0200 Subject: [U-Boot] [PATCH 10/15] sunxi_nand_spl: Remove NAND_SUNXI_SPL_SYNDROME_PARTITIONS_END In-Reply-To: <1439800021.3480.63.camel@hellion.org.uk> References: <1439668968-3882-1-git-send-email-hdegoede@redhat.com> <1439668968-3882-11-git-send-email-hdegoede@redhat.com> <1439800021.3480.63.camel@hellion.org.uk> Message-ID: <55D1A3BD.10301@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 17-08-15 10:27, Ian Campbell wrote: > On Sat, 2015-08-15 at 22:02 +0200, Hans de Goede wrote: >> We currently only use the spl nand code to get u-boot itself, which >> _always_ is located on a syndrome partition. Once we figure out how >> we are going to store the u-boot env on nand, we may need non syndrome >> support, but even then there will likely be better ways to determine >> whether to use syndrome or not (e.g. look at the loadaddr passed in). >> >> Signed-off-by: Hans de Goede > > I suppose I should back reference my question as to whether we should > remove this option (even though it seems to have ended up the other way > round to what I was anticipating). Actually I've been discussing nand partitioning with Boris Brezillon and it looks like that we need may need to use a non syndrome partition for u-boot on some nands, because the ecc options supported by the BROM are too weak for this nand, so we will only use a BROM compatible ecc setting for the spl and use a separate nand partition for u-boot. We're still figuring this out, but I think I'll drop this patch for now, as we may need this after all. Regards, Hans