Commit f1232ac2 authored by Ji-Ze Hong (Peter Hong)'s avatar Ji-Ze Hong (Peter Hong) Committed by Greg Kroah-Hartman

serial: 8250_fintek: Refactoring read/write method

If we need to access SuperIO registers, It should write register offset
to base_addr and read/write value to base_addr + 1 to perform read/write.
We can make it more simply with write/read functions.

This patch add sio_read_reg()/sio_write_reg()/sio_write_mask_reg() to
reduce SuperIO register operation with lot of outb()/inb().
Signed-off-by: default avatarJi-Ze Hong (Peter Hong) <hpeter+linux_kernel@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent fea6dd14
...@@ -49,6 +49,27 @@ struct fintek_8250 { ...@@ -49,6 +49,27 @@ struct fintek_8250 {
u8 key; u8 key;
}; };
static u8 sio_read_reg(struct fintek_8250 *pdata, u8 reg)
{
outb(reg, pdata->base_port + ADDR_PORT);
return inb(pdata->base_port + DATA_PORT);
}
static void sio_write_reg(struct fintek_8250 *pdata, u8 reg, u8 data)
{
outb(reg, pdata->base_port + ADDR_PORT);
outb(data, pdata->base_port + DATA_PORT);
}
static void sio_write_mask_reg(struct fintek_8250 *pdata, u8 reg, u8 mask,
u8 data)
{
u8 tmp;
tmp = (sio_read_reg(pdata, reg) & ~mask) | (mask & data);
sio_write_reg(pdata, reg, tmp);
}
static int fintek_8250_enter_key(u16 base_port, u8 key) static int fintek_8250_enter_key(u16 base_port, u8 key)
{ {
if (!request_muxed_region(base_port, 2, "8250_fintek")) if (!request_muxed_region(base_port, 2, "8250_fintek"))
...@@ -66,22 +87,18 @@ static void fintek_8250_exit_key(u16 base_port) ...@@ -66,22 +87,18 @@ static void fintek_8250_exit_key(u16 base_port)
release_region(base_port + ADDR_PORT, 2); release_region(base_port + ADDR_PORT, 2);
} }
static int fintek_8250_check_id(u16 base_port) static int fintek_8250_check_id(struct fintek_8250 *pdata)
{ {
u16 chip; u16 chip;
outb(VENDOR_ID1, base_port + ADDR_PORT); if (sio_read_reg(pdata, VENDOR_ID1) != VENDOR_ID1_VAL)
if (inb(base_port + DATA_PORT) != VENDOR_ID1_VAL)
return -ENODEV; return -ENODEV;
outb(VENDOR_ID2, base_port + ADDR_PORT); if (sio_read_reg(pdata, VENDOR_ID2) != VENDOR_ID2_VAL)
if (inb(base_port + DATA_PORT) != VENDOR_ID2_VAL)
return -ENODEV; return -ENODEV;
outb(CHIP_ID1, base_port + ADDR_PORT); chip = sio_read_reg(pdata, CHIP_ID1);
chip = inb(base_port + DATA_PORT); chip |= sio_read_reg(pdata, CHIP_ID2) << 8;
outb(CHIP_ID2, base_port + ADDR_PORT);
chip |= inb(base_port + DATA_PORT) << 8;
if (chip != CHIP_ID_0 && chip != CHIP_ID_1) if (chip != CHIP_ID_0 && chip != CHIP_ID_1)
return -ENODEV; return -ENODEV;
...@@ -128,10 +145,8 @@ static int fintek_8250_rs485_config(struct uart_port *port, ...@@ -128,10 +145,8 @@ static int fintek_8250_rs485_config(struct uart_port *port,
if (fintek_8250_enter_key(pdata->base_port, pdata->key)) if (fintek_8250_enter_key(pdata->base_port, pdata->key))
return -EBUSY; return -EBUSY;
outb(LDN, pdata->base_port + ADDR_PORT); sio_write_reg(pdata, LDN, pdata->index);
outb(pdata->index, pdata->base_port + DATA_PORT); sio_write_reg(pdata, RS485, config);
outb(RS485, pdata->base_port + ADDR_PORT);
outb(config, pdata->base_port + DATA_PORT);
fintek_8250_exit_key(pdata->base_port); fintek_8250_exit_key(pdata->base_port);
port->rs485 = *rs485; port->rs485 = *rs485;
...@@ -147,10 +162,12 @@ static int find_base_port(struct fintek_8250 *pdata, u16 io_address) ...@@ -147,10 +162,12 @@ static int find_base_port(struct fintek_8250 *pdata, u16 io_address)
for (i = 0; i < ARRAY_SIZE(addr); i++) { for (i = 0; i < ARRAY_SIZE(addr); i++) {
for (j = 0; j < ARRAY_SIZE(keys); j++) { for (j = 0; j < ARRAY_SIZE(keys); j++) {
pdata->base_port = addr[i];
pdata->key = keys[j];
if (fintek_8250_enter_key(addr[i], keys[j])) if (fintek_8250_enter_key(addr[i], keys[j]))
continue; continue;
if (fintek_8250_check_id(addr[i])) { if (fintek_8250_check_id(pdata)) {
fintek_8250_exit_key(addr[i]); fintek_8250_exit_key(addr[i]);
continue; continue;
} }
...@@ -158,19 +175,13 @@ static int find_base_port(struct fintek_8250 *pdata, u16 io_address) ...@@ -158,19 +175,13 @@ static int find_base_port(struct fintek_8250 *pdata, u16 io_address)
for (k = 0; k < 4; k++) { for (k = 0; k < 4; k++) {
u16 aux; u16 aux;
outb(LDN, addr[i] + ADDR_PORT); sio_write_reg(pdata, LDN, k);
outb(k, addr[i] + DATA_PORT); aux = sio_read_reg(pdata, IO_ADDR1);
aux |= sio_read_reg(pdata, IO_ADDR2) << 8;
outb(IO_ADDR1, addr[i] + ADDR_PORT);
aux = inb(addr[i] + DATA_PORT);
outb(IO_ADDR2, addr[i] + ADDR_PORT);
aux |= inb(addr[i] + DATA_PORT) << 8;
if (aux != io_address) if (aux != io_address)
continue; continue;
fintek_8250_exit_key(addr[i]); fintek_8250_exit_key(addr[i]);
pdata->key = keys[j];
pdata->base_port = addr[i];
pdata->index = k; pdata->index = k;
return 0; return 0;
...@@ -186,24 +197,16 @@ static int find_base_port(struct fintek_8250 *pdata, u16 io_address) ...@@ -186,24 +197,16 @@ static int find_base_port(struct fintek_8250 *pdata, u16 io_address)
static int fintek_8250_set_irq_mode(struct fintek_8250 *pdata, bool level_mode) static int fintek_8250_set_irq_mode(struct fintek_8250 *pdata, bool level_mode)
{ {
int status; int status;
u8 tmp;
status = fintek_8250_enter_key(pdata->base_port, pdata->key); status = fintek_8250_enter_key(pdata->base_port, pdata->key);
if (status) if (status)
return status; return status;
outb(LDN, pdata->base_port + ADDR_PORT); sio_write_reg(pdata, LDN, pdata->index);
outb(pdata->index, pdata->base_port + DATA_PORT); sio_write_mask_reg(pdata, FINTEK_IRQ_MODE, IRQ_SHARE, IRQ_SHARE);
sio_write_mask_reg(pdata, FINTEK_IRQ_MODE, IRQ_MODE_MASK,
outb(FINTEK_IRQ_MODE, pdata->base_port + ADDR_PORT); level_mode ? IRQ_LEVEL_LOW : IRQ_EDGE_HIGH);
tmp = inb(pdata->base_port + DATA_PORT);
tmp &= ~IRQ_MODE_MASK;
tmp |= IRQ_SHARE;
if (!level_mode)
tmp |= IRQ_EDGE_HIGH;
outb(tmp, pdata->base_port + DATA_PORT);
fintek_8250_exit_key(pdata->base_port); fintek_8250_exit_key(pdata->base_port);
return 0; return 0;
} }
......
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