* AT91: PIO base-address should be calculated by processor-specific code
@ 2011-04-29 14:11 Andrew Victor
2011-04-29 14:53 ` Andrew Victor
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Andrew Victor @ 2011-04-29 14:11 UTC (permalink / raw)
To: linux-arm-kernel
For supporting multiple AT91 processors in a single kernel image, the
base-address of the system peripherals is processor-dependent. The GPIO
driver should therefore not reference AT91_VA_BASE_SYS, but rather the
correct peripheral base-address needs to be provided by the
processor-specific initialization code.
Signed-off-by: Andrew Victor <linux@maxim.org.za>
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at572d940hf.c linux-2.6/arch/arm/mach-at91/at572d940hf.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at572d940hf.c 2010-12-24 16:52:59.323332139 +0200
+++ linux-2.6/arch/arm/mach-at91/at572d940hf.c 2011-04-28 14:51:20.626598251 +0200
@@ -279,15 +279,15 @@
static struct at91_gpio_bank at572d940hf_gpio[] = {
{
.id = AT572D940HF_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT572D940HF_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT572D940HF_ID_PIOC,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioC_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91cap9.c linux-2.6/arch/arm/mach-at91/at91cap9.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91cap9.c 2009-06-10 05:05:27.000000000 +0200
+++ linux-2.6/arch/arm/mach-at91/at91cap9.c 2011-04-28 14:52:15.627736528 +0200
@@ -271,19 +271,19 @@
static struct at91_gpio_bank at91cap9_gpio[] = {
{
.id = AT91CAP9_ID_PIOABCD,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioABCD_clk,
}, {
.id = AT91CAP9_ID_PIOABCD,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioABCD_clk,
}, {
.id = AT91CAP9_ID_PIOABCD,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioABCD_clk,
}, {
.id = AT91CAP9_ID_PIOABCD,
- .offset = AT91_PIOD,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
.clock = &pioABCD_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91rm9200.c linux-2.6/arch/arm/mach-at91/at91rm9200.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91rm9200.c 2009-06-10 05:05:27.000000000 +0200
+++ linux-2.6/arch/arm/mach-at91/at91rm9200.c 2011-04-28 14:33:14.161390776 +0200
@@ -240,19 +240,19 @@
static struct at91_gpio_bank at91rm9200_gpio[] = {
{
.id = AT91RM9200_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT91RM9200_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT91RM9200_ID_PIOC,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioC_clk,
}, {
.id = AT91RM9200_ID_PIOD,
- .offset = AT91_PIOD,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
.clock = &pioD_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9260.c linux-2.6/arch/arm/mach-at91/at91sam9260.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9260.c 2011-04-26 20:02:18.348726273 +0200
+++ linux-2.6/arch/arm/mach-at91/at91sam9260.c 2011-04-28 14:53:53.510630116 +0200
@@ -266,15 +266,15 @@
static struct at91_gpio_bank at91sam9260_gpio[] = {
{
.id = AT91SAM9260_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT91SAM9260_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT91SAM9260_ID_PIOC,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioC_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9261.c linux-2.6/arch/arm/mach-at91/at91sam9261.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9261.c 2011-04-26 20:02:18.348726273 +0200
+++ linux-2.6/arch/arm/mach-at91/at91sam9261.c 2011-04-28 14:54:36.792366992 +0200
@@ -244,15 +244,15 @@
static struct at91_gpio_bank at91sam9261_gpio[] = {
{
.id = AT91SAM9261_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT91SAM9261_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT91SAM9261_ID_PIOC,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioC_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9263.c linux-2.6/arch/arm/mach-at91/at91sam9263.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9263.c 2011-04-26 20:02:18.348726273 +0200
+++ linux-2.6/arch/arm/mach-at91/at91sam9263.c 2011-04-28 14:55:50.240519738 +0200
@@ -248,23 +248,23 @@
static struct at91_gpio_bank at91sam9263_gpio[] = {
{
.id = AT91SAM9263_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT91SAM9263_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT91SAM9263_ID_PIOCDE,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioCDE_clk,
}, {
.id = AT91SAM9263_ID_PIOCDE,
- .offset = AT91_PIOD,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
.clock = &pioCDE_clk,
}, {
.id = AT91SAM9263_ID_PIOCDE,
- .offset = AT91_PIOE,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOE,
.clock = &pioCDE_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9g45.c linux-2.6/arch/arm/mach-at91/at91sam9g45.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9g45.c 2010-12-24 16:54:07.564396678 +0200
+++ linux-2.6/arch/arm/mach-at91/at91sam9g45.c 2011-04-28 14:56:43.621718542 +0200
@@ -270,23 +270,23 @@
static struct at91_gpio_bank at91sam9g45_gpio[] = {
{
.id = AT91SAM9G45_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT91SAM9G45_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT91SAM9G45_ID_PIOC,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioC_clk,
}, {
.id = AT91SAM9G45_ID_PIODE,
- .offset = AT91_PIOD,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
.clock = &pioDE_clk,
}, {
.id = AT91SAM9G45_ID_PIODE,
- .offset = AT91_PIOE,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOE,
.clock = &pioDE_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9rl.c linux-2.6/arch/arm/mach-at91/at91sam9rl.c
--- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9rl.c 2011-04-26 20:02:18.348726273 +0200
+++ linux-2.6/arch/arm/mach-at91/at91sam9rl.c 2011-04-28 14:57:24.215612004 +0200
@@ -225,19 +225,19 @@
static struct at91_gpio_bank at91sam9rl_gpio[] = {
{
.id = AT91SAM9RL_ID_PIOA,
- .offset = AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
.clock = &pioA_clk,
}, {
.id = AT91SAM9RL_ID_PIOB,
- .offset = AT91_PIOB,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
.clock = &pioB_clk,
}, {
.id = AT91SAM9RL_ID_PIOC,
- .offset = AT91_PIOC,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
.clock = &pioC_clk,
}, {
.id = AT91SAM9RL_ID_PIOD,
- .offset = AT91_PIOD,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
.clock = &pioD_clk,
}
};
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/generic.h linux-2.6/arch/arm/mach-at91/generic.h
--- linux-2.6.38.at91/arch/arm/mach-at91/generic.h 2011-04-28 17:01:30.074425436 +0200
+++ linux-2.6/arch/arm/mach-at91/generic.h 2011-04-28 14:39:51.365842238 +0200
@@ -56,7 +56,7 @@
struct at91_gpio_bank {
unsigned short id; /* peripheral ID */
- unsigned long offset; /* offset from system peripheral base */
+ void __iomem *regbase; /* base address of PIO register bank */
struct clk *clock; /* associated clock */
};
extern void __init at91_gpio_init(struct at91_gpio_bank *, int nr_banks);
diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/gpio.c linux-2.6/arch/arm/mach-at91/gpio.c
--- linux-2.6.38.at91/arch/arm/mach-at91/gpio.c 2011-04-28 17:01:35.838026699 +0200
+++ linux-2.6/arch/arm/mach-at91/gpio.c 2011-04-28 14:40:55.418627104 +0200
@@ -32,7 +32,6 @@
struct gpio_chip chip;
struct at91_gpio_chip *next; /* Bank sharing same clock */
struct at91_gpio_bank *bank; /* Bank definition */
- void __iomem *regbase; /* Base of register bank */
};
#define to_at91_gpio_chip(c) container_of(c, struct at91_gpio_chip, chip)
@@ -74,7 +73,7 @@
pin -= PIN_BASE;
pin /= 32;
if (likely(pin < gpio_banks))
- return gpio_chip[pin].regbase;
+ return gpio_chip[pin].bank->regbase;
return NULL;
}
@@ -297,7 +296,7 @@
int i;
for (i = 0; i < gpio_banks; i++) {
- void __iomem *pio = gpio_chip[i].regbase;
+ void __iomem *pio = gpio_chip[i].bank->regbase;
backups[i] = __raw_readl(pio + PIO_IMR);
__raw_writel(backups[i], pio + PIO_IDR);
@@ -318,7 +317,7 @@
int i;
for (i = 0; i < gpio_banks; i++) {
- void __iomem *pio = gpio_chip[i].regbase;
+ void __iomem *pio = gpio_chip[i].bank->regbase;
if (!wakeups[i])
clk_enable(gpio_chip[i].bank->clock);
@@ -390,7 +389,7 @@
u32 isr;
at91_gpio = get_irq_chip_data(irq);
- pio = at91_gpio->regbase;
+ pio = at91_gpio->bank->regbase;
for (;;) {
/* Reading ISR acks pending (edge triggered) GPIO interrupts.
@@ -402,7 +401,7 @@
if (!at91_gpio->next)
break;
at91_gpio = at91_gpio->next;
- pio = at91_gpio->regbase;
+ pio = at91_gpio->bank->regbase;
continue;
}
@@ -513,7 +512,7 @@
unsigned id = this->bank->id;
unsigned i;
- __raw_writel(~0, this->regbase + PIO_IDR);
+ __raw_writel(~0, this->bank->regbase + PIO_IDR);
for (i = 0, pin = this->chip.base; i < 32; i++, pin++) {
lockdep_set_class(&irq_desc[pin].lock, &gpio_lock_class);
@@ -545,7 +544,7 @@
unsigned offset)
{
struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
- void __iomem *pio = at91_gpio->regbase;
+ void __iomem *pio = at91_gpio->bank->regbase;
unsigned mask = 1 << offset;
__raw_writel(mask, pio + PIO_ODR);
@@ -556,7 +555,7 @@
unsigned offset, int val)
{
struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
- void __iomem *pio = at91_gpio->regbase;
+ void __iomem *pio = at91_gpio->bank->regbase;
unsigned mask = 1 << offset;
__raw_writel(mask, pio + (val ? PIO_SODR : PIO_CODR));
@@ -567,7 +566,7 @@
static int at91_gpiolib_get(struct gpio_chip *chip, unsigned offset)
{
struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
- void __iomem *pio = at91_gpio->regbase;
+ void __iomem *pio = at91_gpio->bank->regbase;
unsigned mask = 1 << offset;
u32 pdsr;
@@ -578,7 +577,7 @@
static void at91_gpiolib_set(struct gpio_chip *chip, unsigned offset, int val)
{
struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
- void __iomem *pio = at91_gpio->regbase;
+ void __iomem *pio = at91_gpio->bank->regbase;
unsigned mask = 1 << offset;
__raw_writel(mask, pio + (val ? PIO_SODR : PIO_CODR));
@@ -627,8 +626,6 @@
at91_gpio->bank = &data[i];
at91_gpio->chip.base = PIN_BASE + i * 32;
- at91_gpio->regbase = at91_gpio->bank->offset +
- (void __iomem *)AT91_VA_BASE_SYS;
/* enable PIO controller's clock */
clk_enable(at91_gpio->bank->clock);
^ permalink raw reply [flat|nested] 4+ messages in thread
* AT91: PIO base-address should be calculated by processor-specific code
2011-04-29 14:11 AT91: PIO base-address should be calculated by processor-specific code Andrew Victor
@ 2011-04-29 14:53 ` Andrew Victor
2011-04-30 1:28 ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-04 20:53 ` Ryan Mallon
2 siblings, 0 replies; 4+ messages in thread
From: Andrew Victor @ 2011-04-29 14:53 UTC (permalink / raw)
To: linux-arm-kernel
hi,
As an improvement to this patch, it's probably better to also include
this changes:
{
.id = AT91RM9200_ID_PIOA,
- .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
+ .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91RM9200_PIOA,
.clock = &pioA_clk,
}, {
and rename the AT91_PIOx definitions in the headers.
That would (1) denote they're processor-specific values,
and (2) get rid of conflicts when we remove the CPU-header selection
#ifdef from hardware.h
Regards,
Andrew Victor
On Fri, Apr 29, 2011 at 4:11 PM, Andrew Victor <avictor.za@gmail.com> wrote:
> For supporting multiple AT91 processors in a single kernel image, the
> base-address of the system peripherals is processor-dependent. ?The GPIO
> driver should therefore not reference AT91_VA_BASE_SYS, but rather the
> correct peripheral base-address needs to be provided by the
> processor-specific initialization code.
>
>
> Signed-off-by: Andrew Victor <linux@maxim.org.za>
>
>
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at572d940hf.c linux-2.6/arch/arm/mach-at91/at572d940hf.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at572d940hf.c ?2010-12-24 16:52:59.323332139 +0200
> +++ linux-2.6/arch/arm/mach-at91/at572d940hf.c ?2011-04-28 14:51:20.626598251 +0200
> @@ -279,15 +279,15 @@
> ?static struct at91_gpio_bank at572d940hf_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT572D940HF_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT572D940HF_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT572D940HF_ID_PIOC,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioC_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91cap9.c linux-2.6/arch/arm/mach-at91/at91cap9.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91cap9.c ? ? 2009-06-10 05:05:27.000000000 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91cap9.c ? ? 2011-04-28 14:52:15.627736528 +0200
> @@ -271,19 +271,19 @@
> ?static struct at91_gpio_bank at91cap9_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91CAP9_ID_PIOABCD,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioABCD_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91CAP9_ID_PIOABCD,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioABCD_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91CAP9_ID_PIOABCD,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioABCD_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91CAP9_ID_PIOABCD,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOD,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioABCD_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91rm9200.c linux-2.6/arch/arm/mach-at91/at91rm9200.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91rm9200.c ? 2009-06-10 05:05:27.000000000 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91rm9200.c ? 2011-04-28 14:33:14.161390776 +0200
> @@ -240,19 +240,19 @@
> ?static struct at91_gpio_bank at91rm9200_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91RM9200_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91RM9200_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91RM9200_ID_PIOC,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioC_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91RM9200_ID_PIOD,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOD,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioD_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9260.c linux-2.6/arch/arm/mach-at91/at91sam9260.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9260.c ?2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9260.c ?2011-04-28 14:53:53.510630116 +0200
> @@ -266,15 +266,15 @@
> ?static struct at91_gpio_bank at91sam9260_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9260_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9260_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9260_ID_PIOC,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioC_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9261.c linux-2.6/arch/arm/mach-at91/at91sam9261.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9261.c ?2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9261.c ?2011-04-28 14:54:36.792366992 +0200
> @@ -244,15 +244,15 @@
> ?static struct at91_gpio_bank at91sam9261_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9261_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9261_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9261_ID_PIOC,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioC_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9263.c linux-2.6/arch/arm/mach-at91/at91sam9263.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9263.c ?2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9263.c ?2011-04-28 14:55:50.240519738 +0200
> @@ -248,23 +248,23 @@
> ?static struct at91_gpio_bank at91sam9263_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9263_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9263_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9263_ID_PIOCDE,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioCDE_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9263_ID_PIOCDE,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOD,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioCDE_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9263_ID_PIOCDE,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOE,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOE,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioCDE_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9g45.c linux-2.6/arch/arm/mach-at91/at91sam9g45.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9g45.c ?2010-12-24 16:54:07.564396678 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9g45.c ?2011-04-28 14:56:43.621718542 +0200
> @@ -270,23 +270,23 @@
> ?static struct at91_gpio_bank at91sam9g45_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9G45_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9G45_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9G45_ID_PIOC,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioC_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9G45_ID_PIODE,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOD,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioDE_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9G45_ID_PIODE,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOE,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOE,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioDE_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9rl.c linux-2.6/arch/arm/mach-at91/at91sam9rl.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9rl.c ? 2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9rl.c ? 2011-04-28 14:57:24.215612004 +0200
> @@ -225,19 +225,19 @@
> ?static struct at91_gpio_bank at91sam9rl_gpio[] = {
> ? ? ? ?{
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9RL_ID_PIOA,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOA,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioA_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9RL_ID_PIOB,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOB,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioB_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9RL_ID_PIOC,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOC,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioC_clk,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.id ? ? ? ? ? ? = AT91SAM9RL_ID_PIOD,
> - ? ? ? ? ? ? ? .offset ? ? ? ? = AT91_PIOD,
> + ? ? ? ? ? ? ? .regbase ? ? ? ?= (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> ? ? ? ? ? ? ? ?.clock ? ? ? ? ?= &pioD_clk,
> ? ? ? ?}
> ?};
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/generic.h linux-2.6/arch/arm/mach-at91/generic.h
> --- linux-2.6.38.at91/arch/arm/mach-at91/generic.h ? ? ?2011-04-28 17:01:30.074425436 +0200
> +++ linux-2.6/arch/arm/mach-at91/generic.h ? ? ?2011-04-28 14:39:51.365842238 +0200
> @@ -56,7 +56,7 @@
>
> ?struct at91_gpio_bank {
> ? ? ? ?unsigned short id; ? ? ? ? ? ? ?/* peripheral ID */
> - ? ? ? unsigned long offset; ? ? ? ? ? /* offset from system peripheral base */
> + ? ? ? void __iomem *regbase; ? ? ? ? ?/* base address of PIO register bank */
> ? ? ? ?struct clk *clock; ? ? ? ? ? ? ?/* associated clock */
> ?};
> ?extern void __init at91_gpio_init(struct at91_gpio_bank *, int nr_banks);
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/gpio.c linux-2.6/arch/arm/mach-at91/gpio.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/gpio.c 2011-04-28 17:01:35.838026699 +0200
> +++ linux-2.6/arch/arm/mach-at91/gpio.c 2011-04-28 14:40:55.418627104 +0200
> @@ -32,7 +32,6 @@
> ? ? ? ?struct gpio_chip ? ? ? ?chip;
> ? ? ? ?struct at91_gpio_chip ? *next; ? ? ? ? ?/* Bank sharing same clock */
> ? ? ? ?struct at91_gpio_bank ? *bank; ? ? ? ? ?/* Bank definition */
> - ? ? ? void __iomem ? ? ? ? ? ?*regbase; ? ? ? /* Base of register bank */
> ?};
>
> ?#define to_at91_gpio_chip(c) container_of(c, struct at91_gpio_chip, chip)
> @@ -74,7 +73,7 @@
> ? ? ? ?pin -= PIN_BASE;
> ? ? ? ?pin /= 32;
> ? ? ? ?if (likely(pin < gpio_banks))
> - ? ? ? ? ? ? ? return gpio_chip[pin].regbase;
> + ? ? ? ? ? ? ? return gpio_chip[pin].bank->regbase;
>
> ? ? ? ?return NULL;
> ?}
> @@ -297,7 +296,7 @@
> ? ? ? ?int i;
>
> ? ? ? ?for (i = 0; i < gpio_banks; i++) {
> - ? ? ? ? ? ? ? void __iomem ? ?*pio = gpio_chip[i].regbase;
> + ? ? ? ? ? ? ? void __iomem ? ?*pio = gpio_chip[i].bank->regbase;
>
> ? ? ? ? ? ? ? ?backups[i] = __raw_readl(pio + PIO_IMR);
> ? ? ? ? ? ? ? ?__raw_writel(backups[i], pio + PIO_IDR);
> @@ -318,7 +317,7 @@
> ? ? ? ?int i;
>
> ? ? ? ?for (i = 0; i < gpio_banks; i++) {
> - ? ? ? ? ? ? ? void __iomem ? ?*pio = gpio_chip[i].regbase;
> + ? ? ? ? ? ? ? void __iomem ? ?*pio = gpio_chip[i].bank->regbase;
>
> ? ? ? ? ? ? ? ?if (!wakeups[i])
> ? ? ? ? ? ? ? ? ? ? ? ?clk_enable(gpio_chip[i].bank->clock);
> @@ -390,7 +389,7 @@
> ? ? ? ?u32 ? ? ? ? ? ? isr;
>
> ? ? ? ?at91_gpio = get_irq_chip_data(irq);
> - ? ? ? pio = at91_gpio->regbase;
> + ? ? ? pio = at91_gpio->bank->regbase;
>
> ? ? ? ?for (;;) {
> ? ? ? ? ? ? ? ?/* Reading ISR acks pending (edge triggered) GPIO interrupts.
> @@ -402,7 +401,7 @@
> ? ? ? ? ? ? ? ? ? ? ? ?if (!at91_gpio->next)
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?break;
> ? ? ? ? ? ? ? ? ? ? ? ?at91_gpio = at91_gpio->next;
> - ? ? ? ? ? ? ? ? ? ? ? pio = at91_gpio->regbase;
> + ? ? ? ? ? ? ? ? ? ? ? pio = at91_gpio->bank->regbase;
> ? ? ? ? ? ? ? ? ? ? ? ?continue;
> ? ? ? ? ? ? ? ?}
>
> @@ -513,7 +512,7 @@
> ? ? ? ? ? ? ? ?unsigned ? ? ? ?id = this->bank->id;
> ? ? ? ? ? ? ? ?unsigned ? ? ? ?i;
>
> - ? ? ? ? ? ? ? __raw_writel(~0, this->regbase + PIO_IDR);
> + ? ? ? ? ? ? ? __raw_writel(~0, this->bank->regbase + PIO_IDR);
>
> ? ? ? ? ? ? ? ?for (i = 0, pin = this->chip.base; i < 32; i++, pin++) {
> ? ? ? ? ? ? ? ? ? ? ? ?lockdep_set_class(&irq_desc[pin].lock, &gpio_lock_class);
> @@ -545,7 +544,7 @@
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?unsigned offset)
> ?{
> ? ? ? ?struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - ? ? ? void __iomem *pio = at91_gpio->regbase;
> + ? ? ? void __iomem *pio = at91_gpio->bank->regbase;
> ? ? ? ?unsigned mask = 1 << offset;
>
> ? ? ? ?__raw_writel(mask, pio + PIO_ODR);
> @@ -556,7 +555,7 @@
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned offset, int val)
> ?{
> ? ? ? ?struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - ? ? ? void __iomem *pio = at91_gpio->regbase;
> + ? ? ? void __iomem *pio = at91_gpio->bank->regbase;
> ? ? ? ?unsigned mask = 1 << offset;
>
> ? ? ? ?__raw_writel(mask, pio + (val ? PIO_SODR : PIO_CODR));
> @@ -567,7 +566,7 @@
> ?static int at91_gpiolib_get(struct gpio_chip *chip, unsigned offset)
> ?{
> ? ? ? ?struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - ? ? ? void __iomem *pio = at91_gpio->regbase;
> + ? ? ? void __iomem *pio = at91_gpio->bank->regbase;
> ? ? ? ?unsigned mask = 1 << offset;
> ? ? ? ?u32 pdsr;
>
> @@ -578,7 +577,7 @@
> ?static void at91_gpiolib_set(struct gpio_chip *chip, unsigned offset, int val)
> ?{
> ? ? ? ?struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - ? ? ? void __iomem *pio = at91_gpio->regbase;
> + ? ? ? void __iomem *pio = at91_gpio->bank->regbase;
> ? ? ? ?unsigned mask = 1 << offset;
>
> ? ? ? ?__raw_writel(mask, pio + (val ? PIO_SODR : PIO_CODR));
> @@ -627,8 +626,6 @@
>
> ? ? ? ? ? ? ? ?at91_gpio->bank = &data[i];
> ? ? ? ? ? ? ? ?at91_gpio->chip.base = PIN_BASE + i * 32;
> - ? ? ? ? ? ? ? at91_gpio->regbase = at91_gpio->bank->offset +
> - ? ? ? ? ? ? ? ? ? ? ? (void __iomem *)AT91_VA_BASE_SYS;
>
> ? ? ? ? ? ? ? ?/* enable PIO controller's clock */
> ? ? ? ? ? ? ? ?clk_enable(at91_gpio->bank->clock);
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* AT91: PIO base-address should be calculated by processor-specific code
2011-04-29 14:11 AT91: PIO base-address should be calculated by processor-specific code Andrew Victor
2011-04-29 14:53 ` Andrew Victor
@ 2011-04-30 1:28 ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-04 20:53 ` Ryan Mallon
2 siblings, 0 replies; 4+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2011-04-30 1:28 UTC (permalink / raw)
To: linux-arm-kernel
On 16:11 Fri 29 Apr , Andrew Victor wrote:
> For supporting multiple AT91 processors in a single kernel image, the
> base-address of the system peripherals is processor-dependent. The GPIO
> driver should therefore not reference AT91_VA_BASE_SYS, but rather the
> correct peripheral base-address needs to be provided by the
> processor-specific initialization code.
please don't I swtich to early device and clkdev thye are in confilt
and this is clean inside
Best Regards,
J.
^ permalink raw reply [flat|nested] 4+ messages in thread
* AT91: PIO base-address should be calculated by processor-specific code
2011-04-29 14:11 AT91: PIO base-address should be calculated by processor-specific code Andrew Victor
2011-04-29 14:53 ` Andrew Victor
2011-04-30 1:28 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2011-05-04 20:53 ` Ryan Mallon
2 siblings, 0 replies; 4+ messages in thread
From: Ryan Mallon @ 2011-05-04 20:53 UTC (permalink / raw)
To: linux-arm-kernel
On 04/30/2011 02:11 AM, Andrew Victor wrote:
> For supporting multiple AT91 processors in a single kernel image, the
> base-address of the system peripherals is processor-dependent. The GPIO
> driver should therefore not reference AT91_VA_BASE_SYS, but rather the
> correct peripheral base-address needs to be provided by the
> processor-specific initialization code.
>
>
> Signed-off-by: Andrew Victor <linux@maxim.org.za>
With your suggested change to rename AT91_PIOA to AT91RM9200_PIO, etc:
Acked-by: Ryan Mallon <ryan@bluewatersys.com>
>
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at572d940hf.c linux-2.6/arch/arm/mach-at91/at572d940hf.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at572d940hf.c 2010-12-24 16:52:59.323332139 +0200
> +++ linux-2.6/arch/arm/mach-at91/at572d940hf.c 2011-04-28 14:51:20.626598251 +0200
> @@ -279,15 +279,15 @@
> static struct at91_gpio_bank at572d940hf_gpio[] = {
> {
> .id = AT572D940HF_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT572D940HF_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT572D940HF_ID_PIOC,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioC_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91cap9.c linux-2.6/arch/arm/mach-at91/at91cap9.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91cap9.c 2009-06-10 05:05:27.000000000 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91cap9.c 2011-04-28 14:52:15.627736528 +0200
> @@ -271,19 +271,19 @@
> static struct at91_gpio_bank at91cap9_gpio[] = {
> {
> .id = AT91CAP9_ID_PIOABCD,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioABCD_clk,
> }, {
> .id = AT91CAP9_ID_PIOABCD,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioABCD_clk,
> }, {
> .id = AT91CAP9_ID_PIOABCD,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioABCD_clk,
> }, {
> .id = AT91CAP9_ID_PIOABCD,
> - .offset = AT91_PIOD,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> .clock = &pioABCD_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91rm9200.c linux-2.6/arch/arm/mach-at91/at91rm9200.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91rm9200.c 2009-06-10 05:05:27.000000000 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91rm9200.c 2011-04-28 14:33:14.161390776 +0200
> @@ -240,19 +240,19 @@
> static struct at91_gpio_bank at91rm9200_gpio[] = {
> {
> .id = AT91RM9200_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT91RM9200_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT91RM9200_ID_PIOC,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioC_clk,
> }, {
> .id = AT91RM9200_ID_PIOD,
> - .offset = AT91_PIOD,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> .clock = &pioD_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9260.c linux-2.6/arch/arm/mach-at91/at91sam9260.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9260.c 2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9260.c 2011-04-28 14:53:53.510630116 +0200
> @@ -266,15 +266,15 @@
> static struct at91_gpio_bank at91sam9260_gpio[] = {
> {
> .id = AT91SAM9260_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT91SAM9260_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT91SAM9260_ID_PIOC,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioC_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9261.c linux-2.6/arch/arm/mach-at91/at91sam9261.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9261.c 2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9261.c 2011-04-28 14:54:36.792366992 +0200
> @@ -244,15 +244,15 @@
> static struct at91_gpio_bank at91sam9261_gpio[] = {
> {
> .id = AT91SAM9261_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT91SAM9261_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT91SAM9261_ID_PIOC,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioC_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9263.c linux-2.6/arch/arm/mach-at91/at91sam9263.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9263.c 2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9263.c 2011-04-28 14:55:50.240519738 +0200
> @@ -248,23 +248,23 @@
> static struct at91_gpio_bank at91sam9263_gpio[] = {
> {
> .id = AT91SAM9263_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT91SAM9263_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT91SAM9263_ID_PIOCDE,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioCDE_clk,
> }, {
> .id = AT91SAM9263_ID_PIOCDE,
> - .offset = AT91_PIOD,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> .clock = &pioCDE_clk,
> }, {
> .id = AT91SAM9263_ID_PIOCDE,
> - .offset = AT91_PIOE,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOE,
> .clock = &pioCDE_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9g45.c linux-2.6/arch/arm/mach-at91/at91sam9g45.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9g45.c 2010-12-24 16:54:07.564396678 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9g45.c 2011-04-28 14:56:43.621718542 +0200
> @@ -270,23 +270,23 @@
> static struct at91_gpio_bank at91sam9g45_gpio[] = {
> {
> .id = AT91SAM9G45_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT91SAM9G45_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT91SAM9G45_ID_PIOC,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioC_clk,
> }, {
> .id = AT91SAM9G45_ID_PIODE,
> - .offset = AT91_PIOD,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> .clock = &pioDE_clk,
> }, {
> .id = AT91SAM9G45_ID_PIODE,
> - .offset = AT91_PIOE,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOE,
> .clock = &pioDE_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/at91sam9rl.c linux-2.6/arch/arm/mach-at91/at91sam9rl.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/at91sam9rl.c 2011-04-26 20:02:18.348726273 +0200
> +++ linux-2.6/arch/arm/mach-at91/at91sam9rl.c 2011-04-28 14:57:24.215612004 +0200
> @@ -225,19 +225,19 @@
> static struct at91_gpio_bank at91sam9rl_gpio[] = {
> {
> .id = AT91SAM9RL_ID_PIOA,
> - .offset = AT91_PIOA,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOA,
> .clock = &pioA_clk,
> }, {
> .id = AT91SAM9RL_ID_PIOB,
> - .offset = AT91_PIOB,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOB,
> .clock = &pioB_clk,
> }, {
> .id = AT91SAM9RL_ID_PIOC,
> - .offset = AT91_PIOC,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOC,
> .clock = &pioC_clk,
> }, {
> .id = AT91SAM9RL_ID_PIOD,
> - .offset = AT91_PIOD,
> + .regbase = (void __iomem *)AT91_VA_BASE_SYS + AT91_PIOD,
> .clock = &pioD_clk,
> }
> };
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/generic.h linux-2.6/arch/arm/mach-at91/generic.h
> --- linux-2.6.38.at91/arch/arm/mach-at91/generic.h 2011-04-28 17:01:30.074425436 +0200
> +++ linux-2.6/arch/arm/mach-at91/generic.h 2011-04-28 14:39:51.365842238 +0200
> @@ -56,7 +56,7 @@
>
> struct at91_gpio_bank {
> unsigned short id; /* peripheral ID */
> - unsigned long offset; /* offset from system peripheral base */
> + void __iomem *regbase; /* base address of PIO register bank */
> struct clk *clock; /* associated clock */
> };
> extern void __init at91_gpio_init(struct at91_gpio_bank *, int nr_banks);
> diff -urN -x CVS linux-2.6.38.at91/arch/arm/mach-at91/gpio.c linux-2.6/arch/arm/mach-at91/gpio.c
> --- linux-2.6.38.at91/arch/arm/mach-at91/gpio.c 2011-04-28 17:01:35.838026699 +0200
> +++ linux-2.6/arch/arm/mach-at91/gpio.c 2011-04-28 14:40:55.418627104 +0200
> @@ -32,7 +32,6 @@
> struct gpio_chip chip;
> struct at91_gpio_chip *next; /* Bank sharing same clock */
> struct at91_gpio_bank *bank; /* Bank definition */
> - void __iomem *regbase; /* Base of register bank */
> };
>
> #define to_at91_gpio_chip(c) container_of(c, struct at91_gpio_chip, chip)
> @@ -74,7 +73,7 @@
> pin -= PIN_BASE;
> pin /= 32;
> if (likely(pin < gpio_banks))
> - return gpio_chip[pin].regbase;
> + return gpio_chip[pin].bank->regbase;
>
> return NULL;
> }
> @@ -297,7 +296,7 @@
> int i;
>
> for (i = 0; i < gpio_banks; i++) {
> - void __iomem *pio = gpio_chip[i].regbase;
> + void __iomem *pio = gpio_chip[i].bank->regbase;
>
> backups[i] = __raw_readl(pio + PIO_IMR);
> __raw_writel(backups[i], pio + PIO_IDR);
> @@ -318,7 +317,7 @@
> int i;
>
> for (i = 0; i < gpio_banks; i++) {
> - void __iomem *pio = gpio_chip[i].regbase;
> + void __iomem *pio = gpio_chip[i].bank->regbase;
>
> if (!wakeups[i])
> clk_enable(gpio_chip[i].bank->clock);
> @@ -390,7 +389,7 @@
> u32 isr;
>
> at91_gpio = get_irq_chip_data(irq);
> - pio = at91_gpio->regbase;
> + pio = at91_gpio->bank->regbase;
>
> for (;;) {
> /* Reading ISR acks pending (edge triggered) GPIO interrupts.
> @@ -402,7 +401,7 @@
> if (!at91_gpio->next)
> break;
> at91_gpio = at91_gpio->next;
> - pio = at91_gpio->regbase;
> + pio = at91_gpio->bank->regbase;
> continue;
> }
>
> @@ -513,7 +512,7 @@
> unsigned id = this->bank->id;
> unsigned i;
>
> - __raw_writel(~0, this->regbase + PIO_IDR);
> + __raw_writel(~0, this->bank->regbase + PIO_IDR);
>
> for (i = 0, pin = this->chip.base; i < 32; i++, pin++) {
> lockdep_set_class(&irq_desc[pin].lock, &gpio_lock_class);
> @@ -545,7 +544,7 @@
> unsigned offset)
> {
> struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - void __iomem *pio = at91_gpio->regbase;
> + void __iomem *pio = at91_gpio->bank->regbase;
> unsigned mask = 1 << offset;
>
> __raw_writel(mask, pio + PIO_ODR);
> @@ -556,7 +555,7 @@
> unsigned offset, int val)
> {
> struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - void __iomem *pio = at91_gpio->regbase;
> + void __iomem *pio = at91_gpio->bank->regbase;
> unsigned mask = 1 << offset;
>
> __raw_writel(mask, pio + (val ? PIO_SODR : PIO_CODR));
> @@ -567,7 +566,7 @@
> static int at91_gpiolib_get(struct gpio_chip *chip, unsigned offset)
> {
> struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - void __iomem *pio = at91_gpio->regbase;
> + void __iomem *pio = at91_gpio->bank->regbase;
> unsigned mask = 1 << offset;
> u32 pdsr;
>
> @@ -578,7 +577,7 @@
> static void at91_gpiolib_set(struct gpio_chip *chip, unsigned offset, int val)
> {
> struct at91_gpio_chip *at91_gpio = to_at91_gpio_chip(chip);
> - void __iomem *pio = at91_gpio->regbase;
> + void __iomem *pio = at91_gpio->bank->regbase;
> unsigned mask = 1 << offset;
>
> __raw_writel(mask, pio + (val ? PIO_SODR : PIO_CODR));
> @@ -627,8 +626,6 @@
>
> at91_gpio->bank = &data[i];
> at91_gpio->chip.base = PIN_BASE + i * 32;
> - at91_gpio->regbase = at91_gpio->bank->offset +
> - (void __iomem *)AT91_VA_BASE_SYS;
>
> /* enable PIO controller's clock */
> clk_enable(at91_gpio->bank->clock);
>
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Bluewater Systems Ltd - ARM Technology Solution Centre
Ryan Mallon 5 Amuri Park, 404 Barbadoes St
ryan at bluewatersys.com PO Box 13 889, Christchurch 8013
http://www.bluewatersys.com New Zealand
Phone: +64 3 3779127 Freecall: Australia 1800 148 751
Fax: +64 3 3779135 USA 1800 261 2934
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-05-04 20:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-29 14:11 AT91: PIO base-address should be calculated by processor-specific code Andrew Victor
2011-04-29 14:53 ` Andrew Victor
2011-04-30 1:28 ` Jean-Christophe PLAGNIOL-VILLARD
2011-05-04 20:53 ` Ryan Mallon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).