Skip to contentSkip to conversation

serial: atmel_serial: remove atmel open/close hooks

PATCH New Joachim Eastwood 2 participants 2 messages Nov 17, 2012 – Nov 19, 2012

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 serial: atmel_serial: remove atmel open/close hooks

After serial_at91.h was removed the atmel_open/close_hook code is now useless.

---
Hi,

This patch depends on [RFC 3/3] ARM/AVR32: get rid of serial_at91.h patch from Russell King.

Patch based on current linux-next with "[RFC 3/3] ARM/AVR32: get rid of serial_at91.h" patch on top. There is a small conflict with Russell's patch
in next now due to another header change.

If there is another tree this patch should be based on please let me know.

regards
Joachim Eastwood

 drivers/tty/serial/atmel_serial.c | 22 ----------------------
 1 file changed, 22 deletions(-)
drivers/tty/serial/atmel_serial.c +0 -22
index 76e4c05..5c2337d 100644
@@ -113,9 +113,6 @@ static void atmel_stop_rx(struct uart_port *port);
 #define UART_PUT_TCR(port,v)	__raw_writel(v, (port)->membase + ATMEL_PDC_TCR)
 #define UART_GET_TCR(port)	__raw_readl((port)->membase + ATMEL_PDC_TCR)
 
-static int (*atmel_open_hook)(struct uart_port *);
-static void (*atmel_close_hook)(struct uart_port *);
-
 struct atmel_dma_buffer {
 	unsigned char	*buf;
 	dma_addr_t	dma_addr;
@@ -976,18 +973,6 @@ static int atmel_startup(struct uart_port *port)
 		pdc->ofs = 0;
 	}
 
-	/*
-	 * If there is a specific "open" function (to register
-	 * control line interrupts)
-	 */
-	if (atmel_open_hook) {
-		retval = atmel_open_hook(port);
-		if (retval) {
-			free_irq(port->irq, port);
-			return retval;
-		}
-	}
-
 	/* Save current CSR for comparison in atmel_tasklet_func() */
 	atmel_port->irq_status_prev = UART_GET_CSR(port);
 	atmel_port->irq_status = atmel_port->irq_status_prev;
@@ -1062,13 +1047,6 @@ static void atmel_shutdown(struct uart_port *port)
 	 * Free the interrupt
 	 */
 	free_irq(port->irq, port);
-
-	/*
-	 * If there is a specific "close" function (to unregister
-	 * control line interrupts)
-	 */
-	if (atmel_close_hook)
-		atmel_close_hook(port);
 }
 
 /*
Signature · 1 line
1.8.0
#1 lore raw

PATCH 1/1 serial: atmel_serial: remove atmel open/close hooks

On 11/17/2012 01:09 AM, Joachim Eastwood :

Quoted 4 lines

After serial_at91.h was removed the atmel_open/close_hook code is now useless.

Thanks Joachim.

Quoted 67 lines

---
Hi,

This patch depends on [RFC 3/3] ARM/AVR32: get rid of serial_at91.h patch from Russell King.

Patch based on current linux-next with "[RFC 3/3] ARM/AVR32: get rid of serial_at91.h" patch on top. There is a small conflict with Russell's patch
in next now due to another header change.

If there is another tree this patch should be based on please let me know.

regards
Joachim Eastwood

 drivers/tty/serial/atmel_serial.c | 22 ----------------------
 1 file changed, 22 deletions(-)
drivers/tty/serial/atmel_serial.c +0 -22
index 76e4c05..5c2337d 100644
@@ -113,9 +113,6 @@ static void atmel_stop_rx(struct uart_port *port);
 #define UART_PUT_TCR(port,v)	__raw_writel(v, (port)->membase + ATMEL_PDC_TCR)
 #define UART_GET_TCR(port)	__raw_readl((port)->membase + ATMEL_PDC_TCR)
 
-static int (*atmel_open_hook)(struct uart_port *);
-static void (*atmel_close_hook)(struct uart_port *);
-
 struct atmel_dma_buffer {
 	unsigned char	*buf;
 	dma_addr_t	dma_addr;
@@ -976,18 +973,6 @@ static int atmel_startup(struct uart_port *port)
 		pdc->ofs = 0;
 	}
 
-	/*
-	 * If there is a specific "open" function (to register
-	 * control line interrupts)
-	 */
-	if (atmel_open_hook) {
-		retval = atmel_open_hook(port);
-		if (retval) {
-			free_irq(port->irq, port);
-			return retval;
-		}
-	}
-
 	/* Save current CSR for comparison in atmel_tasklet_func() */
 	atmel_port->irq_status_prev = UART_GET_CSR(port);
 	atmel_port->irq_status = atmel_port->irq_status_prev;
@@ -1062,13 +1047,6 @@ static void atmel_shutdown(struct uart_port *port)
 	 * Free the interrupt
 	 */
 	free_irq(port->irq, port);
-
-	/*
-	 * If there is a specific "close" function (to unregister
-	 * control line interrupts)
-	 */
-	if (atmel_close_hook)
-		atmel_close_hook(port);
 }
 
 /*
Signature · 1 line
Nicolas Ferre