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...