Re: [PATCH] si2157: Add support for delivery system SYS_ATSC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Thu, 30 Oct 2014 08:04:29 +0200
Olli Salonen <olli.salonen@xxxxxx> escreveu:

> Hi Mauro,
> 
> No, for ClearQAM the delivery_system should be set to 0x10 and this
> patch does not include that. At the time of submission of that patch I
> only had the trace from the ATSC case.

Ah, ok. Are you planning to submit a patch for it, and the patches adding
support for HVR-955Q?

> 
> ATSC & ClearQAM USB sniffs here:
> http://trsqr.net/olli/hvr955q/

Thanks!

Regards,
Mauro

> 
> Cheers,
> -olli
> 
> On 29 October 2014 11:08, Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx> wrote:
> > Hi Olli,
> >
> > Em Sun, 17 Aug 2014 08:24:49 +0300
> > Olli Salonen <olli.salonen@xxxxxx> escreveu:
> >
> >> Set the property for delivery system also in case of SYS_ATSC. This
> >> behaviour is observed in the sniffs taken with Hauppauge HVR-955Q
> >> Windows driver.
> >>
> >> Signed-off-by: Olli Salonen <olli.salonen@xxxxxx>
> >> ---
> >>  drivers/media/tuners/si2157.c | 3 +++
> >>  1 file changed, 3 insertions(+)
> >>
> >> diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c
> >> index 6c53edb..3b86d59 100644
> >> --- a/drivers/media/tuners/si2157.c
> >> +++ b/drivers/media/tuners/si2157.c
> >> @@ -239,6 +239,9 @@ static int si2157_set_params(struct dvb_frontend *fe)
> >>               bandwidth = 0x0f;
> >>
> >>       switch (c->delivery_system) {
> >> +     case SYS_ATSC:
> >> +                     delivery_system = 0x00;
> >> +                     break;
> >
> > Did you check if it uses the same delivery system also for clear-QAM?
> >
> > If so, this patch is missing SYS_DVBC_ANNEX_B inside this case.
> >
> > Ah, FYI, I merged the demod used on HVR-955Q at a separate topic branch
> > upstream:
> >         http://git.linuxtv.org/cgit.cgi/media_tree.git/log/?h=lgdt3306a
> >
> > Regards,
> > Mauro
--
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




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux