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. ATSC & ClearQAM USB sniffs here: http://trsqr.net/olli/hvr955q/ 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
