[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/3] xen/arm: drivers: scif: Remove unused variables



Hello Dirk,

On 21/06/16 10:15, Dirk Behme wrote:
The two struct members baud and clock_hz are in the end read only
variables nowhere used for anything useful. Removing them makes
the code much simpler without changing any functionality.

From my understanding, this patch is removing code you just added on the previous patch. I would prefer if you squash this patch into #1.

Regards,


Signed-off-by: Dirk Behme <dirk.behme@xxxxxxxxxxxx>
---
  xen/drivers/char/scif-uart.c    | 13 +------------
  xen/include/asm-arm/scif-uart.h |  1 -
  2 files changed, 1 insertion(+), 13 deletions(-)

diff --git a/xen/drivers/char/scif-uart.c b/xen/drivers/char/scif-uart.c
index ca88c0f..bc157fe 100644
--- a/xen/drivers/char/scif-uart.c
+++ b/xen/drivers/char/scif-uart.c
@@ -41,7 +41,7 @@
  #define scif_writew(uart, off, val)    writew((val), (uart)->regs + (off))

  static struct scif_uart {
-    unsigned int baud, clock_hz, data_bits, parity, stop_bits;
+    unsigned int data_bits, parity, stop_bits;
      unsigned int irq;
      char __iomem *regs;
      struct irqaction irqaction;
@@ -87,7 +87,6 @@ static void scif_uart_interrupt(int irq, void *data, struct 
cpu_user_regs *regs)
  static void __init scif_uart_init_preirq(struct serial_port *port)
  {
      struct scif_uart *uart = port->uart;
-    unsigned int divisor;
      uint16_t val;

      /*
@@ -142,14 +141,6 @@ static void __init scif_uart_init_preirq(struct 
serial_port *port)
      }
      scif_writew(uart, SCIF_SCSMR, val);

-    ASSERT( uart->clock_hz > 0 );
-    ASSERT( uart->baud == BAUD_AUTO );
-
-    /* Read current Baud rate */
-    divisor = scif_readw(uart, SCIF_DL);
-    ASSERT( divisor >= 1 && divisor <= (uint16_t)UINT_MAX );
-    uart->baud = uart->clock_hz / (divisor << 4);
-
      /* Setup trigger level for TX/RX FIFOs */
      scif_writew(uart, SCIF_SCFCR, SCFCR_RTRG11 | SCFCR_TTRG11);

@@ -292,8 +283,6 @@ static int __init scif_uart_init(struct dt_device_node *dev,

      uart = &scif_com;

-    uart->clock_hz  = SCIF_CLK_FREQ;
-    uart->baud      = BAUD_AUTO;
      uart->data_bits = 8;
      uart->parity    = PARITY_NONE;
      uart->stop_bits = 1;
diff --git a/xen/include/asm-arm/scif-uart.h b/xen/include/asm-arm/scif-uart.h
index 7a9f639..d030b26 100644
--- a/xen/include/asm-arm/scif-uart.h
+++ b/xen/include/asm-arm/scif-uart.h
@@ -22,7 +22,6 @@
  #define __ASM_ARM_SCIF_UART_H

  #define SCIF_FIFO_MAX_SIZE    16
-#define SCIF_CLK_FREQ         14745600

  /* Register offsets */
  #define SCIF_SCSMR     (0x00)    /* Serial mode register           */


--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.