Adds -fno-strict-aliasing to ARCHOPTIMIZAITON definition for all sim Make.defs file. This is constistent with the setting used in the Make.defs file of all other architectures and should eliminate the build test warnings:
arp/arp_arpin.c: In function 'arp_arpin':
arp/arp_arpin.c:136:13: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
net_ipv4addr_hdrcopy(arp->ah_sipaddr, &dev->d_ipaddr);
^~~~~~~~~~~~~~~~~~~~
arp/arp_format.c: In function 'arp_format':
arp/arp_format.c:96:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
net_ipv4addr_hdrcopy(arp->ah_dipaddr, &ipaddr);
^~~~~~~~~~~~~~~~~~~~
And similar warnings from icmp/icmp_input.c, and tcp/tcp_send.c. -fno-strict-aliasing i required for a warning free build of NuttX code since there is use of punned pointers.
Eliminated unused function tun_ipv6multicast(). This eliminates a warning from the build test:
net/tun.c:1061:13: warning: 'tun_ipv6multicast' defined but not used [-Wunused-function]
static void tun_ipv6multicast(FAR struct tun_device_s *priv)
^~~~~~~~~~~~~~~~~
Author: Alan Carvalho de Assis <acassis@gmail.com>
Fix all nxstyle reported issues
Author: Robin Raymond <robin@opticaltone.com>
Fixed compilation issue with poll fds notification.
https://github.com/apache/incubator-nuttx/issues/483
Author: Xiang Xiao <xiaoxiang@xiaomi.com>
Fix the nxstyle Left/Right brackets in a new lined
Author: Alan Carvalho de Assis <acassis@gmail.com>
Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
arch/z80/src/ez80/ez80_timerisr.c: Fix an error that looks the file got damaged by a bad keystroke in the editor while a string was highlighted.
boards/z80/ez80/z20x/src/ez80_spi.c: Fix some bad conditional compilationi. Prevented W25 chip select from being initialized.
This reverts commit f735584514.
These header changes introduce unacceptable errors:
1. The changes alter the width of the initial block comment. That will cause nxstyle failures on most of the files.
2. The third line of the header is an (optional) short description of content of the the file. This change erroneously removes that line.
Automated header file changes can screw up a lot of files, very quickly.
Commit 797e45d6bf attempted to reduce the frequency of SPI reconfiguration by keeping track of the current frequency and mode settings and changing these settings only when the new settings differ from the old settings.
However, the original commit neglected to save the new frequency setting and, hence, was not effective in what it was trying to do: Every time the SPI setfrequency() method was called, it still looked like a new frequency and the frequency was still reconfigured, even though it hadn't really changed.
