Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753838AbbEKLZX (ORCPT ); Mon, 11 May 2015 07:25:23 -0400 Received: from bhuna.collabora.co.uk ([93.93.135.160]:42780 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753150AbbEKLZV (ORCPT ); Mon, 11 May 2015 07:25:21 -0400 Message-ID: <5550919A.9090703@collabora.co.uk> Date: Mon, 11 May 2015 13:25:14 +0200 From: Javier Martinez Canillas User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.2.0 MIME-Version: 1.0 To: Michal Suchanek , linux-arm-kernel@lists.infradead.org, Russell King , Kukjin Kim , Krzysztof Kozlowski , Lukasz Majewski , Andreas Faerber , Sachin Kamat , Sjoerd Simons , "Rafael J. Wysocki" , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ARM: exynos_defconfig: add options to make wifi usable References: <20150511105034.40580.qmail@dec59.ruk.cuni.cz> In-Reply-To: <20150511105034.40580.qmail@dec59.ruk.cuni.cz> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1587 Lines: 42 Hello Michal, On 05/11/2015 12:22 PM, Michal Suchanek wrote: > The Exynos defconfig includes mwifiex sdio support which is present on > some of the Exynos boards. > > For the WiFi to be usable two extra options are needed. Usermode Your subject line and the commit message are somehow misleading since these options are needed to make the WiFi usable with your current setup. > firmware helper to load out-of-kernel firmware and wireless extensions For example, this is only needed if the in-kernel fw loader is not able to find the firmware but isn't needed if the fw is for example in an initial ramdisk and the kernel is able to load it, built in the kernel or if the mwifiex driver is built as a module. > so the interface can be configured with wireless-tools. > And wireless extensions is deprecated AFAIK and is only needed for old user-space since most tools should had been converted to use the netlink based CONFIG_CFG80211 interface instead. I'm booting a debian jessie and have WiFi working without CFG80211_WEXT for example. That doesn't mean that I'm against your patch (I'm always happy to enable more config options if that makes the defconfig more useful) but the commit message should be accurate about why a change has to be done. > Signed-off-by: Michal Suchanek > --- Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/