Commit d518d9ca authored by Rojhalat Ibrahim's avatar Rojhalat Ibrahim Committed by Philipp Zabel

reset-socfpga: Fix nr_resets property

The SoC-FPGA reset controller driver defines NR_BANKS as 4 and uses that define
for two unrelated purposes. It is used
1. as an increment for reset line banks which are 32-bit registers with 4-byte
aligned addresses.
2. as the total number of reset line banks which together with the number of
resets per bank (32) limits the total number of useable resets to 128 and the
highest useable reset ID to 127.

This is clearly wrong as there are resets with higher IDs than 127 defined in
include/dt-bindings/reset/altr,rst-mgr.h and altr,rst-mgr-a10.h.

The patch introduces a new define BANK_INCREMENT for calculating the register
addresses as before and increases NR_BANKS to 8 for useable reset IDs up to 255.
Signed-off-by: default avatarRojhalat Ibrahim <imr@rtschenk.de>
Signed-off-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
parent 716adfe3
...@@ -25,7 +25,8 @@ ...@@ -25,7 +25,8 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/types.h> #include <linux/types.h>
#define NR_BANKS 4 #define BANK_INCREMENT 4
#define NR_BANKS 8
struct socfpga_reset_data { struct socfpga_reset_data {
spinlock_t lock; spinlock_t lock;
...@@ -46,8 +47,8 @@ static int socfpga_reset_assert(struct reset_controller_dev *rcdev, ...@@ -46,8 +47,8 @@ static int socfpga_reset_assert(struct reset_controller_dev *rcdev,
spin_lock_irqsave(&data->lock, flags); spin_lock_irqsave(&data->lock, flags);
reg = readl(data->membase + (bank * NR_BANKS)); reg = readl(data->membase + (bank * BANK_INCREMENT));
writel(reg | BIT(offset), data->membase + (bank * NR_BANKS)); writel(reg | BIT(offset), data->membase + (bank * BANK_INCREMENT));
spin_unlock_irqrestore(&data->lock, flags); spin_unlock_irqrestore(&data->lock, flags);
return 0; return 0;
...@@ -67,8 +68,8 @@ static int socfpga_reset_deassert(struct reset_controller_dev *rcdev, ...@@ -67,8 +68,8 @@ static int socfpga_reset_deassert(struct reset_controller_dev *rcdev,
spin_lock_irqsave(&data->lock, flags); spin_lock_irqsave(&data->lock, flags);
reg = readl(data->membase + (bank * NR_BANKS)); reg = readl(data->membase + (bank * BANK_INCREMENT));
writel(reg & ~BIT(offset), data->membase + (bank * NR_BANKS)); writel(reg & ~BIT(offset), data->membase + (bank * BANK_INCREMENT));
spin_unlock_irqrestore(&data->lock, flags); spin_unlock_irqrestore(&data->lock, flags);
...@@ -84,7 +85,7 @@ static int socfpga_reset_status(struct reset_controller_dev *rcdev, ...@@ -84,7 +85,7 @@ static int socfpga_reset_status(struct reset_controller_dev *rcdev,
int offset = id % BITS_PER_LONG; int offset = id % BITS_PER_LONG;
u32 reg; u32 reg;
reg = readl(data->membase + (bank * NR_BANKS)); reg = readl(data->membase + (bank * BANK_INCREMENT));
return !(reg & BIT(offset)); return !(reg & BIT(offset));
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment