Re: [PATCH] Add support for Prof Revolution DVB-S2 8000 PCI-E card

2012-08-01 Thread Mariusz Bialonczyk 
On 07/26/2012 08:08 PM, Mariusz Bialonczyk wrote:
 The device is based on STV0903 demodulator, STB6100 tuner
 and CX23885 chipset; subsystem id: 8000:3034
 
 Signed-off-by: Mariusz Bialonczyk ma...@skyboo.net
Igor, can I please get your ACK on this patch?
I was contacting proftuners for information who is the author of the prof8000 
patch (on their website), but i haven't got a clear answer, they only point me 
your hg repo.

regards,
-- 
Mariusz Białończyk
jabber/e-mail: ma...@skyboo.net
http://manio.skyboo.net
--
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-info.html

Re: [PATCH] Add support for Prof Revolution DVB-S2 8000 PCI-E card

2012-07-31 Thread Mariusz Bialonczyk 
On 07/31/2012 05:08 AM, Mauro Carvalho Chehab wrote:
 Em 26-07-2012 15:08, Mariusz Bialonczyk escreveu:
 The device is based on STV0903 demodulator, STB6100 tuner
 and CX23885 chipset; subsystem id: 8000:3034

 Signed-off-by: Mariusz Bialonczyk ma...@skyboo.net
 
 This is the second time I see this patch. It seems very similar to the
 one sent by Wojciech. So, who is the author of this patch?
Hi!
Wojciech Myrda's patch was based on original producer patch from here:
http://www.proftuners.com/sites/default/files/prof8000_0.patch

From my diagnose (diff original and his patch) he rebased it to take
into account later kernel sources.

You ask him to post it again with Signed-off-by line, but he doesn't
respond for over a year (at least to linux media mailing list).

My patch uses stv090x frontend instead and I assumed that I can
ommit additional signed-off-by lines in this case.

If it is not ok, please tell me and I will prepare second patch
with additional Signed-off-by lines.

regards!
-- 
Mariusz Białończyk
jabber/e-mail: ma...@skyboo.net
http://manio.skyboo.net
--
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-info.html

Re: [PATCH] Add support for Prof Revolution DVB-S2 8000 PCI-E card

2012-07-31 Thread Mauro Carvalho Chehab 
Em 31-07-2012 05:32, Mariusz Bialonczyk escreveu:
 On 07/31/2012 05:08 AM, Mauro Carvalho Chehab wrote:
 Em 26-07-2012 15:08, Mariusz Bialonczyk escreveu:
 The device is based on STV0903 demodulator, STB6100 tuner
 and CX23885 chipset; subsystem id: 8000:3034

 Signed-off-by: Mariusz Bialonczyk ma...@skyboo.net

 This is the second time I see this patch. It seems very similar to the
 one sent by Wojciech. So, who is the author of this patch?
 Hi!
 Wojciech Myrda's patch was based on original producer patch from here:
 http://www.proftuners.com/sites/default/files/prof8000_0.patch

Hmm... it seems that the original producer worked for the manufacturers
of this board, as the patch is posted there...

An ack from them to allow adding it at Kernel seems to be required.
 
  From my diagnose (diff original and his patch) he rebased it to take
 into account later kernel sources.
 
 You ask him to post it again with Signed-off-by line, but he doesn't
 respond for over a year (at least to linux media mailing list).
 
 My patch uses stv090x frontend instead and I assumed that I can
 ommit additional signed-off-by lines in this case.

Rebased patches should be mentioning their origin. In a matter of
fact, you need to add a From: field at the beginning of the body of
the email, with the original author there, even without his SOB (Signed-off-by).

It should be noticed that SOB means:

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or

