Date: Fri, 22 Jun 2012 16:56:11 -0700 (PDT)
Message-Id: <20120622.165611.2220193371527786713.davem@davemloft.net> (sfid-20120623_015629_926189_9315BB32)
To: c_manoha@qca.qualcomm.com
Cc: linville@tuxdriver.com, linux-wireless@vger.kernel.org,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: pull request: wireless 2012-06-22
From: David Miller <davem@davemloft.net>
In-Reply-To: <20453.302.671438.533390@gargle.gargle.HOWL>
References: <20120622201716.GB13918@tuxdriver.com>
	<20452.64701.760819.559191@gargle.gargle.HOWL>
	<20453.302.671438.533390@gargle.gargle.HOWL>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Sender: linux-wireless-owner@vger.kernel.org

From: Sujith Manoharan <c_manoha@qca.qualcomm.com>
Date: Sat, 23 Jun 2012 05:05:10 +0530

> Sujith Manoharan wrote:
>> John W. Linville wrote:
>> > Mohammed Shafi Shajakhan (1):
>> >       ath9k_hw: avoid possible infinite loop in ar9003_get_pll_sqsum_dvc
>> 
>> Um, this commit doesn't really fix anything - the timeout is also
>> arbitrary. Can you remove this from the 3.5 queue ? Thanks.
> 
> A review of the patch later, I think it's good. At least there's a
> WARN_ON_ONCE() to indicate the situation in which the measurement fails.
> It might come in handy as a debugging aid.

Thanks for wasting our time.

Make these determinations when changes to into John's tree, not
later when he asks me to pull them into mine.