[U-Boot] [PATCH 04/15] sunxi_nand_spl: Do not bother writing the spare-area reg in syndrome mode

[Ian Campbell <ijc@hellion.org.uk>]

On Sat, 2015-08-15 at 22:02 +0200, Hans de Goede wrote:
> In syndrome mode we set the NFC_SEQ bit in the command register, so the
> spare-area register is not used. Also the value currently being written is
> actual wrong, the ecc sits at "column + CONFIG_NAND_SUNXI_SPL_ECC_PAGE_SIZE"
> not just CONFIG_NAND_SUNXI_SPL_ECC_PAGE_SIZE.
> 
> So the current code only serves to confuse the user -> remove it.
> 
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

There's a bunch of other uses of the syndrome parameter in this
function. Does syndrome=true work even without this particular bit of
code?

I suppose I'm asking, should the paramter and the other uses be
removed? Or should an ASSERT(!syndrome) be added, or am I worrying
about nothing and everything is just fine as it is after this patch?

I suspect the latter, so if that is indeed the case:
    Acked-by: Ian Campbell <    ijc@hellion.org.uk    >