All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anders Thomson <aeriksson2@gmail.com>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: tda8290 regression fix
Date: Sat, 15 Sep 2012 19:44:56 +0200	[thread overview]
Message-ID: <5054BE98.8020708@gmail.com> (raw)
In-Reply-To: <5054BD53.7060109@gmail.com>

On 2012-09-15 19:39, Anders Thomson wrote:
> On 2012-09-15 18:34, Mauro Carvalho Chehab wrote:
> >  >   $ cat /TV_CARD.diff
> >  >   diff --git a/drivers/media/common/tuners/tda8290.c
> >  >   b/drivers/media/common/tuners/tda8290.c
> >  >   index 064d14c..498cc7b 100644
> >  >   --- a/drivers/media/common/tuners/tda8290.c
> >  >   +++ b/drivers/media/common/tuners/tda8290.c
> >  >   @@ -635,7 +635,11 @@ static int tda829x_find_tuner(struct dvb_frontend *fe)
> >  >
> >  >                    dvb_attach(tda827x_attach, fe, priv->tda827x_addr,
> >  >                               priv->i2c_props.adap,&priv->cfg);
> >  >   +               tuner_info("ANDERS: setting switch_addr. was 0x%02x, new
> >  >   0x%02x\n",priv->cfg.switch_addr,priv->i2c_props.addr);
> >  >                    priv->cfg.switch_addr = priv->i2c_props.addr;
> >  >   +               priv->cfg.switch_addr = 0xc2 / 2;
> >
> >  No, this is wrong. The I2C address is passed by the bridge driver or by
> >  the tuner_core attachment, being stored at priv->i2c_props.addr.
> >
> >  What's the driver and card you're using?
> >
>
...and here's the modules I'm using
Module                  Size  Used by
hid_sunplus             1321  0
usbhid                 29765  0
uinput                  6426  2
saa7134_alsa            9359  0
tda1004x               12639  1
saa7134_dvb            22092  0
videobuf_dvb            4106  1 saa7134_dvb
dvb_core               78773  1 videobuf_dvb
ir_kbd_i2c              4473  0
tda827x                 8291  2
tda8290                11906  1
tuner                  13649  1
uvcvideo               54515  0
videobuf2_core         15467  1 uvcvideo
snd_hda_codec_realtek    46581  1
saa7134               149350  2 saa7134_alsa,saa7134_dvb
videobuf_dma_sg         6504  3 saa7134_alsa,saa7134_dvb,saa7134
videobuf_core          12866  3 videobuf_dvb,saa7134,videobuf_dma_sg
snd_hda_intel          20144  1
snd_hda_codec          59409  2 snd_hda_codec_realtek,snd_hda_intel
v4l2_common             4558  2 tuner,saa7134
videodev               68383  4 tuner,uvcvideo,saa7134,v4l2_common
snd_usb_audio          81087  0
lirc_dev                9954  0
ir_mce_kbd_decoder      2838  0
parport_pc             27310  0
rc_imon_mce             1349  0
v4l2_compat_ioctl32     6796  1 videodev
ir_rc6_decoder          1946  0
videobuf2_vmalloc       1812  1 uvcvideo
snd_usbmidi_lib        15516  1 snd_usb_audio
tveeprom               12721  1 saa7134
snd_hwdep               5006  2 snd_hda_codec,snd_usb_audio
asus_atk0110            7054  0
parport                24807  1 parport_pc
i2c_piix4               7680  0
videobuf2_memops        1638  1 videobuf2_vmalloc
ir_rc5_decoder          1433  0
imon                   18432  1
rc_core                10749  8 
ir_kbd_i2c,saa7134,ir_mce_kbd_decoder,rc_imon_mce,ir_rc6_decoder,ir_rc5_decoder,imon
pcspkr                  1659  0
atiixp                  2404  0
rtc_cmos                7570  0
sg                     20980  0
snd_rawmidi            14984  1 snd_usbmidi_lib



  reply	other threads:[~2012-09-15 17:45 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-30 11:27 tda8290 regression fix Anders Thomson
2012-09-15 16:34 ` Mauro Carvalho Chehab
2012-09-15 17:39   ` Anders Thomson
2012-09-15 17:44     ` Anders Thomson [this message]
2012-09-15 17:58     ` Mauro Carvalho Chehab
2012-09-15 18:12       ` Anders Thomson
2012-09-15 22:25         ` Mauro Carvalho Chehab
2012-09-16  8:48           ` Anders Thomson
2012-09-18  9:53             ` Mauro Carvalho Chehab
2012-09-19 13:01               ` Anders Thomson
2012-09-19 16:45                 ` Mauro Carvalho Chehab
2012-09-19 19:25                   ` Anders Thomson
     [not found]                     ` <CAGncdOae+VoAAUWz3x84zUA-TCMeMmNONf_ktNFd1p7c-o5H_A@mail.gmail.com>
2012-09-21 14:49                       ` Mauro Carvalho Chehab
     [not found]                         ` <8ed8c988-fa8c-41fc-9f33-cccdceb1b232@email.android.com>
2012-09-23 11:36                           ` Mauro Carvalho Chehab
2012-09-23 17:54                             ` Anders Thomson
2012-09-23 18:14                               ` Mauro Carvalho Chehab
2012-09-23 18:39                               ` Anders Thomson
2012-09-23 21:06                                 ` Anders Thomson
2012-10-01 16:56                                   ` Anders Thomson
2012-11-15 20:43                                     ` Anders Thomson
2012-12-04 11:00                                       ` Anders Thomson
2012-09-15 18:28       ` Anders Thomson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5054BE98.8020708@gmail.com \
    --to=aeriksson2@gmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.