From patchwork Fri Nov 10 00:32:02 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ben Greear X-Patchwork-Id: 10052177 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8C1326035D for ; Fri, 10 Nov 2017 00:32:42 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7DF7B2B222 for ; Fri, 10 Nov 2017 00:32:42 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 72B982B221; Fri, 10 Nov 2017 00:32:42 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 262192B222 for ; Fri, 10 Nov 2017 00:32:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Date:Message-ID:Subject:From:To: Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=EBPIjD73lq1OS2MPKnrBe3VTUjVAvK/HvIO+EgdSIvw=; b=qL7z3fHTmk2M9g pNtvl5vd1A15UON1y4DodKXplFWHq2hOtUuR7nVTGmvvnuJFBkar4ygWWnG3TvYgWqxq68XHi2uYx t1qD9b6VkicgfL/JlHX+leMagYnDBJnRBSaizJd3VDZn2qS95iyR7jHGSFF5IMe5wFri+Z0SwGfg2 fY0Vhil1Xd79KY3csEZnUKkrO8EtAMon+IO8agxh+dYWeM1uPQ9BuTgBKOckOAfxvRwYcBfrTUoW2 Zv4yqPuShE0UKUFFf+0yhLeywuNjjIv96UqSGNHm/cd2TjhlkFkm82csqc+V9cjVgGb3hnhErnc01 /CbLWAGMgCJfxYC6r0wA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1eCxFI-0006Hu-1r; Fri, 10 Nov 2017 00:32:32 +0000 Received: from mail2.candelatech.com ([208.74.158.173]) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1eCxFD-0005TC-UV for ath10k@lists.infradead.org; Fri, 10 Nov 2017 00:32:30 +0000 Received: from [192.168.100.149] (firewall.candelatech.com [50.251.239.81]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail2.candelatech.com (Postfix) with ESMTPSA id AD8A940A539; Thu, 9 Nov 2017 16:32:02 -0800 (PST) To: "linux-wireless@vger.kernel.org" , erik.stromdahl@gmail.com, ath10k From: Ben Greear Subject: ath10k targaddrs issue Organization: Candela Technologies Message-ID: Date: Thu, 9 Nov 2017 16:32:02 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20171109_163228_059792_276C694B X-CRM114-Status: GOOD ( 10.18 ) X-BeenThere: ath10k@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "ath10k" Errors-To: ath10k-bounces+patchwork-ath10k=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP While poking around in the firmware, I noticed what appears to be an ath10k driver issue, which came in with this commit below. commit 01d6fd6965eaf8d4b3aab681d30e77c53a834162 Author: Erik Stromdahl Date: Wed Apr 26 12:17:55 2017 +0300 ath10k: various sdio related definitions Debug masks for SDIO HIF layer. Address definitions for SDIO/mbox based chipsets. Augmented struct host_interest with more members. Signed-off-by: Erik Stromdahl Signed-off-by: Kalle Valo ... Those variables above may be correct for some firmware, but they are not correct for at least some versions of 10.4, which have this instead: ... A_UINT32 hi_fw_swap; /* 0x104 */ /* host side need to support tx/rx data packet swap */ A_UINT32 hi_txrx_dataswap; /* 0x108 */ /* enable allocram statistics gathering (set to an integer, which * is the number of tracking records that allocram will allocate). */ A_UINT32 hi_allocram_track_max; /* 0x108 */ } I guess this may not be an issue as long as zero'd values are sent to the 10.4 nics, but it is confusing at best. Eric, where did you get these values from? Thanks, Ben diff --git a/drivers/net/wireless/ath/ath10k/targaddrs.h b/drivers/net/wireless/ath/ath10k/targaddrs.h index cbac9e42..8bded5d 100644 --- a/drivers/net/wireless/ath/ath10k/targaddrs.h +++ b/drivers/net/wireless/ath/ath10k/targaddrs.h @@ -205,6 +205,24 @@ struct host_interest { */ /* Bit 1 - unused */ u32 hi_fw_swap; /* 0x104 */ + + /* global arenas pointer address, used by host driver debug */ + u32 hi_dynamic_mem_arenas_addr; /* 0x108 */ + + /* allocated bytes of DRAM use by allocated */ + u32 hi_dynamic_mem_allocated; /* 0x10C */ + + /* remaining bytes of DRAM */ + u32 hi_dynamic_mem_remaining; /* 0x110 */ + + /* memory track count, configured by host */ + u32 hi_dynamic_mem_track_max; /* 0x114 */ + + /* minidump buffer */ + u32 hi_minidump; /* 0x118 */ + + /* bdata's sig and key addr */ + u32 hi_bd_sig_key; /* 0x11c */ } __packed;