Re: [patch v3 1/5] AST2500 DMA UART driver

From: Joe Perches
Date: Wed Jul 03 2019 - 15:16:09 EST


On Wed, 2019-07-03 at 19:49 +0200, Greg KH wrote:
> On Tue, Jun 25, 2019 at 04:14:32PM +0530, sudheer.v wrote:
> > +#define UART_TX_R_POINT(x) (0x40 + (x * 0x20))
> > +#define UART_TX_W_POINT(x) (0x44 + (x * 0x20))
> > +#define UART_TX_SDMA_ADDR(x) (0x48 + (x * 0x20))
> > +#define UART_RX_R_POINT(x) (0x50 + (x * 0x20))
> > +#define UART_RX_W_POINT(x) (0x54 + (x * 0x20))
> > +#define UART_RX_SDMA_ADDR(x) (0x58 + (x * 0x20))
>
> Please use a tab to line these up.

Also x should be surrounded by parentheses

#define UART_TX_R_POINT(x) (0x40 + ((x) * 0x20))

etc...