Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-12 Thread Manu Abraham
On Mon, Dec 12, 2011 at 7:11 PM, Antti Palosaari wrote: > On 12/12/2011 02:55 PM, Manu Abraham wrote: >> >> On Mon, Dec 12, 2011 at 6:13 PM, Andreas Oberritter >>  wrote: >>> >>> On 12.12.2011 05:28, Manu Abraham wrote: On Sat, Dec 10, 2011 at 5:17 PM, Antti Palosaari  wrote: > >

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-12 Thread Antti Palosaari
On 12/12/2011 02:55 PM, Manu Abraham wrote: On Mon, Dec 12, 2011 at 6:13 PM, Andreas Oberritter wrote: On 12.12.2011 05:28, Manu Abraham wrote: On Sat, Dec 10, 2011 at 5:17 PM, Antti Palosaari wrote: On 12/10/2011 06:44 AM, Manu Abraham wrote: static int cxd2820r_set_frontend(struct dvb_

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-12 Thread Manu Abraham
On Mon, Dec 12, 2011 at 6:13 PM, Andreas Oberritter wrote: > On 12.12.2011 05:28, Manu Abraham wrote: >> On Sat, Dec 10, 2011 at 5:17 PM, Antti Palosaari wrote: >>> On 12/10/2011 06:44 AM, Manu Abraham wrote:  static int cxd2820r_set_frontend(struct dvb_frontend *fe, >>> >>> [...]

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-12 Thread Andreas Oberritter
On 12.12.2011 05:28, Manu Abraham wrote: > On Sat, Dec 10, 2011 at 5:17 PM, Antti Palosaari wrote: >> On 12/10/2011 06:44 AM, Manu Abraham wrote: >>> >>> static int cxd2820r_set_frontend(struct dvb_frontend *fe, >> >> [...] >>> >>> + switch (c->delivery_system) { >>> + case SYS_DVBT:

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-11 Thread Manu Abraham
On Sat, Dec 10, 2011 at 5:17 PM, Antti Palosaari wrote: > Hello Manu, > That patch looks now much acceptable than the older for my eyes, since you > removed that .set_state() (change from .set_params() to .set_state()) I > criticized. Thanks! > :-) > > On 12/10/2011 06:44 AM, Manu Abraham wrote

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-10 Thread Mauro Carvalho Chehab
On 10-12-2011 11:09, Antti Palosaari wrote: On 12/10/2011 02:47 PM, Mauro Carvalho Chehab wrote: On 10-12-2011 09:47, Antti Palosaari wrote: Hello Manu, That patch looks now much acceptable than the older for my eyes, since you removed that .set_state() (change from .set_params() to .set_state(

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-10 Thread Antti Palosaari
On 12/10/2011 02:47 PM, Mauro Carvalho Chehab wrote: On 10-12-2011 09:47, Antti Palosaari wrote: Hello Manu, That patch looks now much acceptable than the older for my eyes, since you removed that .set_state() (change from .set_params() to .set_state()) I criticized. Thanks! On 12/10/2011 06:4

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-10 Thread Mauro Carvalho Chehab
On 10-12-2011 09:47, Antti Palosaari wrote: Hello Manu, That patch looks now much acceptable than the older for my eyes, since you removed that .set_state() (change from .set_params() to .set_state()) I criticized. Thanks! On 12/10/2011 06:44 AM, Manu Abraham wrote: static int cxd2820r_set_f

Re: v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-10 Thread Antti Palosaari
Hello Manu, That patch looks now much acceptable than the older for my eyes, since you removed that .set_state() (change from .set_params() to .set_state()) I criticized. Thanks! On 12/10/2011 06:44 AM, Manu Abraham wrote: static int cxd2820r_set_frontend(struct dvb_frontend *fe, [...] +

v4 [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities

2011-12-09 Thread Manu Abraham
From 4fdffdcc77228a140c944c20ce2a9f2b6c5b7658 Mon Sep 17 00:00:00 2001 From: Manu Abraham Date: Thu, 24 Nov 2011 20:29:53 +0530 Subject: [PATCH 09/10] CXD2820r: Query DVB frontend delivery capabilities Override default delivery system information provided by FE_GET_INFO, so that applications can