Hello again, On Sun, Nov 16, 2014 at 10:42:39PM +0100, Uwe Kleine-König wrote: > Also note that the atmel_get_lines_status function can be simplified to > just: > > status = UART_GET_CSR(port); > return mctrl_gpio_get(atmel_port->gpios, &status); thinking again, this is wrong because it uses the Atmel specific register bits for the different flags, but with the generic solution outlined in the previous mail this function wouldn't need to handle the gpio parts at all. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html