From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx51.mymxserver.com ([85.199.173.110]:15667 "EHLO mx51.mymxserver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755619Ab0C3GsH (ORCPT ); Tue, 30 Mar 2010 02:48:07 -0400 From: Holger Schurig To: Daniel Mack Subject: Re: [PATCH] net/wireless/libertas: do not call wiphy_unregister() w/o wiphy_register() Date: Tue, 30 Mar 2010 08:49:07 +0200 Cc: linux-kernel@vger.kernel.org, Dan Williams , "John W. Linville" , Bing Zhao , libertas-dev@lists.infradead.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org References: <1269875658-3324-1-git-send-email-daniel@caiaq.de> In-Reply-To: <1269875658-3324-1-git-send-email-daniel@caiaq.de> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Message-Id: <201003300849.07441.hs4233@mail.mn-solutions.de> Sender: linux-wireless-owner@vger.kernel.org List-ID: > The libertas driver calls wiphy_unregister() without a prior > wiphy_register() when a devices fails initialization. Fix this by > introducing a private flag. Nice. However, I wonder: do we really need a private variable? Does each driver introduce a private variable for this?