Re: [PATCH] Add fc0011 tuner driver

2012-05-07 Thread Rémi Denis-Courmont
Le mardi 8 mai 2012 00:00:31 Michael Büsch, vous avez écrit : > > > + dev_dbg(&priv->i2c->dev, "Tuned to " > > > + "fa=%02X fp=%02X xin=%02X%02X vco=%02X vcosel=%02X " > > > + "vcocal=%02X(%u) bw=%u\n", > > > + (unsigned int)regs[FC11_REG_FA], > > > + (unsigned int)r

Re: [PATCH] Add fc0011 tuner driver

2012-05-07 Thread Michael Büsch
On Mon, 07 May 2012 22:02:18 +0300 Antti Palosaari wrote: > On 02.04.2012 19:14, Michael Büsch wrote: > > This adds support for the Fitipower fc0011 DVB-t tuner. > > > > Signed-off-by: Michael Buesch > > > + unsigned int i, vco_retries; > > + u32 freq = p->frequency / 1000; > > + u32 bandw

Re: [PATCH] Add fc0011 tuner driver

2012-05-07 Thread Antti Palosaari
On 02.04.2012 19:14, Michael Büsch wrote: This adds support for the Fitipower fc0011 DVB-t tuner. Signed-off-by: Michael Buesch + unsigned int i, vco_retries; + u32 freq = p->frequency / 1000; + u32 bandwidth = p->bandwidth_hz / 1000; + u32 fvco, xin, xdiv, xdivr; +

Re: [PATCH] Add fc0011 tuner driver

2012-04-02 Thread Michael Büsch
On Tue, 03 Apr 2012 00:52:08 +0300 Antti Palosaari wrote: > > Well the fc0011 tuner driver still works worse on this af9035 driver > > than on Hans' driver. I have absolutely no idea why this is the case. > > I'm almost certain that it is not a timing issue of some sort. I tried > > a zillion of

Re: [PATCH] Add fc0011 tuner driver

2012-04-02 Thread Antti Palosaari
On 02.04.2012 20:51, Michael Büsch wrote: Likely tuner driver, or demod driver. But as demod tuner initialization tables are likely correct I suspect it is tuner issue at first hand. And secondly my other hardware with TUA9001 performs very well, better than old AF9015 sticks. Well the fc0011 t

Re: [PATCH] Add fc0011 tuner driver

2012-04-02 Thread Michael Büsch
On Mon, 02 Apr 2012 20:40:45 +0300 Antti Palosaari wrote: > h, I think Mauro will at least complain when I ask he to PULL that > master. Personally I would like to see line len something more than 80 > chars, but as checkpatch.pl complains it I have shortened lines despite > very few cases

Re: [PATCH] Add fc0011 tuner driver

2012-04-02 Thread Antti Palosaari
On 02.04.2012 20:20, Michael Büsch wrote: On Mon, 02 Apr 2012 19:56:50 +0300 Antti Palosaari wrote: On 02.04.2012 19:14, Michael Büsch wrote: This adds support for the Fitipower fc0011 DVB-t tuner. Signed-off-by: Michael Buesch Applied, thanks! http://git.linuxtv.org/anttip/media_tree.git/

Re: [PATCH] Add fc0011 tuner driver

2012-04-02 Thread Michael Büsch
On Mon, 02 Apr 2012 19:56:50 +0300 Antti Palosaari wrote: > On 02.04.2012 19:14, Michael Büsch wrote: > > This adds support for the Fitipower fc0011 DVB-t tuner. > > > > Signed-off-by: Michael Buesch > > Applied, thanks! > http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/af9035_e

Re: [PATCH] Add fc0011 tuner driver

2012-04-02 Thread Antti Palosaari
On 02.04.2012 19:14, Michael Büsch wrote: This adds support for the Fitipower fc0011 DVB-t tuner. Signed-off-by: Michael Buesch Applied, thanks! http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/af9035_experimental I looked it through quickly, no big issues. Anyhow, when I ran

[PATCH] Add fc0011 tuner driver

2012-04-02 Thread Michael Büsch
This adds support for the Fitipower fc0011 DVB-t tuner. Signed-off-by: Michael Buesch --- Index: linux/drivers/media/common/tuners/Kconfig === --- linux.orig/drivers/media/common/tuners/Kconfig 2012-04-02 15:55:51.155296579