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
