Re: [PATCH 1/3] m88brs2000 DVB-S frontend and tuner module.
- Subject: Re: [PATCH 1/3] m88brs2000 DVB-S frontend and tuner module.
- From: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
- Date: Thu, 26 Jan 2012 14:56:01 -0200
- Cc: linux-media@xxxxxxxxxxxxxxx
- In-reply-to: <1327228731.2540.3.camel@tvbox>
- User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111222 Thunderbird/9.0
Em 22-01-2012 08:38, Malcolm Priestley escreveu:
> Support for m88brs2000 chip used in lmedm04 driver.
>
> Note there are still lock problems.
>
> Slow channel change due to the large block of registers sent in set_frontend.
>
> Signed-off-by: Malcolm Priestley <tvboxspy@xxxxxxxxx>
> ---
...
> +static int m88rs2000_set_property(struct dvb_frontend *fe,
> + struct dtv_property *p)
> +{
> + dprintk("%s(..)\n", __func__);
> + return 0;
> +}
> +
> +static int m88rs2000_get_property(struct dvb_frontend *fe,
> + struct dtv_property *p)
> +{
> + dprintk("%s(..)\n", __func__);
> + return 0;
> +}
...
Just don't implement set_property/get_property if you're not using them.
Except for that, the code looks ok on my eyes.
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
[Linux Input]
[Video for Linux]
[Mplayer Users]
[Linux USB Devel]
[Linux Audio Users]
[Photos]
[Yosemite Photos]
[Linux Kernel]
[Linux SCSI]
[XFree86]
[Devices]
[Yosemite Backpacking]