ok, got it ! 2018-05-31 4:43 GMT-04:00 Katsuhiro Suzuki <suzuki.katsuhiro@xxxxxxxxxxxxx>: > Hello Abylay, > > I got a mistake in this patch. > > DTV_FREQUENCY for satellite delivery systems, the frequency is in 'kHz' not 'Hz', > so original code is correct. Sorry for confusing... > > > Regards, > -- > Katsuhiro Suzuki > > >> -----Original Message----- >> From: Abylay Ospan <aospan@xxxxxxxx> >> Sent: Wednesday, May 16, 2018 7:58 PM >> To: Suzuki, Katsuhiro/鈴木 勝博 <suzuki.katsuhiro@xxxxxxxxxxxxx> >> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>; linux-media >> <linux-media@xxxxxxxxxxxxxxx>; Masami Hiramatsu <masami.hiramatsu@xxxxxxxxxx>; >> Jassi Brar <jaswinder.singh@xxxxxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; >> linux-kernel@xxxxxxxxxxxxxxx >> Subject: Re: [PATCH] media: helene: fix tuning frequency of satellite >> >> True. >> I'm curious but how did it worked before ... >> Which hardware (dvb adapter) are you using ? >> >> 2018-05-16 4:41 GMT-04:00 Katsuhiro Suzuki <suzuki.katsuhiro@xxxxxxxxxxxxx>: >> > This patch fixes tuning frequency of satellite to kHz. That as same >> > as terrestrial one. >> > >> > Signed-off-by: Katsuhiro Suzuki <suzuki.katsuhiro@xxxxxxxxxxxxx> >> > --- >> > drivers/media/dvb-frontends/helene.c | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/drivers/media/dvb-frontends/helene.c >> b/drivers/media/dvb-frontends/helene.c >> > index 04033f0c278b..0a4f312c4368 100644 >> > --- a/drivers/media/dvb-frontends/helene.c >> > +++ b/drivers/media/dvb-frontends/helene.c >> > @@ -523,7 +523,7 @@ static int helene_set_params_s(struct dvb_frontend *fe) >> > enum helene_tv_system_t tv_system; >> > struct dtv_frontend_properties *p = &fe->dtv_property_cache; >> > struct helene_priv *priv = fe->tuner_priv; >> > - int frequencykHz = p->frequency; >> > + int frequencykHz = p->frequency / 1000; >> > uint32_t frequency4kHz = 0; >> > u32 symbol_rate = p->symbol_rate/1000; >> > >> > -- >> > 2.17.0 >> > >> >> >> >> -- >> Abylay Ospan, >> NetUP Inc. >> http://www.netup.tv > > -- Abylay Ospan, NetUP Inc. http://www.netup.tv
