Messages in this thread | | | Date | Wed, 11 Oct 2006 14:29:59 -0700 | From | Greg KH <> | Subject | Re: [patch 06/67] Video: cx24123: fix PLL divisor setup |
| |
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 <yeasah@schwide.net> > > > > 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 <yeasah@schwide.net> > > Signed-off-by: Steven Toth <stoth@hauppauge.com> > > Signed-off-by: Michael Krufky <mkrufky@linuxtv.org> > > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> > > 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,
thanks,
greg k-h - 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/
| |