public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: YP WU <yp.wu@mediatek.com>
To: <mchehab@kernel.org>
Cc: <Jason-BF.Huang@mediatek.com>, <Lecopzer.Chen@mediatek.com>,
	<francis.lee@mediatek.com>, <gustavoars@kernel.org>,
	<hverkuil-cisco@xs4all.nl>, <leo.hsiao@mediatek.com>,
	<linux-kernel@vger.kernel.org>, <linux-media@vger.kernel.org>,
	<yp.wu@mediatek.com>
Subject: Re: DVBS Blind scan implementation of Linux DVB
Date: Thu, 2 Sep 2021 18:56:36 +0800	[thread overview]
Message-ID: <20210902105636.14850-1-yp.wu@mediatek.com> (raw)
In-Reply-To: <20210702133326.654ac68d@coco.lan>

Hello, Mr.Mauro,
      Frist of all, thanks for your reply and Sorry to reply late.

According to your comment, we found that it still miss some steps about blindscan
For example, we should get tuner's next IF frequency from demod DD during blindscan.
Also, we should get the number of founded transpoders and detail information of each transponder
To complete DVBS blindscan, we proposed to add element below:

1.Create interface between demod DD and user space
We need to add IOCTL command for user space to indicate dvb_core that this command is related to blindscan
Also, we need to add ops for dvb_core to indicate demod dd to set/get parameters related to blindscan
We propose to add code below:

(1)Add IOCTL cmd in frontend.h:
#define FE_BLINDSCAN                   _IOW('o', 84, enum blindscan_action, struct dtv_frontend_properties *p)
/* 
ioctl command which is related to DVBS blindscan
*/

(2)Add member into dvb_frontend_ops in dvb_frontend.h
int (*blindscan)(enum blindscan_action cmd, struct dtv_frontend_properties *p );
/* 
function which is related to DVBS blindscan
*/

2.Indicate demod DD what kind of blindscan command user space send
During blindscan, user space needs to get or set parameters with demod DD
Ex: User space should get next IF frequency from demod DD and set to tuner.
    If transponders are founded in this tuner step, user space should get the number of transponders and information of each transponder
    User space should start blindscan with start&end frequency

Therefore, we need add enum to indicate demod DD that user space want to get or set parameters during blindscan with demod DD 
We propose to add enum in dvb_frontend.h below:

Enum blindscan_action
{
    BLINDSCAN_SET  //Setting parameters during blindscan (set start/end frequency, continue blindscan)
    BLINDSCAN_GET  //Getting parameters during blindscan (get current tuner frequency, get total founded transponders information)
    BLINDSCAN_END  //current tuner frequency reaches end frequency or user wants to stop blindscan
}

3.Additional properties related to blindscan
(1)We should indicate demod DD the end frequency of blindscan, then demod DD would know when to end blindscan
(2)If transponders are founded in this tuner step, we should get the number of transponders and information of each transponder

We propose to add property command and struct member of dtv_frontend_properties below: 
(1)Add dtv property command in frontend.h
#define DTV_END_FREQUENCY  //Setting end IF frequency of blindscan
#define DTV_TRANSPONDER_NUMBER  //Getting number of founded transponders in this tuner step

(2)Add struct member of dtv_frontend_properties in dvb_frontend.h
u32 end_freq  //End IF frequency of blindscan
u8 TP_num  //Number of founded transponders in this tuner step

Do you have any suggestion? 
Thanks for your help.

BRs,
YP

      reply	other threads:[~2021-09-02 10:56 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-02 10:16 DVBS Blind scan implementation of Linux DVB YP WU
2021-07-02 11:33 ` Mauro Carvalho Chehab
2021-09-02 10:56   ` YP WU [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210902105636.14850-1-yp.wu@mediatek.com \
    --to=yp.wu@mediatek.com \
    --cc=Jason-BF.Huang@mediatek.com \
    --cc=Lecopzer.Chen@mediatek.com \
    --cc=francis.lee@mediatek.com \
    --cc=gustavoars@kernel.org \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=leo.hsiao@mediatek.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox