[lkml]   [2017]   [Aug]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v2] arm64: allwinner: h5: add support for NanoPi NEO Plus 2
On Fri, Aug 25, 2017 at 03:28:41PM +0200, Code Kipper wrote:
> On 25 August 2017 at 12:32, Antony Antony <> wrote:

> > +
> > + brcmf: bcrmf@1 {
> > + reg = <1>;
> > + compatible = "brcm,bcm4329-fmac";
> > + };
> Incorrect wifi chip - just deliver what you've tested.

You are right. Now I found the correct chip ID by enabling debug with
Broadcom kernel module 'modprobe brcmfmac debug=0x3ffff6'

The module is Ampak AP6212A with Broadcom 43430 rev=1 inside it.

brcmfmac: brcmf_chip_recognition found AXI chip: BCM43430, rev=1
brcmfmac: brcmf_ops_sdio_probe sdio vendor ID: 0x02d0
brcmfmac: brcmf_ops_sdio_probe sdio device ID: 0xa9a6

IPv6: ADDRCONF(NETDEV_CHANGE): wlan0: link becomes ready

This information mostly match with, except
module is actually AP6212A. Some of the modules may not have 'A' printed on
them. However the ones that identify as rev=1 is a AP6212A.

FYI: there were two things that confused me
- bcm4329-fmac also works.
- I couldn't configure it to join WiFi network with password.

With change I will send a v4. Here is the propssed diff to v3.


- * WiFi driver support could be incomplete,
- * wlan0 is able to see Base Stations, however not able to join.
+ * AMPAK AP6212A WiFi module with BCM43430, rev=1 inside
+ * sdio vendor ID: 0x02d0, sdio device ID: 0xa9a6
brcmf: wifi@1 {
reg = <1>;
- compatible = "brcm,bcm4329-fmac";
+ compatible = "brcm,bcm43430-fmac";

I think now nanopi-neo-plus2.dts, v4, is in a good shape to merge.

thanks for the feedback and review.


 \ /
  Last update: 2017-08-28 14:17    [W:0.074 / U:1.340 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site