Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UART #21

Open
epernia opened this issue Jan 8, 2017 · 1 comment
Open

UART #21

epernia opened this issue Jan 8, 2017 · 1 comment

Comments

@epernia
Copy link
Contributor

epernia commented Jan 8, 2017

Traigo la discusión de los mails:

Martín:

El modulo de UART le faltaria un uart[Write|Read](uart, buffer, size); normal, no es practico hacer un for y llamar a wartWirteByte es bastante ineficiente, cuando seguramente hay formas de hacer esto mas facil usando el HW de la UART (DMA, FIFO por HW, etc)

Eric:

Mantendría las versiones de byte y string, que se hicieron así sencillas para que sea fácil de explicar su comportamiento.

Martín:

En cuanto a la UART, creo que ahi amerita algo mas simple:

uartInit( UART_NAME, speed); // 8bits, no parity, 1 stop

Y si hace falta, solo si hace falta cambiar las cosas:

uartConfig( UART_ NAME, UART_7BITS | UART_PODD | UART_2STOP);

Aca vemos que la semantinca de "config" es mala para estos casos, por lo que propongo tener "Init" y "Config" así podemos iniciarla con parametros por defecto y cambiar esa configuración al vuelo si hace falta.

@epernia
Copy link
Contributor Author

epernia commented Jan 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant