From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.deathmatch.net ([70.167.247.36]:4018 "EHLO mail.deathmatch.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751235AbZAIS3U (ORCPT ); Fri, 9 Jan 2009 13:29:20 -0500 From: "Bob Copeland" To: Kalle Valo Cc: Johannes Berg , linux-wireless , linville@tuxdriver.com Subject: Re: drivers returning wrong results from their _tx op Date: Fri, 9 Jan 2009 13:27:27 -0500 Message-Id: <20090109182142.M2510@bobcopeland.com> (sfid-20090109_192924_395403_24228DE9) In-Reply-To: <87hc48tpwa.fsf@litku.valot.fi> References: <1231193264.3697.41.camel@johannes> <1231246859.3503.0.camel@johannes> <20090109022347.GA5077@hash.localnet> <87hc4828xn.fsf@litku.valot.fi> <20090109153716.M19288@bobcopeland.com> <87hc48tpwa.fsf@litku.valot.fi> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Fri, 09 Jan 2009 18:17:41 +0200, Kalle Valo wrote > I read it. Adding "or NETDEV_TX_BUSY if it fails" is just confusing, > better to state that it just should not fail. Fair enough, but the state of things today is some drivers do. Yes, we're guilty of that in ath5k and it should be fixed. But like ->config(), if the callback should never ever fail, then returning an error code at all is pretty suspect. Would this be better? "This function should return NETDEV_TX_OK except in very limited cases." Then when someone asks which cases, you say "none"... -- Bob Copeland %% www.bobcopeland.com