Hi, On Thu, Mar 20, 2014 at 10:05:33AM -0400, jon@xxxxxxxxxx wrote: > From: Jon Ringle <jringle@xxxxxxxxxxxxx> > > The SC16IS7xx is a slave I2C-bus/SPI interface to a single-channel > high performance UART. The SC16IS7xx’s internal register set is > backward-compatible with the widely used and widely popular 16C450. So couldn't this be just a probe driver for 8250? > +/* SC16IS7XX register definitions */ > +#define SC16IS7XX_RHR_REG (0x00) /* RX FIFO */ > +#define SC16IS7XX_THR_REG (0x00) /* TX FIFO */ > +#define SC16IS7XX_IER_REG (0x01) /* Interrupt enable */ > +#define SC16IS7XX_IIR_REG (0x02) /* Interrupt Identification */ > +#define SC16IS7XX_FCR_REG (0x02) /* FIFO control */ > +#define SC16IS7XX_LCR_REG (0x03) /* Line Control */ > +#define SC16IS7XX_MCR_REG (0x04) /* Modem Control */ > +#define SC16IS7XX_LSR_REG (0x05) /* Line Status */ > +#define SC16IS7XX_MSR_REG (0x06) /* Modem Status */ > +#define SC16IS7XX_SPR_REG (0x07) /* Scratch Pad */ At least there should not be any need to redefine those register or their bits. Just include serial_reg.h. Thanks, -- heikki -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html