[linux-dvb] Re: [video4linux-cvs] [hg:v4l-dvb] Add support for
Opera S1- DVB-USB
abraham.manu at gmail.com
Thu Apr 19 22:55:38 CEST 2007
Mauro Carvalho Chehab wrote:
> Em Qui, 2007-04-19 às 16:41 -0400, Michael Krufky escreveu:
>> Marco Gittler wrote:
>>> this patch has applied the hints from mkrufky (dvb_attach,
>>> and also one working rewrite of the i2c addresses stuff to fit the
>>> kernel i2c reqs.
>>> Signed-off-by: Marco Gittler<g.marco at freenet.de>
>>> diff -r c8b73ec18b42 linux/drivers/media/dvb/dvb-usb/opera1.c
>>> --- a/linux/drivers/media/dvb/dvb-usb/opera1.c Thu Apr 19 12:04:50 2007 -0300
>>> +++ b/linux/drivers/media/dvb/dvb-usb/opera1.c Thu Apr 19 20:38:01 2007 +0200
>>> @@ -25,6 +25,13 @@
>>> #define REG_20_SYMBOLRATE_BYTE1 0x20
>>> #define REG_21_SYMBOLRATE_BYTE2 0x21
>>> +#define ADDR_C0_TUNER (0xc0>>1)
>>> +#define ADDR_D0_PLL (0xd0>>1)
>> I don't like these two #define's. These i2c addresses need only be
>> specified once, in the config structs / frontendfoo_attach calls for the
>> tuner / demod.
>> Better to just put them in as constants like all of the other dvb drivers.
> I prefer the way it is. We should really avoid having magic numbers
> inside the code. The alias here helps to know that 0x60 is tuner addres
> and 0x68 the pll.
Following a project's coding styles and conventions is "respecting" a
More information about the linux-dvb