Merge pull request #116 from olerem/warnings-2016.11.26

Warnings 2016.11.26

af_ath: don't use trash to configure interrupts

The WMI_ENABLE_INTR_CMDID can some time send empty package
to set interrupt to default value.
The problem is that ath_enable_intr_tgt is testing the
data pointer instead of dataleng variable.
We have never data == NULL case, even on zero size package.
(which is probably a bug too)

As result, FW was using some part of memory with undefined value
to configure interrupts. In most cases it was enabling SWBA interrupt
even if driver was trying to disable it.

In my tests, this issue was not really affecting STA or AP mode.

The only measurable difference i got on AP+STA mode. In this case
the FW was sending SWBA event to host at the channel switch in the
bgscan.

Possible effects of this issue:
- slow bgscan. EP3/EP4 have perfomance issue, extra traffic on will
make is slower.
- spreading our AP beacons on other channels.
- more ???

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

add -Wmaybe-uninitialized

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove -Wundef

it is already set here.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Merge pull request #111 from olerem/debian-2016.11.06

Remove FSF snail mail address from licensing texts

Remove FSF snail mail address from licensing texts

The FSF address changed before and might change in the future. Borrow
the recommended wording from GPLv3.

Signed-off-by: Paul Fertser <fercerpav@gmail.com>

Merge pull request #110 from olerem/clean-2016.11.02

Clean 2016.11.02

remove duplicate: cmnos_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove not directly used includes

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: uart_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: hif_pci.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: usb_table.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: magpie_regdump.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: intr_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: hif_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: athos_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused htc_tgt.c

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused buf_pool_dynamic.c

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused cmnos_clock.c

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused OTUS_htc.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused OTUS_misc.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused adf_os_pseudo.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused magpie_mem.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused opt_ah.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused adf_os_bitops.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused adf_os_crypto.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove unused asf_sm.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: hif_gmac.h

same checksum

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: cmnos_api.h

resulting same binary checksum

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: adf_nbuf_pvt.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: clock_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: clock_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: buf_pool_api.h and htc_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: printf_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: romp_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: hif_usb.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: dma_engine_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: dma_lib.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: wdt_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: vdesc_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: opt_ah.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: string_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: regdump.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: timer_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: tasklet_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: vbuf_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: misc_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: mem_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: eeprom_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove duplicate: allocram_api.h

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Merge pull request #106 from olerem/clean-2016.10.15

Clean 2016.10.15

Merge pull request #100 from RyDroid/editorconfig

Adding a file for EditorConfig

Merge pull request #97 from DC-fgervais/xretries

Only increment xretries statistic for unicast frames

docs: remove *png files

debian asks to remove pngs and generate it from svg on make, if needed.
For now, just remove it.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

fix all Wunused-but-set-variable varnings.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove all trailing whitespaces in one run

and convert dos to unix files. no other changes made here.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

add -Wunused-but-set-variable

we can't just now use -Wall. So lets add them step by step.
-Wunused-but-set-variable seems to be a good start :)

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Merge pull request #102 from RyDroid/travis

Adding a file for Travis CI

Merge pull request #104 from olerem/pre_debian-2016.10.05

make toolchain path configurable

make toolchain path configurable

this is needed to make it work with debian
cross-toolchain

Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Adding a file for EditorConfig

Adding a file for Travis CI

Merge pull request #99 from olerem/pre_debian-2016.10.02

Pre debian 2016.10.02

remove unused cmnos_printf.c

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

remove linux/compiler.h, mosty it is not used

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

change licese for atheros files

fix license issues for debian packaging:                                                                                        licensecheck --copyright -r target_firmware

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

change xtansa licesed files

fix license issues for debian packaging:                                                                                        licensecheck --copyright -r target_firmware

With permission from Tensilica, Inc, these files have been relicensed.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

fix missing licenses

fix license issues for debian packaging:                                                                                        licensecheck --copyright -r target_firmware

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Fix FSF adress in GPLv2 files

fix license issues for debian packaging:
licensecheck --copyright -r target_firmware

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

fix path for a sboot file

sboot was move to other location.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

use binutils 2.27

to provide debian package we need to use binutils 2.27.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

CMakeLists.txt: add -fno-lto

this option is needed to make compile work with cross-toolchain properly.
We are not using LTO any way, but for some reasons if cross-tolachain
is compailed without LTO, then FW compilation will fail.

With and without this option FW has same md5sum - So it should be save
to use this option.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Only increment xretries statistic for unicast frames

It looks like the ar5416 only has the AR_ExcessiveRetries flag
to let the outside know it didn't receive an ACK.

So in case of broadcast/multicast it gets set even though it's
not "excessive retry" per se. These frames just don't
acknowledge/retry in a 802.11 transmission.

