Message ID | 20180516084111.28618-1-suzuki.katsuhiro@socionext.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from <linux-media-owner@vger.kernel.org>) id 1fIs0S-0005jA-Qn; Wed, 16 May 2018 08:41:57 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752740AbeEPIlO (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Wed, 16 May 2018 04:41:14 -0400 Received: from mx.socionext.com ([202.248.49.38]:8654 "EHLO mx.socionext.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752574AbeEPIlN (ORCPT <rfc822;linux-media@vger.kernel.org>); Wed, 16 May 2018 04:41:13 -0400 Received: from unknown (HELO kinkan-ex.css.socionext.com) ([172.31.9.52]) by mx.socionext.com with ESMTP; 16 May 2018 17:41:12 +0900 Received: from mail.mfilter.local (m-filter-2 [10.213.24.62]) by kinkan-ex.css.socionext.com (Postfix) with ESMTP id 351761800E3; Wed, 16 May 2018 17:41:12 +0900 (JST) Received: from 172.31.9.53 (172.31.9.53) by m-FILTER with ESMTP; Wed, 16 May 2018 17:41:12 +0900 Received: from yuzu.css.socionext.com (yuzu [172.31.8.45]) by iyokan.css.socionext.com (Postfix) with ESMTP id C6B9D402D8; Wed, 16 May 2018 17:41:11 +0900 (JST) Received: from aegis.e01.socionext.com (unknown [10.213.134.210]) by yuzu.css.socionext.com (Postfix) with ESMTP id 9467C120134; Wed, 16 May 2018 17:41:11 +0900 (JST) From: Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com> To: Abylay Ospan <aospan@netup.ru>, Mauro Carvalho Chehab <mchehab@kernel.org>, linux-media@vger.kernel.org Cc: Masami Hiramatsu <masami.hiramatsu@linaro.org>, Jassi Brar <jaswinder.singh@linaro.org>, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com> Subject: [PATCH] media: helene: fix tuning frequency of satellite Date: Wed, 16 May 2018 17:41:11 +0900 Message-Id: <20180516084111.28618-1-suzuki.katsuhiro@socionext.com> X-Mailer: git-send-email 2.17.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org |
Commit Message
Katsuhiro Suzuki
May 16, 2018, 8:41 a.m. UTC
This patch fixes tuning frequency of satellite to kHz. That as same
as terrestrial one.
Signed-off-by: Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com>
---
drivers/media/dvb-frontends/helene.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
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@socionext.com>: > This patch fixes tuning frequency of satellite to kHz. That as same > as terrestrial one. > > Signed-off-by: Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com> > --- > 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 >
Hello Abylay, > -----Original Message----- > From: Abylay Ospan <aospan@netup.ru> > Sent: Wednesday, May 16, 2018 7:56 PM > To: Suzuki, Katsuhiro/?? ?? <suzuki.katsuhiro@socionext.com> > Cc: Mauro Carvalho Chehab <mchehab@kernel.org>; linux-media > <linux-media@vger.kernel.org>; Masami Hiramatsu <masami.hiramatsu@linaro.org>; > Jassi Brar <jaswinder.singh@linaro.org>; linux-arm-kernel@lists.infradead.org; > linux-kernel@vger.kernel.org > 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 ? > I'm using evaluation boards of my company. So it's not exist in market... Tuner module is SONY SUT-PJ series for ISDB-S/ISDB-T (not for DVB). Regards, -- Katsuhiro Suzuki > 2018-05-16 4:41 GMT-04:00 Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com > <mailto:suzuki.katsuhiro@socionext.com> >: > > > This patch fixes tuning frequency of satellite to kHz. That as same > as terrestrial one. > > Signed-off-by: Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com > <mailto:suzuki.katsuhiro@socionext.com> > > --- > 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 <http://www.netup.tv/>
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@netup.ru> > Sent: Wednesday, May 16, 2018 7:58 PM > To: Suzuki, Katsuhiro/?? ?? <suzuki.katsuhiro@socionext.com> > Cc: Mauro Carvalho Chehab <mchehab@kernel.org>; linux-media > <linux-media@vger.kernel.org>; Masami Hiramatsu <masami.hiramatsu@linaro.org>; > Jassi Brar <jaswinder.singh@linaro.org>; linux-arm-kernel@lists.infradead.org; > linux-kernel@vger.kernel.org > 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@socionext.com>: > > This patch fixes tuning frequency of satellite to kHz. That as same > > as terrestrial one. > > > > Signed-off-by: Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com> > > --- > > 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
ok, got it ! 2018-05-31 4:43 GMT-04:00 Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com>: > 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@netup.ru> >> Sent: Wednesday, May 16, 2018 7:58 PM >> To: Suzuki, Katsuhiro/?? ?? <suzuki.katsuhiro@socionext.com> >> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>; linux-media >> <linux-media@vger.kernel.org>; Masami Hiramatsu <masami.hiramatsu@linaro.org>; >> Jassi Brar <jaswinder.singh@linaro.org>; linux-arm-kernel@lists.infradead.org; >> linux-kernel@vger.kernel.org >> 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@socionext.com>: >> > This patch fixes tuning frequency of satellite to kHz. That as same >> > as terrestrial one. >> > >> > Signed-off-by: Katsuhiro Suzuki <suzuki.katsuhiro@socionext.com> >> > --- >> > 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 > >
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;