Re: [PATCH 1/4] rbtx4938: Add generic GPIO support

[Atsushi Nemoto <anemo@xxxxxxxxxxxxx>]

On Sun, 24 Jun 2007 22:54:29 +0200, Ralf Baechle <ralf@xxxxxxxxxxxxxx> wrote:
> > GPIO 0..15 are for TX4938 PIO pins, GPIO 16..18 are for FPGA-driven
> > chipselect signals for SPI devices.
> 
> Queued also.

Thanks.  And please queue this too.


Subject: [PATCH] rbtx4938: Add mmio barriers for GPIO.

Signed-off-by: Atsushi Nemoto <anemo@xxxxxxxxxxxxx>
---
 arch/mips/tx4938/toshiba_rbtx4938/setup.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/arch/mips/tx4938/toshiba_rbtx4938/setup.c b/arch/mips/tx4938/toshiba_rbtx4938/setup.c
index a835870..330ee43 100644
--- a/arch/mips/tx4938/toshiba_rbtx4938/setup.c
+++ b/arch/mips/tx4938/toshiba_rbtx4938/setup.c
@@ -1021,6 +1021,7 @@ static void rbtx4938_spi_gpio_set(unsigned gpio, int value)
 	else
 		val &= ~(1 << gpio);
 	*rbtx4938_spics_ptr = val;
+	mmiowb();
 	spin_unlock_irqrestore(&rbtx4938_spi_gpio_lock, flags);
 }
 
@@ -1053,6 +1054,7 @@ static void tx4938_gpio_set(unsigned gpio, int value)
 	unsigned long flags;
 	spin_lock_irqsave(&tx4938_gpio_lock, flags);
 	tx4938_gpio_set_raw(gpio, value);
+	mmiowb();
 	spin_unlock_irqrestore(&tx4938_gpio_lock, flags);
 }
 
@@ -1060,6 +1062,7 @@ static int tx4938_gpio_dir_in(unsigned gpio)
 {
 	spin_lock_irq(&tx4938_gpio_lock);
 	tx4938_pioptr->dir &= ~(1 << gpio);
+	mmiowb();
 	spin_unlock_irq(&tx4938_gpio_lock);
 	return 0;
 }
@@ -1069,6 +1072,7 @@ static int tx4938_gpio_dir_out(unsigned int gpio, int value)
 	spin_lock_irq(&tx4938_gpio_lock);
 	tx4938_gpio_set_raw(gpio, value);
 	tx4938_pioptr->dir |= 1 << gpio;
+	mmiowb();
 	spin_unlock_irq(&tx4938_gpio_lock);
 	return 0;
 }