Re: svn commit: r295557 - head/sys/dev/uart

2016-02-16 Thread Ian Lepore
On Tue, 2016-02-16 at 12:46 +1100, Bruce Evans wrote: > On Mon, 15 Feb 2016, Ian Lepore wrote: > > > On Tue, 2016-02-16 at 11:01 +1100, Bruce Evans wrote: > >> On Mon, 15 Feb 2016, Ian Lepore wrote: > >> > >>> On Tue, 2016-02-16 at 09:28 +1100, Bruce Evans wrote: > On Mon, 15 Feb 2016,

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Bruce Evans
On Mon, 15 Feb 2016, Ian Lepore wrote: On Tue, 2016-02-16 at 11:01 +1100, Bruce Evans wrote: On Mon, 15 Feb 2016, Ian Lepore wrote: On Tue, 2016-02-16 at 09:28 +1100, Bruce Evans wrote: On Mon, 15 Feb 2016, Michal Meloun wrote: [...] Please note that ARM architecture does not have

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Ian Lepore
On Tue, 2016-02-16 at 11:01 +1100, Bruce Evans wrote: > On Mon, 15 Feb 2016, Ian Lepore wrote: > > > On Tue, 2016-02-16 at 09:28 +1100, Bruce Evans wrote: > >> On Mon, 15 Feb 2016, Michal Meloun wrote: > >> > >>> [...] > >>> Please note that ARM architecture does not have vectored interrupts, >

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Bruce Evans
On Mon, 15 Feb 2016, Ian Lepore wrote: On Tue, 2016-02-16 at 09:28 +1100, Bruce Evans wrote: On Mon, 15 Feb 2016, Michal Meloun wrote: [...] Please note that ARM architecture does not have vectored interrupts, CPU must read actual interrupt source from external interrupt controller (GIC)

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Ian Lepore
On Tue, 2016-02-16 at 09:28 +1100, Bruce Evans wrote: > On Mon, 15 Feb 2016, Michal Meloun wrote: > > > [...] > > Please note that ARM architecture does not have vectored interrupts, > > CPU must read actual interrupt source from external interrupt > > controller (GIC) register. This register

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Adrian Chadd
On 15 February 2016 at 14:28, Bruce Evans wrote: > > BTW, if arm needs the barrier, then how does it work with > bus_space_barrier() referenced in just 25 files in all of /sys/dev? likely the same reason it does for mips - everything's cache coherent on hardware that has

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Bruce Evans
On Mon, 15 Feb 2016, Michal Meloun wrote: Dne 13.02.2016 v 1:58 Marius Strobl napsal(a): On Sat, Feb 13, 2016 at 06:53:25AM +1100, Bruce Evans wrote: On Fri, 12 Feb 2016, Marius Strobl wrote: On Fri, Feb 12, 2016 at 05:14:58AM +, Michal Meloun wrote: Author: mmel Date: Fri Feb 12

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-15 Thread Michal Meloun
Dne 13.02.2016 v 1:58 Marius Strobl napsal(a): > On Sat, Feb 13, 2016 at 06:53:25AM +1100, Bruce Evans wrote: >> On Fri, 12 Feb 2016, Marius Strobl wrote: >> >>> On Fri, Feb 12, 2016 at 05:14:58AM +, Michal Meloun wrote: Author: mmel Date: Fri Feb 12 05:14:58 2016 New Revision:

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-12 Thread Marius Strobl
On Fri, Feb 12, 2016 at 05:14:58AM +, Michal Meloun wrote: > Author: mmel > Date: Fri Feb 12 05:14:58 2016 > New Revision: 295557 > URL: https://svnweb.freebsd.org/changeset/base/295557 > > Log: > UART: Fix spurious interrupts generated by ns8250 and lpc drivers: >- don't enable

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-12 Thread Bruce Evans
On Fri, 12 Feb 2016, Marius Strobl wrote: On Fri, Feb 12, 2016 at 05:14:58AM +, Michal Meloun wrote: Author: mmel Date: Fri Feb 12 05:14:58 2016 New Revision: 295557 URL: https://svnweb.freebsd.org/changeset/base/295557 Log: UART: Fix spurious interrupts generated by ns8250 and lpc

Re: svn commit: r295557 - head/sys/dev/uart

2016-02-12 Thread Marius Strobl
On Sat, Feb 13, 2016 at 06:53:25AM +1100, Bruce Evans wrote: > On Fri, 12 Feb 2016, Marius Strobl wrote: > > > On Fri, Feb 12, 2016 at 05:14:58AM +, Michal Meloun wrote: > >> Author: mmel > >> Date: Fri Feb 12 05:14:58 2016 > >> New Revision: 295557 > >> URL:

svn commit: r295557 - head/sys/dev/uart

2016-02-11 Thread Michal Meloun
Author: mmel Date: Fri Feb 12 05:14:58 2016 New Revision: 295557 URL: https://svnweb.freebsd.org/changeset/base/295557 Log: UART: Fix spurious interrupts generated by ns8250 and lpc drivers: - don't enable transmitter empty interrupt before filling TX FIFO. - add missing uart_barrier()