linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/1] xc3028: fix center frequency calculation for DTV78 firmware
@ 2011-12-08 12:56 Gianluca Gennari
  2011-12-08 13:00 ` [PATCH 1/1] " Gianluca Gennari
  2011-12-10 14:51 ` [PATCH 0/1] " Mauro Carvalho Chehab
  0 siblings, 2 replies; 5+ messages in thread
From: Gianluca Gennari @ 2011-12-08 12:56 UTC (permalink / raw)
  To: linux-media; +Cc: Mauro Carvalho Chehab

Hi all,
this patch replaces the previous one proposed in the thread "xc3028:
force reload of DTV7 firmware in VHF band with Zarlink demodulator".
The problem is that the firmware DTV78 works fine in UHF band (8 MHz
bandwidth) but is not working at all in VHF band (7 MHz bandwidth).
Reading the comments inside the code, I figured out that the real
problem could be connected to the formula used to calculate the center
frequency offset in VHF band.

In fact, removing this adjustment fixed the problem:

		if ((priv->cur_fw.type & DTV78) && freq < 470000000)
			offset -= 500000;

This is coherent to what was implemented for the DTV7 firmware by an
Australian user:

		if (priv->cur_fw.type & DTV7)
			offset += 500000;

In the end, the center frequency is the same for all firmwares (DTV7,
DTV8, DTV78) and for both 7 and 8 MHz bandwidth.
Probably, a further offset is hardcoded directly into the firmwares, to
compensate the difference between 7 and 8 MHz bandwidth.

The final code looks clean and simple, and there is no need for any
"magic" adjustment:

		if (priv->cur_fw.type & DTV6)
			offset = 1750000;
		else	/* DTV7 or DTV8 or DTV78 */
			offset = 2750000;

Best regards,
Gianluca Gennari

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-12-11 14:15 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-08 12:56 [PATCH 0/1] xc3028: fix center frequency calculation for DTV78 firmware Gianluca Gennari
2011-12-08 13:00 ` [PATCH 1/1] " Gianluca Gennari
2011-12-10 14:51 ` [PATCH 0/1] " Mauro Carvalho Chehab
2011-12-10 18:01   ` Gianluca Gennari
2011-12-11 14:15     ` Mauro Carvalho Chehab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).