On 01/14/2014 08:25 PM, Rafał Miłecki wrote: > 2014/1/14 Arend van Spriel <arend@xxxxxxxxxxxx>: >> On 01/14/2014 07:29 AM, Rafał Miłecki wrote: >>> 2014/1/13 Arend van Spriel <arend@xxxxxxxxxxxx>: >>>> The IOCTL register in the agent/wrapper contains additional bits >>>> that are core specific and use in the core reset sequence. >>> >>> I'm not sure about this. I don't think we want to keep device-specific >>> bits in a commond (bcma's) code. >> >> True. We can put these in brcm80211 includes so both brcmfmac and >> brcmsmac can use it. That said the bcma_core_disable() and >> bcma_core_enable() do need to be corrected to be reliable. Patch #2 in >> this series show what needs to be changed. > > I'm looking at > [PATCH 2/8] brcmfmac: update core reset and disable routines. > and it's unclear for me, what needs to be changed in > bcma_core_[enable|disable]. Could you specify, please? Basically, the whole sequence in bcma_core_enable() and disable() is derived from aiutils.c in the original brcmsmac mainline patch (or at least it has very, very close resemblance). It turned out this sequence was cooked up by software devs, but the hardware documentation states another sequence, which is what Patch #2 implements. For this it needs additional parameters. You know what, I will create a patch for this. Gr. AvS -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
