From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753758AbdBPHoo (ORCPT ); Thu, 16 Feb 2017 02:44:44 -0500 Received: from mo69.mail-out.ovh.net ([178.32.228.69]:46544 "EHLO mo69.mail-out.ovh.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752986AbdBPHon (ORCPT ); Thu, 16 Feb 2017 02:44:43 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Date: Thu, 16 Feb 2017 08:25:42 +0100 From: =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= To: Andy Shevchenko Cc: =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= , Ming Lei , "Luis R . Rodriguez" , Greg KH , Linux Kernel Mailing List , Kalle Valo , Arend van Spriel , "open list:TI WILINK WIRELES..." , brcm80211-dev-list.pdl@broadcom.com Subject: Re: [PATCH 2/2] brcmfmac: don't warn user if loading firmware file with NVRAM fails In-Reply-To: References: <20170215222948.21030-1-zajec5@gmail.com> <20170215222948.21030-2-zajec5@gmail.com> Message-ID: User-Agent: Roundcube Webmail/1.2.3 X-Originating-IP: 194.187.74.233 X-Webmail-UserID: rafal@milecki.pl X-Ovh-Tracer-Id: 6052556427850387103 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrfeelhedrtdefgdduuddvucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuqfggjfdpvefjgfevmfevgfenuceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddm Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017-02-16 02:19, Andy Shevchenko wrote: > On Thu, Feb 16, 2017 at 12:29 AM, Rafał Miłecki > wrote: >> From: Rafał Miłecki >> >> This isn't critical as we use platform NVRAM as fallback and it's very >> common case of all Broadcom home routers. Thanks for the new firmware >> loading function we can achieve this by simply passing FW_OPT_NO_WARN. >> > > What kind of warning you are talking about? > > On Intel Edison brcmfmac isn't backed up by NVRAM (there no such) and > the platform is not ACPI-based. The warning might be crucial for > debugging Wi-Fi issues on such platforms. Sorry, I should have tried better with that commit message. V2 in a second.