, only a tiny fraction
require this option. That's why I didn't change the default behavior.
Cheers,
Rémi Cardona
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo
Hi Antti,
I realized I forgot to CC you on those patches. Thanks for any further
comments you might have.
Cheers,
Rémi Cardona
--
SmartJog | T: +33 1 5868 6229
27 Blvd Hippolyte Marquès, 94200 Ivry-sur-Seine, France www.smartjog.com
| a TDF Group company
--
To unsubscribe from this list: send
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb-frontends/ds3000.c |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/media/dvb-frontends/ds3000.c
b/drivers/media/dvb-frontends/ds3000.c
index 59184a8..c66d731 100644
--- a/drivers/media
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/usb/dvb-usb/dw2102.c | 36 ++--
1 file changed, 26 insertions(+), 10 deletions(-)
diff --git a/drivers/media/usb/dvb-usb/dw2102.c
b/drivers/media/usb/dvb-usb/dw2102.c
index 9382895..937c744
, as we have recently gotten a few S480 cards from TeVii and we
were missing their firmware, and found modinfo to be lacking the info.
Thanks again for any review/comments.
Cheers,
Rémi Cardona
--
SmartJog | T: +33 1 5868 6229
27 Blvd Hippolyte Marquès, 94200 Ivry-sur-Seine, France
(and not recursion as the comments suggest). However, this open-
coded mechanism is anything but race-free and should have used a proper
mutex.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb-frontends/ds3000.c |9 -
1 file changed, 9 deletions(-)
diff --git a/drivers
-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb-frontends/ds3000.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/media/dvb-frontends/ds3000.c
b/drivers/media/dvb-frontends/ds3000.c
index 675..162faaf 100644
--- a/drivers/media
ds3000_readreg() returns negative values in case of i2c failures. The
old code would simply return 0 when failing to read the 0xb2 register,
misleading ds3000_initfe() into believing that the firmware had been
correctly loaded.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb-frontends/ds3000.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/dvb-frontends/ds3000.c
b/drivers/media/dvb-frontends/ds3000.c
index 4c8ac26..46874c7 100644
--- a/drivers/media/dvb-frontends/ds3000
This brings both ds3000_readreg() and ds3000_tuner_readreg() in line
with ds3000_writereg() and ds3000_tuner_writereg() respectively.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb-frontends/ds3000.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions
On 09/16/2012 03:26 AM, Antti Palosaari wrote:
On 09/14/2012 12:27 PM, Rémi Cardona wrote:
Hi all,
Here is an updated and more complete patch series for ds3000. I've
done my testing on top of kernel 3.5.2 and things look ok so far.
Reviews and comments are again more than welcome.
I
Hi all,
Here is an updated and more complete patch series for ds3000. I've
done my testing on top of kernel 3.5.2 and things look ok so far.
Reviews and comments are again more than welcome.
Cheers,
Rémi Cardona
--
SmartJog | T: +33 1 5868 6229
27 Blvd Hippolyte Marquès, 94200 Ivry-sur-Seine
(and not recursion as the comments suggest). However, this open-
coded mechanism is anything but race-free and should have used a proper
mutex.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c |9 -
1 file changed, 9 deletions(-)
diff --git a/drivers
-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/media/dvb/frontends/ds3000.c
b/drivers/media/dvb/frontends/ds3000.c
index 675..162faaf 100644
--- a/drivers/media
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/dvb/frontends/ds3000.c
b/drivers/media/dvb/frontends/ds3000.c
index 4c8ac26..46874c7 100644
--- a/drivers/media/dvb/frontends/ds3000
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c |9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/media/dvb/frontends/ds3000.c
b/drivers/media/dvb/frontends/ds3000.c
index 970963c..3e0e9de 100644
--- a/drivers
loaded.
This should make it more obvious if the chip is in a weird state.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c |8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/media/dvb/frontends/ds3000.c
b/drivers/media
This brings both ds3000_readreg() and ds3000_tuner_readreg() in line
with ds3000_writereg() and ds3000_tuner_writereg() respectively.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions
On 08/30/2012 06:00 PM, Antti Palosaari wrote:
hmm, looks like ds3000_readreg() logic is still a little bit broken. It
checks count of sent messages and compares it to 2. But if I2C-adapter
sends only 1 message or 3 (which should not be possible) function return
that count instead of
Hi Max,
On 08/31/2012 10:29 AM, nibble.max wrote:
As remember that there is a fault in the tuner register read function in
ds3000.c file.
It will cause the read back value wrong.
Well, using 0x12 works out for most of the cards we have in the wild.
Not knowing what 0x11 / 0x12 means, I'd be
On 08/30/2012 11:36 AM, Rémi Cardona wrote:
Since b9bf2eafaad9c1ef02fb3db38c74568be601a43a, the function
ds3000_firmware_ondemand() is called only once during init. This
locking scheme may have been useful when the firmware was loaded at
each tune.
Furthermore, it looks like this 'lock
(and not recursion as the comments suggest). However, this open-
coded mechanism is anything but race-free and should have used a proper
mutex.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers/media/dvb/frontends/ds3000.c |9 -
1 file changed, 9 deletions(-)
diff --git a/drivers
a patch that completely removes the register check (and thus
always loads the firmware at init time) be a preferred alternative?
Thanks for any pointers/reviews.
Rémi Cardona
--
SmartJog | T: +33 1 5868 6229
27 Blvd Hippolyte Marquès, 94200 Ivry-sur-Seine, France
www.smartjog.com | a TDF Group
ds3000_readreg() returns negative values in case of i2c failures. The
old code would simply return 0 when failing to read the 0xb2 register,
misleading ds3000_initfe() into believing that the firmware had been
correctly loaded.
Signed-off-by: Rémi Cardona remi.card...@smartjog.com
---
drivers
Hi Antti,
On 08/30/2012 03:39 PM, Antti Palosaari wrote:
As I understand firmware downloading failure is coming from the fact
that register read fails = fails to detect if firmware is already
running or not.
Well we actually see 2 cases:
- the register read failure (when ds3000_readreg()
25 matches
Mail list logo