Dark Shadow <shadowofdarkness@xxxxxxxxx> wrote:
>On Wed, Jun 8, 2011 at 6:24 PM, Andy Walls <awalls@xxxxxxxxxxxxxxxx>
>wrote:
>> On Wed, 2011-06-08 at 13:18 -0600, Dark Shadow wrote:
>>> On Wed, Jun 8, 2011 at 4:19 AM, Andy Walls <awalls@xxxxxxxxxxxxxxxx>
>wrote:
>>> > Dark Shadow <shadowofdarkness@xxxxxxxxx> wrote:
>>> >
>>> >>I have a capture card that was sold as a Hauppauge HVR-1250
>(according
>>> >>to the box) that I am trying to use but I am having trouble
>getting
>>> >>all it's features at once. When I leave it auto detected by the
>module
>>> >>I have working TV in MythTV even though it thinks it is a 1270 but
>IR
>>> >>isn't setup.
>>> >>
>>> >>dmesg outputs
>>> >>#modprobe cx23885 enable_885_ir=1
>>> >>[ Â Â7.592714] cx23885 driver version 0.0.2 loaded
>>> >>[ Â Â7.592748] cx23885 0000:07:00.0: PCI INT A -> GSI 17 (level,
>low)
>>> >>-> IRQ 17
>>> >>[ Â Â7.592926] CORE cx23885[0]: subsystem: 0070:2211, board:
>Hauppauge
>>> >>WinTV-HVR1270 [card=18,autodetected]
>>> >>[ Â Â7.728163] IR JVC protocol handler initialized
>>> >>[ Â Â7.738971] tveeprom 0-0050: Hauppauge model 22111, rev C2F5,
>>> >>serial# 6429897
>>> >>[ Â Â7.738974] tveeprom 0-0050: MAC address is 00:0d:fe:62:1c:c9
>>> >>[ Â Â7.738975] tveeprom 0-0050: tuner model is NXP 18271C2 (idx
>155,
>>> >>type 54)
>>> >>[ Â Â7.738977] tveeprom 0-0050: TV standards NTSC(M) ATSC/DVB
>Digital
>>> >>(eeprom 0x88)
>>> >>[ Â Â7.738979] tveeprom 0-0050: audio processor is CX23888 (idx
>40)
>>> >>[ Â Â7.738980] tveeprom 0-0050: decoder processor is CX23888 (idx
>34)
>>> >>[ Â Â7.738982] tveeprom 0-0050: has no radio, has IR receiver, has
>no
>>> >>IR transmitter
>>> >>[ Â Â7.738983] cx23885[0]: hauppauge eeprom: model=22111
>>> >>[ Â Â7.738985] cx23885_dvb_register() allocating 1 frontend(s)
>>> >>[ Â Â7.738991] cx23885[0]: cx23885 based dvb card
>>> >>[ Â Â7.961122] IR Sony protocol handler initialized
>>> >>[ Â Â7.977301] tda18271 1-0060: creating new instance
>>> >>[ Â Â7.979325] TDA18271HD/C2 detected @ 1-0060
>>> >>[ Â Â8.209663] DVB: registering new adapter (cx23885[0])
>>> >>[ Â Â8.209668] DVB: registering adapter 0 frontend 0 (LG
>Electronics
>>> >>LGDT3305 VSB/QAM Frontend)...
>>> >>[ Â Â8.210095] cx23885_dev_checkrevision() Hardware revision =
>0xd0
>>> >>[ Â Â8.210101] cx23885[0]/0: found at 0000:07:00.0, rev: 4, irq:
>17,
>>> >>latency: 0, mmio: 0xf7c00000
>>> >>[ Â Â8.210109] cx23885 0000:07:00.0: setting latency timer to 64
>>> >>[ Â Â8.210186] cx23885 0000:07:00.0: irq 49 for MSI/MSI-X
>>
>>> >>#ir-keytable -a /etc/rc_maps.cfg
>>> >>Old keytable cleared
>>> >>Wrote 136 keycode(s) to driver
>>> >>Protocols changed to RC-5
>>
>>> >>I have heard this should show up as a normal keyboard to the
>system
>>> >>but no button presses cause anything to happen to the system and
>>> >>trying lirc with devinput (with devinput lircd.conf) and then
>opening
>>> >>irw doesn't show any button presses either
>>
>>
>>> > Don't force your card to a 1250, if the driver detects it is a
>1270
>>> with a CX23888 chip. ÂNo need to use the enable_885_ir parameter
>with
>>> a CX23888 chip, either. ÂIt only applies for two board models with
>>> actual CX23885 chips.
>>> >
>>> > Use of IR with the CX23888 chip should be realtively trouble free,
>>> *if* the 1270's IR has been enabled in the driver code. ÂIt likely
>has
>>> not been. ÂI don't have the source code in front of me at the moment
>>> to check.
>>> >
>>> > It shouldn't be hard for anyone to patch a few files in the
>cx23885
>>> driver to add it. ÂPatches are welcome...
>>> >
>>
>>> Under auto detect without the enable_885_ir there is no difference
>so
>>> I can only hope someone will add support for it.
>>
>> I wasn't kidding when I said the patch is sholdn't be hard for
>anyone.
>> It is really, really simple cut-and-paste job. ÂIn fact here is an
>> *untested* patch.
>>
>> Regards,
>> Andy
>>
>> cx23885: Add IR Rx support for HVR-1270 boards
>>
>> Signed-off-by: Andy Walls <awalls@xxxxxxxxxxxxxxxx>
>>
>>
>> diff --git a/drivers/media/video/cx23885/cx23885-cards.c
>b/drivers/media/video/cx23885/cx23885-cards.c
>> index ea88722..5635588 100644
>> --- a/drivers/media/video/cx23885/cx23885-cards.c
>> +++ b/drivers/media/video/cx23885/cx23885-cards.c
>> @@ -1097,12 +1097,19 @@ int cx23885_ir_init(struct cx23885_dev *dev)
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1800:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1200:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1400:
>> - Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1275:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1255:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1210:
>> Â Â Â Â Â Â Â Â/* FIXME: Implement me */
>> Â Â Â Â Â Â Â Âbreak;
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> + Â Â Â Â Â Â Â ret = cx23888_ir_probe(dev);
>> + Â Â Â Â Â Â Â if (ret)
>> + Â Â Â Â Â Â Â Â Â Â Â break;
>> + Â Â Â Â Â Â Â dev->sd_ir = cx23885_find_hw(dev, CX23885_HW_888_IR);
>> + Â Â Â Â Â Â Â v4l2_subdev_call(dev->sd_cx25840, core,
>s_io_pin_config,
>> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âir_rx_pin_cfg_count, ir_rx_pin_cfg);
>> + Â Â Â Â Â Â Â break;
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1290:
>> Â Â Â Â Â Â Â Âret = cx23888_ir_probe(dev);
>> @@ -1156,6 +1163,7 @@ int cx23885_ir_init(struct cx23885_dev *dev)
>> Âvoid cx23885_ir_fini(struct cx23885_dev *dev)
>> Â{
>> Â Â Â Âswitch (dev->board) {
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1290:
>> Â Â Â Â Â Â Â Âcx23885_irq_remove(dev, PCI_MSK_IR);
>> @@ -1199,6 +1207,7 @@ int netup_jtag_io(void *device, int tms, int
>tdi, int read_tdo)
>> Âvoid cx23885_ir_pci_int_enable(struct cx23885_dev *dev)
>> Â{
>> Â Â Â Âswitch (dev->board) {
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1290:
>> Â Â Â Â Â Â Â Âif (dev->sd_ir)
>> @@ -1357,6 +1366,7 @@ void cx23885_card_setup(struct cx23885_dev
>*dev)
>> Â Â Â Âcase CX23885_BOARD_NETUP_DUAL_DVBS2_CI:
>> Â Â Â Âcase CX23885_BOARD_NETUP_DUAL_DVB_T_C_CI_RF:
>> Â Â Â Âcase CX23885_BOARD_COMPRO_VIDEOMATE_E800:
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_MYGICA_X8506:
>> Â Â Â Âcase CX23885_BOARD_MAGICPRO_PROHDTVE2:
>> diff --git a/drivers/media/video/cx23885/cx23885-input.c
>b/drivers/media/video/cx23885/cx23885-input.c
>> index e97cafd..bc28d2c 100644
>> --- a/drivers/media/video/cx23885/cx23885-input.c
>> +++ b/drivers/media/video/cx23885/cx23885-input.c
>> @@ -82,6 +82,7 @@ void cx23885_input_rx_work_handler(struct
>cx23885_dev *dev, u32 events)
>> Â Â Â Â Â Â Â Âreturn;
>>
>> Â Â Â Âswitch (dev->board) {
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1290:
>> Â Â Â Âcase CX23885_BOARD_TEVII_S470:
>> @@ -133,6 +134,7 @@ static int cx23885_input_ir_start(struct
>cx23885_dev *dev)
>>
>> Â Â Â Âv4l2_subdev_call(dev->sd_ir, ir, rx_g_parameters, ¶ms);
>> Â Â Â Âswitch (dev->board) {
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1290:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1250:
>> @@ -257,6 +259,7 @@ int cx23885_input_init(struct cx23885_dev *dev)
>> Â Â Â Â Â Â Â Âreturn -ENODEV;
>>
>> Â Â Â Âswitch (dev->board) {
>> + Â Â Â case CX23885_BOARD_HAUPPAUGE_HVR1270:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1850:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1290:
>> Â Â Â Âcase CX23885_BOARD_HAUPPAUGE_HVR1250:
>>
>>
>>
>>
>>
>
>Thank you I just tested those changes and they work (in getting the IR
>to work to the same level as forcing the card model and not cause
>problems for the video)
>
>I still can't use the remote any more then forcing the card model but
>I have a guess on that. According to everything I have read the
>ir-kbd-i2c module is needed but it never auto loads on my system so I
>am thinking it is just not updated to know about this card and may
>only need the same type of changes as the cx23885 module. (I am
>guessing since I can't even program a Hello World) Is there any chance
>you would be able to see if any easy changes could be done to it.
You don't need ir-kbd-i2c for this ir unit. If you don't have the right keymap loaded, the remote won't work though.
The cx23885 module has a parameter called ir_888_debug (or something similar). Set it to a value of around 7 or so. The cx23885 module will log pulse and space measurments to dmesg or var/log/messages. That way you can at least verify pulses are being processed.
If pulses are not being processed, check that the cable is firmly plugged into the card and that you have fresh batteries in the remote.
I don't have a 1270 of my own, so I can't help too much.
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
