This is an automated email from the ASF dual-hosted git repository.

pkarashchenko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git


The following commit(s) were added to refs/heads/master by this push:
     new 893387b2c Fix the minor style issue
893387b2c is described below

commit 893387b2c590c4dda7eb0363ed67b0fbd5919b91
Author: Xiang Xiao <xiaoxi...@xiaomi.com>
AuthorDate: Sat Oct 8 16:31:21 2022 +0800

    Fix the minor style issue
    
    Signed-off-by: Xiang Xiao <xiaoxi...@xiaomi.com>
---
 Makefile                                           |   4 +-
 audioutils/fmsynth/fmsynth_op.c                    |   2 +-
 audioutils/fmsynth/test/fmsynth_alsa_test.c        |   2 +-
 examples/adxl372_test/adxl372_test_main.c          |   8 +-
 examples/apds9960/apds9960_main.c                  |   4 +-
 examples/bridge/bridge_main.c                      |  14 +--
 examples/bridge/host_main.c                        |  36 +++---
 examples/camera/camera_main.c                      |   4 +-
 examples/configdata/configdata_main.c              |   4 +-
 examples/elf/tests/helloxx/hello++5.cxx            |   4 +-
 examples/elf/tests/mutex/mutex.c                   |   4 +-
 examples/foc/foc_main.c                            |   2 +-
 examples/foc/foc_motor_b16.c                       |   2 +-
 examples/foc/foc_motor_f32.c                       |   6 +-
 examples/foc/foc_thr.c                             |  12 +-
 examples/ft80x/ft80x_coprocessor.c                 |   2 +-
 examples/mount/mount_main.c                        |   2 +-
 examples/netloop/lo_listener.c                     |   6 +-
 examples/nettest/nettest_server.c                  |   2 +-
 examples/nrf24l01_btle/nrf24l01_btle.c             |   2 +-
 examples/nxdemo/nxdemo_bkgd.c                      |   2 +-
 examples/nxflat/tests/mutex/mutex.c                |   2 +-
 examples/nxhello/nxhello_bkgd.c                    |   2 +-
 examples/nximage/nximage_bitmap.c                  |   2 +-
 examples/nximage/nximage_bkgd.c                    |   6 +-
 examples/nxtext/nxtext_putc.c                      |   2 +-
 examples/pdcurses/firework_main.c                  |   2 +-
 examples/pdcurses/newdemo_main.c                   |   2 +-
 examples/pipe/interlock_test.c                     |  11 +-
 examples/pipe/redirect_test.c                      |  86 +++++++-------
 examples/pipe/transfer_test.c                      |  26 ++---
 examples/poll/host.c                               |  10 +-
 examples/poll/net_listener.c                       |  47 +++++---
 examples/pwfb/pwfb_main.c                          |   2 +-
 examples/sotest/lib/modprint/modprint.c            |   2 +-
 examples/sotest/lib/sotest/sotest.c                |  12 +-
 examples/tcpblaster/README.md                      |   6 +-
 examples/thttpd/content/Makefile.nxflat            |   2 +-
 examples/udp/udp_server.c                          |   2 +-
 examples/usrsocktest/usrsocktest_block_recv.c      |   3 +-
 examples/usrsocktest/usrsocktest_block_send.c      |   3 +-
 examples/usrsocktest/usrsocktest_daemon.c          |   4 +-
 .../usrsocktest/usrsocktest_wake_with_signal.c     |  16 +--
 examples/ustream/ustream_client.c                  |  15 ++-
 examples/ws2812/ws2812_main.c                      |   8 +-
 examples/xmlrpc/xmlrpc_main.c                      |   2 +-
 games/shift/shift_input_gesture.h                  |   2 +-
 graphics/ft80x/ft80x_regs.c                        |   2 +-
 graphics/nxwidgets/src/cmultilinetextbox.cxx       |   2 +-
 graphics/pdcurs34/pdcurses/pdc_panel.c             |   2 +-
 graphics/pdcurs34/pdcurses/pdc_terminfo.c          |   2 +-
 graphics/twm4nx/src/ciconmgr.cxx                   |   2 +-
 include/audioutils/fmsynth_op.h                    |   2 +-
 include/graphics/nxwidgets/crlepalettebitmap.hxx   |  12 +-
 include/graphics/nxwm/ccalibration.hxx             |   4 +-
 include/graphics/nxwm/cnxterm.hxx                  |  10 +-
 include/graphics/tiff.h                            |  13 ++-
 include/wireless/ieee802154.h                      |   2 +-
 netutils/chat/chat.c                               |   2 +-
 netutils/codecs/md5.c                              |   2 +-
 netutils/discover/discover.c                       |  11 +-
 netutils/esp8266/esp8266.c                         |   2 +-
 netutils/libcurl4nx/curl4nx_easy_perform.c         |   8 +-
 netutils/libcurl4nx/curl4nx_private.h              |  10 +-
 netutils/netcat/netcat_main.c                      |   2 +-
 netutils/netinit/netinit.c                         |  10 +-
 netutils/ntpclient/ntpclient.c                     |   2 +-
 netutils/pppd/ahdlc.c                              |  40 +++----
 netutils/pppd/ipcp.c                               |  86 +++++++-------
 netutils/pppd/ppp.c                                |   2 +-
 netutils/telnetc/telnetc.c                         |  52 +++++----
 netutils/thttpd/cgi-src/redirect.c                 |   2 +-
 netutils/thttpd/cgi-src/ssi.c                      | 125 ++++++++++++---------
 netutils/thttpd/libhttpd.c                         |  28 ++---
 netutils/thttpd/thttpd.c                           |   6 +-
 netutils/usrsock_rpmsg/usrsock_rpmsg_client.c      |   2 +-
 netutils/usrsock_rpmsg/usrsock_rpmsg_server.c      |   2 +-
 netutils/wakeonlan/wol_main.c                      |   2 +-
 netutils/webclient/webclient.c                     |   6 +-
 netutils/xmlrpc/xmlparser.c                        |   2 +-
 nshlib/nsh_console.c                               |   8 +-
 nshlib/nsh_console.h                               |  12 +-
 nshlib/nsh_romfsetc.c                              |   2 +-
 nshlib/nsh_telnetlogin.c                           |   4 +-
 nshlib/nsh_test.c                                  |   2 +-
 platform/mikroe-stm32f4/mikroe_configdata.c        |  12 +-
 system/cfgdata/cfgdata_main.c                      |  38 +++----
 system/i2c/i2ctool.h                               |   4 +-
 system/nxlooper/nxlooper.c                         |   2 +-
 system/nxplayer/nxplayer.c                         |  58 +++++-----
 system/nxplayer/nxplayer_main.c                    |   2 +-
 system/nxrecorder/nxrecorder.c                     |  35 +++---
 system/popen/popen.c                               |   2 +-
 system/spi/spi_exch.c                              |   2 +-
 system/spi/spi_main.c                              |   4 +-
 system/spi/spitool.h                               |   4 +-
 system/termcurses/tcurses_vt100.c                  |  50 ++++-----
 system/termcurses/termcurses.c                     |  16 +--
 system/zmodem/host/crc16.c                         |   3 +-
 system/zmodem/zm_receive.c                         |   4 +-
 system/zmodem/zm_send.c                            |   4 +-
 testing/ostest/pthread_cleanup.c                   |   8 +-
 testing/ostest/pthread_rwlock.c                    |   8 +-
 testing/ostest/pthread_rwlock_cancel.c             |   4 +-
 testing/ostest/timedmqueue.c                       |   4 +-
 testing/smart_test/smart_test.c                    |  12 +-
 tools/check-hash.sh                                |   4 +-
 wireless/bluetooth/btsak/Makefile                  |   2 +-
 wireless/ieee802154/i8sak/Makefile                 |   2 +-
 wireless/ieee802154/i8sak/i8sak_events.c           |   3 +-
 wireless/ieee802154/i8sak/i8sak_main.c             |   4 +-
 wireless/ieee802154/libmac/sixlowpan_getreq.c      |   4 +-
 112 files changed, 605 insertions(+), 576 deletions(-)

diff --git a/Makefile b/Makefile
index beae68005..3f3d002bb 100644
--- a/Makefile
+++ b/Makefile
@@ -187,13 +187,13 @@ clean: $(foreach SDIR, $(CLEANDIRS), $(SDIR)_clean)
 
 distclean: $(foreach SDIR, $(CLEANDIRS), $(SDIR)_distclean)
 ifeq ($(CONFIG_WINDOWS_NATIVE),y)
