Return-path: Received: from mail-pa0-f54.google.com ([209.85.220.54]:36531 "EHLO mail-pa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752765AbbJ0AHD (ORCPT ); Mon, 26 Oct 2015 20:07:03 -0400 Received: by pacfv9 with SMTP id fv9so211718084pac.3 for ; Mon, 26 Oct 2015 17:07:03 -0700 (PDT) Subject: Re: [PATCH] wcn36xx: Remove warning message when dev is NULL for arm64 dma_alloc. To: Bjorn Andersson References: <1445826982-4023-1-git-send-email-fengwei.yin@linaro.org> Cc: linux-wireless , wcn36xx , me@bobcopeland.com, Eugene Krasnikov , Bjorn Andersson From: yfw Message-ID: <562EC022.4030302@linaro.org> (sfid-20151027_010709_383841_79349625) Date: Tue, 27 Oct 2015 08:06:58 +0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 2015/10/27 2:24, Bjorn Andersson wrote: > On Sun, Oct 25, 2015 at 7:36 PM, Yin, Fengwei wrote: >> arm64 has requirement that all the dma operations have actual device. >> Otherwise, following warnning message shown and dma allocation fails: >> >> WARNING: CPU: 0 PID: 954 at arch/arm64/mm/dma-mapping.c:106 __dma_alloc+0x24c/0x258() >> Use an actual device structure for DMA allocation >> Modules linked in: wcn36xx wcn36xx_platform >> CPU: 0 PID: 954 Comm: ifconfig Not tainted 4.0.0+ #14 >> Hardware name: Qualcomm Technologies, Inc. MSM 8916 MTP (DT) >> Call trace: >> [] dump_backtrace+0x0/0x124 >> [] show_stack+0x10/0x1c >> [] dump_stack+0x80/0xc4 >> [] warn_slowpath_common+0x98/0xd0 >> [] warn_slowpath_fmt+0x4c/0x58 >> [] __dma_alloc+0x248/0x258 >> [] wcn36xx_dxe_allocate_mem_pools+0xc4/0x108 [wcn36xx] >> [] wcn36xx_start+0x38/0x240 [wcn36xx] >> [] ieee80211_do_open+0x1b0/0x9a4 >> [] ieee80211_open+0x58/0x68 >> [] __dev_open+0xb0/0x120 >> [] __dev_change_flags+0x88/0x150 >> [] dev_change_flags+0x1c/0x5c >> [] devinet_ioctl+0x644/0x6f0 >> >> Signed-off-by: Yin, Fengwei > > Acked-by: Bjorn Andersson Thanks for the Acked-by. Regards Yin, Fengwei > > Regards, > Bjorn >