This forces the driver to use the hardware synchronized commands.

Signed-off-by: Juergen Borleis <[email protected]>
---
 drivers/serial/serial_mpc5xxx.c | 37 +++++++++++++++++++------------------
 1 file changed, 19 insertions(+), 18 deletions(-)

diff --git a/drivers/serial/serial_mpc5xxx.c b/drivers/serial/serial_mpc5xxx.c
index 18aca87..9d92fed 100644
--- a/drivers/serial/serial_mpc5xxx.c
+++ b/drivers/serial/serial_mpc5xxx.c
@@ -32,6 +32,7 @@
 #include <common.h>
 #include <mach/mpc5xxx.h>
 #include <driver.h>
+#include <io.h>
 #include <init.h>
 #include <console.h>
 #include <xfuncs.h>
@@ -50,8 +51,8 @@ static int __mpc5xxx_serial_setbaudrate(struct mpc5xxx_psc 
*psc, int baudrate)
 
        /* set up UART divisor */
        div = (baseclk + (baudrate/2)) / baudrate;
-       psc->ctur = (div >> 8) & 0xFF;
-       psc->ctlr =  div & 0xff;
+       out_8(&psc->ctur, (div >> 8) & 0xFF);
+       out_8(&psc->ctlr,  div & 0xff);
 
        return 0;
 }
@@ -69,33 +70,33 @@ static int mpc5xxx_serial_setbaudrate(struct console_device 
*cdev, int baudrate)
 static int __mpc5xxx_serial_init(struct mpc5xxx_psc *psc)
 {
        /* reset PSC */
-       psc->command = PSC_SEL_MODE_REG_1;
+       out_8(&psc->command, PSC_SEL_MODE_REG_1);
 
        /* select clock sources */
 #if defined(CONFIG_MGT5100)
-       psc->psc_clock_select = 0xdd00;
+       out_be16(&psc->psc_clock_select, 0xdd00);
 #elif defined(CONFIG_MPC5200)
-       psc->psc_clock_select = 0;
+       out_be16(&psc->psc_clock_select, 0);
 #endif
 
        /* switch to UART mode */
-       psc->sicr = 0;
+       out_be32(&psc->sicr, 0);
 
        /* configure parity, bit length and so on */
 #if defined(CONFIG_MGT5100)
-       psc->mode = PSC_MODE_ERR | PSC_MODE_8_BITS | PSC_MODE_PARNONE;
+       out_8(&psc->mode, PSC_MODE_ERR | PSC_MODE_8_BITS | PSC_MODE_PARNONE);
 #elif defined(CONFIG_MPC5200)
-       psc->mode = PSC_MODE_8_BITS | PSC_MODE_PARNONE;
+       out_8(&psc->mode, PSC_MODE_8_BITS | PSC_MODE_PARNONE);
 #endif
-       psc->mode = PSC_MODE_ONE_STOP;
+       out_8(&psc->mode, PSC_MODE_ONE_STOP);
 
        /* disable all interrupts */
-       psc->psc_imr = 0;
+       out_be16(&psc->psc_imr, 0);
 
        /* reset and enable Rx/Tx */
-//     psc->command = PSC_RST_RX;
-//     psc->command = PSC_RST_TX;
-       psc->command = PSC_RX_ENABLE | PSC_TX_ENABLE;
+//     out_8(&psc->command, PSC_RST_RX);
+//     out_8(&psc->command, PSC_RST_TX);
+       out_8(&psc->command, PSC_RX_ENABLE | PSC_TX_ENABLE);
 
        return 0;
 }
@@ -116,10 +117,10 @@ static void mpc5xxx_serial_putc (struct console_device 
*cdev, const char c)
        struct mpc5xxx_psc *psc = dev->priv;
 
        /* Wait for last character to go. */
-       while (!(psc->psc_status & PSC_SR_TXEMP))
+       while (!(in_be16(&psc->psc_status) & PSC_SR_TXEMP))
                ;
 
-       psc->psc_buffer_8 = c;
+       out_8(&psc->psc_buffer_8, c);
 }
 
 static int mpc5xxx_serial_getc (struct console_device *cdev)
@@ -128,10 +129,10 @@ static int mpc5xxx_serial_getc (struct console_device 
*cdev)
        struct mpc5xxx_psc *psc = dev->priv;
 
        /* Wait for a character to arrive. */
-       while (!(psc->psc_status & PSC_SR_RXRDY))
+       while (!(in_be16(&psc->psc_status) & PSC_SR_RXRDY))
                ;
 
-       return psc->psc_buffer_8;
+       return in_8(&psc->psc_buffer_8);
 }
 
 static int mpc5xxx_serial_tstc (struct console_device *cdev)
@@ -139,7 +140,7 @@ static int mpc5xxx_serial_tstc (struct console_device *cdev)
        struct device_d *dev = cdev->dev;
        struct mpc5xxx_psc *psc = dev->priv;
 
-       return (psc->psc_status & PSC_SR_RXRDY);
+       return in_be16(&psc->psc_status) & PSC_SR_RXRDY;
 }
 
 static int mpc5xxx_serial_probe(struct device_d *dev)
-- 
2.1.0


_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to