Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161388AbWJKVhk (ORCPT ); Wed, 11 Oct 2006 17:37:40 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1161390AbWJKVhk (ORCPT ); Wed, 11 Oct 2006 17:37:40 -0400 Received: from kurby.webscope.com ([204.141.84.54]:5029 "EHLO kirby.webscope.com") by vger.kernel.org with ESMTP id S1161388AbWJKVhh (ORCPT ); Wed, 11 Oct 2006 17:37:37 -0400 Message-ID: <452D63D4.6050300@linuxtv.org> Date: Wed, 11 Oct 2006 17:36:20 -0400 From: Michael Krufky User-Agent: Thunderbird 1.5.0.7 (Windows/20060909) MIME-Version: 1.0 To: Greg KH CC: linux-kernel@vger.kernel.org, stable@kernel.org, v4l-dvb maintainer list Subject: Re: [patch 06/67] Video: cx24123: fix PLL divisor setup References: <20061011204756.642936754@quad.kroah.org> <20061011210353.GG16627@kroah.com> <452D5EF7.80303@linuxtv.org> <20061011212959.GA18006@suse.de> In-Reply-To: <20061011212959.GA18006@suse.de> X-Enigmail-Version: 0.94.0.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1755 Lines: 51 Greg KH wrote: > On Wed, Oct 11, 2006 at 05:15:35PM -0400, Michael Krufky wrote: >> Greg KH wrote: >>> -stable review patch. If anyone has any objections, please let us know. >>> >>> ------------------ >>> From: Yeasah Pell >>> >>> The cx24109 datasheet says: "NOTE: if A=0, then N=N+1" >>> >>> The current code is the result of a misinterpretation of the datasheet to >>> mean exactly the opposite of the requirement -- The actual value of N is 1 >>> greater than the value written when A is 0, so 1 needs to be *subtracted* >>> from it to compensate. >>> >>> Signed-off-by: Yeasah Pell >>> Signed-off-by: Steven Toth >>> Signed-off-by: Michael Krufky >>> Signed-off-by: Greg Kroah-Hartman >> Greg, >> >> When you apply this patch to your 2.6.18.y tree (and also to your >> 2.6.17.y tree) , can you please preceed the patch title with 'DVB' >> instead of 'VIDEO' ? >> >> I'll be sure to specify the subsystem, instead of only the driver name >> in future patches. > > Yes, it's better for you to specifiy it, instead of having me guess at > what it should be classified as :) > > I'll try to go edit the existing patches to fix this, OOPS! I just saw your -stable commit. Slight misunderstanding, Greg... Out of those six patches that I sent to you, only "cx24123: fix PLL divisor setup" is a DVB patch... The remaining 5 patches are V4L patches. Sorry for the confusion. Regards, Michael Krufky - 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/