On 6 April 2016 at 19:04, Marek Vasut <ma...@denx.de> wrote: > The hi address bitfield in the protection rule must be set to > the last address in the region which the rule represents. The > behavior is now in-line with code generated by Quartus 15.1 . > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Dinh Nguyen <dingu...@opensource.altera.com> > Cc: Chin Liang See <cl...@altera.com> > --- > drivers/ddr/altera/sdram.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c > index 2996942..7e4606d 100644 > --- a/drivers/ddr/altera/sdram.c > +++ b/drivers/ddr/altera/sdram.c > @@ -118,7 +118,7 @@ static void sdram_set_rule(struct sdram_prot_rule *prule) > > /* Obtain the address bits */ > lo_addr_bits = prule->sdram_start >> 20ULL; > - hi_addr_bits = prule->sdram_end >> 20ULL; > + hi_addr_bits = (prule->sdram_end - 1) >> 20ULL; > > debug("sdram set rule start %x, %d\n", lo_addr_bits, > prule->sdram_start); > -- > 2.7.0 >
Tested on: SoCKit, DE0_Nano_SoC Tested-by: George Broz <brozgeo at gmail.com> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot