Hi Oldrich,
On Thu, 9 Apr 2009 23:12:51 +0200, Oldrich Jedlicka wrote:
> On Saturday 04 of April 2009 at 14:31:37, Jean Delvare wrote:
> > Now that we instantiate I2C IR devices explicitly, we can skip probing
> > altogether on boards where the I2C IR device address is known. The
> > AVerMedia Cardbus are two of these boards.
> >
> > Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx>
> > ---
> > linux/drivers/media/video/saa7134/saa7134-input.c | 35
> > +++------------------ 1 file changed, 5 insertions(+), 30 deletions(-)
> >
> > ---
> > v4l-dvb.orig/linux/drivers/media/video/saa7134/saa7134-input.c 2009-04-04
> > 10:41:44.000000000 +0200 +++
> > v4l-dvb/linux/drivers/media/video/saa7134/saa7134-input.c 2009-04-04
> > 10:47:10.000000000 +0200 @@ -691,22 +691,6 @@ void
> > saa7134_probe_i2c_ir(struct saa7134
> > I2C_CLIENT_END
> > };
> >
> > - unsigned char subaddr, data;
> > - struct i2c_msg msg_avermedia[] = { {
> > - .addr = 0x40,
> > - .flags = 0,
> > - .len = 1,
> > - .buf = &subaddr,
> > - }, {
> > - .addr = 0x40,
> > - .flags = I2C_M_RD,
> > - .len = 1,
> > - .buf = &data,
> > - } };
> > -
> > - struct i2c_client *client;
> > - int rc;
> > -
> > if (disable_ir) {
> > dprintk("IR has been disabled, not probing for i2c remote\n");
> > return;
> > @@ -753,6 +737,10 @@ void saa7134_probe_i2c_ir(struct saa7134
> > init_data.get_key = get_key_beholdm6xx;
> > init_data.ir_codes = ir_codes_behold;
> > break;
> > + case SAA7134_BOARD_AVERMEDIA_CARDBUS:
> > + case SAA7134_BOARD_AVERMEDIA_CARDBUS_506:
> > + info.addr = 0x40;
> > + break;
> > }
>
> The Avermedia Cardbus (E500 - SAA7134_BOARD_AVERMEDIA_CARDBUS) doesn't have
> remote control as far as I know. The first model was Cardbus Plus (E501R)
> which is not supported (yet), but Grigory Milev reported that it works with
> small patching. I plan to send patches after some more testing.
OK, I've removed case SAA7134_BOARD_AVERMEDIA_CARDBUS from my patch,
thanks for letting me know.
--
Jean Delvare
--
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