(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

So, in the case of a patch made by someone else who didn't sign, the SOB means:

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file;

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.


 
 If it is not ok, please tell me and I will prepare second patch
 with additional Signed-off-by lines.
 
 regards!
 

Regards,
Mauro
--
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-info.html

Re: [PATCH] Add support for Prof Revolution DVB-S2 8000 PCI-E card

2012-07-30 Thread Mauro Carvalho Chehab 
Em 26-07-2012 15:08, Mariusz Bialonczyk escreveu:
 The device is based on STV0903 demodulator, STB6100 tuner
 and CX23885 chipset; subsystem id: 8000:3034
 
 Signed-off-by: Mariusz Bialonczyk ma...@skyboo.net

This is the second time I see this patch. It seems very similar to the
one sent by Wojciech. So, who is the author of this patch?


 ---
   drivers/media/video/cx23885/Kconfig |1 +
   drivers/media/video/cx23885/cx23885-cards.c |   10 +
   drivers/media/video/cx23885/cx23885-dvb.c   |   56 
 +++
   drivers/media/video/cx23885/cx23885.h   |1 +
   4 files changed, 68 insertions(+)
 
 diff --git a/drivers/media/video/cx23885/Kconfig 
 b/drivers/media/video/cx23885/Kconfig
 index b391e9b..510adfe 100644
 --- a/drivers/media/video/cx23885/Kconfig
 +++ b/drivers/media/video/cx23885/Kconfig
 @@ -21,6 +21,7 @@ config VIDEO_CX23885
   select DVB_STV6110 if !DVB_FE_CUSTOMISE
   select DVB_CX24116 if !DVB_FE_CUSTOMISE
   select DVB_STV0900 if !DVB_FE_CUSTOMISE
 + select DVB_STV090x if !DVB_FE_CUSTOMISE
   select DVB_DS3000 if !DVB_FE_CUSTOMISE
   select DVB_STV0367 if !DVB_FE_CUSTOMISE
   select MEDIA_TUNER_MT2131 if !MEDIA_TUNER_CUSTOMISE

You've included 3 files at -dvb:

+#include stv090x.h
+#include stb6100.h
+#include stb6100_cfg.h

but you're selecting only STV090x. It seems you missed stb6100 here.


 diff --git a/drivers/media/video/cx23885/cx23885-cards.c 
 b/drivers/media/video/cx23885/cx23885-cards.c
 index 080e111..50fedff 100644
 --- a/drivers/media/video/cx23885/cx23885-cards.c
 +++ b/drivers/media/video/cx23885/cx23885-cards.c
 @@ -564,6 +564,10 @@ struct cx23885_board cx23885_boards[] = {
   [CX23885_BOARD_TEVII_S471] = {
   .name   = TeVii S471,
   .portb  = CX23885_MPEG_DVB,
 + },
 + [CX23885_BOARD_PROF_8000] = {
 + .name   = Prof Revolution DVB-S2 8000,
 + .portb  = CX23885_MPEG_DVB,
   }
   };
   const unsigned int cx23885_bcount = ARRAY_SIZE(cx23885_boards);
 @@ -776,6 +780,10 @@ struct cx23885_subid cx23885_subids[] = {
   .subvendor = 0xd471,
   .subdevice = 0x9022,
   .card  = CX23885_BOARD_TEVII_S471,
 + }, {
 + .subvendor = 0x8000,
 + .subdevice = 0x3034,
 + .card  = CX23885_BOARD_PROF_8000,
   },
   };
   const unsigned int cx23885_idcount = ARRAY_SIZE(cx23885_subids);
 @@ -1155,6 +1163,7 @@ void cx23885_gpio_setup(struct cx23885_dev *dev)
   cx_set(GP0_IO, 0x00040004);
   break;
   case CX23885_BOARD_TBS_6920:
 + case CX23885_BOARD_PROF_8000:
   cx_write(MC417_CTL, 0x0036);
   cx_write(MC417_OEN, 0x1000);
   cx_set(MC417_RWD, 0x0002);
 @@ -1536,6 +1545,7 @@ void cx23885_card_setup(struct cx23885_dev *dev)
   case CX23885_BOARD_TEVII_S470:
   case CX23885_BOARD_TEVII_S471:
   case CX23885_BOARD_DVBWORLD_2005:
 + case CX23885_BOARD_PROF_8000:
   ts1-gen_ctrl_val  = 0x5; /* Parallel */
   ts1-ts_clk_en_val = 0x1; /* Enable TS_CLK */
   ts1-src_sel_val   = CX23885_SRC_SEL_PARALLEL_MPEG_VIDEO;
 diff --git a/drivers/media/video/cx23885/cx23885-dvb.c 
 b/drivers/media/video/cx23885/cx23885-dvb.c
 index cd54268..ba046ea 100644
 --- a/drivers/media/video/cx23885/cx23885-dvb.c
 +++ b/drivers/media/video/cx23885/cx23885-dvb.c
 @@ -63,6 +63,9 @@
   #include stv0367.h
   #include drxk.h
   #include mt2063.h
 +#include stv090x.h
 +#include stb6100.h
 +#include stb6100_cfg.h
   
   static unsigned int debug;
   
 @@ -489,6 +492,42 @@ static struct xc5000_config mygica_x8506_xc5000_config = 
 {
   .if_khz = 5380,
   };
   
 +static struct stv090x_config prof_8000_stv090x_config = {
 +.device = STV0903,
 +.demod_mode = STV090x_SINGLE,
 +.clk_mode   = STV090x_CLK_EXT,
 +.xtal   = 2700,
 +.address= 0x6A,
 +.ts1_mode   = STV090x_TSMODE_PARALLEL_PUNCTURED,
 +.repeater_level = STV090x_RPTLEVEL_64,
 +.adc1_range = STV090x_ADC_2Vpp,
 +.diseqc_envelope_mode   = false,
 +
 +.tuner_get_frequency= stb6100_get_frequency,
 +.tuner_set_frequency= stb6100_set_frequency,
 +.tuner_set_bandwidth= stb6100_set_bandwidth,
 +.tuner_get_bandwidth= stb6100_get_bandwidth,
 +};
 +
 +static struct stb6100_config prof_8000_stb6100_config = {
 + .tuner_address = 0x60,
 + .refclock = 2700,
 +};
 +
 +static int p8000_set_voltage(struct dvb_frontend *fe, fe_sec_voltage_t 
 voltage)
 +{
 + struct cx23885_tsport *port = fe-dvb-priv;
 + struct cx23885_dev *dev = port-dev;
 +
 + if (voltage == SEC_VOLTAGE_18)
 + cx_write(MC417_RWD, 0x1e00);
 + else if

[PATCH] Add support for Prof Revolution DVB-S2 8000 PCI-E card

2012-07-26 Thread Mariusz Bialonczyk 
The device is based on STV0903 demodulator, STB6100 tuner
and CX23885 chipset; subsystem id: 8000:3034

Signed-off-by: Mariusz Bialonczyk ma...@skyboo.net
---
 drivers/media/video/cx23885/Kconfig |1 +
 drivers/media/video/cx23885/cx23885-cards.c |   10 +
 drivers/media/video/cx23885/cx23885-dvb.c   |   56 +++
 drivers/media/video/cx23885/cx23885.h   |1 +
 4 files changed, 68 insertions(+)

diff --git a/drivers/media/video/cx23885/Kconfig 
b/drivers/media/video/cx23885/Kconfig
index b391e9b..510adfe 100644
--- a/drivers/media/video/cx23885/Kconfig
+++ b/drivers/media/video/cx23885/Kconfig
@@ -21,6 +21,7 @@ config VIDEO_CX23885
select DVB_STV6110 if !DVB_FE_CUSTOMISE
select DVB_CX24116 if !DVB_FE_CUSTOMISE
select DVB_STV0900 if !DVB_FE_CUSTOMISE
+   select DVB_STV090x if !DVB_FE_CUSTOMISE
select DVB_DS3000 if !DVB_FE_CUSTOMISE
select DVB_STV0367 if !DVB_FE_CUSTOMISE
select MEDIA_TUNER_MT2131 if !MEDIA_TUNER_CUSTOMISE
diff --git a/drivers/media/video/cx23885/cx23885-cards.c 
b/drivers/media/video/cx23885/cx23885-cards.c
index 080e111..50fedff 100644
--- a/drivers/media/video/cx23885/cx23885-cards.c
+++ b/drivers/media/video/cx23885/cx23885-cards.c
@@ -564,6 +564,10 @@ struct cx23885_board cx23885_boards[] = {
[CX23885_BOARD_TEVII_S471] = {
.name   = TeVii S471,
.portb  = CX23885_MPEG_DVB,
+   },
+   [CX23885_BOARD_PROF_8000] = {
+   .name   = Prof Revolution DVB-S2 8000,
+   .portb  = CX23885_MPEG_DVB,
}
 };
 const unsigned int cx23885_bcount = ARRAY_SIZE(cx23885_boards);
@@ -776,6 +780,10 @@ struct cx23885_subid cx23885_subids[] = {
.subvendor = 0xd471,
.subdevice = 0x9022,
.card  = CX23885_BOARD_TEVII_S471,
+   }, {
+   .subvendor = 0x8000,
+   .subdevice = 0x3034,
+   .card  = CX23885_BOARD_PROF_8000,
},
 };
 const unsigned int cx23885_idcount = ARRAY_SIZE(cx23885_subids);