This patch makes sure broadcast/multicast transmissions won't
increment the xretries statistic.

Merge pull request #94 from olerem/crypto

add cryptographic assurances when fetching the toolchain

add cryptographic assurances when fetching the toolchain

Previously, anyone tampering with the network could have modified the
toolchain source code in transit and the build would have proceeded as
expected.

With this patch, we can ensure that we're getting the expected
tarballs from upstream.

If the required toolchain packages change in the future, the sha256
digests of the new packages should be updated in Makefile as well.

Updated-by: Oleksij Rempel <linux@rempel-privat.de>

Merge pull request #93 from olerem/gcc-clean-2016.09.19.1

Gcc clean 2016.09.19.1

fix compile warnings for gcc 6.2.0

Most of issues discovered by gcc 6.2.0 are:
warning: implicit declaration of function '...'
which make sense since this functions was never declared,
espesially functions provided by ROM.

In the last case, correct usage of this functions was never
checked by compiler.

So, wellcome in to GCC 6.2.0 age ;)

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Revert "Add a patch for binutils 2.23.1"

This reverts commit 24b475841870cc495417619637c42d65381be38e.
We are using now binutils 2.26.1. So we can drop this patch.

update binutils.patch for 2.26.1

and drop useless update of copy right - it prevent us from
reusing this patch for latest binutils.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

update gcc.patch for gcc 6.2.0

and drop useless update of copy right - it prevent us from
reusing this patch for latest gcc.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

bump gcc to 6.2.0

gcc 6.2.0 builds with texinfo versions 6.x .

update other dependencies too.

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Merge pull request #92 from olerem/date-2016.09.16

remove __DATE__ and __TIME__

remove __DATE__ and __TIME__, make builds reproducible.

__DATE__ and __TIME__ are mostly useless information.
Even more, it makes builds not reprodusable which is evil.

Read more about reproducible builds here:
https://reproducible-builds.org/

Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>

Merge pull request #79 from kpengboy/master

Fix license missed during initial conversion

Fix license missed during initial conversion

Requested by Adrian Chadd

Merge pull request #69 from chunyeow/master

ath9k_htc_firmware: check only the mesh control present subfield

Merge pull request #67 from IanR778/timestamp_fix

Timestamp fix - looks good! Thanks!

Fix the licence - i missed this with my initial conversion
of the licences.

Update build instructions for FreeBSD.

ath9k_htc_firmware: check only the mesh control present subfield

This is related to the previous patch "fix the offset of
CCMP header for mesh data frame". Check only the Mesh
Control Present subfield on QoS Control field to determine
whether it is a mesh data frame.

Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>

Fix: bug in extend_tsf near 0x8000000 values

Bugfix + added more test cases.

FIX: support rstamps from the future too.

 * Extend a 32 bit TSF to nearest 64 bit TSF value.
 * When the adapter is a STATION, its local TSF is periodically modified by
 * the hardware to match the BSS TSF (as received in beacon packets), and
 * rstamp may appear to be from the future or from the past (with reference
 * to the current local TSF) because of jitter. This is mostly noticable in
 * highly congested channels. The code uses signed modulo arithmetic to
 * handle both past/future cases and signed-extension to avoid branches.
 * Test cases:
 * extend(0x0000001200000004, 0x00000006) == 0x0000001200000006
 * extend(0x0000001200000004, 0x00000002) == 0x0000001200000002
 * extend(0x0000001200000004, 0xfffffffe) == 0x00000011fffffffe  ! tsfhigh--
 * extend(0x00000012fffffffc, 0xfffffffe) == 0x00000012fffffffe
 * extend(0x00000012fffffffc, 0xfffffffa) == 0x00000012fffffffa
 * extend(0x00000012fffffffc, 0x00000002) == 0x0000001300000002  ! tsfhigh++

Firmware sometimes returns erroneous timestamps(2)

Fixed incorrect wrapping handling in ath_extend_tsf().

Firmware sometimes returns erroneous timestamps

Added proper synchronization while handling lower 32 bit and upper 32 bit parts in ar5416GetTsf64().

Merge pull request #64 from iazz/master

Fix resetting of txstatus flags.

Fix resetting of txstatus flags.

Merge pull request #61 from sudoman/sudoman-fixes

minimal changes to fix the toolchain build process on some newer systems

bump gcc to newer revision

gcc 4.7.4 builds with texinfo versions 5.x .

fix binutils build with new gcc versions

disabling -Werror while building binutils ignores the following
warnings on systems using -Werror and -Wall by default:

    -Wunused-value

Add a patch for binutils 2.23.1

Without this patch, binutils fails to compile.
This patch will not be needed for binutils 2.24.