diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index e16f116..da7f27f 100644
--- a/drivers/ddr/altera/sdram.c
+++ b/drivers/ddr/altera/sdram.c
@@ -117,8 +117,8 @@
 	writel(ruleno, &sdr_ctrl->prot_rule_rdwr);
 
 	/* Obtain the address bits */
-	lo_addr_bits = (uint32_t)(((prule->sdram_start) >> 20ULL) & 0xFFF);
-	hi_addr_bits = (uint32_t)((((prule->sdram_end-1) >> 20ULL)) & 0xFFF);
+	lo_addr_bits = prule->sdram_start >> 20ULL;
+	hi_addr_bits = (prule->sdram_end - 1) >> 20ULL;
 
 	debug("sdram set rule start %x, %lld\n", lo_addr_bits,
 	      prule->sdram_start);
@@ -138,7 +138,7 @@
 	       &sdr_ctrl->prot_rule_data);
 
 	/* write the rule */
-	writel(ruleno | (1L << 5), &sdr_ctrl->prot_rule_rdwr);
+	writel(ruleno | (1 << 5), &sdr_ctrl->prot_rule_rdwr);
 
 	/* Set rule number to 0 by default */
 	writel(0, &sdr_ctrl->prot_rule_rdwr);
@@ -183,7 +183,7 @@
 	writel(0x0, &sdr_ctrl->protport_default);
 
 	/* Clear all protection rules for warm boot case */
-	memset(&rule, 0, sizeof(struct sdram_prot_rule));
+	memset(&rule, 0, sizeof(rule));
 
 	for (rules = 0; rules < 20; rules++) {
 		rule.rule = rules;