@@ -1155,6 +1163,7 @@ void cx23885_gpio_setup(struct cx23885_dev *dev)
cx_set(GP0_IO, 0x00040004);
break;
case CX23885_BOARD_TBS_6920:
+   case CX23885_BOARD_PROF_8000:
cx_write(MC417_CTL, 0x0036);
cx_write(MC417_OEN, 0x1000);
cx_set(MC417_RWD, 0x0002);
@@ -1536,6 +1545,7 @@ void cx23885_card_setup(struct cx23885_dev *dev)
case CX23885_BOARD_TEVII_S470:
case CX23885_BOARD_TEVII_S471:
case CX23885_BOARD_DVBWORLD_2005:
+   case CX23885_BOARD_PROF_8000:
ts1-gen_ctrl_val  = 0x5; /* Parallel */
ts1-ts_clk_en_val = 0x1; /* Enable TS_CLK */
ts1-src_sel_val   = CX23885_SRC_SEL_PARALLEL_MPEG_VIDEO;
diff --git a/drivers/media/video/cx23885/cx23885-dvb.c 
b/drivers/media/video/cx23885/cx23885-dvb.c
index cd54268..ba046ea 100644
--- a/drivers/media/video/cx23885/cx23885-dvb.c
+++ b/drivers/media/video/cx23885/cx23885-dvb.c
@@ -63,6 +63,9 @@
 #include stv0367.h
 #include drxk.h
 #include mt2063.h
