[PATCH v2 1/4] brcmfmac: Do not print the firmware version as an error

Using pr_err for things which are not errors is a bad idea. E.g. it will cause the plymouth bootsplash screen to drop back to the text console so that the user can see the error, which is not what we normally want to happen. Instead add a new brcmf_info macro and use that. Signed-off-by: Hans de

[PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time

The firmware responding with -EBUSY when trying to add an extra virtual-if is a normal thing, do not print an error for this. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- .../net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c| 14 ++ drivers/net/wireless/br

[PATCH v2 3/4] brcmfmac: Do not complain about country code "00"

The country code gets set to "00" by default at boot, ignore this rather then logging an error about it. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 4 1 file changed, 4 insertions(+) diff --git a/driv

[PATCH 0/4] brcmfmac: Remove a bunch of false positive error logs

Hi All, Here is a small series to stop brcmfmac from spamming dmesg with errors which are not really errors at all. Note the 4th patch actually contains a small behavior change rather then just changing logging, so it needs a bit of extra review attention. Regards, Hans

[PATCH 4/4] brcmfmac: Handle status == BRCMF_E_STATUS_ABORT in cfg80211_escan_handler

ORT and in this case simply cleanly exit the cfg80211_escan_handler. This also avoids a BRCMF_E_STATUS_ABORT event arriving after a new scan has been started causing the new scan to complete prematurely without any data. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- drivers/net

[PATCH 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time

The firmware responding with -EBUSY when trying to add an extra virtual-if is a normal thing, do not print an error for this. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- .../net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c| 14 ++ drivers/net/wireless/br

[PATCH 3/4] brcmfmac: Do not complain about country code "00"

The country code gets set to "00" by default at boot, ignore this rather then logging an error about it. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 4 1 file changed, 4 insertions(+) diff --git a/driv

[PATCH 1/4] brcmfmac: Do not print the firmware version as an error

Using pr_err for things which are not errors is a bad idea. E.g. it will cause the plymouth bootsplash screen to drop back to the text console so that the user can see the error, which is not what we normally want to happen. Instead add a new brcmf_info macro and use that. Signed-off-by: Hans de

[PATCH] net: wireless: rtl8xxxu: Make rtl8xxxu_ampdu_action less chatty

On my home network rtl8xxxu is spamming the log with IEEE80211_AMPDU_RX_START / IEEE80211_AMPDU_RX_STOP every few seconds turn these messages into debug messages. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c | 10 +-

Re: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 30-06-16 12:18, Jonas Gorski wrote: Hi, On 30 June 2016 at 12:04, Hans de Goede <hdego...@redhat.com> wrote: Hi, On 30-06-16 11:58, Kalle Valo wrote: Hans de Goede <hdego...@redhat.com> writes: Hi, On 30-06-16 11:02, Kalle Valo wrote: Priit Laes <pl...@pl

Re: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 30-06-16 11:58, Kalle Valo wrote: Hans de Goede <hdego...@redhat.com> writes: Hi, On 30-06-16 11:02, Kalle Valo wrote: Priit Laes <pl...@plaes.org> writes: What is the size of this nvram file? As it's board specific, I wonder if we can simply include it inside of the DT

Re: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 29-06-16 20:51, 'Arend Van Spriel' via linux-sunxi wrote: On 29-6-2016 20:01, Hans de Goede wrote: Hi, On 29-06-16 19:00, Kalle Valo wrote: Hans de Goede <hdego...@redhat.com> writes: Hi, On 29-06-16 16:42, Jonas Gorski wrote: Hi, On 29 June 2016 at 16:04, Hans de Goede

Re: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 30-06-16 11:02, Kalle Valo wrote: Priit Laes writes: What is the size of this nvram file? As it's board specific, I wonder if we can simply include it inside of the DT verbatim. I remember doing that (in the pre-dtb days, on real open firmware) for the "spidernet"

Re: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 30-06-16 10:46, Kalle Valo wrote: Arend Van Spriel writes: Since we are dealing with a per-board config-file here, which is loaded from the os filesystem we really need to specify a basename here as the list of possible boards is endless, so we cannot

Re: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 29-06-16 19:00, Kalle Valo wrote: Hans de Goede <hdego...@redhat.com> writes: Hi, On 29-06-16 16:42, Jonas Gorski wrote: Hi, On 29 June 2016 at 16:04, Hans de Goede <hdego...@redhat.com> wrote: Add a brcm,nvram_file_name dt property to allow overruling the default nv

Re: [linux-sunxi] Re: [PATCH 2/4] ARM: dts: sun7i-a20-cubietruck: Set brcm,nvram_file_name

HI, On 29-06-16 19:01, Kalle Valo wrote: Hans de Goede <hdego...@redhat.com> writes: The cubietruck uses an ap6210 sdio wifi module, set brcm,nvram_file_name to brcmfmac43362-ap6210.txt so that we can ship the ap6210 specific nvram file in linunx-firmware to get the wifi to wo

Re: [PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

Hi, On 29-06-16 16:42, Jonas Gorski wrote: Hi, On 29 June 2016 at 16:04, Hans de Goede <hdego...@redhat.com> wrote: Add a brcm,nvram_file_name dt property to allow overruling the default nvram filename for sdio devices. The idea is that we can specify a board specific nvram fil

[PATCH 4/4] ARM: dts: sun5i-a10s-auxtek-t004: Set brcm,nvram_file_name

to ship the ap6210 specific nvram file in linux-firmware to get the wifi to work out of the box without users needing to download and install the nvram file themselves. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- arch/arm/boot/dts/sun5i-a10s-auxtek-t004.dts | 6 ++ 1 file chan

[PATCH 3/4] ARM: dts: sun7i-a20-wits-pro-a20-dkt: Set brcm,nvram_file_name

to work out of the box without users needing to download and install the nvram file themselves. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- arch/arm/boot/dts/sun7i-a20-wits-pro-a20-dkt.dts | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/sun7i-a20-wits-pro-a20-dkt.

[PATCH 1/4] brcmfmac: Add brcm,nvram_file_name dt property

of the box, without requiring users to find and then manually install the right nvram file for their board. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- .../devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt | 2 ++ drivers/net/wireless/broadcom/brcm80211/brcmfma

[PATCH 2/4] ARM: dts: sun7i-a20-cubietruck: Set brcm,nvram_file_name

a downside of this patch is that users who have already manually installed the nvram file will need to rename it. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- arch/arm/boot/dts/sun7i-a20-cubietruck.dts | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/sun7i-a20-cubietru

Re: RTL8189ETV

Hi, On 31-08-15 22:31, poma wrote: Let's see if Hans knows what type of this device is RTL8189ETV. Assuming you are talking about the wifi module of the Allwinner H3 based Orangepi2 as shown here: http://linux-sunxi.org/images/9/96/Xunlong_OrangePi2_front.jpg Then we are talking about a

<    1   2