* [PATCH] [media] ngene: fix dvb_pll_attach failure @ 2012-11-27 20:15 Patrice Chotard 2012-12-14 17:28 ` Patrice Chotard 0 siblings, 1 reply; 7+ messages in thread From: Patrice Chotard @ 2012-11-27 20:15 UTC (permalink / raw) To: linux-media, Mauro Carvalho Chehab; +Cc: Patrice Chotard, Frédéric Before dvb_pll_attch call, be sure that drxd demodulator was initialized, otherwise, dvb_pll_attach() will always failed. In dvb_pll_attach(), first thing done is to enable the I2C gate control in order to probe the pll by performing a read access. As demodulator was not initialized, every i2c access failed. Reported-by: frederic.mantegazza@gbiloba.org Signed-off-by: Patrice Chotard <patricechotard@free.fr> --- drivers/media/pci/ngene/ngene-cards.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/pci/ngene/ngene-cards.c b/drivers/media/pci/ngene/ngene-cards.c index 96a13ed..e2192db 100644 --- a/drivers/media/pci/ngene/ngene-cards.c +++ b/drivers/media/pci/ngene/ngene-cards.c @@ -328,6 +328,8 @@ static int demod_attach_drxd(struct ngene_channel *chan) return -ENODEV; } + /* initialized the DRXD demodulator */ + chan->fe->ops.init(chan->fe); if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, &chan->i2c_adapter, feconf->pll_type)) { -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] [media] ngene: fix dvb_pll_attach failure 2012-11-27 20:15 [PATCH] [media] ngene: fix dvb_pll_attach failure Patrice Chotard @ 2012-12-14 17:28 ` Patrice Chotard 2012-12-14 17:55 ` Antti Palosaari 0 siblings, 1 reply; 7+ messages in thread From: Patrice Chotard @ 2012-12-14 17:28 UTC (permalink / raw) To: linux-media, Mauro Carvalho Chehab; +Cc: Frédéric, Patrice Chotard Before dvb_pll_attch call, be sure that drxd demodulator was initialized, otherwise, dvb_pll_attach() will always failed. In dvb_pll_attach(), first thing done is to enable the I2C gate control in order to probe the pll by performing a read access. As demodulator was not initialized, every i2c access failed. Reported-by: frederic.mantegazza@gbiloba.org Signed-off-by: Patrice Chotard <patricechotard@free.fr> --- drivers/media/pci/ngene/ngene-cards.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/pci/ngene/ngene-cards.c b/drivers/media/pci/ngene/ngene-cards.c index 96a13ed..e2192db 100644 --- a/drivers/media/pci/ngene/ngene-cards.c +++ b/drivers/media/pci/ngene/ngene-cards.c @@ -328,6 +328,8 @@ static int demod_attach_drxd(struct ngene_channel *chan) return -ENODEV; } + /* initialized the DRXD demodulator */ + chan->fe->ops.init(chan->fe); if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, &chan->i2c_adapter, feconf->pll_type)) { -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] [media] ngene: fix dvb_pll_attach failure 2012-12-14 17:28 ` Patrice Chotard @ 2012-12-14 17:55 ` Antti Palosaari 2012-12-14 22:31 ` Patrice Chotard 0 siblings, 1 reply; 7+ messages in thread From: Antti Palosaari @ 2012-12-14 17:55 UTC (permalink / raw) To: Patrice Chotard; +Cc: linux-media, Mauro Carvalho Chehab, Frédéric On 12/14/2012 07:28 PM, Patrice Chotard wrote: > Before dvb_pll_attch call, be sure that drxd demodulator was > initialized, otherwise, dvb_pll_attach() will always failed. > > In dvb_pll_attach(), first thing done is to enable the I2C gate > control in order to probe the pll by performing a read access. > As demodulator was not initialized, every i2c access failed. > > Reported-by: frederic.mantegazza@gbiloba.org > Signed-off-by: Patrice Chotard <patricechotard@free.fr> > --- > drivers/media/pci/ngene/ngene-cards.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/media/pci/ngene/ngene-cards.c > b/drivers/media/pci/ngene/ngene-cards.c > index 96a13ed..e2192db 100644 > --- a/drivers/media/pci/ngene/ngene-cards.c > +++ b/drivers/media/pci/ngene/ngene-cards.c > @@ -328,6 +328,8 @@ static int demod_attach_drxd(struct ngene_channel *chan) > return -ENODEV; > } > > + /* initialized the DRXD demodulator */ > + chan->fe->ops.init(chan->fe); > if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, > &chan->i2c_adapter, > feconf->pll_type)) { > I don't like that as this causes again more deviation against normal procedures. If gate open is needed (for probe or id check?) then pll/tuner attach should open it. If that is not easily possible then calling gate_control() before pll attach is allowed. init() is very, very, bad as generally starts whole chip => starts eating power etc. Even better would be to let whole gate-control to responsibility of DVB-core, but unfortunately current situation is quite mess. Gate is operated sometimes by DVB-core (like for init/sleep) and for some cases it is left for responsibility of tuner driver. So on real life there is mixed solutions and for init/sleep gate is even double controlled. regards Antti -- http://palosaari.fi/ ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] [media] ngene: fix dvb_pll_attach failure 2012-12-14 17:55 ` Antti Palosaari @ 2012-12-14 22:31 ` Patrice Chotard 2012-12-14 23:02 ` Antti Palosaari 0 siblings, 1 reply; 7+ messages in thread From: Patrice Chotard @ 2012-12-14 22:31 UTC (permalink / raw) To: Antti Palosaari; +Cc: linux-media, Mauro Carvalho Chehab, Frédéric Le 14/12/2012 18:55, Antti Palosaari a écrit : > On 12/14/2012 07:28 PM, Patrice Chotard wrote: >> Before dvb_pll_attch call, be sure that drxd demodulator was >> initialized, otherwise, dvb_pll_attach() will always failed. >> >> In dvb_pll_attach(), first thing done is to enable the I2C gate >> control in order to probe the pll by performing a read access. >> As demodulator was not initialized, every i2c access failed. >> >> Reported-by: frederic.mantegazza@gbiloba.org >> Signed-off-by: Patrice Chotard <patricechotard@free.fr> >> --- >> drivers/media/pci/ngene/ngene-cards.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/media/pci/ngene/ngene-cards.c >> b/drivers/media/pci/ngene/ngene-cards.c >> index 96a13ed..e2192db 100644 >> --- a/drivers/media/pci/ngene/ngene-cards.c >> +++ b/drivers/media/pci/ngene/ngene-cards.c >> @@ -328,6 +328,8 @@ static int demod_attach_drxd(struct ngene_channel >> *chan) >> return -ENODEV; >> } >> >> + /* initialized the DRXD demodulator */ >> + chan->fe->ops.init(chan->fe); >> if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, >> &chan->i2c_adapter, >> feconf->pll_type)) { >> > > I don't like that as this causes again more deviation against normal > procedures. If gate open is needed (for probe or id check?) then > pll/tuner attach should open it. If that is not easily possible then > calling gate_control() before pll attach is allowed. init() is very, > very, bad as generally starts whole chip => starts eating power etc. > > > Even better would be to let whole gate-control to responsibility of > DVB-core, but unfortunately current situation is quite mess. Gate is > operated sometimes by DVB-core (like for init/sleep) and for some cases > it is left for responsibility of tuner driver. So on real life there is > mixed solutions and for init/sleep gate is even double controlled. > > > regards > Antti > Hi Antti, Unfortunately, opening gate doesn't work with drxd demodulator before its initialization (drxd_init() call). Also, in dvb_pll_attach(), first thing done is opening the gate control. I was aware by calling init() directly in demod_attach_drxd() was not good at the power consumption point of view, but it was the only solution to make it work. To stop the power consumption, it would be possible to call the frontends : } + /* initialized the DRXD demodulator */ + chan->fe->ops.init(chan->fe); if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, &chan->i2c_adapter, feconf->pll_type)) { pr_err("No pll(%d) found!\n", feconf->pll_type); return -ENODEV; } + chan->fe->ops.sleep(chan->fe); return 0; } Thanks for your feedback. Regards Patrice ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] [media] ngene: fix dvb_pll_attach failure 2012-12-14 22:31 ` Patrice Chotard @ 2012-12-14 23:02 ` Antti Palosaari 2012-12-14 23:26 ` Devin Heitmueller 2012-12-15 10:22 ` Patrice Chotard 0 siblings, 2 replies; 7+ messages in thread From: Antti Palosaari @ 2012-12-14 23:02 UTC (permalink / raw) To: Patrice Chotard; +Cc: linux-media, Mauro Carvalho Chehab, Frédéric On 12/15/2012 12:31 AM, Patrice Chotard wrote: > Le 14/12/2012 18:55, Antti Palosaari a écrit : >> On 12/14/2012 07:28 PM, Patrice Chotard wrote: >>> Before dvb_pll_attch call, be sure that drxd demodulator was >>> initialized, otherwise, dvb_pll_attach() will always failed. >>> >>> In dvb_pll_attach(), first thing done is to enable the I2C gate >>> control in order to probe the pll by performing a read access. >>> As demodulator was not initialized, every i2c access failed. >>> >>> Reported-by: frederic.mantegazza@gbiloba.org >>> Signed-off-by: Patrice Chotard <patricechotard@free.fr> >>> --- >>> drivers/media/pci/ngene/ngene-cards.c | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/drivers/media/pci/ngene/ngene-cards.c >>> b/drivers/media/pci/ngene/ngene-cards.c >>> index 96a13ed..e2192db 100644 >>> --- a/drivers/media/pci/ngene/ngene-cards.c >>> +++ b/drivers/media/pci/ngene/ngene-cards.c >>> @@ -328,6 +328,8 @@ static int demod_attach_drxd(struct ngene_channel >>> *chan) >>> return -ENODEV; >>> } >>> >>> + /* initialized the DRXD demodulator */ >>> + chan->fe->ops.init(chan->fe); >>> if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, >>> &chan->i2c_adapter, >>> feconf->pll_type)) { >>> >> >> I don't like that as this causes again more deviation against normal >> procedures. If gate open is needed (for probe or id check?) then >> pll/tuner attach should open it. If that is not easily possible then >> calling gate_control() before pll attach is allowed. init() is very, >> very, bad as generally starts whole chip => starts eating power etc. >> >> >> Even better would be to let whole gate-control to responsibility of >> DVB-core, but unfortunately current situation is quite mess. Gate is >> operated sometimes by DVB-core (like for init/sleep) and for some cases >> it is left for responsibility of tuner driver. So on real life there is >> mixed solutions and for init/sleep gate is even double controlled. >> >> >> regards >> Antti >> > > Hi Antti, > > Unfortunately, opening gate doesn't work with drxd demodulator before > its initialization (drxd_init() call). > Also, in dvb_pll_attach(), first thing done is opening the gate control. > > I was aware by calling init() directly in demod_attach_drxd() was not > good at the power consumption point of view, but it was the only > solution to make it work. > > To stop the power consumption, it would be possible to call the frontends : > > } > > + /* initialized the DRXD demodulator */ > + chan->fe->ops.init(chan->fe); > if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, > &chan->i2c_adapter, > feconf->pll_type)) { > pr_err("No pll(%d) found!\n", feconf->pll_type); > return -ENODEV; > } > + chan->fe->ops.sleep(chan->fe); > return 0; > } > > Thanks for your feedback. That one is better solution... but it is clearly DRXD driver bug - it should offer working I2C gate control after attach() :-( I looked quickly DRXD driver and there is clearly some other places to enhance too. But I suspect there is no maintainer, nor interest from anyone to start fix things properly, so only reasonable way is to add that hack to get it working... Honestly, I hate this kind of hacks :/ That makes our live hard on long run. It goes slowly more and more hard to make any core changes as regressions will happen due to this kind of hacks. So send new patch which put demod chip sleeping after tuner attach. Or even better, find out what is minimal set of commands needed do execute during attach in order to offer working I2C gate (I suspect firmware load is needed). regards Antti -- http://palosaari.fi/ ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] [media] ngene: fix dvb_pll_attach failure 2012-12-14 23:02 ` Antti Palosaari @ 2012-12-14 23:26 ` Devin Heitmueller 2012-12-15 10:22 ` Patrice Chotard 1 sibling, 0 replies; 7+ messages in thread From: Devin Heitmueller @ 2012-12-14 23:26 UTC (permalink / raw) To: Antti Palosaari Cc: Patrice Chotard, Linux Media Mailing List, Mauro Carvalho Chehab, Frédéric On Fri, Dec 14, 2012 at 6:02 PM, Antti Palosaari <crope@iki.fi> wrote: > That one is better solution... > > but it is clearly DRXD driver bug - it should offer working I2C gate control > after attach() :-( I looked quickly DRXD driver and there is clearly some > other places to enhance too. But I suspect there is no maintainer, nor > interest from anyone to start fix things properly, so only reasonable way is > to add that hack to get it working... > > Honestly, I hate this kind of hacks :/ That makes our live hard on long run. > It goes slowly more and more hard to make any core changes as regressions > will happen due to this kind of hacks. > > So send new patch which put demod chip sleeping after tuner attach. Or even > better, find out what is minimal set of commands needed do execute during > attach in order to offer working I2C gate (I suspect firmware load is > needed). Opening the gate at the end of the attach callback should be a trivial exercise. Should just be a call to drxd_config_i2c(fe, enable) at the end of drxd_attach(). Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] [media] ngene: fix dvb_pll_attach failure 2012-12-14 23:02 ` Antti Palosaari 2012-12-14 23:26 ` Devin Heitmueller @ 2012-12-15 10:22 ` Patrice Chotard 1 sibling, 0 replies; 7+ messages in thread From: Patrice Chotard @ 2012-12-15 10:22 UTC (permalink / raw) To: Antti Palosaari; +Cc: linux-media, Mauro Carvalho Chehab, Frédéric Le 15/12/2012 00:02, Antti Palosaari a écrit : > On 12/15/2012 12:31 AM, Patrice Chotard wrote: >> Le 14/12/2012 18:55, Antti Palosaari a écrit : >>> On 12/14/2012 07:28 PM, Patrice Chotard wrote: >>>> Before dvb_pll_attch call, be sure that drxd demodulator was >>>> initialized, otherwise, dvb_pll_attach() will always failed. >>>> >>>> In dvb_pll_attach(), first thing done is to enable the I2C gate >>>> control in order to probe the pll by performing a read access. >>>> As demodulator was not initialized, every i2c access failed. >>>> >>>> Reported-by: frederic.mantegazza@gbiloba.org >>>> Signed-off-by: Patrice Chotard <patricechotard@free.fr> >>>> --- >>>> drivers/media/pci/ngene/ngene-cards.c | 2 ++ >>>> 1 file changed, 2 insertions(+) >>>> >>>> diff --git a/drivers/media/pci/ngene/ngene-cards.c >>>> b/drivers/media/pci/ngene/ngene-cards.c >>>> index 96a13ed..e2192db 100644 >>>> --- a/drivers/media/pci/ngene/ngene-cards.c >>>> +++ b/drivers/media/pci/ngene/ngene-cards.c >>>> @@ -328,6 +328,8 @@ static int demod_attach_drxd(struct ngene_channel >>>> *chan) >>>> return -ENODEV; >>>> } >>>> >>>> + /* initialized the DRXD demodulator */ >>>> + chan->fe->ops.init(chan->fe); >>>> if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, >>>> &chan->i2c_adapter, >>>> feconf->pll_type)) { >>>> >>> >>> I don't like that as this causes again more deviation against normal >>> procedures. If gate open is needed (for probe or id check?) then >>> pll/tuner attach should open it. If that is not easily possible then >>> calling gate_control() before pll attach is allowed. init() is very, >>> very, bad as generally starts whole chip => starts eating power etc. >>> >>> >>> Even better would be to let whole gate-control to responsibility of >>> DVB-core, but unfortunately current situation is quite mess. Gate is >>> operated sometimes by DVB-core (like for init/sleep) and for some cases >>> it is left for responsibility of tuner driver. So on real life there is >>> mixed solutions and for init/sleep gate is even double controlled. >>> >>> >>> regards >>> Antti >>> >> >> Hi Antti, >> >> Unfortunately, opening gate doesn't work with drxd demodulator before >> its initialization (drxd_init() call). >> Also, in dvb_pll_attach(), first thing done is opening the gate control. >> >> I was aware by calling init() directly in demod_attach_drxd() was not >> good at the power consumption point of view, but it was the only >> solution to make it work. >> >> To stop the power consumption, it would be possible to call the >> frontends : >> >> } >> >> + /* initialized the DRXD demodulator */ >> + chan->fe->ops.init(chan->fe); >> if (!dvb_attach(dvb_pll_attach, chan->fe, feconf->pll_address, >> &chan->i2c_adapter, >> feconf->pll_type)) { >> pr_err("No pll(%d) found!\n", feconf->pll_type); >> return -ENODEV; >> } >> + chan->fe->ops.sleep(chan->fe); >> return 0; >> } >> >> Thanks for your feedback. > > That one is better solution... > > but it is clearly DRXD driver bug - it should offer working I2C gate > control after attach() :-( I looked quickly DRXD driver and there is > clearly some other places to enhance too. But I suspect there is no > maintainer, nor interest from anyone to start fix things properly, so > only reasonable way is to add that hack to get it working... > > Honestly, I hate this kind of hacks :/ That makes our live hard on long > run. It goes slowly more and more hard to make any core changes as > regressions will happen due to this kind of hacks. > > So send new patch which put demod chip sleeping after tuner attach. Or > even better, find out what is minimal set of commands needed do execute > during attach in order to offer working I2C gate (I suspect firmware > load is needed). > > > regards > Antti > I fully understand your point of view. I don't promise anything, but as you advice, i will try to find the minimal set of commands to make the gate control working during attach. Patrice ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-12-15 10:22 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-11-27 20:15 [PATCH] [media] ngene: fix dvb_pll_attach failure Patrice Chotard 2012-12-14 17:28 ` Patrice Chotard 2012-12-14 17:55 ` Antti Palosaari 2012-12-14 22:31 ` Patrice Chotard 2012-12-14 23:02 ` Antti Palosaari 2012-12-14 23:26 ` Devin Heitmueller 2012-12-15 10:22 ` Patrice Chotard
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.