+#include stv090x.h
+#include stb6100.h
+#include stb6100_cfg.h
 
 static unsigned int debug;
 
@@ -489,6 +492,42 @@ static struct xc5000_config mygica_x8506_xc5000_config = {
.if_khz = 5380,
 };
 
+static struct stv090x_config prof_8000_stv090x_config = {
+.device = STV0903,
+.demod_mode = STV090x_SINGLE,
+.clk_mode   = STV090x_CLK_EXT,
+.xtal   = 2700,
+.address= 0x6A,
+.ts1_mode   = STV090x_TSMODE_PARALLEL_PUNCTURED,
+.repeater_level = STV090x_RPTLEVEL_64,
+.adc1_range = STV090x_ADC_2Vpp,
+.diseqc_envelope_mode   = false,
+
+.tuner_get_frequency= stb6100_get_frequency,
+.tuner_set_frequency= stb6100_set_frequency,
+.tuner_set_bandwidth= stb6100_set_bandwidth,
+.tuner_get_bandwidth= stb6100_get_bandwidth,
+};
+
+static struct stb6100_config prof_8000_stb6100_config = {
+   .tuner_address = 0x60,
+   .refclock = 2700,
+};
+
+static int p8000_set_voltage(struct dvb_frontend *fe, fe_sec_voltage_t voltage)
+{
+   struct cx23885_tsport *port = fe-dvb-priv;
+   struct cx23885_dev *dev = port-dev;
+
+   if (voltage == SEC_VOLTAGE_18)
+   cx_write(MC417_RWD, 0x1e00);
+   else if (voltage == SEC_VOLTAGE_13)
+   cx_write(MC417_RWD, 0x1a00);
+   else
+   cx_write(MC417_RWD, 0x1800);
+   return 0;
+}
+
 static int cx23885_dvb_set_frontend(struct dvb_frontend *fe)
 {
struct dtv_frontend_properties *p = fe-dtv_property_cache;
@@ -1186,6 +1225,23 @@ static int dvb_register(struct cx23885_tsport *port)
tevii_ds3000_config,