Return-path: Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:26078 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754372Ab3FQRvq convert rfc822-to-8bit (ORCPT ); Mon, 17 Jun 2013 13:51:46 -0400 Date: Mon, 17 Jun 2013 13:51:43 -0400 From: Jason Cooper To: =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= Cc: Franky Lin , rvossen@broadcom.com, arend@broadcom.com, linux-wireless@vger.kernel.org Subject: Re: [RFC PATCH] net: brcmfmac: add sdio chip id 0x4319 Message-ID: <20130617175143.GR31667@titan.lakedaemon.net> (sfid-20130617_195153_366450_8DEAFF9C) References: <20130612222123.GB31667@titan.lakedaemon.net> <20130617161903.GN31667@titan.lakedaemon.net> <51BF498F.1010007@broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, Jun 17, 2013 at 07:45:25PM +0200, Rafał Miłecki wrote: > 2013/6/17 Franky Lin : > > On 06/17/2013 09:19 AM, Jason Cooper wrote: > >> > >> Added Franky... > >> > >> On Wed, Jun 12, 2013 at 06:21:23PM -0400, Jason Cooper wrote: > >>> > >>> All, > >>> > >>> I have a Seagate Wireless Plus I am trying to put a vanilla kernel on > >>> (currently v3.10-rc5 for various omap DT bits). > >>> > >>> I have the board booting and running a debian rootfs on the HD. I just > >>> got mmc to come up and I've discovered that the wireless card is vendor > >>> 0x02d0, device 0x4319. > >>> > >>> My hope, once I got to this point, was that I would be able to use > >>> the mainline, open source driver. Unfortunately, it looks like brcmfmac > >>> lost support for the 0x4319 while it was in staging. > >>> > >>> The commit in question is: > >>> > >>> 4dad253 staging: brcm80211: remove code for unsupported chip > >>> > >>> Is adding it back in a bridge too far? > >> > >> It looks like only two things are needed, adding the chip id and then > >> setting the addresses. Here's a preliminary patch to do just that. > >> Note, the addresses I have used are a straight copy from the 4329. It > >> causes this: > >> > >> [ 12.346466] mmcblk mmc0:0001: no of_node; not parsing pinctrl DT > >> [ 12.353149] brcmfmac_sdio mmc0:0001:1: no of_node; not parsing pinctrl > >> DT > >> [ 12.360717] brcmfmac_sdio mmc0:0001:2: no of_node; not parsing pinctrl > >> DT > >> [ 12.368804] brcmfmac: brcmf_sdio_chip_drivestrengthinit: No SDIO Drive > >> strength init done for chip 4319 rev 1 pmurev 7 > >> [ 12.380462] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte > >> F0:@0x00408=03, Err: -22 > >> [ 12.391906] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte > >> F0:@0x00408=03, Err: -22 > >> [ 12.403289] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte > >> F0:@0x00408=03, Err: -22 > >> [ 12.412261] brcmfmac: brcmf_sdio_regrw_helper: failed with -22 > >> [ 12.418457] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte > >> F0:@0x00408=01, Err: -22 > >> [ 12.429718] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte > >> F0:@0x00408=01, Err: -22 > >> [ 12.441101] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte > >> F0:@0x00408=01, Err: -22 > >> [ 12.450073] brcmfmac: brcmf_sdio_regrw_helper: failed with -22 > >> > >> I've taken a quick look at aosp, the provided GPL broadcom code, and I > >> haven't seen anything for the 4319 wrt to addresses. How hard would it > >> be to get those addresses from you guys? > > > > > > Hi Jason, > > > > The major obstacle of adding 4319 support is the obsolete firmware. Since > > it's an EOL chip we are not planning to add the support to brcmfmac. > > Does it mean, BCM4319 is a "new architecture" chipset, working > similarly to the already supported (in brcmfmac) chipsets? In older versions of the broadcom driver 4329 and 4319 were in the same code base. There was minimal reference to 4319 beyond "I work with this chip id". Hence my attempt. Now I'm wondering how much effort there would be to adding legacy firmware support to brcmfmac. I would only need AP mode to start with (yeah, no big deal ;-) ). thx, Jason.