I'm worried about the stm32_tim_getcounter funtion. It returns always 32 bits. But major stm32 timers have 16 bits counters. I think, it's not a good idea to return the memory behind the TIMx_CNT register.  This changes adds the register size checking.

Work queue:  In a recent change for a problem noted by Pascal Speck, it was noted (again by Pascal Speck) that the cancellation of existing work and replacement with new work must be atomic.  Thanks, Pascal.

Networking:  Fix a race condition.  The accept() operation is performed with the network locked.  However, the network is unlocked BEFORE the connected state is set.  Therefore, a context switch may occur and the socket may no longer be connected when it is marked so.  Noted by Pascal Speck.

ARM syscall logic:  Clear bit 0 in PC settings.  Bit 0 is the thumb mode indication and should not be set in the PC.

Mostly cosmetic changes from review of last PR.  Also same inappropriate use of _info changes to configurable debug macros.  Remember folks, if it has a _ or __ in front, it is not intended for use outside of the header file.  It is an internal private definition.

configs: update STM32L4 configs for ADC changes

Approved-by: Gregory Nutt <gnutt@nuttx.org>

Signed-off-by: Juha Niskanen <juha.niskanen@haltian.com>

Networking:  Clean up some naming that has bothered me for a long time... There are no interrupts and no interrupt handlers in the network.  There are events and event handler (there used to to be interrupt logic in there years ago but that is long, long gone).

Networking:  A placeholder for some missing logic in the previous change related to monitoring network status for dup'ed sockets.  If one of the dup'ed socket's is closed, then network monitor resources associated with that one socket must be recovered.  Also, in the event that socket is being used on one thread, but then closed on another, any threads waiting for events from the socket should be informed of the closure.  That latter requirement is not implemented because current data structures do not support it.

Networking:  Fix a runaway recursion problem introduced the previous fixe for shutting down dup'ed sockets.

Networking:  TCP disconnection callbacks are not retained in a list.  This will support mutiple callbacks per lower-level TCP connection structure.  That is necessary for the cae where a socket is dup'ed and shares the same lower-level connection structure.  NOTE:  There still needs to be a call to tcp_start_monitor() when the socket is dup'ed.

Networking:  Move two more TCP specific files from inet/ to tcp/.  There is other TCP-specific logic in inet/ that should be moved sometime, but those are more entangled.

Networking: Move net/inet/net_monitor.c to net/tcp/tcp_monitor.c in preparation for design change to fix monitoring of duplicated sockets.

stm32_hrtim: add DMA configuration

Approved-by: Gregory Nutt <gnutt@nuttx.org>

work_queue() must cancel existing work prior to queuing new work, otherwise the work queue can become corrupted.  Problem noted by Pascal Speck.

stm32_dac.c: support external triggering for DMA transfer

Approved-by: Gregory Nutt <gnutt@nuttx.org>

Master

* stm32f0/Kconfig: remove references to HRTIM

* STM32F33: missing SYSCFG CFGR3 definitions

* stm32_hrtim.h: remove redundant definitions

* stm32_hrtim.c: fix DAC triggers configuration

* stm32_hritm.c: warning message when default value selected

* stm32_hrtim.c: missing master timer logic

* stm32_hrtim.c: add more assertions

* stm32_dac.c: fix conditional

* stm32_dac.c: conditional logic for timer triggering

* stm32_dac.c: fix TSEL configuration when HRTIM

* stm32_dac.c: unnecessary condition

* stm32_dac.c: DMA request remapping

* stm32_dac.c: fix commpilation errors

* stm32_dac.c: add DMA buffers initialization logic

* stm32_hrtim.c: enable DAC triggering

* analog/comp.c: fix compilation errors when poll disabled

* stm32_hrtim.c: remove doubled assertions

Approved-by: Gregory Nutt <gnutt@nuttx.org>