-       $(Q) ( if exist  external ( \
+       $(Q) (if exist  external ( \
                echo ********************************************************" \
                echo * The external directory/link must be removed manually *" \
                echo ********************************************************" \
        )
 else
-       $(Q) ( if [ -e external ]; then \
+       $(Q) (if [ -e external ]; then \
                echo 
"********************************************************"; \
                echo "* The external directory/link must be removed manually 
*"; \
                echo 
"********************************************************"; \
diff --git a/audioutils/fmsynth/fmsynth_op.c b/audioutils/fmsynth/fmsynth_op.c
index 3f5fda521..6bb200cd9 100644
--- a/audioutils/fmsynth/fmsynth_op.c
+++ b/audioutils/fmsynth/fmsynth_op.c
@@ -30,7 +30,7 @@
  ****************************************************************************/
 
 #define PHASE_ADJUST(th) \
-        ( ((th) < 0 ? (FMSYNTH_PI) - (th) : (th)) % (FMSYNTH_PI * 2) )
+        (((th) < 0 ? (FMSYNTH_PI) - (th) : (th)) % (FMSYNTH_PI * 2))
 
 /****************************************************************************
  * Private Data
diff --git a/audioutils/fmsynth/test/fmsynth_alsa_test.c 
b/audioutils/fmsynth/test/fmsynth_alsa_test.c
index 1de8fcb3c..73e0b22dc 100644
--- a/audioutils/fmsynth/test/fmsynth_alsa_test.c
+++ b/audioutils/fmsynth/test/fmsynth_alsa_test.c
@@ -307,7 +307,7 @@ int main(void)
           dump_count -= SAMPLE_NUM;
         }
 
-      snd_pcm_writei(hndl, (const void *)samples, SAMPLE_NUM);
+      snd_pcm_writei(hndl, samples, SAMPLE_NUM);
     }
 
   snd_pcm_drain(hndl);
diff --git a/examples/adxl372_test/adxl372_test_main.c 
b/examples/adxl372_test/adxl372_test_main.c
index 4ebefb23b..e30155945 100644
--- a/examples/adxl372_test/adxl372_test_main.c
+++ b/examples/adxl372_test/adxl372_test_main.c
@@ -147,7 +147,7 @@ static int adxl372_test(int is_interactive, FAR char *path)
   };
 
   char bfr[32] __attribute__((aligned(2)));  /* REVISIT: GCC dependent 
attribute */
-  FAR struct XYZ *pxyz = (FAR struct XYZ *) bfr;
+  FAR struct XYZ *pxyz = (FAR struct XYZ *)bfr;
   int rc = PASSED;
   int rc_step = PASSED;
 
@@ -335,11 +335,11 @@ static int adxl372_test(int is_interactive, FAR char 
*path)
           goto error_exit;
         }
 
-      printf("ADXL372 = ( %6d, %6d, %6d))\n",
+      printf("ADXL372 = (%6d, %6d, %6d))\n",
              pxyz->d[0], pxyz->d[1], pxyz->d[2]);
-      printf("ADXL372 = ( 0x%04X, 0x%04X, 0x%04X)\n",
+      printf("ADXL372 = (0x%04X, 0x%04X, 0x%04X)\n",
              pxyz->d[0], pxyz->d[1], pxyz->d[2]);
-      printf("ADXL372 raw = ( 0x%02X%02X, 0x%02X%02X, 0x%02X%02X)\n",
+      printf("ADXL372 raw = (0x%02X%02X, 0x%02X%02X, 0x%02X%02X)\n",
              bfr[1], bfr[0], bfr[3], bfr[2], bfr[5], bfr[4]);
 
       if (is_interactive)
diff --git a/examples/apds9960/apds9960_main.c 
b/examples/apds9960/apds9960_main.c
index a4cc30999..24963fe1c 100644
--- a/examples/apds9960/apds9960_main.c
+++ b/examples/apds9960/apds9960_main.c
@@ -53,7 +53,7 @@ int main(int argc, FAR char *argv[])
   int nbytes;
   char gest;
 
-  fd = open(CONFIG_EXAMPLES_APDS9960_DEVNAME, O_RDONLY|O_NONBLOCK);
+  fd = open(CONFIG_EXAMPLES_APDS9960_DEVNAME, O_RDONLY | O_NONBLOCK);
   if (fd < 0)
     {
       int errcode = errno;
@@ -64,7 +64,7 @@ int main(int argc, FAR char *argv[])
 
   while (1)
     {
-      nbytes = read(fd, (void *)&gest, sizeof(gest));
+      nbytes = read(fd, &gest, sizeof(gest));
       if (nbytes == 1)
         {
           switch (gest)
diff --git a/examples/bridge/bridge_main.c b/examples/bridge/bridge_main.c
index 948ef8d42..7eb932c2b 100644
--- a/examples/bridge/bridge_main.c
+++ b/examples/bridge/bridge_main.c
@@ -358,8 +358,8 @@ static int bridge_net1_worker(int argc, char *argv[])
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(recvsd, SOL_SOCKET, SO_REUSEADDR, (void *)&optval,
-                 sizeof(int)) < 0)
+  if (setsockopt(recvsd, SOL_SOCKET, SO_REUSEADDR,
+                 &optval, sizeof(int)) < 0)
     {
       fprintf(stderr, "NET1 ERROR: setsockopt SO_REUSEADDR failure: %d\n",
               errno);
@@ -399,8 +399,8 @@ static int bridge_net1_worker(int argc, char *argv[])
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(sndsd, SOL_SOCKET, SO_REUSEADDR, (void *)&optval,
-                 sizeof(int)) < 0)
+  if (setsockopt(sndsd, SOL_SOCKET, SO_REUSEADDR,
+                 &optval, sizeof(int)) < 0)
     {
       fprintf(stderr, "NET1 ERROR: setsockopt SO_REUSEADDR failure: %d\n",
               errno);
@@ -549,8 +549,8 @@ static int bridge_net2_worker(int argc, char *argv[])
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(recvsd, SOL_SOCKET, SO_REUSEADDR, (void *)&optval,
-                 sizeof(int)) < 0)
+  if (setsockopt(recvsd, SOL_SOCKET, SO_REUSEADDR,
+                 &optval, sizeof(int)) < 0)
     {
       fprintf(stderr, "NET2 ERROR: setsockopt SO_REUSEADDR failure: %d\n",
               errno);
@@ -590,7 +590,7 @@ static int bridge_net2_worker(int argc, char *argv[])
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(sndsd, SOL_SOCKET, SO_REUSEADDR, (void *)&optval,
+  if (setsockopt(sndsd, SOL_SOCKET, SO_REUSEADDR, &optval,
                  sizeof(int)) < 0)
     {
       fprintf(stderr, "NET2 ERROR: setsockopt SO_REUSEADDR failure: %d\n",
diff --git a/examples/bridge/host_main.c b/examples/bridge/host_main.c
index 29705397e..febb75081 100644
--- a/examples/bridge/host_main.c
+++ b/examples/bridge/host_main.c
@@ -80,16 +80,18 @@ int main(int argc, char *argv[])
   sndsd = socket(PF_INET, SOCK_DGRAM, 0);
   if (sndsd < 0)
     {
-      fprintf(stderr, LABEL "ERROR: Failed to create send socket: %d\n", 
errno);
+      fprintf(stderr, LABEL "ERROR: Failed to create send socket: %d\n",
+              errno);
       return EXIT_FAILURE;
     }
 
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(sndsd, SOL_SOCKET, SO_REUSEADDR, (void*)&optval, sizeof(int)) 
< 0)
+  if (setsockopt(sndsd, SOL_SOCKET, SO_REUSEADDR, &optval, sizeof(int)) < 0)
     {
-      fprintf(stderr, LABEL "ERROR: setsockopt SO_REUSEADDR failure: %d\n", 
errno);
+      fprintf(stderr,
+              LABEL "ERROR: setsockopt SO_REUSEADDR failure: %d\n", errno);
       goto errout_with_sendsd;
     }
 
@@ -99,7 +101,8 @@ int main(int argc, char *argv[])
   sender.sin_port        = 0;
   sender.sin_addr.s_addr = htonl(EXAMPLES_BRIDGE_SEND_IPHOST);
 
-  if (bind(sndsd, (struct sockaddr*)&sender, sizeof(struct sockaddr_in)) < 0)
+  if (bind(sndsd, (struct sockaddr *)&sender,
+           sizeof(struct sockaddr_in)) < 0)
     {
       printf(LABEL "bind failure: %d\n", errno);
       goto errout_with_sendsd;
@@ -113,16 +116,18 @@ int main(int argc, char *argv[])
   recvsd = socket(PF_INET, SOCK_DGRAM, 0);
   if (recvsd < 0)
     {
-      fprintf(stderr, LABEL "ERROR: Failed to create receive socket: %d\n", 
errno);
+      fprintf(stderr,
+              LABEL "ERROR: Failed to create receive socket: %d\n", errno);
       goto errout_with_sendsd;
     }
 
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(recvsd, SOL_SOCKET, SO_REUSEADDR, (void*)&optval, 
sizeof(int)) < 0)
+  if (setsockopt(recvsd, SOL_SOCKET, SO_REUSEADDR, &optval, sizeof(int)) < 0)
     {
-      fprintf(stderr, LABEL "ERROR: setsockopt SO_REUSEADDR failure: %d\n", 
errno);
+      fprintf(stderr,
+              LABEL "ERROR: setsockopt SO_REUSEADDR failure: %d\n", errno);
       goto errout_with_recvsd;
     }
 
@@ -132,7 +137,8 @@ int main(int argc, char *argv[])
   receiver.sin_port        = htons(EXAMPLES_BRIDGE_SEND_HOSTPORT);
   receiver.sin_addr.s_addr = htonl(EXAMPLES_BRIDGE_RECV_IPHOST);
 
-  if (bind(recvsd, (struct sockaddr*)&receiver, sizeof(struct sockaddr_in)) < 
0)
+  if (bind(recvsd, (struct sockaddr *)&receiver,
+           sizeof(struct sockaddr_in)) < 0)
     {
       fprintf(stderr, LABEL "ERROR: bind failure: %d\n", errno);
       goto errout_with_recvsd;
@@ -148,8 +154,8 @@ int main(int argc, char *argv[])
   toaddr.sin_port        = htons(EXAMPLES_BRIDGE_RECV_RECVPORT);
   toaddr.sin_addr.s_addr = htonl(EXAMPLES_BRIDGE_RECV_IPADDR);
 
-   nsent = sendto(sndsd, g_sndmessage,  sizeof(g_sndmessage), 0,
-                 (struct sockaddr*)&toaddr, sizeof(struct sockaddr_in));
+  nsent = sendto(sndsd, g_sndmessage, sizeof(g_sndmessage), 0,
+                 (struct sockaddr *)&toaddr, sizeof(struct sockaddr_in));
 
   /* Check for send errors */
 
@@ -173,7 +179,7 @@ int main(int argc, char *argv[])
 
   addrlen = sizeof(struct sockaddr_in);
   nrecvd = recvfrom(recvsd, g_rdbuffer, EXAMPLES_BRIDGE_SEND_IOBUFIZE, 0,
-                    (struct sockaddr*)&fromaddr, &addrlen);
+                    (struct sockaddr *)&fromaddr, &addrlen);
 
   tmpaddr = ntohl(fromaddr.sin_addr.s_addr);
   printf(LABEL "Received %ld bytes from %d.%d.%d.%d:%d\n",
@@ -207,7 +213,7 @@ int main(int argc, char *argv[])
 
   for (i = 0, j = 0; i < nrecvd; i++)
     {
-      if ( g_rdbuffer[i] == ' ' && j >= 64)
+      if (g_rdbuffer[i] == ' ' && j >= 64)
         {
           putchar('\n');
           j = 0;
@@ -234,8 +240,8 @@ int main(int argc, char *argv[])
   return EXIT_SUCCESS;
 
 errout_with_recvsd:
-   close(recvsd);
+  close(recvsd);
 errout_with_sendsd:
-   close(sndsd);
-   return EXIT_FAILURE;
+  close(sndsd);
+  return EXIT_FAILURE;
 }
diff --git a/examples/camera/camera_main.c b/examples/camera/camera_main.c
index 54c6ab5b5..3a9bf1291 100644
--- a/examples/camera/camera_main.c
+++ b/examples/camera/camera_main.c
@@ -84,7 +84,7 @@ static int camera_prepare(int fd, enum v4l2_buf_type type,
                           uint16_t hsize, uint16_t vsize,
                           struct v_buffer **vbuf,
                           uint8_t buffernum, int buffersize);
-static void free_buffer(struct v_buffer  *buffers, uint8_t bufnum);
+static void free_buffer(struct v_buffer *buffers, uint8_t bufnum);
 static int parse_arguments(int argc, char *argv[],
                            int *capture_num, enum v4l2_buf_type *type);
 static int get_camimage(int fd, struct v4l2_buffer *v4l2_buf,
@@ -246,7 +246,7 @@ static int camera_prepare(int fd, enum v4l2_buf_type type,
  *   All free allocated memory of v_buffer.
  ****************************************************************************/
 
-static void free_buffer(struct v_buffer  *buffers, uint8_t bufnum)
+static void free_buffer(struct v_buffer *buffers, uint8_t bufnum)
 {
   uint8_t cnt;
   if (buffers)
diff --git a/examples/configdata/configdata_main.c 
b/examples/configdata/configdata_main.c
index a90118f25..128e03f47 100644
--- a/examples/configdata/configdata_main.c
+++ b/examples/configdata/configdata_main.c
@@ -302,7 +302,7 @@ static inline int
   config.instance = entry->instance;
   config.len = entry->len;
   config.configdata = g_entryimage;
-  ret = ioctl(g_fd, CFGDIOC_SETCONFIG, (unsigned long) &config);
+  ret = ioctl(g_fd, CFGDIOC_SETCONFIG, (unsigned long)&config);
   if (ret < 0)
     {
       entry->id = 0;
@@ -368,7 +368,7 @@ static inline int
   config.instance = entry->instance;
   config.len = entry->len;
   config.configdata = g_entryimage;
-  ret = ioctl(g_fd, CFGDIOC_GETCONFIG, (unsigned long) &config);
+  ret = ioctl(g_fd, CFGDIOC_GETCONFIG, (unsigned long)&config);
   if (ret < 0)
     {
       return ERROR;
diff --git a/examples/elf/tests/helloxx/hello++5.cxx 
b/examples/elf/tests/helloxx/hello++5.cxx
index 153d05c3f..8a7a3ca28 100644
--- a/examples/elf/tests/helloxx/hello++5.cxx
+++ b/examples/elf/tests/helloxx/hello++5.cxx
@@ -89,7 +89,7 @@ static CThingSayer MyThingSayer;
 
 MyException::MyException(std::string r = NULL) : reason(r)
 {
-  if( r.length() > 0 )
+  if(r.length() > 0)
     {
       cout << "MyException(" << r << "):  constructing with reason." << endl;
     }
@@ -154,7 +154,7 @@ void CThingSayer::ThrowThing(void)
 void CThingSayer::ThrowMyThing(const char *czSayThis = NULL)
 {
   cout << "CThingSayer::ThrowMyThing: I am now throwing an MyException (with 
reason)." << endl;
-  throw MyException( string(czSayThis) );
+  throw MyException(tring(czSayThis));
 }
 
 /////////////////////////////////////////////////////////////////////////////
diff --git a/examples/elf/tests/mutex/mutex.c b/examples/elf/tests/mutex/mutex.c
index 752941cac..714dc94c6 100644
--- a/examples/elf/tests/mutex/mutex.c
+++ b/examples/elf/tests/mutex/mutex.c
@@ -42,7 +42,7 @@ static volatile bool bendoftest;
  * Private Functions
  ****************************************************************************/
 
-/* NOTE: it is necessary for functions that are referred to by function 
pointers
+/* NOTE: it is necessary for functions that are referred to by function
  * pointer to be declared with global scope (at least for ARM).  Otherwise,
  * a relocation type that is not supported by ELF is generated by GCC.
  */
@@ -103,7 +103,7 @@ int main(int argc, char **argv)
 
   printf("Starting thread 1\n");
   bendoftest = false;
-  if ((pthread_create(&thread1, NULL, (void*)thread_func, (void*)1)) != 0)
+  if ((pthread_create(&thread1, NULL, (void *)thread_func, (void *)1)) != 0)
     {
       fprintf(stderr, "Error in thread#1 creation\n");
     }
diff --git a/examples/foc/foc_main.c b/examples/foc/foc_main.c
index a675ccd58..1bbbabbc2 100644
--- a/examples/foc/foc_main.c
+++ b/examples/foc/foc_main.c
@@ -217,7 +217,7 @@ static int foc_mq_send(mqd_t mqd, uint8_t msg, FAR void 
*data)
 
   /* Data max 4B */
 
-  tmp = *((FAR uint32_t *) data);
+  tmp = *((FAR uint32_t *)data);
 
   buffer[0] = msg;
   buffer[1] = ((tmp & 0x000000ff) >> 0);
diff --git a/examples/foc/foc_motor_b16.c b/examples/foc/foc_motor_b16.c
index 503ac6167..a3996f6d1 100644
--- a/examples/foc/foc_motor_b16.c
+++ b/examples/foc/foc_motor_b16.c
@@ -948,7 +948,7 @@ int foc_motor_init(FAR struct foc_motor_b16_s *motor,
   /* Initialize motor alignment data */
 
   align_cfg.volt         = ftob16(CONFIG_EXAMPLES_FOC_ALIGN_VOLT / 1000.0f);
-  align_cfg.offset_steps = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ *     \
+  align_cfg.offset_steps = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ * \
                             CONFIG_EXAMPLES_FOC_ALIGN_SEC / 1000);
 
   /* Connect align callbacks */
diff --git a/examples/foc/foc_motor_f32.c b/examples/foc/foc_motor_f32.c
index 3cf39e923..6c9b54374 100644
--- a/examples/foc/foc_motor_f32.c
+++ b/examples/foc/foc_motor_f32.c
@@ -932,7 +932,7 @@ int foc_motor_init(FAR struct foc_motor_f32_s *motor,
   /* Initialize motor alignment data */
 
   align_cfg.volt         = (CONFIG_EXAMPLES_FOC_ALIGN_VOLT / 1000.0f);
-  align_cfg.offset_steps = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ *     \
+  align_cfg.offset_steps = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ * \
                             CONFIG_EXAMPLES_FOC_ALIGN_SEC / 1000);
 
   /* Connect align callbacks */
@@ -973,9 +973,9 @@ int foc_motor_init(FAR struct foc_motor_f32_s *motor,
   ident_cfg.per         = motor->per;
   ident_cfg.res_current = (CONFIG_EXAMPLES_FOC_IDENT_RES_CURRENT / 1000.0f);
   ident_cfg.ind_volt    = (CONFIG_EXAMPLES_FOC_IDENT_IND_VOLTAGE / 1000.0f);
-  ident_cfg.res_steps   = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ *     \
+  ident_cfg.res_steps   = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ * \
                            CONFIG_EXAMPLES_FOC_IDENT_RES_SEC / 1000);
-  ident_cfg.ind_steps   = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ *     \
+  ident_cfg.ind_steps   = (CONFIG_EXAMPLES_FOC_NOTIFIER_FREQ * \
                            CONFIG_EXAMPLES_FOC_IDENT_IND_SEC / 1000);
   ident_cfg.idle_steps  = CONFIG_EXAMPLES_FOC_IDENT_IDLE;
 
diff --git a/examples/foc/foc_thr.c b/examples/foc/foc_thr.c
index a510e8297..11cb71711 100644
--- a/examples/foc/foc_thr.c
+++ b/examples/foc/foc_thr.c
@@ -69,7 +69,7 @@ static int g_fixed16_thr_cntr = 0;
 
 static FAR void *foc_control_thr(FAR void *arg)
 {
-  FAR struct foc_ctrl_env_s *envp = (FAR struct foc_ctrl_env_s *) arg;
+  FAR struct foc_ctrl_env_s *envp = (FAR struct foc_ctrl_env_s *)arg;
   char                       buffer[CONTROL_MQ_MSGSIZE];
   char                       mqname[10];
   int                        ret  = OK;
@@ -276,11 +276,11 @@ bool foc_threads_terminated(void)
 int foc_ctrlthr_init(FAR struct foc_ctrl_env_s *foc, int i, FAR mqd_t *mqd,
                      FAR pthread_t *thread)
 {
-  char                mqname[10];
-  int                 ret = OK;
-  pthread_attr_t      attr;
-  struct mq_attr      mqattr;
-  struct sched_param  param;
+  char               mqname[10];
+  int                ret = OK;
+  pthread_attr_t     attr;
+  struct mq_attr     mqattr;
+  struct sched_param param;
 
   DEBUGASSERT(foc);
   DEBUGASSERT(mqd);
diff --git a/examples/ft80x/ft80x_coprocessor.c 
b/examples/ft80x/ft80x_coprocessor.c
index d9caba6ba..1e56f0a88 100644
--- a/examples/ft80x/ft80x_coprocessor.c
+++ b/examples/ft80x/ft80x_coprocessor.c
@@ -2066,7 +2066,7 @@ int ft80x_coproc_interactive(int fd, FAR struct 
ft80x_dlbuffer_s *buffer)
       cmds.e.button.cmd         = FT80X_CMD_BUTTON;
       cmds.e.button.x           = ydist;
       cmds.e.button.y           = yoffset;
-      cmds.e.button.w           = 10  *width;
+      cmds.e.button.w           = 10 * width;
       cmds.e.button.h           = height;
       cmds.e.button.font        = fontid;
 
diff --git a/examples/mount/mount_main.c b/examples/mount/mount_main.c
index 510e5bb9d..f31dc181b 100644
--- a/examples/mount/mount_main.c
+++ b/examples/mount/mount_main.c
@@ -599,7 +599,7 @@ static void succeed_stat(const char *path)
 
 int main(int argc, FAR char *argv[])
 {
-  int  ret;
+  int ret;
 
 #ifndef CONFIG_EXAMPLES_MOUNT_DEVNAME
   /* Create a RAM disk for the test */
diff --git a/examples/netloop/lo_listener.c b/examples/netloop/lo_listener.c
index e1d2831d7..f10eb1cda 100644
--- a/examples/netloop/lo_listener.c
+++ b/examples/netloop/lo_listener.c
@@ -85,7 +85,9 @@ static bool net_closeclient(struct net_listener_s *nls, int 
sd)
   close(sd);
   FD_CLR(sd, &nls->master);
 
-  /* If we just closed the max SD, then search downward for the next biggest 
SD. */
+  /* If we just closed the max SD, then search downward for the next
+   * biggest SD.
+   */
 
   while (FD_ISSET(nls->mxsd, &nls->master) == false)
     {
@@ -297,7 +299,7 @@ void *lo_listener(pthread_addr_t pvarg)
   memset(&nls, 0, sizeof(struct net_listener_s));
   if (!net_mksocket(&nls))
     {
-       return (void*)1;
+       return (void *)(uintptr_t)1;
     }
 
   /* Initialize the 'master' file descriptor set */
diff --git a/examples/nettest/nettest_server.c 
b/examples/nettest/nettest_server.c
index aa2dd1ba1..1710af57b 100644
--- a/examples/nettest/nettest_server.c
+++ b/examples/nettest/nettest_server.c
@@ -87,7 +87,7 @@ void nettest_server(void)
 
   optval = 1;
   if (setsockopt(listensd, SOL_SOCKET, SO_REUSEADDR,
-                 (void *)&optval, sizeof(int)) < 0)
+                 &optval, sizeof(int)) < 0)
     {
       printf("server: setsockopt SO_REUSEADDR failure: %d\n", errno);
       goto errout_with_listensd;
diff --git a/examples/nrf24l01_btle/nrf24l01_btle.c 
b/examples/nrf24l01_btle/nrf24l01_btle.c
index 49142cac1..9616a9a2f 100644
--- a/examples/nrf24l01_btle/nrf24l01_btle.c
+++ b/examples/nrf24l01_btle/nrf24l01_btle.c
@@ -468,7 +468,7 @@ static void adv_packet(int wl_fd)
 
 FAR void *advertise(FAR void *arg)
 {
-  uint32_t wl_fd = *(uint32_t *)arg;
+  uint32_t wl_fd = *(FAR uint32_t *)arg;
   while (!quit)
     {
       if (current == 2)
diff --git a/examples/nxdemo/nxdemo_bkgd.c b/examples/nxdemo/nxdemo_bkgd.c
index 7d93e2f8f..f94ed1e67 100644
--- a/examples/nxdemo/nxdemo_bkgd.c
+++ b/examples/nxdemo/nxdemo_bkgd.c
@@ -476,7 +476,7 @@ static void nxdemo_demo_4(NXWINDOW hwnd)
   origin.x   = 0;
   origin.y   = 0;
 
-  src[0] = (FAR const void *)g_battery;
+  src[0] = g_battery;
 
   ret = nx_bitmap((NXWINDOW)hwnd,
                   &rect,
diff --git a/examples/nxflat/tests/mutex/mutex.c 
b/examples/nxflat/tests/mutex/mutex.c
index 9ca14fc3b..bbf7102c9 100644
--- a/examples/nxflat/tests/mutex/mutex.c
+++ b/examples/nxflat/tests/mutex/mutex.c
@@ -103,7 +103,7 @@ int main(int argc, char **argv)
 
   printf("Starting thread 1\n");
   bendoftest = false;
-  if ((pthread_create(&thread1, NULL, (void*)thread_func, (void*)1)) != 0)
+  if ((pthread_create(&thread1, NULL, (void*)thread_func, (void *)1)) != 0)
     {
       fprintf(stderr, "Error in thread#1 creation\n");
     }
diff --git a/examples/nxhello/nxhello_bkgd.c b/examples/nxhello/nxhello_bkgd.c
index 1a18925ca..3c224a398 100644
--- a/examples/nxhello/nxhello_bkgd.c
+++ b/examples/nxhello/nxhello_bkgd.c
@@ -399,7 +399,7 @@ void nxhello_hello(NXWINDOW hwnd)
 
           /* Then put the font on the display */
 
-          src[0] = (FAR const void *)glyph;
+          src[0] = glyph;
 #if CONFIG_NX_NPLANES > 1
 # warning "More logic is needed for the case where CONFIG_NX_PLANES > 1"
 #endif
diff --git a/examples/nximage/nximage_bitmap.c 
b/examples/nximage/nximage_bitmap.c
index 34d0bce18..16b339fea 100644
--- a/examples/nximage/nximage_bitmap.c
+++ b/examples/nximage/nximage_bitmap.c
@@ -3692,5 +3692,5 @@ void nximage_blitrow(FAR nxgl_mxpixel_t *run, FAR const 
void **state)
 
   /* Save the start of the next row and return success */
 
-  *state = (FAR const void *)pos;
+  *state = pos;
 }
diff --git a/examples/nximage/nximage_bkgd.c b/examples/nximage/nximage_bkgd.c
index f63ef144f..953d3f033 100644
--- a/examples/nximage/nximage_bkgd.c
+++ b/examples/nximage/nximage_bkgd.c
@@ -300,7 +300,7 @@ void nximage_image(NXWINDOW hwnd)
       dest.pt1.y = pos.y;
       dest.pt2.y = pos.y;
 
-      src[0] = (FAR const void *)g_runs[0].run;
+      src[0] = g_runs[0].run;
 #if CONFIG_NX_NPLANES > 1
 # warning "More logic is needed for the case where CONFIG_NX_PLANES > 1"
 #endif
@@ -342,7 +342,7 @@ void nximage_image(NXWINDOW hwnd)
       dest.pt1.y = pos.y;
       dest.pt2.y = pos.y;
 
-      src[0] = (FAR const void *)g_runs[0].run;
+      src[0] = g_runs[0].run;
 #if CONFIG_NX_NPLANES > 1
 # warning "More logic is needed for the case where CONFIG_NX_PLANES > 1"
 #endif
@@ -363,7 +363,7 @@ void nximage_image(NXWINDOW hwnd)
       dest.pt1.y = pos.y;
       dest.pt2.y = pos.y;
 
-      src[0] = (FAR const void *)g_runs[1].run;
+      src[0] = g_runs[1].run;
 #if CONFIG_NX_NPLANES > 1
 # warning "More logic is needed for the case where CONFIG_NX_PLANES > 1"
 #endif
diff --git a/examples/nxtext/nxtext_putc.c b/examples/nxtext/nxtext_putc.c
index b2727270f..93c8ca402 100644
--- a/examples/nxtext/nxtext_putc.c
+++ b/examples/nxtext/nxtext_putc.c
@@ -570,7 +570,7 @@ void nxtext_fillchar(NXWINDOW hwnd, FAR const struct 
nxgl_rect_s *rect,
 
       /* Blit the font bitmap into the window */
 
-      src = (FAR const void *)glyph->bitmap;
+      src = glyph->bitmap;
       ret = nx_bitmap((NXWINDOW)hwnd, &intersection, &src,
                       &bm->pos, (unsigned int)glyph->stride);
       if (ret < 0)
diff --git a/examples/pdcurses/firework_main.c 
b/examples/pdcurses/firework_main.c
index 7fe718ae3..b7c17992e 100644
--- a/examples/pdcurses/firework_main.c
+++ b/examples/pdcurses/firework_main.c
@@ -161,7 +161,7 @@ int main(int argc, FAR char *argv[])
       init_pair(i, color_table[i], COLOR_BLACK);
     }
 
-  seed = time((time_t *) 0);
+  seed = time(NULL);
   srand(seed);
   flag = 0;
 
diff --git a/examples/pdcurses/newdemo_main.c b/examples/pdcurses/newdemo_main.c
index 1e34c9f01..962515d7a 100644
--- a/examples/pdcurses/newdemo_main.c
+++ b/examples/pdcurses/newdemo_main.c
@@ -303,7 +303,7 @@ int main(int argc, FAR char *argv[])
 
   traceon();
   initscr();
-  seed = time((time_t *) 0);
+  seed = time(NULL);
   srand(seed);
 
   start_color();
diff --git a/examples/pipe/interlock_test.c b/examples/pipe/interlock_test.c
index 02c60b02d..9a7eabcc9 100644
--- a/examples/pipe/interlock_test.c
+++ b/examples/pipe/interlock_test.c
@@ -57,7 +57,6 @@
 static void *null_writer(pthread_addr_t pvarg)
 {
   int fd;
-  void *p;
 
   /* Wait a bit */
 
@@ -73,8 +72,7 @@ static void *null_writer(pthread_addr_t pvarg)
       fprintf(stderr, \
         "null_writer: Failed to open FIFO %s for writing, errno=%d\n",
               FIFO_PATH2, errno);
-      p = (void *) 1;
-      return p;
+      return (void *)(uintptr_t)1;
     }
 
   /* Wait a bit more */
@@ -93,8 +91,7 @@ static void *null_writer(pthread_addr_t pvarg)
   sleep(5);
 
   printf("null_writer: Returning success\n");
-  p = (void *) 0;
-  return p;
+  return NULL;
 }
 
 /****************************************************************************
@@ -113,7 +110,6 @@ int interlock_test(void)
   ssize_t nbytes;
   int fd;
   int ret;
-  void *p;
 
   /* Create a FIFO */
 
@@ -202,8 +198,7 @@ int interlock_test(void)
   else
     {
       printf("interlock_test: writer returned %p\n", value);
-      p = (void *) 0;
-      if (value != p)
+      if (value != NULL)
         {
           ret = 7;
           goto errout_with_fifo;
diff --git a/examples/pipe/redirect_test.c b/examples/pipe/redirect_test.c
index 5e1dbb256..210894344 100644
--- a/examples/pipe/redirect_test.c
+++ b/examples/pipe/redirect_test.c
@@ -73,9 +73,9 @@ static int redirect_reader(int argc, char *argv[])
   ret = close(fdout);
   if (ret != 0)
     {
-       fprintf(stderr, "redirect_reader: failed to close fdout=%d\n",
-               fdout);
-       return 1;
+      fprintf(stderr, "redirect_reader: failed to close fdout=%d\n",
+              fdout);
+      return 1;
     }
 
   /* Re-direct the fdin to stdin */
@@ -83,9 +83,9 @@ static int redirect_reader(int argc, char *argv[])
   ret = dup2(fdin, 0);
   if (ret != 0)
     {
-        fprintf(stderr, "redirect_reader: dup2 failed: %d\n", errno);
-        close(fdin);
-        return 2;
+      fprintf(stderr, "redirect_reader: dup2 failed: %d\n", errno);
+      close(fdin);
+      return 2;
     }
 
   /* Close the original file descriptor */
@@ -93,23 +93,23 @@ static int redirect_reader(int argc, char *argv[])
   ret = close(fdin);
   if (ret != 0)
     {
-       fprintf(stderr, "redirect_reader: failed to close fdin=%d\n", fdin);
-       return 3;
+      fprintf(stderr, "redirect_reader: failed to close fdin=%d\n", fdin);
+      return 3;
     }
 
   /* Then read from stdin until we hit the end of file */
 
   fflush(stdout);
-  for (;;)
+  for (; ; )
     {
       /* Read from stdin */
 
       ret = read(0, buffer, READ_SIZE);
-      if (ret < 0 )
+      if (ret < 0)
         {
-           fprintf(stderr, "redirect_reader: read failed, errno=%d\n",
-                   errno);
-           return 4;
+          fprintf(stderr, "redirect_reader: read failed, errno=%d\n",
+                  errno);
+          return 4;
         }
       else if (ret == 0)
         {
@@ -123,9 +123,9 @@ static int redirect_reader(int argc, char *argv[])
       ret = write(1, buffer, ret);
       if (ret < 0)
         {
-           fprintf(stderr, "redirect_reader: read failed, errno=%d\n",
-                   errno);
-           return 5;
+          fprintf(stderr, "redirect_reader: read failed, errno=%d\n",
+                  errno);
+          return 5;
         }
     }
 
@@ -133,8 +133,8 @@ static int redirect_reader(int argc, char *argv[])
   ret = close(0);
   if (ret != 0)
     {
-       fprintf(stderr, "redirect_reader: failed to close fd=0\n");
-       return 6;
+      fprintf(stderr, "redirect_reader: failed to close fd=0\n");
+      return 6;
     }
 
   sem_post(&g_rddone);
@@ -165,8 +165,8 @@ static int redirect_writer(int argc, char *argv[])
   ret = close(fdin);
   if (ret != 0)
     {
-       fprintf(stderr, "redirect_reader: failed to close fdin=%d\n", fdin);
-       return 1;
+      fprintf(stderr, "redirect_reader: failed to close fdin=%d\n", fdin);
+      return 1;
     }
 
   /* Re-direct the fdout to stdout */
@@ -174,8 +174,8 @@ static int redirect_writer(int argc, char *argv[])
   ret = dup2(fdout, 1);
   if (ret != 0)
     {
-        fprintf(stderr, "redirect_writer: dup2 failed: %d\n", errno);
-        return 2;
+      fprintf(stderr, "redirect_writer: dup2 failed: %d\n", errno);
+      return 2;
     }
 
   /* Close the original file descriptor */
@@ -183,9 +183,9 @@ static int redirect_writer(int argc, char *argv[])
   ret = close(fdout);
   if (ret != 0)
     {
-       fprintf(stderr, "redirect_reader: failed to close fdout=%d\n",
-               fdout);
-       return 3;
+      fprintf(stderr, "redirect_reader: failed to close fdout=%d\n",
+              fdout);
+      return 3;
     }
 
   /* Then write a bunch of stuff to stdout */
@@ -215,8 +215,8 @@ static int redirect_writer(int argc, char *argv[])
   ret = close(1);
   if (ret != 0)
     {
-       fprintf(stderr, "redirect_writer: failed to close fd=1\n");
-       return 4;
+      fprintf(stderr, "redirect_writer: failed to close fd=1\n");
+      return 4;
     }
 
   fprintf(stderr, "redirect_writer: Returning success\n");
@@ -243,12 +243,13 @@ int redirection_test(void)
 
   sem_init(&g_rddone, 0, 0);
 
-    /* Create the pipe */
+  /* Create the pipe */
 
   ret = pipe(fd);
   if (ret < 0)
     {
-      fprintf(stderr, "redirection_test: pipe failed with errno=%d\n", errno);
+      fprintf(stderr, "redirection_test: pipe failed with errno=%d\n",
+              errno);
       return 5;
     }
 
@@ -260,34 +261,35 @@ int redirection_test(void)
 
   /* Start redirect_reader thread */
 
-  printf("redirection_test: Starting redirect_reader task with fd=%d\n", 
fd[0]);
-  readerid = task_create("redirect_reader", 50, CONFIG_EXAMPLES_PIPE_STACKSIZE,
+  printf("redirection_test: Starting redirect_reader task with fd=%d\n",
+         fd[0]);
+  readerid = task_create("redirect_reader",
+                         50, CONFIG_EXAMPLES_PIPE_STACKSIZE,
                          redirect_reader, (FAR char * const *)argv);
   if (readerid < 0)
     {
-      fprintf(stderr,
-              "redirection_test: Failed to create redirect_writer task: %d\n",
-              errno);
+      fprintf(stderr, "redirection_test: "
+              "Failed to create redirect_writer task: %d\n", errno);
       return 1;
     }
 
   /* Start redirect_writer task */
 
-  printf("redirection_test: Starting redirect_writer task with fd=%d\n", 
fd[1]);
-  writerid = task_create("redirect_writer", 50, CONFIG_EXAMPLES_PIPE_STACKSIZE,
+  printf("redirection_test: Starting redirect_writer task with fd=%d\n",
+         fd[1]);
+  writerid = task_create("redirect_writer",
+                         50, CONFIG_EXAMPLES_PIPE_STACKSIZE,
                          redirect_writer, (FAR char * const *)argv);
   if (writerid < 0)
     {
-      fprintf(stderr,
-              "redirection_test: Failed to create redirect_writer task: %d\n",
-              errno);
+      fprintf(stderr, "redirection_test: "
+              "Failed to create redirect_writer task: %d\n", errno);
 
       ret = task_delete(readerid);
       if (ret != 0)
         {
-          fprintf(stderr,
-                  "redirection_test: Failed to delete redirect_reader task 
%d\n",
-                  errno);
+          fprintf(stderr, "redirection_test: "
+                  "Failed to delete redirect_reader task %d\n", errno);
         }
 
       return 2;
diff --git a/examples/pipe/transfer_test.c b/examples/pipe/transfer_test.c
index b4348c077..1fe5ec03f 100644
--- a/examples/pipe/transfer_test.c
+++ b/examples/pipe/transfer_test.c
@@ -68,7 +68,6 @@ static void *transfer_reader(pthread_addr_t pvarg)
   int nbytes;
   int value;
   int ndx;
-  void *p;
 
   printf("transfer_reader: started\n");
   for (nbytes = 0, value = 0; nbytes < NREAD_BYTES; )
@@ -80,8 +79,7 @@ static void *transfer_reader(pthread_addr_t pvarg)
           "transfer_reader: read failed, errno=%d\n", \
           errno);
 
-          p = (void *) 1;
-          return p;
+          return (void *)(uintptr_t)1;
         }
       else if (ret == 0)
         {
@@ -91,8 +89,7 @@ static void *transfer_reader(pthread_addr_t pvarg)
               "transfer_reader: Too few bytes read -- aborting: %d\n", \
               nbytes);
 
-              p = (void *) 2;
-              return p;
+              return (void *)(uintptr_t)2;
             }
             break;
         }
@@ -110,8 +107,7 @@ static void *transfer_reader(pthread_addr_t pvarg)
               "transfer_reader: Byte %d, expected %d, found %d\n",
                     nbytes + ndx, value, buffer[ndx]);
 
-              p = (void *) 3;
-              return p;
+              return (void *)(uintptr_t)3;
             }
 
           value++;
@@ -124,15 +120,13 @@ static void *transfer_reader(pthread_addr_t pvarg)
           "transfer_reader: Too many bytes read -- aborting: %d\n", \
           nbytes);
 
-          p = (void *) 4;
-          return p;
+          return (void *)(uintptr_t)4;
         }
     }
 
   printf("transfer_reader: %d bytes read\n", nbytes);
 
-  p = (void *) 0;
-  return p;
+  return NULL;
 }
 
 /****************************************************************************
@@ -145,7 +139,6 @@ static void *transfer_writer(pthread_addr_t pvarg)
   int fd = (intptr_t)pvarg;
   int ret;
   int i;
-  void *p;
 
   printf("transfer_writer: started\n");
   for (i = 0; i < WRITE_SIZE; i++)
@@ -160,21 +153,18 @@ static void *transfer_writer(pthread_addr_t pvarg)
         {
           fprintf(stderr, \
           "transfer_writer: write failed, errno=%d\n", errno);
-          p = (void *) 1;
-          return p;
+          return (void *)(uintptr_t)1;
         }
       else if (ret != WRITE_SIZE)
         {
           fprintf(stderr, \
           "transfer_writer: Unexpected write size=%d\n", ret);
-          p = (void *) 2;
-          return p;
+          return (void *)(uintptr_t)2;
         }
     }
 
   printf("transfer_writer: %d bytes written\n", NWRITE_BYTES);
-  p = (void *) 0;
-  return p;
+  return NULL;
 }
 
 /****************************************************************************
diff --git a/examples/poll/host.c b/examples/poll/host.c
index cd6edad5d..868fca718 100644
--- a/examples/poll/host.c
+++ b/examples/poll/host.c
@@ -85,11 +85,13 @@ int main(int argc, char **argv, char **envp)
   myaddr.sin_addr.s_addr = inet_addr(TARGETIP);
 
   printf("client: Connecting to %s...\n", TARGETIP);
-  if (connect( sockfd, (struct sockaddr*)&myaddr, sizeof(struct sockaddr_in)) 
< 0)
+  if (connect(sockfd, (struct sockaddr *)&myaddr,
+              sizeof(struct sockaddr_in)) < 0)
     {
       printf("client: connect failure: %d\n", errno);
       goto errout_with_socket;
     }
+
   printf("client: Connected\n");
 
   /* Then send and receive messages */
@@ -110,7 +112,8 @@ int main(int argc, char **argv, char **envp)
         }
       else if (nbytessent != len)
         {
-          printf("client: Bad send length: %d Expected: %d\n", nbytessent, 
len);
+          printf("client: Bad send length: %d Expected: %d\n",
+                 nbytessent, len);
           goto errout_with_socket;
         }
 
@@ -133,7 +136,8 @@ int main(int argc, char **argv, char **envp)
 
       if (nbytesrecvd != len)
         {
-          printf("client: Bad recv length: %d Expected: %d\n", nbytesrecvd, 
len);
+          printf("client: Bad recv length: %d Expected: %d\n",
+                 nbytesrecvd, len);
           goto errout_with_socket;
         }
       else if (memcmp(inbuf, outbuf, len) != 0)
diff --git a/examples/poll/net_listener.c b/examples/poll/net_listener.c
index 3333f699d..f8d0617e5 100644
--- a/examples/poll/net_listener.c
+++ b/examples/poll/net_listener.c
@@ -85,12 +85,15 @@ static bool net_closeclient(struct net_listener_s *nls, int 
sd)
   close(sd);
   FD_CLR(sd, &nls->master);
 
-  /* If we just closed the max SD, then search downward for the next biggest 
SD. */
+  /* If we just closed the max SD, then search downward for the next
+   * biggest SD.
+   */
 
   while (FD_ISSET(nls->mxsd, &nls->master) == false)
     {
       nls->mxsd -= 1;
     }
+
   return true;
 }
 
@@ -107,7 +110,7 @@ static inline bool net_incomingdata(struct net_listener_s 
*nls, int sd)
   /* Read data from the socket */
 
 #ifdef FIONBIO
-  for (;;)
+  for (; ; )
 #endif
     {
       printf("net_listener: Read data from sd=%d\n", sd);
@@ -132,7 +135,7 @@ static inline bool net_incomingdata(struct net_listener_s 
*nls, int sd)
         }
       else
         {
-          nls->buffer[ret]='\0';
+          nls->buffer[ret] = '\0';
           printf("poll_listener: Read '%s' (%d bytes)\n", nls->buffer, ret);
 
           /* Echo the data back to the client */
@@ -144,9 +147,10 @@ static inline bool net_incomingdata(struct net_listener_s 
*nls, int sd)
                 {
                   if (errno != EINTR)
                     {
-                       printf("net_listener: Send failed sd=%d: %d\n", sd, 
errno);
-                       net_closeclient(nls, sd);
-                       return false;
+                      printf("net_listener: Send failed sd=%d: %d\n",
+                             sd, errno);
+                      net_closeclient(nls, sd);
+                      return false;
                     }
                 }
               else
@@ -157,6 +161,7 @@ static inline bool net_incomingdata(struct net_listener_s 
*nls, int sd)
             }
         }
     }
+
   return 0;
 }
 
@@ -171,10 +176,11 @@ static inline bool net_connection(struct net_listener_s 
*nls)
   /* Loop until all connections have been processed */
 
 #ifdef FIONBIO
-  for (;;)
+  for (; ; )
 #endif
     {
-      printf("net_listener: Accepting new connection on sd=%d\n", 
nls->listensd);
+      printf("net_listener: Accepting new connection on sd=%d\n",
+             nls->listensd);
 
       sd = accept(nls->listensd, NULL, NULL);
       if (sd < 0)
@@ -197,6 +203,7 @@ static inline bool net_connection(struct net_listener_s 
*nls)
             {
                nls->mxsd = sd;
             }
+
           return true;
         }
     }
@@ -225,7 +232,8 @@ static inline bool net_mksocket(struct net_listener_s *nls)
   /* Configure the socket */
 
   value = 1;
-  ret = setsockopt(nls->listensd, SOL_SOCKET,  SO_REUSEADDR, (char*)&value, 
sizeof(int));
+  ret = setsockopt(nls->listensd, SOL_SOCKET,  SO_REUSEADDR,
+                   (char *)&value, sizeof(int));
   if (ret < 0)
     {
       printf("net_listener: setsockopt failed: %d\n", errno);
@@ -251,7 +259,8 @@ static inline bool net_mksocket(struct net_listener_s *nls)
   nls->addr.sin_family      = AF_INET;
   nls->addr.sin_addr.s_addr = htonl(INADDR_ANY);
   nls->addr.sin_port        = htons(LISTENER_PORT);
-  ret = bind(nls->listensd, (struct sockaddr *)&nls->addr, sizeof(struct 
sockaddr_in));
+  ret = bind(nls->listensd, (struct sockaddr *)&nls->addr,
+             sizeof(struct sockaddr_in));
   if (ret < 0)
     {
       printf("net_listener: bind failed: %d\n", errno);
@@ -283,8 +292,9 @@ static void net_configure(void)
   uint8_t mac[IFHWADDRLEN];
 #endif
 
-  /* Configure the network */
-  /* Many embedded network interfaces must have a software assigned MAC */
+  /* Configure the network
+   * Many embedded network interfaces must have a software assigned MAC
+   */
 
 #ifdef CONFIG_EXAMPLES_POLL_NOMAC
   mac[0] = 0x00;
@@ -343,7 +353,7 @@ void *net_listener(pthread_addr_t pvarg)
   memset(&nls, 0, sizeof(struct net_listener_s));
   if (!net_mksocket(&nls))
     {
-       return (void*)1;
+      return (void *)(uintptr_t)1;
     }
 
   /* Initialize the 'master' file descriptor set */
@@ -361,13 +371,15 @@ void *net_listener(pthread_addr_t pvarg)
    * on any of the connect sockets.
    */
 
-  for (;;)
+  for (; ; )
     {
       /* Wait on select */
 
-      printf("net_listener: Calling select(), listener sd=%d\n", nls.listensd);
+      printf("net_listener: Calling select(), listener sd=%d\n",
+             nls.listensd);
       memcpy(&nls.working, &nls.master, sizeof(fd_set));
-      ret = select(nls.mxsd + 1, (FAR fd_set*)&nls.working, (FAR fd_set*)NULL, 
(FAR fd_set*)NULL, &timeout);
+      ret = select(nls.mxsd + 1, (FAR fd_set *)&nls.working,
+                   (FAR fd_set*)NULL, (FAR fd_set *)NULL, &timeout);
       if (ret < 0)
         {
           printf("net_listener: select failed: %d\n", errno);
@@ -411,7 +423,7 @@ void *net_listener(pthread_addr_t pvarg)
   /* Cleanup */
 
 #if 0 /* Don't get here */
-   for (i = 0; i <= nls.mxsd; +i++)
+  for (i = 0; i <= nls.mxsd; +i++)
     {
       if (FD_ISSET(i, &nls.master))
         {
@@ -419,5 +431,6 @@ void *net_listener(pthread_addr_t pvarg)
         }
     }
 #endif
+
   return NULL;  /* Keeps some compilers from complaining */
 }
diff --git a/examples/pwfb/pwfb_main.c b/examples/pwfb/pwfb_main.c
index 0d05e1ac9..1683b600e 100644
--- a/examples/pwfb/pwfb_main.c
+++ b/examples/pwfb/pwfb_main.c
@@ -337,7 +337,7 @@ static bool pwfb_putc(FAR struct pwfb_state_s *st,
 
   /* Render the glyph into the window */
 
-  src = (FAR const void *)glyph->bitmap;
+  src = glyph->bitmap;
   ret =  nxtk_bitmapwindow(wndo->hwnd, &bounds, &src,
                            fpos, (unsigned int)glyph->stride);
   if (ret < 0)
diff --git a/examples/sotest/lib/modprint/modprint.c 
b/examples/sotest/lib/modprint/modprint.c
index f326c5cfe..45bf25e82 100644
--- a/examples/sotest/lib/modprint/modprint.c
+++ b/examples/sotest/lib/modprint/modprint.c
@@ -45,7 +45,7 @@ static void modprint(FAR const char *fmt, ...) printflike(1, 
2);
 static const struct symtab_s g_modprint_exports[1] =
 {
   {
-    (FAR const char *)"modprint", (FAR const void *)modprint,
+    "modprint", (FAR const void *)modprint,
   }
 };
 
diff --git a/examples/sotest/lib/sotest/sotest.c 
b/examples/sotest/lib/sotest/sotest.c
index cfca240d2..f4143c3b4 100644
--- a/examples/sotest/lib/sotest/sotest.c
+++ b/examples/sotest/lib/sotest/sotest.c
@@ -60,22 +60,22 @@ static const char g_msg3[] = "Yes, don't be a stranger!";
 static const struct symtab_s g_sotest_exports[6] =
 {
   {
-    (FAR const char *)"testfunc1", (FAR const void *)testfunc1,
+    "testfunc1", (FAR const void *)testfunc1,
   },
   {
-    (FAR const char *)"testfunc2", (FAR const void *)testfunc2,
+    "testfunc2", (FAR const void *)testfunc2,
   },
   {
-    (FAR const char *)"testfunc3", (FAR const void *)testfunc3,
+    "testfunc3", (FAR const void *)testfunc3,
   },
   {
-    (FAR const char *)"g_msg1",    (FAR const void *)g_msg1,
+    "g_msg1",    (FAR const void *)g_msg1,
   },
   {
-    (FAR const char *)"g_msg2",    (FAR const void *)g_msg2,
+    "g_msg2",    (FAR const void *)g_msg2,
   },
   {
-    (FAR const char *)"g_msg3",    (FAR const void *)g_msg3,
+    "g_msg3",    (FAR const void *)g_msg3,
   },
 };
 
diff --git a/examples/tcpblaster/README.md b/examples/tcpblaster/README.md
index 4b699a782..1ad0e3f3d 100644
--- a/examples/tcpblaster/README.md
+++ b/examples/tcpblaster/README.md
@@ -21,7 +21,7 @@ On nuttx:
 nsh> tcpclient
 Connecting to IPv4 Address: 0100000a
 client: Connected
-[2014-07-31 00:16:15.000] 0: Sent 200 4096-byte buffers:    800.0 KB (avg   
4.0 KB) in   0.18 seconds ( 4444.4 KB/second)
+[2014-07-31 00:16:15.000] 0: Sent 200 4096-byte buffers:    800.0 KB (avg   
4.0 KB) in   0.18 seconds (4444.4 KB/second)
 ```
 
 Now on the host you should see something like:
@@ -31,8 +31,8 @@ $ ./tcpserver
 Binding to IPv4 Address: 00000000
 server: Accepting connections on port 5471
 server: Connection accepted -- receiving
-[2020-02-22 16:17:07.000] 0: Received 200 buffers:   502.9 KB (buffer average 
size:   2.5 KB) in   0.12 seconds ( 4194.8 KB/second)
-[2020-02-22 16:17:07.000] 1: Received 200 buffers:   393.1 KB (buffer average 
size:   2.0 KB) in   0.09 seconds ( 4299.4 KB/second)
+[2020-02-22 16:17:07.000] 0: Received 200 buffers:   502.9 KB (buffer average 
size:   2.5 KB) in   0.12 seconds (4194.8 KB/second)
+[2020-02-22 16:17:07.000] 1: Received 200 buffers:   393.1 KB (buffer average 
size:   2.0 KB) in   0.09 seconds (4299.4 KB/second)
 ```
 
 This will tell you the link speed in KB/sec – kilobytes per second. If you want
diff --git a/examples/thttpd/content/Makefile.nxflat 
b/examples/thttpd/content/Makefile.nxflat
index 906a5c0fe..7d4c91961 100644
--- a/examples/thttpd/content/Makefile.nxflat
+++ b/examples/thttpd/content/Makefile.nxflat
@@ -45,7 +45,7 @@ $(foreach DIR, $(SUBDIRS), $(eval $(call 
DIR_template,$(DIR),install)))
 # Install each program in the romfs directory
 
 install: $(foreach DIR, $(SUBDIRS), $(DIR)_install)
-       $(Q) ( for file in $(INSTALL_FILES); do\
+       $(Q) (for file in $(INSTALL_FILES); do \
                install -m 0644 -D $${file} $(ROMFS_DIR)/$${file}; \
        done; )
 
diff --git a/examples/udp/udp_server.c b/examples/udp/udp_server.c
index abde7ad02..f06c0eb34 100644
--- a/examples/udp/udp_server.c
+++ b/examples/udp/udp_server.c
@@ -104,7 +104,7 @@ void udp_server(void)
   /* Set socket to reuse address */
 
   optval = 1;
-  if (setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR, (void *)&optval,
+  if (setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR, &optval,
                  sizeof(int)) < 0)
     {
       printf("server: setsockopt SO_REUSEADDR failure: %d\n", errno);
diff --git a/examples/usrsocktest/usrsocktest_block_recv.c 
b/examples/usrsocktest/usrsocktest_block_recv.c
index d08a3af84..7382a2465 100644
--- a/examples/usrsocktest/usrsocktest_block_recv.c
+++ b/examples/usrsocktest/usrsocktest_block_recv.c
@@ -356,8 +356,7 @@ static void receive_timeout(FAR struct 
usrsocktest_daemon_conf_s *dconf)
 
   tv.tv_sec = 0;
   tv.tv_usec = 100 * 1000;
-  ret = setsockopt(sd, SOL_SOCKET, SO_RCVTIMEO, (FAR const void *)&tv,
-                   sizeof(tv));
+  ret = setsockopt(sd, SOL_SOCKET, SO_RCVTIMEO, &tv, sizeof(tv));
   TEST_ASSERT_EQUAL(0, ret);
 
   /* Receive data from remote */
diff --git a/examples/usrsocktest/usrsocktest_block_send.c 
b/examples/usrsocktest/usrsocktest_block_send.c
index d029ae465..4e6b799a2 100644
--- a/examples/usrsocktest/usrsocktest_block_send.c
+++ b/examples/usrsocktest/usrsocktest_block_send.c
@@ -284,8 +284,7 @@ static void send_timeout(FAR struct 
usrsocktest_daemon_conf_s *dconf)
 
   tv.tv_sec = 0;
   tv.tv_usec = 100 * 1000;
-  ret = setsockopt(sd, SOL_SOCKET, SO_SNDTIMEO, (FAR const void *)&tv,
-                   sizeof(tv));
+  ret = setsockopt(sd, SOL_SOCKET, SO_SNDTIMEO, &tv, sizeof(tv));
   TEST_ASSERT_EQUAL(0, ret);
 
   /* Try send data to remote. */
diff --git a/examples/usrsocktest/usrsocktest_daemon.c 
b/examples/usrsocktest/usrsocktest_daemon.c
index 9bdd8ccfe..b91bc7122 100644
--- a/examples/usrsocktest/usrsocktest_daemon.c
+++ b/examples/usrsocktest/usrsocktest_daemon.c
@@ -2084,10 +2084,10 @@ int usrsocktest_daemon_stop(void)
       goto out;
     }
 
-  item = (void *)sq_peek(&priv->delayed_cmd_threads);
+  item = (FAR struct delayed_cmd_s *)sq_peek(&priv->delayed_cmd_threads);
   while (item)
     {
-      next = (void *)sq_next(&item->node);
+      next = (FAR struct delayed_cmd_s *)sq_next(&item->node);
 
       pthread_mutex_unlock(&daemon_mutex);
       pthread_join(item->tid, &retval);
diff --git a/examples/usrsocktest/usrsocktest_wake_with_signal.c 
b/examples/usrsocktest/usrsocktest_wake_with_signal.c
index 7e49aa28a..8f08210b2 100644
--- a/examples/usrsocktest/usrsocktest_wake_with_signal.c
+++ b/examples/usrsocktest/usrsocktest_wake_with_signal.c
@@ -160,7 +160,7 @@ static void do_usrsock_blocking_socket_thread(FAR void 
*param)
   TEST_ASSERT_EQUAL(ENETDOWN, errno);
 }
 
-static FAR void * usrsock_blocking_socket_thread(FAR void *param)
+static FAR void *usrsock_blocking_socket_thread(FAR void *param)
 {
   do_usrsock_blocking_socket_thread(param);
   return NULL;
@@ -199,7 +199,7 @@ static void do_usrsock_blocking_close_thread(FAR void 
*param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_close_thread(FAR void *param)
+static FAR void *usrsock_blocking_close_thread(FAR void *param)
 {
   do_usrsock_blocking_close_thread(param);
   return NULL;
@@ -243,7 +243,7 @@ static void do_usrsock_blocking_connect_thread(FAR void 
*param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_connect_thread(FAR void *param)
+static FAR void *usrsock_blocking_connect_thread(FAR void *param)
 {
   do_usrsock_blocking_connect_thread(param);
   return NULL;
@@ -289,7 +289,7 @@ static void do_usrsock_blocking_setsockopt_thread(FAR void 
*param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_setsockopt_thread(FAR void *param)
+static FAR void *usrsock_blocking_setsockopt_thread(FAR void *param)
 {
   do_usrsock_blocking_setsockopt_thread(param);
   return NULL;
@@ -337,7 +337,7 @@ static void do_usrsock_blocking_getsockopt_thread(FAR void 
*param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_getsockopt_thread(FAR void *param)
+static FAR void *usrsock_blocking_getsockopt_thread(FAR void *param)
 {
   do_usrsock_blocking_getsockopt_thread(param);
   return NULL;
@@ -386,7 +386,7 @@ static void do_usrsock_blocking_send_thread(FAR void *param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_send_thread(FAR void *param)
+static FAR void *usrsock_blocking_send_thread(FAR void *param)
 {
   do_usrsock_blocking_send_thread(param);
   return NULL;
@@ -435,7 +435,7 @@ static void do_usrsock_blocking_recv_thread(FAR void *param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_recv_thread(FAR void *param)
+static FAR void *usrsock_blocking_recv_thread(FAR void *param)
 {
   do_usrsock_blocking_recv_thread(param);
   return NULL;
@@ -497,7 +497,7 @@ static void do_usrsock_blocking_poll_thread(FAR void *param)
   test_sd[tidx] = -1;
 }
 
-static FAR void * usrsock_blocking_poll_thread(FAR void *param)
+static FAR void *usrsock_blocking_poll_thread(FAR void *param)
 {
   do_usrsock_blocking_poll_thread(param);
   return NULL;
diff --git a/examples/ustream/ustream_client.c 
b/examples/ustream/ustream_client.c
index fb92c9af1..68d4698e4 100644
--- a/examples/ustream/ustream_client.c
+++ b/examples/ustream/ustream_client.c
@@ -59,8 +59,8 @@ int main(int argc, FAR char *argv[])
 
   /* Allocate buffers */
 
-  outbuf = (char*)malloc(SENDSIZE);
-  inbuf  = (char*)malloc(SENDSIZE);
+  outbuf = (FAR char *)malloc(SENDSIZE);
+  inbuf  = (FAR char *)malloc(SENDSIZE);
   if (!outbuf || !inbuf)
     {
       printf("client: failed to allocate buffers\n");
@@ -90,7 +90,7 @@ int main(int argc, FAR char *argv[])
 
   printf("client: Connecting to %s...\n", CONFIG_EXAMPLES_USTREAM_ADDR);
   addrlen += sizeof(sa_family_t) + 1;
-  ret = connect( sockfd, (struct sockaddr *)&myaddr, addrlen);
+  ret = connect(sockfd, (struct sockaddr *)&myaddr, addrlen);
   if (ret < 0)
     {
       printf("client: connect failure: %d\n", errno);
@@ -147,7 +147,8 @@ int main(int argc, FAR char *argv[])
     }
   else if (nbytessent != SENDSIZE)
     {
-      printf("client: Bad send length: %d Expected: %d\n", nbytessent, 
SENDSIZE);
+      printf("client: Bad send length: %d Expected: %d\n",
+             nbytessent, SENDSIZE);
       goto errout_with_socket;
     }
 
@@ -178,7 +179,8 @@ int main(int argc, FAR char *argv[])
 #endif
 
       printf("client: Receiving...\n");
-      nbytesrecvd = recv(sockfd, &inbuf[totalbytesrecvd], SENDSIZE - 
totalbytesrecvd, 0);
+      nbytesrecvd = recv(sockfd, &inbuf[totalbytesrecvd],
+                         SENDSIZE - totalbytesrecvd, 0);
 
       if (nbytesrecvd < 0)
         {
@@ -198,7 +200,8 @@ int main(int argc, FAR char *argv[])
 
   if (totalbytesrecvd != SENDSIZE)
     {
-      printf("client: Bad recv length: %d Expected: %d\n", totalbytesrecvd, 
SENDSIZE);
+      printf("client: Bad recv length: %d Expected: %d\n",
+             totalbytesrecvd, SENDSIZE);
       goto errout_with_socket;
     }
   else if (memcmp(inbuf, outbuf, SENDSIZE) != 0)
diff --git a/examples/ws2812/ws2812_main.c b/examples/ws2812/ws2812_main.c
index c7fae92d1..0bef6d344 100644
--- a/examples/ws2812/ws2812_main.c
+++ b/examples/ws2812/ws2812_main.c
@@ -43,10 +43,10 @@
 
 struct neo_config_s
 {
-  FAR char  *path;
-  int        loops;
-  int        leds;
-  int        delay;
+  FAR char *path;
+  int       loops;
+  int       leds;
+  int       delay;
 };
 
 /****************************************************************************
diff --git a/examples/xmlrpc/xmlrpc_main.c b/examples/xmlrpc/xmlrpc_main.c
index 0d75c5504..555e3928a 100644
--- a/examples/xmlrpc/xmlrpc_main.c
+++ b/examples/xmlrpc/xmlrpc_main.c
@@ -403,7 +403,7 @@ int main(int argc, FAR char *argv[])
 
   setsockopt(listenfd, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on));
 
-  memset((void *)&servaddr, 0, sizeof(servaddr));
+  memset(&servaddr, 0, sizeof(servaddr));
   servaddr.sin_family = AF_INET;
   servaddr.sin_addr.s_addr = htonl(INADDR_ANY);
   servaddr.sin_port = htons(80);
diff --git a/games/shift/shift_input_gesture.h 
b/games/shift/shift_input_gesture.h
index 3547b3cf6..54feb71c1 100644
--- a/games/shift/shift_input_gesture.h
+++ b/games/shift/shift_input_gesture.h
@@ -62,7 +62,7 @@ int dev_read_input(FAR struct input_state_s *dev)
   int nbytes;
   uint8_t gest;
 
-  nbytes = read(dev->fd_gest, (void *)&gest, sizeof(gest));
+  nbytes = read(dev->fd_gest, &gest, sizeof(gest));
   if (nbytes == sizeof(gest))
     {
       dev->dir = gest;
diff --git a/graphics/ft80x/ft80x_regs.c b/graphics/ft80x/ft80x_regs.c
index a811ae812..1e78e9ea3 100644
--- a/graphics/ft80x/ft80x_regs.c
+++ b/graphics/ft80x/ft80x_regs.c
@@ -43,7 +43,7 @@
 
 #define DLSWAP_DELAY_USEC   (10 * 1000)
 
-/* But timeout after 5 seconds ( 5 * 100 * 10 MS = 5 S) */
+/* But timeout after 5 seconds (5 * 100 * 10 MS = 5 S) */
 
 #define DLSWAP_TIMEOUT_CSEC (5 * 100)
 
diff --git a/graphics/nxwidgets/src/cmultilinetextbox.cxx 
b/graphics/nxwidgets/src/cmultilinetextbox.cxx
index c655ff1e5..6668db645 100644
--- a/graphics/nxwidgets/src/cmultilinetextbox.cxx
+++ b/graphics/nxwidgets/src/cmultilinetextbox.cxx
@@ -1225,7 +1225,7 @@ void CMultiLineTextBox::drawCursor(CGraphicsPort *port)
       // Draw cursor
 
       port->invert(cursorX, cursorY,
-                   m_text->getFont()->getCharWidth( getCursorChar()),
+                   m_text->getFont()->getCharWidth(getCursorChar()),
                    m_text->getFont()->getHeight());
     }
 }
diff --git a/graphics/pdcurs34/pdcurses/pdc_panel.c 
b/graphics/pdcurs34/pdcurses/pdc_panel.c
index 59ec33e01..03edaf490 100644
--- a/graphics/pdcurs34/pdcurses/pdc_panel.c
+++ b/graphics/pdcurs34/pdcurses/pdc_panel.c
@@ -164,7 +164,7 @@ typedef struct panel_ctx_s
 #ifdef CONFIG_PDCURSES_MULTITHREAD
 void *pdc_alloc_panel_ctx()
 {
-  return (void *) zalloc(sizeof(struct panel_ctx_s));
+  return zalloc(sizeof(struct panel_ctx_s));
 }
 #endif
 
diff --git a/graphics/pdcurs34/pdcurses/pdc_terminfo.c 
b/graphics/pdcurs34/pdcurses/pdc_terminfo.c
index 87e1c787e..ebdc8c373 100644
--- a/graphics/pdcurs34/pdcurses/pdc_terminfo.c
+++ b/graphics/pdcurs34/pdcurses/pdc_terminfo.c
@@ -84,7 +84,7 @@ TERMINAL *cur_term = NULL;
 #ifdef CONFIG_PDCURSES_MULTITHREAD
 void *pdc_alloc_term_ctx(void)
 {
-  return (void *) zalloc(sizeof(TERMINAL));
+  return zalloc(sizeof(TERMINAL));
 }
 #endif
 
diff --git a/graphics/twm4nx/src/ciconmgr.cxx b/graphics/twm4nx/src/ciconmgr.cxx
index 5cffa328f..72c7c56fe 100644
--- a/graphics/twm4nx/src/ciconmgr.cxx
+++ b/graphics/twm4nx/src/ciconmgr.cxx
@@ -730,7 +730,7 @@ void CIconMgr::insertEntry(FAR struct SWindowEntry *wentry,
       // Insert the new window mid-list, in name order
 
       NXWidgets::CNxString windowName = cwin->getWindowName();
-      if (windowName.compareTo( tmpwin->cwin->getWindowName()) > 0)
+      if (windowName.compareTo(tmpwin->cwin->getWindowName()) > 0)
         {
           wentry->flink = tmpwin;
           wentry->blink = tmpwin->blink;
diff --git a/include/audioutils/fmsynth_op.h b/include/audioutils/fmsynth_op.h
index aad10de16..bbba43216 100644
--- a/include/audioutils/fmsynth_op.h
+++ b/include/audioutils/fmsynth_op.h
@@ -47,7 +47,7 @@ typedef CODE int (*opfunc_t)(int theta);
 
 typedef struct fmsynth_op_s
 {
-  FAR fmsynth_eg_t  *eg;
+  FAR fmsynth_eg_t *eg;
   opfunc_t wavegen;
   struct fmsynth_op_s *cascadeop;
   struct fmsynth_op_s *parallelop;
diff --git a/include/graphics/nxwidgets/crlepalettebitmap.hxx 
b/include/graphics/nxwidgets/crlepalettebitmap.hxx
index d14ba4cc0..627f565e1 100644
--- a/include/graphics/nxwidgets/crlepalettebitmap.hxx
+++ b/include/graphics/nxwidgets/crlepalettebitmap.hxx
@@ -63,12 +63,12 @@ namespace NXWidgets
 
   struct SRlePaletteBitmap
   {
-    uint8_t          bpp;     /**< Bits per pixel */
-    uint8_t          fmt;     /**< Color format */
-    uint8_t          nlut;    /**< Number of colors in the Look-Up Table (LUT) 
*/
-    nxgl_coord_t     width;   /**< Width in pixels */
-    nxgl_coord_t     height;  /**< Height in rows */
-    FAR const void  *lut[2];  /**< Pointers to the beginning of the Look-Up 
Tables (LUTs) */
+    uint8_t         bpp;     /**< Bits per pixel */
+    uint8_t         fmt;     /**< Color format */
+    uint8_t         nlut;    /**< Number of colors in the Look-Up Table (LUT) 
*/
+    nxgl_coord_t    width;   /**< Width in pixels */
+    nxgl_coord_t    height;  /**< Height in rows */
+    FAR const void *lut[2];  /**< Pointers to the beginning of the Look-Up 
Tables (LUTs) */
 
     /**
      * The pointer to the beginning of the RLE data
diff --git a/include/graphics/nxwm/ccalibration.hxx 
b/include/graphics/nxwm/ccalibration.hxx
index 8ee66f2c8..3e6251f53 100644
--- a/include/graphics/nxwm/ccalibration.hxx
+++ b/include/graphics/nxwm/ccalibration.hxx
@@ -396,8 +396,8 @@ namespace NxWM
   class CCalibrationFactory : public IApplicationFactory
   {
   private:
-    CTaskbar      *m_taskbar;      /**< The taskbar */
-    CTouchscreen  *m_touchscreen;  /**< The touchscreen device */
+    CTaskbar     *m_taskbar;      /**< The taskbar */
+    CTouchscreen *m_touchscreen;  /**< The touchscreen device */
 
   public:
     /**
diff --git a/include/graphics/nxwm/cnxterm.hxx 
b/include/graphics/nxwm/cnxterm.hxx
index fb4acfbcb..4f78fd3ce 100644
--- a/include/graphics/nxwm/cnxterm.hxx
+++ b/include/graphics/nxwm/cnxterm.hxx
@@ -63,11 +63,11 @@ namespace NxWM
   class CNxTerm : public IApplication, private IApplicationCallback
   {
   private:
-    CTaskbar            *m_taskbar;     /**< Reference to the "parent" taskbar 
*/
-    CApplicationWindow  *m_window;      /**< Reference to the application 
window */
-    NXTERM               m_nxterm;      /**< NxTerm handle */
-    pid_t                m_pid;         /**< Task ID of the NxTerm thread */
-    int                  m_minor;       /**< Terminal device minor number */
+    CTaskbar           *m_taskbar;     /**< Reference to the "parent" taskbar 
*/
+    CApplicationWindow *m_window;      /**< Reference to the application 
window */
+    NXTERM              m_nxterm;      /**< NxTerm handle */
+    pid_t               m_pid;         /**< Task ID of the NxTerm thread */
+    int                 m_minor;       /**< Terminal device minor number */
 
     /**
      * This is the NxTerm task.  This function first redirects output to the
diff --git a/include/graphics/tiff.h b/include/graphics/tiff.h
index a857dd0fe..553434d98 100644
--- a/include/graphics/tiff.h
+++ b/include/graphics/tiff.h
@@ -66,10 +66,11 @@
 
 /* Values for the IFD tag type */
 
-#define IFD_TAG_NEWSUBFILETYPE    254 /* NewSubfileType, LONG */
+#define IFD_TAG_NEWSUBFILETYPE    254         /* NewSubfileType, LONG */
 #  define TAG_NEWSUBFILETYPE_REDUCED (1 << 0) /* Bit 0: Reduced resolution 
version of image */
 #  define TAG_NEWSUBFILETYPE_SINGLE  (1 << 1) /* Bit 1: Single page of a 
multi-page image */
 #  define TAG_NEWSUBFILETYPE_TRANSP  (1 << 2) /* Bit 2: Defines a transparency 
mask for image */
+
 #define IFD_TAG_SUBFILETYPE       255 /* SubfileType, SHORT */
 #  define TAG_SUBFILETYPE_FULL      1 /*   Full-resolution image data */
 #  define TAG_SUBFILETYPE_REDUCED   2 /*   Reduced-resolution image data */
@@ -145,11 +146,11 @@
 #  define TAG_GRAYRESPUNIT_100KTHS  5 /*   Number represents 
hundred-thousandths of a unit */
 #define IFD_TAG_GRAYRESPONSECURVE 291 /* GrayResponseCurve, SHORT */
 #define IFD_TAG_T4OPTIONS         292 /* T4Options, LONG */
-#  define TAG_T4OPTIONS_2D        (1 << 0) /*   2-dimensional coding */
-#  define TAG_T4OPTIONS_NONE      (1 << 1) /*   Uncompressed mode */
-#  define TAG_T4OPTIONS_FILL      (1 << 2) /*   Fill bits have been added */
+#  define TAG_T4OPTIONS_2D   (1 << 0) /*   2-dimensional coding */
+#  define TAG_T4OPTIONS_NONE (1 << 1) /*   Uncompressed mode */
+#  define TAG_T4OPTIONS_FILL (1 << 2) /*   Fill bits have been added */
 #define IFD_TAG_T6OPTIONS         293 /* T6Options, LONG */
-#  define TAG_T6OPTIONS_NONE      (1 << 1) /*   Uncompressed mode allowed */
+#  define TAG_T6OPTIONS_NONE (1 << 1) /*   Uncompressed mode allowed */
 #define IFD_TAG_RESUNIT           296 /* ResolutionUnit, SHORT (Required) */
 #  define TAG_RESUNIT_NONE          1 /* No absolute unit of measurement */
 #  define TAG_RESUNIT_INCH          2 /* Inch (default) */
@@ -244,7 +245,7 @@ struct tiff_ifdentry_s
 
 
/************************************************************************************/
 
-/* Structures needed to interface with the TIFF file creation library )and also
+/* Structures needed to interface with the TIFF file creation library) and also
  * structures used only internally by the TIFF file creation library).
  */
 
diff --git a/include/wireless/ieee802154.h b/include/wireless/ieee802154.h
index 383aa3fcb..4049cf885 100644
--- a/include/wireless/ieee802154.h
+++ b/include/wireless/ieee802154.h
@@ -120,7 +120,7 @@ int sixlowpan_assoc_resp(int sock, FAR const char *ifname,
 int sixlowpan_disassoc_req(int sock, FAR const char *ifname,
       FAR const struct ieee802154_disassoc_req_s *req);
 int sixlowpan_get_req(int sock, FAR const char *ifname,
-      FAR const struct ieee802154_get_req_s *req);
+      FAR struct ieee802154_get_req_s *req);
 int sixlowpan_gts_req(int sock, FAR const char *ifname,
       FAR const struct ieee802154_gts_req_s *req);
 int sixlowpan_orphan_resp(int sock, FAR const char *ifname,
diff --git a/netutils/chat/chat.c b/netutils/chat/chat.c
index df42aa537..3954498de 100644
--- a/netutils/chat/chat.c
+++ b/netutils/chat/chat.c
@@ -492,7 +492,7 @@ static void chat_flush(FAR struct chat *priv)
   char c;
 
   ninfo("starting\n");
-  while (chat_readb(priv, (FAR char *) &c, 0) == 0);
+  while (chat_readb(priv, &c, 0) == 0);
   ninfo("done\n");
 }
 
diff --git a/netutils/codecs/md5.c b/netutils/codecs/md5.c
index 93133b27e..83e6dc0aa 100644
--- a/netutils/codecs/md5.c
+++ b/netutils/codecs/md5.c
@@ -84,7 +84,7 @@
 /* This is the central step in the MD5 algorithm. */
 
 #  define MD5STEP(f, w, x, y, z, data, s) \
-        ( w += f(x, y, z) + data,  w = w<<s | w>>(32-s),  w += x )
+        (w += f(x, y, z) + data,  w = w<<s | w>>(32-s),  w += x)
 
 /****************************************************************************
  * Private Functions
diff --git a/netutils/discover/discover.c b/netutils/discover/discover.c
index 7613f17a7..9e7aef9b7 100644
--- a/netutils/discover/discover.c
+++ b/netutils/discover/discover.c
@@ -144,7 +144,7 @@ static inline void discover_initresponse(void)
   g_state.response[0] = DISCOVER_PROTO_ID;
   g_state.response[1] = DISCOVER_RESPONSE;
 
-  strncpy((char *) &g_state.response[2], g_state.info.description,
+  strncpy((char *)&g_state.response[2], g_state.info.description,
           DISCOVER_RESPONSE_SIZE - 3);
 
   for (i = 0; i < DISCOVER_RESPONSE_SIZE - 1; i++)
@@ -161,7 +161,7 @@ static int discover_daemon(int argc, char *argv[])
 {
   int sockfd = -1;
   int nbytes;
-  int addrlen = sizeof(struct sockaddr_in);
+  socklen_t addrlen = sizeof(struct sockaddr_in);
   struct sockaddr_in srcaddr;
 
   /* memset(&g_state, 0, sizeof(struct discover_state_s)); */
@@ -187,8 +187,7 @@ static int discover_daemon(int argc, char *argv[])
       /* Read the next packet */
 
       nbytes = recvfrom(sockfd, &g_state.request, sizeof(g_state.request), 0,
-                        (struct sockaddr *) &srcaddr,
-                        (socklen_t *) &addrlen);
+                        (struct sockaddr *)&srcaddr, &addrlen);
       if (nbytes < 0)
         {
           /* On errors (other EINTR), close the socket and try again */
@@ -305,7 +304,7 @@ static inline int discover_socket(void)
 #ifdef HAVE_SO_REUSEADDR
   optval = 1;
   ret = setsockopt(sockfd, SOL_SOCKET,
-                   SO_REUSEADDR, (void *)&optval, sizeof(int));
+                   SO_REUSEADDR, &optval, sizeof(int));
   if (ret < 0)
     {
       nerr("ERROR: setsockopt SO_REUSEADDR failed: %d\n", errno);
@@ -317,7 +316,7 @@ static inline int discover_socket(void)
 #ifdef HAVE_SO_BROADCAST
   optval = 1;
   ret = setsockopt(sockfd, SOL_SOCKET,
-                   SO_BROADCAST, (void *)&optval, sizeof(int));
+                   SO_BROADCAST, &optval, sizeof(int));
   if (ret < 0)
     {
       nerr("ERROR: setsockopt SO_BROADCAST failed: %d\n", errno);
diff --git a/netutils/esp8266/esp8266.c b/netutils/esp8266/esp8266.c
index 163b486d0..617d162a5 100644
--- a/netutils/esp8266/esp8266.c
+++ b/netutils/esp8266/esp8266.c
@@ -145,7 +145,7 @@ typedef struct
 
   /* ESP Got only One ip + 1 for NULL that indicate end of list */
 
-  in_addr_t *     h_addr_list_buf[2];
+  in_addr_t      *h_addr_list_buf[2];
 
   in_addr_t       in_addr;
 } lesp_state_t;
diff --git a/netutils/libcurl4nx/curl4nx_easy_perform.c 
b/netutils/libcurl4nx/curl4nx_easy_perform.c
index 06bb07277..a0db57afc 100644
--- a/netutils/libcurl4nx/curl4nx_easy_perform.c
+++ b/netutils/libcurl4nx/curl4nx_easy_perform.c
@@ -140,15 +140,15 @@ static int curl4nx_is_header(FAR char *buf, int len,
 int curl4nx_easy_perform(FAR struct curl4nx_s *handle)
 {
   struct sockaddr_in server;
-  FILE *             stream;     /* IOSTREAM used to printf in the socket TODO 
avoid */
+  FAR FILE          *stream;     /* IOSTREAM used to printf in the socket TODO 
avoid */
   int                rxoff;      /* Current offset within RX buffer */
   char               tmpbuf[16]; /* Buffer to hold small strings */
   int                tmplen;     /* Number of bytes used in tmpbuffer */
   int                state;      /* Current state of the parser */
   int                ret;        /* Return value from internal calls */
-  char *             headerbuf;
+  FAR char          *headerbuf;
   int                headerlen;
-  char *             end;
+  FAR char          *end;
   int                cret = CURL4NXE_OK; /* Public return value */
   bool               redirected = false; /* Boolean to manage HTTP 
redirections */
   bool               chunked = false;
@@ -506,7 +506,7 @@ int curl4nx_easy_perform(FAR struct curl4nx_s *handle)
                                                   curl4nx_info(
                                                   "Too many redirections\n");
                                                   cret =
-                                                  CURL4NXE_TOO_MANY_REDIRECTS;
+                                                 CURL4NXE_TOO_MANY_REDIRECTS;
                                                   goto close;
                                                 }
 
diff --git a/netutils/libcurl4nx/curl4nx_private.h 
b/netutils/libcurl4nx/curl4nx_private.h
index cdc45f9c1..96fcdd93e 100644
--- a/netutils/libcurl4nx/curl4nx_private.h
+++ b/netutils/libcurl4nx/curl4nx_private.h
@@ -131,7 +131,7 @@ struct curl4nx_s
   uint32_t                    flags;
   bool                        verbose;
   curl4nx_xferinfofunc_f      progressfunc; /* Called when data chunks are 
received */
-  FAR void *                  progressdata;
+  FAR void                    *progressdata;
 
   /* Request side */
 
@@ -145,21 +145,21 @@ struct curl4nx_s
   int                         max_redirs;
 
   curl4nx_iofunc_f            writefunc; /* Called when data has to be 
uploaded */
-  FAR void *                  writedata;
+  FAR void                    *writedata;
 
   /* Response side */
 
   int                         rxbufsize;
-  FAR char *                  rxbuf;
+  FAR char                    *rxbuf;
 
   int                         status;
   unsigned long long          content_length;
 
   curl4nx_iofunc_f            readfunc; /* Called when data has to be 
downloaded */
-  FAR void *                  readdata;
+  FAR void                    *readdata;
 
   curl4nx_iofunc_f            headerfunc; /* Called when a complete HTTP 
header has been received */
-  FAR void *                  headerdata;
+  FAR void                    *headerdata;
 };
 
 /****************************************************************************
diff --git a/netutils/netcat/netcat_main.c b/netutils/netcat/netcat_main.c
index fc1343c6b..4b26c2449 100644
--- a/netutils/netcat/netcat_main.c
+++ b/netutils/netcat/netcat_main.c
@@ -269,7 +269,7 @@ int netcat_client(int argc, char * argv[])
       goto out;
     }
 
-  if (connect(id, (struct sockaddr *) &server, sizeof(server)) < 0)
+  if (connect(id, (struct sockaddr *)&server, sizeof(server)) < 0)
     {
       perror("error: net: Failed to connect");
       result = 4;
diff --git a/netutils/netinit/netinit.c b/netutils/netinit/netinit.c
index 5c16ff3c7..a742bbf6f 100644
--- a/netutils/netinit/netinit.c
+++ b/netutils/netinit/netinit.c
@@ -703,7 +703,7 @@ static void netinit_configure(void)
 
 #ifdef CONFIG_NETINIT_MONITOR
 static void netinit_signal(int signo, FAR siginfo_t *siginfo,
-                               FAR void * context)
+                           FAR void *context)
 {
   int semcount;
   int ret;
@@ -995,10 +995,10 @@ static pthread_addr_t netinit_thread(pthread_addr_t arg)
 int netinit_bringup(void)
 {
 #ifdef CONFIG_NETINIT_THREAD
-  struct sched_param  sparam;
-  pthread_attr_t      attr;
-  pthread_t           tid;
-  int                 ret;
+  struct sched_param sparam;
+  pthread_attr_t     attr;
+  pthread_t          tid;
+  int                ret;
 
   /* Start the network initialization thread to perform the network bring-up
    * asynchronously.
diff --git a/netutils/ntpclient/ntpclient.c b/netutils/ntpclient/ntpclient.c
index c3c32a0da..5cb62fc75 100644
--- a/netutils/ntpclient/ntpclient.c
+++ b/netutils/ntpclient/ntpclient.c
@@ -1148,7 +1148,7 @@ static int ntpc_get_ntp_sample(FAR struct ntp_servers_s 
*srvs,
    * setsockopt() above)
    */
 
-  nbytes = recvfrom(sd, (void *)&recv, sizeof(struct ntp_datagram_s),
+  nbytes = recvfrom(sd, &recv, sizeof(struct ntp_datagram_s),
                     0, &recvaddr.sa, &socklen);
   recv_time = ntp_localtime();
 
diff --git a/netutils/pppd/ahdlc.c b/netutils/pppd/ahdlc.c
index 07ef997c3..1725cffc5 100644
--- a/netutils/pppd/ahdlc.c
+++ b/netutils/pppd/ahdlc.c
@@ -79,8 +79,8 @@ static uint16_t crcadd(uint16_t crcvalue, uint8_t c)
 {
   uint16_t b;
 
-  b = (crcvalue ^ c) & 0xFF;
-  b = (b ^ (b << 4)) & 0xFF;
+  b = (crcvalue ^ c) & 0xff;
+  b = (b ^ (b << 4)) & 0xff;
   b = (b << 8) ^ (b << 3) ^ (b >> 4);
 
   return ((crcvalue >> 8) ^ b);
@@ -149,7 +149,7 @@ uint8_t ahdlc_rx(FAR struct ppp_context_s *ctx, uint8_t c)
         {
           /* Discard character */
 
-          DEBUG1(("Discard because char is < 0x20 hex and asysnc map is 0\n"));
+          DEBUG1(("Discard since char is < 0x20 hex and asysnc map is 0\n"));
           return 0;
         }
 
@@ -182,8 +182,8 @@ uint8_t ahdlc_rx(FAR struct ppp_context_s *ctx, uint8_t c)
               DEBUG1(("\nReceiving packet with good crc value, len %d\n",
                       ctx->ahdlc_rx_count));
 
-              /* we have a good packet, turn off CTS until we are done with 
this
-               * packet
+              /* we have a good packet, turn off CTS until we are done with
+               * this packet
                */
 
               /* CTS_OFF(); */
@@ -212,7 +212,7 @@ uint8_t ahdlc_rx(FAR struct ppp_context_s *ctx, uint8_t c)
                   /* Send up packet */
 
                   ppp_upcall(ctx, (uint16_t)ctx->ahdlc_rx_buffer[0],
-                             (FAR uint8_t *) & ctx->ahdlc_rx_buffer[1],
+                             (FAR uint8_t *)&ctx->ahdlc_rx_buffer[1],
                              (uint16_t)(ctx->ahdlc_rx_count - 1));
                 }
               else
@@ -222,7 +222,7 @@ uint8_t ahdlc_rx(FAR struct ppp_context_s *ctx, uint8_t c)
                   ppp_upcall(ctx,
                              (uint16_t)(ctx->ahdlc_rx_buffer[0] << 8 | ctx->
                                         ahdlc_rx_buffer[1]),
-                             (FAR uint8_t *) & ctx->ahdlc_rx_buffer[2],
+                             (FAR uint8_t *)&ctx->ahdlc_rx_buffer[2],
                              (uint16_t)(ctx->ahdlc_rx_count - 2));
                 }
 
@@ -232,8 +232,8 @@ uint8_t ahdlc_rx(FAR struct ppp_context_s *ctx, uint8_t c)
             }
           else if (ctx->ahdlc_rx_count > 3)
             {
-              DEBUG1(("\nReceiving packet with bad crc value, was 0x%04x len 
%d\n",
-                     ctx->ahdlc_rx_crc, ctx->ahdlc_rx_count));
+              DEBUG1(("\nReceiving packet with bad crc value, was 0x%04x "
+                      "len %d\n", ctx->ahdlc_rx_crc, ctx->ahdlc_rx_count));
 #ifdef PPP_STATISTICS
               ++ctx->ahdlc_crc_error;
 #endif
@@ -312,16 +312,14 @@ void ahdlc_tx_char(struct ppp_context_s *ctx, uint16_t 
protocol, uint8_t c)
 
   ctx->ahdlc_tx_crc = crcadd(ctx->ahdlc_tx_crc, c);
 
-  /* See if we need to escape char, we always escape 0x7d and 0x7e, in the case
-   * of char < 0x20 we only support async map of default or none, so escape if
-   * ASYNC map is not set.  We may want to modify this to support a bitmap set
+  /* See if we need to escape char, we always escape 0x7d and 0x7e, in the
+   * case of char < 0x20 we only support async map of default or none, so
+   * escape if ASYNC map is not set.  We may want to modify this to support
    * ASYNC map.
    */
 
-  if ((c == 0x7d) || (c == 0x7e) || ((c < 0x20) && ((protocol == LCP) ||
-                                                    (ctx->
-                                                     ahdlc_flags &
-                                                     PPP_TX_ASYNC_MAP) == 0)))
+  if ((c == 0x7d) || (c == 0x7e) || ((c < 0x20) &&
+      ((protocol == LCP) || (ctx->ahdlc_flags & PPP_TX_ASYNC_MAP) == 0)))
     {
       /* Send escape char and xor byte by 0x20 */
 
@@ -344,19 +342,21 @@ void ahdlc_tx_char(struct ppp_context_s *ctx, uint16_t 
protocol, uint8_t c)
  ****************************************************************************/
 
 uint8_t ahdlc_tx(struct ppp_context_s *ctx, uint16_t protocol,
-                 FAR uint8_t * header, FAR uint8_t * buffer, uint16_t 
headerlen,
-                 uint16_t datalen)
+                 FAR uint8_t * header, FAR uint8_t * buffer,
+                 uint16_t headerlen, uint16_t datalen)
 {
   uint16_t i;
   uint8_t c;
 
-  DEBUG1(("\nAHDLC_TX - transmit frame, protocol 0x%04x, length %d  offline 
%d\n",
+  DEBUG1(("\nAHDLC_TX - transmit frame, protocol 0x%04x, length %d "
+          "offline %d\n",
          protocol, datalen + headerlen, ctx->ahdlc_tx_offline));
 
   if (AHDLC_TX_OFFLINE && (ctx->ahdlc_tx_offline++ > AHDLC_TX_OFFLINE))
     {
       ctx->ahdlc_tx_offline = 0;
-      DEBUG1(("\nAHDLC_TX to many outstanding TX packets => 
ppp_reconnect()\n"));
+      DEBUG1(("\nAHDLC_TX to many outstanding TX packets => "
+              "ppp_reconnect()\n"));
       ppp_reconnect(ctx);
       return 0;
     }
diff --git a/netutils/pppd/ipcp.c b/netutils/pppd/ipcp.c
index 3f1768f3a..b4f4fe34e 100644
--- a/netutils/pppd/ipcp.c
+++ b/netutils/pppd/ipcp.c
@@ -87,7 +87,7 @@ static const uint8_t g_ipcplist[] =
 #if PPP_DEBUG
 void printip(struct in_addr ip2)
 {
-  char *ip = (FAR uint8_t *) & ip2.s_addr;
+  FAR unt8_t *ip = (FAR uint8_t *)&ip2.s_addr;
   DEBUG1((" %d.%d.%d.%d ", ip[0], ip[1], ip[2], ip[3]));
 }
 #else
@@ -173,10 +173,10 @@ void ipcp_rx(FAR struct ppp_context_s *ctx, FAR uint8_t * 
buffer,
 
               ++bptr;
 #ifdef IPCP_GET_PEER_IP
-              ((FAR uint8_t *) & ctx->peer_ip)[0] = *bptr++;
-              ((FAR uint8_t *) & ctx->peer_ip)[1] = *bptr++;
-              ((FAR uint8_t *) & ctx->peer_ip)[2] = *bptr++;
-              ((FAR uint8_t *) & ctx->peer_ip)[3] = *bptr++;
+              ((FAR uint8_t *)&ctx->peer_ip)[0] = *bptr++;
+              ((FAR uint8_t *)&ctx->peer_ip)[1] = *bptr++;
+              ((FAR uint8_t *)&ctx->peer_ip)[2] = *bptr++;
+              ((FAR uint8_t *)&ctx->peer_ip)[3] = *bptr++;
 
               DEBUG1(("Peer IP "));
               printip(ctx->peer_ip);
@@ -275,29 +275,29 @@ void ipcp_rx(FAR struct ppp_context_s *ctx, FAR uint8_t * 
buffer,
               /* Dump length */
 
               bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[0] = *bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[1] = *bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[2] = *bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[3] = *bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[0] = *bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[1] = *bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[2] = *bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[3] = *bptr++;
               break;
 
 #  ifdef IPCP_GET_PRI_DNS
             case IPCP_PRIMARY_DNS:
               bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[0] = *bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[1] = *bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[2] = *bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[3] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[0] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[1] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[2] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[3] = *bptr++;
               break;
 #  endif
 
 #  ifdef IPCP_GET_SEC_DNS
             case IPCP_SECONDARY_DNS:
               bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[0] = *bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[1] = *bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[2] = *bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[3] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[0] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[1] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[2] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[3] = *bptr++;
               break;
 #  endif
 
@@ -346,22 +346,22 @@ void ipcp_rx(FAR struct ppp_context_s *ctx, FAR uint8_t * 
buffer,
 
               bptr++;
 
-              ((FAR uint8_t *) & ctx->local_ip)[0] = (char)*bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[1] = (char)*bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[2] = (char)*bptr++;
-              ((FAR uint8_t *) & ctx->local_ip)[3] = (char)*bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[0] = (char)*bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[1] = (char)*bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[2] = (char)*bptr++;
+              ((FAR uint8_t *)&ctx->local_ip)[3] = (char)*bptr++;
 
-              netlib_ifup((char *)ctx->ifname);
-              netlib_set_ipv4addr((char *)ctx->ifname, &ctx->local_ip);
+              netlib_ifup((FAR char *)ctx->ifname);
+              netlib_set_ipv4addr((FAR char *)ctx->ifname, &ctx->local_ip);
               break;
 
 #ifdef IPCP_GET_PRI_DNS
             case IPCP_PRIMARY_DNS:
               bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[0] = *bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[1] = *bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[2] = *bptr++;
-              ((FAR uint8_t *) & ctx->pri_dns_addr)[3] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[0] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[1] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[2] = *bptr++;
+              ((FAR uint8_t *)&ctx->pri_dns_addr)[3] = *bptr++;
               netlib_set_ipv4dnsaddr(&ctx->pri_dns_addr);
               break;
 #endif
@@ -369,10 +369,10 @@ void ipcp_rx(FAR struct ppp_context_s *ctx, FAR uint8_t * 
buffer,
 #ifdef IPCP_GET_SEC_DNS
             case IPCP_SECONDARY_DNS:
               bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[0] = *bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[1] = *bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[2] = *bptr++;
-              ((FAR uint8_t *) & ctx->sec_dns_addr)[3] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[0] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[1] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[2] = *bptr++;
+              ((FAR uint8_t *)&ctx->sec_dns_addr)[3] = *bptr++;
               netlib_set_ipv4dnsaddr(&ctx->sec_dns_addr);
               break;
 #endif
@@ -486,10 +486,10 @@ void ipcp_task(FAR struct ppp_context_s *ctx, FAR uint8_t 
* buffer)
 
           *bptr++ = IPCP_IPADDRESS;
           *bptr++ = 0x6;
-          *bptr++ = (uint8_t)((FAR uint8_t *) & ctx->local_ip)[0];
-          *bptr++ = (uint8_t)((FAR uint8_t *) & ctx->local_ip)[1];
-          *bptr++ = (uint8_t)((FAR uint8_t *) & ctx->local_ip)[2];
-          *bptr++ = (uint8_t)((FAR uint8_t *) & ctx->local_ip)[3];
+          *bptr++ = (uint8_t)((FAR uint8_t *)&ctx->local_ip)[0];
+          *bptr++ = (uint8_t)((FAR uint8_t *)&ctx->local_ip)[1];
+          *bptr++ = (uint8_t)((FAR uint8_t *)&ctx->local_ip)[2];
+          *bptr++ = (uint8_t)((FAR uint8_t *)&ctx->local_ip)[3];
 
 #ifdef IPCP_GET_PRI_DNS
           if ((ctx->ipcp_state & IPCP_PRI_DNS_BIT) == 0)
@@ -498,10 +498,10 @@ void ipcp_task(FAR struct ppp_context_s *ctx, FAR uint8_t 
* buffer)
 
               *bptr++ = IPCP_PRIMARY_DNS;
               *bptr++ = 0x6;
-              *bptr++ = ((FAR uint8_t *) & ctx->pri_dns_addr)[0];
-              *bptr++ = ((FAR uint8_t *) & ctx->pri_dns_addr)[1];
-              *bptr++ = ((FAR uint8_t *) & ctx->pri_dns_addr)[2];
-              *bptr++ = ((FAR uint8_t *) & ctx->pri_dns_addr)[3];
+              *bptr++ = ((FAR uint8_t *)&ctx->pri_dns_addr)[0];
+              *bptr++ = ((FAR uint8_t *)&ctx->pri_dns_addr)[1];
+              *bptr++ = ((FAR uint8_t *)&ctx->pri_dns_addr)[2];
+              *bptr++ = ((FAR uint8_t *)&ctx->pri_dns_addr)[3];
             }
 #endif
 
@@ -512,10 +512,10 @@ void ipcp_task(FAR struct ppp_context_s *ctx, FAR uint8_t 
* buffer)
 
               *bptr++ = IPCP_SECONDARY_DNS;
               *bptr++ = 0x6;
-              *bptr++ = ((FAR uint8_t *) & ctx->sec_dns_addr)[0];
-              *bptr++ = ((FAR uint8_t *) & ctx->sec_dns_addr)[1];
-              *bptr++ = ((FAR uint8_t *) & ctx->sec_dns_addr)[2];
-              *bptr++ = ((FAR uint8_t *) & ctx->sec_dns_addr)[3];
+              *bptr++ = ((FAR uint8_t *)&ctx->sec_dns_addr)[0];
+              *bptr++ = ((FAR uint8_t *)&ctx->sec_dns_addr)[1];
+              *bptr++ = ((FAR uint8_t *)&ctx->sec_dns_addr)[2];
+              *bptr++ = ((FAR uint8_t *)&ctx->sec_dns_addr)[3];
             }
 #endif
 
diff --git a/netutils/pppd/ppp.c b/netutils/pppd/ppp.c
index 12a2349fe..0e27617fd 100644
--- a/netutils/pppd/ppp.c
+++ b/netutils/pppd/ppp.c
@@ -113,7 +113,7 @@ static void ppp_reject_protocol(FAR struct ppp_context_s 
*ctx,
   /* pkt->id = tid++;  write tid */
 
   pkt->len = htons(count + 6);
-  *((FAR uint16_t *) (&pkt->data[0])) = htons(protocol);
+  *((FAR uint16_t *)(&pkt->data[0])) = htons(protocol);
 
   ahdlc_tx(ctx, LCP, buffer, 0, (uint16_t)(count + 6), 0);
 }
diff --git a/netutils/telnetc/telnetc.c b/netutils/telnetc/telnetc.c
index ce59bbad5..5051e3b8f 100644
--- a/netutils/telnetc/telnetc.c
+++ b/netutils/telnetc/telnetc.c
@@ -236,7 +236,7 @@ enum telnet_error_e _init_zlib(FAR struct telnet_s *telnet, 
int deflate,
 
   /* Allocate zstream box */
 
-  if ((z = (z_stream *) calloc(1, sizeof(z_stream))) == 0)
+  if ((z = (z_stream *)calloc(1, sizeof(z_stream))) == 0)
     {
       return _error(telnet, __LINE__, __func__, TELNET_ENOMEM, err_fatal,
                     "malloc() failed: %d", errno);
@@ -381,8 +381,8 @@ static inline int _check_telopt(FAR struct telnet_s *telnet,
 
 /* Retrieve RFC1143 option state */
 
-static inline struct telnet_rfc1143_s _get_rfc1143(FAR struct telnet_s *telnet,
-                                                   unsigned char telopt)
+static inline struct telnet_rfc1143_s
+_get_rfc1143(FAR struct telnet_s *telnet, unsigned char telopt)
 {
   struct telnet_rfc1143_s empty;
   int i;
@@ -425,9 +425,9 @@ static inline void _set_rfc1143(FAR struct telnet_s *telnet,
 
   /* we're going to need to track state for it, so grow the queue by 4 (four)
    * elements and put the telopt into it; bail on allocation error.  we go by
-   * four because it seems like a reasonable guess as to the number of enabled
-   * options for most simple code, and it allows for an acceptable number of
-   * reallocations for complex code.
+   * four because it seems like a reasonable guess as to the number of
+   * enabled options for most simple code, and it allows for an acceptable
+   * number of reallocations for complex code.
    */
 
   qtmp = (struct telnet_rfc1143_s *)
@@ -1307,7 +1307,9 @@ static void _process(FAR struct telnet_s *telnet, FAR 
const char *buffer,
 
         case TELNET_STATE_DATA:
 
-          /* On an IAC byte, pass through all pending bytes and switch states 
*/
+          /* On an IAC byte, pass through all pending bytes and switch
+           * states
+           */
 
           if (byte == TELNET_IAC)
             {
@@ -1421,8 +1423,9 @@ static void _process(FAR struct telnet_s *telnet, FAR 
const char *buffer,
               /* In 1998 MCCP used TELOPT 85 and the protocol defined an
                * invalid subnegotiation sequence (IAC SB 85 WILL SE) to start
                * compression. Subsequently MCCP version 2 was created in 2000
-               * using TELOPT 86 and a valid subnegotiation (IAC SB 86 IAC SE).
-               * libtelnet for now just captures and discards MCCPv1 sequences.
+               * using TELOPT 86 and a valid subnegotiation (IAC SB 86 IAC
+               * SE). libtelnet for now just captures and discards MCCPv1
+               * sequences.
                */
 
               start = i + 2;
@@ -1455,10 +1458,11 @@ static void _process(FAR struct telnet_s *telnet, FAR 
const char *buffer,
 
               if (_subnegotiate(telnet) != 0)
                 {
-                  /* Any remaining bytes in the buffer are compressed. we have
-                   * to re-invoke telnet_recv to get those bytes inflated and
-                   * abort trying to process the remaining compressed bytes in
-                   * the current _process buffer argument.
+                  /* Any remaining bytes in the buffer are compressed. we
+                   * have to re-invoke telnet_recv to get those bytes
+                   * inflated and abort trying to process the remaining
+                   * compressed bytes in the current _process buffer
+                   * argument.
                    */
 
                   telnet_recv(telnet, &buffer[start], size - start);
@@ -1483,9 +1487,9 @@ static void _process(FAR struct telnet_s *telnet, FAR 
const char *buffer,
                 }
               break;
 
-              /* Something else -- protocol error.  attempt to process content
-               * in subnegotiation buffer, then evaluate the given command as
-               * an IAC code.
+              /* Something else -- protocol error.  attempt to process
+               * content in subnegotiation buffer, then evaluate the given
+               * command as an IAC code.
                */
 
             default:
@@ -1508,9 +1512,9 @@ static void _process(FAR struct telnet_s *telnet, FAR 
const char *buffer,
                 }
               else
                 {
-                  /* Recursive call to get the current input byte processed as
-                   * a regular IAC command.  we could use a goto, but that
-                   * would be gross.
+                  /* Recursive call to get the current input byte
+                   * processed as a regular IAC command.  we could use a
+                   * goto, but that would be gross.
                    */
 
                   _process(telnet, (char *)&byte, 1);
@@ -1877,8 +1881,8 @@ void telnet_subnegotiation(FAR struct telnet_s *telnet, 
unsigned char telopt,
   _sendu(telnet, bytes + 3, 2);
 
 #if defined(HAVE_ZLIB)
-  /* If we're a proxy and we just sent the COMPRESS2 marker, we must make sure
-   * all further data is compressed if not already.
+  /* If we're a proxy and we just sent the COMPRESS2 marker, we must make
+   * sure all further data is compressed if not already.
    */
 
   if (telnet->flags & TELNET_FLAG_PROXY && telopt == TELNET_TELOPT_COMPRESS2)
@@ -1934,9 +1938,9 @@ void telnet_begin_compress2(FAR struct telnet_s *telnet)
       return;
     }
 
-  /* Send compression marker.  we send directly to the event handler instead of
-   * passing through _send because _send would result in the compress marker
-   * itself being compressed.
+  /* Send compression marker.  we send directly to the event handler instead
+   * of passing through _send because _send would result in the compress
+   * marker itself being compressed.
    */
 
   ev.type        = TELNET_EV_SEND;
diff --git a/netutils/thttpd/cgi-src/redirect.c 
b/netutils/thttpd/cgi-src/redirect.c
index 42df5f650..943ac17c4 100644
--- a/netutils/thttpd/cgi-src/redirect.c
+++ b/netutils/thttpd/cgi-src/redirect.c
@@ -193,7 +193,7 @@ int main(int argc, char *argv[])
   /* Open the redirects file. */
 
   fp = fopen(".redirects", "r");
-  if (fp == (FILE *) 0)
+  if (fp == NULL)
     {
       internal_error("Couldn't open .redirects file.");
       errcode = 3;
diff --git a/netutils/thttpd/cgi-src/ssi.c b/netutils/thttpd/cgi-src/ssi.c
index 7104af2cc..9a2bb729f 100644
--- a/netutils/thttpd/cgi-src/ssi.c
+++ b/netutils/thttpd/cgi-src/ssi.c
@@ -99,81 +99,82 @@ static void internal_error(char *reason)
 {
   char *title = "500 Internal Error";
 
-  printf("\
-<HTML><HEAD><TITLE>%s</TITLE></HEAD>\n\
-<BODY><H2>%s</H2>\n\
-Something unusual went wrong during a server-side-includes request:\n\
-<BLOCKQUOTE>\n\
-%s\n\
-</BLOCKQUOTE>\n\
-</BODY></HTML>\n", title, title, reason);
+  printf("<HTML><HEAD><TITLE>%s</TITLE></HEAD>\n"
+    "<BODY><H2>%s</H2>\n"
+    "Something unusual went wrong during a server-side-includes request:\n"
+    "<BLOCKQUOTE>\n"
+    "%s\n"
+    "</BLOCKQUOTE>\n"
+    "</BODY></HTML>\n",
+    title, title, reason);
 }
 
 static void not_found(char *filename)
 {
   char *title = "404 Not Found";
 
-  printf("\
-<HTML><HEAD><TITLE>%s</TITLE></HEAD>\n\
-<BODY><H2>%s</H2>\n\
-The requested server-side-includes filename, %s,\n\
-does not seem to exist.\n\
-</BODY></HTML>\n", title, title, filename);
+  printf("<HTML><HEAD><TITLE>%s</TITLE></HEAD>\n"
+         "<BODY><H2>%s</H2>\n"
+         "The requested server-side-includes filename, %s,\n"
+         "does not seem to exist.\n"
+         "</BODY></HTML>\n",
+         title, title, filename);
 }
 
 static void not_found2(char *directive, char *tag, char *filename)
 {
   char *title = "Not Found";
 
-  printf("\
-<HR><H2>%s</H2>\n\
-The filename requested in a %s %s directive, %s,\n\
-does not seem to exist.\n\
-<HR>\n", title, directive, tag, filename);
+  printf("<HR><H2>%s</H2>\n"
+         "The filename requested in a %s %s directive, %s,\n"
+         "does not seem to exist.\n"
+         "<HR>\n",
+         title, directive, tag, filename);
 }
 
 static void not_permitted(char *directive, char *tag, char *val)
 {
   char *title = "Not Permitted";
 
-  printf("\
-<HR><H2>%s</H2>\n\
-The filename requested in the %s %s=%s directive\n\
-may not be fetched.\n\
-<HR>\n", title, directive, tag, val);
+  printf("<HR><H2>%s</H2>\n"
+         "The filename requested in the %s %s=%s directive\n"
+         "may not be fetched.\n"
+         "<HR>\n",
+         title, directive, tag, val);
 }
 
 static void unknown_directive(char *filename, char *directive)
 {
   char *title = "Unknown Directive";
 
-  printf("\
-<HR><H2>%s</H2>\n\
-The requested server-side-includes filename, %s,\n\
-tried to use an unknown directive, %s.\n\
-<HR>\n", title, filename, directive);
+  printf("<HR><H2>%s</H2>\n"
+         "The requested server-side-includes filename, %s,\n"
+         "tried to use an unknown directive, %s.\n"
+         "<HR>\n",
+         title, filename, directive);
 }
 
 static void unknown_tag(char *filename, char *directive, char *tag)
 {
   char *title = "Unknown Tag";
 
-  printf("\
-<HR><H2>%s</H2>\n\
-The requested server-side-includes filename, %s,\n\
-tried to use the directive %s with an unknown tag, %s.\n\
-<HR>\n", title, filename, directive, tag);
+  printf("<HR><H2>%s</H2>\n"
+         "The requested server-side-includes filename, %s,\n"
+         "tried to use the directive %s with an unknown tag, %s.\n"
+         "<HR>\n",
+         title, filename, directive, tag);
 }
 
-static void unknown_value(char *filename, char *directive, char *tag, char 
*val)
+static void unknown_value(char *filename, char *directive,
+                          char *tag, char *val)
 {
   char *title = "Unknown Value";
 
-  printf("\
-<HR><H2>%s</H2>\n\
-The requested server-side-includes filename, %s,\n\
-tried to use the directive %s %s with an unknown value, %s.\n\
-<HR>\n", title, filename, directive, tag, val);
+  printf("<HR><H2>%s</H2>\n"
+         "The requested server-side-includes filename, %s,\n"
+         "tried to use the directive %s %s with an unknown value, %s.\n"
+         "<HR>\n",
+         title, filename, directive, tag, val);
 }
 
 static int get_filename(char *vfilename, char *filename,
@@ -223,6 +224,7 @@ static int get_filename(char *vfilename, char *filename,
           not_permitted(directive, tag, val);
           return -1;
         }
+
       if (fl + 1 + strlen(val) >= fnsize)
         {
           return -1;
@@ -235,6 +237,7 @@ static int get_filename(char *vfilename, char *filename,
           cp = &fn[strlen(fn)];
           *cp = '/';
         }
+
       strcpy(++cp, val);
     }
   else
@@ -242,6 +245,7 @@ static int get_filename(char *vfilename, char *filename,
       unknown_tag(filename, directive, tag);
       return -1;
     }
+
   return 0;
 }
 
@@ -269,12 +273,13 @@ static int check_filename(char *filename)
           cgi_pattern = CGI_PATTERN;
         }
 #endif /* CGI_PATTERN */
+
       inited = 1;
     }
 
   /* ../ is not permitted. */
 
-  if (strstr(filename, "../") !=NULL)
+  if (strstr(filename, "../") != NULL)
     {
       return 0;
     }
@@ -285,7 +290,8 @@ static int check_filename(char *filename)
   fnl = strlen(filename);
   if (strcmp(filename, CONFIG_AUTH_FILE) == 0 ||
       (fnl >= sizeof(CONFIG_AUTH_FILE) &&
-       strcmp(&filename[fnl - sizeof(CONFIG_AUTH_FILE) + 1], CONFIG_AUTH_FILE) 
== 0 &&
+       strcmp(&filename[fnl - sizeof(CONFIG_AUTH_FILE) + 1],
+              CONFIG_AUTH_FILE) == 0 &&
        filename[fnl - sizeof(CONFIG_AUTH_FILE)] == '/'))
     {
       return 0;
@@ -342,6 +348,7 @@ static int check_filename(char *filename)
     {
       return 0;
     }
+
   return 1;
 }
 
@@ -440,7 +447,8 @@ static void do_include(FILE *instream, char *vfilename, 
char *filename,
 
   /* Inserts the text of another document into the parsed document. */
 
-  ret = get_filename(vfilename, filename, directive, tag, val, g_iobuffer1, 
BUFFER_SIZE);
+  ret = get_filename(vfilename, filename, directive, tag, val, g_iobuffer1,
+                     BUFFER_SIZE);
   if (ret < 0)
     {
       return;
@@ -453,7 +461,7 @@ static void do_include(FILE *instream, char *vfilename, 
char *filename,
     }
 
   instream2 = fopen(g_iobuffer1, "r");
-  if (instream2 == (FILE *) 0)
+  if (instream2 == NULL)
     {
       not_found2(directive, tag, g_iobuffer1);
       return;
@@ -461,7 +469,7 @@ static void do_include(FILE *instream, char *vfilename, 
char *filename,
 
   if (strcmp(tag, "virtual") == 0)
     {
-      if (strlen(val) <BUFFER_SIZE)
+      if (strlen(val) < BUFFER_SIZE)
         {
           strcpy(g_iobuffer2, val);
         }
@@ -482,6 +490,7 @@ static void do_include(FILE *instream, char *vfilename, 
char *filename,
               cp = &g_iobuffer2[strlen(g_iobuffer2)];
               *cp = '/';
             }
+
           strcpy(++cp, val);
         }
       else
@@ -500,8 +509,9 @@ static void do_echo(FILE *instream, char *vfilename, char 
*filename,
   char *cp;
 
   /* Prints the value of one of the include variables.  Any dates are
-   * printed subject to the currently configured g_timeformat.  The only valid
-   * tag is var, whose value is the name of the variable you wish to echo.
+   * printed subject to the currently configured g_timeformat.  The only
+   * valid tag is var, whose value is the name of the variable you wish
+   * to echo.
    */
 
   if (strcmp(tag, "var") != 0)
@@ -585,7 +595,8 @@ static void do_fsize(FILE *instream, char *vfilename, char 
*filename,
 
   /* Prints the size of the specified file. */
 
-  ret = get_filename(vfilename, filename, directive, tag, val, g_iobuffer1, 
BUFFER_SIZE);
+  ret = get_filename(vfilename, filename, directive, tag, val, g_iobuffer1,
+                     BUFFER_SIZE);
   if (ret < 0)
     {
       return;
@@ -607,7 +618,8 @@ static void do_flastmod(FILE *instream, char *vfilename, 
char *filename,
 
   /* Prints the last modification date of the specified file. */
 
-  ret = get_filename(vfilename, filename, directive, tag, val, g_iobuffer1, 
BUFFER_SIZE);
+  ret = get_filename(vfilename, filename, directive, tag, val, g_iobuffer1,
+                     BUFFER_SIZE);
   if (ret < 0)
     {
       return;
@@ -618,6 +630,7 @@ static void do_flastmod(FILE *instream, char *vfilename, 
char *filename,
       not_found2(directive, tag, g_iobuffer1);
       return;
     }
+
   show_time(g_sb.st_mtime, 0);
 }
 
@@ -635,7 +648,7 @@ static void parse(FILE *instream, char *vfilename, char 
*filename, char *str)
 
   ntags = 0;
   cp = directive;
-  for (;;)
+  for (; ; )
     {
       cp = strpbrk(cp, " \t\n\r\"");
       if (cp == (char *)0)
@@ -718,11 +731,13 @@ static void parse(FILE *instream, char *vfilename, char 
*filename, char *str)
       switch (dirn)
         {
         case DI_CONFIG:
-          do_config(instream, vfilename, filename, directive, g_tags[i], val);
+          do_config(instream, vfilename, filename, directive, g_tags[i],
+                    val);
           break;
 
         case DI_INCLUDE:
-          do_include(instream, vfilename, filename, directive, g_tags[i], val);
+          do_include(instream, vfilename, filename, directive, g_tags[i],
+                     val);
           break;
 
         case DI_ECHO:
@@ -734,7 +749,8 @@ static void parse(FILE *instream, char *vfilename, char 
*filename, char *str)
           break;
 
         case DI_FLASTMOD:
-          do_flastmod(instream, vfilename, filename, directive, g_tags[i], 
val);
+          do_flastmod(instream, vfilename, filename, directive, g_tags[i],
+                      val);
           break;
         }
     }
@@ -915,12 +931,13 @@ int main(int argc, char *argv[])
       path_info = "";
     }
 
-  g_url = (char*)malloc(strlen(script_name) + strlen(path_info) + 1);
+  g_url = (char *)malloc(strlen(script_name) + strlen(path_info) + 1);
   if (!g_url)
     {
       internal_error("Out of memory.");
       return 2;
     }
+
   sprintf(g_url, "%s%s", script_name, path_info);
 
   /* Get the name of the file to parse. */
diff --git a/netutils/thttpd/libhttpd.c b/netutils/thttpd/libhttpd.c
index 561256cad..eab4fdbd4 100644
--- a/netutils/thttpd/libhttpd.c
+++ b/netutils/thttpd/libhttpd.c
@@ -3123,21 +3123,21 @@ void httpd_destroy_conn(httpd_conn *hc)
 {
   if (hc->initialized)
     {
-      httpd_free((void *)hc->read_buf);
-      httpd_free((void *)hc->decodedurl);
-      httpd_free((void *)hc->origfilename);
-      httpd_free((void *)hc->expnfilename);
-      httpd_free((void *)hc->encodings);
-      httpd_free((void *)hc->pathinfo);
-      httpd_free((void *)hc->query);
-      httpd_free((void *)hc->accept);
-      httpd_free((void *)hc->accepte);
-      httpd_free((void *)hc->reqhost);
-      httpd_free((void *)hc->hostdir);
-      httpd_free((void *)hc->remoteuser);
-      httpd_free((void *)hc->buffer);
+      httpd_free(hc->read_buf);
+      httpd_free(hc->decodedurl);
+      httpd_free(hc->origfilename);
+      httpd_free(hc->expnfilename);
+      httpd_free(hc->encodings);
+      httpd_free(hc->pathinfo);
+      httpd_free(hc->query);
+      httpd_free(hc->accept);
+      httpd_free(hc->accepte);
+      httpd_free(hc->reqhost);
+      httpd_free(hc->hostdir);
+      httpd_free(hc->remoteuser);
+      httpd_free(hc->buffer);
 #ifdef CONFIG_THTTPD_TILDE_MAP2
-      httpd_free((void *)hc->altdir);
+      httpd_free(hc->altdir);
 #endif /* CONFIG_THTTPD_TILDE_MAP2 */
       hc->initialized = 0;
     }
diff --git a/netutils/thttpd/thttpd.c b/netutils/thttpd/thttpd.c
index bc07f0ca0..07cb4da5f 100644
--- a/netutils/thttpd/thttpd.c
+++ b/netutils/thttpd/thttpd.c
@@ -152,7 +152,7 @@ static void shut_down(void)
       if (connects[cnum].hc != NULL)
         {
           httpd_destroy_conn(connects[cnum].hc);
-          httpd_free((void *)connects[cnum].hc);
+          httpd_free(connects[cnum].hc);
           connects[cnum].hc = NULL;
         }
     }
@@ -170,7 +170,7 @@ static void shut_down(void)
     }
 
   tmr_destroy();
-  httpd_free((void *)connects);
+  httpd_free(connects);
 }
 
 static int handle_newconnect(FAR struct timeval *tv, int listen_fd)
@@ -638,7 +638,7 @@ static void linger_clear_connection(ClientData client_data,
   struct connect_s *conn;
 
   ninfo("Clear connection\n");
-  conn = (struct connect_s *) client_data.p;
+  conn = (struct connect_s *)client_data.p;
   conn->linger_timer = NULL;
   really_clear_connection(conn);
 }
diff --git a/netutils/usrsock_rpmsg/usrsock_rpmsg_client.c 
b/netutils/usrsock_rpmsg/usrsock_rpmsg_client.c
index 249f575b2..731cf976b 100644
--- a/netutils/usrsock_rpmsg/usrsock_rpmsg_client.c
+++ b/netutils/usrsock_rpmsg/usrsock_rpmsg_client.c
@@ -262,7 +262,7 @@ int main(int argc, char *argv[])
       while (1)
         {
           struct pollfd pfd;
-          void  *buf;
+          FAR void *buf;
           uint32_t len;
 
           /* Wait the packet ready */
diff --git a/netutils/usrsock_rpmsg/usrsock_rpmsg_server.c 
b/netutils/usrsock_rpmsg/usrsock_rpmsg_server.c
index dd37df9b0..4782636b0 100644
--- a/netutils/usrsock_rpmsg/usrsock_rpmsg_server.c
+++ b/netutils/usrsock_rpmsg/usrsock_rpmsg_server.c
@@ -442,7 +442,7 @@ static int usrsock_rpmsg_sendto_handler(struct 
rpmsg_endpoint *ept,
           else
             {
               ret = psock_sendto(&priv->socks[req->usockid],
-                  (const void *)(req + 1) + req->addrlen, req->buflen,
+                  (const char *)(req + 1) + req->addrlen, req->buflen,
                   req->flags,
                   req->addrlen ? (const struct sockaddr *)(req + 1) : NULL,
                   req->addrlen);
diff --git a/netutils/wakeonlan/wol_main.c b/netutils/wakeonlan/wol_main.c
index 668e10a96..e756e1780 100644
--- a/netutils/wakeonlan/wol_main.c
+++ b/netutils/wakeonlan/wol_main.c
@@ -66,7 +66,7 @@ static int send_wol(FAR struct ether_addr *dest,
     }
 
   ret = setsockopt(sockfd, SOL_SOCKET, SO_BROADCAST,
-    (void *)&optval, sizeof(optval));
+                   &optval, sizeof(optval));
   if (ret < 0)
     {
       fprintf(stderr, "failed to set socket options: %s\n", strerror(errno));
diff --git a/netutils/webclient/webclient.c b/netutils/webclient/webclient.c
index 95b72b469..a24c877cd 100644
--- a/netutils/webclient/webclient.c
+++ b/netutils/webclient/webclient.c
@@ -1420,8 +1420,7 @@ int webclient_perform(FAR struct webclient_context *ctx)
                   /* Check return value one by one */
 
                   ret = setsockopt(conn->sockfd, SOL_SOCKET, SO_RCVTIMEO,
-                                  (FAR const void *)&tv,
-                                   sizeof(struct timeval));
+                                   &tv, sizeof(struct timeval));
                   if (ret != 0)
                     {
                       ret = -errno;
@@ -1430,8 +1429,7 @@ int webclient_perform(FAR struct webclient_context *ctx)
                     }
 
                   ret = setsockopt(conn->sockfd, SOL_SOCKET, SO_SNDTIMEO,
-                                  (FAR const void *)&tv,
-                                  sizeof(struct timeval));
+                                   &tv, sizeof(struct timeval));
                   if (ret != 0)
                     {
                       ret = -errno;
diff --git a/netutils/xmlrpc/xmlparser.c b/netutils/xmlrpc/xmlparser.c
index 7405e8168..51e283cd6 100644
--- a/netutils/xmlrpc/xmlparser.c
+++ b/netutils/xmlrpc/xmlparser.c
@@ -310,7 +310,7 @@ static int xmlrpc_parsemethod(struct parsebuf_s *pbuf)
   int ret = XMLRPC_PARSE_ERROR;
   int type;
 
-  memset((void *)&g_xmlcall, 0, sizeof(struct xmlrpc_s));
+  memset(&g_xmlcall, 0, sizeof(struct xmlrpc_s));
 
   /* Look for the methodName tag */
 
diff --git a/nshlib/nsh_console.c b/nshlib/nsh_console.c
index d383c8e15..7fa918e3c 100644
--- a/nshlib/nsh_console.c
+++ b/nshlib/nsh_console.c
@@ -45,10 +45,10 @@
 #ifdef CONFIG_FILE_STREAM
 struct serialsave_s
 {
-  int    cn_errfd;     /* Re-directed error output file descriptor */
-  int    cn_outfd;     /* Re-directed output file descriptor */
-  FILE  *cn_errstream; /* Re-directed error output stream */
-  FILE  *cn_outstream; /* Re-directed output stream */
+  int   cn_errfd;     /* Re-directed error output file descriptor */
+  int   cn_outfd;     /* Re-directed output file descriptor */
+  FILE *cn_errstream; /* Re-directed error output stream */
+  FILE *cn_outstream; /* Re-directed output stream */
 };
 #endif
 
diff --git a/nshlib/nsh_console.h b/nshlib/nsh_console.h
index 895f98d51..219ff5832 100644
--- a/nshlib/nsh_console.h
+++ b/nshlib/nsh_console.h
@@ -150,15 +150,15 @@ struct console_stdio_s
 
 #ifdef CONFIG_FILE_STREAM
 #ifdef CONFIG_NSH_ALTCONDEV
-  int    cn_confd;     /* Console I/O file descriptor */
+  int   cn_confd;     /* Console I/O file descriptor */
 #endif
-  int    cn_outfd;     /* Output file descriptor (possibly redirected) */
-  int    cn_errfd;     /* Error Output file descriptor (possibly redirected) */
+  int   cn_outfd;     /* Output file descriptor (possibly redirected) */
+  int   cn_errfd;     /* Error Output file descriptor (possibly redirected) */
 #ifdef CONFIG_NSH_ALTCONDEV
-  FILE  *cn_constream; /* Console I/O stream (possibly redirected) */
+  FILE *cn_constream; /* Console I/O stream (possibly redirected) */
 #endif
-  FILE  *cn_outstream; /* Output stream */
-  FILE  *cn_errstream; /* Error Output stream */
+  FILE *cn_outstream; /* Output stream */
+  FILE *cn_errstream; /* Error Output stream */
 #endif
 
 #ifdef CONFIG_NSH_VARS
diff --git a/nshlib/nsh_romfsetc.c b/nshlib/nsh_romfsetc.c
index 166ca075f..90d4e7a18 100644
--- a/nshlib/nsh_romfsetc.c
+++ b/nshlib/nsh_romfsetc.c
@@ -78,7 +78,7 @@
 
 int nsh_romfsetc(void)
 {
-  int  ret;
+  int ret;
 
 #ifndef CONFIG_NSH_CROMFSETC
   struct boardioc_romdisk_s desc;
diff --git a/nshlib/nsh_telnetlogin.c b/nshlib/nsh_telnetlogin.c
index 4b4b0b496..fc032cec6 100644
--- a/nshlib/nsh_telnetlogin.c
+++ b/nshlib/nsh_telnetlogin.c
@@ -58,12 +58,12 @@
 static void nsh_telnetecho(FAR struct console_stdio_s *pstate,
                            uint8_t is_use)
 {
-  uint8_t optbuf[4];
+  char optbuf[4];
   optbuf[0] = TELNET_IAC;
   optbuf[1] = (is_use == TELNET_USE_ECHO) ? TELNET_WILL : TELNET_DO;
   optbuf[2] = 1;
   optbuf[3] = 0;
-  fputs((char *)optbuf, pstate->cn_outstream);
+  fputs(optbuf, pstate->cn_outstream);
   fflush(pstate->cn_outstream);
 }
 
diff --git a/nshlib/nsh_test.c b/nshlib/nsh_test.c
index d2f48db1d..b23b3d7cd 100644
--- a/nshlib/nsh_test.c
+++ b/nshlib/nsh_test.c
@@ -181,7 +181,7 @@ static inline int unaryexpression(FAR struct nsh_vtbl_s 
*vtbl,
 {
   struct stat buf;
   FAR char *fullpath;
-  int   ret;
+  int ret;
 
   /* -n STRING */
 
diff --git a/platform/mikroe-stm32f4/mikroe_configdata.c 
b/platform/mikroe-stm32f4/mikroe_configdata.c
index 77cec6525..be54fd0bf 100644
--- a/platform/mikroe-stm32f4/mikroe_configdata.c
+++ b/platform/mikroe-stm32f4/mikroe_configdata.c
@@ -94,10 +94,10 @@ int platform_setconfig(enum config_data_e id, int instance,
 
   config.id         = (enum config_data_e)id;
   config.instance   = instance;
-  config.configdata = (FAR uint8_t *) configdata;
+  config.configdata = (FAR uint8_t *)configdata;
   config.len        = datalen;
 
-  ret = ioctl(fd, CFGDIOC_SETCONFIG, (unsigned long) &config);
+  ret = ioctl(fd, CFGDIOC_SETCONFIG, (unsigned long)&config);
   close(fd);
   return ret;
 
@@ -200,9 +200,9 @@ int platform_getconfig(enum config_data_e id, int instance,
 #endif
 
 #ifdef CONFIG_MIKROE_STM32F4_CONFIGDATA_PART
-  struct config_data_s  config;
-  int                   ret;
-  int                   fd;
+  struct config_data_s config;
+  int                  ret;
+  int                  fd;
 
   /* Try to open the /dev/config device file */
 
@@ -220,7 +220,7 @@ int platform_getconfig(enum config_data_e id, int instance,
   config.configdata = configdata;
   config.len = datalen;
 
-  ret = ioctl(fd, CFGDIOC_GETCONFIG, (unsigned long) &config);
+  ret = ioctl(fd, CFGDIOC_GETCONFIG, (unsigned long)&config);
   close(fd);
   return ret;
 
diff --git a/system/cfgdata/cfgdata_main.c b/system/cfgdata/cfgdata_main.c
index 2c93a9bb2..962fce1ce 100644
--- a/system/cfgdata/cfgdata_main.c
+++ b/system/cfgdata/cfgdata_main.c
@@ -237,11 +237,11 @@ static void cfgdatacmd_parse_byte_array(struct 
config_data_s *cfg,
 
 static void cfgdatacmd_set(int argc, char *argv[])
 {
-  int                   ret;
-  int                   fd;
-  int                   x;
-  struct config_data_s  cfg;
-  uint8_t               data[32];
+  int                  ret;
+  int                  fd;
+  int                  x;
+  struct config_data_s cfg;
+  uint8_t              data[32];
 
 #ifdef CONFIG_MTD_CONFIG_NAMED
 
@@ -369,9 +369,9 @@ static void cfgdatacmd_set(int argc, char *argv[])
 
 static void cfgdatacmd_unset(int argc, char *argv[])
 {
-  int                   ret;
-  int                   fd;
-  struct config_data_s  cfg;
+  int                  ret;
+  int                  fd;
+  struct config_data_s cfg;
 
 #ifdef CONFIG_MTD_CONFIG_NAMED
   /* Copy the name to the cfg struct */
@@ -432,11 +432,11 @@ static void cfgdatacmd_unset(int argc, char *argv[])
 
 static void cfgdatacmd_print(int argc, char *argv[])
 {
-  int                   ret;
-  int                   fd;
-  int                   x;
-  struct config_data_s  cfg;
-  bool                  isstring;
+  int                  ret;
+  int                  fd;
+  int                  x;
+  struct config_data_s cfg;
+  bool                 isstring;
 
 #ifdef CONFIG_MTD_CONFIG_NAMED
 
@@ -541,12 +541,12 @@ static void cfgdatacmd_print(int argc, char *argv[])
 
 static void cfgdatacmd_show_all_config_items(void)
 {
-  int                   ret;
-  int                   fd;
-  int                   x;
-  struct config_data_s  cfg;
-  char                  fmtstr[24];
-  bool                  isstring;
+  int                  ret;
+  int                  fd;
+  int                  x;
+  struct config_data_s cfg;
+  char                 fmtstr[24];
+  bool                 isstring;
 
   /* Try to open the /dev/config file */
 
diff --git a/system/i2c/i2ctool.h b/system/i2c/i2ctool.h
index fdad24a70..e1596457a 100644
--- a/system/i2c/i2ctool.h
+++ b/system/i2c/i2ctool.h
@@ -134,8 +134,8 @@ struct i2ctool_s
   /* Output streams */
 
 #ifdef CONFIG_I2CTOOL_OUTDEV
-  int    ss_outfd;     /* Output file descriptor */
-  FILE  *ss_outstream; /* Output stream */
+  int   ss_outfd;     /* Output file descriptor */
+  FILE *ss_outstream; /* Output stream */
 #endif
 };
 
diff --git a/system/nxlooper/nxlooper.c b/system/nxlooper/nxlooper.c
index 86ede96ff..5efb39d7f 100644
--- a/system/nxlooper/nxlooper.c
+++ b/system/nxlooper/nxlooper.c
@@ -759,7 +759,7 @@ int nxlooper_setvolume(FAR struct nxlooper_s *plooper, 
uint16_t volume)
 #ifndef CONFIG_AUDIO_EXCLUDE_PAUSE_RESUME
 int nxlooper_pause(FAR struct nxlooper_s *plooper)
 {
-  int   ret = OK;
+  int ret = OK;
 
   if (plooper->loopstate == NXLOOPER_STATE_LOOPING)
     {
diff --git a/system/nxplayer/nxplayer.c b/system/nxplayer/nxplayer.c
index 7c09fa7dd..64665db31 100644
--- a/system/nxplayer/nxplayer.c
+++ b/system/nxplayer/nxplayer.c
@@ -74,9 +74,9 @@
 #ifdef CONFIG_NXPLAYER_FMT_FROM_EXT
 struct nxplayer_ext_fmt_s
 {
-  const char  *ext;
-  uint16_t    format;
-  CODE int    (*getsubformat)(int fd);
+  FAR const char *ext;
+  uint16_t       format;
+  CODE int       (*getsubformat)(int fd);
 };
 #endif
 
@@ -179,10 +179,8 @@ static int _open_with_http(const char *fullurl)
   tv.tv_sec  = 10; /* TODO */
   tv.tv_usec = 0;
 
-  setsockopt(s, SOL_SOCKET, SO_RCVTIMEO, (FAR const void *)&tv,
-             sizeof(struct timeval));
-  setsockopt(s, SOL_SOCKET, SO_SNDTIMEO, (FAR const void *)&tv,
-             sizeof(struct timeval));
+  setsockopt(s, SOL_SOCKET, SO_RCVTIMEO, &tv, sizeof(struct timeval));
+  setsockopt(s, SOL_SOCKET, SO_SNDTIMEO, &tv, sizeof(struct timeval));
 
   struct sockaddr_in server;
   server.sin_family = AF_INET;
@@ -472,8 +470,8 @@ static int nxplayer_opendevice(FAR struct nxplayer_s 
*pplayer, int format,
 #ifdef CONFIG_AUDIO_FORMAT_MIDI
 int nxplayer_getmidisubformat(int fd)
 {
-  char    type[2];
-  int     ret;
+  char type[2];
+  int  ret;
 
   /* Seek to location 8 in the file (the format type) */
 
@@ -516,9 +514,9 @@ static inline int nxplayer_fmtfromextension(FAR struct 
nxplayer_s *pplayer,
                                             FAR const char *pfilename,
                                             FAR int *subfmt)
 {
-  const char  *pext;
-  uint8_t      x;
-  uint8_t      c;
+  FAR const char *pext;
+  uint8_t         x;
+  uint8_t         c;
 
   /* Find the file extension, if any */
 
@@ -750,23 +748,23 @@ static int nxplayer_enqueuebuffer(FAR struct nxplayer_s 
*pplayer,
  *
  ****************************************************************************/
 
-static void *nxplayer_playthread(pthread_addr_t pvarg)
+static FAR void *nxplayer_playthread(pthread_addr_t pvarg)
 {
-  struct nxplayer_s           *pplayer = (struct nxplayer_s *)pvarg;
-  struct audio_msg_s          msg;
-  struct audio_buf_desc_s     buf_desc;
-  ssize_t                     size;
-  bool                        running = true;
-  bool                        streaming = true;
-  bool                        failed = false;
-  struct ap_buffer_info_s     buf_info;
-  FAR struct ap_buffer_s      **buffers;
-  unsigned int                prio;
+  FAR struct nxplayer_s   *pplayer = (FAR struct nxplayer_s *)pvarg;
+  struct audio_msg_s      msg;
+  struct audio_buf_desc_s buf_desc;
+  ssize_t                 size;
+  bool                    running = true;
+  bool                    streaming = true;
+  bool                    failed = false;
+  struct ap_buffer_info_s buf_info;
+  FAR struct ap_buffer_s  **buffers;
+  unsigned int            prio;
 #ifdef CONFIG_DEBUG_FEATURES
-  int                         outstanding = 0;
+  int                     outstanding = 0;
 #endif
-  int                         x;
-  int                         ret;
+  int                     x;
+  int                     ret;
 
   audinfo("Entry\n");
 
@@ -1419,7 +1417,7 @@ int nxplayer_setbalance(FAR struct nxplayer_s *pplayer, 
uint16_t balance)
 #ifndef CONFIG_AUDIO_EXCLUDE_PAUSE_RESUME
 int nxplayer_pause(FAR struct nxplayer_s *pplayer)
 {
-  int   ret = OK;
+  int ret = OK;
 
   if (pplayer->state == NXPLAYER_STATE_PLAYING)
     {
@@ -1705,8 +1703,8 @@ int nxplayer_setdevice(FAR struct nxplayer_s *pplayer,
 #ifndef CONFIG_AUDIO_EXCLUDE_STOP
 int nxplayer_stop(FAR struct nxplayer_s *pplayer)
 {
-  struct audio_msg_s  term_msg;
-  FAR void            *value;
+  struct audio_msg_s term_msg;
+  FAR void           *value;
 
   DEBUGASSERT(pplayer != NULL);
 
@@ -2111,7 +2109,7 @@ FAR struct nxplayer_s *nxplayer_create(void)
 
   /* Allocate the memory */
 
-  pplayer = (FAR struct nxplayer_s *) malloc(sizeof(struct nxplayer_s));
+  pplayer = (FAR struct nxplayer_s *)malloc(sizeof(struct nxplayer_s));
   if (pplayer == NULL)
     {
       return NULL;
diff --git a/system/nxplayer/nxplayer_main.c b/system/nxplayer/nxplayer_main.c
index f7d9145f3..f28cfe239 100644
--- a/system/nxplayer/nxplayer_main.c
+++ b/system/nxplayer/nxplayer_main.c
@@ -256,7 +256,7 @@ static const int g_nxplayer_cmd_count = 
sizeof(g_nxplayer_cmds) /
 
 static int nxplayer_cmd_play(FAR struct nxplayer_s *pplayer, char *parg)
 {
-  int     ret;
+  int ret;
 
   /* Try to play the file specified */
 
diff --git a/system/nxrecorder/nxrecorder.c b/system/nxrecorder/nxrecorder.c
index b1527b82b..7f4054537 100644
--- a/system/nxrecorder/nxrecorder.c
+++ b/system/nxrecorder/nxrecorder.c
@@ -223,23 +223,23 @@ static int nxrecorder_enqueuebuffer(FAR struct 
nxrecorder_s *precorder,
  *
  ****************************************************************************/
 
-static void *nxrecorder_recordthread(pthread_addr_t pvarg)
+static FAR void *nxrecorder_recordthread(pthread_addr_t pvarg)
 {
-  struct nxrecorder_s         *precorder = (struct nxrecorder_s *)pvarg;
-  struct audio_msg_s          msg;
-  struct audio_buf_desc_s     buf_desc;
-  ssize_t                     size;
-  bool                        running = true;
-  bool                        streaming = true;
-  bool                        failed = false;
-  struct ap_buffer_info_s     buf_info;
-  FAR struct ap_buffer_s      **pbuffers;
-  unsigned int                prio;
+  FAR struct nxrecorder_s *precorder = (FAR struct nxrecorder_s *)pvarg;
+  struct audio_msg_s      msg;
+  struct audio_buf_desc_s buf_desc;
+  ssize_t                 size;
+  bool                    running = true;
+  bool                    streaming = true;
+  bool                    failed = false;
+  struct ap_buffer_info_s buf_info;
+  FAR struct ap_buffer_s  **pbuffers;
+  unsigned int            prio;
 #ifdef CONFIG_DEBUG_FEATURES
-  int                         outstanding = 0;
+  int                     outstanding = 0;
 #endif
-  int                         x;
-  int                         ret;
+  int                     x;
+  int                     ret;
 
   audinfo("Entry\n");
 
@@ -711,8 +711,8 @@ int nxrecorder_setdevice(FAR struct nxrecorder_s *precorder,
 #ifndef CONFIG_AUDIO_EXCLUDE_STOP
 int nxrecorder_stop(FAR struct nxrecorder_s *precorder)
 {
-  struct audio_msg_s  term_msg;
-  FAR void            *value;
+  struct audio_msg_s term_msg;
+  FAR void           *value;
 
   DEBUGASSERT(precorder != NULL);
 
@@ -956,8 +956,7 @@ FAR struct nxrecorder_s *nxrecorder_create(void)
 
   /* Allocate the memory */
 
-  precorder = (FAR struct nxrecorder_s *) malloc(
-                                           sizeof(struct nxrecorder_s));
+  precorder = (FAR struct nxrecorder_s *)malloc(sizeof(struct nxrecorder_s));
   if (precorder == NULL)
     {
       return NULL;
diff --git a/system/popen/popen.c b/system/popen/popen.c
index f02ca3b81..338ce5dcb 100644
--- a/system/popen/popen.c
+++ b/system/popen/popen.c
@@ -68,7 +68,7 @@ struct popen_file_s
  *   were created within the popen() call using the fork() function, and the
  *   child invoked the sh utility using the call:
  *
- *     execl(shell path, "sh", "-c", command, (char *)0);
+ *     execl(shell path, "sh", "-c", command, NULL);
  *
  *   where shell path is an unspecified pathname for the sh utility.
  *
diff --git a/system/spi/spi_exch.c b/system/spi/spi_exch.c
index 9b3772b93..768fccfca 100644
--- a/system/spi/spi_exch.c
+++ b/system/spi/spi_exch.c
@@ -41,7 +41,7 @@
  ****************************************************************************/
 
 #define ISHEX(x) ((((x)>='0') && ((x)<='9')) || ((toupper(x)>='A') && 
(toupper(x)<='F')))
-#define HTOI(x) ( (((x)>='0') && ((x)<='9')) ? (x)-'0':toupper(x)-'A'+10 )
+#define HTOI(x)  ((((x)>='0') && ((x)<='9')) ? (x)-'0':toupper(x)-'A'+10)
 
 int spicmd_exch(FAR struct spitool_s *spitool, int argc, FAR char **argv)
 {
diff --git a/system/spi/spi_main.c b/system/spi/spi_main.c
index 68a6461c3..63aa45d62 100644
--- a/system/spi/spi_main.c
+++ b/system/spi/spi_main.c
@@ -186,8 +186,8 @@ static int spi_execute(FAR struct spitool_s *spitool, int 
argc,
 {
   FAR const struct cmdmap_s *cmdmap;
   FAR const char            *cmd;
-  cmd_t                  handler;
-  int                    ret;
+  cmd_t                      handler;
+  int                        ret;
 
   /* The form of argv is:
    *
diff --git a/system/spi/spitool.h b/system/spi/spitool.h
index b1cb66b43..7af9f7953 100644
--- a/system/spi/spitool.h
+++ b/system/spi/spitool.h
@@ -138,8 +138,8 @@ struct spitool_s
   /* Output streams */
 
 #ifdef CONFIG_SPITOOL_OUTDEV
-  int    ss_outfd;     /* Output file descriptor */
-  FILE  *ss_outstream; /* Output stream */
+  int   ss_outfd;     /* Output file descriptor */
+  FILE *ss_outstream; /* Output stream */
 #endif
 };
 
diff --git a/system/termcurses/tcurses_vt100.c 
b/system/termcurses/tcurses_vt100.c
index 3c7b49115..3ee01d4c7 100644
--- a/system/termcurses/tcurses_vt100.c
+++ b/system/termcurses/tcurses_vt100.c
@@ -736,7 +736,7 @@ static int tcurses_vt100_clear(FAR struct termcurses_s 
*dev, int type)
   int ret = -ENOSYS;
   int fd;
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd = priv->out_fd;
 
   /* Perform operation based on type */
@@ -908,7 +908,7 @@ static int tcurses_vt100_setcolors(FAR struct termcurses_s 
*dev,
   int  fd;
   char str[48];
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd   = priv->out_fd;
 
   /* Test if FG color to be set */
@@ -961,7 +961,7 @@ static int tcurses_vt100_getwinsize(FAR struct termcurses_s 
*dev,
   int  delay;
   int  len;
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd   = priv->out_fd;
 
   /* First try the TIOCGWINSZ ioctl */
@@ -1055,7 +1055,7 @@ static int tcurses_vt100_setattributes(FAR struct 
termcurses_s *dev,
   int fd;
   char str[48];
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd   = priv->out_fd;
 
   /* Test for cursor hide */
@@ -1127,22 +1127,22 @@ static int tcurses_vt100_getkeycode(FAR struct 
termcurses_s *dev,
                                     FAR int *keymodifiers)
 {
   FAR struct tcurses_vt100_s  *priv;
-  const struct keycodes_s     *pkeycodes;
-  int                         ret;
-  int                         fd;
-  bool                        esc_seq = false;
-  bool                        ctrl_seq = false;
-  bool                        ismodifier;
-  int                         keycode;
-  int                         k;
-  int                         x;
-  int                         start;
-  fd_set                      rfds;
-  struct                      timeval tv;
-  char                        ch;
-  char                        buildkey[16];
-  int                         keybuildcount;
-  static const char modtable[7] =
+  FAR const struct keycodes_s *pkeycodes;
+  int                          ret;
+  int                          fd;
+  bool                         esc_seq = false;
+  bool                         ctrl_seq = false;
+  bool                         ismodifier;
+  int                          keycode;
+  int                          k;
+  int                          x;
+  int                          start;
+  fd_set                       rfds;
+  struct timeval               tv;
+  char                         ch;
+  char                         buildkey[16];
+  int                          keybuildcount;
+  static const char            modtable[7] =
   {
     PDC_KEY_MODIFIER_SHIFT,
     PDC_KEY_MODIFIER_ALT,
@@ -1153,7 +1153,7 @@ static int tcurses_vt100_getkeycode(FAR struct 
termcurses_s *dev,
     PDC_KEY_MODIFIER_ALT | PDC_KEY_MODIFIER_CONTROL | PDC_KEY_MODIFIER_SHIFT
   };
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd   = priv->in_fd;
 
   /* Watch stdin (fd 0) to see when it has input. */
@@ -1414,13 +1414,13 @@ static int tcurses_vt100_getkeycode(FAR struct 
termcurses_s *dev,
 
 static bool tcurses_vt100_checkkey(FAR struct termcurses_s *dev)
 {
-  FAR struct tcurses_vt100_s  *priv;
+  FAR struct tcurses_vt100_s *priv;
   int                         ret;
   int                         fd;
   fd_set                      rfds;
   struct                      timeval tv;
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd   = priv->in_fd;
 
   /* Test for queued characters */
@@ -1483,7 +1483,7 @@ FAR struct termcurses_s *tcurses_vt100_initialize(int 
in_fd, int out_fd)
   priv->out_fd   = out_fd;
   priv->keycount = 0;
 
-  return (FAR struct termcurses_s *) priv;
+  return (FAR struct termcurses_s *)priv;
 }
 
 
/************************************************************************************
@@ -1499,7 +1499,7 @@ static int tcurses_vt100_terminate(FAR struct 
termcurses_s *dev)
   FAR struct tcurses_vt100_s *priv;
   int  fd;
 
-  priv = (FAR struct tcurses_vt100_s *) dev;
+  priv = (FAR struct tcurses_vt100_s *)dev;
   fd   = priv->out_fd;
 
   /* Set default foreground and background colors.
diff --git a/system/termcurses/termcurses.c b/system/termcurses/termcurses.c
index 0b7b7692c..c9d49be4e 100644
--- a/system/termcurses/termcurses.c
+++ b/system/termcurses/termcurses.c
@@ -78,7 +78,7 @@ int termcurses_initterm(FAR const char *term_type, int in_fd, 
int out_fd,
         {
           /* Default to vt100 as a last resort */
 
-          term_type = (FAR const char *) "vt100";
+          term_type = "vt100";
         }
     }
 
@@ -126,7 +126,7 @@ int termcurses_initterm(FAR const char *term_type, int 
in_fd, int out_fd,
 
 int termcurses_deinitterm(FAR struct termcurses_s *term)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
   int result = OK;
 
   /* Call the dev function */
@@ -154,7 +154,7 @@ int termcurses_deinitterm(FAR struct termcurses_s *term)
 
 int termcurses_moveyx(FAR struct termcurses_s *term, int row, int col)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
 
   /* Call the dev function */
 
@@ -177,7 +177,7 @@ int termcurses_moveyx(FAR struct termcurses_s *term, int 
row, int col)
 int termcurses_setcolors(FAR struct termcurses_s *term,
                          FAR struct termcurses_colors_s *colors)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
 
   /* Call the dev function */
 
@@ -199,7 +199,7 @@ int termcurses_setcolors(FAR struct termcurses_s *term,
 
 int termcurses_setattribute(FAR struct termcurses_s *term, unsigned long 
attrib)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
 
   /* Call the dev function */
 
@@ -221,7 +221,7 @@ int termcurses_setattribute(FAR struct termcurses_s *term, 
unsigned long attrib)
 
 int termcurses_getwinsize(FAR struct termcurses_s *term, FAR struct winsize 
*winsz)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
 
   /* Call the dev function */
 
@@ -244,7 +244,7 @@ int termcurses_getwinsize(FAR struct termcurses_s *term, 
FAR struct winsize *win
 int termcurses_getkeycode(FAR struct termcurses_s *term, FAR int *specialkey,
       int *keymodifiers)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
 
   /* Call the dev function */
 
@@ -266,7 +266,7 @@ int termcurses_getkeycode(FAR struct termcurses_s *term, 
FAR int *specialkey,
 
 bool termcurses_checkkey(FAR struct termcurses_s *term)
 {
-  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *) term;
+  FAR struct termcurses_dev_s *dev = (FAR struct termcurses_dev_s *)term;
 
   /* Call the dev function */
 
diff --git a/system/zmodem/host/crc16.c b/system/zmodem/host/crc16.c
index f37609996..0f353001c 100644
--- a/system/zmodem/host/crc16.c
+++ b/system/zmodem/host/crc16.c
@@ -99,11 +99,12 @@ static uint16_t crc16_tab[256] =
   0x6e17,  0x7e36,  0x4e55,  0x5e74,  0x2e93,  0x3eb2,  0x0ed1,  0x1ef0
 };
 
-#define updcrc(cp, crc) ( crc16_tab[((crc >> 8) & 255)] ^ (crc << 8) ^ cp)
+#define updcrc(cp, crc) (crc16_tab[((crc >> 8) & 255)] ^ (crc << 8) ^ cp)
 
 
/************************************************************************************************
  * Public Functions
  
************************************************************************************************/
+
 
/************************************************************************************************
  * Name: crc16part
  *
diff --git a/system/zmodem/zm_receive.c b/system/zmodem/zm_receive.c
index b8ab47bf3..47568aaf7 100644
--- a/system/zmodem/zm_receive.c
+++ b/system/zmodem/zm_receive.c
@@ -422,7 +422,7 @@ static int zmr_zsrintdata(FAR struct zm_state_s *pzm)
   pzmr->attn = NULL;
   if (pzm->pktbuf[0] != '\0')
     {
-      pzmr->attn = strdup((char *)pzm->pktbuf);
+      pzmr->attn = strdup((FAR char *)pzm->pktbuf);
     }
 
   /* And send ZACK */
@@ -1057,7 +1057,7 @@ static int zmr_zstderr(FAR struct zm_state_s *pzm)
   zmdbg("ZMR_STATE %d\n", pzm->state);
 
   pzm->pktbuf[pzm->pktlen] = '\0';
-  fprintf(stderr, "Message: %s", (char *)pzm->pktbuf);
+  fprintf(stderr, "Message: %s", (FAR char *)pzm->pktbuf);
   return OK;
 }
 
diff --git a/system/zmodem/zm_send.c b/system/zmodem/zm_send.c
index 32555c539..02e6eeee6 100644
--- a/system/zmodem/zm_send.c
+++ b/system/zmodem/zm_send.c
@@ -777,7 +777,7 @@ static int zms_sendfilename(FAR struct zm_state_s *pzm)
           (unsigned long)pzms->filesize);
 #endif
 
-  ptr += strlen((char *)ptr);
+  ptr += strlen((FAR char *)ptr);
   *ptr++ = '\0';
 
   len =  ptr - pzm->scratch;
@@ -1596,7 +1596,7 @@ ZMSHANDLE zms_initialize(int remfd)
        * or command if it were not already active.
        */
 
-      nwritten = zm_remwrite(pzm->remfd, (uint8_t *) "rz\r", 3);
+      nwritten = zm_remwrite(pzm->remfd, (FAR uint8_t *)"rz\r", 3);
       if (nwritten < 0)
         {
           zmdbg("ERROR: zm_remwrite failed: %d\n", (int)nwritten);
diff --git a/testing/ostest/pthread_cleanup.c b/testing/ostest/pthread_cleanup.c
index ccb8fe957..61b19743b 100644
--- a/testing/ostest/pthread_cleanup.c
+++ b/testing/ostest/pthread_cleanup.c
@@ -40,9 +40,9 @@ struct sync_s
  * Private Functions
  ****************************************************************************/
 
-static void cleanup(FAR void * data)
+static void cleanup(FAR void *data)
 {
-  FAR struct sync_s *sync = (FAR struct sync_s *) data;
+  FAR struct sync_s *sync = (FAR struct sync_s *)data;
   int status;
 
   /* Note:  The behavior of canceling pthread_cond_wait() with asynchronous
@@ -69,9 +69,9 @@ static void cleanup(FAR void * data)
     }
 }
 
-static void *cleanup_thread(FAR void * data)
+static void *cleanup_thread(FAR void *data)
 {
-  FAR struct sync_s *sync = (FAR struct sync_s *) data;
+  FAR struct sync_s *sync = (FAR struct sync_s *)data;
   int status;
 
   status = pthread_mutex_lock(&sync->lock);
diff --git a/testing/ostest/pthread_rwlock.c b/testing/ostest/pthread_rwlock.c
index 326fa3cd8..89de53a31 100644
--- a/testing/ostest/pthread_rwlock.c
+++ b/testing/ostest/pthread_rwlock.c
@@ -49,7 +49,7 @@ static int g_race_cond_thread_pos;
 
 static FAR void *race_cond_thread1(FAR void *data)
 {
-  FAR struct race_cond_s *rc = (FAR struct race_cond_s *) data;
+  FAR struct race_cond_s *rc = (FAR struct race_cond_s *)data;
   int status;
 
   /* Runs 1st */
@@ -147,7 +147,7 @@ static FAR void *race_cond_thread1(FAR void *data)
 
 static FAR void *race_cond_thread2(FAR void *data)
 {
-  FAR struct race_cond_s *rc = (FAR struct race_cond_s *) data;
+  FAR struct race_cond_s *rc = (FAR struct race_cond_s *)data;
   int status;
 
   status = sem_wait(rc->sem2);
@@ -294,7 +294,7 @@ static void test_two_threads(void)
   pthread_join(thread2, NULL);
 }
 
-static void * timeout_thread1(FAR void * data)
+static FAR void *timeout_thread1(FAR void *data)
 {
   FAR struct race_cond_s *rc = (FAR struct race_cond_s *)data;
   int status;
@@ -320,7 +320,7 @@ static void * timeout_thread1(FAR void * data)
   return NULL;
 }
 
-static void * timeout_thread2(FAR void * data)
+static FAR void *timeout_thread2(FAR void *data)
 {
   FAR struct race_cond_s *rc = (FAR struct race_cond_s *)data;
   struct timespec time;
diff --git a/testing/ostest/pthread_rwlock_cancel.c 
b/testing/ostest/pthread_rwlock_cancel.c
index b087e47e3..c3ff042da 100644
--- a/testing/ostest/pthread_rwlock_cancel.c
+++ b/testing/ostest/pthread_rwlock_cancel.c
@@ -40,7 +40,7 @@ struct sync_s
  * Private Functions
  ****************************************************************************/
 
-static void * timeout_thread1(FAR void * data)
+static FAR void *timeout_thread1(FAR void *data)
 {
   FAR struct sync_s *sync = (FAR struct sync_s *)data;
   struct timespec time;
@@ -63,7 +63,7 @@ static void * timeout_thread1(FAR void * data)
   return NULL;
 }
 
-static void * timeout_thread2(FAR void * data)
+static FAR void *timeout_thread2(FAR void *data)
 {
   FAR struct sync_s *sync = (FAR struct sync_s *)data;
   struct timespec time;
diff --git a/testing/ostest/timedmqueue.c b/testing/ostest/timedmqueue.c
index 15a421ef5..7c9289407 100644
--- a/testing/ostest/timedmqueue.c
+++ b/testing/ostest/timedmqueue.c
@@ -358,7 +358,7 @@ void timedmqueue_test(void)
 
   printf("timedmqueue_test: Waiting for sender to complete\n");
   pthread_join(sender, &result);
-  if (result != (void *)0)
+  if (result != NULL)
     {
       printf("timedmqueue_test: ERROR sender thread exited with %d errors\n",
              (int)((intptr_t)result));
@@ -396,7 +396,7 @@ void timedmqueue_test(void)
 
   printf("timedmqueue_test: Waiting for receiver to complete\n");
   pthread_join(receiver, &result);
-  if (result != (void *)0)
+  if (result != NULL)
     {
       printf("timedmqueue_test: ERROR receiver thread exited "
              "with %d errors\n", (int)((intptr_t)result));
diff --git a/testing/smart_test/smart_test.c b/testing/smart_test/smart_test.c
index da7d0a25f..38d63d66f 100644
--- a/testing/smart_test/smart_test.c
+++ b/testing/smart_test/smart_test.c
@@ -117,12 +117,12 @@ static int smart_create_test_file(char *filename)
 
 static int smart_seek_test(char *filename)
 {
-  FILE     *fd;
-  char      readstring[80];
-  char      cmpstring[80];
-  int       index;
-  int       x;
-  int       ret = OK;
+  FILE *fd;
+  char  readstring[80];
+  char  cmpstring[80];
+  int   index;
+  int   x;
+  int   ret = OK;
 
   fd = fopen(filename, "r");
   if (fd == NULL)
diff --git a/tools/check-hash.sh b/tools/check-hash.sh
index df5b2e5aa..cdf56bb05 100755
--- a/tools/check-hash.sh
+++ b/tools/check-hash.sh
@@ -59,11 +59,11 @@ esac
 if [ `which ${hash_algo}sum 2> /dev/null` ]; then
     hash_algo_cmd="${hash_algo}sum"
 elif [ `which shasum 2> /dev/null` ]; then
-    hash_algo_len=$( echo ${hash_algo} | cut -c 4- )
+    hash_algo_len=$(echo ${hash_algo} | cut -c 4-)
     hash_algo_cmd="shasum -a ${hash_algo_len}"
 fi
 
-calc_hash=$( ${hash_algo_cmd} "${file_to_check}" | cut -d' ' -f1 )
+calc_hash=$(${hash_algo_cmd} "${file_to_check}" | cut -d' ' -f1)
 
 # Does it match expected hash?
 
diff --git a/wireless/bluetooth/btsak/Makefile 
b/wireless/bluetooth/btsak/Makefile
index 9edd1f4c0..cd483f8f5 100644
--- a/wireless/bluetooth/btsak/Makefile
+++ b/wireless/bluetooth/btsak/Makefile
@@ -20,7 +20,7 @@
 
 include $(APPDIR)/Make.defs
 
-# Bluetooth SAK (swiss army knife )
+# Bluetooth SAK (swiss army knife)
 
 CSRCS   = btsak_advertise.c btsak_features.c btsak_gatt.c btsak_info.c
 CSRCS  += btsak_scan.c btsak_security.c
diff --git a/wireless/ieee802154/i8sak/Makefile 
b/wireless/ieee802154/i8sak/Makefile
index 5238b4169..b120e2154 100644
--- a/wireless/ieee802154/i8sak/Makefile
+++ b/wireless/ieee802154/i8sak/Makefile
@@ -20,7 +20,7 @@
 
 include $(APPDIR)/Make.defs
 
-# IEEE 802.15.4  SAK (swiss army knife )
+# IEEE 802.15.4  SAK (swiss army knife)
 
 PROGNAME = $(CONFIG_IEEE802154_I8SAK_PROGNAME)
 PRIORITY = $(CONFIG_IEEE802154_I8SAK_PRIORITY)
diff --git a/wireless/ieee802154/i8sak/i8sak_events.c 
b/wireless/ieee802154/i8sak/i8sak_events.c
index 7b9ad57e6..25b42cf73 100644
--- a/wireless/ieee802154/i8sak/i8sak_events.c
+++ b/wireless/ieee802154/i8sak/i8sak_events.c
@@ -222,8 +222,7 @@ int i8sak_eventlistener_start(FAR struct i8sak_s *i8sak)
   i8sak->eventlistener_run = true;
 
   ret = pthread_create(&i8sak->eventlistener_threadid,
-                       NULL, i8sak_eventthread,
-                       (void *)i8sak);
+                       NULL, i8sak_eventthread, i8sak);
   if (ret != 0)
     {
       fprintf(stderr, "failed to start event thread: %d\n", ret);
diff --git a/wireless/ieee802154/i8sak/i8sak_main.c 
b/wireless/ieee802154/i8sak/i8sak_main.c
index ed255910e..7c1841f9c 100644
--- a/wireless/ieee802154/i8sak/i8sak_main.c
+++ b/wireless/ieee802154/i8sak/i8sak_main.c
@@ -765,7 +765,7 @@ static int i8sak_daemon(int argc, FAR char *argv[])
           i8sak->startblaster = false;
 
           ret = pthread_create(&i8sak->blaster_threadid, NULL,
-                               i8sak_blaster_thread, (void *)i8sak);
+                               i8sak_blaster_thread, i8sak);
           if (ret != 0)
             {
               fprintf(stderr, "failed to start blaster thread: %d\n", ret);
@@ -779,7 +779,7 @@ static int i8sak_daemon(int argc, FAR char *argv[])
           i8sak->startsniffer = false;
 
           ret = pthread_create(&i8sak->sniffer_threadid, NULL,
-                               i8sak_sniffer_thread, (void *)i8sak);
+                               i8sak_sniffer_thread, i8sak);
           if (ret != 0)
             {
               fprintf(stderr, "failed to start sniffer thread: %d\n", ret);
diff --git a/wireless/ieee802154/libmac/sixlowpan_getreq.c 
b/wireless/ieee802154/libmac/sixlowpan_getreq.c
index 071e72b8d..9f07f5ca1 100644
--- a/wireless/ieee802154/libmac/sixlowpan_getreq.c
+++ b/wireless/ieee802154/libmac/sixlowpan_getreq.c
@@ -40,7 +40,7 @@
  ****************************************************************************/
 
 int sixlowpan_get_req(int sock, FAR const char *ifname,
-                      FAR const struct ieee802154_get_req_s *req)
+                      FAR struct ieee802154_get_req_s *req)
 {
   struct ieee802154_netmac_s arg;
   int ret;
@@ -61,7 +61,7 @@ int sixlowpan_get_req(int sock, FAR const char *ifname,
       fprintf(stderr, "MAC802154IOC_MLME_GET_REQUEST failed: %d\n", ret);
     }
 
-  memcpy((void *)req, &arg.u.getreq, sizeof(struct ieee802154_get_req_s));
+  memcpy(req, &arg.u.getreq, sizeof(struct ieee802154_get_req_s));
 
   return ret;
 }

Reply via email to