On 6.6.2018 14:41, Michal Simek wrote: > Writing zero and NULLs to already initialized fields is not needed. > Remove this additional writes. > > Signed-off-by: Michal Simek <michal.simek@xxxxxxxxxx> > --- > > Changes in v2: > - new patch - it can be sent separately too > > drivers/tty/serial/xilinx_uartps.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c > index 8a3e34234e98..5f116f3ecd4a 100644 > --- a/drivers/tty/serial/xilinx_uartps.c > +++ b/drivers/tty/serial/xilinx_uartps.c > @@ -1510,15 +1510,12 @@ static int cdns_uart_probe(struct platform_device *pdev) > > /* At this point, we've got an empty uart_port struct, initialize it */ > spin_lock_init(&port->lock); > - port->membase = NULL; > - port->irq = 0; > port->type = PORT_UNKNOWN; > port->iotype = UPIO_MEM32; > port->flags = UPF_BOOT_AUTOCONF; > port->ops = &cdns_uart_ops; > port->fifosize = CDNS_UART_FIFO_SIZE; > port->line = id; > - port->dev = NULL; > > /* > * Register the port. > Alan, Rob, Greg: Any comment about this RFC? Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Xilinx Microblaze Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html